SECURE_BOOT: Correct reading of ITS bit
authorAneesh Bansal <aneesh.bansal@freescale.com>
Mon, 12 Oct 2015 16:35:50 +0000 (22:05 +0530)
committerYork Sun <yorksun@freescale.com>
Fri, 30 Oct 2015 16:19:47 +0000 (09:19 -0700)
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>
Reviewed-by: York Sun <yorksun@freescale.com>
board/freescale/common/fsl_validate.c

index 465676f..73b6718 100644 (file)
@@ -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