From a851bbeec607fa6b388137e786d822d5c27f471a Mon Sep 17 00:00:00 2001 From: Shiji Yang Date: Mon, 19 May 2025 08:09:47 +0800 Subject: [PATCH] mediatek: filogic: fix PCIe node dtc warnings Add missing #address-cells and #size-cells to fix the following dtc warnings: ../dts/mt7986a-smartrg-bonanza-peak.dtsi:359.4-27: Warning (reg_format): /soc/pcie@11280000/pcie@0,0/mt7915@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) ../dts/mt7988a-smartrg-mt-stuart.dtsi:551.4-27: Warning (reg_format): /soc/pcie@11300000/pcie@0,0/mt7996@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) ../dts/mt7988a-arcadyan-mozart.dts:331.4-27: Warning (reg_format): /soc/pcie@11300000/pcie@0,0/mt7996@1,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) ../dts/mt7988d-asus-zenwifi-bt8.dtsi:291.4-27: Warning (reg_format): /soc/pcie@11300000/pcie@0,0/mt7996@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso:72.5-28: Warning (reg_format): /fragment@2/__overlay__/pcie@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso:76.6-29: Warning (reg_format): /fragment@2/__overlay__/pcie@0,0/wifi@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso:88.5-28: Warning (reg_format): /fragment@3/__overlay__/pcie@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso:92.6-29: Warning (reg_format): /fragment@3/__overlay__/pcie@0,0/wifi@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-rfb-spim-nand-factory.dtso:47.5-28: Warning (reg_format): /fragment@1/__overlay__/pcie@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-rfb-spim-nand-factory.dtso:51.6-29: Warning (reg_format): /fragment@1/__overlay__/pcie@0,0/wifi@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) ../dts/mt7986a-tplink-tl-xtr8488.dts:305.4-27: Warning (reg_format): /soc/pcie@11280000/pcie@0,0/wifi@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) ../dts/mt7986a-zyxel-ex5700-telenor.dts:268.4-27: Warning (reg_format): /soc/pcie@11280000/pcie@0,0/wifi@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso:77.5-28: Warning (reg_format): /fragment@2/__overlay__/pcie@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso:95.5-28: Warning (reg_format): /fragment@3/__overlay__/pcie@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) mt7988a-rfb-spim-nand-factory.dtso:49.5-28: Warning (reg_format): /fragment@1/__overlay__/pcie@0,0:reg: property has invalid length (20 bytes) (#address-cells == 2, #size-cells == 1) Signed-off-by: Shiji Yang --- .../mediatek/dts/mt7986a-smartrg-bonanza-peak.dtsi | 2 ++ .../linux/mediatek/dts/mt7986a-tplink-tl-xtr8488.dts | 2 ++ .../mediatek/dts/mt7986a-zyxel-ex5700-telenor.dts | 2 ++ target/linux/mediatek/dts/mt7988a-arcadyan-mozart.dts | 4 +++- .../linux/mediatek/dts/mt7988a-smartrg-mt-stuart.dtsi | 2 ++ .../linux/mediatek/dts/mt7988d-asus-zenwifi-bt8.dtsi | 2 ++ .../mediatek/mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso | 10 ++++++++++ .../dts/mediatek/mt7988a-rfb-spim-nand-factory.dtso | 5 +++++ 8 files changed, 28 insertions(+), 1 deletion(-) diff --git a/target/linux/mediatek/dts/mt7986a-smartrg-bonanza-peak.dtsi b/target/linux/mediatek/dts/mt7986a-smartrg-bonanza-peak.dtsi index dd6cccade8c..41468449de3 100644 --- a/target/linux/mediatek/dts/mt7986a-smartrg-bonanza-peak.dtsi +++ b/target/linux/mediatek/dts/mt7986a-smartrg-bonanza-peak.dtsi @@ -356,6 +356,8 @@ slot0: pcie@0,0 { reg = <0x0000 0 0 0 0>; + #address-cells = <3>; + #size-cells = <2>; radio0: mt7915@0,0 { reg = <0x0000 0 0 0 0>; diff --git a/target/linux/mediatek/dts/mt7986a-tplink-tl-xtr8488.dts b/target/linux/mediatek/dts/mt7986a-tplink-tl-xtr8488.dts index 1b26b25f49e..e3d3aa37b2b 100644 --- a/target/linux/mediatek/dts/mt7986a-tplink-tl-xtr8488.dts +++ b/target/linux/mediatek/dts/mt7986a-tplink-tl-xtr8488.dts @@ -298,6 +298,8 @@ status = "okay"; pcie@0,0 { + #address-cells = <3>; + #size-cells = <2>; reg = <0x0000 0 0 0 0>; wifi@0,0 { diff --git a/target/linux/mediatek/dts/mt7986a-zyxel-ex5700-telenor.dts b/target/linux/mediatek/dts/mt7986a-zyxel-ex5700-telenor.dts index 29d37e93157..20e9146e2a3 100644 --- a/target/linux/mediatek/dts/mt7986a-zyxel-ex5700-telenor.dts +++ b/target/linux/mediatek/dts/mt7986a-zyxel-ex5700-telenor.dts @@ -261,6 +261,8 @@ status = "okay"; pcie@0,0 { + #address-cells = <3>; + #size-cells = <2>; reg = <0x0000 0 0 0 0>; wifi@0,0 { diff --git a/target/linux/mediatek/dts/mt7988a-arcadyan-mozart.dts b/target/linux/mediatek/dts/mt7988a-arcadyan-mozart.dts index 8b5f0526b20..f6752e340aa 100644 --- a/target/linux/mediatek/dts/mt7988a-arcadyan-mozart.dts +++ b/target/linux/mediatek/dts/mt7988a-arcadyan-mozart.dts @@ -326,8 +326,10 @@ pcie@0,0 { reg = <0x0000 0 0 0 0>; + #address-cells = <3>; + #size-cells = <2>; - mt7996@1,0 { + mt7996@0,0 { reg = <0x0000 0 0 0 0>; nvmem-cells = <&eeprom_factory_0>; nvmem-cell-names = "eeprom"; diff --git a/target/linux/mediatek/dts/mt7988a-smartrg-mt-stuart.dtsi b/target/linux/mediatek/dts/mt7988a-smartrg-mt-stuart.dtsi index 92ebc3a864c..058a96f253c 100644 --- a/target/linux/mediatek/dts/mt7988a-smartrg-mt-stuart.dtsi +++ b/target/linux/mediatek/dts/mt7988a-smartrg-mt-stuart.dtsi @@ -548,6 +548,8 @@ pcie@0,0 { reg = <0x0000 0 0 0 0>; + #address-cells = <3>; + #size-cells = <2>; mt7996@0,0 { reg = <0x0000 0 0 0 0>; diff --git a/target/linux/mediatek/dts/mt7988d-asus-zenwifi-bt8.dtsi b/target/linux/mediatek/dts/mt7988d-asus-zenwifi-bt8.dtsi index a8de1ee99a4..e865f2a678a 100644 --- a/target/linux/mediatek/dts/mt7988d-asus-zenwifi-bt8.dtsi +++ b/target/linux/mediatek/dts/mt7988d-asus-zenwifi-bt8.dtsi @@ -286,6 +286,8 @@ pcie@0,0 { reg = <0x0000 0 0 0 0>; + #address-cells = <3>; + #size-cells = <2>; mt7996_wifi: mt7996@0,0 { reg = <0x0000 0 0 0 0>; diff --git a/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso b/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso index baba44e5950..49ae14b72d6 100644 --- a/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso +++ b/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-bananapi-bpi-r4-wifi-mt7996a.dtso @@ -68,7 +68,12 @@ fragment@2 { target = <&pcie0>; __overlay__ { + #address-cells = <3>; + #size-cells = <2>; + pcie@0,0 { + #address-cells = <3>; + #size-cells = <2>; reg = <0x0000 0 0 0 0>; wifi@0,0 { @@ -84,7 +89,12 @@ fragment@3 { target = <&pcie1>; __overlay__ { + #address-cells = <3>; + #size-cells = <2>; + pcie@0,0 { + #address-cells = <3>; + #size-cells = <2>; reg = <0x0000 0 0 0 0>; wifi@0,0 { diff --git a/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-rfb-spim-nand-factory.dtso b/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-rfb-spim-nand-factory.dtso index 3fe75aca361..a94d1659d58 100644 --- a/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-rfb-spim-nand-factory.dtso +++ b/target/linux/mediatek/files-6.6/arch/arm64/boot/dts/mediatek/mt7988a-rfb-spim-nand-factory.dtso @@ -43,7 +43,12 @@ fragment@1 { target = <&pcie0>; __overlay__ { + #address-cells = <3>; + #size-cells = <2>; + pcie@0,0 { + #address-cells = <3>; + #size-cells = <2>; reg = <0x0000 0 0 0 0>; wifi@0,0 {