Re: [RFC PATCH] ASoC: Intel: Skylake: fix Kconfigs, make HDaudio codec optional

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux