All of lore.kernel.org
 help / color / mirror / Atom feed
From: Vinod Koul <vinod.koul@intel.com>
To: Lars-Peter Clausen <lars@metafoo.de>
Cc: "Jean-Francois Moine" <moinejf@free.fr>,
	alsa-devel@alsa-project.org,
	"Russell King - ARM Linux ‎" <linux@arm.linux.org.uk>,
	"David Airlie" <airlied@linux.ie>,
	broonie@kernel.org, "Arnaud Pouliquen" <arnaud.pouliquen@st.com>,
	lgirdwood@gmail.com, dri-devel@lists.freedesktop.org,
	peter.ujfalusi@ti.com, tomi.valkeinen@ti.com,
	"Jyri Sarha" <jsarha@ti.com>
Subject: Re: HDMI codec, way forward?
Date: Sun, 18 Oct 2015 20:32:13 +0530	[thread overview]
Message-ID: <20151018150213.GN27370@localhost> (raw)
In-Reply-To: <5620EE2F.8070504@metafoo.de>

On Fri, Oct 16, 2015 at 02:31:43PM +0200, Lars-Peter Clausen wrote:
> On 10/16/2015 01:50 PM, Jyri Sarha wrote:
> > After reading the ELCE Audio mini conf minutes [1] I gather that HDMI
> > audio was not discussed after all.
> 
> It was discussed, but rather shortly and only in the context of the HDA.
> (Adding Vinod to Cc, who presented yet another approach to the problem last
> week)

Thanks for adding me :)

Based on the links sent by Jyri, I did look up the series from Jyri and
Arnaud. I have tried to summarize the approach but please do correct me if I
got something wrong

There are similarities but both are trying to add an interface between audio
and display.

Jyri's approach to add generic IEC code makes sense and drives reuse. I kind
of didn't like adding rates and formats for DAIs, if they are placeholders
then it is okay but otherwise we should read and set them from ELD. Also I
have reservations about hdmi_codec_ops and this being set by drm driver, why
not use component interface for these things and let data be shared!

Arnaud's approach to add N,CTS helpers is great. But having calls to drm
from audio side for drm enabling wouldn't have been required if we have
component interface here.

If we have these calls on a component interface and use these as generic ops
for the audio-drm interface rather than i915 which is implemented today we
don't need to export symbols and create a dependency between these two side.

Also I would again like you folks to review the HDA HDMI ASoC driver RFC I
posted recently [1]. We use component interface and have been adding more
callback to interface. David added notification [2] and we plan to add ELD
query as well soonish.

[1]: http://mailman.alsa-project.org/pipermail/alsa-devel/2015-October/098730.html
[2]: http://mailman.alsa-project.org/pipermail/alsa-devel/2015-August/097042.html

-- 
~Vinod

  parent reply	other threads:[~2015-10-18 14:59 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-10-16 11:50 HDMI codec, way forward? Jyri Sarha
2015-10-16 12:31 ` Lars-Peter Clausen
2015-10-16 12:43   ` [alsa-devel] " Takashi Iwai
2015-10-16 13:37   ` Russell King - ARM Linux
2015-10-16 14:12     ` Mark Brown
2015-10-18 15:08     ` Vinod Koul
2015-10-18 15:20       ` Russell King - ARM Linux
2015-10-18 16:13         ` Vinod Koul
2015-10-18 16:22           ` Russell King - ARM Linux
2015-10-18 17:16           ` Russell King - ARM Linux
2015-10-19 13:20             ` [alsa-devel] " Takashi Iwai
2015-10-20  3:38               ` Vinod Koul
2015-10-20  8:08                 ` [alsa-devel] " Russell King - ARM Linux
2015-10-20 14:01                   ` Vinod Koul
2015-10-21  9:11                     ` [alsa-devel] " Jani Nikula
2015-10-21 15:52                       ` Vinod Koul
2015-10-21  9:27                     ` [alsa-devel] " Russell King - ARM Linux
2015-10-21 13:41                       ` Takashi Iwai
2015-10-21 14:03                         ` Russell King - ARM Linux
2015-10-21 14:10                           ` Takashi Iwai
2015-10-21 14:37                             ` Takashi Iwai
2015-10-21 15:36                               ` Daniel Vetter
2015-10-21 16:46                                 ` Takashi Iwai
2015-10-21 16:48                                   ` Takashi Iwai
2015-10-21 16:50                                     ` Takashi Iwai
2015-10-21 14:37                             ` Russell King - ARM Linux
2015-10-21 16:19                               ` Vinod Koul
2015-10-21 17:34                                 ` Russell King - ARM Linux
2015-10-21 17:59                                   ` [alsa-devel] " Takashi Iwai
2015-10-21 18:19                                     ` Russell King - ARM Linux
2015-10-21 20:37                                       ` Takashi Iwai
2015-10-21 21:04                                         ` Russell King - ARM Linux
2015-10-22  7:18                                           ` Daniel Vetter
2015-10-16 14:06   ` Mark Brown
2015-10-18 15:02   ` Vinod Koul [this message]
2015-10-19 20:14     ` Jyri Sarha
2015-10-16 13:51 ` Arnaud Pouliquen
2015-10-16 14:15   ` Mark Brown
2015-10-18 15:07     ` Vinod Koul
2015-10-19 13:10   ` Jyri Sarha
2015-10-19 13:31     ` Russell King - ARM Linux

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=20151018150213.GN27370@localhost \
    --to=vinod.koul@intel.com \
    --cc=airlied@linux.ie \
    --cc=alsa-devel@alsa-project.org \
    --cc=arnaud.pouliquen@st.com \
    --cc=broonie@kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=jsarha@ti.com \
    --cc=lars@metafoo.de \
    --cc=lgirdwood@gmail.com \
    --cc=linux@arm.linux.org.uk \
    --cc=moinejf@free.fr \
    --cc=peter.ujfalusi@ti.com \
    --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 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.