375 lines
9.3 KiB
Diff
375 lines
9.3 KiB
Diff
--- a/drivers/mtd/spi-nor/fsl-quadspi.c
|
|
+++ b/drivers/mtd/spi-nor/fsl-quadspi.c
|
|
@@ -719,16 +719,10 @@ static int fsl_qspi_read(struct spi_nor
|
|
{
|
|
struct fsl_qspi *q = nor->priv;
|
|
u8 cmd = nor->read_opcode;
|
|
- int ret;
|
|
|
|
dev_dbg(q->dev, "cmd [%x],read from (0x%p, 0x%.8x, 0x%.8x),len:%d\n",
|
|
cmd, q->ahb_base, q->chip_base_addr, (unsigned int)from, len);
|
|
|
|
- /* Wait until the previous command is finished. */
|
|
- ret = nor->wait_till_ready(nor);
|
|
- if (ret)
|
|
- return ret;
|
|
-
|
|
/* Read out the data directly from the AHB buffer.*/
|
|
memcpy(buf, q->ahb_base + q->chip_base_addr + from, len);
|
|
|
|
@@ -744,16 +738,6 @@ static int fsl_qspi_erase(struct spi_nor
|
|
dev_dbg(nor->dev, "%dKiB at 0x%08x:0x%08x\n",
|
|
nor->mtd->erasesize / 1024, q->chip_base_addr, (u32)offs);
|
|
|
|
- /* Wait until finished previous write command. */
|
|
- ret = nor->wait_till_ready(nor);
|
|
- if (ret)
|
|
- return ret;
|
|
-
|
|
- /* Send write enable, then erase commands. */
|
|
- ret = nor->write_reg(nor, SPINOR_OP_WREN, NULL, 0, 0);
|
|
- if (ret)
|
|
- return ret;
|
|
-
|
|
ret = fsl_qspi_runcmd(q, nor->erase_opcode, offs, 0);
|
|
if (ret)
|
|
return ret;
|
|
--- a/drivers/mtd/spi-nor/spi-nor.c
|
|
+++ b/drivers/mtd/spi-nor/spi-nor.c
|
|
@@ -165,81 +165,69 @@ static inline int set_4byte(struct spi_n
|
|
return nor->write_reg(nor, SPINOR_OP_BRWR, nor->cmd_buf, 1, 0);
|
|
}
|
|
}
|
|
-
|
|
-static int spi_nor_wait_till_ready(struct spi_nor *nor)
|
|
+static inline int spi_nor_sr_ready(struct spi_nor *nor)
|
|
{
|
|
- unsigned long deadline;
|
|
- int sr;
|
|
-
|
|
- deadline = jiffies + MAX_READY_WAIT_JIFFIES;
|
|
-
|
|
- do {
|
|
- cond_resched();
|
|
+ int sr = read_sr(nor);
|
|
+ if (sr < 0)
|
|
+ return sr;
|
|
+ else
|
|
+ return !(sr & SR_WIP);
|
|
+}
|
|
|
|
- sr = read_sr(nor);
|
|
- if (sr < 0)
|
|
- break;
|
|
- else if (!(sr & SR_WIP))
|
|
- return 0;
|
|
- } while (!time_after_eq(jiffies, deadline));
|
|
+static inline int spi_nor_fsr_ready(struct spi_nor *nor)
|
|
+{
|
|
+ int fsr = read_fsr(nor);
|
|
+ if (fsr < 0)
|
|
+ return fsr;
|
|
+ else
|
|
+ return fsr & FSR_READY;
|
|
+}
|
|
|
|
- return -ETIMEDOUT;
|
|
+static int spi_nor_ready(struct spi_nor *nor)
|
|
+{
|
|
+ int sr, fsr;
|
|
+ sr = spi_nor_sr_ready(nor);
|
|
+ if (sr < 0)
|
|
+ return sr;
|
|
+ fsr = nor->flags & SNOR_F_USE_FSR ? spi_nor_fsr_ready(nor) : 1;
|
|
+ if (fsr < 0)
|
|
+ return sr;
|
|
+ return sr && fsr;
|
|
}
|
|
|
|
-static int spi_nor_wait_till_fsr_ready(struct spi_nor *nor)
|
|
+/*
|
|
+ * Service routine to read status register until ready, or timeout occurs.
|
|
+ * Returns non-zero if error.
|
|
+ */
|
|
+static int spi_nor_wait_till_ready(struct spi_nor *nor)
|
|
{
|
|
unsigned long deadline;
|
|
- int sr;
|
|
- int fsr;
|
|
+ int ret;
|
|
|
|
deadline = jiffies + MAX_READY_WAIT_JIFFIES;
|
|
|
|
do {
|
|
cond_resched();
|
|
|
|
- sr = read_sr(nor);
|
|
- if (sr < 0) {
|
|
- break;
|
|
- } else if (!(sr & SR_WIP)) {
|
|
- fsr = read_fsr(nor);
|
|
- if (fsr < 0)
|
|
- break;
|
|
- if (fsr & FSR_READY)
|
|
- return 0;
|
|
- }
|
|
+ ret = spi_nor_ready(nor);
|
|
+ if (ret < 0)
|
|
+ return ret;
|
|
+ if (ret)
|
|
+ return 0;
|
|
} while (!time_after_eq(jiffies, deadline));
|
|
|
|
return -ETIMEDOUT;
|
|
}
|
|
|
|
/*
|
|
- * Service routine to read status register until ready, or timeout occurs.
|
|
- * Returns non-zero if error.
|
|
- */
|
|
-static int wait_till_ready(struct spi_nor *nor)
|
|
-{
|
|
- return nor->wait_till_ready(nor);
|
|
-}
|
|
-
|
|
-/*
|
|
* Erase the whole flash memory
|
|
*
|
|
* Returns 0 if successful, non-zero otherwise.
|
|
*/
|
|
static int erase_chip(struct spi_nor *nor)
|
|
{
|
|
- int ret;
|
|
-
|
|
dev_dbg(nor->dev, " %lldKiB\n", (long long)(nor->mtd->size >> 10));
|
|
|
|
- /* Wait until finished previous write command. */
|
|
- ret = wait_till_ready(nor);
|
|
- if (ret)
|
|
- return ret;
|
|
-
|
|
- /* Send write enable, then erase commands. */
|
|
- write_enable(nor);
|
|
-
|
|
return nor->write_reg(nor, SPINOR_OP_CHIP_ERASE, NULL, 0, 0);
|
|
}
|
|
|
|
@@ -292,6 +280,8 @@ static int spi_nor_erase(struct mtd_info
|
|
if (ret)
|
|
return ret;
|
|
|
|
+ write_enable(nor);
|
|
+
|
|
/* whole-chip erase? */
|
|
if (len == mtd->size) {
|
|
if (erase_chip(nor)) {
|
|
@@ -299,6 +289,10 @@ static int spi_nor_erase(struct mtd_info
|
|
goto erase_err;
|
|
}
|
|
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
+ if (ret)
|
|
+ goto erase_err;
|
|
+
|
|
/* REVISIT in some cases we could speed up erasing large regions
|
|
* by using SPINOR_OP_SE instead of SPINOR_OP_BE_4K. We may have set up
|
|
* to use "small sector erase", but that's not always optimal.
|
|
@@ -314,9 +308,15 @@ static int spi_nor_erase(struct mtd_info
|
|
|
|
addr += mtd->erasesize;
|
|
len -= mtd->erasesize;
|
|
+
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
+ if (ret)
|
|
+ goto erase_err;
|
|
}
|
|
}
|
|
|
|
+ write_disable(nor);
|
|
+
|
|
spi_nor_unlock_and_unprep(nor, SPI_NOR_OPS_ERASE);
|
|
|
|
instr->state = MTD_ERASE_DONE;
|
|
@@ -341,11 +341,6 @@ static int spi_nor_lock(struct mtd_info
|
|
if (ret)
|
|
return ret;
|
|
|
|
- /* Wait until finished previous command */
|
|
- ret = wait_till_ready(nor);
|
|
- if (ret)
|
|
- goto err;
|
|
-
|
|
status_old = read_sr(nor);
|
|
|
|
if (offset < mtd->size - (mtd->size / 2))
|
|
@@ -388,11 +383,6 @@ static int spi_nor_unlock(struct mtd_inf
|
|
if (ret)
|
|
return ret;
|
|
|
|
- /* Wait until finished previous command */
|
|
- ret = wait_till_ready(nor);
|
|
- if (ret)
|
|
- goto err;
|
|
-
|
|
status_old = read_sr(nor);
|
|
|
|
if (offset+len > mtd->size - (mtd->size / 64))
|
|
@@ -704,11 +694,6 @@ static int sst_write(struct mtd_info *mt
|
|
if (ret)
|
|
return ret;
|
|
|
|
- /* Wait until finished previous write command. */
|
|
- ret = wait_till_ready(nor);
|
|
- if (ret)
|
|
- goto time_out;
|
|
-
|
|
write_enable(nor);
|
|
|
|
nor->sst_write_second = false;
|
|
@@ -720,7 +705,7 @@ static int sst_write(struct mtd_info *mt
|
|
|
|
/* write one byte. */
|
|
nor->write(nor, to, 1, retlen, buf);
|
|
- ret = wait_till_ready(nor);
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
if (ret)
|
|
goto time_out;
|
|
}
|
|
@@ -732,7 +717,7 @@ static int sst_write(struct mtd_info *mt
|
|
|
|
/* write two bytes. */
|
|
nor->write(nor, to, 2, retlen, buf + actual);
|
|
- ret = wait_till_ready(nor);
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
if (ret)
|
|
goto time_out;
|
|
to += 2;
|
|
@@ -741,7 +726,7 @@ static int sst_write(struct mtd_info *mt
|
|
nor->sst_write_second = false;
|
|
|
|
write_disable(nor);
|
|
- ret = wait_till_ready(nor);
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
if (ret)
|
|
goto time_out;
|
|
|
|
@@ -752,7 +737,7 @@ static int sst_write(struct mtd_info *mt
|
|
nor->program_opcode = SPINOR_OP_BP;
|
|
nor->write(nor, to, 1, retlen, buf + actual);
|
|
|
|
- ret = wait_till_ready(nor);
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
if (ret)
|
|
goto time_out;
|
|
write_disable(nor);
|
|
@@ -780,11 +765,6 @@ static int spi_nor_write(struct mtd_info
|
|
if (ret)
|
|
return ret;
|
|
|
|
- /* Wait until finished previous write command. */
|
|
- ret = wait_till_ready(nor);
|
|
- if (ret)
|
|
- goto write_err;
|
|
-
|
|
write_enable(nor);
|
|
|
|
page_offset = to & (nor->page_size - 1);
|
|
@@ -803,16 +783,20 @@ static int spi_nor_write(struct mtd_info
|
|
if (page_size > nor->page_size)
|
|
page_size = nor->page_size;
|
|
|
|
- wait_till_ready(nor);
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
+ if (ret)
|
|
+ goto write_err;
|
|
+
|
|
write_enable(nor);
|
|
|
|
nor->write(nor, to + i, page_size, retlen, buf + i);
|
|
}
|
|
}
|
|
|
|
+ ret = spi_nor_wait_till_ready(nor);
|
|
write_err:
|
|
spi_nor_unlock_and_unprep(nor, SPI_NOR_OPS_WRITE);
|
|
- return 0;
|
|
+ return ret;
|
|
}
|
|
|
|
static int macronix_quad_enable(struct spi_nor *nor)
|
|
@@ -825,7 +809,7 @@ static int macronix_quad_enable(struct s
|
|
nor->cmd_buf[0] = val | SR_QUAD_EN_MX;
|
|
nor->write_reg(nor, SPINOR_OP_WRSR, nor->cmd_buf, 1, 0);
|
|
|
|
- if (wait_till_ready(nor))
|
|
+ if (spi_nor_wait_till_ready(nor))
|
|
return 1;
|
|
|
|
ret = read_sr(nor);
|
|
@@ -907,8 +891,6 @@ static int spi_nor_check(struct spi_nor
|
|
|
|
if (!nor->read_id)
|
|
nor->read_id = spi_nor_read_id;
|
|
- if (!nor->wait_till_ready)
|
|
- nor->wait_till_ready = spi_nor_wait_till_ready;
|
|
|
|
return 0;
|
|
}
|
|
@@ -983,9 +965,8 @@ int spi_nor_scan(struct spi_nor *nor, co
|
|
else
|
|
mtd->_write = spi_nor_write;
|
|
|
|
- if ((info->flags & USE_FSR) &&
|
|
- nor->wait_till_ready == spi_nor_wait_till_ready)
|
|
- nor->wait_till_ready = spi_nor_wait_till_fsr_ready;
|
|
+ if (info->flags & USE_FSR)
|
|
+ nor->flags |= SNOR_F_USE_FSR;
|
|
|
|
#ifdef CONFIG_MTD_SPI_NOR_USE_4K_SECTORS
|
|
/* prefer "small sector" erase if possible */
|
|
--- a/include/linux/mtd/spi-nor.h
|
|
+++ b/include/linux/mtd/spi-nor.h
|
|
@@ -116,6 +116,10 @@ enum spi_nor_ops {
|
|
SPI_NOR_OPS_UNLOCK,
|
|
};
|
|
|
|
+enum spi_nor_option_flags {
|
|
+ SNOR_F_USE_FSR = BIT(0),
|
|
+};
|
|
+
|
|
/**
|
|
* struct spi_nor - Structure for defining a the SPI NOR layer
|
|
* @mtd: point to a mtd_info structure
|
|
@@ -129,6 +133,7 @@ enum spi_nor_ops {
|
|
* @program_opcode: the program opcode
|
|
* @flash_read: the mode of the read
|
|
* @sst_write_second: used by the SST write operation
|
|
+ * @flags: flag options for the current SPI-NOR (SNOR_F_*)
|
|
* @cfg: used by the read_xfer/write_xfer
|
|
* @cmd_buf: used by the write_reg
|
|
* @prepare: [OPTIONAL] do some preparations for the
|
|
@@ -141,7 +146,6 @@ enum spi_nor_ops {
|
|
* @write_reg: [DRIVER-SPECIFIC] write data to the register
|
|
* @read_id: [REPLACEABLE] read out the ID data, and find
|
|
* the proper spi_device_id
|
|
- * @wait_till_ready: [REPLACEABLE] wait till the NOR becomes ready
|
|
* @read: [DRIVER-SPECIFIC] read data from the SPI NOR
|
|
* @write: [DRIVER-SPECIFIC] write data to the SPI NOR
|
|
* @erase: [DRIVER-SPECIFIC] erase a sector of the SPI NOR
|
|
@@ -160,6 +164,7 @@ struct spi_nor {
|
|
u8 program_opcode;
|
|
enum read_mode flash_read;
|
|
bool sst_write_second;
|
|
+ u32 flags;
|
|
struct spi_nor_xfer_cfg cfg;
|
|
u8 cmd_buf[SPI_NOR_MAX_CMD_SIZE];
|
|
|
|
@@ -173,7 +178,6 @@ struct spi_nor {
|
|
int (*write_reg)(struct spi_nor *nor, u8 opcode, u8 *buf, int len,
|
|
int write_enable);
|
|
const struct spi_device_id *(*read_id)(struct spi_nor *nor);
|
|
- int (*wait_till_ready)(struct spi_nor *nor);
|
|
|
|
int (*read)(struct spi_nor *nor, loff_t from,
|
|
size_t len, size_t *retlen, u_char *read_buf);
|