diff options
| author | Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> | 2019-10-31 23:37:27 +0300 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2020-01-23 10:22:57 +0300 |
| commit | 1883f6047d8cf7907e8fd8558e5dcb92f6d5db5d (patch) | |
| tree | b6c03555327e58cdcbdf381b754e1a998da9bb4b /drivers | |
| parent | acc5e1a86abaf6483db48ac6e8a225ca018df11c (diff) | |
| download | linux-1883f6047d8cf7907e8fd8558e5dcb92f6d5db5d.tar.xz | |
mtd: cfi_cmdset_0002: only check errors when ready in cfi_check_err_status()
commit 72914a8cff7e1d910c58e125e15a0da409e3135f upstream.
Cypress S26K{L|S}P{128|256|512}S datasheet says that the error bits in
the status register are only valid when the "device ready" bit 7 is set.
Add the check for the device ready bit in cfi_check_err_status() as that
function isn't always called with this bit set.
Fixes: 4844ef80305d ("mtd: cfi_cmdset_0002: Add support for polling status register")
Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Signed-off-by: Vignesh Raghavendra <vigneshr@ti.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/mtd/chips/cfi_cmdset_0002.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/mtd/chips/cfi_cmdset_0002.c b/drivers/mtd/chips/cfi_cmdset_0002.c index cf8c8be40a9c..0ac374e70573 100644 --- a/drivers/mtd/chips/cfi_cmdset_0002.c +++ b/drivers/mtd/chips/cfi_cmdset_0002.c @@ -136,6 +136,10 @@ static void cfi_check_err_status(struct map_info *map, struct flchip *chip, cfi->device_type, NULL); status = map_read(map, adr); + /* The error bits are invalid while the chip's busy */ + if (!map_word_bitsset(map, status, CMD(CFI_SR_DRB))) + return; + if (map_word_bitsset(map, status, CMD(0x3a))) { unsigned long chipstatus = MERGESTATUS(status); |