diff --git a/apps/chrg/applications/thread/chrg_north.c b/apps/chrg/applications/thread/chrg_north.c index c7bf5f0..6b06f2f 100644 --- a/apps/chrg/applications/thread/chrg_north.c +++ b/apps/chrg/applications/thread/chrg_north.c @@ -160,48 +160,6 @@ void chrg_source_setting(rt_uint8_t *frame_data, eIDX_SOU_CH idx, pSW->sub = IDX_SET_SOURCE_STOP_STATE; } } -////解析负载返回数据 -//void chrg_sink_parse(rt_uint8_t *frame_data, eIDX_SOU_CH idx, -// struct chrg_switch_t *pSW, struct chrg_north_t *pNOR) -//{ -// rt_uint16_t volt_mv = ((frame_data[4] << 8) | frame_data[5])*10; -// static uint8_t volt_low[TOTAL_SOU_CHS] = {0}; -// static uint8_t no_curr[TOTAL_SOU_CHS] = {0}; -// pSW->VZ = ((frame_data[14] << 8) | frame_data[15]); -// pSW->VF = ((frame_data[16] << 8) | frame_data[17]); - -//#if DEBUG_NORTH -// LOG_D("Sink[%d] real volt: %d mV", idx, volt_mv); -//#endif -// -// if(pSW->sink.On_work == 1) { -// if(pSW->sink.test.test_mode == 0){ -// if (volt_mv <= WORK_MIN_VOLT) { // 电压过低,禁止进入工作状态机 -// volt_low[idx]++; -// //if(volt_low[idx]>=5) pSW->sink.work_mode = SINK_WORK_LOW; // 复位工作状态机 -// } -// else{ // 电压稳定时执行工作状态机(继电器/工作模式切换) -// volt_low[idx] = 0; -// pSW->change_flag = 0; -// if((pSW->sink.pro_loadv - volt_mv) <= 2000){ -// pSW->sink.work_mode = SINK_WORK_RUNING; -// if(pSW->Now_current<=200){ -// if(no_curr[idx]++>=5){ -// pSW->sink.work_mode = SINK_WORK_WAIT; -// no_curr[idx] = 0; -// } -// } -// else no_curr[idx] = 0; -// } -// } -// } -// } -// else{ -// volt_low[idx] = 0; -//// pSW->sink.work_mode = SINK_WORK_STOP; -// } -// chrg_sink_work(idx,pSW); -//} //解析负载返回数据 void chrg_sink_parse(rt_uint8_t *frame_data, eIDX_SOU_CH idx, diff --git a/apps/chrg/applications/thread/chrg_roll_nor.c b/apps/chrg/applications/thread/chrg_roll_nor.c index 7bfcb09..736cebb 100644 --- a/apps/chrg/applications/thread/chrg_roll_nor.c +++ b/apps/chrg/applications/thread/chrg_roll_nor.c @@ -355,7 +355,10 @@ void chrg_roll_nor_set(rt_uint8_t ch, struct chrg_switch_t *pSW,rt_uint8_t *enab break; case IDX_SET_SOURCE_PD: chrg_roll_nor_source_pro_set(pSW); - pSW->source.On_Flag = 1; + if (pSW->sub == IDX_GET_SOURCE_VC && + chrg_source_init_submit((eIDX_SOU_CH)ch, pSW) == 0) { + pSW->source.On_Flag = 1; + } break; case IDX_GET_SOURCE_VC: source_get_vc(pSW->source.vc.set_voltage, pSW->source.vc.set_current); diff --git a/apps/chrg/applications/utils/chrg_source.c b/apps/chrg/applications/utils/chrg_source.c index 76ebabf..62e09dd 100644 --- a/apps/chrg/applications/utils/chrg_source.c +++ b/apps/chrg/applications/utils/chrg_source.c @@ -13,6 +13,31 @@ #include "chrg_utils.h" #include "chrg_roll_nor.h" #include "chrg_source.h" +#include "chrg_eload.h" +#include "chrg_roll_sou.h" + +int chrg_source_init_submit(rt_uint8_t idx, struct chrg_switch_t *pSW) +{ + rt_uint16_t regs[5] = { + REG_ELOAD, + REG_MODE, + REG_VOLTAGE_OUT + 1, + REG_CURRENT_OUT + 1, + REG_WORK + }; + rt_uint16_t vals[5] = { + ELOAD_SOURCE, + pSW->CV_mode, + pSW->source.max_vol, + pSW->source.protective_curr, + WORK_STOP + }; + + if ((idx >= TOTAL_SOU_CHS) || (pSW == RT_NULL)) { + return -1; + } + return chrg_sou_com_batch_submit(idx, regs, vals, 5); +} rt_uint8_t Pro_Group_VCMM[11][8] = { {0} // 11个挡位,8个参数(src, pps, vmin_H,vmin_L, vmax_H,vmax_L,cur_set_H,cur_set_L) }; diff --git a/apps/chrg/applications/utils/chrg_source.h b/apps/chrg/applications/utils/chrg_source.h index b4bc84c..8ed2c4f 100644 --- a/apps/chrg/applications/utils/chrg_source.h +++ b/apps/chrg/applications/utils/chrg_source.h @@ -249,6 +249,9 @@ extern void parse_power_pdo_gear(rt_uint8_t protoCmd, rt_uint8_t *pData, rt_uint extern void parse_multi_gear_more(rt_uint8_t protoCmd, rt_uint8_t *pData, rt_uint16_t reqLen,uint8_t *pro_gear_idx,uint8_t *pro,rt_uint16_t *protect_ma); extern void parse_power_ufcs(rt_uint8_t *pData, rt_uint16_t reqLen, uint8_t *pro_gear_idx, uint8_t *pro,rt_uint16_t *protect_ma); extern int source_set_pd_new_com(rt_uint8_t pro, struct chrg_switch_t *pSW); + +/* 提交 Source 功率板初始化参数;北向协议设置完成后调用。 */ +extern int chrg_source_init_submit(rt_uint8_t idx, struct chrg_switch_t *pSW); #endif diff --git a/docs/2026-07-21-source-state-machine-remediation-plan.md b/docs/2026-07-21-source-state-machine-remediation-plan.md new file mode 100644 index 0000000..43f98d0 --- /dev/null +++ b/docs/2026-07-21-source-state-machine-remediation-plan.md @@ -0,0 +1,121 @@ +# Source 状态机改造与角色切换维护计划 + +## 1. 改造目标 + +将电源(Source)控制整理为与负载(Sink)类似的独立状态机,但保留两者业务差异: + +- 北向协议板只负责角色/CC 线选择、协议设置及周期读取协议板电压电流。 +- 功率板角色、模式、电压、电流、继电器和启停全部由 `chrg_source.c` 管理。 +- 所有 Source 南向寄存器命令统一使用 `chrg_sou_com_batch_submit()`,不再散落调用 `chrg_set_sou_reg()`。 +- Source 不判断带载电流。协议板返回电压有效则进入 `RUNNING`;启动后持续无电压则回到 `WAIT`。 +- 协议、协议电压或协议电流发生变化时,重新进入 `INIT`。 +- Source/Sink 不停机切换时,必须先停止旧角色,确认完成后再初始化新角色。 + +## 2. 当前已完成的修改 + +当前工作区已完成第一阶段,尚未形成完整状态机: + +1. `apps/chrg/applications/utils/chrg_source.c` + - 新增 `chrg_source_init_submit(rt_uint8_t idx, struct chrg_switch_t *pSW)`。 + - 通过一个批次下发 `REG_ELOAD`、`REG_MODE`、`REG_VOLTAGE_OUT + 1`、`REG_CURRENT_OUT + 1` 和 `REG_WORK = WORK_STOP`。 +2. `apps/chrg/applications/utils/chrg_source.h` + - 增加上述函数声明。 +3. `apps/chrg/applications/thread/chrg_roll_nor.c` + - Source 协议设置流程进入 `IDX_GET_SOURCE_VC` 后提交南向初始化批次。 + - 仅当批次提交成功时设置 `source.On_Flag = 1`。 + +注意:当前仍有旧 Source 路径在 `chrg_north.c`、`chrg_roll_nor.c` 和 `chrg_lcd.c` 中直接调用 `chrg_set_sou_reg()`。在完成后续步骤前,新旧逻辑会同时存在,不能作为最终版本发布。 + +## 3. 目标状态机 + +在 `chrg_source.h` 中增加明确的 Source 状态: + +```c +typedef enum { + SOURCE_WORK_STOP = 0, + SOURCE_WORK_WAIT, + SOURCE_WORK_INIT, + SOURCE_WORK_START, + SOURCE_WORK_RUNNING, + SOURCE_WORK_STOPPING, + SOURCE_WORK_FAULT +} source_work_mode_t; +``` + +在 `chrg_src_t` 中增加每通道状态及参数快照:`work_mode`、`last_protocol`、`last_voltage`、`last_current`、`zero_volt_count`、`retry_count`。参数快照只能在对应命令确认完成后更新。 + +状态动作: + +| 状态 | 动作 | 后续状态 | +|---|---|---| +| `INIT` | 批量设置 Source 角色、模式、电压、电流及继电器,保持停止 | `START` | +| `START` | 批量下发 `REG_WORK = WORK_START` | `WAIT` | +| `WAIT` | 北向周期读取协议板 VC | 有电压进入 `RUNNING` | +| `RUNNING` | 更新 VC;检测协议/电压/电流变化 | 变化进入 `INIT` | +| `STOPPING` | 批量停止输出、关闭功率板继电器 | `STOP` | +| `STOP` | 清理使能并断开北向继电器 | 保持 | + +建议电压大于 `WORK_MIN_VOLT` 即视为有效。电压为零连续 3 次再由 `RUNNING` 回到 `WAIT`,避免单帧异常造成状态抖动。 + +## 4. COM 端改造 + +修改 `chrg_mb_write_sys()`: + +- 当目标角色为 Source 且当前未工作时,只设置 `pSW->sub = IDX_SET_SOURCE_CC_SET`,由北向依次完成 CC 和协议设置;协议应答完成后才进入 `SOURCE_WORK_INIT`。 +- 同角色下,比较协议、协议电压、协议电流。任一变化则进入 `SOURCE_WORK_INIT`;相同参数不得重复初始化。 +- 不要在收到 `SYS_REG_ROLE_SWITCH` 时立即覆盖实际角色。应保存 `target_role`,在参数生效时启动切换流程。 +- 切换期间返回 `MB_EX_SLAVE_BUSY`,或者只更新影子参数并等待本次切换完成。 + +角色切换顺序: + +```text +Source -> Sink: Source STOPPING -> STOP -> 切换角色/继电器 -> Sink INIT +Sink -> Source: Sink STOP -> 切换角色/继电器 -> 北向 CC/协议 -> Source INIT +``` + +禁止 Source 和 Sink 的功率板继电器、工作命令在同一通道同时有效。 + +## 5. 北向流程改造 + +精简 `chrg_roll_nor_set()` 的 Source 分支,只保留: + +1. `IDX_SET_SOURCE_CC_SET`:设置 CC 线,成功后进入 `IDX_SET_SOURCE_PD`。 +2. `IDX_SET_SOURCE_PD`:设置协议,应答成功后设置 `SOURCE_WORK_INIT`。 +3. `IDX_GET_SOURCE_VC`:读取协议板电压电流。 + +删除 Source 分支中的 `IDX_SET_SOURCE_ID`、`IDX_SET_SOURCE_CV`、`IDX_SER_SOURCE_VOLT`、`IDX_SER_SOURCE_CURR`、`IDX_SET_SOURCE_WORK` 南向设置动作。校准、控制台和 Sink 分支不在本次范围内。 + +`chrg_north.c` 只解析 Source VC:更新 `source.vc`、`Now_voltage`、`Now_current`,并根据电压推进 `WAIT/RUNNING`。移除 `chrg_source_standard_output()`、`chrg_source_fast_output()` 和旧的分步停机控制,避免解析函数直接下发南向命令。 + +## 6. 南向完成确认 + +`chrg_sou_com_batch_submit()` 返回成功仅表示入队成功,不表示功率板已经执行。需要为每通道增加批次完成序号或完成事件: + +- `INIT` 批次全部应答匹配后才能进入 `START`。 +- `START` 应答成功后才能进入 `WAIT`。 +- `STOPPING` 批次完成后才能断开北向继电器和修改实际角色。 +- CRC 错误、寄存器不匹配或超时不得推进状态;达到重试上限后进入 `FAULT`。 + +## 7. 推荐实施顺序 + +1. 增加 Source 状态枚举、参数快照及 `chrg_source_work()`。 +2. 将 INIT、START、STOPPING 的南向动作全部迁移到 `chrg_source.c`。 +3. 增加南向批次完成通知,再改为应答驱动状态转换。 +4. 精简北向 Source 子状态,只保留 CC、协议和 VC。 +5. 修改 COM 同角色参数差异及跨角色切换。 +6. 清理 `chrg_lcd.c` 中重复的 Source 启停和角色寄存器写入。 +7. 编译并在硬件验证后删除旧字段和旧函数。 + +## 8. 验证清单 + +- Keil 编译 `apps/chrg/chrg.uvprojx`,确认无未定义符号和头文件循环依赖。 +- 单通道验证 Source 首次启动、停止、相同参数重复生效。 +- 分别改变协议、协议电压和协议电流,确认只执行一次重新 INIT。 +- 模拟协议板电压持续为零,确认进入 WAIT 且不会反复提交相同批次。 +- 验证 Source -> Sink、Sink -> Source,示波器确认切换期间无输出重叠。 +- 四通道同时运行并交叉切换,检查南向队列饥饿、响应错配、超时和缓存串通道。 +- 回归 Sink 正常负载、动态测试、无电流重启及低压处理。 + +## 9. 接手注意事项 + +开始修改前先执行 `git status --short` 和 `git diff`。当前 `chrg_north.c` 还包含此前 Sink 连续确认次数改为 3 的修改及注释清理,不能在 Source 重构时回退。不要提交 Keil `build/`、`Obj/`、AXF、BIN 或 MAP 等生成文件。