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

From: Miquel Raynal <[email-masked]>
Date: Tue, 2 Apr 2019 09:59:36 +0200

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-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel