Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751174AbdCPCfy (ORCPT ); Wed, 15 Mar 2017 22:35:54 -0400 Received: from mail.kernel.org ([198.145.29.136]:60108 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750885AbdCPCfw (ORCPT ); Wed, 15 Mar 2017 22:35:52 -0400 Date: Thu, 16 Mar 2017 10:34:59 +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 V4] dts/imx6q-b850v3: Use megachips-stdpxxxx-ge-b850v3-fw bridges (LVDS-DP++) Message-ID: <20170316023457.GU3618@dragon> References: <20170314092302.7317-1-peter.senna@collabora.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170314092302.7317-1-peter.senna@collabora.com> 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: 2895 Lines: 133 On Tue, Mar 14, 2017 at 10:23:02AM +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 To match i.MX patch style, the subject prefix should be something like 'ARM: dts: imx6q-b850v3: ...'. > --- > Subject was [PATCH V4 4/4] ... > But patches 1, 2, and 3 are already upstream and were not resent. > > Changes from V3: > - Newline fixes > - Removed 'status = "okay";' from '&mux2_i2c2' > - Sorted device nodes in order of unit address > > Unchanged since V1. > arch/arm/boot/dts/imx6q-b850v3.dts | 66 ++++++++++++++++++++++++++++++++++++++ > 1 file changed, 66 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6q-b850v3.dts b/arch/arm/boot/dts/imx6q-b850v3.dts > index d78514c..ef67b47 100644 > --- a/arch/arm/boot/dts/imx6q-b850v3.dts > +++ b/arch/arm/boot/dts/imx6q-b850v3.dts > @@ -72,6 +72,14 @@ > fsl,data-mapping = "spwg"; > fsl,data-width = <24>; > status = "okay"; > + > + port@4 { > + reg = <4>; > + Great, it's been fixed. > + lvds0_out: endpoint { > + remote-endpoint = <&stdp4028_in>; > + }; > + }; > }; > }; > > @@ -142,3 +150,61 @@ > reg = <0x4a>; > }; > }; > + > +&mux2_i2c2 { > + clock-frequency = <100000>; > + > + stdp2690@72 { > + 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>; But this one ... > + stdp2690_in: endpoint { > + remote-endpoint = <&stdp4028_out>; > + }; > + }; > + > + port@1 { > + reg = <1>; ... here ... > + stdp2690_out: endpoint { > + /* Connector for external display */ > + }; > + }; > + }; > + }; > + > + stdp4028@73 { > + compatible = "megachips,stdp4028-ge-b850v3-fw"; > + #address-cells = <1>; > + #size-cells = <0>; > + reg = <0x73>; > + interrupt-parent = <&gpio2>; > + interrupts = <0 IRQ_TYPE_LEVEL_HIGH>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; ... here ... > + stdp4028_in: endpoint { > + remote-endpoint = <&lvds0_out>; > + }; > + }; > + > + port@1 { > + reg = <1>; ... and here. I fixed up all of them and applied patch. Shawn > + stdp4028_out: endpoint { > + remote-endpoint = <&stdp2690_in>; > + }; > + }; > + }; > + }; > +}; > -- > 2.9.3 >