Subject: Re: [PATCH v3 13/40] mtd: nand: Add more parameters to the nand_ecc_props structure

From: Miquel Raynal <[email-masked]>
Date: Thu, 16 Jan 2020 09:59:53 +0100

Hi Boris,

Boris Brezillon <[email-masked]> wrote on Sat, 12 Oct
2019 11:37:45 +0200:

> On Thu, 19 Sep 2019 21:31:13 +0200
> Miquel Raynal <[email-masked]> wrote:
> 
> > Prepare the migration to the generic ECC framework by adding more
> > fields to the nand_ecc_props structure which will be used widely to
> > describe different kind of ECC properties.
> > 
> > Signed-off-by: Miquel Raynal <[email-masked]>
> > ---
> >  include/linux/mtd/nand.h | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> > 
> > diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
> > index 7072f14239e5..11cd7cc81a7a 100644
> > --- a/include/linux/mtd/nand.h
> > +++ b/include/linux/mtd/nand.h
> > @@ -129,12 +129,20 @@ struct nand_page_io_req {
> >  
> >  /**
> >   * struct nand_ecc_props - NAND ECC properties
> > + * @provider: ECC engine provider type
> > + * @placement: OOB placement (if relevant)
> > + * @algo: ECC algorithm (if relevant)
> >   * @strength: ECC strength
> >   * @step_size: Number of bytes per step
> > + * @flags: Misc properties
> >   */
> >  struct nand_ecc_props {
> > +	unsigned int provider;
> > +	unsigned int placement;
> > +	unsigned int algo;  
> 
> Hm, we should have enums here, and it's better to introduce the
> provider/placement/algo definitions along with the fields.

Absolutely! Good catch, this is fixed.

Thanks,
Miquèl

_______________________________________________
linux-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel