* [PATCH] arm64: dts: broadcom: bcm4906-netgear-r8000p: Drop unnecessary "ranges" in partition node
@ 2026-01-08 23:15 Rob Herring (Arm)
2026-01-09 2:18 ` Florian Fainelli
0 siblings, 1 reply; 2+ messages in thread
From: Rob Herring (Arm) @ 2026-01-08 23:15 UTC (permalink / raw)
To: Krzysztof Kozlowski, Conor Dooley, William Zhang, Anand Gore,
Kursad Oney, Florian Fainelli, Rafał Miłecki,
Broadcom internal kernel review list
Cc: devicetree, linux-arm-kernel, linux-kernel
"ranges" is only valid for MMIO addresses as it is used for translating
addresses to CPU address. Even if a partial translation was supported,
the DT is incorrect here as the nvmem-layout node would also need
"ranges". So drop "ranges" and the associated cell size properties.
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
One more dts fix to add after reworking the partition bindings.
arch/arm64/boot/dts/broadcom/bcmbca/bcm4906-netgear-r8000p.dts | 3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/arm64/boot/dts/broadcom/bcmbca/bcm4906-netgear-r8000p.dts b/arch/arm64/boot/dts/broadcom/bcmbca/bcm4906-netgear-r8000p.dts
index a5f9ec92bd5e..c6d76ba04903 100644
--- a/arch/arm64/boot/dts/broadcom/bcmbca/bcm4906-netgear-r8000p.dts
+++ b/arch/arm64/boot/dts/broadcom/bcmbca/bcm4906-netgear-r8000p.dts
@@ -146,9 +146,6 @@ partitions {
partition@0 {
label = "cferom";
reg = <0x0 0x100000>;
- #address-cells = <1>;
- #size-cells = <1>;
- ranges = <0 0x0 0x100000>;
nvmem-layout {
compatible = "fixed-layout";
--
2.51.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] arm64: dts: broadcom: bcm4906-netgear-r8000p: Drop unnecessary "ranges" in partition node
2026-01-08 23:15 [PATCH] arm64: dts: broadcom: bcm4906-netgear-r8000p: Drop unnecessary "ranges" in partition node Rob Herring (Arm)
@ 2026-01-09 2:18 ` Florian Fainelli
0 siblings, 0 replies; 2+ messages in thread
From: Florian Fainelli @ 2026-01-09 2:18 UTC (permalink / raw)
To: Rob Herring (Arm), Krzysztof Kozlowski, Conor Dooley,
William Zhang, Anand Gore, Kursad Oney, Rafał Miłecki,
Broadcom internal kernel review list
Cc: devicetree, linux-arm-kernel, linux-kernel
On 1/8/2026 3:15 PM, Rob Herring (Arm) wrote:
> "ranges" is only valid for MMIO addresses as it is used for translating
> addresses to CPU address. Even if a partial translation was supported,
> the DT is incorrect here as the nvmem-layout node would also need
> "ranges". So drop "ranges" and the associated cell size properties.
>
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Applied to
https://github.com/Broadcom/stblinux/commits/devicetree-arm64/next, thanks!
--
Florian
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-01-09 2:19 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-08 23:15 [PATCH] arm64: dts: broadcom: bcm4906-netgear-r8000p: Drop unnecessary "ranges" in partition node Rob Herring (Arm)
2026-01-09 2:18 ` Florian Fainelli
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox