[PATCH 0/2] Remove duplicate function calls and useless message

[PATCH 0/2] Remove duplicate function calls and useless message

[Qiang Ma]

From: Qiang Ma <maqianga@uniontech.com>

This patchset removes duplicate function calls and useless debug message

1. The efi_set_mapping_permissions() does not need to be called in
efi_create_mapping(), because efi_memattr_apply_permissions() will be invoked.

The calling relationship is as follows:

==>efi_create_mapping()
   ==>efi_set_mapping_permissions()
==>efi_memattr_apply_permissions()
   ==>efi_set_mapping_permissions()

2. Based on the first point, efi_create_mapping() always returns 0,
so there is no need to print the failure return message.

Qiang Ma (2):
  ARM/efi: Remove duplicate permission settings
  efi/arm*: Remove the useless failure return message print

 arch/arm/kernel/efi.c              | 5 -----
 drivers/firmware/efi/arm-runtime.c | 7 +------
 2 files changed, 1 insertion(+), 11 deletions(-)

-- 
2.20.1

Re: [PATCH 0/2] Remove duplicate function calls and useless message

[Russell King (Oracle)]

On Thu, Oct 23, 2025 at 04:20:51PM +0800, Qiang Ma wrote:
> From: Qiang Ma <maqianga@uniontech.com>
> 
> This patchset removes duplicate function calls and useless debug message

This isn't a patch set. This is a cover message with no patches threaded
to it. This makes it difficult to find the patches.

Please ensure that patches are threaded to the cover message.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!