Re: [PATCH] ALSA: hda - Revert DSP detection on legacy HD-audio driver

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

 



On Mon, 31 Dec 2018 19:17:20 +0100,
Takashi Iwai wrote:
> 
> This essentially reverts the commits
>   c337104b1a16 ("ALSA: HD-Audio: SKL+: abort probe if DSP is present
>   and Skylake driver selected")
> and
>   d82b51c855a2 ("ALSA: HD-Audio: SKL+: force HDaudio legacy or SKL+
>   driver selection")
> for the path of legacy HD-audio controller (snd-hda-intel).
> 
> The automatic DSP detection and skip of binding with the legacy driver
> caused regressions on several machines like Dell XPS13.  They give the
> PCI class 0x40380 indicating the availability of DSP while they don't
> work with ASoC SKL driver (yet).
> 
> As the support of ASoC driver for such devices isn't available, it's
> better to revert the whole DSP-detection-and-skip behavior of the
> legacy driver, so that we can get the old good driver working on such
> devices.
> 
> The pci_binding option for ASoC SKL driver is still kept so that it
> can work without blacklisting.
> 
> Fixes: c337104b1a16 ("ALSA: HD-Audio: SKL+: abort probe if DSP is present and Skylake driver selected")
> Reported-by: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
> Reported-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> Reported-by: Azat Khuzhin <dohardgopro@xxxxxxxxx>
> Cc: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>
> Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>

I seem to have forgotten to exclude the global header change, which
broke the build in sound/soc/intel/skylake.  Will submit a v2 patch.

Sorry for the trouble.


Takashi
_______________________________________________
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