On Tue, Jun 02, 2020 at 10:18:28AM +0200, Lars Povlsen wrote: > > Serge Semin writes: > > > Hello Lars, > > > > On Wed, May 13, 2020 at 04:00:21PM +0200, Lars Povlsen wrote: > >> This is an add-on series to the main SoC Sparx5 series > >> (Message-ID: <20200513125532.24585-1-lars.povl...@microchip.com>). > >> > >> The series add support for Sparx5 on top of the existing > >> ocelot/jaguar2 spi driver. > >> > >> It spins off the existing support for the MSCC platforms into a > >> separate driver, as adding new platforms from the MSCC/Microchip > >> product lines will further complicate (clutter) the original driver. > >> > >> New YAML dt-bindings are provided for the resulting driver. > >> > >> It is expected that the DT patches are to be taken directly by the arm-soc > >> maintainers. > > > > Regarding our cooperation. It can be implemented as follows. Since your > > patchset > > is less cumbersome than mine and is more ready to be integrated into the > > generic DW > > APB SSI code, it would be better to first make it through Mark', Andy' and > > my reviews > > to be further merged into the kernel version of the driver. After that I'll > > have > > my code altered so it could be applied on top of your patches. When > > everything > > is done we'll have a more comprehensive DW APB SSI driver with poll-based > > PIO operations support, new features like rx-delay, etc. > > > > Hi Serge! > > I think I would be able to work on the SPI patches this week. Should I > base it on the current spi-next or 5.7? Then address the comments and > send out a new revision? > > Thanks for reaching out.
Sorry for a delay. I had to finish urgent tasks first. I'll give my review comments shortly today. -Sergey > > ---Lars > > > Thank you one more time for the series you've shared with us. Let's see > > what can > > be done to improve it... > > > > -Sergey > > > >> > >> Lars Povlsen (10): > >> spi: dw: Add support for polled operation via no IRQ specified in DT > >> spi: dw: Add support for RX sample delay register > >> spi: dw: Add support for client driver memory operations > >> dt-bindings: spi: Add bindings for spi-dw-mchp > >> spi: spi-dw-mmio: Spin off MSCC platforms into spi-dw-mchp > >> dt-bindings: spi: spi-dw-mchp: Add Sparx5 support > >> spi: spi-dw-mchp: Add Sparx5 support > >> arm64: dts: sparx5: Add SPI controller > >> arm64: dts: sparx5: Add spi-nor support > >> arm64: dts: sparx5: Add spi-nand devices > >> > >> .../bindings/spi/mscc,ocelot-spi.yaml | 89 ++++ > >> .../bindings/spi/snps,dw-apb-ssi.txt | 7 +- > >> MAINTAINERS | 2 + > >> arch/arm64/boot/dts/microchip/sparx5.dtsi | 37 ++ > >> .../boot/dts/microchip/sparx5_pcb125.dts | 16 + > >> .../boot/dts/microchip/sparx5_pcb134.dts | 22 + > >> .../dts/microchip/sparx5_pcb134_board.dtsi | 9 + > >> .../boot/dts/microchip/sparx5_pcb135.dts | 23 + > >> .../dts/microchip/sparx5_pcb135_board.dtsi | 9 + > >> arch/mips/configs/generic/board-ocelot.config | 2 +- > >> drivers/spi/Kconfig | 7 + > >> drivers/spi/Makefile | 1 + > >> drivers/spi/spi-dw-mchp.c | 399 ++++++++++++++++++ > >> drivers/spi/spi-dw-mmio.c | 93 ---- > >> drivers/spi/spi-dw.c | 31 +- > >> drivers/spi/spi-dw.h | 4 + > >> 16 files changed, 644 insertions(+), 107 deletions(-) > >> create mode 100644 > >> Documentation/devicetree/bindings/spi/mscc,ocelot-spi.yaml > >> create mode 100644 drivers/spi/spi-dw-mchp.c > >> > >> -- > >> 2.26.2 > >> > >> _______________________________________________ > >> linux-arm-kernel mailing list > >> linux-arm-ker...@lists.infradead.org > >> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > > -- > Lars Povlsen, > Microchip
