arm64: dts: broadcom: bcm4906-netgear-r8000p: Drop unnecessary "ranges" in partition node

[PATCH] arm64: dts: broadcom: bcm4906-netgear-r8000p: Drop unnecessary "ranges" in partition node

Posted by Rob Herring (Arm) 1 month ago

"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

Re: [PATCH] arm64: dts: broadcom: bcm4906-netgear-r8000p: Drop unnecessary "ranges" in partition node

Posted by Florian Fainelli 1 month ago


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