From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932454AbdK2KYH (ORCPT ); Wed, 29 Nov 2017 05:24:07 -0500 Received: from mga03.intel.com ([134.134.136.65]:61536 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932323AbdK2KYD (ORCPT ); Wed, 29 Nov 2017 05:24:03 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.44,472,1505804400"; d="scan'208";a="181501766" Date: Wed, 29 Nov 2017 15:57:34 +0530 From: Vinod Koul To: Pierre-Louis Bossart Cc: alsa-devel@alsa-project.org, tiwai@suse.de, broonie@kernel.org, torvalds@linux-foundation.org, liam.r.girdwood@linux.intel.com, andriy.shevchenko@linux.intel.com, arnd@arndb.de, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH v2 1/7] ASoC: Intel: Fix Kconfig with top-level selector Message-ID: <20171129102734.GA32417@localhost> References: <20171129014551.30552-1-pierre-louis.bossart@linux.intel.com> <20171129014551.30552-2-pierre-louis.bossart@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171129014551.30552-2-pierre-louis.bossart@linux.intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 28, 2017 at 07:45:45PM -0600, Pierre-Louis Bossart wrote: I am not sure about top level being default to Y... > config SND_SST_ATOM_HIFI2_PLATFORM > tristate "Intel ASoC SST driver for HiFi2 platforms (*field, *trail)" > - depends on SND_SOC_INTEL_SST_TOPLEVEL && X86 > + depends on X86 > select SND_SOC_COMPRESS > + select SND_SOC_INTEL_COMMON > > config SND_SOC_INTEL_SKYLAKE > tristate "Intel ASoC SST driver for SKL/BXT/KBL/GLK/CNL" > - depends on SND_SOC_INTEL_SST_TOPLEVEL && PCI && ACPI > + depends on PCI && ACPI > select SND_HDA_EXT_CORE > select SND_HDA_DSP_LOADER > select SND_SOC_TOPOLOGY > select SND_SOC_INTEL_SST > + select SND_SOC_INTEL_COMMON > + help > + If you have a Intel Skylake/Broxton/ApolloLake/KabyLake/ > + GeminiLake or CannonLake platform with the DSP enabled in the BIOS > + then enable this option by saying Y or m. this is good stuff, helps in improving UX vastly. Btw can we have select ALL machine also as an option for people who dont want to select a specfic one, that one will really help for a better UX > + > +endif ## SND_SOC_INTEL_SST_TOPLEVEL > > # ASoC codec drivers > source "sound/soc/intel/boards/Kconfig" > + > +# configs common to SST and SOF to compile sound/soc/intel/common > +# directory and use matching tables > + > +config SND_SOC_INTEL_COMMON > + tristate > + select SND_SOC_ACPI_INTEL_MATCH if ACPI common selects only MATCH > + # this option controls the compilation of the sound/soc/intel/common > + # directory and is not meant to be selected by the user. It is > + # not filtered out on purpose by the top-level selector since > + # it will be selected by SST or SOF platform driver options > + > +config SND_SOC_ACPI_INTEL_MATCH > + tristate > + select SND_SOC_ACPI if ACPI then why keep common, lets remove one level and have SND_SOC_ACPI_INTEL_MATCH selected. ACPI is must have at top level so we can add depends on that symbol -- ~Vinod