From: Jyri Sarha <jsarha@ti.com>
To: Russell King - ARM Linux <linux@arm.linux.org.uk>
Cc: alsa-devel@alsa-project.org, peter.ujfalusi@ti.com,
devicetree@vger.kernel.org, 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: 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 16:26:50 +0200 [thread overview]
Message-ID: <56D5A6AA.2030907@ti.com> (raw)
In-Reply-To: <56D41F1A.2030100@ti.com>
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.
The example assumes a dt-include file with integer definitions for
TDA998x_SPDIF and TDA998x_I2S.
tda19988 {
compatible = "nxp,tda998x";
reg = <0x70>;
pinctrl-names = "default", "off";
pinctrl-0 = <&nxp_hdmi_bonelt_pins>;
pinctrl-1 = <&nxp_hdmi_bonelt_off_pins>;
video-ports = <0x230145>;
/* Required by simple-card. The number of sound-dai
cells must agree with the number of audio ports
described in audio-ports property. */
#sound-dai-cells = <2>;
/* DAI-format AP_ENA reg value */
audio-ports = < TDA998x_SPDIF 0x04
TDA998x_I2S 0x03>;
port {
hdmi_0: endpoint@0 {
remote-endpoint = <&lcdc_0>;
};
};
};
Russell, Rob? Which one would you prefer, the old graph ports binding or
this simpler version?
Best regards,
Jyri
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2016-03-01 14:26 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 ` Jyri Sarha [this message]
2016-03-01 15:35 ` Alternative binding proposal for tda998x audio (Was: Re: [PATCH RFC v5 4/8] drm/i2c: tda998x: Add support of a DT graph of ports) Jean-Francois Moine
2016-03-01 15:51 ` Jyri Sarha
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=56D5A6AA.2030907@ti.com \
--to=jsarha@ti.com \
--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=peter.ujfalusi@ti.com \
--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).