[alsa-devel] [RFC PATCH] hda_intel: ratelimit "spurious response" message

[Takashi Iwai]

At Thu, 14 Nov 2013 09:52:34 -0800,
Joe Perches wrote:
> 
> On Thu, 2013-11-14 at 18:28 +0100, Takashi Iwai wrote:
> > At Fri, 08 Nov 2013 20:41:34 -0800,
> > Joe Perches wrote:
> []
> > > !!HDA-Intel Codec information
> > > !!---------------------------
> > > --startcollapse--
> > > 
> > > Codec: ATI ID aa01
> > 
> > You didn't enable CONFIG_SND_HDA_CODEC_HDMI, right?
> 
> No, it's enabled.
> 
> > Build with it, at least.
> 
> $ grep CONFIG_SND_HDA_CODEC_HDMI /boot/config-3.12.0 
> CONFIG_SND_HDA_CODEC_HDMI=y

Weird.  Try to build all sound drivers as modules and see whether the
ID string of HDMI codec is still shown like that or a proper name like
"ATI R6xx HDMI".

If this works, could you try the patch below and see whether it works
better now with built-in kernel?


Takashi

---
diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
index afb90f48867f..80736e5af548 100644
--- a/sound/pci/hda/hda_codec.c
+++ b/sound/pci/hda/hda_codec.c
@@ -976,6 +976,7 @@ find_codec_preset(struct hda_codec *codec)
 	mutex_unlock(&preset_mutex);
 
 	if (mod_requested < HDA_MODREQ_MAX_COUNT) {
+#ifdef CONFIG_SND_HDA_INTEL_MODULE
 		char name[32];
 		if (!mod_requested)
 			snprintf(name, sizeof(name), "snd-hda-codec-id:%08x",
@@ -986,6 +987,7 @@ find_codec_preset(struct hda_codec *codec)
 		request_module(name);
 		mod_requested++;
 		goto again;
+#endif /* CONFIG_SND_HDA_INTEL_MODULE */
 	}
 	return NULL;
 }