From: Heinrich Schuchardt <xypron.glpk@xxxxxx> Shadowing variables is generally frowned upon. Let's simply reuse the existing loop counter i instead of shadowing it. Signed-off-by: Heinrich Schuchardt <xypron.glpk@xxxxxx> Link: https://lore.kernel.org/r/20200223221324.156086-1-xypron.glpk@xxxxxx Signed-off-by: Ard Biesheuvel <ardb@xxxxxxxxxx> --- drivers/firmware/efi/efi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/firmware/efi/efi.c b/drivers/firmware/efi/efi.c index 5f77cb8756ef..91f546dc13d4 100644 --- a/drivers/firmware/efi/efi.c +++ b/drivers/firmware/efi/efi.c @@ -611,7 +611,6 @@ int __init efi_config_parse_tables(const efi_config_table_t *config_tables, while (prsv) { struct linux_efi_memreserve *rsv; u8 *p; - int i; /* * Just map a full page: that is what we will get -- 2.17.1