On Sun, Nov 18, 2018 at 11:38 PM Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> wrote: > > 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@xxxxxxxxxxxxxxx> This looks correct to me to solve the problem I reported. Reported-by: Arnd Bergmann <arnd@xxxxxxxx> Acked-by: Arnd Bergmann <arnd@xxxxxxxx> _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel