> From: Utkarsh Gupta <utkarsh.gu...@nxp.com> > The sec_in32 preprocessor is defined as follows in include/fsl_sec.h file: > When address "a" is calculated using math for ex: addition of base address and > an offset, then casting is applied only to the first address which in this > example is base address. > caam_ccbvid_reg = sec_in32(CONFIG_SYS_FSL_SEC_ADDR + CAAM_CCBVID_OFFSET) > resolves to: > caam_ccbvid_reg = in_le32((ulong *)(ulong)CONFIG_SYS_FSL_SEC_ADDR + > CAAM_CCBVID_OFFSET) > instead it should resolve to: > caam_ccbvid_reg = in_le32((ulong *)(ulong)(CONFIG_SYS_FSL_SEC_ADDR + > CAAM_CCBVID_OFFSET)) > Thus add parenthesis around the address "a" so that however the address is > calculated, the casting is applied to the final calculated address. > Reviewed-by: Horia Geanta <horia.gea...@nxp.com> > Reviewed-by: Ye Li <ye...@nxp.com> > Signed-off-by: Utkarsh Gupta <utkarsh.gu...@nxp.com> > Signed-off-by: Peng Fan <peng....@nxp.com> Applied to u-boot-imx, master, thanks !
Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Erika Unter HRB 165235 Munich, Office: Kirchenstr.5, 82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de =====================================================================