[U-Boot] [PATCH] SECURE_BOOT: Correct reading of ITS bit

The ITS bit was being read incorrectly beacause of operator precedence. The same ahs been corrected.
Signed-off-by: Lawish Deshmukh lawish.deshmukh@freescale.com Signed-off-by: Aneesh Bansal aneesh.bansal@freescale.com --- board/freescale/common/fsl_validate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/board/freescale/common/fsl_validate.c b/board/freescale/common/fsl_validate.c index 5283648..b528fb7 100644 --- a/board/freescale/common/fsl_validate.c +++ b/board/freescale/common/fsl_validate.c @@ -246,7 +246,7 @@ static void fsl_secboot_image_verification_failure(void) struct ccsr_sfp_regs *sfp_regs = (void *)(CONFIG_SYS_SFP_ADDR); u32 sts = sec_mon_in32(&sec_mon_regs->hp_stat);
- u32 its = sfp_in32(&sfp_regs->ospr) & ITS_MASK >> ITS_BIT; + u32 its = (sfp_in32(&sfp_regs->ospr) & ITS_MASK) >> ITS_BIT;
/* * Read the SEC_MON status register

On 10/12/2015 09:35 AM, Aneesh Bansal wrote:
The ITS bit was being read incorrectly beacause of operator precedence. The same ahs been corrected.
Signed-off-by: Lawish Deshmukh lawish.deshmukh@freescale.com Signed-off-by: Aneesh Bansal aneesh.bansal@freescale.com
Applied to u-boot-fsl-qoriq. Awaiting upstream. Thanks.
York
participants (2)
-
Aneesh Bansal
-
York Sun