linux-omap.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Mark Brown <broonie@opensource.wolfsonmicro.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: alsa-devel <alsa-devel@alsa-project.org>,
	Ricardo Neri <ricardo.neri@ti.com>,
	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>
Subject: Re: [RFC] ASoC: snd_soc_jack for HDMI audio: does it make	sense?
Date: Tue, 21 Aug 2012 13:05:12 +0100	[thread overview]
Message-ID: <20120821120511.GB7995@opensource.wolfsonmicro.com> (raw)
In-Reply-To: <s5hlih895kt.wl%tiwai@suse.de>


[-- Attachment #1.1: Type: text/plain, Size: 1566 bytes --]

On Tue, Aug 21, 2012 at 07:28:34AM +0200, Takashi Iwai wrote:
> Ricardo Neri wrote:

> > I was wondering about how much sense does it make to you guys use a 
> > snd_soc_jack in this case?

> HD-audio already uses the generic jack event for the HDMI/DP
> connection change notification as well, so I think it would make sense
> in general.

The whole problem here is that we don't *have* a generic jack interface.
We've got:

 - sound/core/jack.c which was written to be a generic API and is used
   by everything that does jack support currently.

 - sound/core/ctljack.c which was added later and provides separate
   in-kernel and userspace APIs and is currently only used by HDA.

 - extcon which does have a good reason to be a separate API since that
   it's not audio specific (and is likely to be picked up by Android as
   the code was originally taken from there); it's currently not
   supported by the frameworks in ALSA.  I'd suggest Pulse should be using
   it too.

This is a complete shambles for both driver authors and userspace, the
ABI varies randomly with drivers and in theory driver authors have to
implement everything three times which is just nuts.

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.

Short term for drivers used on embedded systems I'd have to recommend
extcon rather than anything ALSA-specific.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  reply	other threads:[~2012-08-21 12:05 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 [this message]
2012-08-21 12:30     ` David Henningsson
2012-08-21 13:16       ` Mark Brown
2012-08-22  1:24         ` Ricardo Neri
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=20120821120511.GB7995@opensource.wolfsonmicro.com \
    --to=broonie@opensource.wolfsonmicro.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=lrg@ti.com \
    --cc=peter.ujfalusi@ti.com \
    --cc=ricardo.neri@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).