mbox series

[00/12] dmaengine: remove usage of dma_slave_config direction

Message ID 20180911083536.16482-1-vkoul@kernel.org (mailing list archive)
Headers show
Series dmaengine: remove usage of dma_slave_config direction | expand

Message

Vinod Koul Sept. 11, 2018, 8:35 a.m. UTC 
dma_slave_config direction field was marked as deprecated sometime back,
though no new users are being added, some old ones still persist.

So, here is first set of removal batch, please test and ack.

Some drivers and users are next to be done.

One note on renasas drivers, I found it bit difficult to remove it due
to usage of a layer between drivers, so if some Renasas folks can patch
these drivers, it would be great


CC: Ludovic Desroches <ludovic.desroches@microchip.com>
CC: Ray Jui <rjui@broadcom.com>
CC: Scott Branden <sbranden@broadcom.com>
CC: Eric Anholt <eric@anholt.net>
CC: Linus Walleij <linus.walleij@linaro.org>
CC: Viresh Kumar <vireshk@kernel.org>
CC: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
CC: Fabio Estevam <fabio.estevam@nxp.com>
CC: Zhangfei Gao <zhangfei.gao@linaro.org>
CC: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
CC: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
CC: Simon Horman <horms+renesas@verge.net.au>

Vinod Koul (12):
  dmaengine: at_hdmac: remove dma_slave_config direction usage
  dmaengine: bcm2835: remove dma_slave_config direction usage
  dmaengine: coh901318: remove dma_slave_config direction usage
  dmaengine: jz4740: remove dma_slave_config direction usage
  dmaengine: dw: remove dma_slave_config direction usage
  dmaengine: fsl-edma: remove dma_slave_config direction usage
  dmaengine: hsu: remove dma_slave_config direction usage
  dmaengine: idma: remove dma_slave_config direction usage
  dmaengine: imx-dma: remove dma_slave_config direction usage
  dmaengine: k3dma: remove dma_slave_config direction usage
  dmaengine: ep93xx_dma: remove dma_slave_config direction usage
  dmaengine: mmp_tdma: remove dma_slave_config direction usage

 drivers/dma/at_hdmac.c    |  2 +-
 drivers/dma/bcm2835-dma.c |  8 -----
 drivers/dma/coh901318.c   | 28 ++++++++++++---
 drivers/dma/dma-jz4740.c  | 21 ++++++++++--
 drivers/dma/dw/core.c     |  5 ---
 drivers/dma/ep93xx_dma.c  | 21 +++++++++++-
 drivers/dma/fsl-edma.c    | 87 +++++++++++++++++++++++++----------------------
 drivers/dma/hsu/hsu.c     |  4 ---
 drivers/dma/idma64.c      |  4 ---
 drivers/dma/imx-dma.c     | 20 +++++++++--
 drivers/dma/k3dma.c       | 23 ++++++++++---
 drivers/dma/mmp_tdma.c    | 22 ++++++++++--
 12 files changed, 164 insertions(+), 81 deletions(-)

Comments

Vinod Koul Oct. 7, 2018, 1:39 p.m. UTC | #1 
On 11-09-18, 14:05, Vinod Koul wrote:
> dma_slave_config direction field was marked as deprecated sometime back,
> though no new users are being added, some old ones still persist.
> 
> So, here is first set of removal batch, please test and ack.
> 
> Some drivers and users are next to be done.
> 
> One note on renasas drivers, I found it bit difficult to remove it due
> to usage of a layer between drivers, so if some Renasas folks can patch
> these drivers, it would be great

I have not heard any objections, so merging this with ACKs/tested
received.
Geert Uytterhoeven Nov. 12, 2018, 3:36 p.m. UTC | #2 
Hi Vinod,

CC SuperH

On Sun, Oct 7, 2018 at 3:39 PM Vinod <vkoul@kernel.org> wrote:
> On 11-09-18, 14:05, Vinod Koul wrote:
> > dma_slave_config direction field was marked as deprecated sometime back,
> > though no new users are being added, some old ones still persist.
> >
> > So, here is first set of removal batch, please test and ack.
> >
> > Some drivers and users are next to be done.
> >
> > One note on renasas drivers, I found it bit difficult to remove it due
> > to usage of a layer between drivers, so if some Renasas folks can patch
> > these drivers, it would be great

Upon closer look, none of this is used anymore on Renesas ARM SoCs, only
on legacy SuperH SoCs.

So I'd like to defer this to the SuperH maintainers, and people who do have
old SH hardware.

Thanks!

Gr{oetje,eeting}s,

                        Geert