On Fri, Nov 16, 2012 at 4:03 PM, Viresh Kumar <[email protected]> wrote: > On 16 November 2012 19:29, Andy Shevchenko > <[email protected]> wrote: >> This function helps to distinguish the slave type of transfer by checking the >> direction parameter. >> >> Signed-off-by: Andy Shevchenko <[email protected]> >> Cc: Nicolas Ferre <[email protected]> >> Cc: Mika Westerberg <[email protected]> >> Cc: Guennadi Liakhovetski <[email protected]> >> Cc: Linus Walleij <[email protected]> >> --- >> include/linux/dmaengine.h | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h >> index 6402b18..3e1a9026 100644 >> --- a/include/linux/dmaengine.h >> +++ b/include/linux/dmaengine.h >> @@ -618,6 +618,11 @@ static inline int dmaengine_slave_config(struct >> dma_chan *chan, >> (unsigned long)config); >> } >> >> +static inline bool is_slave_xfer(enum dma_transfer_direction direction) >> +{ >> + return (direction == DMA_MEM_TO_DEV) || (direction == >> DMA_DEV_TO_MEM); >> +} > > Reviewed-by: Viresh Kumar <[email protected]> > > Though i still have vote to include DMA_DEV_TO_DEV in thist list :)
I propose to do this later when we have a real user of it. Otherwise it requires to do additional (sometimes useless) checks. -- With Best Regards, Andy Shevchenko -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

