Messages in this thread | | | From | Laurent Pinchart <> | Subject | Re: [PATCH 3/8] dt-bindings: display: renesas,lvds: Add LVDS binding for D3 | Date | Thu, 15 Feb 2018 16:02:49 +0200 |
| |
Hi Kieran,
Thank you for the patch.
On Thursday, 15 February 2018 10:38:18 EET Kieran Bingham wrote: > From: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > > The D3 supports two LVDS channels. Extend the binding to support them. > > Signed-off-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> > --- > 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
This should be "renesas,r8a77995-lvds". I've already fixed the series you've based this patch on.
Apart from this,
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
and taken in my tree with the compatible string fixed.
> > - reg: Base address and length for the memory-mapped registers > - clocks: A phandle + clock-specifier pair for the functional clock
-- Regards,
Laurent Pinchart
| |