Re: [PATCH 4.19 07/86] ALSA: hda: do not override bus codec_mask in link_get()

From: Pavel Machek
Date: Tue Mar 10 2020 - 18:33:37 EST


Hi!

> From: Kai Vehmanen <kai.vehmanen@xxxxxxxxxxxxxxx>
>
> [ Upstream commit 43bcb1c0507858cdc95e425017dcc33f8105df39 ]
>
> snd_hdac_ext_bus_link_get() does not work correctly in case
> there are multiple codecs on the bus. It unconditionally
> resets the bus->codec_mask value. As per documentation in
> hdaudio.h and existing use in client code, this field should
> be used to store bit flag of detected codecs on the bus.
>
> By overwriting value of the codec_mask, information on all
> detected codecs is lost. No current user of hdac is impacted,
> but use of bus->codec_mask is planned in future patches
> for SOF.

Given that no users are impacted, is this stable material?

Best regards,
Pavel

--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

Attachment: signature.asc
Description: PGP signature