Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753007AbdCMMnR (ORCPT ); Mon, 13 Mar 2017 08:43:17 -0400 Received: from mail.kernel.org ([198.145.29.136]:43584 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752917AbdCMMmQ (ORCPT ); Mon, 13 Mar 2017 08:42:16 -0400 Date: Mon, 13 Mar 2017 20:41:37 +0800 From: Shawn Guo To: Peter Senna Tschudin Cc: airlied@linux.ie, architt@codeaurora.org, akpm@linux-foundation.org, daniel.vetter@ffwll.ch, davem@davemloft.net, devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org, enric.balletbo@collabora.com, eballetbo@gmail.com, galak@codeaurora.org, gregkh@linuxfoundation.org, heiko@sntech.de, ijc+devicetree@hellion.org.uk, javier@dowhile0.org, jslaby@suse.cz, kernel@pengutronix.de, linux-arm-kernel@lists.infradead.org, linux@armlinux.org.uk, linux-kernel@vger.kernel.org, linux@roeck-us.net, mark.rutland@arm.com, martin.donnelly@ge.com, martyn.welch@collabora.co.uk, mchehab@osg.samsung.com, pawel.moll@arm.com, peter.senna@gmail.com, p.zabel@pengutronix.de, thierry.reding@gmail.com, rmk+kernel@armlinux.org.uk, robh+dt@kernel.org, tiwai@suse.com, treding@nvidia.com, laurent.pinchart@ideasonboard.com, Rob Herring , Fabio Estevam Subject: Re: [PATCH V3 4/4] dts/imx6q-b850v3: Use megachips-stdpxxxx-ge-b850v3-fw bridges (LVDS-DP++) Message-ID: <20170313124134.GF3618@dragon> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2854 Lines: 128 On Fri, Mar 03, 2017 at 04:57:10PM +0100, Peter Senna Tschudin wrote: > Configures the megachips-stdpxxxx-ge-b850v3-fw bridges on the GE > B850v3 dts file. > > Cc: Laurent Pinchart > Cc: Martyn Welch > Cc: Martin Donnelly > Cc: Javier Martinez Canillas > Cc: Enric Balletbo i Serra > Cc: Philipp Zabel > Cc: Rob Herring > Cc: Fabio Estevam > Signed-off-by: Peter Senna Tschudin > --- > Unchanged since V1. > > arch/arm/boot/dts/imx6q-b850v3.dts | 68 ++++++++++++++++++++++++++++++++++++++ > 1 file changed, 68 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6q-b850v3.dts b/arch/arm/boot/dts/imx6q-b850v3.dts > index b237429..3ec54da 100644 > --- a/arch/arm/boot/dts/imx6q-b850v3.dts > +++ b/arch/arm/boot/dts/imx6q-b850v3.dts > @@ -72,6 +72,13 @@ > fsl,data-mapping = "spwg"; > fsl,data-width = <24>; > status = "okay"; > + > + port@4 { > + reg = <4>; Have a newline between property list and child node. > + lvds0_out: endpoint { > + remote-endpoint = <&stdp4028_in>; > + }; > + }; > }; > }; > > @@ -146,3 +153,64 @@ > &usdhc2 { > status = "disabled"; > }; > + > +&mux2_i2c2 { > + status = "okay"; This okay status doesn't seem really necessary to me. If it does, please put it at the end of property list. > + clock-frequency = <100000>; > + > + stdp4028@73 { > + compatible = "megachips,stdp4028-ge-b850v3-fw"; > + #address-cells = <1>; > + #size-cells = <0>; > + Drop this newline, as in general we do not add newline in middle of property list. > + reg = <0x73>; > + Ditto > + interrupt-parent = <&gpio2>; > + interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + stdp4028_in: endpoint { > + remote-endpoint = <&lvds0_out>; > + }; > + }; Have a newline between nodes. > + port@1 { > + reg = <1>; > + stdp4028_out: endpoint { > + remote-endpoint = <&stdp2690_in>; > + }; > + }; > + }; > + }; > + > + stdp2690@72 { Sort device nodes under bus in order of unit address. That said, stdp2690@72 should go before stdp4028@73. Shawn > + compatible = "megachips,stdp2690-ge-b850v3-fw"; > + #address-cells = <1>; > + #size-cells = <0>; > + > + reg = <0x72>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + stdp2690_in: endpoint { > + remote-endpoint = <&stdp4028_out>; > + }; > + }; > + > + port@1 { > + reg = <1>; > + stdp2690_out: endpoint { > + /* Connector for external display */ > + }; > + }; > + }; > + }; > +}; > -- > 2.9.3 >