Re: [PATCH v2 3/4] mtd: rawnand: mtk: Replace max_sector_size with sector size array

[Boris Brezillon <boris.brezillon@bootlin.com>]

On Mon, 16 Apr 2018 15:41:03 +0800
Xiaolei Li <xiaolei.li@mediatek.com> wrote:

> Remove max_sector_size from struct mtk_nfc_caps, and use sector size
> array and number of sector size to show sector sizes that controller
> supports.
> 
> Signed-off-by: Xiaolei Li <xiaolei.li@mediatek.com>

I don't see the point of this patch. I was seeing a clear benefit when
you were moving to the generic ECC selection code, but since you've
dropped this part from the patch series, this patch becomes useless.

Am I missing something?

> ---
>  drivers/mtd/nand/raw/mtk_nand.c | 32 +++++++++++++++++++++++++++-----
>  1 file changed, 27 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/mtd/nand/raw/mtk_nand.c b/drivers/mtd/nand/raw/mtk_nand.c
> index 6977da3..03a4df2 100644
> --- a/drivers/mtd/nand/raw/mtk_nand.c
> +++ b/drivers/mtd/nand/raw/mtk_nand.c
> @@ -106,10 +106,11 @@
>  struct mtk_nfc_caps {
>  	const u8 *spare_size;
>  	u8 num_spare_size;
> +	const int *sector_size;
> +	int num_sector_size;
>  	u8 pageformat_spare_shift;
>  	u8 nfi_clk_div;
>  	u8 max_sector;
> -	u32 max_sector_size;
>  };
>  
>  struct mtk_nfc_bad_mark_ctl {
> @@ -178,6 +179,14 @@ struct mtk_nfc {
>  	16, 26, 27, 28
>  };
>  
> +static const int sector_size_max_1k[] = {
> +	1024, 512
> +};
> +
> +static const int sector_size_max_512[] = {
> +	512
> +};
> +
>  static inline struct mtk_nfc_nand_chip *to_mtk_nand(struct nand_chip *nand)
>  {
>  	return container_of(nand, struct mtk_nfc_nand_chip, nand);
> @@ -258,6 +267,16 @@ static inline u8 nfi_readb(struct mtk_nfc *nfc, u32 reg)
>  	return readb_relaxed(nfc->regs + reg);
>  }
>  
> +static int mtk_nfc_max_sector_size(struct mtk_nfc *nfc)
> +{
> +	int i, sector_size = 0;
> +
> +	for (i = 0; i < nfc->caps->num_sector_size; i++)
> +		sector_size = max(sector_size, nfc->caps->sector_size[i]);
> +
> +	return sector_size;
> +}
> +
>  static void mtk_nfc_hw_reset(struct mtk_nfc *nfc)
>  {
>  	struct device *dev = nfc->dev;
> @@ -1207,7 +1226,7 @@ static int mtk_nfc_ecc_init(struct device *dev, struct mtd_info *mtd)
>  		 */
>  		if (nand->ecc.size < 1024) {
>  			if (mtd->writesize > 512 &&
> -			    nfc->caps->max_sector_size > 512) {
> +			    mtk_nfc_max_sector_size(nfc) > 512) {
>  				nand->ecc.size = 1024;
>  				nand->ecc.strength <<= 1;
>  			} else {
> @@ -1389,28 +1408,31 @@ static int mtk_nfc_nand_chips_init(struct device *dev, struct mtk_nfc *nfc)
>  static const struct mtk_nfc_caps mtk_nfc_caps_mt2701 = {
>  	.spare_size = spare_size_mt2701,
>  	.num_spare_size = 16,
> +	.sector_size = sector_size_max_1k,
> +	.num_sector_size = 2,
>  	.pageformat_spare_shift = 4,
>  	.nfi_clk_div = 1,
>  	.max_sector = 16,
> -	.max_sector_size = 1024,
>  };
>  
>  static const struct mtk_nfc_caps mtk_nfc_caps_mt2712 = {
>  	.spare_size = spare_size_mt2712,
>  	.num_spare_size = 19,
> +	.sector_size = sector_size_max_1k,
> +	.num_sector_size = 2,
>  	.pageformat_spare_shift = 16,
>  	.nfi_clk_div = 2,
>  	.max_sector = 16,
> -	.max_sector_size = 1024,
>  };
>  
>  static const struct mtk_nfc_caps mtk_nfc_caps_mt7622 = {
>  	.spare_size = spare_size_mt7622,
>  	.num_spare_size = 4,
> +	.sector_size = sector_size_max_512,
> +	.num_sector_size = 1,
>  	.pageformat_spare_shift = 4,
>  	.nfi_clk_div = 1,
>  	.max_sector = 8,
> -	.max_sector_size = 512,
>  };
>  
>  static const struct of_device_id mtk_nfc_id_table[] = {