ASoC: Intel: Skylake: fix Kconfigs, make HDaudio codec optional
authorPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Sun, 18 Nov 2018 22:38:09 +0000 (16:38 -0600)
committerMark Brown <broonie@kernel.org>
Tue, 20 Nov 2018 16:52:19 +0000 (16:52 +0000)
commit8c4e7c2ee8096b5ca8214418f287b3878d578cc0
tree13cc04a8cd6a3e43d38d0e32df29ad1ab2950ba7
parent5bb9b84b0796336b18c8b8651628009eace70202
ASoC: Intel: Skylake: fix Kconfigs, make HDaudio codec optional

The Skylake driver currently has a set of problems supporting
load/unload modules. We need to make the HDaudio codec support
optional to help narrow down the issues.

Support for HDaudio codecs also leads to a Kconfig issue. We want the
hdac_hda codec to be compilable independently of Skylake (e.g. with
ALL_CODECS) but when Skylake is selected as built-in the hdac_hda
codec needs to use the same option due a a code dependency

Solve both problems by adding a user-selectable boolean Kconfig,
select HDAC_HDA as needed and make the HDaudio codec support in the
Skylake driver optional. Tests on a Chell Chromebook device without
HDaudio show no regression for speaker and HDMI playback.

This is submitted as an RFC to allow for comments and more validation.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/intel/Kconfig
sound/soc/intel/boards/Kconfig
sound/soc/intel/skylake/skl.c