>On Thu, Dec 13, 2018 at 09:45:04AM +0000, Schrempf Frieder wrote: >> + Shawn, Sascha >> >> On 10.12.18 17:31, Schrempf Frieder wrote: >> > Hi FSL/ARM maintainers, >> > >> > On 10.12.18 17:26, Schrempf Frieder wrote: >> >> From: Frieder Schrempf <frieder.schrempf@xxxxxxxxxx> >> >> >> >> We have prepared a new driver for the FSL QSPI controller that works under >> >> the SPI framework. The current devicetrees are broken because they use an >> >> inconsistent scheme for assigning the reg properties. >> >> >> >> As, in contrast to the current driver, the new driver will use the reg >> >> properties to select the correct chip, we need to fix them. >> >> >> >> We also add spi-[tx/rx]-bus-width to make sure the full buswidth will be used >> >> and while at it clean up and remove some unused properties. >> > >> > could you please consider these devicetree patches for v4.21? >> >> Any chance to get this merged? Otherwise we would need to postpone the >> new driver even further. > >As it came up a little bit late (the same day I sent pull request for >4.21), I was considering it as 4.22 material. But as you mentioned that >the series is non-intrusive, I can try to send it to arm-soc folk for >4.21 merge window, and then that will be their call. But before I do >it, I need you to double confirm that the series will not break anything >that is already working. I can confirm, that the series does only remove or add properties to the devicetree, that are currently not used by the driver. Therefore it won't break anything. Thanks, Frieder > >Shawn > >> >> All the changes to the devicetrees in this series are non-intrusive and >> uncritical, anyway. >> >> Thanks, >> Frieder >> >> > >> > We have the new FSL QSPI driver under the SPI framework ([1]) ready, but >> > we need some small fixes to the devicetrees that are currently broken. >> > >> > As the new driver depends on these changes (mainly the change to the reg >> > properties) and it is not possible to add compatibility for the old >> > bindings, it would be really nice to have them in v4.21, so we can queue >> > the driver for v4.22 or whatever the next release will be called. >> > >> > Please also see [2] for some more explanation. >> > >> > Thanks, >> > Frieder >> > >> > [1] https://patchwork.ozlabs.org/cover/1007641/ >> > [2] https://patchwork.ozlabs.org/patch/922817/#1925445 >> > >> >> >> >> Frieder Schrempf (5): >> >> arm64: dts: Remove unused properties from FSL QSPI driver nodes >> >> ARM: dts: Remove unused properties from FSL QSPI driver nodes >> >> ARM: dts: imx6sx-sdb: Fix the reg properties for the FSL QSPI nodes >> >> arm64: dts: Add spi-[tx/rx]-bus-width for the FSL QSPI controller >> >> ARM: dts: Add spi-[tx/rx]-bus-width for the FSL QSPI controller >> >> >> >> arch/arm/boot/dts/imx6sx-sdb-reva.dts | 8 ++++++-- >> >> arch/arm/boot/dts/imx6sx-sdb.dts | 8 ++++++-- >> >> arch/arm/boot/dts/imx6ul-14x14-evk.dtsi | 2 ++ >> >> arch/arm/boot/dts/ls1021a-moxa-uc-8410a.dts | 5 ++--- >> >> arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 3 ++- >> >> arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts | 4 ++-- >> >> arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts | 6 ++++-- >> >> arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi | 4 ++++ >> >> 8 files changed, 28 insertions(+), 12 deletions(-) >> >> >> > ______________________________________________________ >> > Linux MTD discussion mailing list >> > http://lists.infradead.org/mailman/listinfo/linux-mtd/ >> >