fpv: give the 8812eu its own gk7205v200 build - #133
Merged
Conversation
openipc-ai
requested review from
cronyx,
flyrouter and
viktorxda
as code owners
August 27, 2026 17:30
PR Summary by QodoEnable RTL88X2EU adapters on gk7205v200 FPV
AI Description
High-Level Assessment
Files changed (1)
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can ask Qodo to dismiss a finding you disagree with, with your reason on record |
openipc-ai
force-pushed
the
fpv/gk7205v200-rtl88x2eu
branch
from
August 27, 2026 17:40
3f49c9a to
90e0e31
Compare
#61 asks for 8812eu support on gk7205v200 FPV. #70 and the first version of this PR both tried to satisfy it by adding BR2_PACKAGE_RTL88X2EU_OPENIPC=y next to the existing BR2_PACKAGE_RTL8812AU_OPENIPC=y. CI built that and it does not fit: - uImage: [1988KB/2048KB] - rootfs.squashfs: [5504KB/5120KB] -- size exceeded by: 384KB headroom: kernel=59KB rootfs=-384KB Two heavy kernel Wi-Fi drivers do not go in one 8M image. The driver itself is fine -- 8812eu.ko links and installs -- it is purely flash budget, and there is no 384KB of slack to find in a 5MB rootfs that was already at its cap. The tree had already said so and I misread it. Every defconfig here that ships both drivers is 16M: all the ssc338q, ssc30kq, ssc377qe and ssc378qe fpv/apfpv/rubyfpv targets. Every 8M one ships 8812au alone: gk7205v200/v210/v300 fpv and venc, hi3516ev200_fpv, hi3516ev300_fpv. The "missing" 8812eu on the Goke configs was never an oversight; it is what 8M allows. So this is a separate build instead of a fatter one. gk7205v200_fpv_8812eu is gk7205v200_fpv with one line changed, the driver swapped rather than added. The existing gk7205v200_fpv is untouched, so nobody using an 8812au adapter is affected. Nothing else needs to move for it: - Registration is automatic; the defconfig is the registration. - The excludes list is resolved as ${SOC_MODEL}_${VARIANT}.list by firmware's rootfs_script.sh, and VARIANT stays "fpv", so it reuses gk7205v200_fpv.list. That list strips neither driver. - Release assets do not collide. master.yml keys on the underscore count: gk7205v200_fpv has COMMON=1 and uploads the canonical openipc.gk7205v200-nor-fpv.tgz, while this one has COMMON=2 and is renamed to gk7205v200_fpv_8812eu-nor.tgz. Same for its sizes.json. Only gk7205v200, which is what #61 asked for. The other four 8M FPV targets have the same constraint and could take the same treatment if anyone asks for it.
openipc-ai
force-pushed
the
fpv/gk7205v200-rtl88x2eu
branch
from
August 27, 2026 18:02
90e0e31 to
33ce34f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#61 asks for 8812eu support on gk7205v200 FPV. #70 and the first version of this PR both tried to satisfy it by adding
BR2_PACKAGE_RTL88X2EU_OPENIPC=ynext to the existingBR2_PACKAGE_RTL8812AU_OPENIPC=y. CI built that and it does not fit:Two heavy kernel Wi-Fi drivers do not go in one 8M image. The driver itself is fine —
8812eu.kolinks and installs — it is purely flash budget, and there is no 384KB of slack to find in a 5MB rootfs that was already at its cap.The tree had already said so, and I misread it. Every defconfig here that ships both drivers is 16M: all the ssc338q, ssc30kq, ssc377qe and ssc378qe fpv/apfpv/rubyfpv targets. Every 8M one ships 8812au alone: gk7205v200/v210/v300 fpv and venc, hi3516ev200_fpv, hi3516ev300_fpv. The "missing" 8812eu on the Goke configs was never an oversight — it is what 8M allows.
What this does instead
gk7205v200_fpv_8812euisgk7205v200_fpvwith one line changed, the driver swapped rather than added:The existing
gk7205v200_fpvis untouched, so nobody using an 8812au adapter is affected.Nothing else needs to move for it
--self-testpasses at 108 devices, and the diff narrows to exactly the new target, so the build in this PR is the proof:${OPENIPC_SOC_MODEL}_${OPENIPC_VARIANT}.listby firmware'sgeneral/scripts/rootfs_script.sh, andVARIANTstays"fpv", so this reusesgk7205v200_fpv.list. That list strips neither driver.master.ymlkeys on the underscore count:gk7205v200_fpvhasCOMMON=1and uploads the canonicalopenipc.gk7205v200-nor-fpv.tgz, while this one hasCOMMON=2and is renamed togk7205v200_fpv_8812eu-nor.tgz. Same for itssizes.json.Only gk7205v200, which is what #61 asked for. The other four 8M FPV targets have the same constraint and could take the same treatment if anyone asks.