Re: [PATCH v5 00/11] serial: 8250: split LPSS to 8250_lpss, enable DMA on Quark

From: Andy Shevchenko
Date: Thu May 12 2016 - 11:05:48 EST


On Fri, 2016-05-06 at 18:17 +0300, Andy Shevchenko wrote:
> This is combined series of two things:
> - split out the Intel LPSS specific driver from 8250_pci into
> 8250_lpss
> - enable DMA support on Intel Quark UART
>
> The patch has been tested on few Intel SoCs / platforms. In any case I
> would
> like to ask Bryan to do independent test.
>
> This is targeting serial subsystem, thus it would be nice to get and
> Ack from
> Vinod first. Moreover, the series depends on [1] that is now in
> Vinod's
> slave-dma tree.
>
> That's why I asked Vinod to create immutable tag / branch for the [1]
> and the
> dependants (at least one more, which is sata_dwc_460ex) can use it.
>
> The series can be reached in the branch located at [2].
>
> [1] http://www.spinics.net/lists/kernel/msg2244475.html
> [2] https://bitbucket.org/andy-shev/linux/branch/topic%2Fdw%2Fqrk
>

Do we have a chance to get this? Requirement, i.e. topic/dw branch of
Vinod's slave-dma tree is available and has all prerequisites.

> Since v4:
> - fix an issue what kernel test bot reported (lkp)
> - add a list of supported SoC to Kconfig (Bryan)
> - rename polarity -> hs_polarity (Bryan)
>
> Since v3:
> - rebase on top of recent tty-next branch
> - add Reviewed-by tag from Bryan
>
> Since v2:
> - add tags
> - rebase on top of new version of [1]
>
> Since v1:
> - address most of Peter's comments (mostly changelog to patch 8)
> - add tag to patch 5
> - drop patch 6 from the series to be separately dealt with
>
> Andy Shevchenko (11):
> Â dmaengine: dw: keep copy of custom slave config in dwc
> Â dmaengine: dw: provide probe(), remove() stubs for users
> Â dmaengine: dw: set polarity of handshake interface
> Â dmaengine: dw: override LLP support if asked in platform data
> Â serial: 8250_dma: switch to new dmaengine_terminate_* API
> Â serial: 8250_dma: adjust DMA address of the UART
> Â serial: 8250: enable AFE on ports where FIFO is 16 bytes
> Â serial: 8250_lpss: split LPSS driver to separate module
> Â serial: 8250_lpss: move Quark code from PCI driver
> Â serial: 8250_lpss: enable MSI for Intel Quark
> Â serial: 8250_lpss: enable DMA on Intel Quark UART
>
> Âdrivers/dma/dw/core.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ43 +++--
> Âdrivers/dma/dw/regs.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ5 +-
> Âdrivers/tty/serial/8250/8250.hÂÂÂÂÂÂÂ|ÂÂÂ5 +
> Âdrivers/tty/serial/8250/8250_dma.cÂÂÂ|ÂÂ14 +-
> Âdrivers/tty/serial/8250/8250_lpss.cÂÂ| 357
> +++++++++++++++++++++++++++++++++++
> Âdrivers/tty/serial/8250/8250_pci.cÂÂÂ| 257 +------------------------
> Âdrivers/tty/serial/8250/8250_port.cÂÂ|ÂÂÂ9 +-
> Âdrivers/tty/serial/8250/KconfigÂÂÂÂÂÂ|ÂÂ16 +-
> Âdrivers/tty/serial/8250/MakefileÂÂÂÂÂ|ÂÂÂ1 +
> Âinclude/linux/dma/dw.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ5 +
> Âinclude/linux/platform_data/dma-dw.h |ÂÂÂ4 +
> Â11 files changed, 430 insertions(+), 286 deletions(-)
> Âcreate mode 100644 drivers/tty/serial/8250/8250_lpss.c
>

--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy