Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753751AbcD1Vsb (ORCPT ); Thu, 28 Apr 2016 17:48:31 -0400 Received: from mail.kernel.org ([198.145.29.136]:60948 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753687AbcD1Vs2 (ORCPT ); Thu, 28 Apr 2016 17:48:28 -0400 Date: Thu, 28 Apr 2016 16:48:23 -0500 From: Rob Herring To: Akshay Bhat Cc: p.zabel@pengutronix.de, airlied@linux.ie, steve_longerbeam@mentor.com, dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 2/2] dt-bindings: imx: ldb: Add ddc-i2c-bus property Message-ID: <20160428214823.GA841@rob-hp-laptop> References: <1461788614-7784-1-git-send-email-akshay.bhat@timesys.com> <1461788614-7784-2-git-send-email-akshay.bhat@timesys.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1461788614-7784-2-git-send-email-akshay.bhat@timesys.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1289 Lines: 35 On Wed, Apr 27, 2016 at 04:23:34PM -0400, Akshay Bhat wrote: > Document the ddc-i2c-bus property used by imx-ldb driver to read EDID > information via I2C interface. > > Signed-off-by: Akshay Bhat > --- > > v3: > Newly added. > > Documentation/devicetree/bindings/display/imx/ldb.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/display/imx/ldb.txt b/Documentation/devicetree/bindings/display/imx/ldb.txt > index 0a175d9..a407462 100644 > --- a/Documentation/devicetree/bindings/display/imx/ldb.txt > +++ b/Documentation/devicetree/bindings/display/imx/ldb.txt > @@ -62,6 +62,7 @@ Required properties: > display-timings are used instead. > > Optional properties (required if display-timings are used): The required part doesn't make sense if you add this, but... > + - ddc-i2c-bus: phandle of an I2C controller used for DDC EDID probing Really, this should be part of a connector node since i2c goes from the i2c controller to a connector and is not part of the display controller. > - display-timings : A node that describes the display timings as defined in > Documentation/devicetree/bindings/display/display-timing.txt. > - fsl,data-mapping : should be "spwg" or "jeida" > -- > 2.8.1 >