Skip to contentSkip to conversation

SECURE_BOOT: Correct reading of ITS bit

PATCH New Aneesh Bansal 2 participants 2 messages Oct 12, 2015 – Oct 30, 2015

lore
Share & tools
lore

Aneesh Bansal

author
#0 lore raw

PATCH 1/1 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 <[email protected]>
Signed-off-by Aneesh Bansal <[email protected]>
---
 board/freescale/common/fsl_validate.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
board/freescale/common/fsl_validate.c +1 -1
index 5283648..b528fb7 100644
@@ -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
Signature · 1 line
1.8.1.4