From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Peter Ujfalusi <peter.ujfalusi@nokia.com>
Cc: ext Takashi Iwai <tiwai@suse.de>,
alsa-devel@alsa-project.org,
"Aaltonen Matti.J (Nokia-MS/Tampere)"
<matti.j.aaltonen@nokia.com>,
"lrg@slimlogic.co.uk" <lrg@slimlogic.co.uk>
Subject: Re: [PATCH v2 0/1] *** SUBJECT HERE ***
Date: Thu, 22 Jul 2010 12:02:10 +0100 [thread overview]
Message-ID: <20100722110210.GE4737@rakim.wolfsonmicro.main> (raw)
In-Reply-To: <201007221357.16840.peter.ujfalusi@nokia.com>
On Thu, Jul 22, 2010 at 01:57:16PM +0300, Peter Ujfalusi wrote:
> I think the whole SNDRV_PCM_INFO_HALF_DUPLEX bit cleaning/setting can be left
> out as such.
That would be even simpler, of course!
> My point is, that in wl1273_startup, the driver anyway will refuse the non
> correct stream for FM mode. It is checking, that only playback can be started,
> when the FM is in TX mode, and allows only capture, if the chip has been
> configured to FM RX mode. In BT mode the codec can work in full duplex.
...and hopefully userspace won't be trying to do any of the unsupported
things anyway at a higher level than ALSA so that's probably OK. I
guess in principle the device can support full duplex even with FM, it's
just that the output is going to get discarded so it's not *useful* so
there's not such a need to make sure userspace is told about the
restriction anyway - the CPU driver can do the DMA which is all that's
directly visible to the application.
next prev parent reply other threads:[~2010-07-22 11:02 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-07-22 9:38 [PATCH v2 0/1] *** SUBJECT HERE *** Matti J. Aaltonen
2010-07-22 9:38 ` [PATCH v2 1/1] ASoC: TI WL1273 FM Radio Codec Matti J. Aaltonen
2010-07-22 9:48 ` [PATCH v2 0/1] *** SUBJECT HERE *** Mark Brown
2010-07-22 10:57 ` Peter Ujfalusi
2010-07-22 11:02 ` Mark Brown [this message]
2010-07-22 11:07 ` Peter Ujfalusi
2010-07-22 11:30 ` Takashi Iwai
2010-07-22 12:04 ` Mark Brown
2010-07-22 11:18 ` matti.j.aaltonen
2010-07-22 11:25 ` Peter Ujfalusi
2010-07-22 11:37 ` matti.j.aaltonen
2010-07-22 12:02 ` Mark Brown
2010-07-22 12:11 ` matti.j.aaltonen
2010-07-22 13:05 ` Mark Brown
2010-07-22 14:13 ` Matti J. Aaltonen
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=20100722110210.GE4737@rakim.wolfsonmicro.main \
--to=broonie@opensource.wolfsonmicro.com \
--cc=alsa-devel@alsa-project.org \
--cc=lrg@slimlogic.co.uk \
--cc=matti.j.aaltonen@nokia.com \
--cc=peter.ujfalusi@nokia.com \
--cc=tiwai@suse.de \
/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.