* [PATCH] maestro3 2.4.4-pre5 fixes
@ 2001-04-21 14:54 Zach Brown
0 siblings, 0 replies; only message in thread
From: Zach Brown @ 2001-04-21 14:54 UTC (permalink / raw)
To: torvalds; +Cc: linux-kernel, maestro-users
The maestro3 driver has a error cleanup bug that would leave its reboot
notifier registerd after exiting init_module with an error. Bayad.
It also had a minor bug where it didn't explicitly set the codec->id
before probing..
this patch vs 2.4.4-pre5 (I hope, its really vs jeff's cvs :)) fixes both.
Linus, please apply.
--
zach
Index: maestro3.c
===================================================================
RCS file: /cvsroot/gkernel/linux_2_4/drivers/sound/maestro3.c,v
retrieving revision 1.1.1.24.6.1
diff -u -r1.1.1.24.6.1 maestro3.c
--- maestro3.c 2001/04/20 01:40:38 1.1.1.24.6.1
+++ maestro3.c 2001/04/21 14:36:21
@@ -2307,6 +2307,8 @@
codec->private_data = card;
codec->codec_read = m3_ac97_read;
codec->codec_write = m3_ac97_write;
+ /* someday we should support secondary codecs.. */
+ codec->id = 0;
if (ac97_probe_codec(codec) == 0) {
printk(KERN_ERR PFX "codec probe failed\n");
@@ -2933,6 +2935,7 @@
if (!pci_register_driver(&m3_pci_driver)) {
pci_unregister_driver(&m3_pci_driver);
+ unregister_reboot_notifier(&m3_reboot_nb);
return -ENODEV;
}
return 0;
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2001-04-21 14:54 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-04-21 14:54 [PATCH] maestro3 2.4.4-pre5 fixes Zach Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®