Commit 37df0949 authored by Hui Wang's avatar Hui Wang Committed by Takashi Iwai

Revert "ALSA: hda - drop def association and sequence from pinconf comparing"

This reverts commit c687200b.

Dropping the def association and sequence from pinconf comparing is a
bit risky, It will introduce a greater risk of catching unwanted
machines.

And in addition, so far no BIOS experts give us an explicit answer
whether it makes senses to compare these two fields or not.

For safety reason, we revert this commit.
Signed-off-by: default avatarHui Wang <hui.wang@canonical.com>
Signed-off-by: default avatarTakashi Iwai <tiwai@suse.de>
parent 39617837
...@@ -844,8 +844,7 @@ static bool pin_config_match(struct hda_codec *codec, ...@@ -844,8 +844,7 @@ static bool pin_config_match(struct hda_codec *codec,
{ {
for (; pins->nid; pins++) { for (; pins->nid; pins++) {
u32 def_conf = snd_hda_codec_get_pincfg(codec, pins->nid); u32 def_conf = snd_hda_codec_get_pincfg(codec, pins->nid);
u32 mask = 0xffffff00; if (pins->val != def_conf)
if ((pins->val & mask) != (def_conf & mask))
return false; return false;
} }
return true; return true;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment