2001-11-20 18:02:57

by Marty Pauley

[permalink] [raw]
Subject: Re: [PATCH] OOPS in i810_audio during resume

Hello

When resuming from APM suspend with the i810_audio loaded I got an OOPS that
killed apmd. This simple patch stops the OOPS (so APM keeps working) but it
doesn't solve the problem.

'codec' is null is because it previously failed to initialise in
i810_ac97_init: i810_ac97_probe_and_powerup fails for the second codec; Alex
Bligh mentioned this a few weeks ago.


--- /usr/src/linux/drivers/sound/i810_audio.c.orig Thu Nov 15 01:38:31 2001
+++ /usr/src/linux/drivers/sound/i810_audio.c Thu Nov 15 01:45:07 2001
@@ -2873,6 +2873,10 @@
if(!i810_ac97_exists(card,num_ac97)) {
if(num_ac97) continue;
else BUG();
+ }
+ if(!codec) {
+ printk("i810_audio: cannot resume null codec %d\n", num_ac97);
+ continue;
}
if(!i810_ac97_probe_and_powerup(card,codec)) BUG();

--
Marty