From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jayachandran C Subject: Re: [PATCH v2] arm64: dts: cavium: fix PCI bus dtc warnings Date: Fri, 28 Jul 2017 01:43:05 +0000 Message-ID: <20170728014304.GA6857@localhost> References: <20170726210943.6875-1-robh@kernel.org> <20170726210943.6875-7-robh@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20170726210943.6875-7-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Rob Herring Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org List-Id: devicetree@vger.kernel.org On Wed, Jul 26, 2017 at 04:09:43PM -0500, Rob Herring wrote: > dtc recently added PCI bus checks. Fix these warnings: > > arch/arm64/boot/dts/cavium/thunder2-99xx.dtb: Warning (pci_bridge): Node /pci missing bus-range for PCI bridge > arch/arm64/boot/dts/cavium/thunder2-99xx.dtb: Warning (unit_address_vs_reg): Node /pci has a reg or ranges property, but no unit name > > Signed-off-by: Rob Herring > Cc: Jayachandran C > --- > v2: > - Reapply broadcom vulkan changes to thunder2. > > arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi b/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi > index 4220fbdcb24a..ea50c1320bf7 100644 > --- a/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi > +++ b/arch/arm64/boot/dts/cavium/thunder2-99xx.dtsi > @@ -98,7 +98,7 @@ > clock-output-names = "clk125mhz"; > }; > > - pci { > + pci@30000000 { > compatible = "pci-host-ecam-generic"; > device_type = "pci"; > #interrupt-cells = <1>; > @@ -118,6 +118,7 @@ > ranges = > <0x02000000 0 0x40000000 0 0x40000000 0 0x20000000 > 0x43000000 0x40 0x00000000 0x40 0x00000000 0x20 0x00000000>; > + bus-range = <0 0xff>; > interrupt-map-mask = <0 0 0 7>; > interrupt-map = > /* addr pin ic icaddr icintr */ Thanks for fixing this up. Acked-by: Jayachandran C The unit name issue is there with memory node on thunder2-99xx.dts as well, I can fix this up if you are not planning to. JC -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html