mbox series

[GIT,PULL] mtd: spi-nor: changes for v6.7-rc1

Message ID 60bc9593-445d-47c9-ac47-0b2e2b11756e@linaro.org
State Accepted
Delegated to: Miquel Raynal
Headers show
Series [GIT,PULL] mtd: spi-nor: changes for v6.7-rc1 | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git tags/spi-nor/for-6.7

Message

Tudor Ambarus Oct. 18, 2023, 7:26 a.m. UTC
Hello!

Please find the SPI NOR PR for v6.7-rc1 below.
Cheers,
ta

The following changes since commit ce9ecca0238b140b88f43859b211c9fdfd8e5b70:

  Linux 6.6-rc2 (2023-09-17 14:40:24 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git
tags/spi-nor/for-6.7

for you to fetch changes up to 6823a8383420263bc061865027836755615a275f:

  mtd: spi-nor: micron-st: use SFDP table for mt25qu512a (2023-10-18
07:47:22 +0300)

----------------------------------------------------------------
For SPI NOR we cleaned the flash info entries in order to have
them slimmer and self explanatory. In order to make the entries
as slim as possible, we introduced sane default values so that
the actual flash entries don't need to specify them. We now use
a flexible macro to specify the flash ID instead of the previous
INFOx() macros that had hardcoded ID lengths.

Instead of:
-       { "w25q512nwm", INFO(0xef8020, 0, 64 * 1024, 0)
-               OTP_INFO(256, 3, 0x1000, 0x1000) },

We now use:
+               .id = SNOR_ID(0xef, 0x80, 0x20),
+               .name = "w25q512nwm",
+               .otp = SNOR_OTP(256, 3, 0x1000, 0x1000),

We also removed some flash entries: the very old Catalyst
SPI EEPROMs that were introduced once with the SPI-NOR subsystem,
and a Fujitsu MRAM. Both should use the at25 EEPROM driver.
The latter even has device tree bindings for the at25 driver.

We made sure that the conversion didn't introduce any unwanted
changes by comparing the .rodata segment before and after the
conversion. The patches landed in linux-next immediately after
v6.6-rc2, we haven't seen any regressions yet.

Apart of the autumn cleaning we introduced a new flash entry,
at25ff321a, and added block protection support for mt25qu512a.

----------------------------------------------------------------
Mamta Shukla (2):
      mtd: spi-nor: micron-st: enable lock/unlock for mt25qu512a
      mtd: spi-nor: micron-st: use SFDP table for mt25qu512a

Michael Walle (41):
      mtd: spi-nor: remove catalyst 'flashes'
      mtd: spi-nor: remove Fujitsu MB85RS1MT support
      mtd: spi-nor: xilinx: use SPI_NOR_ID() in S3AN_INFO()
      mtd: spi-nor: xilinx: remove addr_nbytes from S3AN_INFO()
      mtd: spi-nor: convert .n_sectors to .size
      mtd: spi-nor: default page_size to 256 bytes
      mtd: spi-nor: store .n_banks in struct spi_nor_flash_parameter
      mtd: spi-nor: default .n_banks to 1
      mtd: spi-nor: push 4k SE handling into spi_nor_select_uniform_erase()
      mtd: spi-nor: make sector_size optional
      mtd: spi-nor: drop .parse_sfdp
      mtd: spi-nor: introduce (temporary) INFO0()
      mtd: spi-nor: move the .id and .id_len into an own structure
      mtd: spi-nor: rename .otp_org to .otp and make it a pointer
      mtd: spi-nor: add SNOR_ID() and SNOR_OTP()
      mtd: spi-nor: remove or move flash_info comments
      mtd: spi-nor: atmel: convert flash_info to new format
      mtd: spi-nor: eon: convert flash_info to new format
      mtd: spi-nor: esmt: convert flash_info to new format
      mtd: spi-nor: everspin: convert flash_info to new format
      mtd: spi-nor: gigadevice: convert flash_info to new format
      mtd: spi-nor: intel: convert flash_info to new format
      mtd: spi-nor: issi: convert flash_info to new format
      mtd: spi-nor: macronix: convert flash_info to new format
      mtd: spi-nor: micron-st: convert flash_info to new format
      mtd: spi-nor: spansion: convert flash_info to new format
      mtd: spi-nor: sst: convert flash_info to new format
      mtd: spi-nor: winbond: convert flash_info to new format
      mtd: spi-nor: xilinx: use new macros in S3AN_INFO()
      mtd: spi-nor: xmc: convert flash_info to new format
      mtd: spi-nor: atmel: sort flash_info database
      mtd: spi-nor: eon: sort flash_info database
      mtd: spi-nor: gigadevice: sort flash_info database
      mtd: spi-nor: issi: sort flash_info database
      mtd: spi-nor: macronix: sort flash_info database
      mtd: spi-nor: micron-st: sort flash_info database
      mtd: spi-nor: spansion: sort flash_info database
      mtd: spi-nor: sst: sort flash_info database
      mtd: spi-nor: winbond: sort flash_info entries
      mtd: spi-nor: atmel: drop duplicate entry
      mtd: spi-nor: core: get rid of the INFOx() macros

Nicolas Ferre (1):
      mtd: spi-nor: atmel: add at25ff321a entry

 drivers/mtd/spi-nor/Makefile     |   2 -
 drivers/mtd/spi-nor/atmel.c      | 121 ++++++++++++-------
 drivers/mtd/spi-nor/catalyst.c   |  24 ----
 drivers/mtd/spi-nor/core.c       |  63 ++++------
 drivers/mtd/spi-nor/core.h       | 138 ++++++++++-----------
 drivers/mtd/spi-nor/eon.c        |  74 ++++++++----
 drivers/mtd/spi-nor/esmt.c       |  29 +++--
 drivers/mtd/spi-nor/everspin.c   |  28 ++++-
 drivers/mtd/spi-nor/fujitsu.c    |  21 ----
 drivers/mtd/spi-nor/gigadevice.c |  82 ++++++++-----
 drivers/mtd/spi-nor/intel.c      |  23 ++--
 drivers/mtd/spi-nor/issi.c       | 118 +++++++++++-------
 drivers/mtd/spi-nor/macronix.c   | 220 +++++++++++++++++++++++-----------
 drivers/mtd/spi-nor/micron-st.c  | 423
+++++++++++++++++++++++++++++++++++++++++++----------------------
 drivers/mtd/spi-nor/spansion.c   | 392
+++++++++++++++++++++++++++++++++++++-----------------------
 drivers/mtd/spi-nor/sst.c        | 164 +++++++++++++++----------
 drivers/mtd/spi-nor/swp.c        |  13 +-
 drivers/mtd/spi-nor/sysfs.c      |   6 +-
 drivers/mtd/spi-nor/winbond.c    | 288
++++++++++++++++++++++++++++----------------
 drivers/mtd/spi-nor/xilinx.c     |  32 ++---
 drivers/mtd/spi-nor/xmc.c        |  19 +--
 21 files changed, 1407 insertions(+), 873 deletions(-)
 delete mode 100644 drivers/mtd/spi-nor/catalyst.c
 delete mode 100644 drivers/mtd/spi-nor/fujitsu.c

Comments

Miquel Raynal Oct. 18, 2023, 8:13 a.m. UTC | #1
Hi Tudor,

> For SPI NOR we cleaned the flash info entries in order to have
> them slimmer and self explanatory. In order to make the entries
> as slim as possible, we introduced sane default values so that
> the actual flash entries don't need to specify them. We now use
> a flexible macro to specify the flash ID instead of the previous
> INFOx() macros that had hardcoded ID lengths.
> 
> Instead of:
> -       { "w25q512nwm", INFO(0xef8020, 0, 64 * 1024, 0)
> -               OTP_INFO(256, 3, 0x1000, 0x1000) },
> 
> We now use:
> +               .id = SNOR_ID(0xef, 0x80, 0x20),
> +               .name = "w25q512nwm",
> +               .otp = SNOR_OTP(256, 3, 0x1000, 0x1000),
> 
> We also removed some flash entries: the very old Catalyst
> SPI EEPROMs that were introduced once with the SPI-NOR subsystem,
> and a Fujitsu MRAM. Both should use the at25 EEPROM driver.
> The latter even has device tree bindings for the at25 driver.
> 
> We made sure that the conversion didn't introduce any unwanted
> changes by comparing the .rodata segment before and after the
> conversion. The patches landed in linux-next immediately after
> v6.6-rc2, we haven't seen any regressions yet.
> 
> Apart of the autumn cleaning we introduced a new flash entry,
> at25ff321a, and added block protection support for mt25qu512a.

Thanks a lot for adapting the PR text according to Linus' remarks. I'll
try to do the same :-)

Thanks,
Miquèl
Miquel Raynal Nov. 4, 2023, 11:05 a.m. UTC | #2
Hi Tudor,

tudor.ambarus@linaro.org wrote on Wed, 18 Oct 2023 10:26:16 +0300:

> Hello!
> 
> Please find the SPI NOR PR for v6.7-rc1 below.

Pulled into mtd/next, thanks!

Miquèl