[RFC 8/8] debug: mtd: spi-nor: add BUG_ON() prints to check for !ready

Brian Norris
2014-08-07T01:21:06Z

I hacked around the wait-till-ready sequencing just now, so let's add
some debug checks for now.

These probably shouldn't be included in mainline (or at least they
should be toned down to something less drastic; WARN_ON() perhaps?). I'm
just using these for test purposes.

Signed-off-by: Brian Norris <computersforpeace@gmail.com>
---
This patch is really just for testing. I don't think we really need it,
although I suppose some form of it couldn't hurt...

drivers/mtd/spi-nor/spi-nor.c | 14 ++++++++++++++
1 file changed, 14 insertions(+)

diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
index d08d9f8bb9bd..c2e53e02fe27 100644
--- a/drivers/mtd/spi-nor/spi-nor.c
+++ b/drivers/mtd/spi-nor/spi-nor.c
@@ -226,6 +226,8 @@ static int erase_chip(struct spi_nor *nor)
{
dev_dbg(nor->dev, " %lldKiB\n", (long long)(nor->mtd->size >> 10));

+ BUG_ON(spi_nor_ready(nor) <= 0);
+
return nor->write_reg(nor, SPINOR_OP_CHIP_ERASE, NULL, 0, 0);
}

@@ -278,6 +280,8 @@ static int spi_nor_erase(struct mtd_info *mtd, struct erase_info *instr)
if (ret)
return ret;

+ BUG_ON(spi_nor_ready(nor) <= 0);
+
write_enable(nor);

/* whole-chip erase? */
@@ -339,6 +343,8 @@ static int spi_nor_lock(struct mtd_info *mtd, loff_t ofs, uint64_t len)
if (ret)
return ret;

+ BUG_ON(spi_nor_ready(nor) <= 0);
+
status_old = read_sr(nor);

if (offset < mtd->size - (mtd->size / 2))
@@ -381,6 +387,8 @@ static int spi_nor_unlock(struct mtd_info *mtd, loff_t ofs, uint64_t len)
if (ret)
return ret;

+ BUG_ON(spi_nor_ready(nor) <= 0);
+
status_old = read_sr(nor);

if (offset+len > mtd->size - (mtd->size / 64))
@@ -678,6 +686,8 @@ static int spi_nor_read(struct mtd_info *mtd, loff_t from, size_t len,
if (ret)
return ret;

+ BUG_ON(spi_nor_ready(nor) <= 0);
+
ret = nor->read(nor, from, len, retlen, buf);

spi_nor_unlock_and_unprep(nor, SPI_NOR_OPS_READ);
@@ -697,6 +707,8 @@ static int sst_write(struct mtd_info *mtd, loff_t to, size_t len,
if (ret)
return ret;

+ BUG_ON(spi_nor_ready(nor) <= 0);
+
write_enable(nor);

nor->sst_write_second = false;
@@ -768,6 +780,8 @@ static int spi_nor_write(struct mtd_info *mtd, loff_t to, size_t len,
if (ret)
return ret;

+ BUG_ON(spi_nor_ready(nor) <= 0);
+
write_enable(nor);

page_offset = to & (nor->page_size - 1);
--
1.9.1