[PATCH v2 1/1] spi: use dmaengine_desc_set_callback()
Andy Shevchenko
andriy.shevchenko at linux.intel.com
Wed Nov 4 12:44:13 PST 2026
On Wed, Nov 04, 2026 at 06:25:32PM +0100, Krzysztof Kozlowski wrote:
> On 04/11/2026 16:22, Frank.Li at oss.nxp.com wrote:
...
> > diff --git a/drivers/spi/atmel-quadspi.c b/drivers/spi/atmel-quadspi.c
> > index 762a3fb0fca81..de205e04a8600 100644
> > --- a/drivers/spi/atmel-quadspi.c
> > +++ b/drivers/spi/atmel-quadspi.c
> > @@ -790,8 +790,7 @@ static int atmel_qspi_dma_xfer(struct atmel_qspi *aq, struct dma_chan *chan,
> > }
> >
> > reinit_completion(&aq->dma_completion);
> > - tx->callback = atmel_qspi_dma_callback;
> > - tx->callback_param = aq;
> > + dmaengine_desc_set_callback(tx, atmel_qspi_dma_callback, NULL, aq);
>
> Andy,
> These three lines look correct.
>
> LGTM:
>
> Reviewed-by: Krzysztof Kozlowski <krzk at kernel.org>
As
Reviewed-by: Krzysztof Kozlowski <krzk at kernel.org> # atmel-quadspi
Note, it's in your power to ask the author to split out what you maintain
to a separate change. So, it's your choice what to do, Ack the full patch,
give a partial Rb, ask for a split, ignore reviewing (often means silent
agreement when maintainer just applies that kind of a patch)
--
With Best Regards,
Andy Shevchenko
More information about the linux-riscv
mailing list