diff options
author | Florian Fainelli <f.fainelli@gmail.com> | 2016-07-08 13:36:39 -0400 |
---|---|---|
committer | Brian Norris <computersforpeace@gmail.com> | 2016-07-16 23:40:16 -0400 |
commit | 422485da15e7e9e6f71a5efd1aa2248595cb486d (patch) | |
tree | 222970dce27fdc3b2033daf73a25c266593d10bb /drivers/mtd | |
parent | 79ad07d45743721010e766e65dc004ad249bd429 (diff) |
mtd: nand: brcmnand: Change BUG_ON in brcmnand_send_cmd
Change the BUG_ON() condition in brcmnand_send_cmd() which checks for
the interrupt status "controller ready" bit to a WARN_ON.
There is no good reason to kill the system when this condition occur
because we could have systems which listed the NAND controller as
available (e.g: from Device Tree), but the NAND chip could be
malfunctioning and not responding.
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Acked-by: Brian Norris <computersforpeace@gmail.com>
Reviewed-by: Kamal Dasu <kdasu.kdev@gmail.com>
Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers/mtd')
-rw-r--r-- | drivers/mtd/nand/brcmnand/brcmnand.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mtd/nand/brcmnand/brcmnand.c b/drivers/mtd/nand/brcmnand/brcmnand.c index faca01d6e0f9..8eb2c64df38c 100644 --- a/drivers/mtd/nand/brcmnand/brcmnand.c +++ b/drivers/mtd/nand/brcmnand/brcmnand.c | |||
@@ -1165,7 +1165,7 @@ static void brcmnand_send_cmd(struct brcmnand_host *host, int cmd) | |||
1165 | ctrl->cmd_pending = cmd; | 1165 | ctrl->cmd_pending = cmd; |
1166 | 1166 | ||
1167 | intfc = brcmnand_read_reg(ctrl, BRCMNAND_INTFC_STATUS); | 1167 | intfc = brcmnand_read_reg(ctrl, BRCMNAND_INTFC_STATUS); |
1168 | BUG_ON(!(intfc & INTFC_CTLR_READY)); | 1168 | WARN_ON(!(intfc & INTFC_CTLR_READY)); |
1169 | 1169 | ||
1170 | mb(); /* flush previous writes */ | 1170 | mb(); /* flush previous writes */ |
1171 | brcmnand_write_reg(ctrl, BRCMNAND_CMD_START, | 1171 | brcmnand_write_reg(ctrl, BRCMNAND_CMD_START, |