On Wed, Feb 10, 2016 at 10:56:26AM +0100, Romain Izard wrote: > Documentation/devicetree/bindings/mtd/atmel-nand.txt | 3 ++- > drivers/mtd/nand/atmel_nand.c | 19 +++++++++++++++++-- > drivers/mtd/nand/atmel_nand_ecc.h | 1 + > 3 files changed, 20 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mtd/atmel-nand.txt b/Documentation/devicetree/bindings/mtd/atmel-nand.txt > index e1887b7d7e53..d53aba98fbc9 100644 > --- a/Documentation/devicetree/bindings/mtd/atmel-nand.txt > +++ b/Documentation/devicetree/bindings/mtd/atmel-nand.txt > @@ -27,7 +27,8 @@ Optional properties: > - atmel,has-pmecc : boolean to enable Programmable Multibit ECC hardware, > capable of BCH encoding and decoding, on devices where it is present. > - atmel,pmecc-cap : error correct capability for Programmable Multibit ECC > - Controller. Supported values are: 2, 4, 8, 12, 24. > + Controller. Supported values are: 2, 4, 8, 12, 24. If the compatible string Looks like you could be using the common 'nand-ecc-strength' binding here. > + is "atmel,sama5d2-nand", 32 is also valid. > - atmel,pmecc-sector-size : sector size for ECC computation. Supported values > are: 512, 1024. And 'nand-ecc-step-size' here. > - atmel,pmecc-lookup-table-offset : includes two offsets of lookup table in ROM Of course we can't break backward compatibility, but it'd be nice if you made sure this worked with the generic bindings. Regards, Brian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html