| Message ID | 1544459182-23655-1-git-send-email-frieder.schrempf@kontron.de |
|---|---|
| Headers | show |
| Series | Prepare devicetrees for the new FSL QSPI driver | expand |
Hi FSL/ARM maintainers, On 10.12.18 17:26, Schrempf Frieder wrote: > From: Frieder Schrempf <frieder.schrempf@kontron.de> > > 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? 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(-) >
+ 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@kontron.de> >> >> 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. 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/ >
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@kontron.de> > >> > >> 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. 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/ > >
>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@kontron.de> >> >> >> >> 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/ >> >
On Sat, Dec 15, 2018 at 10:04:55AM +0000, Schrempf Frieder wrote: > 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. Okay, applied the series. Shawn
From: Frieder Schrempf <frieder.schrempf@kontron.de> 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. 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(-)