Re: [PATCH 6/4] dt-bindings: display: renesas: lvds: RZ/G2E needs renesas,companion too

From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Geert Uytterhoeven <geert@linux-m68k.org>,
	Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>,
	"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
	<devicetree@vger.kernel.org>, Rob Herring <robh+dt@kernel.org>,
	Fabrizio Castro <fabrizio.castro@bp.renesas.com>,
	DRI Development <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 6/4] dt-bindings: display: renesas: lvds: RZ/G2E needs renesas,companion too
Date: Tue, 30 Jun 2020 02:42:04 +0300	[thread overview]
Message-ID: <20200629234204.GJ21452@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20200629081140.GG227119@ravnborg.org>

Hi Sam,

On Mon, Jun 29, 2020 at 10:11:40AM +0200, Sam Ravnborg wrote:
> On Thu, May 14, 2020 at 08:44:19AM +0200, Geert Uytterhoeven wrote:
> > On Thu, May 14, 2020 at 1:39 AM Laurent Pinchart wrote:
> > > From: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > >
> > > Document RZ/G2E support for property renesas,companion.
> > >
> > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> > > Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> > 
> > > --- a/Documentation/devicetree/bindings/display/bridge/renesas,lvds.yaml
> > > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,lvds.yaml
> > > @@ -81,9 +81,9 @@ properties:
> > >      $ref: /schemas/types.yaml#/definitions/phandle
> > >      description:
> > >        phandle to the companion LVDS encoder. This property is mandatory
> > > -      for the first LVDS encoder on D3 and E3 SoCs, and shall point to
> > > -      the second encoder to be used as a companion in dual-link mode. It
> > > -      shall not be set for any other LVDS encoder.
> > > +      for the first LVDS encoder on D3, R-Car E3 and RZ/G2E SoCs, and shall
> > 
> > R-Car D3 and E3, and RZ/G2E SoCs
> > 
> > > +      point to the second encoder to be used as a companion in dual-link mode.
> > > +      It shall not be set for any other LVDS encoder.
> 
> Laurent, I assume you will re-spin a ne version wth the changelog
> corrected.

Yes I will. Thanks for checking.

-- 
Regards,

Laurent Pinchart