From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kieran Bingham Subject: Re: [PATCH 3/8] dt-bindings: display: renesas,lvds: Add LVDS binding for D3 Date: Thu, 15 Feb 2018 08:45:33 +0000 Message-ID: References: <1518683903-10681-1-git-send-email-kbingham@kernel.org> <1518683903-10681-4-git-send-email-kbingham@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1518683903-10681-4-git-send-email-kbingham@kernel.org> Content-Language: en-GB Sender: linux-renesas-soc-owner@vger.kernel.org To: Kieran Bingham , linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: Laurent Pinchart , Simon Horman , geert@glider.be, Kieran Bingham , David Airlie , Rob Herring , Mark Rutland , Laurent Pinchart , "open list:DRM DRIVERS" , open list List-Id: devicetree@vger.kernel.org On 15/02/18 08:38, Kieran Bingham wrote: > From: Kieran Bingham > > The D3 supports two LVDS channels. Extend the binding to support them. > > Signed-off-by: Kieran Bingham > --- > Documentation/devicetree/bindings/display/bridge/renesas,lvds.txt | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/display/bridge/renesas,lvds.txt b/Documentation/devicetree/bindings/display/bridge/renesas,lvds.txt > index 79860f58a7ad..0dcf488b70df 100644 > --- a/Documentation/devicetree/bindings/display/bridge/renesas,lvds.txt > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,lvds.txt > @@ -14,6 +14,7 @@ Required properties: > - compatible : Shall contain one of > - "renesas,lvds-r8a7795" for R8A7795 (R-Car H3) compatible LVDS encoders > - "renesas,lvds-r8a7796" for R8A7796 (R-Car M3-W) compatible LVDS encoders > + - "renesas,lvds-r8a77995" for R8A77995 (R-Car D3) compatible LVDS encoders Hi Laurent, Are we unable to have a generic lvds-gen3 here? (Although to perhaps answer my own question I see that the D3/E3 have extra registers) Also "lets pretend" that I intentionally separated out the LVDS updates to the rcar_lvds_of_table :) -- Kieran > > - reg: Base address and length for the memory-mapped registers > - clocks: A phandle + clock-specifier pair for the functional clock >