From: Arnd Bergmann <arnd@kernel.org>
To: Vinod Koul <vkoul@kernel.org>
Cc: Arnd Bergmann <arnd@arndb.de>, Andy Gross <agross@kernel.org>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Baolin Wang <baolin.wang7@gmail.com>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Chunyan Zhang <zhang.lyra@gmail.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Hyun Kwon <hyun.kwon@xilinx.com>,
Jaroslav Kysela <perex@perex.cz>,
Jon Hunter <jonathanh@nvidia.com>,
Lars-Peter Clausen <lars@metafoo.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Laxman Dewangan <ldewangan@nvidia.com>,
Manivannan Sadhasivam <mani@kernel.org>,
Mark Brown <broonie@kernel.org>,
Michal Simek <michal.simek@xilinx.com>,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
Orson Zhai <orsonzhai@gmail.com>,
Robert Jarzmik <robert.jarzmik@free.fr>,
Scott Branden <sbranden@broadcom.com>,
Takashi Iwai <tiwai@suse.com>,
Thierry Reding <thierry.reding@gmail.com>,
alsa-devel@alsa-project.org,
bcm-kernel-feedback-list@broadcom.com, dmaengine@vger.kernel.org,
dri-devel@lists.freedesktop.org,
linux-arm-kernel@lists.infradead.org,
linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-mmc@vger.kernel.org, linux-mtd@lists.infradead.org,
linux-rpi-kernel@lists.infradead.org,
linux-serial@vger.kernel.org, linux-spi@vger.kernel.org,
linux-staging@lists.linux.dev, linux-tegra@vger.kernel.org
Subject: [PATCH v2 00/11] dmaengine: kill off dma_slave_config->slave_id
Date: Mon, 22 Nov 2021 23:21:52 +0100 [thread overview]
Message-ID: <20211122222203.4103644-1-arnd@kernel.org> (raw)
From: Arnd Bergmann <arnd@arndb.de>
I recently came across some new uses of the 'slave_id' field that
I had (almost) removed a few years ago. There are no legitimate
uses of this field in the kernel, only a few stale references and
two drivers that abuse the field as a side-channel between the
dmaengine driver and its client.
Let's change the xilinx and qualcomm drivers to use the documented
side-channel (peripheral_data) instead, and remove the remnants of
it to prevent new users from coming in.
I think I got all the necessary Acks on v1 already, so if there are
no further concerns, please merge this through the dmaengine tree
for v5.17, or pull from
git://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git tags/dma-slave-id-removal
Arnd
Arnd Bergmann (13):
ASoC: tegra20-spdif: stop setting slave_id
ASoC: dai_dma: remove slave_id field
spi: pic32: stop setting dma_config->slave_id
mmc: bcm2835: stop setting chan_config->slave_id
dmaengine: shdma: remove legacy slave_id parsing
dmaengine: pxa/mmp: stop referencing config->slave_id
dmaengine: sprd: stop referencing config->slave_id
dmaengine: qcom-adm: stop abusing slave_id config
dmaengine: xilinx_dpdma: stop using slave_id field
dmaengine: tegra20-apb: stop checking config->slave_id
dmaengine: remove slave_id config field
---
Changes in v2:
- drop obsolete drivers/staging/patch
- split tegra change into a separate patch
- improve some patch descriptions.
drivers/dma/mmp_pdma.c | 6 ---
drivers/dma/pxa_dma.c | 7 ----
drivers/dma/qcom/qcom_adm.c | 56 ++++++++++++++++++++++----
drivers/dma/sh/shdma-base.c | 8 ----
drivers/dma/sprd-dma.c | 3 --
drivers/dma/tegra20-apb-dma.c | 6 ---
drivers/dma/xilinx/xilinx_dpdma.c | 17 +++++---
drivers/gpu/drm/xlnx/zynqmp_disp.c | 9 ++++-
drivers/mmc/host/bcm2835.c | 2 -
drivers/mtd/nand/raw/qcom_nandc.c | 14 ++++++-
drivers/spi/spi-pic32.c | 2 -
drivers/tty/serial/msm_serial.c | 15 ++++++-
include/linux/dma/qcom_adm.h | 12 ++++++
include/linux/dma/xilinx_dpdma.h | 11 +++++
include/linux/dmaengine.h | 4 --
include/linux/platform_data/asoc-s3c.h | 6 ---
include/sound/dmaengine_pcm.h | 4 --
sound/core/pcm_dmaengine.c | 26 +-----------
sound/soc/tegra/tegra20_spdif.c | 1 -
27 files changed, 134 insertions(+), 146 deletions(-)
create mode 100644 include/linux/dma/qcom_adm.h
create mode 100644 include/linux/dma/xilinx_dpdma.h
--
2.29.2
Cc: Andy Gross <agross@kernel.org>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Baolin Wang <baolin.wang7@gmail.com>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Chunyan Zhang <zhang.lyra@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Hyun Kwon <hyun.kwon@xilinx.com>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Jon Hunter <jonathanh@nvidia.com>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Cc: Manivannan Sadhasivam <mani@kernel.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Michal Simek <michal.simek@xilinx.com>
Cc: Nicolas Saenz Julienne <nsaenz@kernel.org>
Cc: Orson Zhai <orsonzhai@gmail.com>
Cc: Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: alsa-devel@alsa-project.org
Cc: bcm-kernel-feedback-list@broadcom.com
Cc: dmaengine@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-mmc@vger.kernel.org
Cc: linux-mtd@lists.infradead.org
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-serial@vger.kernel.org
Cc: linux-spi@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Cc: linux-tegra@vger.kernel.org
WARNING: multiple messages have this Message-ID
From: Arnd Bergmann <arnd@kernel.org>
To: Vinod Koul <vkoul@kernel.org>
Cc: alsa-devel@alsa-project.org, dri-devel@lists.freedesktop.org,
Bjorn Andersson <bjorn.andersson@linaro.org>,
linux-tegra@vger.kernel.org,
Thierry Reding <thierry.reding@gmail.com>,
linux-mtd@lists.infradead.org,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
linux-spi@vger.kernel.org,
Robert Jarzmik <robert.jarzmik@free.fr>,
Chunyan Zhang <zhang.lyra@gmail.com>,
linux-staging@lists.linux.dev,
Michal Simek <michal.simek@xilinx.com>,
Jon Hunter <jonathanh@nvidia.com>, Andy Gross <agross@kernel.org>,
bcm-kernel-feedback-list@broadcom.com,
linux-serial@vger.kernel.org, Orson Zhai <orsonzhai@gmail.com>,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
Arnd Bergmann <arnd@arndb.de>,
Manivannan Sadhasivam <mani@kernel.org>,
linux-arm-msm@vger.kernel.org, dmaengine@vger.kernel.org,
Mark Brown <broonie@kernel.org>,
linux-rpi-kernel@lists.infradead.org,
Jaroslav Kysela <perex@perex.cz>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
linux-arm-kernel@lists.infradead.org,
Scott Branden <sbranden@broadcom.com>,
Hyun Kwon <hyun.kwon@xilinx.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-mmc@vger.kernel.org, Takashi Iwai <tiwai@suse.com>,
linux-kernel@vger.kernel.org,
Laxman Dewangan <ldewangan@nvidia.com>,
Baolin Wang <baolin.wang7@gmail.com>
Subject: [PATCH v2 00/11] dmaengine: kill off dma_slave_config->slave_id
Date: Mon, 22 Nov 2021 23:21:52 +0100 [thread overview]
Message-ID: <20211122222203.4103644-1-arnd@kernel.org> (raw)
From: Arnd Bergmann <arnd@arndb.de>
I recently came across some new uses of the 'slave_id' field that
I had (almost) removed a few years ago. There are no legitimate
uses of this field in the kernel, only a few stale references and
two drivers that abuse the field as a side-channel between the
dmaengine driver and its client.
Let's change the xilinx and qualcomm drivers to use the documented
side-channel (peripheral_data) instead, and remove the remnants of
it to prevent new users from coming in.
I think I got all the necessary Acks on v1 already, so if there are
no further concerns, please merge this through the dmaengine tree
for v5.17, or pull from
git://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git tags/dma-slave-id-removal
Arnd
Arnd Bergmann (13):
ASoC: tegra20-spdif: stop setting slave_id
ASoC: dai_dma: remove slave_id field
spi: pic32: stop setting dma_config->slave_id
mmc: bcm2835: stop setting chan_config->slave_id
dmaengine: shdma: remove legacy slave_id parsing
dmaengine: pxa/mmp: stop referencing config->slave_id
dmaengine: sprd: stop referencing config->slave_id
dmaengine: qcom-adm: stop abusing slave_id config
dmaengine: xilinx_dpdma: stop using slave_id field
dmaengine: tegra20-apb: stop checking config->slave_id
dmaengine: remove slave_id config field
---
Changes in v2:
- drop obsolete drivers/staging/patch
- split tegra change into a separate patch
- improve some patch descriptions.
drivers/dma/mmp_pdma.c | 6 ---
drivers/dma/pxa_dma.c | 7 ----
drivers/dma/qcom/qcom_adm.c | 56 ++++++++++++++++++++++----
drivers/dma/sh/shdma-base.c | 8 ----
drivers/dma/sprd-dma.c | 3 --
drivers/dma/tegra20-apb-dma.c | 6 ---
drivers/dma/xilinx/xilinx_dpdma.c | 17 +++++---
drivers/gpu/drm/xlnx/zynqmp_disp.c | 9 ++++-
drivers/mmc/host/bcm2835.c | 2 -
drivers/mtd/nand/raw/qcom_nandc.c | 14 ++++++-
drivers/spi/spi-pic32.c | 2 -
drivers/tty/serial/msm_serial.c | 15 ++++++-
include/linux/dma/qcom_adm.h | 12 ++++++
include/linux/dma/xilinx_dpdma.h | 11 +++++
include/linux/dmaengine.h | 4 --
include/linux/platform_data/asoc-s3c.h | 6 ---
include/sound/dmaengine_pcm.h | 4 --
sound/core/pcm_dmaengine.c | 26 +-----------
sound/soc/tegra/tegra20_spdif.c | 1 -
27 files changed, 134 insertions(+), 146 deletions(-)
create mode 100644 include/linux/dma/qcom_adm.h
create mode 100644 include/linux/dma/xilinx_dpdma.h
--
2.29.2
Cc: Andy Gross <agross@kernel.org>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Baolin Wang <baolin.wang7@gmail.com>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Chunyan Zhang <zhang.lyra@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Hyun Kwon <hyun.kwon@xilinx.com>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Jon Hunter <jonathanh@nvidia.com>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Cc: Manivannan Sadhasivam <mani@kernel.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Michal Simek <michal.simek@xilinx.com>
Cc: Nicolas Saenz Julienne <nsaenz@kernel.org>
Cc: Orson Zhai <orsonzhai@gmail.com>
Cc: Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: alsa-devel@alsa-project.org
Cc: bcm-kernel-feedback-list@broadcom.com
Cc: dmaengine@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-mmc@vger.kernel.org
Cc: linux-mtd@lists.infradead.org
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-serial@vger.kernel.org
Cc: linux-spi@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Cc: linux-tegra@vger.kernel.org
WARNING: multiple messages have this Message-ID
From: Arnd Bergmann <arnd@kernel.org>
To: Vinod Koul <vkoul@kernel.org>
Cc: Arnd Bergmann <arnd@arndb.de>, Andy Gross <agross@kernel.org>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Baolin Wang <baolin.wang7@gmail.com>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Chunyan Zhang <zhang.lyra@gmail.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Hyun Kwon <hyun.kwon@xilinx.com>,
Jaroslav Kysela <perex@perex.cz>,
Jon Hunter <jonathanh@nvidia.com>,
Lars-Peter Clausen <lars@metafoo.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Laxman Dewangan <ldewangan@nvidia.com>,
Manivannan Sadhasivam <mani@kernel.org>,
Mark Brown <broonie@kernel.org>,
Michal Simek <michal.simek@xilinx.com>,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
Orson Zhai <orsonzhai@gmail.com>,
Robert Jarzmik <robert.jarzmik@free.fr>,
Scott Branden <sbranden@broadcom.com>,
Takashi Iwai <tiwai@suse.com>,
Thierry Reding <thierry.reding@gmail.com>,
alsa-devel@alsa-project.org,
bcm-kernel-feedback-list@broadcom.com, dmaengine@vger.kernel.org,
dri-devel@lists.freedesktop.org,
linux-arm-kernel@lists.infradead.org,
linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-mmc@vger.kernel.org, linux-mtd@lists.infradead.org,
linux-rpi-kernel@lists.infradead.org,
linux-serial@vger.kernel.org, linux-spi@vger.kernel.org,
linux-staging@lists.linux.dev, linux-tegra@vger.kernel.org
Subject: [PATCH v2 00/11] dmaengine: kill off dma_slave_config->slave_id
Date: Mon, 22 Nov 2021 23:21:52 +0100 [thread overview]
Message-ID: <20211122222203.4103644-1-arnd@kernel.org> (raw)
From: Arnd Bergmann <arnd@arndb.de>
I recently came across some new uses of the 'slave_id' field that
I had (almost) removed a few years ago. There are no legitimate
uses of this field in the kernel, only a few stale references and
two drivers that abuse the field as a side-channel between the
dmaengine driver and its client.
Let's change the xilinx and qualcomm drivers to use the documented
side-channel (peripheral_data) instead, and remove the remnants of
it to prevent new users from coming in.
I think I got all the necessary Acks on v1 already, so if there are
no further concerns, please merge this through the dmaengine tree
for v5.17, or pull from
git://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git tags/dma-slave-id-removal
Arnd
Arnd Bergmann (13):
ASoC: tegra20-spdif: stop setting slave_id
ASoC: dai_dma: remove slave_id field
spi: pic32: stop setting dma_config->slave_id
mmc: bcm2835: stop setting chan_config->slave_id
dmaengine: shdma: remove legacy slave_id parsing
dmaengine: pxa/mmp: stop referencing config->slave_id
dmaengine: sprd: stop referencing config->slave_id
dmaengine: qcom-adm: stop abusing slave_id config
dmaengine: xilinx_dpdma: stop using slave_id field
dmaengine: tegra20-apb: stop checking config->slave_id
dmaengine: remove slave_id config field
---
Changes in v2:
- drop obsolete drivers/staging/patch
- split tegra change into a separate patch
- improve some patch descriptions.
drivers/dma/mmp_pdma.c | 6 ---
drivers/dma/pxa_dma.c | 7 ----
drivers/dma/qcom/qcom_adm.c | 56 ++++++++++++++++++++++----
drivers/dma/sh/shdma-base.c | 8 ----
drivers/dma/sprd-dma.c | 3 --
drivers/dma/tegra20-apb-dma.c | 6 ---
drivers/dma/xilinx/xilinx_dpdma.c | 17 +++++---
drivers/gpu/drm/xlnx/zynqmp_disp.c | 9 ++++-
drivers/mmc/host/bcm2835.c | 2 -
drivers/mtd/nand/raw/qcom_nandc.c | 14 ++++++-
drivers/spi/spi-pic32.c | 2 -
drivers/tty/serial/msm_serial.c | 15 ++++++-
include/linux/dma/qcom_adm.h | 12 ++++++
include/linux/dma/xilinx_dpdma.h | 11 +++++
include/linux/dmaengine.h | 4 --
include/linux/platform_data/asoc-s3c.h | 6 ---
include/sound/dmaengine_pcm.h | 4 --
sound/core/pcm_dmaengine.c | 26 +-----------
sound/soc/tegra/tegra20_spdif.c | 1 -
27 files changed, 134 insertions(+), 146 deletions(-)
create mode 100644 include/linux/dma/qcom_adm.h
create mode 100644 include/linux/dma/xilinx_dpdma.h
--
2.29.2
Cc: Andy Gross <agross@kernel.org>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Baolin Wang <baolin.wang7@gmail.com>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Chunyan Zhang <zhang.lyra@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Hyun Kwon <hyun.kwon@xilinx.com>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Jon Hunter <jonathanh@nvidia.com>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Cc: Manivannan Sadhasivam <mani@kernel.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Michal Simek <michal.simek@xilinx.com>
Cc: Nicolas Saenz Julienne <nsaenz@kernel.org>
Cc: Orson Zhai <orsonzhai@gmail.com>
Cc: Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: alsa-devel@alsa-project.org
Cc: bcm-kernel-feedback-list@broadcom.com
Cc: dmaengine@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-mmc@vger.kernel.org
Cc: linux-mtd@lists.infradead.org
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-serial@vger.kernel.org
Cc: linux-spi@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Cc: linux-tegra@vger.kernel.org
______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/
WARNING: multiple messages have this Message-ID
From: Arnd Bergmann <arnd@kernel.org>
To: Vinod Koul <vkoul@kernel.org>
Cc: Arnd Bergmann <arnd@arndb.de>, Andy Gross <agross@kernel.org>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Baolin Wang <baolin.wang7@gmail.com>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Chunyan Zhang <zhang.lyra@gmail.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Hyun Kwon <hyun.kwon@xilinx.com>,
Jaroslav Kysela <perex@perex.cz>,
Jon Hunter <jonathanh@nvidia.com>,
Lars-Peter Clausen <lars@metafoo.de>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Laxman Dewangan <ldewangan@nvidia.com>,
Manivannan Sadhasivam <mani@kernel.org>,
Mark Brown <broonie@kernel.org>,
Michal Simek <michal.simek@xilinx.com>,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
Orson Zhai <orsonzhai@gmail.com>,
Robert Jarzmik <robert.jarzmik@free.fr>,
Scott Branden <sbranden@broadcom.com>,
Takashi Iwai <tiwai@suse.com>,
Thierry Reding <thierry.reding@gmail.com>,
alsa-devel@alsa-project.org,
bcm-kernel-feedback-list@broadcom.com, dmaengine@vger.kernel.org,
dri-devel@lists.freedesktop.org,
linux-arm-kernel@lists.infradead.org,
linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-mmc@vger.kernel.org, linux-mtd@lists.infradead.org,
linux-rpi-kernel@lists.infradead.org,
linux-serial@vger.kernel.org, linux-spi@vger.kernel.org,
linux-staging@lists.linux.dev, linux-tegra@vger.kernel.org
Subject: [PATCH v2 00/11] dmaengine: kill off dma_slave_config->slave_id
Date: Mon, 22 Nov 2021 23:21:52 +0100 [thread overview]
Message-ID: <20211122222203.4103644-1-arnd@kernel.org> (raw)
From: Arnd Bergmann <arnd@arndb.de>
I recently came across some new uses of the 'slave_id' field that
I had (almost) removed a few years ago. There are no legitimate
uses of this field in the kernel, only a few stale references and
two drivers that abuse the field as a side-channel between the
dmaengine driver and its client.
Let's change the xilinx and qualcomm drivers to use the documented
side-channel (peripheral_data) instead, and remove the remnants of
it to prevent new users from coming in.
I think I got all the necessary Acks on v1 already, so if there are
no further concerns, please merge this through the dmaengine tree
for v5.17, or pull from
git://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git tags/dma-slave-id-removal
Arnd
Arnd Bergmann (13):
ASoC: tegra20-spdif: stop setting slave_id
ASoC: dai_dma: remove slave_id field
spi: pic32: stop setting dma_config->slave_id
mmc: bcm2835: stop setting chan_config->slave_id
dmaengine: shdma: remove legacy slave_id parsing
dmaengine: pxa/mmp: stop referencing config->slave_id
dmaengine: sprd: stop referencing config->slave_id
dmaengine: qcom-adm: stop abusing slave_id config
dmaengine: xilinx_dpdma: stop using slave_id field
dmaengine: tegra20-apb: stop checking config->slave_id
dmaengine: remove slave_id config field
---
Changes in v2:
- drop obsolete drivers/staging/patch
- split tegra change into a separate patch
- improve some patch descriptions.
drivers/dma/mmp_pdma.c | 6 ---
drivers/dma/pxa_dma.c | 7 ----
drivers/dma/qcom/qcom_adm.c | 56 ++++++++++++++++++++++----
drivers/dma/sh/shdma-base.c | 8 ----
drivers/dma/sprd-dma.c | 3 --
drivers/dma/tegra20-apb-dma.c | 6 ---
drivers/dma/xilinx/xilinx_dpdma.c | 17 +++++---
drivers/gpu/drm/xlnx/zynqmp_disp.c | 9 ++++-
drivers/mmc/host/bcm2835.c | 2 -
drivers/mtd/nand/raw/qcom_nandc.c | 14 ++++++-
drivers/spi/spi-pic32.c | 2 -
drivers/tty/serial/msm_serial.c | 15 ++++++-
include/linux/dma/qcom_adm.h | 12 ++++++
include/linux/dma/xilinx_dpdma.h | 11 +++++
include/linux/dmaengine.h | 4 --
include/linux/platform_data/asoc-s3c.h | 6 ---
include/sound/dmaengine_pcm.h | 4 --
sound/core/pcm_dmaengine.c | 26 +-----------
sound/soc/tegra/tegra20_spdif.c | 1 -
27 files changed, 134 insertions(+), 146 deletions(-)
create mode 100644 include/linux/dma/qcom_adm.h
create mode 100644 include/linux/dma/xilinx_dpdma.h
--
2.29.2
Cc: Andy Gross <agross@kernel.org>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Baolin Wang <baolin.wang7@gmail.com>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Chunyan Zhang <zhang.lyra@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Hyun Kwon <hyun.kwon@xilinx.com>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Jon Hunter <jonathanh@nvidia.com>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Cc: Manivannan Sadhasivam <mani@kernel.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Michal Simek <michal.simek@xilinx.com>
Cc: Nicolas Saenz Julienne <nsaenz@kernel.org>
Cc: Orson Zhai <orsonzhai@gmail.com>
Cc: Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: alsa-devel@alsa-project.org
Cc: bcm-kernel-feedback-list@broadcom.com
Cc: dmaengine@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-mmc@vger.kernel.org
Cc: linux-mtd@lists.infradead.org
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-serial@vger.kernel.org
Cc: linux-spi@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Cc: linux-tegra@vger.kernel.org
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID
From: Arnd Bergmann <arnd@kernel.org>
To: Vinod Koul <vkoul@kernel.org>
Cc: alsa-devel@alsa-project.org, dri-devel@lists.freedesktop.org,
Bjorn Andersson <bjorn.andersson@linaro.org>,
linux-tegra@vger.kernel.org,
Thierry Reding <thierry.reding@gmail.com>,
linux-mtd@lists.infradead.org,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
linux-spi@vger.kernel.org,
Robert Jarzmik <robert.jarzmik@free.fr>,
Lars-Peter Clausen <lars@metafoo.de>,
Chunyan Zhang <zhang.lyra@gmail.com>,
linux-staging@lists.linux.dev,
Michal Simek <michal.simek@xilinx.com>,
Jon Hunter <jonathanh@nvidia.com>, Andy Gross <agross@kernel.org>,
bcm-kernel-feedback-list@broadcom.com,
linux-serial@vger.kernel.org, Orson Zhai <orsonzhai@gmail.com>,
Nicolas Saenz Julienne <nsaenz@kernel.org>,
Arnd Bergmann <arnd@arndb.de>,
Manivannan Sadhasivam <mani@kernel.org>,
linux-arm-msm@vger.kernel.org, dmaengine@vger.kernel.org,
Mark Brown <broonie@kernel.org>,
linux-rpi-kernel@lists.infradead.org,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
linux-arm-kernel@lists.infradead.org,
Scott Branden <sbranden@broadcom.com>,
Hyun Kwon <hyun.kwon@xilinx.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-mmc@vger.kernel.org, Takashi Iwai <tiwai@suse.com>,
linux-kernel@vger.kernel.org,
Laxman Dewangan <ldewangan@nvidia.com>,
Baolin Wang <baolin.wang7@gmail.com>
Subject: [PATCH v2 00/11] dmaengine: kill off dma_slave_config->slave_id
Date: Mon, 22 Nov 2021 23:21:52 +0100 [thread overview]
Message-ID: <20211122222203.4103644-1-arnd@kernel.org> (raw)
From: Arnd Bergmann <arnd@arndb.de>
I recently came across some new uses of the 'slave_id' field that
I had (almost) removed a few years ago. There are no legitimate
uses of this field in the kernel, only a few stale references and
two drivers that abuse the field as a side-channel between the
dmaengine driver and its client.
Let's change the xilinx and qualcomm drivers to use the documented
side-channel (peripheral_data) instead, and remove the remnants of
it to prevent new users from coming in.
I think I got all the necessary Acks on v1 already, so if there are
no further concerns, please merge this through the dmaengine tree
for v5.17, or pull from
git://git.kernel.org/pub/scm/linux/kernel/git/arnd/asm-generic.git tags/dma-slave-id-removal
Arnd
Arnd Bergmann (13):
ASoC: tegra20-spdif: stop setting slave_id
ASoC: dai_dma: remove slave_id field
spi: pic32: stop setting dma_config->slave_id
mmc: bcm2835: stop setting chan_config->slave_id
dmaengine: shdma: remove legacy slave_id parsing
dmaengine: pxa/mmp: stop referencing config->slave_id
dmaengine: sprd: stop referencing config->slave_id
dmaengine: qcom-adm: stop abusing slave_id config
dmaengine: xilinx_dpdma: stop using slave_id field
dmaengine: tegra20-apb: stop checking config->slave_id
dmaengine: remove slave_id config field
---
Changes in v2:
- drop obsolete drivers/staging/patch
- split tegra change into a separate patch
- improve some patch descriptions.
drivers/dma/mmp_pdma.c | 6 ---
drivers/dma/pxa_dma.c | 7 ----
drivers/dma/qcom/qcom_adm.c | 56 ++++++++++++++++++++++----
drivers/dma/sh/shdma-base.c | 8 ----
drivers/dma/sprd-dma.c | 3 --
drivers/dma/tegra20-apb-dma.c | 6 ---
drivers/dma/xilinx/xilinx_dpdma.c | 17 +++++---
drivers/gpu/drm/xlnx/zynqmp_disp.c | 9 ++++-
drivers/mmc/host/bcm2835.c | 2 -
drivers/mtd/nand/raw/qcom_nandc.c | 14 ++++++-
drivers/spi/spi-pic32.c | 2 -
drivers/tty/serial/msm_serial.c | 15 ++++++-
include/linux/dma/qcom_adm.h | 12 ++++++
include/linux/dma/xilinx_dpdma.h | 11 +++++
include/linux/dmaengine.h | 4 --
include/linux/platform_data/asoc-s3c.h | 6 ---
include/sound/dmaengine_pcm.h | 4 --
sound/core/pcm_dmaengine.c | 26 +-----------
sound/soc/tegra/tegra20_spdif.c | 1 -
27 files changed, 134 insertions(+), 146 deletions(-)
create mode 100644 include/linux/dma/qcom_adm.h
create mode 100644 include/linux/dma/xilinx_dpdma.h
--
2.29.2
Cc: Andy Gross <agross@kernel.org>
Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Baolin Wang <baolin.wang7@gmail.com>
Cc: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Chunyan Zhang <zhang.lyra@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Hyun Kwon <hyun.kwon@xilinx.com>
Cc: Jaroslav Kysela <perex@perex.cz>
Cc: Jon Hunter <jonathanh@nvidia.com>
Cc: Lars-Peter Clausen <lars@metafoo.de>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Cc: Laxman Dewangan <ldewangan@nvidia.com>
Cc: Manivannan Sadhasivam <mani@kernel.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Michal Simek <michal.simek@xilinx.com>
Cc: Nicolas Saenz Julienne <nsaenz@kernel.org>
Cc: Orson Zhai <orsonzhai@gmail.com>
Cc: Robert Jarzmik <robert.jarzmik@free.fr>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Takashi Iwai <tiwai@suse.com>
Cc: Thierry Reding <thierry.reding@gmail.com>
Cc: Vinod Koul <vkoul@kernel.org>
Cc: alsa-devel@alsa-project.org
Cc: bcm-kernel-feedback-list@broadcom.com
Cc: dmaengine@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-arm-msm@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: linux-mmc@vger.kernel.org
Cc: linux-mtd@lists.infradead.org
Cc: linux-rpi-kernel@lists.infradead.org
Cc: linux-serial@vger.kernel.org
Cc: linux-spi@vger.kernel.org
Cc: linux-staging@lists.linux.dev
Cc: linux-tegra@vger.kernel.org
next reply other threads:[~2021-11-22 22:22 UTC|newest]
Thread overview: 90+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-22 22:21 Arnd Bergmann [this message]
2021-11-22 22:21 ` [PATCH v2 00/11] dmaengine: kill off dma_slave_config->slave_id Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` [PATCH v2 01/11] ASoC: tegra20-spdif: stop setting slave_id Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-24 16:32 ` Dmitry Osipenko
2021-11-24 16:32 ` Dmitry Osipenko
2021-11-24 16:32 ` Dmitry Osipenko
2021-11-24 16:32 ` Dmitry Osipenko
2021-11-24 16:47 ` Arnd Bergmann
2021-11-24 16:47 ` Arnd Bergmann
2021-11-24 16:47 ` Arnd Bergmann
2021-11-24 16:47 ` Arnd Bergmann
2021-11-24 17:36 ` Dmitry Osipenko
2021-11-24 17:36 ` Dmitry Osipenko
2021-11-24 17:36 ` Dmitry Osipenko
2021-11-24 17:36 ` Dmitry Osipenko
2021-11-24 17:39 ` Dmitry Osipenko
2021-11-24 17:39 ` Dmitry Osipenko
2021-11-24 17:39 ` Dmitry Osipenko
2021-11-24 17:39 ` Dmitry Osipenko
2021-11-22 22:21 ` [PATCH v2 02/11] ASoC: dai_dma: remove slave_id field Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` [PATCH v2 03/11] spi: pic32: stop setting dma_config->slave_id Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` [PATCH v2 04/11] mmc: bcm2835: stop setting chan_config->slave_id Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-23 11:18 ` Ulf Hansson
2021-11-23 11:18 ` Ulf Hansson
2021-11-23 11:18 ` Ulf Hansson
2021-11-23 11:18 ` Ulf Hansson
2021-11-23 11:18 ` Ulf Hansson
2021-11-22 22:21 ` [PATCH v2 05/11] dmaengine: shdma: remove legacy slave_id parsing Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` [PATCH v2 06/11] dmaengine: pxa/mmp: stop referencing config->slave_id Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` [PATCH v2 07/11] dmaengine: sprd: " Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:21 ` Arnd Bergmann
2021-11-22 22:22 ` [PATCH v2 08/11] dmaengine: qcom-adm: stop abusing slave_id config Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` [PATCH v2 09/11] dmaengine: xilinx_dpdma: stop using slave_id field Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` [PATCH v2 10/11] dmaengine: tegra20-apb: stop checking config->slave_id Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` [PATCH v2 11/11] dmaengine: remove slave_id config field Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 22:22 ` Arnd Bergmann
2021-11-22 23:01 ` [PATCH v2 00/11] dmaengine: kill off dma_slave_config->slave_id Mark Brown
2021-11-22 23:01 ` Mark Brown
2021-11-22 23:01 ` Mark Brown
2021-11-22 23:01 ` Mark Brown
2021-11-22 23:01 ` Mark Brown
2021-12-13 4:59 ` Vinod Koul
2021-12-13 4:59 ` Vinod Koul
2021-12-13 4:59 ` Vinod Koul
2021-12-13 4:59 ` Vinod Koul
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211122222203.4103644-1-arnd@kernel.org \
--to=arnd@kernel.org \
--cc=agross@kernel.org \
--cc=alsa-devel@alsa-project.org \
--cc=andriy.shevchenko@linux.intel.com \
--cc=arnd@arndb.de \
--cc=baolin.wang7@gmail.com \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=bjorn.andersson@linaro.org \
--cc=broonie@kernel.org \
--cc=dmaengine@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=hyun.kwon@xilinx.com \
--cc=jonathanh@nvidia.com \
--cc=lars@metafoo.de \
--cc=laurent.pinchart@ideasonboard.com \
--cc=ldewangan@nvidia.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=linux-rpi-kernel@lists.infradead.org \
--cc=linux-serial@vger.kernel.org \
--cc=linux-spi@vger.kernel.org \
--cc=linux-staging@lists.linux.dev \
--cc=linux-tegra@vger.kernel.org \
--cc=mani@kernel.org \
--cc=michal.simek@xilinx.com \
--cc=nsaenz@kernel.org \
--cc=orsonzhai@gmail.com \
--cc=perex@perex.cz \
--cc=robert.jarzmik@free.fr \
--cc=sbranden@broadcom.com \
--cc=thierry.reding@gmail.com \
--cc=tiwai@suse.com \
--cc=vkoul@kernel.org \
--cc=zhang.lyra@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.