[PATCH] armv8: sec_firmware: drop unneeded ARMV8_SEC_FIRMWARE_SUPPORT check

Michael Walle michael at walle.cc
Wed Apr 6 15:26:58 CEST 2022


Am 2022-04-06 15:09, schrieb Tom Rini:
> On Fri, Nov 26, 2021 at 05:11:36PM +0100, Michael Walle wrote:
> 
>> This module will only be compiled if this symbol is set. Remove the
>> useless check. While at it, invert the return code, to return 0 if the
>> function succeeds. The only user of this function in
>> arch/arm/cpu/armv8/fsl-layerscape/fdt.c doesn't check the return code
>> anyway, so its safe change it.
>> 
>> Signed-off-by: Michael Walle <michael at walle.cc>
>> ---
>>  arch/arm/cpu/armv8/sec_firmware.c | 20 +++++++++-----------
>>  1 file changed, 9 insertions(+), 11 deletions(-)
> 
> Currently this causes a large number of platforms to fail to build such
> as ls1012aqds_tfa_SECURE_BOOT, please fix and resubmit, thanks.

Ahh, right, I guess that is because that ifdeffery is needed
to distiguish the SPL build. In this case, this can be dropped.

Sorry for the noise,
-michael


More information about the U-Boot mailing list