Re: linux-next: build warnings after merge of the broadcom tree

From: Stephen Rothwell <sfr@canb.auug.org.au>
To: "Rafał Miłecki" <rafal@milecki.pl>
Cc: Florian Fainelli <f.fainelli@gmail.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux Next Mailing List <linux-next@vger.kernel.org>,
	Olof Johansson <olof@lixom.net>, Arnd Bergmann <arnd@arndb.de>,
	ARM <linux-arm-kernel@lists.infradead.org>
Subject: Re: linux-next: build warnings after merge of the broadcom tree
Date: Wed, 31 Aug 2022 09:16:54 +1000	[thread overview]
Message-ID: <20220831091654.45d5ed41@canb.auug.org.au> (raw)
In-Reply-To: <20220815105419.4df1005b@canb.auug.org.au>

[-- Attachment #1: Type: text/plain, Size: 2670 bytes --]

Hi all,

On Mon, 15 Aug 2022 10:54:19 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> On Mon, 25 Jul 2022 12:33:48 +0200 Rafał Miłecki <rafal@milecki.pl> wrote:
> >
> > On 25.07.2022 01:59, Stephen Rothwell wrote:  
> > > After merging the broadcom tree, today's linux-next build (arm
> > > multi_v7_defconfig) produced these 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)    
> > 
> > This is expected. My commit ef126d3f58d25 ("ARM: dts: BCM5301X: Add
> > basic PCI controller properties") reduced following warnings:
> > 
> > arch/arm/boot/dts/bcm47081-buffalo-wzr-600dhp2.dtb: pcie@12000: 'device_type' is a required property
> >          From schema: /lib/python3.9/site-packages/dtschema/schemas/pci/pci-bus.yaml
> > arch/arm/boot/dts/bcm47081-buffalo-wzr-600dhp2.dtb: pcie@12000: 'ranges' is a required property
> >          From schema: /lib/python3.9/site-packages/dtschema/schemas/pci/pci-bus.yaml
> > arch/arm/boot/dts/bcm47081-buffalo-wzr-600dhp2.dtb: pcie@12000: '#address-cells' is a required property
> >          From schema: /lib/python3.9/site-packages/dtschema/schemas/pci/pci-bus.yaml
> > arch/arm/boot/dts/bcm47081-buffalo-wzr-600dhp2.dtb: pcie@12000: '#size-cells' is a required property
> >          From schema: /lib/python3.9/site-packages/dtschema/schemas/pci/pci-bus.yaml
> > 
> > 
> > down to this one:
> > 
> > arch/arm/boot/dts/bcm47081-buffalo-wzr-600dhp2.dtb: pcie@12000: 'ranges' is a required property
> >          From schema: /lib/python3.9/site-packages/dtschema/schemas/pci/pci-bus.yaml
> > 
> > 
> > and basically does the right thing (adds required properties).
> > 
> > 
> > I'm fully aware "ranges" need to be added (it's mentioned in the commit)
> > and it's one of next things on my BCM5301X list.
> > 
> > So while my commits triggers that problem it also reduces warnings so
> > I'd say it's acceptable.

Not from my point of view:  I was not getting any warnings and now I am
getting 254 lines of warnings :-(

> So, is something being done about these introduced warnings?

I am still seeing these warnings.

The above commit is now

  61dc1e3850a6 ("ARM: dts: BCM5301X: Add basic PCI controller properties")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]