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@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