Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions br-ext-chip-goke/configs/gk7201v200_lite_defconfig
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,9 @@ BR2_PACKAGE_MOTORS=y
BR2_PACKAGE_OPUS_OPENIPC=y
BR2_PACKAGE_OPUS_OPENIPC_FIXED_POINT=y
BR2_PACKAGE_VTUND_OPENIPC=y
# Wi-Fi: the G3-WQ-G's AltoBeam ATBM6012B-X, USB. Fits the split image only
# (#2668); the fixed-slot package outgrows its 5 MiB rootfs and is skipped.
BR2_PACKAGE_ATBM603X_COMB_OPENIPC=y
Comment on lines +71 to +73

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Action required

1. The wi-fi image lacks board proof 📘 Rule violation ☼ Reliability

gk7201v200_lite_defconfig now selects BR2_PACKAGE_ATBM603X_COMB_OPENIPC, changing the shipped
firmware for the G3-WQ-G Wi-Fi board. The PR states that no camera test was run and that the module
was not loaded, so the altered image's boot, USB detection and association remain unverified.
Agent Prompt
Issue description
The defconfig enables a Wi-Fi driver that changes the firmware image, but the PR explicitly states that no camera test was run and the module was not loaded. The required hardware evidence for boot, USB detection, and Wi-Fi association is therefore missing.

Fix Focus Areas
- br-ext-chip-goke/configs/gk7201v200_lite_defconfig[71-73]

Recommended Fix
Test the split image on the affected G3-WQ-G camera, load the Wi-Fi module, verify detection of USB device `007a:888b`, and verify association; then add the resulting boot and runtime evidence to the PR description.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Agreed, and the description says it too: this has not run on a camera. The owner of a G3-WQ-G from #1896 is going to test it over UART. The checks are the split image booting, 007a:888b enumerating, atbm603x_comb_wifi_usb binding, and wlan0 associating and getting a lease. Their output goes in the description before this merges, so I'm leaving this thread open until then.

Comment thread
widgetii marked this conversation as resolved.
Resolved
BR2_PACKAGE_YAML_CLI=y
BR2_ROOTFS_POST_IMAGE_SCRIPT="$(BR2_EXTERNAL_GENERAL_PATH)/scripts/post-image-split.sh"
BR2_OPENIPC_NOR_FIXED_PACKAGE=y
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
# OpenIPC/atbm_60xx's atbm603x_comb/ directory: AltoBeam's apollo driver
# (DRIVER_VER 2874) with the AsmLite and Ares-B firmware, as Khadas ships it.
ATBM603X_COMB_OPENIPC_SITE = $(call github,openipc,atbm_60xx,$(ATBM603X_COMB_OPENIPC_VERSION))
ATBM603X_COMB_OPENIPC_VERSION = 543b02dc8871a1bc4df90cd56e07305b95c3ec9c
ATBM603X_COMB_OPENIPC_VERSION = bd7644ecade21991a5cef9513acd53e27237a438

ATBM603X_COMB_OPENIPC_LICENSE = GPL-2.0

Expand Down
Loading