Comment 456 for bug 1958019

Nope.

I'm going to have to generate a patch from this and submit it to
alsa-devel. Since it seems the previous patch is in, this will just be a
one line fix.

I might have time this weekend.

On 10/6/21 12:27 PM, <email address hidden> wrote:
> https://bugzilla.kernel.org/show_bug.cgi?id=208555
>
> --- Comment #442 from Vincent Morel (<email address hidden>) ---
> (In reply to Takashi Iwai from comment #441)
>> Both must be identical from the code perspective.
>>
>> But, looking at the verb sequence there, I doubt whether the 0xb020 call is
>> really correct. If it follows the similar pattern as others, it should be
>> rather setting the COEF index 0x26 instead, i.e.:
>>
>> --- a/sound/pci/hda/patch_realtek.c
>> +++ b/sound/pci/hda/patch_realtek.c
>> @@ -8345,7 +8345,7 @@ static const struct hda_fixup alc269_fixups[] = {
>> .v.verbs = (const struct hda_verb[]) {
>> { 0x20, AC_VERB_SET_COEF_INDEX, 0x24 },
>> { 0x20, AC_VERB_SET_PROC_COEF, 0x41 },
>> - { 0x20, AC_VERB_SET_PROC_COEF, 0xb020 },
>> + { 0x20, AC_VERB_SET_COEF_INDEX, 0x26 },
>> { 0x20, AC_VERB_SET_PROC_COEF, 0x2 },
>> { 0x20, AC_VERB_SET_PROC_COEF, 0x0 },
>> { 0x20, AC_VERB_SET_PROC_COEF, 0x0 },
> That make sense! I translated this into simple verb to test on a config with
> no
> sound and it works.
> Just compiled a new kernel with the update and it works also!
>
> Thanks Takashi Iwai! Is this update will be be pushed to the next kernel?
>