[PATCH v2 0/5] mtd: rawnand: sunxi: Fix user data length for H6

[Michael Nazzareno Trimarchi]

Hi

On Mon, Apr 27, 2026 at 3:47 PM Andre Przywara <andre.przywara at arm.com>
wrote:

> Hi Michael,
>
> On 4/27/26 14:31, Michael Nazzareno Trimarchi wrote:
> > Hi Andre
> >
> > On Mon, Apr 27, 2026 at 2:28 PM Andre Przywara <andre.przywara at arm.com
> > <mailto:andre.przywara at arm.com>> wrote:
> >
> >     Dario, Michael,
> >
> >     are you going to take those patches eventually, or are they supposed
> to
> >     go through the sunxi tree? Somehow patchwork put this on my plate,
> but
> >     MAINTAINERS says it's yours.
> >
> >     I will have a final look at those patches, but I think they are fine
> >     since their counterparts passed the kernel reviews ;-)
> >
> >
> > Company projects are stressing me out. I will check today with Dario
>
> Sorry to hear that! I can easily take care of this series, if you are
> fine with that. It would touch only those sunxi specific files anyway.
>
>
Yes it's ok, good that you can pick them

Michael


> Cheers,
> Andre
>
> > Michael
> >
> >     Cheers,
> >     Andre
> >
> >     On 3/27/26 15:05, Richard Genoud wrote:
> >      > This series introduces the correct usage of user data length for
> >      > H6/H616.
> >      > If the older SoCs must use a fixed value of 4 user data bytes for
> >     each
> >      > ECC step, the H6 can have a different value for each step.
> >      >
> >      > In the first round, the behavior was kept as the one used by A10.
> >      > But we have multiple choices:
> >      > - do like A10 like before (and not use all OOB)
> >      > - do like the vendor kernel (different from A10, but not using
> >     all OOB)
> >      > - use the whole OOB for user data bytes
> >      >
> >      > In this second round, the third approach is implemented.
> >      >
> >      > All patches but the 4th are code cleaning.
> >      > The 4th patch enables using a variable user data length for
> H6/H616
> >      > along with maximizing it.
> >      >
> >      > A similar series has been accepted for Linux:
> >      > https://lore.kernel.org/lkml/20260317142437.580204-1-
> >     richard.genoud at bootlin.com/ <https://lore.kernel.org/
> >     lkml/20260317142437.580204-1-richard.genoud at bootlin.com/>
> >      >
> >      > Changes from v1:
> >      > - gather reviewed-by
> >      > - clean sunxi_nand_chip_init()
> >      > - use all possible remaining OOB space for user data bytes
> >      >
> >      > Richard Genoud (5):
> >      >    mtd: rawnand: sunxi: Replace hard coded value by a define
> >      >    mtd: rawnand: sunxi: make the code mode self-explanatory
> >      >    mtd: rawnand: sunxi: clean sunxi_nand_chip_init()
> >      >    mtd: rawnand: sunxi: introduce variable user data length
> >      >    mtd: nand: raw: sunxi_spl: remove user data length reset
> >      >
> >      >   drivers/mtd/nand/raw/sunxi_nand.c     | 292 ++++++++++++++++++
> >     +-------
> >      >   drivers/mtd/nand/raw/sunxi_nand.h     |   3 +
> >      >   drivers/mtd/nand/raw/sunxi_nand_spl.c | 129 +++++++++---
> >      >   3 files changed, 324 insertions(+), 100 deletions(-)
> >      >
> >      >
> >      > base-commit: ba7bf918dafcd093ad733b07ba490baeb20cf5da
> >
> >
> >
> > --
> > Michael Nazzareno Trimarchi
> > Co-Founder & Chief Executive Officer
> > M. +39 347 913 2170
> > michael at amarulasolutions.com <mailto:michael at amarulasolutions.com>
> > __________________________________
> >
> > Amarula Solutions BV
> > Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
> > T. +31 (0)85 111 9172
> > info at amarulasolutions.com <mailto:info at amarulasolutions.com>
> > www.amarulasolutions.com <http://www.amarulasolutions.com>
>
>

-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael at amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info at amarulasolutions.com
www.amarulasolutions.com