Hi Marcus, marcus.folkesson@xxxxxxxxx wrote on Tue, 01 Oct 2024 12:42:27 +0200: > Some chips, e.g. Micron MT29F1G08ABBFAH4, has a mandatory on-die ECC. > Add "on-die" as ECC engine type in order to be compatible with those. > > Signed-off-by: Marcus Folkesson <marcus.folkesson@xxxxxxxxx> > --- > Documentation/devicetree/bindings/mtd/davinci-nand.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/mtd/davinci-nand.txt b/Documentation/devicetree/bindings/mtd/davinci-nand.txt > index eb8e2ff4dbd2901b3c396f2e66c1f590a32dcf67..9afda5cd956494c6b3171bdbaecaeb289acd64ea 100644 > --- a/Documentation/devicetree/bindings/mtd/davinci-nand.txt > +++ b/Documentation/devicetree/bindings/mtd/davinci-nand.txt > @@ -44,6 +44,7 @@ Recommended properties : > - "none" > - "soft" > - "hw" > + - "on-die" This file is very legacy and this addition would be totally unneeded if that file had been converted to yaml earlier. Just referencing nand-controller.yaml will authorize nand-ecc-mode = "on-die" (while still marking it legacy). Would you mind converting this file please? Thanks, Miquèl