On 7/21/23 13:34, Péter Ujfalusi wrote: > > > On 19/07/2023 19:41, Maarten Lankhorst wrote: >> Add missing pci_set_drv to NULL call on error. >> >> Signed-off-by: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx> >> --- >> sound/pci/hda/hda_intel.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c >> index ef831770ca7da..0d2d6bc6c75ef 100644 >> --- a/sound/pci/hda/hda_intel.c >> +++ b/sound/pci/hda/hda_intel.c >> @@ -2188,6 +2188,7 @@ static int azx_probe(struct pci_dev *pci, >> return 0; >> >> out_free: >> + pci_set_drvdata(pci, NULL); > The original patch added this: > f4c482a4d0b3 ("ALSA: hda - Fix yet another race of vga_switcheroo registration") > > but got removed later by: > 20a24225d8f9 ("ALSA: PCI: Remove superfluous pci_set_drvdata(pci, NULL) at remove") > > and partially added back (to azx_remove) by: > e81478bbe7a1 ("ALSA: hda: fix general protection fault in azx_runtime_idle") > > I guess, it should do not harm to add it back... > >> snd_card_free(card); >> return err; >> } > > Reviewed-by: Peter Ujfalusi <peter.ujfalusi@xxxxxxxxxxxxxxx> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>