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

[Takashi Iwai <tiwai@suse.de>]

[Mistakenly posted to alsa-devel, now resent to the correct addreses]

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 doesn'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