Hi Boris, Boris Brezillon <boris.brezillon at bootlin.com> wrote on Wed, 18 Jul 2018 10:42:15 +0200: > Basing the "mandatory on-die" detection on ID byte 2 does not work, > because Micron has plenty of NANDs using the same device ID code, and > not all of them have forcibly enabled on-die ECC. > > Since the "Array Operation" feature does not provide the "ECC > enabled/disabled" bit when the ECC can't be disabled, let's try to use > the "ECC enabled/disabled" bit in the READ_ID bytes. > > It seems that this bit is dynamically updated on NANDs where on-die ECC > can freely be enabled/disabled, so let's hope it stays at one when we > have a NAND with on-die ECC forcibly enabled. > > Fixes: 51f3b3970a8c ("mtd: rawnand: micron: detect forced on-die ECC") > Signed-off-by: Boris Brezillon <boris.brezillon at bootlin.com> > Tested-by: Chris Packham <chris.packham at alliedtelesis.co.nz> > --- Series applied to nand/next in place of the previous -buggy- series, with your SoB added. Actually as you answered the patches with it, patchwork/git-pwc added it by itself! Thanks, Miqu?l