Re: [PATCH 00/54] ARM: dts: Last round of DT schema fixes

Maxime Ripard
2021-07-29T12:05:49Z

On Wed, Jul 21, 2021 at 10:48:26AM -0600, Rob Herring wrote:
> On Wed, Jul 21, 2021 at 04:03:30PM +0200, Maxime Ripard wrote:
> > Hi,
> >
> > Here's another round of schema warnings fixes for the Allwinner platform.
> >
> > There's a fair share of new schemas in there since the schema tools now warn
> > when a compatible is not documented in a schema.
> >
> > We don't have any warning anymore if we use the OPP binding Rob submitted, and
> > since that means we have all our devices properly validated I don't expect more
> > fixes now, aside from the usual bunch of regressions.
>
> Great! You deserve a prize.
>
> I want to start enabling the schema checks by default. (Though then I'd
> have to worry about new warnings.) This should be pretty easy to do
> where we have subdirs per family, but for arm32 we'd need to move dts
> files to subdirs if we don't want a flag per dts file.

That's definitely something that I'd be interested in. It's very easy
for a warning / error to slip through during review, so having some
builders somewhere reporting new issues would be awesome.

On arm32 though, we have per-family defconfig so maybe we can opt-in by
defconfig? It should work in most case

Maxime