From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753265AbeADTDj (ORCPT + 1 other); Thu, 4 Jan 2018 14:03:39 -0500 Received: from mail.free-electrons.com ([62.4.15.54]:55864 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752663AbeADTDg (ORCPT ); Thu, 4 Jan 2018 14:03:36 -0500 Date: Thu, 4 Jan 2018 20:03:24 +0100 From: Maxime Ripard To: Jernej =?utf-8?Q?=C5=A0krabec?= Cc: Daniel Vetter , David Airlie , Chen-Yu Tsai , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Mark Rutland , Rob Herring , linux-arm-kernel@lists.infradead.org, plaes@plaes.org, icenowy@aosc.io, Thomas Petazzoni , devicetree@vger.kernel.org, thierry.reding@gmail.com, Laurent Pinchart Subject: Re: [PATCH v5 03/12] dt-bindings: display: sun4i-drm: Add LVDS properties Message-ID: <20180104190324.zrrsmdamd7ch2qe5@flea.lan> References: <1927811.df6406f6u0@jernej-laptop> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="ldgtus3ucqyhfy4m" Content-Disposition: inline In-Reply-To: <1927811.df6406f6u0@jernej-laptop> User-Agent: NeoMutt/20171208 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: --ldgtus3ucqyhfy4m Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Sat, Dec 30, 2017 at 12:45:19PM +0100, Jernej =C5=A0krabec wrote: > Hi Maxime, >=20 > Dne =C4=8Detrtek, 21. december 2017 ob 12:02:29 CET je Maxime Ripard napi= sal(a): > > Some clocks and resets supposed to drive the LVDS logic in the display > > engine have been overlooked when the driver was first introduced. > >=20 > > Add those additional resources to the binding, and we'll deal with the = ABI > > stability in the code. > >=20 > > Reviewed-by: Chen-Yu Tsai > > Reviewed-by: Rob Herring > > Signed-off-by: Maxime Ripard > > --- > > Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt | 9 +++= ++++- > > 1 file changed, 9 insertions(+) > >=20 > > diff --git a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.= txt > > b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt index > > 50cc72ee1168..1e21cfaac9e2 100644 > > --- a/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > +++ b/Documentation/devicetree/bindings/display/sunxi/sun4i-drm.txt > > @@ -121,6 +121,15 @@ Required properties: > > On SoCs other than the A33 and V3s, there is one more clock required: > > - 'tcon-ch1': The clock driving the TCON channel 1 > >=20 > > +On SoCs that support LVDS (all SoCs but the A13, H3, H5 and V3s), you > > +need one more reset line: > > + - 'lvds': The reset line driving the LVDS logic > > + > > +And on the SoCs newer than the A31 (sun6i and sun8i families), you > > +need one more clock line: > > + - 'lvds-alt': An alternative clock source, separate from the TCON > > channel 0 + clock, that can be used to drive the LVDS c= lock >=20 > I think this wording is imprecise, since A83T is part of the sun8i family= , but=20 > from the code (patch 7) and DT changes (patch 9) you do, it doesn't need = this=20 > property. >=20 > Maybe it would be just easier to enumerate all compatibles which needs th= is=20 > property?=20 You're right, but the rest of the document uses the SoC name instead. In order to remain consistent, I listed the (currently supported) SoCs that need that property and applied that patch. Thanks! Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com --ldgtus3ucqyhfy4m Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEE0VqZU19dR2zEVaqr0rTAlCFNr3QFAlpOenwACgkQ0rTAlCFN r3RxhQ//YyAWFnFRiS3JBUiX3MTXPY82O6mXnq3lXs6n6z3SlLlJavs4A0WKZe9c KerJTXCT/iA5LAiKfqnY8aymVh9bgowScie/0Cpn7YovttlSrheexVM0TQFq8vTz 1mKwOrvjH1b4C2yKZfwL6T8TKPGIFwW9SIJ6sD2XliEhjAKGLDEumakVVOlpK+aE jHO4B4rlPyrv1qZaVcAFLwHXvzpYqsPdtEprvgPPmME/+PjuBtKpvWk86EKjlejc 7P3IvOSgFvAbtPJfia+MRjLljgK/O/hVwwL7OzAWqed7QiGJizEWrWo+HB7H2/YK b7qz+8JmhV/7Jfyi7zQ/CJk8eN0O+FfgjX/j1sEP3v0fSeQvprWn24eeKMfPg6ag wT7xIPiwF/9tVj2+/kF2H2V0A47nY/L0Ltw+lf6JHMmcHr/VXD4qofgDphYU5utM Dblgw2jJ4Z4YtryuiKtgJWy2wHUqT2zgBCppWmX75fDxt1VKET5he2wSvvAISnyq ca/fj5xpTxhxyHyqYuLOI+SsSII3TU7geOs1jdFv8ZyYWxc6l/sY5Ng0q1h2iDKp 5BobXEH7GnBWU+N5NoJVqADRo0KagsBsbM5VNeX2pOljyiJ3e+ZhYo+ArdQxiFN2 CkjemSKYfg26uJK2o9MadzdjIMGoRGLmB9+yQ5XNAQN27zqozmM= =NF2V -----END PGP SIGNATURE----- --ldgtus3ucqyhfy4m--