* [PATCH] MIPS: CI20: DTS: Correcting IW8103 Wifi binding
@ 2020-07-05 10:32 agriveaux
2020-07-06 11:15 ` H. Nikolaus Schaller
0 siblings, 1 reply; 4+ messages in thread
From: agriveaux @ 2020-07-05 10:32 UTC (permalink / raw)
To: robh+dt, mark.rutland, ralf, paul.burton, jhogan, agriveaux
Cc: linux-mips, linux-kernel, devicetree
From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
Use brcm,bcm4329-fmac instead of brcm,bcm4330-fmac.
Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
---
arch/mips/boot/dts/ingenic/ci20.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/mips/boot/dts/ingenic/ci20.dts b/arch/mips/boot/dts/ingenic/ci20.dts
index 75f5bfbf2c37..82a1f126b778 100644
--- a/arch/mips/boot/dts/ingenic/ci20.dts
+++ b/arch/mips/boot/dts/ingenic/ci20.dts
@@ -116,8 +116,8 @@
pinctrl-0 = <&pins_mmc1>;
brcmf: wifi@1 {
-/* reg = <4>;*/
- compatible = "brcm,bcm4330-fmac";
+ reg = <1>;
+ compatible = "brcm,bcm4329-fmac";
vcc-supply = <&wlan0_power>;
device-wakeup-gpios = <&gpd 9 GPIO_ACTIVE_HIGH>;
shutdown-gpios = <&gpf 7 GPIO_ACTIVE_LOW>;
--
2.20.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] MIPS: CI20: DTS: Correcting IW8103 Wifi binding
2020-07-05 10:32 [PATCH] MIPS: CI20: DTS: Correcting IW8103 Wifi binding agriveaux
@ 2020-07-06 11:15 ` H. Nikolaus Schaller
2020-07-06 20:22 ` Alexandre GRIVEAUX
0 siblings, 1 reply; 4+ messages in thread
From: H. Nikolaus Schaller @ 2020-07-06 11:15 UTC (permalink / raw)
To: agriveaux
Cc: Rob Herring, Mark Rutland, Ralf Baechle, Paul Burton,
James Hogan, linux-mips, Linux Kernel Mailing List,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Alexandre,
> Am 05.07.2020 um 12:32 schrieb agriveaux@deutnet.info:
>
> From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>
> Use brcm,bcm4329-fmac instead of brcm,bcm4330-fmac.
>
> Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
> ---
> arch/mips/boot/dts/ingenic/ci20.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/mips/boot/dts/ingenic/ci20.dts b/arch/mips/boot/dts/ingenic/ci20.dts
> index 75f5bfbf2c37..82a1f126b778 100644
> --- a/arch/mips/boot/dts/ingenic/ci20.dts
> +++ b/arch/mips/boot/dts/ingenic/ci20.dts
> @@ -116,8 +116,8 @@
> pinctrl-0 = <&pins_mmc1>;
>
> brcmf: wifi@1 {
> -/* reg = <4>;*/
> - compatible = "brcm,bcm4330-fmac";
> + reg = <1>;
> + compatible = "brcm,bcm4329-fmac";
> vcc-supply = <&wlan0_power>;
> device-wakeup-gpios = <&gpd 9 GPIO_ACTIVE_HIGH>;
> shutdown-gpios = <&gpf 7 GPIO_ACTIVE_LOW>;
Do you have it working with a v5.8 kernel?
I don't see any activity to detect the module or load firmware.
Does it rely on some other patch?
BR and thanks,
Nikolaus
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] MIPS: CI20: DTS: Correcting IW8103 Wifi binding
2020-07-06 11:15 ` H. Nikolaus Schaller
@ 2020-07-06 20:22 ` Alexandre GRIVEAUX
2020-07-11 20:24 ` H. Nikolaus Schaller
0 siblings, 1 reply; 4+ messages in thread
From: Alexandre GRIVEAUX @ 2020-07-06 20:22 UTC (permalink / raw)
To: H. Nikolaus Schaller
Cc: Rob Herring, Mark Rutland, Ralf Baechle, Paul Burton,
James Hogan, linux-mips, Linux Kernel Mailing List,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Le 06/07/2020 à 13:15, H. Nikolaus Schaller a écrit :
> Hi Alexandre,
>
>> Am 05.07.2020 um 12:32 schrieb agriveaux@deutnet.info:
>>
>> From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>>
>> Use brcm,bcm4329-fmac instead of brcm,bcm4330-fmac.
>>
>> Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>> ---
>> arch/mips/boot/dts/ingenic/ci20.dts | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/arch/mips/boot/dts/ingenic/ci20.dts b/arch/mips/boot/dts/ingenic/ci20.dts
>> index 75f5bfbf2c37..82a1f126b778 100644
>> --- a/arch/mips/boot/dts/ingenic/ci20.dts
>> +++ b/arch/mips/boot/dts/ingenic/ci20.dts
>> @@ -116,8 +116,8 @@
>> pinctrl-0 = <&pins_mmc1>;
>>
>> brcmf: wifi@1 {
>> -/* reg = <4>;*/
>> - compatible = "brcm,bcm4330-fmac";
>> + reg = <1>;
>> + compatible = "brcm,bcm4329-fmac";
>> vcc-supply = <&wlan0_power>;
>> device-wakeup-gpios = <&gpd 9 GPIO_ACTIVE_HIGH>;
>> shutdown-gpios = <&gpf 7 GPIO_ACTIVE_LOW>;
> Do you have it working with a v5.8 kernel?
>
> I don't see any activity to detect the module or load firmware.
>
> Does it rely on some other patch?
>
> BR and thanks,
> Nikolaus
>
Hi Nikolaus
At this time the patch have been only "tested" for error will doing make:
make ARCH=mips CROSS_COMPILE=mipsel-linux-gnu- olddefconfig && make
ARCH=mips CROSS_COMPILE=mipsel-linux-gnu- -j8 && make ARCH=mips
CROSS_COMPILE=mipsel-linux-gnu- -j8 uImage
The .config come from creator-ci20 kernel 'config-3.18.3-ci20-1'
Even with the right DT
(Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt)
it's need some config with brcm enabled I gess.
I need to do some investigation will trying the uImage this week,
unfortunaly kernel developpement is not my main work, it's a hobby.
Thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] MIPS: CI20: DTS: Correcting IW8103 Wifi binding
2020-07-06 20:22 ` Alexandre GRIVEAUX
@ 2020-07-11 20:24 ` H. Nikolaus Schaller
0 siblings, 0 replies; 4+ messages in thread
From: H. Nikolaus Schaller @ 2020-07-11 20:24 UTC (permalink / raw)
To: Alexandre GRIVEAUX
Cc: Rob Herring, Mark Rutland, Ralf Baechle, Paul Burton,
James Hogan, linux-mips, Linux Kernel Mailing List,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Alexandre,
> Am 06.07.2020 um 22:22 schrieb Alexandre GRIVEAUX <agriveaux@deutnet.info>:
>
> Le 06/07/2020 à 13:15, H. Nikolaus Schaller a écrit :
>> Hi Alexandre,
>>
>>> Am 05.07.2020 um 12:32 schrieb agriveaux@deutnet.info:
>>>
>>> From: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>>>
>>> Use brcm,bcm4329-fmac instead of brcm,bcm4330-fmac.
>>>
>>> Signed-off-by: Alexandre GRIVEAUX <agriveaux@deutnet.info>
>>> ---
>>> arch/mips/boot/dts/ingenic/ci20.dts | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/arch/mips/boot/dts/ingenic/ci20.dts b/arch/mips/boot/dts/ingenic/ci20.dts
>>> index 75f5bfbf2c37..82a1f126b778 100644
>>> --- a/arch/mips/boot/dts/ingenic/ci20.dts
>>> +++ b/arch/mips/boot/dts/ingenic/ci20.dts
>>> @@ -116,8 +116,8 @@
>>> pinctrl-0 = <&pins_mmc1>;
>>>
>>> brcmf: wifi@1 {
>>> -/* reg = <4>;*/
>>> - compatible = "brcm,bcm4330-fmac";
>>> + reg = <1>;
>>> + compatible = "brcm,bcm4329-fmac";
>>> vcc-supply = <&wlan0_power>;
>>> device-wakeup-gpios = <&gpd 9 GPIO_ACTIVE_HIGH>;
>>> shutdown-gpios = <&gpf 7 GPIO_ACTIVE_LOW>;
>> Do you have it working with a v5.8 kernel?
>>
>> I don't see any activity to detect the module or load firmware.
>>
>> Does it rely on some other patch?
>>
>> BR and thanks,
>> Nikolaus
>>
> Hi Nikolaus
>
>
> At this time the patch have been only "tested" for error will doing make:
>
> make ARCH=mips CROSS_COMPILE=mipsel-linux-gnu- olddefconfig && make
> ARCH=mips CROSS_COMPILE=mipsel-linux-gnu- -j8 && make ARCH=mips
> CROSS_COMPILE=mipsel-linux-gnu- -j8 uImage
>
>
> The .config come from creator-ci20 kernel 'config-3.18.3-ci20-1'
>
>
> Even with the right DT
> (Documentation/devicetree/bindings/net/wireless/brcm,bcm43xx-fmac.txt)
> it's need some config with brcm enabled I gess.
>
>
> I need to do some investigation will trying the uImage this week,
> unfortunaly kernel developpement is not my main work, it's a hobby.
No problem. For me, the CI20 is also a hobby project :)
Here is some information about the CI20 WiFi and firmware:
https://elinux.org/CI20_Hardware#WiFi.2FBT
https://elinux.org/CI20_upstream#WiFi_firmware
So to be it looks as if the
compatible = "brcm,bcm4330-fmac";
is correct. But the reg = <4> or <1> is something we have to find out.
BR,
Nikolaus
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-07-11 20:24 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-05 10:32 [PATCH] MIPS: CI20: DTS: Correcting IW8103 Wifi binding agriveaux
2020-07-06 11:15 ` H. Nikolaus Schaller
2020-07-06 20:22 ` Alexandre GRIVEAUX
2020-07-11 20:24 ` H. Nikolaus Schaller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).