Re: [PATCH v5 3/5] dt-bindings: wireless: bcm4329-fmac: Use wireless-controller.yaml schema

From: Johannes Berg
Date: Thu Apr 24 2025 - 15:10:19 EST


On Thu, 2025-04-24 at 14:04 -0500, Rob Herring wrote:
> On Thu, Apr 24, 2025 at 1:39 PM Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:
> >
> > On Thu, 2025-04-24 at 13:26 -0500, Rob Herring wrote:
> > > While it seems the reviews of the series caused more warnings for
> > > Apple, in general, schemas creating warnings is not breaking things.
> > > In a way, the whole point is to create warnings because if the .dts
> > > files were perfect already we wouldn't need schemas. The main
> > > requirement for schemas is only that they don't create warnings for
> > > the examples. There's still too many for .dts files to worry about it
> > > (and there's intermittent warnings from things getting merged via
> > > different trees).
> > >
> >
> > Oh, sure, but now if you want to apply the fixes you probably have to
> > wait for the broken patches in my tree to percolate all the way through
> > to Linus, then back to your tree, and then you can apply the fixes?
>
> No, I never take .dts changes. They all go via the individual platform
> trees. It's a bit weird if Krzysztof refers to the commit that's not
> in the linear history, but that shouldn't hold things up. The issues
> exist with or without the schema change. They might even be backported
> to stable while the schema change won't be.

Ah. OK, so never mind then. I'll assume whatever Krzysztof did will go
through some other tree and just go on with things :)

Thanks for the explanations!

johannes