[PATCH] mtd: spi-nor-ids: Deduplicate w25q16dw entry

[Michal Simek]

On 9/6/24 23:09, Marek Vasut wrote:
> The w25q16dw entry exists twice in spi_nor_ids, remove the less
> complete variant of the entry and keep only one copy of it.
> 
> Fixes: baef13ec9d59 ("mtd: spi-nor-ids: Add support for flashes tested by xilinx")
> Signed-off-by: Marek Vasut <marex at denx.de>
> ---
> Cc: Andre Przywara <andre.przywara at arm.com>
> Cc: Ashok Reddy Soma <ashok.reddy.soma at amd.com>
> Cc: Dhruva Gole <d-gole at ti.com>
> Cc: Jagan Teki <jagan at amarulasolutions.com>
> Cc: Jim Liu <jim.t90615 at gmail.com>
> Cc: Jonas Karlman <jonas at kwiboo.se>
> Cc: Michal Simek <michal.simek at amd.com>
> Cc: Neal Frager <neal.frager at amd.com>
> Cc: Prasad Kummari <prasad.kummari at amd.com>
> Cc: Teik Heng Chong <teik.heng.chong at intel.com>
> Cc: Tom Rini <trini at konsulko.com>
> Cc: Vignesh R <vigneshr at ti.com>
> Cc: u-boot at lists.denx.de
> ---
>   drivers/mtd/spi/spi-nor-ids.c | 8 ++------
>   1 file changed, 2 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/mtd/spi/spi-nor-ids.c b/drivers/mtd/spi/spi-nor-ids.c
> index 88709a52b3a..d1389e681d6 100644
> --- a/drivers/mtd/spi/spi-nor-ids.c
> +++ b/drivers/mtd/spi/spi-nor-ids.c
> @@ -430,11 +430,6 @@ const struct flash_info spi_nor_ids[] = {
>   	{ INFO("w25x05", 0xef3010, 0, 64 * 1024,  1,  SECT_4K) },
>   	{ INFO("w25x40", 0xef3013, 0, 64 * 1024,  8,  SECT_4K) },
>   	{ INFO("w25x16", 0xef3015, 0, 64 * 1024,  32, SECT_4K) },
> -	{
> -		INFO("w25q16dw", 0xef6015, 0, 64 * 1024,  32,
> -			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
> -			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
> -	},
>   	{ INFO("w25x32", 0xef3016, 0, 64 * 1024,  64, SECT_4K) },
>   	{ INFO("w25q20cl", 0xef4012, 0, 64 * 1024,  4, SECT_4K) },
>   	{ INFO("w25q20bw", 0xef5012, 0, 64 * 1024,  4, SECT_4K) },
> @@ -442,7 +437,8 @@ const struct flash_info spi_nor_ids[] = {
>   	{ INFO("w25q32", 0xef4016, 0, 64 * 1024,  64, SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) },
>   	{
>   		INFO("w25q16dw", 0xef6015, 0, 64 * 1024,  32,
> -			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ)
> +			SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ |
> +			SPI_NOR_HAS_LOCK | SPI_NOR_HAS_TB)
>   	},
>   	{
>   		INFO("w25q32dw", 0xef6016, 0, 64 * 1024,  64,


Reviewed-by: Michal Simek <michal.simek at amd.com>

Thanks,
Michal