[PATCH v4 0/2] Add support for read retry

Cheng Ming Lin posted 2 patches 9 months, 4 weeks ago
There is a newer version of this series
drivers/mtd/nand/spi/core.c     | 35 ++++++++++++++-
drivers/mtd/nand/spi/macronix.c | 79 ++++++++++++++++++++++++++-------
include/linux/mtd/spinand.h     | 14 ++++++
3 files changed, 111 insertions(+), 17 deletions(-)
[PATCH v4 0/2] Add support for read retry
Posted by Cheng Ming Lin 9 months, 4 weeks ago
From: Cheng Ming Lin <chengminglin@mxic.com.tw>

When the host ECC fails to correct the data error of NAND device,
there's a special read for data recovery method which can be setup
by the host for the next read. There are several retry levels that
can be attempted until the lost data is recovered or definitely
assumed lost.

For more detailed information, please refer to the link below:
Link: https://www.macronix.com/Lists/Datasheet/Attachments/9034/MX35LF1G24AD,%203V,%201Gb,%20v1.4.pdf

v4:
* If set_read_retry fails, it should back to read_retry level 0

v3:
* If set_read_retry fails, it should return an error

v2:
* Remove fixups
* Remove the function of init_read_retry

Cheng Ming Lin (2):
  mtd: spi-nand: Add read retry support
  mtd: spi-nand: macronix: Add support for read retry

 drivers/mtd/nand/spi/core.c     | 35 ++++++++++++++-
 drivers/mtd/nand/spi/macronix.c | 79 ++++++++++++++++++++++++++-------
 include/linux/mtd/spinand.h     | 14 ++++++
 3 files changed, 111 insertions(+), 17 deletions(-)

-- 
2.25.1
Re: [PATCH v4 0/2] Add support for read retry
Posted by Miquel Raynal 9 months, 3 weeks ago
Hello,

On 21/02/2025 at 17:32:58 +08, Cheng Ming Lin <linchengming884@gmail.com> wrote:

> From: Cheng Ming Lin <chengminglin@mxic.com.tw>
>
> When the host ECC fails to correct the data error of NAND device,
> there's a special read for data recovery method which can be setup
> by the host for the next read. There are several retry levels that
> can be attempted until the lost data is recovered or definitely
> assumed lost.

I am sorry but you need to rebase on latest -rc1 or maybe even nand/next
because it does not apply.

Thanks,
Miquèl