新增 Thales MV32-W-A 和 MV32-W-B 模组支持 - #244
Open
ArlongLi wants to merge 15 commits into
Open
Conversation
modem_ctrl.sh loads cmds/<vendor>.sh; missing cmds/thales.sh logged 'cmds file for vendor thales not found'. Add wrappers for the MV32-W command set (SETCONFIG/MODESWITCH/SLMODE/BAND_PREF/SWITCH_SLOT/TEMP/DEBUG/RESET plus common ATI/CGSN/CPIN/COPS/CNUM/CIMI/ICCID/CGMM/CGMI).
Drop the t99w368 (foxconn MV32-W) entries added earlier: normalize_model_name mapping in modem_scand.c, usb/pcie modem_support.json entries, and support_list.md row. mv32-w-a / mv32-w-b (thales) remain. foxconn.sh 90d5 mode-switch logic is upstream original code and is kept.
Per MV32-W-A HW guide, the PCIe configuration on Linux exposes DIAG/RmNet/Modem/NMEA (MBIM is Windows-only). Set pcie modes for mv32-w-a/mv32-w-b from mbim to rmnet; usb modes unchanged.
MV32-W-A/B do not work on PCIe with current firmware. Remove pcie entries from modem_support.json and mark pcie as unsupported in support_list.md. USB entries unchanged.
Owner
|
感谢贡献，我明天起床review一下。若方便可以参考 #236 通过 qmodem_collect 提供一份脱敏后的 fixture，后续这些行为就可以直接进入自动回归测试。 |
Owner
|
你好，感谢贡献。工作日比较忙，周末帮您review一下。或者您简单描述一下您的改动，用 #236 的测试用例收集工具收集一下您的测试用例。 |
AT+COPS? returns '????' in long-alphanumeric format when the module cannot resolve the PLMN name. Set COPS to numeric (AT+COPS=3,2) before querying, then map the MCCMNC (e.g. 46001) to the carrier name. Matches quectel/gosuncn.
The frontend ubus path (rpcd/qmodem -> load_vendor_script) only sourced vendor/<vendor>.sh but never cmds/<vendor>.sh, so vendor-specific cmd_* wrappers (e.g. cmd_cops_query) were undefined and ISP was silently dropped. Load cmds/<vendor>.sh after the vendor script, matching modem_ctrl.sh.
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
2 participants
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.
修改内容