On Fri, Jun 9, 2023 at 4:35 AM Takashi Iwai tiwai@suse.de wrote:
When you load snd-hda-intel module with "model=,generic" option (the command is intentional), does the codec content show up?
Also, assuming it being some timing issue, adding the delay forcibly like below has any influence on the AFG error?
Takashi
--- a/sound/pci/hda/hda_proc.c +++ b/sound/pci/hda/hda_proc.c @@ -792,6 +792,7 @@ static void print_codec_info(struct snd_info_entry *entry, snd_iprintf(buffer, "State of AFG node 0x%02x:\n", fg); print_power_state(buffer, codec, fg);
msleep(100); // XXX nodes = snd_hda_get_sub_nodes(codec, fg, &nid); if (! nid || nodes < 0) { snd_iprintf(buffer, "Invalid AFG subtree\n");
I attached three more alsa-info dumps. In all of them, the kernel had my partial revert patch and "snd_hda_intel.power_save=0" on the kernel command line. The ones with "generic" in the name additionally had "snd_hda_intel.model=,generic" on the kernel command line. The ones with "delay" in the name additionally had your above delay patch.
It looks like the delay didn't change anything in the dumps, but snd_hda_intel.model=,generic did result in "Invalid AFG subtree" not being there anymore.
Regards,
Joseph C. Sible