Hi Rob, thanks for your feedback.
> From: Rob Herring <robh...@kernel.org> > Sent: 07 April 2021 19:27 > Subject: Re: [PATCH v5 2/5] media: dt-bindings: media: renesas,drif: > Convert to json-schema > > On Thu, Jan 14, 2021 at 7:02 AM Geert Uytterhoeven <ge...@linux-m68k.org> > wrote: > > > > Hi Fabrizio, Rob, > > > > On Wed, Oct 21, 2020 at 3:53 PM Fabrizio Castro > > <fabrizio.castro...@renesas.com> wrote: > > > Convert the Renesas DRIF bindings to DT schema and update > > > MAINTAINERS accordingly. > > > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro...@renesas.com> > > > Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad...@bp.renesas.com> > > > Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> > > > Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be> > > > Reviewed-by: Rob Herring <r...@kernel.org> > > > > Thanks for your patch! > > > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/media/renesas,drif.yaml > > > > > + clock-names: > > > + maxItems: 1 > > > + items: > > > + - const: fck > > > > With latest dt-schema, "make dt_binding_check" complains: > > > > Documentation/devicetree/bindings/media/renesas,drif.yaml: > > properties:clock-names:maxItems: False schema does not allow 1 > > Documentation/devicetree/bindings/media/renesas,drif.yaml: > > ignoring, error in schema: properties: clock-names: maxItems > > Seems this just got applied, and now this error is in linux-next. I'll send a patch to fix the problem shortly. Thanks, Fab > > > > > Using > > > > clock-names: > > const: fck > > > > Fixes that. > > > > However, I'm wondering why I do not get a complaint about the similar > > clock/clock-names in > > Documentation/devicetree/bindings/display/bridge/renesas,lvds.yaml. > > Because they're part of an else branch? > > Probably. if/then/else schemas have fewer checks as they can be > incomplete (only additional constraints on the top-level schema). > > Rob