Message-ID: <87zili3be2.wl%kuninori.morimoto.gx@renesas.com>
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To: John Stultz <john.stultz@linaro.org>
CC: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        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 =?ISO-8859-1?Q?Syrj=E4l=E4?= 
        <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>
Subject: Re: [PATCH 0/4 v2] Audio support for adv7511 hdmi bridge
In-Reply-To: <CALAqxLXi8=g0=T_M+puN2+vUWKgcY11BnHEFQf_ahA+cp39bYw@mail.gmail.com>
References: <1472514096-10915-1-git-send-email-john.stultz@linaro.org>
        <3891360.bEMoZb6Wzz@avalon>
        <CALAqxLWR_6fmdUA=rjGSsjTbUZ4WEMBNTJVR2WCHTKyrEAzWHA@mail.gmail.com>
        <1728900.jTuP8Bd5Ss@avalon>
        <874m4942gc.wl%kuninori.morimoto.gx@renesas.com>
        <CALAqxLXi8=g0=T_M+puN2+vUWKgcY11BnHEFQf_ahA+cp39bYw@mail.gmail.com>
User-Agent: Wanderlust/2.15.9 Emacs/24.3 Mule/6.0
MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue")
Content-Type: text/plain; charset="US-ASCII"
Date: Wed, 2 Nov 2016 01:43:26 +0000
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Nov 2016 01:43:26.1627 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-Transport-CrossTenantHeadersStamped: KL1PR0601MB1446
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1381
Lines: 31


Hi John

> > Above patch is using normal simple-card style for HDMI sound, but as Laurent
> > said we want to use same DT style for HDMI video and sound (= OF graph style).
> > Fortunately, I posted patch-set for OF graph support on sound card yesterday.
> > Can you check this ?
> > http://www.spinics.net/lists/devicetree/msg146131.html
> > The points are
> >  - sound can use OF graph style DT
> >  - sound-card DT is no longer needed
> >  - it needs type = "sound" property
> >
> > patch-set [0/23] is this
> > http://www.spinics.net/lists/devicetree/msg146113.html
> 
> Thanks for the pointers! If I understand this correctly, the OF graph
> simple-card method would replace my current simple-card dts usage for
> hikey?   Other then just having another out-of-tree patchset to track,
> I don't have any objection to trying to use the OF graph simple-card
> method in the hikey dts (though I suspect I'll have to pester you for
> help when I give it a shot).

I posted v2 patchset OF graph simple-card which doesn't have "type" property.
Anyway, if you use OF graph simple-card, your "CPU" side driver need to
call asoc_simple_card_try_to_probe_graph_card() to probing it.

And this is not 100% mandatory, but your CPU and/or Codec driver
want to adjust OF graph style for parsing. The position is like this

simple-card :: dai-link <-> OF graph simple-card :: port/endpoint