This series adds support for the SpacemiT K1 SoC QSPI. This IP is
generally compatible with the Freescale QSPI driver, requiring three
minor changes to enable it to be supported. The changes are:
- Adding support for optional resets
- Having the clock *not* be disabled when changing its rate
- Allowing the size of storage blocks written to flash chips
to be set to something different from the AHB buffer size
This version of the series mainly adds sign-offs provided for v3.
-Alex
This series is available here:
https://github.com/riscstar/linux/tree/outgoing/qspi-v4
Version 4 incorporates comments recieved during review of v3:
- Moved the allOf block below the required section in patch 2
- Added Frank Li's Reviewed-by on patch 2, 4, and 5
- Added Paul Walmsley's Acked-by on patch 9
Here is version 3:
https://lore.kernel.org/lkml/20251024191550.194946-1-elder@riscstar.com/
Version 3 addresses comments recieved during review of v2:
- This version fully tested and confirmed to work on BPI-F3
- Added Acked-by from Conor on patches 1 and 2
- Added Reviewed-by from Frank on patches 1 and 6
- Moved the reset property conditional under allOf in the binding
- In patch 5, needs_clk_disable() is no longer inline
- Tweaked the description in patch 5
- Renamed a local variable to be "sfa_size" in patch 6
- Changed the subject line in patch 6
- The sfa_size field was not being assigned in spacemit_k1_data;
now it is set to SZ_1K
Here is version 2:
https://lore.kernel.org/lkml/20251023175922.528868-1-elder@riscstar.com/
Version 2 addresses comments recieved during review of v1:
- The "reset" property now only applies to spacemit,k1-qspi compatible.
- Patch 1 (previously patch 2) now points out that this is the first
non-Freescale device using the binding.
- Added Frank Li's Reviewed-by on patch 3.
- A quirk flag has been renamed to be QUADSPI_QUIRK_SKIP_CLK_DISABLE.
- The predicate for that quirk now returns bool type.
- All other similar predicates now return bool type; this is done in a
new patch (patch 4).
- If non-zero, new field fsl_qspi_devtype_data->sfa_size defines the
size of the serial flash regions, rather than ahb_buf_size.
- A continued line in the Kconfig is now aligned.
- Patch descriptions are wrapped at 75 columns.
Here is version 1:
https://lore.kernel.org/lkml/20251020165152.666221-1-elder@riscstar.com/
Alex Elder (9):
dt-bindings: spi: fsl-qspi: support SpacemiT K1
dt-bindings: spi: fsl-qspi: add optional resets
spi: fsl-qspi: add optional reset support
spi: fsl-qspi: switch predicates to bool
spi: fsl-qspi: add a clock disable quirk
spi: fsl-qspi: introduce sfa_size devtype data
spi: fsl-qspi: support the SpacemiT K1 SoC
riscv: dts: spacemit: enable K1 SoC QSPI on BPI-F3
riscv: defconfig: enable SPI_FSL_QUADSPI as a module
.../bindings/spi/fsl,spi-fsl-qspi.yaml | 21 ++++-
.../boot/dts/spacemit/k1-bananapi-f3.dts | 6 ++
arch/riscv/boot/dts/spacemit/k1-pinctrl.dtsi | 21 +++++
arch/riscv/boot/dts/spacemit/k1.dtsi | 16 ++++
arch/riscv/configs/defconfig | 1 +
drivers/spi/Kconfig | 3 +-
drivers/spi/spi-fsl-qspi.c | 88 +++++++++++++------
7 files changed, 127 insertions(+), 29 deletions(-)
base-commit: 8fec172c82c2b5f6f8e47ab837c1dc91ee3d1b87
--
2.48.1