> From: Utkarsh Gupta <[email protected]> > 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 <[email protected]> > Reviewed-by: Ye Li <[email protected]> > Signed-off-by: Utkarsh Gupta <[email protected]> > Signed-off-by: Peng Fan <[email protected]> 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: [email protected] =====================================================================
