Re: [PATCH v2 0/2] Fix dtc warnings when building the LAN966x device tree overlay
From: Philipp Zabel
Date: Mon Nov 04 2024 - 08:48:03 EST
On Do, 2024-10-31 at 16:01 +0100, Herve Codina wrote:
> Hi Philipp,
>
>
> On Tue, 29 Oct 2024 09:43:34 +0100
> Herve Codina <herve.codina@xxxxxxxxxxx> wrote:
>
> > dtc generates 3 kinds of warnings when it builds the LAN966x dtso.
> >
> > - missing or empty reg/ranges property
> > .../pci-ep-bus@0/cpu_clk: missing or empty reg/ranges property
> > .../pci-ep-bus@0/ddr_clk: missing or empty reg/ranges property
> > .../pci-ep-bus@0/sys_clk: missing or empty reg/ranges property
> > Patch 1 in this series fixes these warnings
> >
> > - Missing interrupt-parent
> > .../pci-ep-bus@0/oic@e00c0120: Missing interrupt-parent
> > This warning was quickly silenced by Philipp [1].
> > Patch 2 in this series fixes the warning and should replace the patch
> > applied by Philipp to silence the warning.
> >
> > - Warning (avoid_unnecessary_addr_size)
> > /fragment@0/__overlay__: unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" property
> > This warning should be fixed in dtc.
> > A patch has already be sent by Philip to fix it [2].
> >
> > [1] https://lore.kernel.org/all/57793bb01e02f03e215dfa6f8783df18034ae2ea.camel@xxxxxxxxxxxxxx/
> > [2] https://lore.kernel.org/devicetree-compiler/20241025161307.3629901-1-p.zabel@xxxxxxxxxxxxxx/T/#u
> >
>
> Both patches in this series have been reviewed by Rob.
>
> I think it is a green light to have them applied in the reset tree.
> Your opinion?
Applied to reset/next, thanks!
[1/2] misc: lan966x_pci: Fix dtc warns 'missing or empty reg/ranges property'
https://git.pengutronix.de/cgit/pza/linux/commit/?id=344ea0d36b8c
[2/2] misc: lan966x_pci: Fix dtc warn 'Missing interrupt-parent'
https://git.pengutronix.de/cgit/pza/linux/commit/?id=cf3e10cc0e88
I've dropped the Makefile patch ("misc: Silence warning when building
the LAN966x device tree overlay").
regards
Philipp