On 18.10.2019 09:32, Horia Geanta wrote: > On 10/17/2019 12:21 PM, Laurentiu Tudor wrote: >> From: Laurentiu Tudor <laurentiu.tu...@nxp.com> >> >> If SEC FW support is not enabled (ARMV8_SEC_FIRMWARE_SUPPORT=n) this >> compilation error happens: >> arch/arm/include/asm/arch-fsl-layerscape/fsl_icid.h:169:4: error: >> 'CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT' undeclared here (not in a function) >> >> Fix it by adding an intermediate macro to handle the problem. >> >> Signed-off-by: Laurentiu Tudor <laurentiu.tu...@nxp.com> >> --- >> arch/arm/include/asm/arch-fsl-layerscape/fsl_icid.h | 8 +++++++- >> 1 file changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/arch/arm/include/asm/arch-fsl-layerscape/fsl_icid.h >> b/arch/arm/include/asm/arch-fsl-layerscape/fsl_icid.h >> index 37e2fe4e66..398bb4eb86 100644 >> --- a/arch/arm/include/asm/arch-fsl-layerscape/fsl_icid.h >> +++ b/arch/arm/include/asm/arch-fsl-layerscape/fsl_icid.h >> @@ -164,9 +164,15 @@ extern int fman_icid_tbl_sz; >> QDMA_BASE_ADDR + QMAN_CQSIDR_REG + 4, \ >> QDMA_BASE_ADDR, QDMA_IS_LE) >> >> +#ifdef CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT >> +#define SEC_FW_SUPPORT CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT >> +#else >> +#define SEC_FW_SUPPORT 0 >> +#endif >> + >> #define SET_SEC_JR_ICID_ENTRY(jr_num, streamid) \ >> SET_ICID_ENTRY( \ >> - (CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT && \ >> + (SEC_FW_SUPPORT && \ >> (FSL_SEC_JR##jr_num##_OFFSET == \ >> SEC_JR3_OFFSET + CONFIG_SYS_FSL_SEC_OFFSET) \ >> ? NULL \ >> > How about using CONFIG_IS_ENABLED(ARMV8_SEC_FIRMWARE_SUPPORT) instead of > CONFIG_ARMV8_SEC_FIRMWARE_SUPPORT? >
That's pretty cool, didn't new about it. Sure I'll use it. --- Thanks & Best Regards, Laurentiu _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
