linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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.
>

  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).