From nobody Mon Feb 9 16:20:45 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1690810565231686.4606117888437; Mon, 31 Jul 2023 06:36:05 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.573061.897408 (Exim 4.92) (envelope-from ) id 1qQT3p-0000t7-1c; Mon, 31 Jul 2023 13:35:45 +0000 Received: by outflank-mailman (output) from mailman id 573061.897408; Mon, 31 Jul 2023 13:35:45 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qQT3o-0000su-T7; Mon, 31 Jul 2023 13:35:44 +0000 Received: by outflank-mailman (input) for mailman id 573061; Mon, 31 Jul 2023 13:35:43 +0000 Received: from se1-gles-flk1-in.inumbo.com ([94.247.172.50] helo=se1-gles-flk1.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1qQT3n-0008Hl-C2 for xen-devel@lists.xenproject.org; Mon, 31 Jul 2023 13:35:43 +0000 Received: from support.bugseng.com (mail.bugseng.com [162.55.131.47]) by se1-gles-flk1.inumbo.com (Halon) with ESMTPS id 247e0a1a-2fa7-11ee-8613-37d641c3527e; Mon, 31 Jul 2023 15:35:41 +0200 (CEST) Received: from nico.bugseng.com (unknown [147.123.100.131]) by support.bugseng.com (Postfix) with ESMTPSA id 9E1844EE0743; Mon, 31 Jul 2023 15:35:40 +0200 (CEST) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 247e0a1a-2fa7-11ee-8613-37d641c3527e From: Nicola Vetrini To: xen-devel@lists.xenproject.org Cc: sstabellini@kernel.org, michal.orzel@amd.com, xenia.ragiadakou@amd.com, ayan.kumar.halder@amd.com, consulting@bugseng.com, Nicola Vetrini , Julien Grall , Bertrand Marquis , Volodymyr Babchuk , "Daniel P. Smith" Subject: [XEN PATCH 4/4] arm/efi: address MISRA C:2012 Rule 5.3 Date: Mon, 31 Jul 2023 15:35:02 +0200 Message-Id: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-ZM-MESSAGEID: 1690810567532100001 Content-Type: text/plain; charset="utf-8" Rule 5.3 has the following headline: "An identifier declared in an inner scope shall not hide an identifier declared in an outer scope" The parameter 'fdt' in static function within this file is removed, as they served no purpose and shadowed the homonymous variable. For the same reason the local variable in 'lookup_fdt_config_table' is dropped and the function is now consequently static void. No functional changes. Signed-off-by: Nicola Vetrini --- xen/arch/arm/efi/efi-boot.h | 29 +++++++++++++++-------------- 1 file changed, 15 insertions(+), 14 deletions(-) diff --git a/xen/arch/arm/efi/efi-boot.h b/xen/arch/arm/efi/efi-boot.h index 6126a71400..ec328885a3 100644 --- a/xen/arch/arm/efi/efi-boot.h +++ b/xen/arch/arm/efi/efi-boot.h @@ -52,7 +52,7 @@ static struct file __initdata dtbfile; static void __initdata *fdt; static void __initdata *memmap; =20 -static int __init setup_chosen_node(void *fdt, int *addr_cells, int *size_= cells) +static int __init setup_chosen_node(int *addr_cells, int *size_cells) { int node; const struct fdt_property *prop; @@ -114,7 +114,7 @@ static int __init setup_chosen_node(void *fdt, int *add= r_cells, int *size_cells) * Set a single 'reg' property taking into account the * configured addr and size cell sizes. */ -static int __init fdt_set_reg(void *fdt, int node, int addr_cells, +static int __init fdt_set_reg(int node, int addr_cells, int size_cells, uint64_t addr, uint64_t len) { __be32 val[4]; /* At most 2 64 bit values to be stored */ @@ -138,13 +138,16 @@ static int __init fdt_set_reg(void *fdt, int node, in= t addr_cells, return(fdt_setprop(fdt, node, "reg", val, sizeof(*cellp) * (cellp - va= l))); } =20 -static void __init *lookup_fdt_config_table(EFI_SYSTEM_TABLE *sys_table) +/* + * Set the variable 'fdt' if a matching guid is found. + */ +static void __init lookup_fdt_config_table(EFI_SYSTEM_TABLE *sys_table) { static const EFI_GUID __initconst fdt_guid =3D DEVICE_TREE_GUID; EFI_CONFIGURATION_TABLE *tables; - void *fdt =3D NULL; int i; =20 + fdt =3D NULL; tables =3D sys_table->ConfigurationTable; for ( i =3D 0; i < sys_table->NumberOfTableEntries; i++ ) { @@ -154,7 +157,6 @@ static void __init *lookup_fdt_config_table(EFI_SYSTEM_= TABLE *sys_table) break; } } - return fdt; } =20 static bool __init meminfo_add_bank(struct meminfo *mem, @@ -228,7 +230,6 @@ static EFI_STATUS __init efi_process_memory_map_bootinf= o(EFI_MEMORY_DESCRIPTOR * * and memory map information. */ EFI_STATUS __init fdt_add_uefi_nodes(EFI_SYSTEM_TABLE *sys_table, - void *fdt, EFI_MEMORY_DESCRIPTOR *memory_= map, UINTN map_size, UINTN desc_size, @@ -383,7 +384,7 @@ static void __init efi_arch_process_memory_map(EFI_SYST= EM_TABLE *SystemTable, if ( EFI_ERROR(status) ) blexit(L"EFI memory map processing failed"); =20 - status =3D fdt_add_uefi_nodes(SystemTable, fdt, map, map_size, desc_si= ze, + status =3D fdt_add_uefi_nodes(SystemTable, map, map_size, desc_size, desc_ver); if ( EFI_ERROR(status) ) PrintErrMesg(L"Updating FDT failed", status); @@ -542,7 +543,7 @@ static void __init efi_arch_handle_module(const struct = file *file, =20 if ( file =3D=3D &dtbfile ) return; - chosen =3D setup_chosen_node(fdt, &addr_len, &size_len); + chosen =3D setup_chosen_node(&addr_len, &size_len); if ( chosen < 0 ) blexit(L"Unable to setup chosen node"); =20 @@ -557,7 +558,7 @@ static void __init efi_arch_handle_module(const struct = file *file, if ( fdt_setprop(fdt, node, "compatible", ramdisk_compat, sizeof(ramdisk_compat)) < 0 ) blexit(L"Unable to set compatible property."); - if ( fdt_set_reg(fdt, node, addr_len, size_len, ramdisk.addr, + if ( fdt_set_reg(node, addr_len, size_len, ramdisk.addr, ramdisk.size) < 0 ) blexit(L"Unable to set reg property."); } @@ -572,7 +573,7 @@ static void __init efi_arch_handle_module(const struct = file *file, if ( fdt_setprop(fdt, node, "compatible", xsm_compat, sizeof(xsm_compat)) < 0 ) blexit(L"Unable to set compatible property."); - if ( fdt_set_reg(fdt, node, addr_len, size_len, xsm.addr, + if ( fdt_set_reg(node, addr_len, size_len, xsm.addr, xsm.size) < 0 ) blexit(L"Unable to set reg property."); } @@ -589,7 +590,7 @@ static void __init efi_arch_handle_module(const struct = file *file, blexit(L"Unable to set compatible property."); if ( options && fdt_setprop_string(fdt, node, "bootargs", options)= < 0 ) blexit(L"Unable to set bootargs property."); - if ( fdt_set_reg(fdt, node, addr_len, size_len, kernel.addr, + if ( fdt_set_reg(node, addr_len, size_len, kernel.addr, kernel.size) < 0 ) blexit(L"Unable to set reg property."); } @@ -757,7 +758,7 @@ static int __init handle_module_node(const EFI_LOADED_I= MAGE *loaded_image, return ERROR_RENAME_MODULE_NAME; } =20 - if ( fdt_set_reg(fdt, module_node_offset, reg_addr_cells, reg_size_cel= ls, + if ( fdt_set_reg(module_node_offset, reg_addr_cells, reg_size_cells, file->addr, file->size) < 0 ) { PrintMessage(L"Unable to set module reg property."); @@ -862,7 +863,7 @@ static int __init efi_check_dt_boot(const EFI_LOADED_IM= AGE *loaded_image) EFI_FILE_HANDLE dir_handle =3D NULL; =20 /* Check for the chosen node in the current DTB */ - chosen =3D setup_chosen_node(fdt, &addr_len, &size_len); + chosen =3D setup_chosen_node(&addr_len, &size_len); if ( chosen < 0 ) { PrintMessage(L"Unable to setup chosen node"); @@ -951,7 +952,7 @@ static bool __init efi_arch_use_config_file(EFI_SYSTEM_= TABLE *SystemTable) * node to decide whether to skip the UEFI Xen configuration file or n= ot. */ =20 - fdt =3D lookup_fdt_config_table(SystemTable); + lookup_fdt_config_table(SystemTable); dtbfile.ptr =3D fdt; dtbfile.need_to_free =3D false; /* Config table memory can't be freed.= */ =20 --=20 2.34.1