Re: [PATCH] checks: Avoid warnings for reg override in __overlay__

Brian Norris <[email protected]> Wed, 17 Jun 2026 16:35:47 -0700
Newsgroups org.kernel.vger.devicetree-compiler
Message-ID <[email protected]>
Hi Rob,

On Wed, Jun 17, 2026 at 05:54:42PM -0500, Rob Herring wrote:
> I would argue (and did the last time this came up IIRC) the overlay

I gave a quick look for prior art, but didn't go far enough. I see this
was a similar conversation:

[PATCH v2] checks: Suppress warnings on overlay fragments
https://lore.kernel.org/all/[email protected]/

I don't think it had a satisfying conclusion though.

> should target the parent node instead and then you can put in
> #address-cells and #size-cells in the overlay to make it pass checks
> (and make 'reg' parsable without applying the overlay).

This implies we can't actually target the appropriate node via phandle
any more, and so we lose the ergonomics that phandles provide. Where
previouly an overlay could be resilient to node renaming and other sorts
of incompatibilities between a dtb and a dtbo (the overlay wouldn't
apply if &foo isn't found), now we'd have to open-code the node name and
maybe even its parent node name in the overlay. If either of those were
wrong ... we wouldn't notice at all, unless there's an obvious
functional breakage as a result.

I acknowledge that it's difficult to parse and validate dtbos when they
are low on context. But I don't see why we should emit false warnings
for the possibly-correct, and more ergnomic approach.

Brian