Edward O'Callaghan has submitted this change and it was merged. ( https://review.coreboot.org/c/flashrom/+/33836 )
Change subject: flashchips: Drop dead code of AT26DF321 ......................................................................
flashchips: Drop dead code of AT26DF321
The definition for the AT26DF321 has been commented out since it was first added in 2008. The chip now appears to be obsolete, being marked "obsolete" and unstocked at Digikey. It is also only referred to in historical documents on the manufacturer's website (microchip.com).
To avoid further bitrot of this dead code, drop it.
Signed-off-by: Alan Green avg@google.com Change-Id: Ib30b3a16f25de5def508d90ec9375563b1d4d384 Reviewed-on: https://review.coreboot.org/c/flashrom/+/33836 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Edward O'Callaghan quasisec@chromium.org --- M flashchips.c M flashchips.h 2 files changed, 1 insertion(+), 20 deletions(-)
Approvals: build bot (Jenkins): Verified Edward O'Callaghan: Looks good to me, approved
diff --git a/flashchips.c b/flashchips.c index f3aab52..ac65510 100644 --- a/flashchips.c +++ b/flashchips.c @@ -1709,6 +1709,7 @@ .voltage = {2700, 3600}, },
+ /*The AT26DF321 has the same ID as the AT25DF321. */ { .vendor = "Atmel", .name = "AT25DF321", @@ -2510,25 +2511,6 @@ .voltage = {2700, 3600}, },
- /*The AT26DF321 has the same ID as the AT25DF321. */ - /*{ - .vendor = "Atmel", - .name = "AT26DF321", - .bustype = BUS_SPI, - .manufacture_id = ATMEL_ID, - .model_id = ATMEL_AT26DF321, - .total_size = 4096, - .page_size = 256, - .feature_bits = FEATURE_WRSR_WREN, - .tested = TEST_UNTESTED, - .probe = probe_spi_rdid, - .probe_timing = TIMING_ZERO, - .printlock = spi_prettyprint_status_register_at26df081a, - .unlock = spi_disable_blockprotect, - .write = spi_chip_write_256, - .read = spi_chip_read, - },*/ - { .vendor = "Atmel", .name = "AT26F004", diff --git a/flashchips.h b/flashchips.h index a1b38e0..1c7ba99 100644 --- a/flashchips.h +++ b/flashchips.h @@ -156,7 +156,6 @@ #define ATMEL_AT26DF081A 0x4501 #define ATMEL_AT26DF161 0x4600 #define ATMEL_AT26DF161A 0x4601 -#define ATMEL_AT26DF321 0x4700 /* Same as 25DF321 */ #define ATMEL_AT26F004 0x0400 #define ATMEL_AT29LV512 0x3D #define ATMEL_AT29LV010A 0x35 /* Same as AT29BV010A, the latter works down to 2.7V */