From mboxrd@z Thu Jan  1 00:00:00 1970
From: Boris Brezillon <boris.brezillon@bootlin.com>
Date: Fri, 22 Jun 2018 14:03:36 +0200
Subject: [U-Boot] [RFC PATCH 15/20] mtd: spinand: Add initial support
 for the MX35LF2GE4AB chip
In-Reply-To: <20180606153040.21397-16-miquel.raynal@bootlin.com>
References: <20180606153040.21397-1-miquel.raynal@bootlin.com>
 <20180606153040.21397-16-miquel.raynal@bootlin.com>
Message-ID: <20180622140336.3b9bfba0@bbrezillon>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Wed,  6 Jun 2018 17:30:35 +0200
Miquel Raynal <miquel.raynal@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@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 ^.

> +		     NAND_ECCREQ(4, 512),
> +		     SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
> +					      &write_cache_variants,
> +					      &update_cache_variants),
> +		     SPINAND_HAS_QE_BIT,
> +		     SPINAND_ECCINFO(&mx35lfxge4ab_ooblayout, NULL)),
>  };
>  
>  static int macronix_spinand_detect(struct spinand_device *spinand)