From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from galahad.ideasonboard.com ([185.26.127.97]:47368 "EHLO galahad.ideasonboard.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751422AbeBWJgl (ORCPT ); Fri, 23 Feb 2018 04:36:41 -0500 From: Laurent Pinchart To: Sergei Shtylyov Cc: Laurent Pinchart , dri-devel@lists.freedesktop.org, linux-renesas-soc@vger.kernel.org Subject: Re: [PATCH v4 11/16] ARM: dts: r8a7791: Convert to new LVDS DT bindings Date: Fri, 23 Feb 2018 11:37:26 +0200 Message-ID: <3129673.WMkrksIBVj@avalon> In-Reply-To: <3469a2fd-f1d7-cf69-e028-053d6bdd5550@cogentembedded.com> References: <20180220231046.32638-1-laurent.pinchart+renesas@ideasonboard.com> <20180220231046.32638-12-laurent.pinchart+renesas@ideasonboard.com> <3469a2fd-f1d7-cf69-e028-053d6bdd5550@cogentembedded.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: Hi Sergei, On Friday, 23 February 2018 11:32:54 EET Sergei Shtylyov wrote: > On 2/21/2018 2:10 AM, Laurent Pinchart wrote: > > The internal LVDS encoder now has DT bindings separate from the DU. Port > > the device tree over to the new model. > > > > Signed-off-by: Laurent Pinchart > > > > --- > > Changes since v3: > > > > - Added power-domains and resets properties to LVDS nodes > > > > Changes since v2: > > > > - Fixed LVDS compatible string > > > > Changes since v1: > > > > - Remove the DU reg-names property > > --- > > > > arch/arm/boot/dts/r8a7791-koelsch.dts | 10 ++++++++-- > > arch/arm/boot/dts/r8a7791-porter.dts | 16 +++++++++++++--- > > arch/arm/boot/dts/r8a7791.dtsi | 36 +++++++++++++++++++++++++---- > > 3 files changed, 51 insertions(+), 11 deletions(-) > > > > diff --git a/arch/arm/boot/dts/r8a7791-koelsch.dts > > b/arch/arm/boot/dts/r8a7791-koelsch.dts index a50924d12b6f..c42ff25e3aa3 > > 100644 > > --- a/arch/arm/boot/dts/r8a7791-koelsch.dts > > +++ b/arch/arm/boot/dts/r8a7791-koelsch.dts > > @@ -337,8 +337,7 @@ > > > > clocks = <&cpg CPG_MOD 724>, <&cpg CPG_MOD 723>, <&cpg CPG_MOD 726>, > > > > <&x13_clk>, <&x2_clk>; > > > > - clock-names = "du.0", "du.1", "lvds.0", > > - "dclkin.0", "dclkin.1"; > > + clock-names = "du.0", "du.1", "dclkin.0", "dclkin.1"; > > Er, you only deleted the LVDS clock name but not LVDS clock itself? Oops :-/ Will be fixed in the next version. Does it show that I don't have a Koelsch board to run tests ? :-) > > diff --git a/arch/arm/boot/dts/r8a7791.dtsi > > b/arch/arm/boot/dts/r8a7791.dtsi index 008a260f86a5..8660f8a329c7 100644 > > --- a/arch/arm/boot/dts/r8a7791.dtsi > > +++ b/arch/arm/boot/dts/r8a7791.dtsi > > [...] -- Regards, Laurent Pinchart