From: Takashi Iwai <takashi.iwai@gmail•com>
To: Colin Leroy <colin@colino•net>
Cc: Takashi Iwai <tiwai@suse•de>,
"debian-powerpc@lists•debian.org"
<debian-powerpc@lists•debian.org>, Andrew Morton <akpm@osdl•org>,
linuxppc-dev list <linuxppc-dev@ozlabs•org>
Subject: Re: [PATCH 2.6.12-rc3] pmac: save master volume on sleep
Date: Sun, 24 Apr 2005 18:43:32 +0200 [thread overview]
Message-ID: <6dc076840504240943598b05f2@mail.gmail.com> (raw)
In-Reply-To: <20050424182456.34e92bd2@jack.colino.net>
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 <colin@colino•net> wrote:
> On 22 Apr 2005 at 10h04, Benjamin Herrenschmidt wrote:
>=20
> Hi,
>=20
> > > 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.
>=20
> Takashi: any news or comments on this?
>=20
> Thanks,
> --
> Colin
>=20
>
prev parent reply other threads:[~2005-04-24 16:50 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-04-21 18:04 [PATCH 2.6.12-rc3] pmac: save master volume on sleep Colin Leroy
2005-04-22 0:17 ` Benjamin Herrenschmidt
2005-04-24 16:24 ` Colin Leroy
2005-04-24 16:43 ` Takashi Iwai [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6dc076840504240943598b05f2@mail.gmail.com \
--to=takashi.iwai@gmail$(echo .)com \
--cc=akpm@osdl$(echo .)org \
--cc=colin@colino$(echo .)net \
--cc=debian-powerpc@lists$(echo .)debian.org \
--cc=linuxppc-dev@ozlabs$(echo .)org \
--cc=tiwai@suse$(echo .)de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox