Message ID | 20210721140424.725744-1-maxime@cerno.tech (mailing list archive) |
---|---|
Headers | show |
Series | ARM: dts: Last round of DT schema fixes | expand |
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. Rob
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