| Message ID | 20210309053116.1486347-1-hs@denx.de (mailing list archive) |
|---|---|
| Headers | show |
| Series | enable flexspi support on imx8mp | expand |
Hi Heiko, On 21-03-09 06:31, Heiko Schocher wrote: > > This series enables support for the SPI NOR on the > imx8mp based phyboard-pollux-rdk board. > > Patches new in v2: > "spi: fspi: enable fspi driver for on imx8mp" > which adds own compatible entry for imx8mp > > and seperate in own patch the documentation entry in > patch "dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller" > as checkpatch says: > > warning: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst Thanks for picking up the comments :) Did you missed to send them or did you used an other Cc: and To: for the new patches? Regards, Marco
Hello Marco, On 09.03.21 09:19, Marco Felsch wrote: > Hi Heiko, > > On 21-03-09 06:31, Heiko Schocher wrote: >> >> This series enables support for the SPI NOR on the >> imx8mp based phyboard-pollux-rdk board. >> >> Patches new in v2: >> "spi: fspi: enable fspi driver for on imx8mp" >> which adds own compatible entry for imx8mp >> >> and seperate in own patch the documentation entry in >> patch "dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller" >> as checkpatch says: >> >> warning: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst > > Thanks for picking up the comments :) Did you missed to send them or did > you used an other Cc: and To: for the new patches? Damn, I use patman tool from u-boot source whoch generates cc list and missed to add you explicitely to Cc... sorry They are all on linux-arm-kernel: http://lists.infradead.org/pipermail/linux-arm-kernel/2021-March/643290.html http://lists.infradead.org/pipermail/linux-arm-kernel/2021-March/643291.html http://lists.infradead.org/pipermail/linux-arm-kernel/2021-March/643292.html http://lists.infradead.org/pipermail/linux-arm-kernel/2021-March/643293.html bye, Heiko
On Tue, Mar 09, 2021 at 06:31:12AM +0100, Heiko Schocher wrote: > > This series enables support for the SPI NOR on the > imx8mp based phyboard-pollux-rdk board. > > Patches new in v2: > "spi: fspi: enable fspi driver for on imx8mp" > which adds own compatible entry for imx8mp > > and seperate in own patch the documentation entry in > patch "dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller" > as checkpatch says: > > warning: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst > > > Changes in v2: > - work in comments from Marco > add own compatible entry for imx8mp > - work in comments from Marco > - add own compatible entry "nxp,imx8mp-fspi" > - reworked order of properties as Marco mentioned > - work in comments from Marco and Teresa > - rename node into "'som_flash: flash@0 { }" > - compatible is now first entry > - removed #size-cells and #address-cells > as no child node. If bootloader adds them bootloader > can add them too. > > Heiko Schocher (4): > spi: fspi: enable fspi driver for on imx8mp > dt-bindings: spi: add compatible entry for imx8mp in FlexSPI > controller > arm64: dts: imx8mp: add flexspi node > arm64: imx8mp: imx8mp-phycore-som enable spi nor Two DTS patch look good. Ping me when driver and bindings changes are accepted. Shawn
Hello Shawn, On 15.03.21 07:47, Shawn Guo wrote: > On Tue, Mar 09, 2021 at 06:31:12AM +0100, Heiko Schocher wrote: >> >> This series enables support for the SPI NOR on the >> imx8mp based phyboard-pollux-rdk board. >> >> Patches new in v2: >> "spi: fspi: enable fspi driver for on imx8mp" >> which adds own compatible entry for imx8mp >> >> and seperate in own patch the documentation entry in >> patch "dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller" >> as checkpatch says: >> >> warning: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst >> >> >> Changes in v2: >> - work in comments from Marco >> add own compatible entry for imx8mp >> - work in comments from Marco >> - add own compatible entry "nxp,imx8mp-fspi" >> - reworked order of properties as Marco mentioned >> - work in comments from Marco and Teresa >> - rename node into "'som_flash: flash@0 { }" >> - compatible is now first entry >> - removed #size-cells and #address-cells >> as no child node. If bootloader adds them bootloader >> can add them too. >> >> Heiko Schocher (4): >> spi: fspi: enable fspi driver for on imx8mp >> dt-bindings: spi: add compatible entry for imx8mp in FlexSPI >> controller >> arm64: dts: imx8mp: add flexspi node >> arm64: imx8mp: imx8mp-phycore-som enable spi nor > > Two DTS patch look good. Ping me when driver and bindings changes are > accepted. Thanks! Hmm.. I have not splitted this series into 2 series... should I do this now? bye, Heiko
On Mon, Mar 15, 2021 at 08:10:37AM +0100, Heiko Schocher wrote: > Hello Shawn, > > On 15.03.21 07:47, Shawn Guo wrote: > > On Tue, Mar 09, 2021 at 06:31:12AM +0100, Heiko Schocher wrote: > >> > >> This series enables support for the SPI NOR on the > >> imx8mp based phyboard-pollux-rdk board. > >> > >> Patches new in v2: > >> "spi: fspi: enable fspi driver for on imx8mp" > >> which adds own compatible entry for imx8mp > >> > >> and seperate in own patch the documentation entry in > >> patch "dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller" > >> as checkpatch says: > >> > >> warning: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst > >> > >> > >> Changes in v2: > >> - work in comments from Marco > >> add own compatible entry for imx8mp > >> - work in comments from Marco > >> - add own compatible entry "nxp,imx8mp-fspi" > >> - reworked order of properties as Marco mentioned > >> - work in comments from Marco and Teresa > >> - rename node into "'som_flash: flash@0 { }" > >> - compatible is now first entry > >> - removed #size-cells and #address-cells > >> as no child node. If bootloader adds them bootloader > >> can add them too. > >> > >> Heiko Schocher (4): > >> spi: fspi: enable fspi driver for on imx8mp > >> dt-bindings: spi: add compatible entry for imx8mp in FlexSPI > >> controller > >> arm64: dts: imx8mp: add flexspi node > >> arm64: imx8mp: imx8mp-phycore-som enable spi nor > > > > Two DTS patch look good. Ping me when driver and bindings changes are > > accepted. > > Thanks! > > Hmm.. I have not splitted this series into 2 series... should I do > this now? No, you do not need to. I sent the message only because I didn't receive patch #1 and #2, so I will not be aware of their status. Shawn
This series enables support for the SPI NOR on the imx8mp based phyboard-pollux-rdk board. Patches new in v2: "spi: fspi: enable fspi driver for on imx8mp" which adds own compatible entry for imx8mp and seperate in own patch the documentation entry in patch "dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller" as checkpatch says: warning: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst Changes in v2: - work in comments from Marco add own compatible entry for imx8mp - work in comments from Marco - add own compatible entry "nxp,imx8mp-fspi" - reworked order of properties as Marco mentioned - work in comments from Marco and Teresa - rename node into "'som_flash: flash@0 { }" - compatible is now first entry - removed #size-cells and #address-cells as no child node. If bootloader adds them bootloader can add them too. Heiko Schocher (4): spi: fspi: enable fspi driver for on imx8mp dt-bindings: spi: add compatible entry for imx8mp in FlexSPI controller arm64: dts: imx8mp: add flexspi node arm64: imx8mp: imx8mp-phycore-som enable spi nor .../devicetree/bindings/spi/spi-nxp-fspi.txt | 1 + .../dts/freescale/imx8mp-phycore-som.dtsi | 25 +++++++++++++++++++ arch/arm64/boot/dts/freescale/imx8mp.dtsi | 16 ++++++++++++ drivers/spi/spi-nxp-fspi.c | 1 + 4 files changed, 43 insertions(+)