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@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