-
Notifications
You must be signed in to change notification settings - Fork 533
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
Remove unused ESP8089 firmware [V42] #13005
base: master
Are you sure you want to change the base?
Conversation
The firmware is not used, because Batocera's RK3326 kernel: https://github.com/shantigilbert/hardkernel-linux/blob/d167de7358c9c53d5e6592978f7cdde7674c420b/drivers/net/wireless/esp8089/esp_main.c#L178 uses different file names than in the repo referenced in this package: https://github.com/jwrdegoede/esp8089/tree/38cb0c10d70754392932a52d9335a29eed9c3b94/firmware
i'm not sure about this as i added the firmware for a specific device that required that firmware for working wifi. |
@dmanlfc Which device was it? And are you sure adding this firmware actually fixed the wifi? Looks like this firmware was added in dfc611b (commit text is "add missing esp8089 firmware for rk3326") and this is around 4 months after wifi on RK3326 boards was broken by batocera-linux/buildroot@2ae792d . I.e. Looks like at the time of adding this firmware wifi on RK3326 boards was actually broken due to a completely different reason - missing wpa_supplicant_k44 package. P.S. AFAIK ESP8089 firmware is embedded in kernel files eagle_fw1.h, eagle_fw2.h and eagle_fw3.h |
the report from the user suggested their wifi was fixed, hence i added it. does it cause a conflict? if not, let's keep it for now. |
@dmanlfc Awesome! These might be much better news than initially seems! Looks like:
Seems like the user who contacted you is using a different kernel! Do you have any other explanations? Can you please ask the user:
P.S. No - these files don't cause any conflicts as they are not used by the current Batocera "stock" kernel.
|
Actually, now that I think about it, it could have been before we backtracked on mainline back to the 4.4 kernel. So it could be that the firmware files were required at the time. I've had a lot of sleeps since then ;-) We'll need to explore / experiment with a LTS 6.x kernel at some stage for future ease of support. |
The firmware is not used, because Batocera's RK3326 kernel:
https://github.com/shantigilbert/hardkernel-linux/blob/d167de7358c9c53d5e6592978f7cdde7674c420b/drivers/net/wireless/esp8089/esp_main.c#L178
uses different file names than in the repo referenced in this package: https://github.com/jwrdegoede/esp8089/tree/38cb0c10d70754392932a52d9335a29eed9c3b94/firmware
As this is a minor cleanup, it should probably go into v42.