Re: [PATCH 31/39] dmaengine: ste_dma40: Replace ST-E's home-brewDMA direction defs with generic ones

From: Vinod Koul
Date: Thu May 16 2013 - 03:19:15 EST


On Thu, May 16, 2013 at 08:06:38AM +0100, Lee Jones wrote:
> On Thu, 16 May 2013, Vinod Koul wrote:
>
> > On Wed, May 15, 2013 at 10:51:54AM +0100, Lee Jones wrote:
> > > STEDMA40_*_TO_* direction definitions are identical in all but name to
> > > the pre-defined generic DMA_*_TO_* ones. Let's make things easy by not
> > > duplicating such things.
> > >
> > > Cc: Vinod Koul <vinod.koul@xxxxxxxxx>
> > > Cc: Dan Williams <djbw@xxxxxx>
> > > Cc: Per Forlin <per.forlin@xxxxxxxxxxxxxx>
> > > Cc: Rabin Vincent <rabin@xxxxxx>
> > > Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx>
> > Nice :)
>
> :)
>
> > 1) I dont see the STE macro getting removed, why do we need it
>
> They are removed later in the set, once their use has been removed
> from platform code and all the other drivers.
>
> > 2) last i checked the direction values had a bit idfference b/w what you are
> > using and what dmaengine defines, so hopefully that is taken care of..
>
> Yes, no problem.
>
> <place Ack here> ;)
Acked-by: Vinod Koul <vinod.koul@xxxxxxxxx>

--
~Vinod
>
> --
> Lee Jones
> Linaro ST-Ericsson Landing Team Lead
> Linaro.org â Open source software for ARM SoCs
> Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/