Re: [PATCH]: Suspend2 Merge: Device driver fixes 2/2

[Takashi Iwai <tiwai@suse.de>]

At Thu, 16 Sep 2004 20:45:36 +1000,
Nigel Cunningham wrote:
> 
> -----Forwarded Message-----
> > From: Bernard Blackham <bernard@blackham.com.au>
> > To: Michael Frank <mhf@linuxmail.org>
> > Cc: softwaresuspend-devel@lists.berlios.de
> > Subject: Re: [SoftwareSuspend-devel] ALSA ali5451 and sis PM fundamentally broken?
> > Date: Sun, 08 Aug 2004 22:33:54 +0800
> > 
> > On Fri, Jul 16, 2004 at 01:24:09PM +0800, Michael Frank wrote:
> > > Built the current Gentoo ALSA package 1.05a on 2.4.24.
> > > 
> > > On ali5451 it is difficult to get the driver to work
> > > at all - takes several loads. Does not survive suspend.
> > 
> > With the latest 2.6.7 and swsusp 2.0.0.103, my sound card's
> > (ali5451) suspend/resume began eratticaly failing again. So I
> > started tinkering some more, and I've come up with this patch (in
> > addition to the one liner already in 2.0.0.103-for-2.6.7) that makes
> > sound reliably suspend and resume every time *and* playback is not
> > interrupted! :)
> > 
> > The line "snd_pcm_suspend_all(chip->pcm)" was the culprit for
> > stopping playback and requiring it to be restarted. It seems common
> > to some other alsa drivers too, so removing it may help with similar
> > problems experienced by others.

No, snd_pcm_suspend_all() must be called.  All PCM streams are
supposed to be moved to SUSPENDED state when the suspend happens.
Keeping other states don't guaratee the proper resume process.

The problem of ali5451 driver was that it forgot to call
snd_power_change_state() at the end of suspend callback.
This was already fixed.

So, don't apply this patch, please.


Takashi


-------------------------------------------------------
This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170
Project Admins to receive an Apple iPod Mini FREE for your judgement on
who ports your project to Linux PPC the best. Sponsored by IBM.
Deadline: Sept. 24. Go here: http://sf.net/ppc_contest.php