[PATCH 1/3 v4] drm/panel: Add generic DSI panel YAML bindings
Rob Herring
robh at kernel.org
Wed Oct 30 19:45:54 UTC 2019
On Wed, Oct 30, 2019 at 8:50 AM Linus Walleij <linus.walleij at linaro.org> wrote:
>
> On Fri, Oct 25, 2019 at 9:20 PM Rob Herring <robh at kernel.org> wrote:
>
> > [Me]
> > > + dsi-controller at 55aa55aa {
> > > + compatible = "acme,foo";
> >
> > Eventually, I want to start reporting if compatible strings aren't
> > documented (i.e. matched to a schema). This will break then...
>
> OK I drop this.
>
> > > + reg = <0x55aa55aa>;
> >
> > This will fail because a size cell is expected.
>
> Do you mean it will fail the day we actually check reg properties to
> be properly formed under the circumstances?
>
> Because:
> HOSTLD scripts/dtc/dtc
> CHKDT Documentation/devicetree/bindings/display/panel/sony,acx424akp.yaml
> CHKDT Documentation/devicetree/bindings/display/dsi-controller.yaml
> SCHEMA Documentation/devicetree/bindings/processed-schema.yaml
> (...)
>
> It passes just fine.
Yes, but the examples are built and validated after this. With the
other errors currently, you're erroring out before this happens.
> I get a bit nervous since the schema is all about being strict about
> stuff so I get the feeling that "should have failed".
It's actually dtc that will complain here because it's aware of
#*-cells values. Can't do that with schema.
Rob
More information about the dri-devel
mailing list