From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.codeaurora.org ([198.145.29.96]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1f9Ccv-0005Cq-0V for linux-mtd@lists.infradead.org; Thu, 19 Apr 2018 16:42:04 +0000 From: Kalle Valo To: Geert Uytterhoeven Cc: Christoph Hellwig , Marek Szyprowski , Robin Murphy , Felipe Balbi , Greg Kroah-Hartman , Andrew Morton , Mark Brown , Liam Girdwood , Tejun Heo , Herbert Xu , "David S . Miller" , Bartlomiej Zolnierkiewicz , Stefan Richter , Alan Tull , Moritz Fischer , Wolfram Sang , Jonathan Cameron , Joerg Roedel , Matias Bjorling , Jassi Brar , Mauro Carvalho Chehab , Ulf Hansson , David Woodhouse , Brian Norris , Marek Vasut , Cyrille Pi tchen , Boris Brezillon , Richard Weinberger , Ohad Ben-Cohen , Bjorn Andersson , Eric Anholt , Stefan Wahren , iommu@lists.linux-foundation.org, linux-usb@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, linux-wireless@vger.kernel.org Subject: Re: [PATCH v3 15/20] net: Remove depends on HAS_DMA in case of platform dependency In-Reply-To: <1523987360-18760-16-git-send-email-geert@linux-m68k.org> (Geert Uytterhoeven's message of "Tue, 17 Apr 2018 19:49:15 +0200") References: <1523987360-18760-1-git-send-email-geert@linux-m68k.org> <1523987360-18760-16-git-send-email-geert@linux-m68k.org> Date: Thu, 19 Apr 2018 19:41:15 +0300 Message-ID: <87wox3i1xw.fsf@kamboji.qca.qualcomm.com> MIME-Version: 1.0 Content-Type: text/plain List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , (adding linux-wireless) Geert Uytterhoeven writes: > 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. > > Signed-off-by: Geert Uytterhoeven > Reviewed-by: Mark Brown > Acked-by: Robin Murphy > --- > v3: > - Rebase to v4.17-rc1, > - Drop obsolete note about FSL_FMAN, > > v2: > - Add Reviewed-by, Acked-by, > - Drop RFC state, > - Split per subsystem. > --- > drivers/net/ethernet/amd/Kconfig | 2 +- > drivers/net/ethernet/apm/xgene-v2/Kconfig | 1 - > drivers/net/ethernet/apm/xgene/Kconfig | 1 - > drivers/net/ethernet/arc/Kconfig | 6 ++++-- > drivers/net/ethernet/broadcom/Kconfig | 2 -- > drivers/net/ethernet/calxeda/Kconfig | 2 +- > drivers/net/ethernet/hisilicon/Kconfig | 2 +- > drivers/net/ethernet/marvell/Kconfig | 8 +++----- > drivers/net/ethernet/mellanox/mlxsw/Kconfig | 2 +- > drivers/net/ethernet/renesas/Kconfig | 2 -- > drivers/net/wireless/broadcom/brcm80211/Kconfig | 1 - > drivers/net/wireless/quantenna/qtnfmac/Kconfig | 2 +- > 12 files changed, 12 insertions(+), 19 deletions(-) For wireless: Acked-by: Kalle Valo Leaving the hunks for linux-wireless list to see: > diff --git a/drivers/net/wireless/broadcom/brcm80211/Kconfig b/drivers/net/wireless/broadcom/brcm80211/Kconfig > index 9d99eb42d9176f0f..6acba67bca07abd7 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/Kconfig > +++ b/drivers/net/wireless/broadcom/brcm80211/Kconfig > @@ -60,7 +60,6 @@ config BRCMFMAC_PCIE > bool "PCIE bus interface support for FullMAC driver" > depends on BRCMFMAC > depends on PCI > - depends on HAS_DMA > select BRCMFMAC_PROTO_MSGBUF > select FW_LOADER > ---help--- > diff --git a/drivers/net/wireless/quantenna/qtnfmac/Kconfig b/drivers/net/wireless/quantenna/qtnfmac/Kconfig > index 025fa6018550895a..8d1492a90bd135c0 100644 > --- a/drivers/net/wireless/quantenna/qtnfmac/Kconfig > +++ b/drivers/net/wireless/quantenna/qtnfmac/Kconfig > @@ -7,7 +7,7 @@ config QTNFMAC > config QTNFMAC_PEARL_PCIE > tristate "Quantenna QSR10g PCIe support" > default n > - depends on HAS_DMA && PCI && CFG80211 > + depends on PCI && CFG80211 > select QTNFMAC > select FW_LOADER > select CRC32 -- Kalle Valo