[Buildroot] [PATCH] package/linux-firmware: Add Wilocity 6210 11ad firmware
Yann E. MORIN
yann.morin.1998 at free.fr
Tue Mar 31 20:03:32 UTC 2020
Baruch, All,
On 2020-03-30 17:31 +0300, Baruch Siach spake thusly:
> Cc: Yann E. MORIN <yann.morin.1998 at free.fr>
> Cc: Yegor Yefremov <yegorslists at googlemail.com>
> Signed-off-by: Baruch Siach <baruch at tkos.co.il>
> ---
> package/linux-firmware/Config.in | 5 +++++
> package/linux-firmware/linux-firmware.mk | 5 +++++
> 2 files changed, 10 insertions(+)
>
> diff --git a/package/linux-firmware/Config.in b/package/linux-firmware/Config.in
> index fb884842d66b..914b514f80bd 100644
> --- a/package/linux-firmware/Config.in
> +++ b/package/linux-firmware/Config.in
> @@ -345,6 +345,11 @@ config BR2_PACKAGE_LINUX_FIRMWARE_TI_WL18XX
> help
> Firmware files for WLAN and Bluetooth modules wl18xx
>
> +config BR2_PACKAGE_LINUX_FIRMWARE_WILOCITY_WIL6210
> + bool "Wilocity 6210"
I've been reminded recently that the options are all prefixed with the
vendor, followed by the model name/number, so in this case, it should
be:
config BR2_PACKAGE_LINUX_FIRMWARE_QUALCOM_WIL6210
bool "Qualcom Atheros wil6210"
So I was about to rename and apply, but then I noticed something weird
with the exiosting Atheros and Qualcom firmwares. We have:
config BR2_PACKAGE_LINUX_FIRMWARE_ATHEROS_10K_QCA6174
bool "Atheros 10k (QCA6174)"
help
Firmware files for the Atheros QCA6174 module
And:
config BR2_PACKAGE_LINUX_FIRMWARE_QUALCOMM_6174
bool "Qualcomm Atheros 6174"
help
Firmware files for the Qualcomm Atheros QCA6174 chip
And indeed they both installs files from ath10k/QCA6174: the former
installs a selected set of files by name, while the latter installs
the whole directory (using _FILES instaed of DIRS, at that).
Would wyou want to have a look amd fix that, then fix your patch and
resummit, please?
Regards,
Yann E. MORIN.
> + help
> + Firmware files for Qualcomm Atheros 11ad family of chips
> +
> endmenu
>
> menu "Ethernet firmware"
> diff --git a/package/linux-firmware/linux-firmware.mk b/package/linux-firmware/linux-firmware.mk
> index 21300ddc72de..e553a2a06383 100644
> --- a/package/linux-firmware/linux-firmware.mk
> +++ b/package/linux-firmware/linux-firmware.mk
> @@ -373,6 +373,11 @@ LINUX_FIRMWARE_FILES += \
> LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENCE.ti-connectivity
> endif
>
> +ifeq ($(BR2_PACKAGE_LINUX_FIRMWARE_WILOCITY_WIL6210),y)
> +LINUX_FIRMWARE_FILES += wil6210.*
> +LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENSE.QualcommAtheros_ath10k
> +endif
> +
> ifeq ($(BR2_PACKAGE_LINUX_FIRMWARE_IWLWIFI_3160),y)
> LINUX_FIRMWARE_FILES += iwlwifi-3160-*.ucode
> LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENCE.iwlwifi_firmware
> --
> 2.25.1
>
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list