Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933371AbbKMKP0 (ORCPT ); Fri, 13 Nov 2015 05:15:26 -0500 Received: from mx2.suse.de ([195.135.220.15]:53396 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932299AbbKMKPU (ORCPT ); Fri, 13 Nov 2015 05:15:20 -0500 Date: Thu, 12 Nov 2015 21:38:17 +0100 Message-ID: From: Takashi Iwai To: "Dan Williams" Cc: , , , "Jaroslav Kysela" , "Jeff Moyer" Subject: Re: [PATCH] sound: depend on ZONE_DMA In-Reply-To: <20151112201357.11045.22705.stgit@dwillia2-desk3.jf.intel.com> References: <20151112201357.11045.22705.stgit@dwillia2-desk3.jf.intel.com> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.5 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4557 Lines: 144 On Thu, 12 Nov 2015 21:13:57 +0100, Dan Williams wrote: > > There are several sound drivers that 'select ZONE_DMA'. This is > backwards as ZONE_DMA is an architecture capability exported to drivers. > Switch the polarity of the dependency to disable these drivers when the > architecture does not support ZONE_DMA. This was discovered in the > context of testing/enabling devm_memremap_pages() which depends on > ZONE_DEVICE. ZONE_DEVICE in turn depends on !ZONE_DMA. Makes sense. I applied it now, thanks. Takashi > > Cc: Jaroslav Kysela > Cc: Takashi Iwai > Cc: > Reported-by: Jeff Moyer > Signed-off-by: Dan Williams > --- > sound/pci/Kconfig | 24 ++++++++++++------------ > 1 file changed, 12 insertions(+), 12 deletions(-) > > diff --git a/sound/pci/Kconfig b/sound/pci/Kconfig > index edfc1b8d553e..656ce39bddbc 100644 > --- a/sound/pci/Kconfig > +++ b/sound/pci/Kconfig > @@ -25,7 +25,7 @@ config SND_ALS300 > select SND_PCM > select SND_AC97_CODEC > select SND_OPL3_LIB > - select ZONE_DMA > + depends on ZONE_DMA > help > Say 'Y' or 'M' to include support for Avance Logic ALS300/ALS300+ > > @@ -50,7 +50,7 @@ config SND_ALI5451 > tristate "ALi M5451 PCI Audio Controller" > select SND_MPU401_UART > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for the integrated AC97 sound > device on motherboards using the ALi M5451 Audio Controller > @@ -155,7 +155,7 @@ config SND_AZT3328 > select SND_PCM > select SND_RAWMIDI > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for Aztech AZF3328 (PCI168) > soundcards. > @@ -463,7 +463,7 @@ config SND_EMU10K1 > select SND_HWDEP > select SND_RAWMIDI > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y to include support for Sound Blaster PCI 512, Live!, > Audigy and E-mu APS (partially supported) soundcards. > @@ -479,7 +479,7 @@ config SND_EMU10K1X > tristate "Emu10k1X (Dell OEM Version)" > select SND_AC97_CODEC > select SND_RAWMIDI > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for the Dell OEM version of the > Sound Blaster Live!. > @@ -513,7 +513,7 @@ config SND_ES1938 > select SND_OPL3_LIB > select SND_MPU401_UART > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for soundcards based on ESS Solo-1 > (ES1938, ES1946, ES1969) chips. > @@ -525,7 +525,7 @@ config SND_ES1968 > tristate "ESS ES1968/1978 (Maestro-1/2/2E)" > select SND_MPU401_UART > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for soundcards based on ESS Maestro > 1/2/2E chips. > @@ -612,7 +612,7 @@ config SND_ICE1712 > select SND_MPU401_UART > select SND_AC97_CODEC > select BITREVERSE > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for soundcards based on the > ICE1712 (Envy24) chip. > @@ -700,7 +700,7 @@ config SND_LX6464ES > config SND_MAESTRO3 > tristate "ESS Allegro/Maestro3" > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for soundcards based on ESS Maestro 3 > (Allegro) chips. > @@ -806,7 +806,7 @@ config SND_SIS7019 > tristate "SiS 7019 Audio Accelerator" > depends on X86_32 > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for the SiS 7019 Audio Accelerator. > > @@ -818,7 +818,7 @@ config SND_SONICVIBES > select SND_OPL3_LIB > select SND_MPU401_UART > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for soundcards based on the S3 > SonicVibes chip. > @@ -830,7 +830,7 @@ config SND_TRIDENT > tristate "Trident 4D-Wave DX/NX; SiS 7018" > select SND_MPU401_UART > select SND_AC97_CODEC > - select ZONE_DMA > + depends on ZONE_DMA > help > Say Y here to include support for soundcards based on Trident > 4D-Wave DX/NX or SiS 7018 chips. > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/