Skip to content

Commit db2eee6

Browse files
committed
ALSA: hda: Show the codec quirk info at probing
Lots of HD-audio devices need the device-specific quirk, and it's often helpful to know which quirk is applied. But currently the driver shows it only as a debug output, hence we'd have to enable the debug option at each time we want to see it (and the output becomes too messy due to other debug messages). This patch changes the messages to the info level, so that they appear at probing normally. Link: https://patch.msgid.link/20241126141010.12567-1-tiwai@suse.de Signed-off-by: Takashi Iwai <tiwai@suse.de>
1 parent a166f80 commit db2eee6

File tree

1 file changed

+11
-9
lines changed

1 file changed

+11
-9
lines changed

sound/pci/hda/hda_auto_parser.c

Lines changed: 11 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -933,6 +933,7 @@ void snd_hda_pick_pin_fixup(struct hda_codec *codec,
933933
bool match_all_pins)
934934
{
935935
const struct snd_hda_pin_quirk *pq;
936+
const char *name = NULL;
936937

937938
if (codec->fixup_id != HDA_FIXUP_ID_NOT_SET)
938939
return;
@@ -946,9 +947,10 @@ void snd_hda_pick_pin_fixup(struct hda_codec *codec,
946947
codec->fixup_id = pq->value;
947948
#ifdef CONFIG_SND_DEBUG_VERBOSE
948949
codec->fixup_name = pq->name;
949-
codec_dbg(codec, "%s: picked fixup %s (pin match)\n",
950-
codec->core.chip_name, codec->fixup_name);
950+
name = pq->name;
951951
#endif
952+
codec_info(codec, "%s: picked fixup %s (pin match)\n",
953+
codec->core.chip_name, name ? name : "");
952954
codec->fixup_list = fixlist;
953955
return;
954956
}
@@ -1015,8 +1017,8 @@ void snd_hda_pick_fixup(struct hda_codec *codec,
10151017
if (codec->modelname && !strcmp(codec->modelname, "nofixup")) {
10161018
id = HDA_FIXUP_ID_NO_FIXUP;
10171019
fixlist = NULL;
1018-
codec_dbg(codec, "%s: picked no fixup (nofixup specified)\n",
1019-
codec->core.chip_name);
1020+
codec_info(codec, "%s: picked no fixup (nofixup specified)\n",
1021+
codec->core.chip_name);
10201022
goto found;
10211023
}
10221024

@@ -1026,8 +1028,8 @@ void snd_hda_pick_fixup(struct hda_codec *codec,
10261028
if (!strcmp(codec->modelname, models->name)) {
10271029
id = models->id;
10281030
name = models->name;
1029-
codec_dbg(codec, "%s: picked fixup %s (model specified)\n",
1030-
codec->core.chip_name, codec->fixup_name);
1031+
codec_info(codec, "%s: picked fixup %s (model specified)\n",
1032+
codec->core.chip_name, name);
10311033
goto found;
10321034
}
10331035
models++;
@@ -1085,9 +1087,9 @@ void snd_hda_pick_fixup(struct hda_codec *codec,
10851087
#ifdef CONFIG_SND_DEBUG_VERBOSE
10861088
name = q->name;
10871089
#endif
1088-
codec_dbg(codec, "%s: picked fixup %s for %s %04x:%04x\n",
1089-
codec->core.chip_name, name ? name : "",
1090-
type, q->subvendor, q->subdevice);
1090+
codec_info(codec, "%s: picked fixup %s for %s %04x:%04x\n",
1091+
codec->core.chip_name, name ? name : "",
1092+
type, q->subvendor, q->subdevice);
10911093
found:
10921094
codec->fixup_id = id;
10931095
codec->fixup_list = fixlist;

0 commit comments

Comments
 (0)