On Tue, Sep 11, 2018 at 02:05:29PM +0530, Vinod Koul wrote: > dma_slave_config direction was marked as deprecated quite some > time back, remove the usage from this driver so that the field > can be removed It looks like we assign direction field in custom struct any time when we call one of ->*prep*() functions, so, it's safe to be removed from ->config(). Acked-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > > Signed-off-by: Vinod Koul <vkoul@xxxxxxxxxx> > --- > CC: Viresh Kumar <vireshk@xxxxxxxxxx> > CC: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > > drivers/dma/dw/core.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/dma/dw/core.c b/drivers/dma/dw/core.c > index f43e6dafe446..d0c3e50b39fb 100644 > --- a/drivers/dma/dw/core.c > +++ b/drivers/dma/dw/core.c > @@ -886,12 +886,7 @@ static int dwc_config(struct dma_chan *chan, struct dma_slave_config *sconfig) > */ > u32 s = dw->pdata->is_idma32 ? 1 : 2; > > - /* Check if chan will be configured for slave transfers */ > - if (!is_slave_direction(sconfig->direction)) > - return -EINVAL; > - > memcpy(&dwc->dma_sconfig, sconfig, sizeof(*sconfig)); > - dwc->direction = sconfig->direction; > > sc->src_maxburst = sc->src_maxburst > 1 ? fls(sc->src_maxburst) - s : 0; > sc->dst_maxburst = sc->dst_maxburst > 1 ? fls(sc->dst_maxburst) - s : 0; > -- > 2.14.4 > -- With Best Regards, Andy Shevchenko
