Re: [PATCH v3 02/14] dt-bindings: display, renesas,du: Document cmms property

From: Laurent Pinchart
Date: Wed Aug 28 2019 - 04:28:57 EST


On Wed, Aug 28, 2019 at 09:32:23AM +0200, Geert Uytterhoeven wrote:
> On Tue, Aug 27, 2019 at 10:29 PM Rob Herring <robh@xxxxxxxxxx> wrote:
> > On Sun, Aug 25, 2019 at 03:51:42PM +0200, Jacopo Mondi wrote:
> > > Document the newly added 'cmms' property which accepts a list of phandle
> > > and channel index pairs that point to the CMM units available for each
> > > Display Unit output video channel.
> > >
> > > Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>
> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
> > > ---
> > > Documentation/devicetree/bindings/display/renesas,du.txt | 5 +++++
> > > 1 file changed, 5 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/display/renesas,du.txt b/Documentation/devicetree/bindings/display/renesas,du.txt
> > > index c97dfacad281..c2265e2a1af2 100644
> > > --- a/Documentation/devicetree/bindings/display/renesas,du.txt
> > > +++ b/Documentation/devicetree/bindings/display/renesas,du.txt
> > > @@ -45,6 +45,10 @@ Required Properties:
> > > instance that serves the DU channel, and the channel index identifies the
> > > LIF instance in that VSP.
> > >
> > > + - cmms: A list of phandles to the CMM instances present in the SoC, one
> > > + for each available DU channel. The property shall not be specified for
> > > + SoCs that do not provide any CMM (such as V3M and V3H).
> >
> > renesas,cmms
>
> So I guess we really wanted to have the prefix for the vsps property, too?

Yes, we should have :-( My bad.

--
Regards,

Laurent Pinchart