Re: [PATCH v3 12/40] mtd: nand: Rename a core structure

[Boris Brezillon <boris.brezillon@xxxxxxxxxxxxx>]

On Thu, 16 Jan 2020 09:53:01 +0100
Miquel Raynal <miquel.raynal@xxxxxxxxxxx> wrote:

> Hi Boris,
> 
> Boris Brezillon <boris.brezillon@xxxxxxxxxxxxx> wrote on Sat, 12 Oct
> 2019 11:35:30 +0200:
> 
> > On Thu, 19 Sep 2019 21:31:12 +0200
> > Miquel Raynal <miquel.raynal@xxxxxxxxxxx> wrote:
> >   
> > > Prepare the migration to a generic ECC engine by renaming the
> > > nand_ecc_req structure into nand_ecc_props. This structure will be the
> > > base of a wider 'nand_ecc' structure.
> > > 
> > > Signed-off-by: Miquel Raynal <miquel.raynal@xxxxxxxxxxx>
> > > ---
> > >  include/linux/mtd/nand.h    | 8 ++++----
> > >  include/linux/mtd/spinand.h | 2 +-
> > >  2 files changed, 5 insertions(+), 5 deletions(-)
> > > 
> > > diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
> > > index 531c1799bf2c..7072f14239e5 100644
> > > --- a/include/linux/mtd/nand.h
> > > +++ b/include/linux/mtd/nand.h
> > > @@ -128,11 +128,11 @@ struct nand_page_io_req {
> > >  };
> > >  
> > >  /**
> > > - * struct nand_ecc_req - NAND ECC requirements
> > > + * struct nand_ecc_props - NAND ECC properties
> > >   * @strength: ECC strength
> > > - * @step_size: ECC step/block size
> > > + * @step_size: Number of bytes per step
> > >   */
> > > -struct nand_ecc_req {
> > > +struct nand_ecc_props {
> > >  	unsigned int strength;
> > >  	unsigned int step_size;
> > >  };
> > > @@ -191,7 +191,7 @@ struct nand_ops {
> > >  struct nand_device {
> > >  	struct mtd_info mtd;
> > >  	struct nand_memory_organization memorg;
> > > -	struct nand_ecc_req eccreq;
> > > +	struct nand_ecc_props eccreq;    
> > 
> > Let's rename this field too: s/eccreq/eccprops/  
> 
> This is just a transition step, this field is being replaced by a much
> wider structure two patches later. The impact of renaming this field is
> huge compared to its interest so I'd prefer not to do it.

Fair enough. I guess it's no longer named eccreq after the transition.

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