dmaengine: pxa/mmp: stop referencing config->slave_id
authorArnd Bergmann <arnd@arndb.de>
Mon, 22 Nov 2021 22:21:58 +0000 (23:21 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 27 Jan 2022 09:54:07 +0000 (10:54 +0100)
[ Upstream commit 134c37fa250a87a7e77c80a7c59ae16c462e46e0 ]

The last driver referencing the slave_id on Marvell PXA and MMP platforms
was the SPI driver, but this stopped doing so a long time ago, so the
TODO from the earlier patch can no be removed.

Fixes: b729bf34535e ("spi/pxa2xx: Don't use slave_id of dma_slave_config")
Fixes: 13b3006b8ebd ("dma: mmp_pdma: add filter function")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Acked-by: Mark Brown <broonie@kernel.org>
Link: https://lore.kernel.org/r/20211122222203.4103644-7-arnd@kernel.org
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/dma/mmp_pdma.c
drivers/dma/pxa_dma.c

index b84303be8edf55d5ed1dfdf80e5705d8f30fc918..4eb63f1ad224765d690256e9b5eef55d0abe50c7 100644 (file)
@@ -728,12 +728,6 @@ static int mmp_pdma_config_write(struct dma_chan *dchan,
 
        chan->dir = direction;
        chan->dev_addr = addr;
-       /* FIXME: drivers should be ported over to use the filter
-        * function. Once that's done, the following two lines can
-        * be removed.
-        */
-       if (cfg->slave_id)
-               chan->drcmr = cfg->slave_id;
 
        return 0;
 }
index 349fb312c8725678a2a1268a4371047dbcb2ed11..b4ef4f19f7decb9c007a3ecfa369ba974fd1faca 100644 (file)
@@ -911,13 +911,6 @@ static void pxad_get_config(struct pxad_chan *chan,
                *dcmd |= PXA_DCMD_BURST16;
        else if (maxburst == 32)
                *dcmd |= PXA_DCMD_BURST32;
-
-       /* FIXME: drivers should be ported over to use the filter
-        * function. Once that's done, the following two lines can
-        * be removed.
-        */
-       if (chan->cfg.slave_id)
-               chan->drcmr = chan->cfg.slave_id;
 }
 
 static struct dma_async_tx_descriptor *