Repository navigation
Add gk7202v300_lite_xm-rm1 (XM IPC-RM1-BLK7202V3-M43A-WIFI, SV6115C Wi-Fi) - #205
Conversation
Xiongmai pan/tilt dome board: GK7202V300 + SC223A, 8 MB NOR, Wi-Fi only (iComm SV6115C on USB, 8065:6011). Cloned from the Cootli CAMV0103 board, which shares the IR-cut/IR LED/speaker pins and the pan-motor pinmux. - GPIOs from the stock firmware (loadpublic gpio_pin=, LIBXMVIDEO.json): pan 13 15 14 12, tilt 55 54 53 52 (gpio-motors), IR-cut 56/58, IR LED 16, photoresistor 57, speaker amp 51. - Drops wireguard, vtund and the non-gpio motor tools to fit 8 MB. Wi-Fi needs OpenIPC/ssv6x5x#1 (WSDK 12148, SV6115C support). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
/review |
Code Review by Qodo
1. Camera loses its SC223A driver
|
| @@ -0,0 +1,45 @@ | |||
| /etc/sensors/4l_sc223a_i2c_1080p.ini | |||
There was a problem hiding this comment.
1. Camera loses its sc223a driver 📘 Rule violation ≡ Correctness
The new exclude list removes both the SC223A four-lane sensor configuration and libsns_sc223a_4l.so even though the device uses an SC223A sensor and the selected Goke driver package installs that library. Any build using this list reaches the camera without the sensor files needed for SC223A detection and streaming.
Agent Prompt
## Issue description
The device-specific exclude list removes the SC223A four-lane configuration and shared library required by this board's SC223A sensor.
## Fix Focus Areas
- devices/gk7202v300_lite_xm-rm1/general/scripts/excludes/gk7202v300_lite.list[1-1]
- devices/gk7202v300_lite_xm-rm1/general/scripts/excludes/gk7202v300_lite.list[35-35]
## Recommended Fix
Remove the `/etc/sensors/4l_sc223a_i2c_1080p.ini` and `/usr/lib/sensors/libsns_sc223a_4l.so` entries from the exclude list, then verify that the resulting image still fits within the 8 MB flash layout.
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
| ZTE K545 T31X SC4336 ATBM6012B_USB NOR_16M done | ||
| 4G Camera XG521 V1.2 GK7202V300 GC1054 EC800E-CN_USB NOR_8M done | ||
| XM IPG-G3-WR GK7202V300 JXH63 ATBM60321S_USB NOR_8M done | ||
| XM IPC-RM1-BLK7202V3 GK7202V300 SC223A SV6115C_USB NOR_8M video-yes, wifi-yes (ssv6x5x#1), pan/tilt+ir+audio untested |
There was a problem hiding this comment.
2. The clone inventory omits this board 📘 Rule violation ⚙ Maintainability
The README adds the XM IPC-RM1 board to the device table but does not map it in the compatibility and clones table to the existing Cootli board it was cloned from. Readers using the published inventory cannot discover that this rebrand shares the supported Cootli implementation.
Agent Prompt
## Issue description
The new board is documented in the main device table, but its rebrand relationship to `gk7202v300_lite_cootli_camv0103` is absent from the README compatibility and clones table.
## Fix Focus Areas
- README.md[120-120]
- README.md[124-142]
## Recommended Fix
Add a compatibility-table row mapping the XM IPC-RM1-BLK7202V3-M43A-WIFI board to the existing Cootli CAMV0103 firmware/device entry, using the repository's established clone-table format.
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
b1bebf6 to
f85b949
Compare
The list was copied from cootli_camv0103 and had drifted from the package. Seven entries named files that are no longer installed (60fps/, WDR/ x2, iq/f23.ini, iq/gc2053.ini, libsns_f23.so, libsns_h63.so), while high-fps/*, iq/ov2735.ini, iq/sc2239.ini, the gc1054/gc4023/gc5603/imx327/mis2008/os02g10/sp2308 profiles and 19 opensdk-built sensor libraries shipped unlisted. It now lists every installed sensor file except the two this board uses: sc223a_i2c_1080p.ini and libsns_sc223a.so. Both IMX335 name layouts are listed, because which one a build produces depends on the SoC. The three wildcard lines are removed. rootfs_script.sh tests each entry as a literal path, so they never matched anything, and the generic gk7202v300 kernel config builds no i2c, scsi or f2fs modules for them to strip anyway.
f85b949 to
ea8b75d
Compare
|
I pushed ea8b75d to this branch: it refreshes |
…s today (#206) The list had drifted from the package. Seven entries named files that are no longer installed (60fps/, WDR/ x2, iq/f23.ini, iq/gc2053.ini, libsns_f23.so, libsns_h63.so), while high-fps/*, iq/ov2735.ini, iq/sc2239.ini, the gc1054/gc4023/gc5603/imx327/mis2008/os02g10/sp2308 profiles and 19 opensdk-built sensor libraries shipped unlisted. It now lists every installed sensor file except the two this board uses: sc223a_i2c_1080p.ini and libsns_sc223a.so. Both IMX335 name layouts are listed, because which one a build produces depends on the SoC. The three wildcard lines are removed. rootfs_script.sh tests each entry as a literal path, so they never matched anything, and the generic gk7202v300 kernel config builds no i2c, scsi or f2fs modules for them to strip anyway. Same list as gk7202v300_lite_xm-rm1 (#205), the other SC223A board on this SoC.
|
Merged. On Qodo's two notes: the board's SC223A loads |
Device
Xiongmai
IPC-RM1-BLK7202V3-M43A-WIFI: a pan/tilt dome board.8065:6011, efuse SKU 0x57F7FFFF).This device is cloned from
gk7202v300_lite_cootli_camv0103. That board shares the IR-cut / IR LED / speaker pins and the GPIO12/14/15 pinmux (S80pinworks) that the pan motor needs.Pins (from the stock firmware)
loadpublic→XmMotor.ko gpio_pin=LIBXMVIDEO.jsonS35pinworks)The stock driver steps the motors in the same 8-phase half-step sequence as
gpio-motors. Travel: x_stepmax 3950 / y_stepmax 1000 half-steps; there are no limit switches.Dependency
Wi-Fi needs OpenIPC/ssv6x5x#1. The current 11204 driver cannot bring up the SV6115C. The package builds
ssv6x5xat HEAD, so nothing here changes when that PR merges.Tested (local
make BOARD=gk7202v300_lite_xm-rm1, firmware468eccc+ ssv6x5x#1)ci-matrix.py --self-testok;Untested, because the board was on the bench without the motors, IR board or speaker attached:
The README status says so.
🤖 Generated with Claude Code