Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

package/linux-firmware: add iwlwifi gl firmware #59

Merged

Conversation

antst
Copy link
Contributor

@antst antst commented Nov 12, 2024

This commit integrates support for the iwlwifi gl firmware files to support the wifi chipsets for the intel BE200 type of hardware. Thus, this change adds BR2_PACKAGE_LINUX_FIRMWARE_IWLWIFI_GL.

To be honest, personally, I'd add this to already existing umbrella for Intel 6/6E adapters, instead of adding yet another build switch. Because, either we enable all Intel 6/6E adapters, or we enable them one by one.

But, just in case, made it with additional switch.
Alternatively I can rework into other version.

Copy link
Member

@sairon sairon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, it looks great, just a couple of nits in the comments.

Comment on lines 544 to 545

LINUX_FIRMWARE_GL_UCODE_API_MAX = 83
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd like to keep this aligned with kernel naming to make updates a little easier. Also, keep it in one block:

Suggested change
LINUX_FIRMWARE_GL_UCODE_API_MAX = 83
LINUX_FIRMWARE_IWL_BZ_UCODE_API_MAX = 83

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one is incomplete, I'll commit it right way

LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENCE.iwlwifi_firmware
endif


Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove extra newline:

Suggested change

@@ -648,6 +650,14 @@ LINUX_FIRMWARE_FILES += iwlwifi-QuZ-*.ucode
LINUX_FIRMWARE_ALL_LICENSE_FILES += LICENCE.iwlwifi_firmware
endif

ifeq ($(BR2_PACKAGE_LINUX_FIRMWARE_IWLWIFI_GL),y)
LINUX_FIRMWARE_FILES += \
iwlwifi-gl-c0-fm-c0-$(LINUX_FIRMWARE_GL_UCODE_API_MAX).ucode\
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As above:

Suggested change
iwlwifi-gl-c0-fm-c0-$(LINUX_FIRMWARE_GL_UCODE_API_MAX).ucode\
iwlwifi-gl-c0-fm-c0-$(LINUX_FIRMWARE_IWL_BZ_UCODE_API_MAX).ucode\

@home-assistant home-assistant bot marked this pull request as draft November 13, 2024 08:42
@home-assistant
Copy link

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

@sairon
Copy link
Member

sairon commented Nov 13, 2024

Just a comment to this:

To be honest, personally, I'd add this to already existing umbrella for Intel 6/6E adapters, instead of adding yet another build switch. Because, either we enable all Intel 6/6E adapters, or we enable them one by one.

It makes sense to me as well, however, in our fork we add the extra logic for deploying only a single microcode version. That's something that seemed to be irrelevant for upstream (and Intel cards are quite uncommon in traditional Buildroot targets anyway), so extra work is needed to merge changes on Buildroot update anyway. I have an idea in my head for some time already to create a HAOS-specific package for Intel firmwares which would make it possible separate these differences to a file that's fully controlled by us.

@sairon
Copy link
Member

sairon commented Nov 13, 2024

Also, please squash all the changes into a single patch, so we can fast-forward merge this PR.

This commit integrates support for the iwlwifi gl firmware files
to support the wifi chipsets for the intel BE200 type of hardware.
Thus, this change adds BR2_PACKAGE_LINUX_FIRMWARE_IWLWIFI_GL.
@antst antst force-pushed the 2024.02.x-haos-iwl-BE200 branch from 8ef4427 to 1d7407c Compare November 13, 2024 14:46
@antst
Copy link
Contributor Author

antst commented Nov 13, 2024

Made requested changes.

@antst
Copy link
Contributor Author

antst commented Nov 13, 2024

To be honest, I am also unhappy with "IWLWIFI_GL", but naming is not my thingy )))

@sairon sairon merged commit 1d7407c into home-assistant:2024.02.x-haos Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants