Re: [PATCH 1/2] dt-bindings: kbuild: Pass DT_SCHEMA_FILES to dt-validate

[Geert Uytterhoeven]

Hi Rob,

On Fri, Mar 4, 2022 at 3:31 PM Rob Herring <robh@xxxxxxxxxx> wrote:
> On Fri, Mar 04, 2022 at 03:05:13PM +0100, Geert Uytterhoeven wrote:
> > On Fri, Mar 4, 2022 at 2:57 PM Rob Herring <robh@xxxxxxxxxx> wrote:
> > > On Fri, Mar 04, 2022 at 10:32:29AM +0100, Geert Uytterhoeven wrote:
> > > > On Thu, Mar 3, 2022 at 11:43 PM Rob Herring <robh@xxxxxxxxxx> wrote:
> > > > > In preparation for supporting validation of DTB files, the full
> > > > > processed schema will always be needed in order to extract type
> > > > > information from it. Therefore, the processed schema containing only
> > > > > what DT_SCHEMA_FILES specifies won't work. Instead, dt-validate has
> > > > > gained an option, -l or --limit, to specify which schema(s) to use for
> > > > > validation.
> > > > >
> > > > > As the command line option is new, we the minimum dtschema version must be
> > > > > updated.
> > > > >
> > > > > Cc: Masahiro Yamada <masahiroy@xxxxxxxxxx>
> > > > > Signed-off-by: Rob Herring <robh@xxxxxxxxxx>

> > With this series applied, the various salvator-xs DTS files are now
> > throwing up:
> >
> >     sata: size (19) error for type phandle
> >     backlight: size (11) error for type phandle
>
> Those come from the code decoding the properties[1]. Unfortunately, I
> haven't come up with a prettier way to report those with the filename. I
> may just remove it because if decoding the property fails, we'll get
> schema errors later on anyways.
>
> But I don't see any 'sata' properties in the DTS files and 'backlight'
> is a node. Are you building with '-@'? I probably need to skip
> __symbols__ nodes. The overlay side is handled because examples are
> built as overlays (to allow unresolved phandles).

Yes I am. Dropping the "-@" fixes the issue.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds