From: Arnd Bergmann <a...@arndb.de>

Setting slave_id makes no sense with DT based probing, and
should eventually get removed entirely. Address this driver
by no longer setting the field here.

I could not find which DMA driver is used on PIC32, if it's
in the tree at all, but none of the obvious ones even care
about slave_id any more.

Acked-by: Mark Brown <broo...@kernel.org>
Signed-off-by: Arnd Bergmann <a...@arndb.de>
---
 drivers/spi/spi-pic32.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/spi/spi-pic32.c b/drivers/spi/spi-pic32.c
index 5eb7b61bbb4d..f86433b29260 100644
--- a/drivers/spi/spi-pic32.c
+++ b/drivers/spi/spi-pic32.c
@@ -370,7 +370,6 @@ static int pic32_spi_dma_config(struct pic32_spi *pic32s, 
u32 dma_width)
        cfg.src_addr_width = dma_width;
        cfg.dst_addr_width = dma_width;
        /* tx channel */
-       cfg.slave_id = pic32s->tx_irq;
        cfg.direction = DMA_MEM_TO_DEV;
        ret = dmaengine_slave_config(master->dma_tx, &cfg);
        if (ret) {
@@ -378,7 +377,6 @@ static int pic32_spi_dma_config(struct pic32_spi *pic32s, 
u32 dma_width)
                return ret;
        }
        /* rx channel */
-       cfg.slave_id = pic32s->rx_irq;
        cfg.direction = DMA_DEV_TO_MEM;
        ret = dmaengine_slave_config(master->dma_rx, &cfg);
        if (ret)
-- 
2.29.2

Reply via email to