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

splitter brcm Wi-Fi firmware to firmware-brcm-wifi #1646

Closed
wants to merge 0 commits into from

Conversation

junzhuimx
Copy link
Contributor

firmware-nxp-wifi contains Wi-Fi firmware from NXP only

@otavio otavio requested a review from thochstein August 29, 2023 11:35
@thochstein
Copy link
Contributor

Hi Jun, what is the benefit of moving this package to a separate recipe?

@junzhuimx
Copy link
Contributor Author

firmware-nxp-wifi is defined for NXP Wi-Fi but bcm4359 is a "black sheep", as it is used for Cypress (Broadcom) Wi-Fi module.
And BRCM4359 does not upstream to linux-firmware.git, so the separate recipe should be more reasonable.

@otavio
Copy link
Member

otavio commented Aug 30, 2023

I don't really see the point of splitting the recipe up if it's all coming from the same repository. It seems like it would just make things more complicated and harder to manage in the long run. Maybe we could find a way to upstream the firmware instead?

@junzhuimx
Copy link
Contributor Author

@otavio
There is no more updates for bcm4359 firmware in imx-firmware.git. So after thus splitting, no more maintain needs for the recipe "firmware-brcm-wifi" until Cypress upstream them to linux-firmware.git.
We only focus on the firmware-nxp-wifi, which provides the latest NXP firmware for community/customers to "earlier" access.

@otavio
Copy link
Member

otavio commented Sep 8, 2023

#1645 seems to address all issues. Can we close this?

@junzhuimx
Copy link
Contributor Author

@otavio
#1645 is for NXP Wi-Fi firmware only.
This PR is just splitter BRCM firmware to new recipe. and does not conflict with #1645

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.

3 participants