Re: [PATCH 1/4] dt-bindings: mtd: Add YAML schemas for the generic NAND options

[Miquel Raynal <miquel.raynal@xxxxxxxxxxx>]

Hi Maxime,

> > > +      nand-ecc-strength:
> > > +        $ref: /schemas/types.yaml#/definitions/uint32
> > > +        description:
> > > +          Number of bits to correct per ECC step.  
> >
> > Is there a range we can define here? Certainly there's a minimum
> > values of at least 1.  
> 
> It doesn't look like there is from a quick grep. The DT seems to be in
> the 4 - 32 range, but I'm not sure if it would make sense to have
> something higher.
> 
> I'll let Miquel and Boris comment.

A value of zero would not have any meaning IMHO. 1 is definitely the
minimum (ECC Hamming algorithm). There is technically no maximum.


Thanks,
Miquèl

______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/