diff mbox series

ARM: dts: BCM5301X: Add missing "ranges property for PCIe nodes

Message ID 20220920210213.3268525-1-f.fainelli@gmail.com (mailing list archive)
State New, archived
Headers show
Series ARM: dts: BCM5301X: Add missing "ranges property for PCIe nodes | expand

Commit Message

Florian Fainelli Sept. 20, 2022, 9:02 p.m. UTC
Add empty ranges properties to silence such warnings:

arch/arm/boot/dts/bcm5301x.dtsi:240.21-246.5: Warning (pci_bridge): /axi@18000000/pcie@12000: missing ranges for PCI bridge (or not a bridge)
arch/arm/boot/dts/bcm5301x.dtsi:248.21-254.5: Warning (pci_bridge): /axi@18000000/pcie@13000: missing ranges for PCI bridge (or not a bridge)
arch/arm/boot/dts/bcm5301x.dtsi:256.21-262.5: Warning (pci_bridge): /axi@18000000/pcie@14000: missing ranges for PCI bridge (or not a bridge)

to silence warnings. This is not a functional issue since the driver does not
make use of the OF infrastructure for registering the PCIe controller(s) on
this chip, but uses the BCMA bus.

Fixes: 61dc1e3850a6 ("ARM: dts: BCM5301X: Add basic PCI controller properties")
Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
Change-Id: Ia4eba6d21b72620d2fd8d5c3548c1777d3c1c357
---
 arch/arm/boot/dts/bcm5301x.dtsi | 3 +++
 1 file changed, 3 insertions(+)

Comments

Florian Fainelli Sept. 20, 2022, 9:03 p.m. UTC | #1
On 9/20/22 14:02, Florian Fainelli wrote:
> Add empty ranges properties to silence such warnings:
> 
> arch/arm/boot/dts/bcm5301x.dtsi:240.21-246.5: Warning (pci_bridge): /axi@18000000/pcie@12000: missing ranges for PCI bridge (or not a bridge)
> arch/arm/boot/dts/bcm5301x.dtsi:248.21-254.5: Warning (pci_bridge): /axi@18000000/pcie@13000: missing ranges for PCI bridge (or not a bridge)
> arch/arm/boot/dts/bcm5301x.dtsi:256.21-262.5: Warning (pci_bridge): /axi@18000000/pcie@14000: missing ranges for PCI bridge (or not a bridge)
> 
> to silence warnings. This is not a functional issue since the driver does not
> make use of the OF infrastructure for registering the PCIe controller(s) on
> this chip, but uses the BCMA bus.
> 
> Fixes: 61dc1e3850a6 ("ARM: dts: BCM5301X: Add basic PCI controller properties")
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> Change-Id: Ia4eba6d21b72620d2fd8d5c3548c1777d3c1c357

If this is OK, I will fix up the subject and the commit message to not 
have the Change-Id and avoid repeating the "silence such warnings". Thanks!
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/bcm5301x.dtsi b/arch/arm/boot/dts/bcm5301x.dtsi
index a06184b8e0d9..3fb1448b449f 100644
--- a/arch/arm/boot/dts/bcm5301x.dtsi
+++ b/arch/arm/boot/dts/bcm5301x.dtsi
@@ -240,6 +240,7 @@  chipcommon: chipcommon@0 {
 		pcie0: pcie@12000 {
 			reg = <0x00012000 0x1000>;
 			device_type = "pci";
+			ranges = <0 0 0 0 0 0>;
 
 			#address-cells = <3>;
 			#size-cells = <2>;
@@ -248,6 +249,7 @@  pcie0: pcie@12000 {
 		pcie1: pcie@13000 {
 			reg = <0x00013000 0x1000>;
 			device_type = "pci";
+			ranges = <0 0 0 0 0 0>;
 
 			#address-cells = <3>;
 			#size-cells = <2>;
@@ -256,6 +258,7 @@  pcie1: pcie@13000 {
 		pcie2: pcie@14000 {
 			reg = <0x00014000 0x1000>;
 			device_type = "pci";
+			ranges = <0 0 0 0 0 0>;
 
 			#address-cells = <3>;
 			#size-cells = <2>;