On Mon, Oct 07, 2024 at 11:52:21AM -0400, Frank Li wrote: > On Mon, Oct 07, 2024 at 06:06:45PM +0300, Andy Shevchenko wrote: ... > > * Please note in any other slave case, you have to setup chan->private > > * with 'struct imx_dma_data' in your own filter function if you want to > > - * request dma channel by dma_request_channel() rather than > > - * dma_request_slave_channel(). Othwise, 'MEMCPY in case?' will appear > > - * to warn you to correct your filter function. > > + * request DMA channel by dma_request_channel(), otherwise, 'MEMCPY in > > + * case?' will appear to warn you to correct your filter function. > > It just change comments, why combined with dmaengine.h change. Because this comment is related to what is done below. ... > > struct dma_chan *chan; > > > > - chan = dma_request_slave_channel(dev, name); > > - if (chan) > > + chan = dma_request_chan(dev, name); Here is no more dma_request_slave_channel() calls as in the mentioned comment. > > + if (!IS_ERR(chan)) > > return chan; -- With Best Regards, Andy Shevchenko
