diff mbox

[2/6] package/linux-firmware: bump version for new firmware

Message ID 97ba0d959c1ec97eea764c84f57d1e2d5e99bff4.1370289131.git.yann.morin.1998@free.fr
State Accepted
Commit 3e650b8861542ba613bddad3d9b51d456bdeadc9
Headers show

Commit Message

Yann E. MORIN June 3, 2013, 8:15 p.m. UTC
From: "Yann E. MORIN" <yann.morin.1998@free.fr>

Bump version to get a few new firmware files for some WiFi adapters.

In the rtl81xx familly, it's hard to differentiate each members: I have
a USB stick that identifies itself as TRL8188CUS, but is handled by the
rtl8192ce driver. So I moved all of rtl81xx firmwares under the same
option to avoid confusion.

It will be the responsibility of the user to eventually get rid of
unwanted firmware files in a post-build script, if needed.

Ditto for rtl87xx.

Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
---
 package/linux-firmware/Config.in         | 12 ++++++------
 package/linux-firmware/linux-firmware.mk | 15 +++++++++------
 2 files changed, 15 insertions(+), 12 deletions(-)

Comments

Peter Korsgaard June 3, 2013, 9:09 p.m. UTC | #1
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes:

 Yann> From: "Yann E. MORIN" <yann.morin.1998@free.fr>
 Yann> Bump version to get a few new firmware files for some WiFi adapters.

 Yann> In the rtl81xx familly, it's hard to differentiate each members: I have
 Yann> a USB stick that identifies itself as TRL8188CUS, but is handled by the
 Yann> rtl8192ce driver. So I moved all of rtl81xx firmwares under the same
 Yann> option to avoid confusion.

 Yann> It will be the responsibility of the user to eventually get rid of
 Yann> unwanted firmware files in a post-build script, if needed.

 Yann> Ditto for rtl87xx.

Committed, thanks.
Thomas Petazzoni June 4, 2013, 7:13 a.m. UTC | #2
Dear Yann E. MORIN,

On Mon,  3 Jun 2013 22:15:48 +0200, Yann E. MORIN wrote:

> -config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8192
> -	bool "Realtek 8192"
> +config BR2_PACKAGE_LINUX_FIRMWARE_RTL_81XX
> +	bool "Realtek 81xx"
>  	help
> -	  Realtek 8192
> +	  Realtek 8188/8192
>  
> -config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8712
> -	bool "Realtek 8712"
> +config BR2_PACKAGE_LINUX_FIRMWARE_RTL_87XX
> +	bool "Realtek 87xx"
>  	help
> -	  Realtek 8712
> +	  Realtek 8712/8723

Warning, warning, warning: renaming of Kconfig options without
Config.in.legacy updates!

I must say I'm proud to be the one who mentions that, even after Peter
reviewed and committed the patch :-)

Best regards,

Thomas
Peter Korsgaard June 4, 2013, 7:25 a.m. UTC | #3
>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> Dear Yann E. MORIN,
 Thomas> On Mon,  3 Jun 2013 22:15:48 +0200, Yann E. MORIN wrote:

 >> -config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8192
 >> -	bool "Realtek 8192"
 >> +config BR2_PACKAGE_LINUX_FIRMWARE_RTL_81XX
 >> +	bool "Realtek 81xx"
 >> help
 >> -	  Realtek 8192
 >> +	  Realtek 8188/8192
 >> 
 >> -config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8712
 >> -	bool "Realtek 8712"
 >> +config BR2_PACKAGE_LINUX_FIRMWARE_RTL_87XX
 >> +	bool "Realtek 87xx"
 >> help
 >> -	  Realtek 8712
 >> +	  Realtek 8712/8723

 Thomas> Warning, warning, warning: renaming of Kconfig options without
 Thomas> Config.in.legacy updates!

 Thomas> I must say I'm proud to be the one who mentions that, even after Peter
 Thomas> reviewed and committed the patch :-)

Gaah, I even thought about it when I saw it, but then forgot about it
again.

Yann, will you send a patch adding Config.in.legacy handling for the old
names?
Yann E. MORIN June 4, 2013, 8:09 a.m. UTC | #4
Thomas, Peter, All,

On Tuesday 04 June 2013 09:25:05 Peter Korsgaard wrote:
> >>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:
> 
>  Thomas> Dear Yann E. MORIN,
>  Thomas> On Mon,  3 Jun 2013 22:15:48 +0200, Yann E. MORIN wrote:
> 
>  >> -config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8192
>  >> -	bool "Realtek 8192"
>  >> +config BR2_PACKAGE_LINUX_FIRMWARE_RTL_81XX
>  >> +	bool "Realtek 81xx"
>  >> help
>  >> -	  Realtek 8192
>  >> +	  Realtek 8188/8192
>  >> 
>  >> -config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8712
>  >> -	bool "Realtek 8712"
>  >> +config BR2_PACKAGE_LINUX_FIRMWARE_RTL_87XX
>  >> +	bool "Realtek 87xx"
>  >> help
>  >> -	  Realtek 8712
>  >> +	  Realtek 8712/8723
> 
>  Thomas> Warning, warning, warning: renaming of Kconfig options without
>  Thomas> Config.in.legacy updates!

Gahhh... :-(
I even thought about it at the time, and just forgot... :-(

>  Thomas> I must say I'm proud to be the one who mentions that, even after Peter
>  Thomas> reviewed and committed the patch :-)
> 
> Gaah, I even thought about it when I saw it, but then forgot about it
> again.
> 
> Yann, will you send a patch adding Config.in.legacy handling for the old
> names?

Yes, this evening, when back at home.

Regards,
Yann E. MORIN.
diff mbox

Patch

diff --git a/package/linux-firmware/Config.in b/package/linux-firmware/Config.in
index 9998f22..d9bb095 100644
--- a/package/linux-firmware/Config.in
+++ b/package/linux-firmware/Config.in
@@ -61,15 +61,15 @@  config BR2_PACKAGE_LINUX_FIRMWARE_RALINK_RT2XX
 	help
 	  Ralink rt27xx/rt28xx/rt30xx
 
-config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8192
-	bool "Realtek 8192"
+config BR2_PACKAGE_LINUX_FIRMWARE_RTL_81XX
+	bool "Realtek 81xx"
 	help
-	  Realtek 8192
+	  Realtek 8188/8192
 
-config BR2_PACKAGE_LINUX_FIRMWARE_RTL_8712
-	bool "Realtek 8712"
+config BR2_PACKAGE_LINUX_FIRMWARE_RTL_87XX
+	bool "Realtek 87xx"
 	help
-	  Realtek 8712
+	  Realtek 8712/8723
 
 config BR2_PACKAGE_LINUX_FIRMWARE_TI_WL127X
 	bool "TI wl127X"
diff --git a/package/linux-firmware/linux-firmware.mk b/package/linux-firmware/linux-firmware.mk
index 0cb08b4..0ba2b82 100644
--- a/package/linux-firmware/linux-firmware.mk
+++ b/package/linux-firmware/linux-firmware.mk
@@ -4,7 +4,7 @@ 
 #
 #############################################################
 
-LINUX_FIRMWARE_VERSION = 65a5163b8bb77bd7759efa3ca780867fcaed89b0
+LINUX_FIRMWARE_VERSION = 07ea598af5b9dde3acdf279846b062fa1b2987b8
 LINUX_FIRMWARE_SITE = http://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git
 LINUX_FIRMWARE_SITE_METHOD = git
 
@@ -22,16 +22,19 @@  LINUX_FIRMWARE_FILES_$(BR2_PACKAGE_LINUX_FIRMWARE_RALINK_RT2XX) += \
 	rt2860.bin rt2870.bin rt3070.bin rt3071.bin rt3090.bin 	\
 	LICENCE.ralink-firmware.txt
 
-# rtl8192
-LINUX_FIRMWARE_FILES_$(BR2_PACKAGE_LINUX_FIRMWARE_RTL_8192) += \
+# rtl81xx
+LINUX_FIRMWARE_FILES_$(BR2_PACKAGE_LINUX_FIRMWARE_RTL_81XX) += \
 	rtlwifi/rtl8192cfw.bin rtlwifi/rtl8192cfwU.bin 		\
 	rtlwifi/rtl8192cfwU_B.bin rtlwifi/rtl8192cufw.bin	\
 	rtlwifi/rtl8192defw.bin rtlwifi/rtl8192sefw.bin		\
+	rtlwifi/rtl8188efw.bin					\
 	LICENCE.rtlwifi_firmware.txt
 
-# rtl8712
-LINUX_FIRMWARE_FILES_$(BR2_PACKAGE_LINUX_FIRMWARE_RTL_8712) += \
-	rtlwifi/rtl8712u.bin LICENCE.rtlwifi_firmware.txt
+# rtl87xx
+LINUX_FIRMWARE_FILES_$(BR2_PACKAGE_LINUX_FIRMWARE_RTL_87XX) += \
+	rtlwifi/rtl8712u.bin rtlwifi/rtl8723fw.bin		\
+	rtlwifi/rtl8723fw_B.bin					\
+	LICENCE.rtlwifi_firmware.txt
 
 # ar7010
 LINUX_FIRMWARE_FILES_$(BR2_PACKAGE_LINUX_FIRMWARE_ATHEROS_7010) += \