Re: [PATCH v2 0/5] dmaengine: add ACPI DMA helpers and use them in dw_dmac

[Vinod Koul <vinod.koul@intel.com>]

On Tue, Apr 16, 2013 at 08:11:57PM +0200, Rafael J. Wysocki wrote:
> On Tuesday, April 16, 2013 03:14:47 PM Andy Shevchenko wrote:
> > On Mon, 2013-04-15 at 22:06 +0530, Vinod Koul wrote: 
> > > On Tue, Apr 09, 2013 at 02:05:42PM +0300, Andy Shevchenko wrote:
> > > > There is a patch series which introduces ACPI DMA helpers in similar way like
> > > > we have for DeviceTree.
> > > > 
> > > > In addition it applies this to the first user, namely dw_dmac driver.
> > > Applied w/o 3 & 5.2 they failed, can you pls rebase and resend
> > > 
> > 
> > Actually those two (at least 5/5) can't be applied on top of slave-dma.
> > The patches requires slave-dma and linux-pm together. I heard it's
> > usually achieved by creating a specific branch in one subsystem
> > (linux-pm in our case) for another.
> > 
> > Rafael, what could we do here?
> 
> I suppose Vinod can pull from my acpi-lpss (which is guaranteed not to be
> rebased at this point) and apply the series on top of that.
Or I can wait for rc1 and apply afterwards...?
> 
> Thanks,
> Rafael
> 
> 
> > > > Since v1:
> > > >  - address one Vinod's comment
> > > >  - replace 5/6 & 6/6 by 5/5 from Rafael
> > > >  - tested on Intel Lynxpoint system
> > > > 
> > > > Andy Shevchenko (4):
> > > >   dma: acpi-dma: introduce ACPI DMA helpers
> > > >   dmaengine: call acpi_dma_request_slave_channel as well
> > > >   dma: acpi-dma: parse CSRT to extract additional resources
> > > >   dw_dmac: add ACPI support
> > > > 
> > > > Rafael J. Wysocki (1):
> > > >   ACPI / LPSS: add support of shared clock
> > > > 
> > > >  Documentation/acpi/enumeration.txt     |  77 ++++++
> > > >  drivers/acpi/Makefile                  |   1 -
> > > >  drivers/acpi/acpi_lpss.c               |  26 +-
> > > >  drivers/acpi/csrt.c                    | 159 ------------
> > > >  drivers/acpi/internal.h                |   1 -
> > > >  drivers/acpi/scan.c                    |   1 -
> > > >  drivers/clk/x86/clk-lpt.c              |  15 +-
> > > >  drivers/dma/Kconfig                    |   4 +
> > > >  drivers/dma/Makefile                   |   1 +
> > > >  drivers/dma/acpi-dma.c                 | 445 +++++++++++++++++++++++++++++++++
> > > >  drivers/dma/dmaengine.c                |   6 +
> > > >  drivers/dma/dw_dmac.c                  |  68 +++--
> > > >  drivers/dma/dw_dmac_regs.h             |   1 -
> > > >  include/linux/acpi_dma.h               | 120 +++++++++
> > > >  include/linux/platform_data/clk-lpss.h |   5 +
> > > >  15 files changed, 741 insertions(+), 189 deletions(-)
> > > >  delete mode 100644 drivers/acpi/csrt.c
> > > >  create mode 100644 drivers/dma/acpi-dma.c
> > > >  create mode 100644 include/linux/acpi_dma.h
> > > > 
> > 
> > 
> -- 
> I speak only for myself.
> Rafael J. Wysocki, Intel Open Source Technology Center.