mbox series

[0/2] ARM: dts: BCM5301X: Fix PCIe controller warnings

Message ID 20240430230714.3923578-1-florian.fainelli@broadcom.com (mailing list archive)
Headers show
Series ARM: dts: BCM5301X: Fix PCIe controller warnings | expand

Message

Florian Fainelli April 30, 2024, 11:07 p.m. UTC 
Hi Rafal,

Here is a counter proposal to your patch series:

https://lore.kernel.org/all/20240423110238.32148-1-zajec5@gmail.com/

which is a little bit smaller in terms of lines changed. Note that with
either my patches or yours, we still have the following warnings show
up:

arch/arm/boot/dts/broadcom/bcm4709-netgear-r8000.dts:133.15-145.4:
Warning (pci_device_reg): /axi@18000000/pcie@18012000/bridge@0,0,0: PCI
unit address format error, expected "0,0"
arch/arm/boot/dts/broadcom/bcm4709-netgear-r8000.dts:152.15-191.4:
Warning (pci_device_reg): /axi@18000000/pcie@18013000/bridge@1,0,0: PCI
unit address format error, expected "0,0"

Thanks!

Florian Fainelli (1):
  ARM: dts: BCM5301X: Relocate PCIe nodes to top-level

Rafał Miłecki (1):
  ARM: dts: BCM5301X: Describe PCIe controllers fully

 arch/arm/boot/dts/broadcom/bcm-ns.dtsi | 97 +++++++++++++++-----------
 1 file changed, 58 insertions(+), 39 deletions(-)

Comments

Florian Fainelli May 31, 2024, 9:33 p.m. UTC | #1 
On 4/30/24 16:07, Florian Fainelli wrote:
> Hi Rafal,
> 
> Here is a counter proposal to your patch series:
> 
> https://lore.kernel.org/all/20240423110238.32148-1-zajec5@gmail.com/
> 
> which is a little bit smaller in terms of lines changed. Note that with
> either my patches or yours, we still have the following warnings show
> up:
> 
> arch/arm/boot/dts/broadcom/bcm4709-netgear-r8000.dts:133.15-145.4:
> Warning (pci_device_reg): /axi@18000000/pcie@18012000/bridge@0,0,0: PCI
> unit address format error, expected "0,0"
> arch/arm/boot/dts/broadcom/bcm4709-netgear-r8000.dts:152.15-191.4:
> Warning (pci_device_reg): /axi@18000000/pcie@18013000/bridge@1,0,0: PCI
> unit address format error, expected "0,0"

Rafal, any chance you could test those patches? Thanks

> 
> Thanks!
> 
> Florian Fainelli (1):
>    ARM: dts: BCM5301X: Relocate PCIe nodes to top-level
> 
> Rafał Miłecki (1):
>    ARM: dts: BCM5301X: Describe PCIe controllers fully
> 
>   arch/arm/boot/dts/broadcom/bcm-ns.dtsi | 97 +++++++++++++++-----------
>   1 file changed, 58 insertions(+), 39 deletions(-)
>