From: Jyri Sarha <jsarha@ti.com>
To: Jean-Francois Moine <moinejf@free.fr>
Cc: Rob Herring <robh@kernel.org>,
alsa-devel@alsa-project.org,
Russell King - ARM Linux <linux@arm.linux.org.uk>,
peter.ujfalusi@ti.com, devicetree@vger.kernel.org,
airlied@linux.ie, tomi.valkeinen@ti.com, arnaud.pouliquen@st.com,
dri-devel@lists.freedesktop.org, liam.r.girdwood@linux.intel.com,
tony@atomide.com, broonie@kernel.org, bcousson@baylibre.com,
linux-omap@vger.kernel.org
Subject: Re: Alternative binding proposal for tda998x audio (Was: Re: [PATCH RFC v5 4/8] drm/i2c: tda998x: Add support of a DT graph of ports)
Date: Tue, 1 Mar 2016 17:51:09 +0200 [thread overview]
Message-ID: <56D5BA6D.8090904@ti.com> (raw)
In-Reply-To: <20160301163553.3ef298fb2c0bb25ef662a41d@free.fr>
On 03/01/16 17:35, Jean-Francois Moine wrote:
> On Tue, 1 Mar 2016 16:26:50 +0200
> Jyri Sarha <jsarha@ti.com> wrote:
>
>> Ok, here is just one more simple alternative for tda998x audio binding.
>> I feel that the graph ports binding for audio does not make sense
>> without a graph based ASoC machine driver implementation. The ASoC
>> simple-card is already here and it so widely used that there is no
>> getting rid of that any time soon. This proposal provides the same
>> functionality as the patch in the root of this thread, but in a simpler
>> way and is equally compatible with simple-card.
>
> Hi Jyri,
>
> The graph port binding for the tda998x works fine with the simple card
> driver when multi-codec support is added. See
> http://mailman.alsa-project.org/pipermail/alsa-devel/2015-January/085855.html
>
I know that it works, I have used it myself until now, but it is not
needed and there is no driver that parses audio port endpoints. I see no
point specifying something in the binding that is not used and there no
specific plan to ever use it.
AFAIU my proposed binding should work equally well with simple-card,
with or without multi-codec support.
Best regards,
Jyri
next prev parent reply other threads:[~2016-03-01 15:51 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-17 14:49 [PATCH RFC v5 0/8] Implement generic ASoC HDMI codec and use it in tda998x Jyri Sarha
2016-02-17 14:49 ` [PATCH RFC v5 1/8] ALSA: pcm: add IEC958 channel status helper for hw_params Jyri Sarha
2016-02-17 14:49 ` [PATCH RFC v5 2/8] ASoC: hdmi-codec: Add hdmi-codec for external HDMI-encoders Jyri Sarha
2016-02-22 14:22 ` Arnaud Pouliquen
2016-03-08 16:59 ` Arnaud Pouliquen
[not found] ` <cover.1455720381.git.jsarha-l0cyMroinI0@public.gmane.org>
2016-02-17 14:49 ` [PATCH RFC v5 3/8] ASoC: hdmi-codec: Add audio abort() callback for video side to use Jyri Sarha
2016-02-17 14:49 ` [PATCH RFC v5 4/8] drm/i2c: tda998x: Add support of a DT graph of ports Jyri Sarha
[not found] ` <842e221030a0b14bc862790eb2f5bc97bb29c012.1455720381.git.jsarha-l0cyMroinI0@public.gmane.org>
2016-02-18 14:35 ` Rob Herring
2016-02-18 15:18 ` Jean-Francois Moine
2016-02-18 15:32 ` Russell King - ARM Linux
2016-02-25 13:42 ` Jyri Sarha
[not found] ` <56CF04DA.8080905-l0cyMroinI0@public.gmane.org>
2016-02-26 0:43 ` Russell King - ARM Linux
2016-02-26 10:14 ` Jyri Sarha
2016-02-26 11:21 ` Russell King - ARM Linux
2016-02-29 10:36 ` Jyri Sarha
2016-03-01 14:26 ` Alternative binding proposal for tda998x audio (Was: Re: [PATCH RFC v5 4/8] drm/i2c: tda998x: Add support of a DT graph of ports) Jyri Sarha
2016-03-01 15:35 ` Jean-Francois Moine
2016-03-01 15:51 ` Jyri Sarha [this message]
2016-03-01 16:16 ` Jean-Francois Moine
2016-03-01 18:29 ` Jyri Sarha
[not found] ` <56D5DF7D.7090608-l0cyMroinI0@public.gmane.org>
2016-03-01 19:26 ` Jean-Francois Moine
2016-03-02 8:34 ` Jyri Sarha
[not found] ` <20160301171630.630a47dc00060645f89bb8ab-GANU6spQydw@public.gmane.org>
2016-03-02 4:29 ` Mark Brown
2016-02-17 14:49 ` [PATCH RFC v5 5/8] drm/i2c: tda998x: Remove include/sound/tda998x.h and fix graph parsing Jyri Sarha
2016-02-17 14:49 ` [PATCH RFC v5 6/8] drm/i2c: tda998x: Improve tda998x_configure_audio() audio related pdata Jyri Sarha
2016-02-17 14:49 ` [PATCH RFC v5 7/8] drm/i2c: tda998x: Register ASoC HDMI codec for audio functionality Jyri Sarha
2016-02-17 14:49 ` [PATCH RFC v5 8/8] ARM: dts: am335x-boneblack: Add HDMI audio support Jyri Sarha
2016-02-17 19:36 ` Robert Nelson
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=56D5BA6D.8090904@ti.com \
--to=jsarha@ti.com \
--cc=airlied@linux.ie \
--cc=alsa-devel@alsa-project.org \
--cc=arnaud.pouliquen@st.com \
--cc=bcousson@baylibre.com \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=liam.r.girdwood@linux.intel.com \
--cc=linux-omap@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=moinejf@free.fr \
--cc=peter.ujfalusi@ti.com \
--cc=robh@kernel.org \
--cc=tomi.valkeinen@ti.com \
--cc=tony@atomide.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).