Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761731AbcJROmo (ORCPT ); Tue, 18 Oct 2016 10:42:44 -0400 Received: from galahad.ideasonboard.com ([185.26.127.97]:59567 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1761553AbcJROma (ORCPT ); Tue, 18 Oct 2016 10:42:30 -0400 From: Laurent Pinchart To: John Stultz Cc: lkml , David Airlie , Archit Taneja , Wolfram Sang , Srinivas Kandagatla , Ville =?ISO-8859-1?Q?Syrj=E4l=E4?= , Boris Brezillon , Andy Green , Dave Long , Guodong Xu , Zhangfei Gao , Mark Brown , Lars-Peter Clausen , Jose Abreu , "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 Message-ID: <1728900.jTuP8Bd5Ss@avalon> User-Agent: KMail/4.14.10 (Linux/4.4.6-gentoo; KDE/4.14.24; x86_64; ; ) In-Reply-To: References: <1472514096-10915-1-git-send-email-john.stultz@linaro.org> <3891360.bEMoZb6Wzz@avalon> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1072 Lines: 36 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