Message-ID: <87insqjbr9.wl%kuninori.morimoto.gx@renesas.com>
From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To: Jean-Francois Moine <moinejf@free.fr>
CC: Rob Herring <robh+dt@kernel.org>, Mark Brown <broonie@kernel.org>,
        Linux-DT <devicetree@vger.kernel.org>,
        Linux-ALSA <alsa-devel@alsa-project.org>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Linux-Kernel <linux-kernel@vger.kernel.org>,
        Simon <horms@verge.net.au>,
        Laurent <laurent.pinchart@ideasonboard.com>,
        Grant Likely <grant.likely@linaro.org>,
        Frank Rowand <frowand.list@gmail.com>,
        Guennadi <g.liakhovetski@gmx.de>
Subject: Re: [alsa-devel] [PATCH 0/23] ASoC: add OF graph base simple-card
In-Reply-To: <20161018091030.ed9eea657480f6a20bcfea81@free.fr>
References: <87shrv4c8x.wl%kuninori.morimoto.gx@renesas.com>
        <20161018091030.ed9eea657480f6a20bcfea81@free.fr>
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: Tue, 18 Oct 2016 08:44:36 +0000
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2016 08:44:36.2512 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PS1PR0601MB1451
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1084
Lines: 28


Hi Jean-Francois

> > These are OF graph base simple-card patch-set.
> >  1) -  3) : simple-scu-card cleanup
> >  4) - 10) : soc-core prepare for OF graph card
> > 11) - 17) : OF graph new feature
> > 18) - 23) : OF graph base simple-card
(snip)
> >       11) Documentation: of: add type property
> >       12) of_graph: add of_graph_get_remote_endpoint()
> >       13) of_graph: add of_graph_port_type_is()
> >       14) of_graph: add of_graph_get_port_parent()
> >       15) of_graph: add of_graph_get_top_port()
> >       16) of_graph: add for_each_of_port() / for_each_of_endpoint_in_port()
> >       17) of_graph: add of_graph_get_endpoint_count()
(snip)
> The patches 11..17 put trivial functions in the OF core. As they are
> used by only a few systems, it would be better to define them as
> macros, or in a module or in the simple-card-utils.

We can replace few drivers to use it. I will post it if these were accepted.
And let's follow Rob's opinion. Actually, these patches had posted
few month ago, and these already got OK from him.

Best regards
---
Kuninori Morimoto