All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Rene Rebe <rene.rebe@gmx.net>
Cc: alsa-devel@lists.sourceforge.net
Subject: Re: [PATCH] pmac mixer update from shadow register on resume and switching DRC on headphone plug
Date: Mon, 02 Aug 2004 12:55:22 +0200	[thread overview]
Message-ID: <s5h3c35u7vp.wl@alsa2.suse.de> (raw)
In-Reply-To: <410A6190.1020600@gmx.net>

At Fri, 30 Jul 2004 16:56:16 +0200,
Rene Rebe wrote:
> 
> Hi all,
> 
> the attached patch improved the pmac driver by implementing updating the 
> mixer from the shadow register after a resume or headphone interrupt, as 
> well as automatically selecting DRC on headphone plug.
> 
> For normal line-out one does not want the DRC, but it should 
> automatically be reenabled after the headphone is unplugged: Otherwise 
> the power to the internal speakers is high enought to destroy them 
> (happend once to my iBook - when it still had warrenty ... ;-)
> 
> I needed a work-queue / task-queue since it can not be done in the 
> interrupt context - this is done in the way the OSS driver does it, too. 
> I hope this is acceptable.

That's fine, but we keep 2.6 clean from older kernel codes.
The mixed code is the result after patch, which you can find in
alsa-driver tree as *.patch file.

Could you make them separate, and send the patch to tumbler.c and the
new tumber.patch?
i.e. alsa-kernel/ppc/tumbler.c contains 2.6-only code and the stuff
within "#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)" is added by
alsa-driver/ppc/tumbler.patch.


Takashi


-------------------------------------------------------
This SF.Net email is sponsored by OSTG. Have you noticed the changes on
Linux.com, ITManagersJournal and NewsForge in the past few weeks? Now,
one more big change to announce. We are now OSTG- Open Source Technology
Group. Come see the changes on the new OSTG site. www.ostg.com

      reply	other threads:[~2004-08-02 10:55 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-30 14:56 [PATCH] pmac mixer update from shadow register on resume and switching DRC on headphone plug Rene Rebe
2004-08-02 10:55 ` 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=s5h3c35u7vp.wl@alsa2.suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=rene.rebe@gmx.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.