mtd: nand: fsl_ifc: Fix nand waitfunc return value
authorJagdish Gediya <jagdish.gediya@nxp.com>
Tue, 1 May 2018 19:50:57 +0000 (01:20 +0530)
committerYork Sun <york.sun@nxp.com>
Wed, 9 May 2018 14:17:51 +0000 (09:17 -0500)
As per the IFC hardware manual, Most significant byte in nand_fsr
register is the outcome of NAND READ STATUS command.

So status value need to be shifted as per the nand framework
requirement.

Signed-off-by: Jagdish Gediya <jagdish.gediya@nxp.com>
Reviewed-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
Reviewed-by: York Sun <york.sun@nxp.com>
drivers/mtd/nand/fsl_ifc_nand.c

index 3cb487510ac6ee74e36e5cf209095f1c04986326..a47226bd2158106bb4efaa2eac556faad82c60af 100644 (file)
@@ -700,6 +700,7 @@ static int fsl_ifc_wait(struct mtd_info *mtd, struct nand_chip *chip)
        struct fsl_ifc_ctrl *ctrl = priv->ctrl;
        struct fsl_ifc_runtime *ifc = ctrl->regs.rregs;
        u32 nand_fsr;
+       int status;
 
        if (ctrl->status != IFC_NAND_EVTER_STAT_OPC)
                return NAND_STATUS_FAIL;
@@ -720,10 +721,10 @@ static int fsl_ifc_wait(struct mtd_info *mtd, struct nand_chip *chip)
                return NAND_STATUS_FAIL;
 
        nand_fsr = ifc_in32(&ifc->ifc_nand.nand_fsr);
+       status = nand_fsr >> 24;
 
        /* Chip sometimes reporting write protect even when it's not */
-       nand_fsr = nand_fsr | NAND_STATUS_WP;
-       return nand_fsr;
+       return status | NAND_STATUS_WP;
 }
 
 static int fsl_ifc_read_page(struct mtd_info *mtd, struct nand_chip *chip,