[Patch v4 0/7] Transition of fsl qspi driver to spi-mem framework
Jagan Teki
jagan at amarulasolutions.com
Sun Jan 26 14:49:53 CET 2020
Hi Vignesh,
On Mon, Jan 13, 2020 at 12:57 PM Kuldeep Singh <kuldeep.singh at nxp.com> wrote:
>
> This entire patch series migrate freescale qspi driver to spi-mem
> framework.
>
> v4 version of series include removal of buildman failure on LS2080AQDS
> build which was observed in cleanup patches. Also, more clear commit
> message of patch 5.
>
> v3 version of series includes correction of copyright in qspi driver and
> also move SPI_FLASH_SPANSION from header to defconfigs in same patch.
>
> v2 version of series includes changes in qspi driver to have 1k size
> instead of complete flash size so as to make driver independent of flash
> size. This also makes it align with linux version of driver. Also added
> support for imx platforms to set TDH bits correctly. There are other minor
> changes in commit messages.
>
> Dependency on patches[1][2]. These patches are required to resolve booting
> crash observed in LS1012ARDB. One crash was related to pfe driver as it was
> accessing flash memory directly and other was based on environment.
> [1] https://patchwork.ozlabs.org/patch/1219462/
> [2] https://patchwork.ozlabs.org/patch/1208299/
>
> Patch 1 adds new qspi driver incorporating spi-mem framework and also
> removal of old driver which was based on spi-nor. The driver is a ported
> version of linux qspi driver. Initial port was done by Frieder. Now, no
> more direct memory access to spi-nor memory is possible i.e accessing flash
> memory using absolute address is not possible.
>
> Patch 2 removes unused qspi config options.
>
> Patch 3 moves FSL_QSPI to defconfig instead of defining it in header files.
>
> Patch 4 removes unused num-cs property from imx platforms.
>
> Patch 5 enables SPI_FLASH_SPANSION in ls1012a defconfig as FSL_QSPI is
> already enabled.
>
> Patch 6 enables SPI_FLASH_SPANSION in defconfigs of LS1046a boards instead
> of defining in header files.
>
> Patch 7 updates the device-tree properties treewide for layerscape boards
> by aligning with linux device-tree properties.
>
> Frieder Schrempf (1):
> imx: imx6sx: Remove unused 'num-cs' property
>
> Kuldeep Singh (6):
> spi: Transform the FSL QuadSPI driver to use the SPI MEM API
> treewide: Remove unused FSL QSPI config options
> configs: ls1043a: Move CONFIG_FSL_QSPI to defconfig
> configs: ls1012a: Enable CONFIG_SPI_FLASH_SPANSION in defconfigs
> configs: ls1046a: Move SPI_FLASH_SPANSION to defconfig
> treewide: Update fsl qspi node dt properties as per spi-mem driver
>
> arch/arm/dts/fsl-ls1012a-2g5rdb.dts | 5 +-
> arch/arm/dts/fsl-ls1012a-frdm.dtsi | 5 +-
> arch/arm/dts/fsl-ls1012a-qds.dtsi | 5 +-
> arch/arm/dts/fsl-ls1012a-rdb.dtsi | 5 +-
> arch/arm/dts/fsl-ls1012a.dtsi | 4 +-
> arch/arm/dts/fsl-ls1043a-qds.dtsi | 5 +-
> arch/arm/dts/fsl-ls1043a.dtsi | 6 +-
> arch/arm/dts/fsl-ls1046a-frwy.dts | 5 +-
> arch/arm/dts/fsl-ls1046a-qds.dtsi | 5 +-
> arch/arm/dts/fsl-ls1046a-rdb.dts | 5 +-
> arch/arm/dts/fsl-ls1046a.dtsi | 4 +-
> arch/arm/dts/fsl-ls1088a-qds.dts | 5 +-
> arch/arm/dts/fsl-ls1088a-rdb.dts | 5 +-
> arch/arm/dts/fsl-ls1088a.dtsi | 2 +-
> arch/arm/dts/fsl-ls2080a-qds.dts | 5 +-
> arch/arm/dts/fsl-ls2080a.dtsi | 4 +-
> arch/arm/dts/fsl-ls2088a-rdb-qspi.dts | 5 +-
> arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi | 2 -
> arch/arm/dts/imx6sx-sdb-u-boot.dtsi | 2 -
> arch/arm/dts/ls1021a-twr.dtsi | 5 +-
> arch/arm/dts/ls1021a.dtsi | 6 +-
> arch/arm/include/asm/arch-fsl-layerscape/config.h | 1 -
> arch/arm/include/asm/arch-ls102xa/config.h | 1 -
> configs/ls1012a2g5rdb_qspi_defconfig | 1 +
> configs/ls1012a2g5rdb_tfa_defconfig | 1 +
> configs/ls1012afrdm_qspi_defconfig | 1 +
> configs/ls1012afrdm_tfa_defconfig | 1 +
> configs/ls1012aqds_qspi_defconfig | 1 +
> configs/ls1012aqds_tfa_SECURE_BOOT_defconfig | 1 +
> configs/ls1012aqds_tfa_defconfig | 1 +
> configs/ls1012ardb_qspi_SECURE_BOOT_defconfig | 1 +
> configs/ls1012ardb_qspi_defconfig | 1 +
> configs/ls1012ardb_tfa_SECURE_BOOT_defconfig | 1 +
> configs/ls1012ardb_tfa_defconfig | 1 +
> configs/ls1043aqds_qspi_defconfig | 1 +
> configs/ls1043aqds_sdcard_qspi_defconfig | 1 +
> configs/ls1043aqds_tfa_SECURE_BOOT_defconfig | 2 +
> configs/ls1043aqds_tfa_defconfig | 1 +
> configs/ls1046aqds_qspi_defconfig | 1 +
> configs/ls1046aqds_sdcard_qspi_defconfig | 1 +
> configs/ls1046aqds_tfa_SECURE_BOOT_defconfig | 1 +
> configs/ls1046aqds_tfa_defconfig | 1 +
> configs/ls1046ardb_qspi_SECURE_BOOT_defconfig | 1 +
> configs/ls1046ardb_qspi_defconfig | 1 +
> configs/ls1046ardb_tfa_SECURE_BOOT_defconfig | 1 +
> configs/ls1046ardb_tfa_defconfig | 1 +
> drivers/spi/fsl_qspi.c | 1574 ++++++++-------------
> drivers/spi/fsl_qspi.h | 145 --
> include/configs/ls1012a_common.h | 17 +-
> include/configs/ls1012afrwy.h | 3 -
> include/configs/ls1012ardb.h | 3 -
> include/configs/ls1021aiot.h | 6 -
> include/configs/ls1021aqds.h | 11 -
> include/configs/ls1021atwr.h | 10 -
> include/configs/ls1043aqds.h | 10 -
> include/configs/ls1046afrwy.h | 9 -
> include/configs/ls1046aqds.h | 19 -
> include/configs/ls1046ardb.h | 20 -
> include/configs/ls1088a_common.h | 6 -
> include/configs/ls1088aqds.h | 8 -
> include/configs/ls1088ardb.h | 18 -
> include/configs/ls2080aqds.h | 5 -
> include/configs/ls2080ardb.h | 6 +-
> include/configs/mx6sxsabreauto.h | 6 -
> include/configs/mx6sxsabresd.h | 11 -
> include/configs/mx6ul_14x14_evk.h | 6 -
> include/configs/mx6ullevk.h | 6 -
> include/configs/mx7dsabresd.h | 8 -
> include/configs/pcm052.h | 7 -
> include/configs/vf610twr.h | 8 -
> scripts/config_whitelist.txt | 5 -
> 71 files changed, 666 insertions(+), 1382 deletions(-)
> delete mode 100644 drivers/spi/fsl_qspi.h
Look fine to me, do you have any comments?
More information about the U-Boot
mailing list