[U-Boot] [RFC PATCH 15/20] mtd: spinand: Add initial support for the MX35LF2GE4AB chip

[Miquel Raynal]

Hi Boris,

On Fri, 22 Jun 2018 14:03:36 +0200, Boris Brezillon
<boris.brezillon at bootlin.com> wrote:

> On Wed,  6 Jun 2018 17:30:35 +0200
> Miquel Raynal <miquel.raynal at bootlin.com> wrote:
> 
> > Add support for the MX35LF2GE4AB chip, which is similar to its cousin
> > MX35LF1GE4AB, with two planes instead of one.
> > 
> > Signed-off-by: Miquel Raynal <miquel.raynal at bootlin.com>
> > ---
> >  drivers/mtd/nand/spi/macronix.c | 20 ++++++++++++++------
> >  1 file changed, 14 insertions(+), 6 deletions(-)
> > 
> > diff --git a/drivers/mtd/nand/spi/macronix.c b/drivers/mtd/nand/spi/macronix.c
> > index dd351dcb6c..d761b99d26 100644
> > --- a/drivers/mtd/nand/spi/macronix.c
> > +++ b/drivers/mtd/nand/spi/macronix.c
> > @@ -27,13 +27,13 @@ static SPINAND_OP_VARIANTS(update_cache_variants,
> >  		SPINAND_PROG_LOAD_X4(false, 0, NULL, 0),
> >  		SPINAND_PROG_LOAD(false, 0, NULL, 0));
> >  
> > -static int mx35lf1ge4ab_ooblayout_ecc(struct mtd_info *mtd, int section,
> > +static int mx35lfxge4ab_ooblayout_ecc(struct mtd_info *mtd, int section,
> >  				      struct mtd_oob_region *region)
> >  {
> >  	return -ERANGE;
> >  }
> >  
> > -static int mx35lf1ge4ab_ooblayout_free(struct mtd_info *mtd, int section,
> > +static int mx35lfxge4ab_ooblayout_free(struct mtd_info *mtd, int section,
> >  				       struct mtd_oob_region *region)
> >  {
> >  	if (section)
> > @@ -45,9 +45,9 @@ static int mx35lf1ge4ab_ooblayout_free(struct mtd_info *mtd, int section,
> >  	return 0;
> >  }
> >  
> > -static const struct mtd_ooblayout_ops mx35lf1ge4ab_ooblayout = {
> > -	.ecc = mx35lf1ge4ab_ooblayout_ecc,
> > -	.free = mx35lf1ge4ab_ooblayout_free,
> > +static const struct mtd_ooblayout_ops mx35lfxge4ab_ooblayout = {
> > +	.ecc = mx35lfxge4ab_ooblayout_ecc,
> > +	.free = mx35lfxge4ab_ooblayout_free,
> >  };
> >  
> >  static int mx35lf1ge4ab_get_eccsr(struct spinand_device *spinand, u8 *eccsr)
> > @@ -102,8 +102,16 @@ static const struct spinand_info macronix_spinand_table[] = {
> >  					      &write_cache_variants,
> >  					      &update_cache_variants),
> >  		     SPINAND_HAS_QE_BIT,
> > -		     SPINAND_ECCINFO(&mx35lf1ge4ab_ooblayout,
> > +		     SPINAND_ECCINFO(&mx35lfxge4ab_ooblayout,
> >  				     mx35lf1ge4ab_ecc_get_status)),
> > +	SPINAND_INFO("MX35LF2GE4AB", 0x22,
> > +		     NAND_MEMORG(1, 2048, 64, 64, 1024, 2, 1, 1),  
> 
> 		    I think it should be 2048 here ^.

I just checked, I think you are right. I'll edit this for the next
version.

Thanks,
Miquèl