Hi Geert,
On Thu, May 14, 2020 at 09:02:54PM +0200, Geert Uytterhoeven wrote:
> On Thu, May 14, 2020 at 5:17 PM Laurent Pinchart wrote:
> > On Thu, May 14, 2020 at 09:31:53AM +0200, Geert Uytterhoeven wrote:
> > > On Thu, May 14, 2020 at 1:29 AM Laurent Pinchart wrote:
> > > > Convert the Renesas R-Car LVDS encoder text binding to YAML.
> > > >
> > > > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>
> > > > Acked-by: Maxime Ripard <mripard@xxxxxxxxxx>
> > >
> > > > --- /dev/null
> > > > +++ b/Documentation/devicetree/bindings/display/bridge/renesas,lvds.yaml
> > >
> > > > +examples:
> > > > + - |
> > > > + #include <dt-bindings/clock/renesas-cpg-mssr.h>
> > > > + #include <dt-bindings/power/r8a7795-sysc.h>
> > > > +
> > > > + lvds@feb90000 {
> > > > + compatible = "renesas,r8a7795-lvds";
> > > > + reg = <0 0xfeb90000 0 0x14>;
> > >
> > > #{address,size}-cells = <1> for examples.
> > > Applies to all nodes below, too.
> >
> > Why ?
>
> See "[PATCH 5/5] dt-bindings: Fix incorrect 'reg' property sizes"
> https://lore.kernel.org/linux-devicetree/20200512204543.22090-5-robh@xxxxxxxxxx/
I don't like this much as it creates invalid examples, but until we have
a better solution, I'll reduce the number of cells.
--
Regards,
Laurent Pinchart
