All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 00/11] dmaengine: kill off dma_slave_config->slave_id
@ 2021-11-15  8:53 ` Arnd Bergmann
  0 siblings, 0 replies; 176+ messages in thread
From: Arnd Bergmann @ 2021-11-15  8:53 UTC (permalink / raw)
  To: Vinod Koul
  Cc: Arnd Bergmann, Andy Gross, Andy Shevchenko, Baolin Wang,
	Bjorn Andersson, Chunyan Zhang, Greg Kroah-Hartman, Hyun Kwon,
	Jaroslav Kysela, Jon Hunter, Lars-Peter Clausen,
	Laurent Pinchart, Laxman Dewangan, Manivannan Sadhasivam,
	Mark Brown, Michal Simek, Nicolas Saenz Julienne, Orson Zhai,
	Robert Jarzmik, Scott Branden, Takashi Iwai, Thierry Reding,
	alsa-devel, bcm-kernel-feedback-list, dmaengine, dri-devel,
	linux-arm-kernel, linux-arm-msm, linux-kernel, linux-mmc,
	linux-mtd, linux-rpi-kernel, linux-serial, linux-spi,
	linux-staging, linux-tegra

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.

As the last patch in the series depends on all the others, it would
be nice have everything merged into the dmaengine tree for v5.17.

       Arnd

Arnd Bergmann (11):
  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
  staging: ralink-gdma: stop using slave_id config
  dmaengine: remove slave_id config field

 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         | 12 +++--
 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/staging/ralink-gdma/ralink-gdma.c | 12 ++---
 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/sound/dmaengine_pcm.h             |  2 -
 sound/core/pcm_dmaengine.c                |  5 +-
 sound/soc/tegra/tegra20_spdif.c           |  1 -
 19 files changed, 119 insertions(+), 68 deletions(-)
 create mode 100644 include/linux/dma/qcom_adm.h
 create mode 100644 include/linux/dma/xilinx_dpdma.h

-- 
2.30.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

^ permalink raw reply	[flat|nested] 176+ messages in thread

end of thread, other threads:[~2021-11-25  9:33 UTC | newest]

Thread overview: 176+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-15  8:53 [PATCH 00/11] dmaengine: kill off dma_slave_config->slave_id Arnd Bergmann
2021-11-15  8:53 ` Arnd Bergmann
2021-11-15  8:53 ` Arnd Bergmann
2021-11-15  8:53 ` Arnd Bergmann
2021-11-15  8:53 ` Arnd Bergmann
2021-11-15  8:53 ` [PATCH 01/11] ASoC: dai_dma: remove slave_id field Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15 10:14   ` Lars-Peter Clausen
2021-11-15 10:14     ` Lars-Peter Clausen
2021-11-15 10:14     ` Lars-Peter Clausen
2021-11-15 10:14     ` Lars-Peter Clausen
2021-11-15 10:14     ` Lars-Peter Clausen
2021-11-15 10:42     ` Arnd Bergmann
2021-11-15 10:42       ` Arnd Bergmann
2021-11-15 10:42       ` Arnd Bergmann
2021-11-15 10:42       ` Arnd Bergmann
2021-11-15 10:42       ` Arnd Bergmann
2021-11-15 11:53       ` Lars-Peter Clausen
2021-11-15 11:53         ` Lars-Peter Clausen
2021-11-15 11:53         ` Lars-Peter Clausen
2021-11-15 11:53         ` Lars-Peter Clausen
2021-11-15 11:53         ` Lars-Peter Clausen
2021-11-15 14:46         ` Dmitry Osipenko
2021-11-15 14:46           ` Dmitry Osipenko
2021-11-15 14:46           ` Dmitry Osipenko
2021-11-15 14:46           ` Dmitry Osipenko
2021-11-15 14:46           ` Dmitry Osipenko
2021-11-15 15:15           ` Arnd Bergmann
2021-11-15 15:15             ` Arnd Bergmann
2021-11-15 15:15             ` Arnd Bergmann
2021-11-15 15:15             ` Arnd Bergmann
2021-11-15 15:15             ` Arnd Bergmann
2021-11-15  8:53 ` [PATCH 02/11] spi: pic32: stop setting dma_config->slave_id Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15 13:29   ` Mark Brown
2021-11-15 13:29     ` Mark Brown
2021-11-15 13:29     ` Mark Brown
2021-11-15 13:29     ` Mark Brown
2021-11-15 13:29     ` Mark Brown
2021-11-15  8:53 ` [PATCH 03/11] mmc: bcm2835: stop setting chan_config->slave_id Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  9:55   ` nicolas saenz julienne
2021-11-15  9:55     ` nicolas saenz julienne
2021-11-15  9:55     ` nicolas saenz julienne
2021-11-15  9:55     ` nicolas saenz julienne
2021-11-15  9:55     ` nicolas saenz julienne
2021-11-15 13:09   ` Ulf Hansson
2021-11-15 13:09     ` Ulf Hansson
2021-11-15 13:09     ` Ulf Hansson
2021-11-15 13:09     ` Ulf Hansson
2021-11-15 13:09     ` Ulf Hansson
2021-11-15  8:53 ` [PATCH 04/11] dmaengine: shdma: remove legacy slave_id parsing Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  9:09   ` Laurent Pinchart
2021-11-15  9:09     ` Laurent Pinchart
2021-11-15  9:09     ` Laurent Pinchart
2021-11-15  9:09     ` Laurent Pinchart
2021-11-15  9:09     ` Laurent Pinchart
2021-11-15  8:53 ` [PATCH 05/11] dmaengine: pxa/mmp: stop referencing config->slave_id Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53 ` [PATCH 06/11] dmaengine: sprd: " Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-16  2:34   ` Baolin Wang
2021-11-16  2:34     ` Baolin Wang
2021-11-16  2:34     ` Baolin Wang
2021-11-16  2:34     ` Baolin Wang
2021-11-16  2:34     ` Baolin Wang
2021-11-15  8:53 ` [PATCH 07/11] dmaengine: qcom-adm: stop abusing slave_id config Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-15  8:53   ` Arnd Bergmann
2021-11-19 12:42   ` kernel test robot
2021-11-19 12:42     ` kernel test robot
2021-11-19 12:42     ` kernel test robot
2021-11-19 12:42     ` kernel test robot
2021-11-25  7:57   ` kernel test robot
2021-11-25  7:57     ` kernel test robot
2021-11-25  7:57     ` kernel test robot
2021-11-25  7:57     ` kernel test robot
2021-11-25  8:25     ` Arnd Bergmann
2021-11-25  8:25       ` Arnd Bergmann
2021-11-25  8:25       ` Arnd Bergmann
2021-11-25  8:25       ` Arnd Bergmann
2021-11-25  9:32       ` Vinod Koul
2021-11-25  9:32         ` Vinod Koul
2021-11-25  9:32         ` Vinod Koul
2021-11-25  9:32         ` Vinod Koul
2021-11-15  8:54 ` [PATCH 08/11] dmaengine: xilinx_dpdma: stop using slave_id field Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  9:14   ` Laurent Pinchart
2021-11-15  9:14     ` Laurent Pinchart
2021-11-15  9:14     ` Laurent Pinchart
2021-11-15  9:14     ` Laurent Pinchart
2021-11-15  9:14     ` Laurent Pinchart
2021-11-15 10:21     ` Arnd Bergmann
2021-11-15 10:21       ` Arnd Bergmann
2021-11-15 10:21       ` Arnd Bergmann
2021-11-15 10:21       ` Arnd Bergmann
2021-11-15 10:21       ` Arnd Bergmann
2021-11-15 11:49       ` Laurent Pinchart
2021-11-15 11:49         ` Laurent Pinchart
2021-11-15 11:49         ` Laurent Pinchart
2021-11-15 11:49         ` Laurent Pinchart
2021-11-15 11:49         ` Laurent Pinchart
2021-11-15 12:38         ` Arnd Bergmann
2021-11-15 12:38           ` Arnd Bergmann
2021-11-15 12:38           ` Arnd Bergmann
2021-11-15 12:38           ` Arnd Bergmann
2021-11-15 12:38           ` Arnd Bergmann
2021-11-15 13:05           ` Laurent Pinchart
2021-11-15 13:05             ` Laurent Pinchart
2021-11-15 13:05             ` Laurent Pinchart
2021-11-15 13:05             ` Laurent Pinchart
2021-11-15 13:05             ` Laurent Pinchart
2021-11-15 13:39             ` Arnd Bergmann
2021-11-15 13:39               ` Arnd Bergmann
2021-11-15 13:39               ` Arnd Bergmann
2021-11-15 13:39               ` Arnd Bergmann
2021-11-15 13:39               ` Arnd Bergmann
2021-11-16  4:28       ` Vinod Koul
2021-11-16  4:28         ` Vinod Koul
2021-11-16  4:28         ` Vinod Koul
2021-11-16  4:28         ` Vinod Koul
2021-11-16  4:28         ` Vinod Koul
2021-11-15  8:54 ` [PATCH 09/11] dmaengine: tegra20-apb: stop checking config->slave_id Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54 ` [PATCH 10/11] staging: ralink-gdma: stop using slave_id config Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  9:55   ` Sergio Paracuellos
2021-11-15  9:55     ` Sergio Paracuellos
2021-11-15  9:55     ` Sergio Paracuellos
2021-11-15  9:55     ` Sergio Paracuellos
2021-11-15  9:55     ` Sergio Paracuellos
2021-11-15 10:23     ` Arnd Bergmann
2021-11-15 10:23       ` Arnd Bergmann
2021-11-15 10:23       ` Arnd Bergmann
2021-11-15 10:23       ` Arnd Bergmann
2021-11-15 10:23       ` Arnd Bergmann
2021-11-15  8:54 ` [PATCH 11/11] dmaengine: remove slave_id config field Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  8:54   ` Arnd Bergmann
2021-11-15  9:18   ` Laurent Pinchart
2021-11-15  9:18     ` Laurent Pinchart
2021-11-15  9:18     ` Laurent Pinchart
2021-11-15  9:18     ` Laurent Pinchart
2021-11-15  9:18     ` Laurent Pinchart

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.