[alsa-devel] [PATCH v7 7/7] ASoC: Intel: Skylake: add option to control HDAudio + DSP usage

Takashi Iwai tiwai at suse.de
Wed Aug 22 10:03:19 CEST 2018


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 at linux.intel.com>
> ---
>  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


More information about the Alsa-devel mailing list