Re: [PATCH v1 0/8] spi: Rework DMA mapped flag

From: Neil Armstrong
Date: Mon Jun 03 2024 - 05:11:09 EST


On 31/05/2024 21:42, Andy Shevchenko wrote:
The first part of the series (patches 1 to 7) is an introduction
of a new helper followed by the user conversion.

This consolidates the same code and also makes patch 8 (last one)
be localised to the SPI core part.

The last patch is the main rework to get rid of a recently introduced
hack with a dummy SG list and move to the transfer-based DMA mapped
flag.

That said, the patches 1 to 7 may be applied right away since they
have no functional change intended, while the last one needs more
testing and reviewing.

Andy Shevchenko (8):
spi: Introduce internal spi_xfer_is_dma_mapped() helper
spi: dw: Use new spi_xfer_is_dma_mapped() helper
spi: ingenic: Use new spi_xfer_is_dma_mapped() helper
spi: omap2-mcspi: Use new spi_xfer_is_dma_mapped() helper
spi: pxa2xx: Use new spi_xfer_is_dma_mapped() helper
spi: pci1xxxx: Use new spi_xfer_is_dma_mapped() helper
spi: qup: Use new spi_xfer_is_dma_mapped() helper
spi: Rework per message DMA mapped flag to be per transfer

drivers/spi/internals.h | 8 ++++
drivers/spi/spi-dw-core.c | 4 +-
drivers/spi/spi-ingenic.c | 4 +-
drivers/spi/spi-omap2-mcspi.c | 8 ++--
drivers/spi/spi-pci1xxxx.c | 5 ++-
drivers/spi/spi-pxa2xx.c | 6 +--
drivers/spi/spi-qup.c | 9 ++---
drivers/spi/spi.c | 73 +++++++++++++----------------------
include/linux/spi/spi.h | 11 ++++--
9 files changed, 59 insertions(+), 69 deletions(-)


I applied the serie on next-20240603, it worked fine:

Tested-by: Neil Armstrong <neil.armstrong@xxxxxxxxxx> # on SM8650-QRD

Thanks,
Neil