From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: John Stultz <john.stultz@linaro.org>
Cc: kuninori.morimoto.gx@renesas.com,
Guodong Xu <guodong.xu@linaro.org>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
lkml <linux-kernel@vger.kernel.org>,
Jose Abreu <joabreu@synopsys.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Mark Brown <broonie@kernel.org>,
Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
Dave Long <dave.long@linaro.org>, Andy Green <andy@warmcat.com>,
Zhangfei Gao <zhangfei.gao@linaro.org>
Subject: Re: [PATCH 0/4 v2] Audio support for adv7511 hdmi bridge
Date: Tue, 18 Oct 2016 17:42:25 +0300 [thread overview]
Message-ID: <1728900.jTuP8Bd5Ss@avalon> (raw)
In-Reply-To: <CALAqxLWR_6fmdUA=rjGSsjTbUZ4WEMBNTJVR2WCHTKyrEAzWHA@mail.gmail.com>
Hi John,
(CC'ing Morimoto-san)
On Tuesday 06 Sep 2016 15:17:58 John Stultz wrote:
> On Tue, Aug 30, 2016 at 2:23 AM, Laurent Pinchart wrote:
> > Hi John,
> >
> > Thank you for the patches.
>
> Thanks so much for the review! I'm reworking the patchset now and will
> be sending out an updated set soon!
>
> >> This patchset, along with the i2s driver and dts changes allows
> >> HDMI audio to work on the HiKey board.
> >
> > Where are the dts changes ?
>
> Here's what I'm using to get it working:
> https://git.linaro.org/people/john.stultz/android-dev.git/commitdiff/1024cb4
> 85b6f00a7e355ce60425f04a584481148
Thank you.
We need to standardize DT bindings for HDMI sound output. Morimoto-san, could
you have a look at the DT bindings proposal for HDMI sound output with the
ADV7511, and comment on whether it matches the approach you've taken for HDMI
sound output on R-Car Gen3 ?
> But again, that's dependent on the k3dma driver (queued), the hi6210
> i2s driver (still being reworked), and adv7511 audio (this patchset).
--
Regards,
Laurent Pinchart
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: John Stultz <john.stultz@linaro.org>
Cc: lkml <linux-kernel@vger.kernel.org>,
"David Airlie" <airlied@linux.ie>,
"Archit Taneja" <architt@codeaurora.org>,
"Wolfram Sang" <wsa+renesas@sang-engineering.com>,
"Srinivas Kandagatla" <srinivas.kandagatla@linaro.org>,
"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"Boris Brezillon" <boris.brezillon@free-electrons.com>,
"Andy Green" <andy@warmcat.com>,
"Dave Long" <dave.long@linaro.org>,
"Guodong Xu" <guodong.xu@linaro.org>,
"Zhangfei Gao" <zhangfei.gao@linaro.org>,
"Mark Brown" <broonie@kernel.org>,
"Lars-Peter Clausen" <lars@metafoo.de>,
"Jose Abreu" <joabreu@synopsys.com>,
"dri-devel@lists.freedesktop.org"
<dri-devel@lists.freedesktop.org>,
kuninori.morimoto.gx@renesas.com
Subject: Re: [PATCH 0/4 v2] Audio support for adv7511 hdmi bridge
Date: Tue, 18 Oct 2016 17:42:25 +0300 [thread overview]
Message-ID: <1728900.jTuP8Bd5Ss@avalon> (raw)
In-Reply-To: <CALAqxLWR_6fmdUA=rjGSsjTbUZ4WEMBNTJVR2WCHTKyrEAzWHA@mail.gmail.com>
Hi John,
(CC'ing Morimoto-san)
On Tuesday 06 Sep 2016 15:17:58 John Stultz wrote:
> On Tue, Aug 30, 2016 at 2:23 AM, Laurent Pinchart wrote:
> > Hi John,
> >
> > Thank you for the patches.
>
> Thanks so much for the review! I'm reworking the patchset now and will
> be sending out an updated set soon!
>
> >> This patchset, along with the i2s driver and dts changes allows
> >> HDMI audio to work on the HiKey board.
> >
> > Where are the dts changes ?
>
> Here's what I'm using to get it working:
> https://git.linaro.org/people/john.stultz/android-dev.git/commitdiff/1024cb4
> 85b6f00a7e355ce60425f04a584481148
Thank you.
We need to standardize DT bindings for HDMI sound output. Morimoto-san, could
you have a look at the DT bindings proposal for HDMI sound output with the
ADV7511, and comment on whether it matches the approach you've taken for HDMI
sound output on R-Car Gen3 ?
> But again, that's dependent on the k3dma driver (queued), the hi6210
> i2s driver (still being reworked), and adv7511 audio (this patchset).
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2016-10-18 14:42 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-29 23:41 [PATCH 0/4 v2] Audio support for adv7511 hdmi bridge John Stultz
2016-08-29 23:41 ` John Stultz
2016-08-29 23:41 ` [PATCH 1/4 v2] drm/bridge: adv7511: Move the common data structures to header file John Stultz
2016-08-29 23:41 ` John Stultz
2016-08-30 8:56 ` Laurent Pinchart
2016-08-30 8:56 ` Laurent Pinchart
2016-09-06 22:49 ` John Stultz
2016-09-06 22:49 ` John Stultz
2016-08-29 23:41 ` [PATCH 2/4 v2] drm/bridge: adv7511: Add Audio support John Stultz
2016-08-30 9:21 ` Laurent Pinchart
2016-08-30 9:21 ` Laurent Pinchart
2016-08-29 23:41 ` [PATCH 3/4 v2] drm/bridge: adv7511: Enable the audio data and clock pads on adv7533 John Stultz
2016-08-29 23:41 ` John Stultz
2016-08-29 23:41 ` [PATCH 4/4 v2] drm/bridge: adv7511: Initialize audio packet " John Stultz
2016-09-02 9:49 ` Archit Taneja
2016-09-02 9:49 ` Archit Taneja
2016-09-06 22:58 ` John Stultz
2016-08-30 9:23 ` [PATCH 0/4 v2] Audio support for adv7511 hdmi bridge Laurent Pinchart
2016-08-30 9:23 ` Laurent Pinchart
2016-09-06 22:17 ` John Stultz
2016-10-18 14:42 ` Laurent Pinchart [this message]
2016-10-18 14:42 ` Laurent Pinchart
2016-10-19 0:26 ` Kuninori Morimoto
2016-11-01 21:41 ` John Stultz
2016-11-01 21:41 ` John Stultz
2016-11-02 1:43 ` Kuninori Morimoto
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=1728900.jTuP8Bd5Ss@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=andy@warmcat.com \
--cc=broonie@kernel.org \
--cc=dave.long@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=guodong.xu@linaro.org \
--cc=joabreu@synopsys.com \
--cc=john.stultz@linaro.org \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=linux-kernel@vger.kernel.org \
--cc=srinivas.kandagatla@linaro.org \
--cc=wsa+renesas@sang-engineering.com \
--cc=zhangfei.gao@linaro.org \
/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.