RE: [PATCH v2 10/21] lightnvm: Remove depends on HAS_DMA in case of platform dependency
From: Madalin-cristian Bucur <madalin.bucur@nxp.com>
Date: 2018-03-19 05:27:16
Also in:
alsa-devel, linux-block, linux-crypto, linux-fpga, linux-i2c, linux-ide, linux-iio, linux-iommu, linux-media, linux-mmc, linux-remoteproc, linux-scsi, linux-serial, linux-spi, linux-usb, lkml, netdev
-----Original Message-----
From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org]
On Behalf Of Geert Uytterhoeven
Sent: Friday, March 16, 2018 3:52 PM
To: Christoph Hellwig <hch@lst.de>; Marek Szyprowski
[off-list ref]; Robin Murphy [off-list ref];
Felipe Balbi [off-list ref]; Greg Kroah-Hartman
[off-list ref]; James E . J . Bottomley
[off-list ref]; Martin K . Petersen
[off-list ref]; Andrew Morton <akpm@linux-
foundation.org>; Mark Brown [off-list ref]; Liam Girdwood
[off-list ref]; Tejun Heo [off-list ref]; Herbert Xu
[off-list ref]; David S . Miller [off-list ref];
Bartlomiej Zolnierkiewicz [off-list ref]; Stefan Richter
[off-list ref]; Alan Tull [off-list ref]; Moritz Fischer
[off-list ref]; Wolfram Sang [off-list ref]; Jonathan Cameron
[off-list ref]; Joerg Roedel [off-list ref]; Matias Bjorling
[off-list ref]; Jassi Brar [off-list ref]; Mauro Carvalho
Chehab [off-list ref]; Ulf Hansson [off-list ref]; David
Woodhouse [off-list ref]; Brian Norris
[off-list ref]; Marek Vasut [off-list ref];
Cyrille Pitchen [off-list ref]; Boris Brezillon
[off-list ref]; Richard Weinberger [off-list ref];
Kalle Valo [off-list ref]; Ohad Ben-Cohen [off-list ref];
Bjorn Andersson [off-list ref]; Eric Anholt [off-list ref];
Stefan Wahren [off-list ref]
Cc: iommu@lists.linux-foundation.org; linux-usb@vger.kernel.org; linux-
scsi@vger.kernel.org; alsa-devel@alsa-project.org; linux-ide@vger.kernel.org;
linux-crypto@vger.kernel.org; linux-fbdev@vger.kernel.org; linux1394-
devel@lists.sourceforge.net; linux-fpga@vger.kernel.org; linux-
i2c@vger.kernel.org; linux-iio@vger.kernel.org; linux-block@vger.kernel.org;
linux-media@vger.kernel.org; linux-mmc@vger.kernel.org; linux-
mtd@lists.infradead.org; netdev@vger.kernel.org; linux-
remoteproc@vger.kernel.org; linux-serial@vger.kernel.org; linux-
spi@vger.kernel.org; devel@driverdev.osuosl.org; linux-
kernel@vger.kernel.org; Geert Uytterhoeven [off-list ref]
Subject: [PATCH v2 10/21] lightnvm: Remove depends on HAS_DMA in case of
platform dependency
Remove dependencies on HAS_DMA where a Kconfig symbol depends on
another
symbol that implies HAS_DMA, and, optionally, on "|| COMPILE_TEST".
In most cases this other symbol is an architecture or platform specific
symbol, or PCI.
Generic symbols and drivers without platform dependencies keep their
dependencies on HAS_DMA, to prevent compiling subsystems or drivers that
cannot work anyway.
This simplifies the dependencies, and allows to improve compile-testing.
Notes:
- FSL_FMAN keeps its dependency on HAS_DMA, as it calls set_dma_ops(),
which does not exist if HAS_DMA=n (Do we need a dummy? The use of
set_dma_ops() in this driver is questionable),Hi, The set_dma_ops() is no longer required in the fsl/fman, I'll send a patch to remove it. Thanks
quoted hunk ↗ jump to hunk
- SND_SOC_LPASS_IPQ806X and SND_SOC_LPASS_PLATFORM loose their dependency on HAS_DMA, as they are selected from SND_SOC_APQ8016_SBC. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Reviewed-by: Mark Brown <broonie@kernel.org> Acked-by: Robin Murphy <robin.murphy@arm.com> --- v2: - Add Reviewed-by, Acked-by, - Drop RFC state, - Split per subsystem. --- drivers/lightnvm/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/drivers/lightnvm/Kconfig b/drivers/lightnvm/Kconfig index 10c08982185a572f..9c03f35d9df113c6 100644 --- a/drivers/lightnvm/Kconfig +++ b/drivers/lightnvm/Kconfig@@ -4,7 +4,7 @@ menuconfig NVM bool "Open-Channel SSD target support" - depends on BLOCK && HAS_DMA && PCI + depends on BLOCK && PCI select BLK_DEV_NVME help Say Y here to get to enable Open-channel SSDs. --2.7.4