[v3,40/41] mtd: spi-nor: atmel: drop duplicate entry

Message ID	20230807-mtd-flash-info-db-rework-v3-40-e60548861b10@kernel.org
State	Accepted
Delegated to:	Ambarus Tudor
Headers	show Return-Path: <linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org> From: Michael Walle <mwalle@kernel.org> Date: Fri, 08 Sep 2023 12:16:58 +0200 Subject: [PATCH v3 40/41] mtd: spi-nor: atmel: drop duplicate entry MIME-Version: 1.0 Message-Id: <20230807-mtd-flash-info-db-rework-v3-40-e60548861b10@kernel.org> References: <20230807-mtd-flash-info-db-rework-v3-0-e60548861b10@kernel.org> In-Reply-To: <20230807-mtd-flash-info-db-rework-v3-0-e60548861b10@kernel.org> To: Tudor Ambarus <tudor.ambarus@linaro.org>, Pratyush Yadav <pratyush@kernel.org>, Miquel Raynal <miquel.raynal@bootlin.com>, Richard Weinberger <richard@nod.at>, Vignesh Raghavendra <vigneshr@ti.com> Cc: linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, Michael Walle <mwalle@kernel.org> preview: The Atmel AT26DF321 and AT25DF321 have the same ID. Both were just discovered by reading their IDs, that is, there is no probing by name. Thus only the first one (the AT25DF321) in the list was ever p [...] Content analysis details: (-5.2 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at https://www.dnswl.org/, high trust [139.178.84.217 listed in list.dnswl.org] 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record -0.0 SPF_PASS SPF: sender matches SPF record -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from envelope-from domain 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.0 DKIMWL_WL_HIGH DKIMwl.org - High trust sender Precedence: list Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-mtd" <linux-mtd-bounces@lists.infradead.org> Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org
Series	mtd: spi-nor: clean the flash_info database up \| expand [v3,00/41] mtd: spi-nor: clean the flash_info database up [v3,01/41] mtd: spi-nor: remove catalyst 'flashes' [v3,02/41] mtd: spi-nor: remove Fujitsu MB85RS1MT support [v3,03/41] mtd: spi-nor: xilinx: use SPI_NOR_ID() in S3AN_INFO() [v3,04/41] mtd: spi-nor: xilinx: remove addr_nbytes from S3AN_INFO() [v3,05/41] mtd: spi-nor: convert .n_sectors to .size [v3,06/41] mtd: spi-nor: default page_size to 256 bytes [v3,07/41] mtd: spi-nor: store .n_banks in struct spi_nor_flash_parameter [v3,08/41] mtd: spi-nor: default .n_banks to 1 [v3,09/41] mtd: spi-nor: push 4k SE handling into spi_nor_select_uniform_erase() [v3,10/41] mtd: spi-nor: make sector_size optional [v3,11/41] mtd: spi-nor: drop .parse_sfdp [v3,12/41] mtd: spi-nor: introduce (temporary) INFO0() [v3,13/41] mtd: spi-nor: move the .id and .id_len into an own structure [v3,14/41] mtd: spi-nor: rename .otp_org to .otp and make it a pointer [v3,15/41] mtd: spi-nor: add SNOR_ID() and SNOR_OTP() [v3,16/41] mtd: spi-nor: remove or move flash_info comments [v3,17/41] mtd: spi-nor: atmel: convert flash_info to new format [v3,18/41] mtd: spi-nor: eon: convert flash_info to new format [v3,19/41] mtd: spi-nor: esmt: convert flash_info to new format [v3,20/41] mtd: spi-nor: everspin: convert flash_info to new format [v3,21/41] mtd: spi-nor: gigadevice: convert flash_info to new format [v3,22/41] mtd: spi-nor: intel: convert flash_info to new format [v3,23/41] mtd: spi-nor: issi: convert flash_info to new format [v3,24/41] mtd: spi-nor: macronix: convert flash_info to new format [v3,25/41] mtd: spi-nor: micron-st: convert flash_info to new format [v3,26/41] mtd: spi-nor: spansion: convert flash_info to new format [v3,27/41] mtd: spi-nor: sst: convert flash_info to new format [v3,28/41] mtd: spi-nor: winbond: convert flash_info to new format [v3,29/41] mtd: spi-nor: xilinx: use new macros in S3AN_INFO() [v3,30/41] mtd: spi-nor: xmc: convert flash_info to new format [v3,31/41] mtd: spi-nor: atmel: sort flash_info database [v3,32/41] mtd: spi-nor: eon: sort flash_info database [v3,33/41] mtd: spi-nor: gigadevice: sort flash_info database [v3,34/41] mtd: spi-nor: issi: sort flash_info database [v3,35/41] mtd: spi-nor: macronix: sort flash_info database [v3,36/41] mtd: spi-nor: micron-st: sort flash_info database [v3,37/41] mtd: spi-nor: spansion: sort flash_info database [v3,38/41] mtd: spi-nor: sst: sort flash_info database [v3,39/41] mtd: spi-nor: winbond: sort flash_info entries [v3,40/41] mtd: spi-nor: atmel: drop duplicate entry [v3,41/41] mtd: spi-nor: core: get rid of the INFOx() macros

Message ID

20230807-mtd-flash-info-db-rework-v3-40-e60548861b10@kernel.org

State

Accepted

Delegated to:

Ambarus Tudor

Headers

From: Michael Walle <mwalle@kernel.org>
Date: Fri, 08 Sep 2023 12:16:58 +0200
Subject: [PATCH v3 40/41] mtd: spi-nor: atmel: drop duplicate entry
MIME-Version: 1.0
Message-Id: <20230807-mtd-flash-info-db-rework-v3-40-e60548861b10@kernel.org>
References: <20230807-mtd-flash-info-db-rework-v3-0-e60548861b10@kernel.org>
In-Reply-To: <20230807-mtd-flash-info-db-rework-v3-0-e60548861b10@kernel.org>
To: Tudor Ambarus <tudor.ambarus@linaro.org>,
 Pratyush Yadav <pratyush@kernel.org>,
 Miquel Raynal <miquel.raynal@bootlin.com>,
 Richard Weinberger <richard@nod.at>, Vignesh Raghavendra <vigneshr@ti.com>
Cc: linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
 Michael Walle <mwalle@kernel.org>
Precedence: list
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: "linux-mtd" <linux-mtd-bounces@lists.infradead.org>
Errors-To: linux-mtd-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org

Series

mtd: spi-nor: clean the flash_info database up | expand

Commit Message

Michael Walle Sept. 8, 2023, 10:16 a.m. UTC

The Atmel AT26DF321 and AT25DF321 have the same ID. Both were just
discovered by reading their IDs, that is, there is no probing by name.
Thus only the first one (the AT25DF321) in the list was ever probed.
Luckily, the AT25DF is also the newer series. Drop the AT26DF321.

Signed-off-by: Michael Walle <mwalle@kernel.org>
Reviewed-by: Tudor Ambarus <tudor.ambarus@linaro.org>
---
v3:
 - double checked that at26df321 was last in the list
---
 drivers/mtd/spi-nor/atmel.c | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/drivers/mtd/spi-nor/atmel.c b/drivers/mtd/spi-nor/atmel.c
index 18e904962d0e..95f0e139284e 100644
--- a/drivers/mtd/spi-nor/atmel.c
+++ b/drivers/mtd/spi-nor/atmel.c
@@ -206,13 +206,6 @@  static const struct flash_info atmel_nor_parts[] = {
 		.flags = SPI_NOR_HAS_LOCK | SPI_NOR_SWP_IS_VOLATILE,
 		.no_sfdp_flags = SECT_4K,
 		.fixups = &atmel_nor_global_protection_fixups
-	}, {
-		.id = SNOR_ID(0x1f, 0x47, 0x00),
-		.name = "at26df321",
-		.size = SZ_4M,
-		.flags = SPI_NOR_HAS_LOCK | SPI_NOR_SWP_IS_VOLATILE,
-		.no_sfdp_flags = SECT_4K,
-		.fixups = &atmel_nor_global_protection_fixups
 	}, {
 		.id = SNOR_ID(0x1f, 0x47, 0x01),
 		.name = "at25df321a",

[v3,40/41] mtd: spi-nor: atmel: drop duplicate entry

Commit Message

Patch