Hi Colin, the patch looks OK to me too. Could you resend this to Jaroslav to apply it to ALSA tree, since I'm in vacation...?
Thanks. Takashi On 4/24/05, Colin Leroy <[EMAIL PROTECTED]> wrote: > On 22 Apr 2005 at 10h04, Benjamin Herrenschmidt wrote: > > Hi, > > > > Hi, > > > > > > Ben's patch that shutdowns master switch and restores it after > > > resume ("pmac: Improve sleep code of tumbler driver") isn't enough > > > here on an iBook (snapper chip). > > > > > > The master switch is correctly saved and restored, but somehow > > > tumbler_put_master_volume() gets called just after > > > tumbler_set_master_volume() and sets mix->master_vol[*] to 0. So, on > > > resuming, the master switch is reenabled, but the volume is set to > > > 0. > > > > > > Here's a patch that also saves and restores master_vol. > > > Thanks, > > > > Looks good. > > Takashi: any news or comments on this? > > Thanks, > -- > Colin > >