From: Daniel Mack <daniel@caiaq.de>
To: Mark Brown <broonie@sirena.org.uk>
Cc: alsa-devel@alsa-project.org, Timur Tabi <timur@freescale.com>
Subject: Re: [PATCH 2/3] ASoC: cs4270: add Master Playback Switch
Date: Fri, 24 Apr 2009 15:59:22 +0200 [thread overview]
Message-ID: <20090424135922.GG10450@buzzloop.caiaq.de> (raw)
In-Reply-To: <20090424135439.GG26440@sirena.org.uk>
On Fri, Apr 24, 2009 at 02:54:40PM +0100, Mark Brown wrote:
> > I'll send a new patch for that, providing this feature in a slightly
> > more complex fashion. However, I don't want to remove the other
> > function. If the soc core decides to mute the codec, it should be able
> > to do so.
>
> It's not the core that needs it, it's the DAC that normally needs the
> core to mute it (modulo random floating signals that it might parse) -
> if cs4270 is happy without the mute then the core doesn't care if it's
> not there.
This codec has pins which are usually connected to some external logic
that can switch off aplifiers and the like when the code is in its mute
state. Which is a nice feature: once you stop the ALSA output,
everything behind the codec in the analog section is put into a
power saving mode and is magically revived once ALSA starts using it
again. That's why I want to leave it in there, even if the codecs does
not necessarily need it.
Daniel
next prev parent reply other threads:[~2009-04-24 13:59 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-24 13:00 [PATCH 1/3] ASoC: cs4270: fix Master Capture Switch polarity Daniel Mack
2009-04-24 13:00 ` [PATCH 2/3] ASoC: cs4270: add Master Playback Switch Daniel Mack
2009-04-24 13:02 ` Daniel Mack
2009-04-24 13:17 ` Mark Brown
2009-04-24 13:52 ` Daniel Mack
2009-04-24 13:54 ` Mark Brown
2009-04-24 13:59 ` Daniel Mack [this message]
2009-04-24 14:37 ` [PATCH 1/2] ASoC: add SOC_DOUBLE_EXT macro Daniel Mack
2009-04-24 14:37 ` [PATCH 2/2] ASoC: cs4270: add Master Playback Switch Daniel Mack
2009-04-27 13:55 ` Timur Tabi
2009-04-24 15:45 ` [PATCH 1/3] ASoC: cs4270: fix Master Capture Switch polarity Timur Tabi
2009-04-24 16:08 ` Daniel Mack
2009-04-24 16:13 ` Timur Tabi
2009-04-24 16:21 ` Daniel Mack
2009-04-26 9:49 ` Mark Brown
2009-04-26 11:08 ` Takashi Iwai
2009-04-26 11:21 ` Mark Brown
2009-04-26 16:56 ` Takashi Iwai
2009-04-26 11:30 ` Daniel Mack
2009-04-26 16:52 ` Takashi Iwai
2009-04-27 9:37 ` Daniel Mack
2009-04-27 9:38 ` Mark Brown
2009-04-27 12:54 ` Timur Tabi
2009-04-27 13:53 ` Timur Tabi
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=20090424135922.GG10450@buzzloop.caiaq.de \
--to=daniel@caiaq.de \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@sirena.org.uk \
--cc=timur@freescale.com \
/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.