On Fri, 17 Aug 2018 22:37:23 +0200, Pierre-Louis Bossart wrote: > > Add option to explicitly enable HDAudio + DSP usage. > > Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> > --- > sound/soc/intel/Kconfig | 9 +++++++++ > sound/soc/intel/skylake/skl.c | 8 ++++++++ > 2 files changed, 17 insertions(+) > > diff --git a/sound/soc/intel/Kconfig b/sound/soc/intel/Kconfig > index 0caa1f4eb94d..3ee6a407eb08 100644 > --- a/sound/soc/intel/Kconfig > +++ b/sound/soc/intel/Kconfig > @@ -117,6 +117,15 @@ config SND_SOC_INTEL_SKYLAKE > GeminiLake or CannonLake platform with the DSP enabled in the BIOS > then enable this option by saying Y or m. > > +config SND_SOC_INTEL_SKYLAKE_HDA_DSP > + bool "Enable HDAudio Codec + DSP support" > + depends on SND_SOC_INTEL_SKYLAKE > + help > + If you have a Intel Skylake+ platform with the DSP enabled in > + the BIOS and an HDAudio codec, then enable this option by saying Y > + This option will only have an effect if there are no ACPI-enumerated > + audio devices (I2C, SoundWire). As mentioned in the previous patchset, this kind of control should be better provided as a module option. If this influences on the module dependency, it'd make sense to do as Kconfig. But this doesn't look so, and it's rather a sort of runtime control to match with the deployed hardware configuration. thanks, Takashi _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
