Hi Love,
On 7/11/26 8:36 AM, Kumar, Love wrote:
Hi Michal,
On 7/9/2026 11:23 AM, Michal Simek wrote:
Hi Love,
On 7/2/26 15:08, Bastien Curutchet wrote:
This driver is deeply broken in many ways. Miquèl has cleaned it up by:
- flagging the most hideous parts (where Micron on-die ECC support is
hardcoded)
- enabling DT support, eg. for reading the ECC configuration
- fixing software ECC support
- aligning the software ECC OOB layout to the rest of the world and
allow a working interoperability in Linux.
I then added a patch to fix writes when soft ECC is on.
Signed-off-by: Miquel Raynal (DAVE.eu) <[email protected]>
Signed-off-by: Bastien Curutchet (DAVE.eu) <[email protected]>
---
Changes in v3:
- Add patch 6 to fix write issues in soft ECC.
- Link to v2:
https://lore.kernel.org/r/[email protected]
Changes in v2:
- Fix the sponsor: s/DAVE/DAVE.eu/, no other change.
- Link to v1:
https://lore.kernel.org/r/[email protected]
---
Bastien Curutchet (DAVE.eu) (1):
mtd: rawnand: nand_base: Don't overwrite read/write_page_raw() for soft
ECC
Miquel Raynal (DAVE.eu) (5):
mtd: rawnand: zynq: Align the layout structures with the driver
namespace
mtd: rawnand: zynq: Add comments about very bad practices
mtd: rawnand: zynq: Make sure an of node is attached
mtd: rawnand: zynq: Do not fail the probe for no reason
mtd: rawnand: zynq: Fix Software ECC engine support
drivers/mtd/nand/raw/nand_base.c | 6 +-
drivers/mtd/nand/raw/zynq_nand.c | 115
++++++++++-----------------------------
2 files changed, 32 insertions(+), 89 deletions(-)
---
base-commit: 4433253ecf2041f9362a763bb6cb79960921ac7e
change-id: 20260522-dave-upstream-nand-fixes-60a5336517a2
Best regards,
Please test this series.
We are again seeing the NAND write issue with zynq-zc770-xm011 board.
Below is the log snippet:
Zynq> nand write 0x80000 0x0 0x126f80
nand write 0x80000 0x0 0x126f80
NAND write: device 0 offset 0x0, size 0x126f80
NAND write to offset 0 failed -5
The exact same command works in my setup:
```
Zynq> nand write 0x80000 0x0 0x126f80
NAND write: device 0 offset 0x0, size 0x126f80
1208192 bytes written: OK
```
Can you please share your device-tree / defconfig ? I'd like to dig into
this a bit.
I'd also be interested in seeing the output of 'mtd list' and 'nand
info' on your side.
Best regards,
Bastien
