From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752004AbbDIHUf (ORCPT ); Thu, 9 Apr 2015 03:20:35 -0400 Received: from mail-wi0-f177.google.com ([209.85.212.177]:33859 "EHLO mail-wi0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751592AbbDIHUb (ORCPT ); Thu, 9 Apr 2015 03:20:31 -0400 Date: Thu, 9 Apr 2015 09:20:26 +0200 From: Thierry Reding To: Liu Ying Cc: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux@arm.linux.org.uk, kernel@pengutronix.de, p.zabel@pengutronix.de, shawn.guo@linaro.org, mturquette@linaro.org, airlied@linux.ie, andy.yan@rock-chips.com, stefan.wahren@i2se.com, a.hajda@samsung.com, sboyd@codeaurora.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH RFC v9 14/20] Documentation: dt-bindings: Add bindings for Himax HX8369A DRM panel driver Message-ID: <20150409072025.GB12103@ulmo> References: <1423720903-24806-1-git-send-email-Ying.Liu@freescale.com> <1423720903-24806-15-git-send-email-Ying.Liu@freescale.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="v9Ux+11Zm5mwPlX6" Content-Disposition: inline In-Reply-To: <1423720903-24806-15-git-send-email-Ying.Liu@freescale.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --v9Ux+11Zm5mwPlX6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Feb 12, 2015 at 02:01:37PM +0800, Liu Ying wrote: > This patch adds device tree bindings for Himax HX8369A DRM panel driver. >=20 > Signed-off-by: Liu Ying > --- > v8->v9: > * Rebase onto the imx-drm/next branch of Philipp Zabel's open git reposi= tory. >=20 > v7->v8: > * None. >=20 > v6->v7: > * None. >=20 > v5->v6: > * None. >=20 > v4->v5: > * Merge the bs[3:0]-gpios properties into one property - bs-gpios. > This addresses Andrzej Hajda's comment. >=20 > v3->v4: > * Newly introduced in v4. This is separated from the relevant driver pa= tch > in v3 to address Stefan Wahren's comment. >=20 > .../devicetree/bindings/panel/himax,hx8369a.txt | 39 ++++++++++++++++= ++++++ > 1 file changed, 39 insertions(+) > create mode 100644 Documentation/devicetree/bindings/panel/himax,hx8369a= =2Etxt >=20 > diff --git a/Documentation/devicetree/bindings/panel/himax,hx8369a.txt b/= Documentation/devicetree/bindings/panel/himax,hx8369a.txt > new file mode 100644 > index 0000000..3a44b70 > --- /dev/null > +++ b/Documentation/devicetree/bindings/panel/himax,hx8369a.txt > @@ -0,0 +1,39 @@ > +Himax HX8369A WVGA 16.7M color TFT single chip driver with internal GRAM > + > +Himax HX8369A is a WVGA resolution driving controller. > +It is designed to provide a single chip solution that combines a source > +driver and power supply circuits to drive a TFT dot matrix LCD with > +480RGBx864 dots at the maximum. > + > +The HX8369A supports several interface modes, including MPU MIPI DBI Type > +A/B mode, MIPI DPI/DBI Type C mode, MIPI DSI video mode, MIPI DSI command > +mode and MDDI mode. The interface mode is selected by the external hardw= are > +pins BS[3:0]. > + > +Currently, only the MIPI DSI video mode is supported. This doesn't make much sense. The binding is supposed to describe the hardware, so saying "only video mode is supported" is weird. Perhaps if you have no way to test other modes you could reword as: This version of the device tree binding only specifies MIPI DSI video mode. Then again, would the device tree content be different based on the video mode? > +Required properties: > + - compatible: should be a panel's compatible string I don't understand this. If this chip isn't a panel, why should the compatible string contain the panel's compatible string? Is this some kind of bridge chip that can drive different panels? I guess I'll see when I look at the driver. Anyway, if it isn't a panel it shouldn't have the panel's compatible string. > + - reg: the virtual channel number of a DSI peripheral, as described in= [1] > + - reset-gpios: a GPIO spec for the reset pin, as described in [2] > + > +Optional properties: > + - vdd1-supply: I/O and interface power supply > + - vdd2-supply: analog power supply > + - vdd3-supply: logic power supply > + - dsi-vcc-supply: DSI and MDDI power supply > + - vpp-supply: OTP programming voltage > + - bs-gpios: a GPIO spec for the pins BS[3:0], as described in [2] > + > +[1] Documentation/devicetree/bindings/mipi/dsi/mipi-dsi-bus.txt > +[2] Documentation/devicetree/bindings/gpio/gpio.txt > + > +Example: > + panel { > + compatible =3D "truly,tft480800-16-e-dsi"; > + reg =3D <0>; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&pinctrl_mipi_panel>; > + reset-gpios =3D <&gpio6 11 GPIO_ACTIVE_LOW>; > + bs-gpios =3D <0>, <0>, <&gpio6 14 GPIO_ACTIVE_HIGH>, <0>; > + }; Again this doesn't make sense. You're mixing two things here: the HIMAX chip that is presumably a bridge and the panel connected to it. Thierry --v9Ux+11Zm5mwPlX6 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAABCAAGBQJVJig4AAoJEN0jrNd/PrOh+4EP/RkR8rV2vF6yai828m1fQT37 C8bAlZxC0kHk9emfTK7Ye8Lv7lqWrJrDsGW7B7cXLJZPkbiILLDxt4pm9US7lx1Q Zpp231URjpAXcfQvquD2Ul7I1jGAF2+TwXLknlenBYBQupClTTeg4EjotbD0PxGR WKA7kR0p/Mp+GnhGBc52w+t3k39ProfIo1Q6goKtBYYuFmh/N7pJ0vpRnpg52S1b PwfepnHwkNnXwVxiHPnk4I29vEW6FCSAMEBJjTEvdVZhiHiiouyBXYir7GTNkVpC 8KwcxQyvVCA5lbxEIi748j+BkffU5Ium1h7aGp1Ho3HarZLCdGnQsOp5sbBh37Zf IKRfnuZt1k8cJ9Nn4KVGglYwiy6bS0ocOxDaedEH/NTF6IkVoh6yKAZc3vRVIrvY XCUcxGaRUh6w3tKzDVht8wgrWRgmRqitYP/rK7jv1RiKDHOWXbM/LbdQh8HraZ3C 56B2ASOctL1ZFa1x4BtbMVBZLx+B7I0MmBcUGDn7aUJoqOWVw1aYRXI0jUtXk8Dm HcH6FvqKtHdbfz2HxK64Lvkn30rO98J47AMDquDg40uCQzqTGG1EF1t110RP/DH5 5N1KVJBiyHwg+CNQ4sEIFJklhcwB5YV9yhrBSNJV3oSdLdm6Hh2olLHMLNNVQJ74 VREGSaJIpwI5NxbiGUUC =WNwu -----END PGP SIGNATURE----- --v9Ux+11Zm5mwPlX6--