From: Ricardo Neri <ricardo.neri@ti.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: alsa-devel <alsa-devel@alsa-project.org>,
Takashi Iwai <tiwai@suse.de>,
Peter Ujfalusi <peter.ujfalusi@ti.com>,
"Valkeinen, Tomi" <tomi.valkeinen@ti.com>,
"Guiriec, Sebastien" <s-guiriec@ti.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
Liam Girdwood <lrg@ti.com>,
David Henningsson <david.henningsson@canonical.com>
Subject: Re: [RFC] ASoC: snd_soc_jack for HDMI audio: does it make sense?
Date: Tue, 21 Aug 2012 20:24:45 -0500 [thread overview]
Message-ID: <503434DD.8030003@ti.com> (raw)
In-Reply-To: <20120821131625.GD7995@opensource.wolfsonmicro.com>
On 08/21/2012 08:16 AM, Mark Brown wrote:
> On Tue, Aug 21, 2012 at 02:30:51PM +0200, David Henningsson wrote:
>> On 08/21/2012 02:05 PM, Mark Brown wrote:
>
>>> - sound/core/ctljack.c which was added later and provides separate
>>> in-kernel and userspace APIs and is currently only used by HDA.
>
>> This is wrong. PulseAudio uses the new ctljack API. I started out
>> with an implementation which used the input device API
>> (sound/core/jack.c), but since some people did not like this API,
>> the ctljack API was designed and implemented for PA to use, which it
>> does (since PulseAudio 2.0 - the input device API implementation in
>> PulseAudio was never merged upstream).
>
> ...and the only thing using this API to generate events is HDA which is
> what I was talking about here given that this is a question from a
> driver point of view.
>
>>> What I'd like to see happening is that we merge ctljack into jack (since
>>> only HDA is going to be affected by that change it seems like the right
>>> direction to make the merge) and also add extcon support, I have looked
>>> at the extcon support.
>
>> I don't know either why we have two different interfaces for jack
>> detection (not counting extcon) for driver writers. I think we
>> should merge jack and ctljack, as long as that does not mean you
>> break anything I'm relying on in ctljack.
>
>> Maybe something for discussion at Plumbers?
>
> We discussed this last time we all met and came to the above conclusion :/
>
>>> Short term for drivers used on embedded systems I'd have to recommend
>>> extcon rather than anything ALSA-specific.
>
>> I thought that when Takashi did the new ctljack interface, that
>> would effectively deprecate the old input device interface, and that
>> ctljack was the new recommendation.
>
> Android currently uses the out of tree version of the extcon ABI (like I
> say it's where the code originated), and some OSs do use the input ABI
> also but realistically if you've got to pick one Android is where the
> market is at.
So, it seems that the way to go is extcon. I guess that ALSA will use
extcon just like today snd_jack uses the input driver. is this correct?
Is there any chance that ctrljack will propagate the events through
extcon? Is there any early implementation that I could look at? I am
asking to know how feasible is to use ctljack today and be compatible
with extcon in the future.
Thanks!
Ricardo
>
> To my knowledge no embedded OS is using the control jacks, it's possible
> that there's something using them as a function of using PulseAudio but
> the ones I'm famililar with currently ignore PulseAudio routing and just
> use the mixing facilities. Given the dependency on alsa-lib it'd be an
> inconvenient ABI to adopt for most of them.
>
next prev parent reply other threads:[~2012-08-22 1:24 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-21 1:47 [RFC] ASoC: snd_soc_jack for HDMI audio: does it make sense? Ricardo Neri
2012-08-21 5:28 ` [alsa-devel] " Takashi Iwai
2012-08-21 12:05 ` Mark Brown
2012-08-21 12:30 ` [alsa-devel] " David Henningsson
2012-08-21 13:16 ` Mark Brown
2012-08-22 1:24 ` Ricardo Neri [this message]
2012-08-22 16:40 ` Mark Brown
2012-08-24 7:10 ` [alsa-devel] " Arun Raghavan
2012-08-27 18:55 ` Mark Brown
2012-08-21 6:01 ` Tomi Valkeinen
2012-08-21 12:39 ` Clark, Rob
2012-08-21 13:18 ` Mark Brown
2012-08-22 0:58 ` Ricardo Neri
2012-08-22 7:55 ` [alsa-devel] " Takashi Iwai
2012-08-24 1:44 ` Ricardo Neri
2012-08-24 2:57 ` Stephen Warren
2012-08-24 5:21 ` [alsa-devel] " Takashi Iwai
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=503434DD.8030003@ti.com \
--to=ricardo.neri@ti.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=david.henningsson@canonical.com \
--cc=linux-omap@vger.kernel.org \
--cc=lrg@ti.com \
--cc=peter.ujfalusi@ti.com \
--cc=s-guiriec@ti.com \
--cc=tiwai@suse.de \
--cc=tomi.valkeinen@ti.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).