diff mbox

linux-firmware: bump version

Message ID 1473962213-21519-1-git-send-email-gustavo@zacarias.com.ar
State Accepted
Commit ad0162623327fadd65b50a6007a5dfc5c52bd0a1
Headers show

Commit Message

Gustavo Zacarias Sept. 15, 2016, 5:56 p.m. UTC
And copy firmware for BCM43430 and BCM43438, hence rpi3 wifi support at
last.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
---
 package/linux-firmware/linux-firmware.mk | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Peter Korsgaard Sept. 15, 2016, 8:16 p.m. UTC | #1
>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 > And copy firmware for BCM43430 and BCM43438, hence rpi3 wifi support at
 > last.

 > Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>

Committed, thanks.

Will you send a patch to add BR2_PACKAGE_LINUX_FIRMWARE_BRCM_BCM43XX to
the rpi3 defconfig?
Gustavo Zacarias Sept. 16, 2016, 5:54 p.m. UTC | #2
On 2016-09-15 17:16, Peter Korsgaard wrote:

> Committed, thanks.
> 
> Will you send a patch to add BR2_PACKAGE_LINUX_FIRMWARE_BRCM_BCM43XX to
> the rpi3 defconfig?

Hi.
Well, without some $WiFi package (hostapd, wpa_supplicant, 
wireless-tools) just copying the firmware serves no purpose, though it 
can be done for completeness.
That being said, i lack a rpi3.
Regards.
Floris Bos Sept. 16, 2016, 6:11 p.m. UTC | #3
On 09/16/2016 07:54 PM, Gustavo Zacarias wrote:
> On 2016-09-15 17:16, Peter Korsgaard wrote:
>
>> Committed, thanks.
>>
>> Will you send a patch to add BR2_PACKAGE_LINUX_FIRMWARE_BRCM_BCM43XX to
>> the rpi3 defconfig?
>
> Hi.
> Well, without some $WiFi package (hostapd, wpa_supplicant, 
> wireless-tools) just copying the firmware serves no purpose, though it 
> can be done for completeness.
> That being said, i lack a rpi3.

And unless that changed recently, just the .bin the patch added wouldn't 
be sufficient for Pi 3.
Also need: 
https://github.com/RPi-Distro/firmware-nonfree/blob/master/brcm80211/brcm/brcmfmac43430-sdio.txt


Yours sincerely,

Floris Bos
Peter Korsgaard Sept. 16, 2016, 6:14 p.m. UTC | #4
>>>>> "Gustavo" == Gustavo Zacarias <gustavo@zacarias.com.ar> writes:

 > On 2016-09-15 17:16, Peter Korsgaard wrote:
 >> Committed, thanks.
 >> 
 >> Will you send a patch to add BR2_PACKAGE_LINUX_FIRMWARE_BRCM_BCM43XX to
 >> the rpi3 defconfig?

 > Hi.
 > Well, without some $WiFi package (hostapd, wpa_supplicant,
 > wireless-tools) just copying the firmware serves no purpose, though it
 > can be done for completeness.

True. We already enable wpa-supplicant for a number of defconfigs, so
probably that is the best option.

 > That being said, i lack a rpi3.

Me too.
Peter Korsgaard Sept. 17, 2016, 9:04 a.m. UTC | #5
>>>>> "Floris" == Floris Bos <bos@je-eigen-domein.nl> writes:

 > On 09/16/2016 07:54 PM, Gustavo Zacarias wrote:
 >> On 2016-09-15 17:16, Peter Korsgaard wrote:
 >> 
 >>> Committed, thanks.
 >>> 
 >>> Will you send a patch to add BR2_PACKAGE_LINUX_FIRMWARE_BRCM_BCM43XX to
 >>> the rpi3 defconfig?
 >> 
 >> Hi.
 >> Well, without some $WiFi package (hostapd, wpa_supplicant,
 >> wireless-tools) just copying the firmware serves no purpose, though
 >> it can be done for completeness.
 >> That being said, i lack a rpi3.

 > And unless that changed recently, just the .bin the patch added
 > wouldn't be sufficient for Pi 3.
 > Also need:
 > https://github.com/RPi-Distro/firmware-nonfree/blob/master/brcm80211/brcm/brcmfmac43430-sdio.txt

Hmm, perhaps. Do you have a rpi 3 to test on? The linux-firmware commit
doesn't mention anything about it:

http://git.kernel.org/cgit/linux/kernel/git/firmware/linux-firmware.git/commit/?id=c4c07a8d1128d50a5c2885ceea1abbebaa82f820

But the kernel does reference the .txt file as well:

drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c:BRCMF_FW_NVRAM_DEF(43430, "brcmfmac43430-sdio.bin", "brcmfmac43430-sdio.txt");
Floris Bos Sept. 17, 2016, 10:59 a.m. UTC | #6
On 09/17/2016 11:04 AM, Peter Korsgaard wrote:
>   > On 09/16/2016 07:54 PM, Gustavo Zacarias wrote:
>   >> On 2016-09-15 17:16, Peter Korsgaard wrote:
>   >>
>   >>> Committed, thanks.
>   >>>
>   >>> Will you send a patch to add BR2_PACKAGE_LINUX_FIRMWARE_BRCM_BCM43XX to
>   >>> the rpi3 defconfig?
>   >>
>   >> Hi.
>   >> Well, without some $WiFi package (hostapd, wpa_supplicant,
>   >> wireless-tools) just copying the firmware serves no purpose, though
>   >> it can be done for completeness.
>   >> That being said, i lack a rpi3.
>
>   > And unless that changed recently, just the .bin the patch added
>   > wouldn't be sufficient for Pi 3.
>   > Also need:
>   > https://github.com/RPi-Distro/firmware-nonfree/blob/master/brcm80211/brcm/brcmfmac43430-sdio.txt
>
> Hmm, perhaps. Do you have a rpi 3 to test on? The linux-firmware commit
> doesn't mention anything about it:
>
> http://git.kernel.org/cgit/linux/kernel/git/firmware/linux-firmware.git/commit/?id=c4c07a8d1128d50a5c2885ceea1abbebaa82f820

==
[    6.785594] usbcore: registered new interface driver brcmfmac
[    6.786851] brcmfmac_sdio mmc1:0001:1: Direct firmware load for 
brcm/brcmfmac43430-sdio.txt failed with error -2
[    7.789831] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000): 
clkctl 0x50
[    8.802492] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000): 
clkctl 0x50
==

Again, maybe they changed things very recently in a newer kernel branch 
or something.
But it simply does not work with the standard rpi one unless you also 
have the .txt


Yours sincerely,

Floris Bos
Peter Korsgaard Sept. 17, 2016, 12:15 p.m. UTC | #7
>>>>> "Floris" == Floris Bos <bos@je-eigen-domein.nl> writes:

Hi,

 >> Hmm, perhaps. Do you have a rpi 3 to test on? The linux-firmware commit
 >> doesn't mention anything about it:
 >> 
 >> http://git.kernel.org/cgit/linux/kernel/git/firmware/linux-firmware.git/commit/?id=c4c07a8d1128d50a5c2885ceea1abbebaa82f820

 > ==
 > [    6.785594] usbcore: registered new interface driver brcmfmac
 > [    6.786851] brcmfmac_sdio mmc1:0001:1: Direct firmware load for
 > brcm/brcmfmac43430-sdio.txt failed with error -2
 > [    7.789831] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000):
 > clkctl 0x50
 > [    8.802492] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000):
 > clkctl 0x50
 > ==

 > Again, maybe they changed things very recently in a newer kernel
 > branch or something.
 > But it simply does not work with the standard rpi one unless you also
 > have the .txt

Ok, thanks. Does it work with the .bin file from linux-firmware and the
.txt file from RPi-Distro?
Floris Bos Sept. 17, 2016, 12:25 p.m. UTC | #8
On 09/17/2016 02:15 PM, Peter Korsgaard wrote:
>>>>>> "Floris" == Floris Bos <bos@je-eigen-domein.nl> writes:
> Hi,
>
>   >> Hmm, perhaps. Do you have a rpi 3 to test on? The linux-firmware commit
>   >> doesn't mention anything about it:
>   >>
>   >> http://git.kernel.org/cgit/linux/kernel/git/firmware/linux-firmware.git/commit/?id=c4c07a8d1128d50a5c2885ceea1abbebaa82f820
>
>   > ==
>   > [    6.785594] usbcore: registered new interface driver brcmfmac
>   > [    6.786851] brcmfmac_sdio mmc1:0001:1: Direct firmware load for
>   > brcm/brcmfmac43430-sdio.txt failed with error -2
>   > [    7.789831] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000):
>   > clkctl 0x50
>   > [    8.802492] brcmfmac: brcmf_sdio_htclk: HT Avail timeout (1000000):
>   > clkctl 0x50
>   > ==
>
>   > Again, maybe they changed things very recently in a newer kernel
>   > branch or something.
>   > But it simply does not work with the standard rpi one unless you also
>   > have the .txt
>
> Ok, thanks. Does it work with the .bin file from linux-firmware and the
> .txt file from RPi-Distro?

Yes, that combination does work.


Yours sincerely,

Floris Bos
diff mbox

Patch

diff --git a/package/linux-firmware/linux-firmware.mk b/package/linux-firmware/linux-firmware.mk
index 8c983e4..d5a035e 100644
--- a/package/linux-firmware/linux-firmware.mk
+++ b/package/linux-firmware/linux-firmware.mk
@@ -4,7 +4,7 @@ 
 #
 ################################################################################
 
-LINUX_FIRMWARE_VERSION = c883a6b6186bb2415761d287cbac773062911212
+LINUX_FIRMWARE_VERSION = c4c07a8d1128d50a5c2885ceea1abbebaa82f820
 LINUX_FIRMWARE_SITE = http://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git
 LINUX_FIRMWARE_SITE_METHOD = git
 
@@ -381,7 +381,8 @@  LINUX_FIRMWARE_FILES += \
 	brcm/brcmfmac43242a.bin brcm/brcmfmac43340-sdio.bin \
 	brcm/brcmfmac43362-sdio.bin brcm/brcmfmac43455-sdio.bin \
 	brcm/brcmfmac43569.bin brcm/brcmfmac43570-pcie.bin \
-	brcm/brcmfmac43602-pcie.ap.bin brcm/brcmfmac43602-pcie.bin
+	brcm/brcmfmac43602-pcie.ap.bin brcm/brcmfmac43602-pcie.bin \
+	brcm/brcmfmac43430-sdio.bin
 LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENCE.broadcom_bcm43xx
 endif