[PATCH 4/6] ARM: dts: BCM5301X: EA9200: add wifi definitions

Rosen Penev posted 6 patches 2 weeks, 4 days ago
[PATCH 4/6] ARM: dts: BCM5301X: EA9200: add wifi definitions
Posted by Rosen Penev 2 weeks, 4 days ago
brcm,ccode-map and ieee80211-freq-limit are needed to be specified on
some of them for proper operation.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
---
 .../dts/broadcom/bcm4709-linksys-ea9200.dts   | 50 +++++++++++++++++++
 1 file changed, 50 insertions(+)

diff --git a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
index 617fb55bc52f..7b1363aa1144 100644
--- a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
+++ b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
@@ -93,6 +93,56 @@ led-3 {
 	};
 };
 
+&pcie_bridge0 {
+	pcie@0,0 {
+		device_type = "pci';
+		reg = <0x0000 0 0 0 0>;
+		bus-range = <0x01 0xff>;
+
+		#address-cells = <3>;
+		#size-cells = <2>;
+		ranges;
+
+		pcie@1,0 {
+			device_type = "pci';
+			reg = <0x800 0 0 0 0>;
+
+			#address-cells = <3>;
+			#size-cells = <2>;
+
+			wifi@0,0 {
+				compatible = "brcm,bcm4366-fmac", "brcm,bcm4329-fmac";
+				reg = <0x0000 0 0 0 0>;
+				ieee80211-freq-limit = <5170000 5250000>;
+				brcm,ccode-map = "JP-JP-78", "US-Q2-86";
+			};
+		};
+
+		pcie@2,0 {
+			device_type = "pci';
+			reg = <0x1000 0 0 0 0>;
+
+			#address-cells = <3>;
+			#size-cells = <2>;
+
+			wifi@0,0 {
+				compatible = "brcm,bcm4366-fmac", "brcm,bcm4329-fmac";
+				reg = <0x0000 0 0 0 0>;
+				brcm,ccode-map = "JP-JP-78", "US-Q2-86";
+			};
+		};
+	};
+};
+
+&pcie_bridge1 {
+	wifi@0,0 {
+		compatible = "brcm,bcm4366-fmac", "brcm,bcm4329-fmac";
+		reg = <0x0000 0 0 0 0>;
+		ieee80211-freq-limit = <5735000 5835000>;
+		brcm,ccode-map = "JP-JP-78", "US-Q2-86";
+	};
+};
+
 &usb3_phy {
 	status = "okay";
 };
-- 
2.53.0
Re: [PATCH 4/6] ARM: dts: BCM5301X: EA9200: add wifi definitions
Posted by Florian Fainelli 2 weeks, 2 days ago
From: Florian Fainelli <f.fainelli@gmail.com>

On Wed, 18 Mar 2026 20:53:22 -0700, Rosen Penev <rosenp@gmail.com> wrote:
> brcm,ccode-map and ieee80211-freq-limit are needed to be specified on
> some of them for proper operation.
> 
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> ---

Applied to https://github.com/Broadcom/stblinux/commits/devicetree/next, thanks!
--
Florian
Re: [PATCH 4/6] ARM: dts: BCM5301X: EA9200: add wifi definitions
Posted by Rosen Penev 2 weeks, 3 days ago
On Wed, Mar 18, 2026 at 8:53 PM Rosen Penev <rosenp@gmail.com> wrote:
>
> brcm,ccode-map and ieee80211-freq-limit are needed to be specified on
> some of them for proper operation.
>
> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> ---
>  .../dts/broadcom/bcm4709-linksys-ea9200.dts   | 50 +++++++++++++++++++
>  1 file changed, 50 insertions(+)
>
> diff --git a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
> index 617fb55bc52f..7b1363aa1144 100644
> --- a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
> +++ b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
> @@ -93,6 +93,56 @@ led-3 {
>         };
>  };
>
> +&pcie_bridge0 {
> +       pcie@0,0 {
> +               device_type = "pci';
> +               reg = <0x0000 0 0 0 0>;
> +               bus-range = <0x01 0xff>;
> +
> +               #address-cells = <3>;
> +               #size-cells = <2>;
> +               ranges;
> +
> +               pcie@1,0 {
> +                       device_type = "pci';
There are typos here. "pci' vs "pci". Should I resend?
> +                       reg = <0x800 0 0 0 0>;
> +
> +                       #address-cells = <3>;
> +                       #size-cells = <2>;
> +
> +                       wifi@0,0 {
> +                               compatible = "brcm,bcm4366-fmac", "brcm,bcm4329-fmac";
> +                               reg = <0x0000 0 0 0 0>;
> +                               ieee80211-freq-limit = <5170000 5250000>;
> +                               brcm,ccode-map = "JP-JP-78", "US-Q2-86";
> +                       };
> +               };
> +
> +               pcie@2,0 {
> +                       device_type = "pci';
> +                       reg = <0x1000 0 0 0 0>;
> +
> +                       #address-cells = <3>;
> +                       #size-cells = <2>;
> +
> +                       wifi@0,0 {
> +                               compatible = "brcm,bcm4366-fmac", "brcm,bcm4329-fmac";
> +                               reg = <0x0000 0 0 0 0>;
> +                               brcm,ccode-map = "JP-JP-78", "US-Q2-86";
> +                       };
> +               };
> +       };
> +};
> +
> +&pcie_bridge1 {
> +       wifi@0,0 {
> +               compatible = "brcm,bcm4366-fmac", "brcm,bcm4329-fmac";
> +               reg = <0x0000 0 0 0 0>;
> +               ieee80211-freq-limit = <5735000 5835000>;
> +               brcm,ccode-map = "JP-JP-78", "US-Q2-86";
> +       };
> +};
> +
>  &usb3_phy {
>         status = "okay";
>  };
> --
> 2.53.0
>
Re: [PATCH 4/6] ARM: dts: BCM5301X: EA9200: add wifi definitions
Posted by Florian Fainelli 2 weeks, 2 days ago
On 3/19/26 17:29, Rosen Penev wrote:
> On Wed, Mar 18, 2026 at 8:53 PM Rosen Penev <rosenp@gmail.com> wrote:
>>
>> brcm,ccode-map and ieee80211-freq-limit are needed to be specified on
>> some of them for proper operation.
>>
>> Signed-off-by: Rosen Penev <rosenp@gmail.com>
>> ---
>>   .../dts/broadcom/bcm4709-linksys-ea9200.dts   | 50 +++++++++++++++++++
>>   1 file changed, 50 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
>> index 617fb55bc52f..7b1363aa1144 100644
>> --- a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
>> +++ b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
>> @@ -93,6 +93,56 @@ led-3 {
>>          };
>>   };
>>
>> +&pcie_bridge0 {
>> +       pcie@0,0 {
>> +               device_type = "pci';
>> +               reg = <0x0000 0 0 0 0>;
>> +               bus-range = <0x01 0xff>;
>> +
>> +               #address-cells = <3>;
>> +               #size-cells = <2>;
>> +               ranges;
>> +
>> +               pcie@1,0 {
>> +                       device_type = "pci';
> There are typos here. "pci' vs "pci". Should I resend?

Not necessary, fixed it up while applying, thanks!
-- 
Florian
Re: [PATCH 4/6] ARM: dts: BCM5301X: EA9200: add wifi definitions
Posted by Florian Fainelli 2 weeks, 2 days ago
On 3/20/26 10:29, Florian Fainelli wrote:
> On 3/19/26 17:29, Rosen Penev wrote:
>> On Wed, Mar 18, 2026 at 8:53 PM Rosen Penev <rosenp@gmail.com> wrote:
>>>
>>> brcm,ccode-map and ieee80211-freq-limit are needed to be specified on
>>> some of them for proper operation.
>>>
>>> Signed-off-by: Rosen Penev <rosenp@gmail.com>
>>> ---
>>>   .../dts/broadcom/bcm4709-linksys-ea9200.dts   | 50 +++++++++++++++++++
>>>   1 file changed, 50 insertions(+)
>>>
>>> diff --git a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts b/ 
>>> arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
>>> index 617fb55bc52f..7b1363aa1144 100644
>>> --- a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
>>> +++ b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
>>> @@ -93,6 +93,56 @@ led-3 {
>>>          };
>>>   };
>>>
>>> +&pcie_bridge0 {
>>> +       pcie@0,0 {
>>> +               device_type = "pci';
>>> +               reg = <0x0000 0 0 0 0>;
>>> +               bus-range = <0x01 0xff>;
>>> +
>>> +               #address-cells = <3>;
>>> +               #size-cells = <2>;
>>> +               ranges;
>>> +
>>> +               pcie@1,0 {
>>> +                       device_type = "pci';
>> There are typos here. "pci' vs "pci". Should I resend?
> 
> Not necessary, fixed it up while applying, thanks!

Meh, this was not even build tested, even fixing that gives me:

  ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- make CHECK_DTBS=y 
broadcom/bcm4709-linksys-ea9200.dtb
   DTC [C] arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dtb
Lexical error: 
arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts:107.19-22 
Unexpected 'pci'
Error: arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts:107.19-22 
syntax error
FATAL ERROR: Unable to parse input tree

Please resubmit patches 4 and 6 since I dropped them as a result.
-- 
Florian
Re: [PATCH 4/6] ARM: dts: BCM5301X: EA9200: add wifi definitions
Posted by Rosen Penev 2 weeks, 2 days ago
On Fri, Mar 20, 2026 at 2:34 PM Florian Fainelli
<florian.fainelli@broadcom.com> wrote:
>
> On 3/20/26 10:29, Florian Fainelli wrote:
> > On 3/19/26 17:29, Rosen Penev wrote:
> >> On Wed, Mar 18, 2026 at 8:53 PM Rosen Penev <rosenp@gmail.com> wrote:
> >>>
> >>> brcm,ccode-map and ieee80211-freq-limit are needed to be specified on
> >>> some of them for proper operation.
> >>>
> >>> Signed-off-by: Rosen Penev <rosenp@gmail.com>
> >>> ---
> >>>   .../dts/broadcom/bcm4709-linksys-ea9200.dts   | 50 +++++++++++++++++++
> >>>   1 file changed, 50 insertions(+)
> >>>
> >>> diff --git a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts b/
> >>> arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
> >>> index 617fb55bc52f..7b1363aa1144 100644
> >>> --- a/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
> >>> +++ b/arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts
> >>> @@ -93,6 +93,56 @@ led-3 {
> >>>          };
> >>>   };
> >>>
> >>> +&pcie_bridge0 {
> >>> +       pcie@0,0 {
> >>> +               device_type = "pci';
> >>> +               reg = <0x0000 0 0 0 0>;
> >>> +               bus-range = <0x01 0xff>;
> >>> +
> >>> +               #address-cells = <3>;
> >>> +               #size-cells = <2>;
> >>> +               ranges;
> >>> +
> >>> +               pcie@1,0 {
> >>> +                       device_type = "pci';
> >> There are typos here. "pci' vs "pci". Should I resend?
> >
> > Not necessary, fixed it up while applying, thanks!
>
> Meh, this was not even build tested, even fixing that gives me:
Was build tested on kernel 6.12: https://github.com/openwrt/openwrt/pull/22223
>
>   ARCH=arm CROSS_COMPILE=arm-linux-gnueabihf- make CHECK_DTBS=y
> broadcom/bcm4709-linksys-ea9200.dtb
>    DTC [C] arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dtb
> Lexical error:
> arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts:107.19-22
> Unexpected 'pci'
> Error: arch/arm/boot/dts/broadcom/bcm4709-linksys-ea9200.dts:107.19-22
> syntax error
> FATAL ERROR: Unable to parse input tree
>
> Please resubmit patches 4 and 6 since I dropped them as a result.
There are three "pci' mistakes that are copy/paste in this patch. I'll resubmit.
> --
> Florian