| Message ID | 1578900390-12119-1-git-send-email-kuldeep.singh@nxp.com |
|---|---|
| Headers | show |
| Series | Transition of fsl qspi driver to spi-mem framework | expand |
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?
Hi Kuldeep, 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 Seems like defconfig changes of these were depends on net changes isn't it? if yes, we need to wait for them to merge first. Jagan.
Hi Jagan, > -----Original Message----- > From: Jagan Teki <jagan at amarulasolutions.com> > Sent: Monday, January 27, 2020 12:50 PM > To: Kuldeep Singh <kuldeep.singh at nxp.com> > Cc: U-Boot-Denx <u-boot at lists.denx.de>; Priyanka Jain > <priyanka.jain at nxp.com>; Ashish Kumar <ashish.kumar at nxp.com>; Stefan > Roese <sr at denx.de>; Schrempf Frieder <frieder.schrempf at kontron.de>; > Vignesh R <vigneshr at ti.com> > Subject: [EXT] Re: [Patch v4 0/7] Transition of fsl qspi driver to spi-mem > framework > > Caution: EXT Email > > Hi Kuldeep, > > 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://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > > hwork.ozlabs.org%2Fpatch%2F1219462%2F&data=02%7C01%7Ckuldeep.s > ingh > > %40nxp.com%7C94b5e5528efc47df25ea08d7a2f94efd%7C686ea1d3bc2b4c6 > fa92cd9 > > > 9c5c301635%7C0%7C0%7C637157063972042137&sdata=DZFAEmt0sA4c > cCPmu%2F > > cArl99B02G2KmiAUYou1RXXBI%3D&reserved=0 > > [2] > > https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > > > hwork.ozlabs.org%2Fpatch%2F1208299%2F&data=02%7C01%7Ckuldeep.s > ingh > > %40nxp.com%7C94b5e5528efc47df25ea08d7a2f94efd%7C686ea1d3bc2b4c6 > fa92cd9 > > > 9c5c301635%7C0%7C0%7C637157063972042137&sdata=3qr7QKERZgk8 > V83QbYMM > > Nb4xM4rUaqm2v3lZ5gzsGAQ%3D&reserved=0 > > > > 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 > > Seems like defconfig changes of these were depends on net changes isn't it? if > yes, we need to wait for them to merge first. Actually, net change is required to resolve the booting crash on LS1012ARDB with this driver. This series can be applied even without net pfe patch. Thanks Kuldeep > > Jagan.
Hi, On 27.01.20 10:20, Kuldeep Singh wrote: > Hi Jagan, > >> -----Original Message----- >> From: Jagan Teki <jagan at amarulasolutions.com> >> Sent: Monday, January 27, 2020 12:50 PM >> To: Kuldeep Singh <kuldeep.singh at nxp.com> >> Cc: U-Boot-Denx <u-boot at lists.denx.de>; Priyanka Jain >> <priyanka.jain at nxp.com>; Ashish Kumar <ashish.kumar at nxp.com>; Stefan >> Roese <sr at denx.de>; Schrempf Frieder <frieder.schrempf at kontron.de>; >> Vignesh R <vigneshr at ti.com> >> Subject: [EXT] Re: [Patch v4 0/7] Transition of fsl qspi driver to spi-mem >> framework >> >> Caution: EXT Email >> >> Hi Kuldeep, >> >> 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://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc >>> >> hwork.ozlabs.org%2Fpatch%2F1219462%2F&data=02%7C01%7Ckuldeep.s >> ingh >>> %40nxp.com%7C94b5e5528efc47df25ea08d7a2f94efd%7C686ea1d3bc2b4c6 >> fa92cd9 >>> >> 9c5c301635%7C0%7C0%7C637157063972042137&sdata=DZFAEmt0sA4c >> cCPmu%2F >>> cArl99B02G2KmiAUYou1RXXBI%3D&reserved=0 >>> [2] >>> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc >>> >> hwork.ozlabs.org%2Fpatch%2F1208299%2F&data=02%7C01%7Ckuldeep.s >> ingh >>> %40nxp.com%7C94b5e5528efc47df25ea08d7a2f94efd%7C686ea1d3bc2b4c6 >> fa92cd9 >>> >> 9c5c301635%7C0%7C0%7C637157063972042137&sdata=3qr7QKERZgk8 >> V83QbYMM >>> Nb4xM4rUaqm2v3lZ5gzsGAQ%3D&reserved=0 >>> >>> 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 >> >> Seems like defconfig changes of these were depends on net changes isn't it? if >> yes, we need to wait for them to merge first. > > Actually, net change is required to resolve the booting crash on LS1012ARDB with this driver. > This series can be applied even without net pfe patch. It could be applied without and as you sad it would break the boot for LS1012ARDB. Therefore no, I don't think we should apply patches that knowingly break things, just because changes elsewhere are not merged yet. So can the maintainers (Joe, Jagan, ...) please figure out how to get [1] merged so we don't block this patch any longer? Thanks, Frieder [1]: https://patchwork.ozlabs.org/patch/1222025/
On Mon, Jan 27, 2020 at 3:17 PM Schrempf Frieder <frieder.schrempf at kontron.de> wrote: > > Hi, > > On 27.01.20 10:20, Kuldeep Singh wrote: > > Hi Jagan, > > > >> -----Original Message----- > >> From: Jagan Teki <jagan at amarulasolutions.com> > >> Sent: Monday, January 27, 2020 12:50 PM > >> To: Kuldeep Singh <kuldeep.singh at nxp.com> > >> Cc: U-Boot-Denx <u-boot at lists.denx.de>; Priyanka Jain > >> <priyanka.jain at nxp.com>; Ashish Kumar <ashish.kumar at nxp.com>; Stefan > >> Roese <sr at denx.de>; Schrempf Frieder <frieder.schrempf at kontron.de>; > >> Vignesh R <vigneshr at ti.com> > >> Subject: [EXT] Re: [Patch v4 0/7] Transition of fsl qspi driver to spi-mem > >> framework > >> > >> Caution: EXT Email > >> > >> Hi Kuldeep, > >> > >> 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://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > >>> > >> hwork.ozlabs.org%2Fpatch%2F1219462%2F&data=02%7C01%7Ckuldeep.s > >> ingh > >>> %40nxp.com%7C94b5e5528efc47df25ea08d7a2f94efd%7C686ea1d3bc2b4c6 > >> fa92cd9 > >>> > >> 9c5c301635%7C0%7C0%7C637157063972042137&sdata=DZFAEmt0sA4c > >> cCPmu%2F > >>> cArl99B02G2KmiAUYou1RXXBI%3D&reserved=0 > >>> [2] > >>> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatc > >>> > >> hwork.ozlabs.org%2Fpatch%2F1208299%2F&data=02%7C01%7Ckuldeep.s > >> ingh > >>> %40nxp.com%7C94b5e5528efc47df25ea08d7a2f94efd%7C686ea1d3bc2b4c6 > >> fa92cd9 > >>> > >> 9c5c301635%7C0%7C0%7C637157063972042137&sdata=3qr7QKERZgk8 > >> V83QbYMM > >>> Nb4xM4rUaqm2v3lZ5gzsGAQ%3D&reserved=0 > >>> > >>> 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 > >> > >> Seems like defconfig changes of these were depends on net changes isn't it? if > >> yes, we need to wait for them to merge first. > > > > Actually, net change is required to resolve the booting crash on LS1012ARDB with this driver. > > This series can be applied even without net pfe patch. > > It could be applied without and as you sad it would break the boot for > LS1012ARDB. > > Therefore no, I don't think we should apply patches that knowingly break > things, just because changes elsewhere are not merged yet. > > So can the maintainers (Joe, Jagan, ...) please figure out how to get > [1] merged so we don't block this patch any longer? I would say, I can apply this spi driver change (since I'm belongs) for now. and rest of config and arch changes would take care of associated board and arch maintainers. Jagan.