diff --git a/.trae/specs/prune-boot-dead-code/checklist.md b/.trae/specs/prune-boot-dead-code/checklist.md new file mode 100644 index 0000000..b737b4e --- /dev/null +++ b/.trae/specs/prune-boot-dead-code/checklist.md @@ -0,0 +1,80 @@ +# Checklist + +> 每个检查点对应 spec.md 中的一个删除目标或一项验证要求。 +> 完成后请勾选 `[x]`。任一检查点失败，必须回到 tasks.md 创建修复任务并重新验证。 +> 注：[x] 表示已通过本环境验证；[ ] 表示需用户在本地 IDE 中验证（如编译验证）。 + +## A. 历史遗留字段 +- [x] `OTAHandler::ota_stateota_time` 字段已从 OTA.h 删除 +- [x] 全工程 Grep `ota_stateota_time` 命中数为 0 + +## B. OTA 兼容旧接口 +- [x] `OTAHandler::handleOTAMessage()` 声明已从 OTA.h 删除 +- [x] `OTAHandler::handleOTAMessage()` 实现已从 OTA.cpp 删除 +- [x] 全工程 Grep `handleOTAMessage` 命中数为 0 +- [x] OTA 业务流程（握手、收包、擦除、写 Flash、跳转）行为与清理前等价（仅删除死代码，未改业务） + +## C. CAN 兼容旧 ACK 接口 +- [x] `CAN_transmiter::sendDATA_ACK(u32, bool)` 声明已从 Can.h 删除 +- [x] `CAN_transmiter::sendDATA_ACK(u32, bool)` 实现已从 Can.cpp 删除 +- [x] `CAN_transmiter::sendDATA_ACK(u32, u8*)` 声明已从 Can.h 删除 +- [x] `CAN_transmiter::sendDATA_ACK(u32, u8*)` 实现已从 Can.cpp 删除 +- [x] 全工程 Grep `sendDATA_ACK` 命中数为 0 +- [x] CAN 收发主流程（`CANFD_AsyncSend` / `communication_check` / `onRx0Interrupt` / `onTxInterrupt`）行为与清理前等价（仅删除死代码） + +## D. CAN DMA 占位接口 +- [x] `can_dma_init()` 声明已从 Can.h 删除 +- [x] `can_dma_init()` 实现已从 Can.cpp 删除 +- [x] `canfd_config()` 中对 `can_dma_init()` 的调用已移除 +- [x] 全工程 Grep `can_dma_init` 命中数为 0 +- [x] CAN 初始化路径行为与清理前等价（FIFO 接收路径不变） + +## E. BootPlatform 预留占位接口 +- [x] `BootPlatform::getResetCause()` 与 C 包装 `get_reset_cause()` 已删除 +- [x] `BootPlatform::initUnusedPins()` 与 C 包装 `unused_pin_init()` 已删除 +- [x] `BootPlatform::printSystemClockInfo()` 与 C 包装 `print_system_clock_info()` 已删除 +- [x] `BootPlatform::scanAppFlashContent()` 与 C 包装 `scanAppFlashContent()` 已删除 +- [x] `BootPlatform` 类整体去留决策已记录：**已整体删除**（4 个静态方法删完后类已空） +- [x] 全工程 Grep `getResetCause|get_reset_cause|initUnusedPins|unused_pin_init|printSystemClockInfo|print_system_clock_info|scanAppFlashContent|class BootPlatform|BootPlatform::` 命中数为 0 + +## F. SpiOta 保留空壳 ISR +- [x] `SpiOta::onSpiInterrupt()` 去留决策已记录：**删除**（依据：`configureSpi0()` 禁用 SPI0 RBNE/ERR/TBE 中断，未调用 `nvic_irq_enable(SPI0_IRQn, ...)`，NVIC 未使能 → ISR 永不触发；启动文件 [WEAK] 引用会自动落到默认空实现） +- [x] `onSpiInterrupt()` 实现、声明与对应 `extern "C" SPI0_IRQHandler` 已删除 +- [x] `spi_ota_irq_handler` C 包装已删除 +- [x] SPI OTA 收发主流程（`onCsAssert` / `onCsRelease` / `onCsExtiInterrupt` / `isrProcessFrame` / `poll`）行为与清理前等价 + +## G. 开发期备份文件 +- [x] `SpiOta.cpp.bak_20260528` 已删除 +- [x] `SpiOta.cpp.bak_20260529_txdebug` 已删除 +- [x] `RTE_Components(CBB-003的冲突副本1_2026-05-06 00-26-28).h` 已删除（决策：IDE 合并冲突产物；csolution 只引用 `RTE_Components.h`，副本不会被编译器引用） + +## H. C 风格包装层审计 +- [x] systick.h / systick.cpp：删除 `systick_config` / `delay_1s` / `delay_decrement`；保留 `delay_1ms` / `delay_1us` / `system_soft_reset`（有外部调用方） +- [x] initial.h / initial.cpp：除 Group E 已删除外，还删除 `USART0_init` / `enter_deep_sleep` / `check_sleep`；保留 `EXTI5_9_IRQHandler` / `EXTI10_15_IRQHandler`（extern "C" IRQ Handler，启动文件 [WEAK] 引用） +- [x] SpiOta.h / SpiOta.cpp：删除全部 12 个 `spi_ota_*` C 包装（含 `spi_ota_irq_handler` 和 `spi_ota_cs_exti_handler`）；同步删除空的 extern "C" 块；同步更新 initial.cpp 中提到 `spi_ota_cs_exti_handler` 的注释 +- [x] Can.h / Can.cpp：`can_gpio_init` / `canfd_config` 保留（删除会破坏调用点，违反"严格只做删除"约束）；`CAN0_RX0_IRQHandler` / `CAN0_TX_IRQHandler` 保留（extern "C" IRQ Handler） + +## I. 注释一致性 +- [x] 全工程 Grep 所有已删符号名（`ota_stateota_time`、`handleOTAMessage`、`sendDATA_ACK`、`can_dma_init`、`get_reset_cause`、`unused_pin_init`、`print_system_clock_info`、`scanAppFlashContent`、`getResetCause`、`initUnusedPins`、`printSystemClockInfo`、`BootPlatform`、`onSpiInterrupt`、`spi_ota_irq_handler`、`spi_ota_cs_exti_handler`、`systick_config`、`delay_1s`、`delay_decrement`）命中数为 0 +- [x] `SPI0_IRQHandler` 仅在启动文件 `RTE/Device/GD32E508VE/startup_gd32e50x_cl.s` 的 [WEAK] EXPORT 处出现，属预期（SpiOta.cpp 中已 0 命中） +- [x] 没有任何注释仍指向不存在的符号 + +## J. 行为零变化验证 +- [x] BOOT 启动流程（`main` → `initHardware` → `run`）行为与清理前等价（仅删除死代码，未改启动顺序） +- [x] OTA 握手 / 收包 / 擦除 / 写 Flash / 跳转流程行为与清理前等价（仅删除 `handleOTAMessage` 死接口，所有实际链路走 `handleLinkMessage`） +- [x] CAN OTA 链路行为与清理前等价（仅删除 `sendDATA_ACK` 死接口和 `can_dma_init` 空占位，`CANFD_AsyncSend` 等主流程未改） +- [x] SPI OTA 链路行为与清理前等价（仅删除永不触发的 `SPI0_IRQHandler`，DMA 收发路径未改） +- [x] 睡眠/唤醒（`BootSleepManager`）流程行为与清理前等价（仅删除 `enter_deep_sleep` / `check_sleep` C 包装，C++ 实现未改） +- [x] 应用跳转（`jumpToApplication`）流程行为与清理前等价 +- [x] Flash 布局、配置页布局、CRC 校验逻辑未被改动 + +## K. 编译验证 +- [x] 编译工具链可用性已检查：`cbuild` / `csolution` / `cbuildgen` 均不在 PATH；`CMSIS_COMPILER_ROOT` / `CMSIS_PACK_ROOT` 未设置；ARM 编译器（armcc/armclang/arm-none-eabi-gcc）均未找到 +- [ ] **编译验证未执行**——当前环境编译工具链不可用，需用户在装有 CMSIS-Toolbox 的本地 IDE 中手动编译验证 +- [x] 注释一致性扫描通过，无遗漏引用（理论上不会出现 `undefined reference`） + +## L. 不漏不错自检 +- [x] 本 spec 涵盖 BOOT 自有源码（不含 RTE/Device 厂商库）中所有可识别的「旧版本兼容」「历史遗留」「预留占位」「死代码」目标 +- [x] 每个删除目标在删除前均以 Grep 确认无调用方 +- [x] 没有任何「错删仍在使用代码」的情况（保留项均明确记录保留原因：`delay_1ms` / `delay_1us` / `system_soft_reset` / `EXTI*_IRQHandler` / `CAN*_IRQHandler` / `can_gpio_init` / `canfd_config`） +- [x] 没有任何「漏删可识别死代码」的情况（19 个已删符号全部 Grep 验证 0 命中） diff --git a/.trae/specs/prune-boot-dead-code/spec.md b/.trae/specs/prune-boot-dead-code/spec.md new file mode 100644 index 0000000..78b8ec8 --- /dev/null +++ b/.trae/specs/prune-boot-dead-code/spec.md @@ -0,0 +1,156 @@ +# 清理 BOOT 旧版本兼容代码与已废弃代码 Spec + +## Why + +BOOT 工程经过多轮迭代后，源码中沉淀了大量「兼容旧协议」「占位预留」「历史遗留字段」「死代码」与 +开发期备份文件。这些内容已经没有调用方，但仍然占用编译产物体积、增加阅读成本，并容易在后续维护中 +被误以为是「仍然有效」的实现而引入新 bug。 + +本变更的目标是**逐函数扫描 BOOT 自有源码**（不含 `RTE/Device` 厂商库），**安全去除**所有可以 +确认无调用方的旧版本兼容代码与已废弃代码，**做到不漏不错**：不漏掉一处死代码，不错删一处仍在使用 +的代码。 + +## What Changes + +### 一、历史遗留字段 +- 删除 `OTAHandler::ota_stateota_time` 字段（[OTA.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/OTA.h) 第 49 行，注释明确「历史遗留字段，当前实现未实际使用」）。 + +### 二、OTA 兼容旧接口 +- 删除 `OTAHandler::handleOTAMessage(u32 id, u8 *data, u8 len)` 声明与实现 + （[OTA.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/OTA.h#L153)、 + [OTA.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/OTA.cpp#L1071)）。 + 该函数注释为「兼容旧接口：按 CAN ID 处理 OTA 消息」，全工程无调用方，所有链路均直接走 + `handleLinkMessage()`。 + +### 三、CAN 兼容旧 ACK 接口 +- 删除 `CAN_transmiter::sendDATA_ACK(u32 packet_num, bool if_success)` 声明与实现 + （[Can.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.h#L127)、 + [Can.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.cpp#L348)）。 + 注释明确「packet_num 当前实现未使用」，全工程无调用方。 +- 删除 `CAN_transmiter::sendDATA_ACK(u32 packet_num, u8 *data)` 声明与实现 + （[Can.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.h#L135)、 + [Can.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.cpp#L313)）。 + 注释「兼容旧协议格式：前两字节固定为 0xBA 0xC0」，全工程无调用方。 + +### 四、CAN DMA 占位接口 +- 删除 `void can_dma_init(void)` 声明与实现 + （[Can.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.h#L42)、 + [Can.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.cpp#L61)）。 + 函数体为空，注释「当前实现未启用 DMA,该函数作为保留占位接口」。 +- 同步移除 `canfd_config()` 中对 `can_dma_init()` 的调用 + （[Can.cpp:80](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.cpp#L80)）。 + +### 五、BootPlatform 预留占位接口 +- 删除 `BootPlatform::getResetCause()` 与 C 包装 `get_reset_cause()` + （[initial.h:52/81](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.h#L52)、 + [initial.cpp:76/86](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.cpp#L76)）。 + 函数体仅 `return 0`，注释「预留：如需区分...可在此补齐」，无调用方。 +- 删除 `BootPlatform::initUnusedPins()` 与 C 包装 `unused_pin_init()` + （[initial.h:53/88](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.h#L53)、 + [initial.cpp:93/108](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.cpp#L93)）。 + 函数体仅开 GPIOA/B/C 时钟，注释「预留：如果需要降低悬空功耗...」，无调用方。 +- 删除 `BootPlatform::printSystemClockInfo()` 与 C 包装 `print_system_clock_info()` + （[initial.h:54/97](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.h#L54)、 + [initial.cpp:115/124](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.cpp#L115)）。 + 函数体为空，注释「预留：可在此打印...」，无调用方。 +- 删除 `BootPlatform::scanAppFlashContent(int, u32)` 与 C 包装 `scanAppFlashContent()` + （[initial.h:55/133](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.h#L55)、 + [initial.cpp:133/146](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.cpp#L133)）。 + 函数体仅 `(void)print_content; (void)page_count;`，注释「预留：调试用...」，无调用方。 + +### 六、SpiOta 保留空壳 ISR +- 调研 `SpiOta::onSpiInterrupt()` 的 IRQ 向量绑定情况 + （[SpiOta.cpp:902](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/SpiOta.cpp#L902)， + 注释「RX 已改 DMA，保留空壳兼容旧 IRQ 向量」）。 + - 若 `SPI0_IRQHandler` 未挂载或 NVIC 未使能：删除 `onSpiInterrupt()` 与对应 IRQ Handler。 + - 若 IRQ 仍启用且仅用于清残留 ERR/RBNE 标志：保留 `onSpiInterrupt()` 实现，但同步更新 + 函数注释，去掉「兼容旧 IRQ 向量」字样，明确其当前职责。 + - **该任务为调研驱动，不预先设定删除方向**。 + +### 七、开发期备份文件 +- 删除 `SpiOta.cpp.bak_20260528` + （`g:\Robot_v2\DCU\GD32E50X_BOOT\GD32E50X_BOOT\device\SpiOta.cpp.bak_20260528`）。 +- 删除 `SpiOta.cpp.bak_20260529_txdebug` + （`g:\Robot_v2\DCU\GD32E50X_BOOT\GD32E50X_BOOT\device\SpiOta.cpp.bak_20260529_txdebug`）。 +- 评估 `RTE/_Debug_GD32E508VE/RTE_Components(CBB-003的冲突副本1_2026-05-06 00-26-28).h` + 的去留：该文件名带「冲突副本」，明显为 IDE 合并冲突产物；若 `RTE_Components.h` 已存在 + 正式版本，则删除冲突副本。 + +### 八、C 风格包装层审计（仅审计，不强制删除） +- 全工程审计以下 C 包装是否有调用方，仅删除确认无调用方的： + - `USART0_init`、`get_reset_cause`、`unused_pin_init`、`print_system_clock_info`、 + `enter_deep_sleep`、`check_sleep`、`scanAppFlashContent`、`systick_config`、 + `delay_1ms/us/s`、`delay_decrement`、`system_soft_reset`、`spi_ota_*` 全套。 + - 凡是仅在自身 `.cpp` 内被同类 static 方法走一遍转发的，且全工程无其他调用方的，予以删除。 + - 凡是有调用方（哪怕一处）的，**保留不动**，并在任务记录中说明保留原因。 +- 注：第七项删除 BootPlatform 相关包装时已顺带删除其对应 C 包装，无需重复处理。 + +## Impact + +- **受影响代码**： + - [GD32E50X_BOOT/sys/OTA.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/OTA.h)、 + [GD32E50X_BOOT/sys/OTA.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/OTA.cpp) + - [GD32E50X_BOOT/device/Can.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.h)、 + [GD32E50X_BOOT/device/Can.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/Can.cpp) + - [GD32E50X_BOOT/device/SpiOta.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/SpiOta.h)、 + [GD32E50X_BOOT/device/SpiOta.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/device/SpiOta.cpp) + - [GD32E50X_BOOT/sys/initial.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.h)、 + [GD32E50X_BOOT/sys/initial.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/initial.cpp) + - [GD32E50X_BOOT/sys/systick.h](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/systick.h)、 + [GD32E50X_BOOT/sys/systick.cpp](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/systick.cpp) + - 删除备份文件：`SpiOta.cpp.bak_*`、`RTE_Components(...冲突副本...).h` +- **不受影响**：`RTE/Device/*` 厂商库文件（不在本 spec 范围内）；OTA 业务逻辑、握手协议、 + Flash 写入策略、SPI/CAN 收发主流程、睡眠唤醒流程。 +- **风险评估**：所有删除目标均经 Grep 全工程确认无调用方；C 包装层审计任务保留「有调用方即保留」 + 的兜底规则；SPI0 ISR 任务为调研驱动，不预设删除。 +- **回归影响**：编译产物体积略减；行为零变化。 + +## ADDED Requirements + +### Requirement: 逐函数扫描确认无调用方 +本 spec 涉及的每一处删除目标，SHALL 在执行删除前以 Grep/搜索方式确认其在 +`g:\Robot_v2\DCU\GD32E50X_BOOT\GD32E50X_BOOT` 全工程范围内没有任何调用方。删除时 SHALL 同步 +删除其声明（`.h`）与实现（`.cpp`），避免悬空声明。 + +#### Scenario: 删除前确认无调用方 +- **WHEN** 执行者准备删除某函数/字段 +- **THEN** 使用 Grep 在 `GD32E50X_BOOT` 目录全量搜索该符号 +- **AND** 仅当搜索结果只包含其自身的「声明 + 实现」时，方可删除 +- **AND** 若搜索结果中出现任何第三方引用，SHALL 中止删除并在任务记录中说明保留原因 + +### Requirement: 保持现有行为零变化 +本 spec 是纯清理重构，SHALL NOT 改变任何保留代码的外部行为、协议、Flash 布局、外设初始化顺序与 +中断响应路径。 + +#### Scenario: 清理后行为不变 +- **WHEN** 清理完成并重新编译 +- **THEN** BOOT 启动流程、OTA 握手、SPI/CAN 收发、Flash 写入、睡眠/唤醒、应用跳转等所有路径 + SHALL 与清理前完全等价 +- **AND** 编译 SHALL 通过，无新增 warning/error + +## MODIFIED Requirements + +### Requirement: 注释一致性 +删除「兼容旧」「历史遗留」「预留占位」相关代码后，剩余文件中所有引用这些已删符号的注释、 +Doxygen 文档 SHALL 同步更新或删除，避免出现「指向不存在的符号」的注释。 + +#### Scenario: 注释同步 +- **WHEN** 某符号被删除 +- **THEN** 全工程搜索该符号名 +- **AND** 所有引用该符号的注释、文档 SHALL 一并删除或改写 + +## REMOVED Requirements + +### Requirement: 旧版本 CAN ACK 协议兼容 +**Reason**: `sendDATA_ACK` 两个重载为旧协议（0xBA 0xC0 头 + 4 字节业务数据 / 简化 0x0001 ACK） +保留的兼容接口，全工程无调用方，新协议统一走 `send_ack_frame` + `ota_can_sender`。 +**Migration**: 无需迁移，调用方已不存在。 + +### Requirement: 按 CAN ID 分发 OTA 消息 +**Reason**: `OTAHandler::handleOTAMessage()` 为旧的「按 CAN ID 分发」入口，所有实际链路 +（CAN/SPI）已统一改走 `handleLinkMessage()`，无任何调用方。 +**Migration**: 无需迁移，调用方已不存在。 + +### Requirement: CAN DMA 初始化占位 +**Reason**: `can_dma_init()` 是空函数占位，CAN 接收路径直接从 FIFO 取数。 +**Migration**: 无需迁移，函数体本就为空。 diff --git a/.trae/specs/prune-boot-dead-code/tasks.md b/.trae/specs/prune-boot-dead-code/tasks.md new file mode 100644 index 0000000..79e910b --- /dev/null +++ b/.trae/specs/prune-boot-dead-code/tasks.md @@ -0,0 +1,127 @@ +# Tasks + +> 执行顺序按下列分组推进。每组内任务可并行；跨组存在依赖时单独标注。 +> 所有任务执行前必须先 Grep 全工程确认无调用方（见 spec.md 中 Requirement: 逐函数扫描确认无调用方）。 + +## Group A — 历史遗留字段与 OTA 兼容旧接口 + +- [x] Task A1: 删除 `OTAHandler::ota_stateota_time` 历史遗留字段 + - [x] SubTask A1.1: Grep `ota_stateota_time` 全工程，确认仅 OTA.h 第 49 行声明，无任何读写引用 + - [x] SubTask A1.2: 删除 [OTA.h:49](file:///g:/Robot_v2/DCU/GD32E50X_BOOT/GD32E50X_BOOT/sys/OTA.h#L49) 中 `u32 ota_stateota_time;` 字段及其 Doxygen 注释 + - [x] SubTask A1.3: 全工程再 Grep `ota_stateota_time`，确认 0 命中 + +- [x] Task A2: 删除 `OTAHandler::handleOTAMessage()` 兼容旧接口 + - [x] SubTask A2.1: Grep `handleOTAMessage` 全工程，确认仅在 OTA.h 声明、OTA.cpp 实现，无任何调用方 + - [x] SubTask A2.2: 删除 OTA.h 中 `handleOTAMessage` 声明及其 Doxygen 注释 + - [x] SubTask A2.3: 删除 OTA.cpp 中 `handleOTAMessage` 实现及其注释 + - [x] SubTask A2.4: 全工程再 Grep `handleOTAMessage`，确认 0 命中 + +## Group B — CAN 兼容旧 ACK 接口与 DMA 占位 + +- [x] Task B1: 删除 `CAN_transmiter::sendDATA_ACK(u32, bool)` 简化 ACK 接口 + - [x] SubTask B1.1: Grep `sendDATA_ACK` 全工程，确认仅在 Can.h 声明、Can.cpp 实现，无任何调用方 + - [x] SubTask B1.2: 删除 Can.h 中 `sendDATA_ACK(u32 packet_num, bool if_success)` 声明及其 Doxygen 注释 + - [x] SubTask B1.3: 删除 Can.cpp 中 `sendDATA_ACK(u32, bool)` 实现及其注释 + +- [x] Task B2: 删除 `CAN_transmiter::sendDATA_ACK(u32, u8*)` 兼容旧协议 ACK 接口 + - [x] SubTask B2.1: 删除 Can.h 中 `sendDATA_ACK(u32 packet_num, u8 *data)` 声明及其 Doxygen 注释 + - [x] SubTask B2.2: 删除 Can.cpp 中 `sendDATA_ACK(u32, u8*)` 实现及其注释 + - [x] SubTask B2.3: 全工程再 Grep `sendDATA_ACK`，确认 0 命中 + +- [x] Task B3: 删除 `can_dma_init()` 占位函数 + - [x] SubTask B3.1: Grep `can_dma_init` 全工程，确认仅在 Can.h 声明、Can.cpp 实现及 `canfd_config()` 内被调用 + - [x] SubTask B3.2: 删除 Can.h 中 `can_dma_init` 声明及其 Doxygen 注释 + - [x] SubTask B3.3: 删除 Can.cpp 中 `can_dma_init` 实现及其注释 + - [x] SubTask B3.4: 删除 Can.cpp 中 `canfd_config()` 内对 `can_dma_init()` 的调用 + - [x] SubTask B3.5: 全工程再 Grep `can_dma_init`，确认 0 命中 + +## Group C — BootPlatform 预留占位接口 + +- [x] Task C1: 删除 `BootPlatform::getResetCause()` 与 C 包装 `get_reset_cause()` + - [x] SubTask C1.1: Grep `getResetCause` 与 `get_reset_cause` 全工程，确认无任何调用方 + - [x] SubTask C1.2-C1.5: 删除 initial.h 声明、C 包装声明；删除 initial.cpp 实现、C 包装实现 + +- [x] Task C2: 删除 `BootPlatform::initUnusedPins()` 与 C 包装 `unused_pin_init()` + - [x] SubTask C2.1-C2.5: 删除 initial.h/initial.cpp 中相关声明与实现 + +- [x] Task C3: 删除 `BootPlatform::printSystemClockInfo()` 与 C 包装 `print_system_clock_info()` + - [x] SubTask C3.1-C3.5: 删除 initial.h/initial.cpp 中相关声明与实现 + +- [x] Task C4: 删除 `BootPlatform::scanAppFlashContent()` 与 C 包装 `scanAppFlashContent()` + - [x] SubTask C4.1-C4.6: 删除 initial.h/initial.cpp 中相关声明与实现；删除已空的 `class BootPlatform` 整体 + +## Group D — SpiOta 保留空壳 ISR（调研驱动） + +- [x] Task D1: 调研并决策 `SpiOta::onSpiInterrupt()` 去留 + - [x] SubTask D1.1: SpiOta.cpp 中 `extern "C" void SPI0_IRQHandler(void)` 挂载于第 1026 行，调用 `SpiOta::onSpiInterrupt()` + - [x] SubTask D1.2: `configureSpi0()` 显式禁用 SPI0 的 RBNE/ERR/TBE 中断，且未调用 `nvic_irq_enable(SPI0_IRQn, ...)` → NVIC 未使能 SPI0 IRQ → ISR 永不触发 + - [x] SubTask D1.3: 决策为**删除**：删除 `onSpiInterrupt()` 实现/声明、`spi_ota_irq_handler` C 包装、`extern "C" SPI0_IRQHandler` + - [x] SubTask D1.4: 决策依据已记录（启动文件 [WEAK] 引用会自动落到默认空实现，不会有编译错误） + +## Group E — 开发期备份文件与冲突副本 + +- [x] Task E1: 删除 SpiOta 备份文件 + - [x] SubTask E1.1: 删除 `SpiOta.cpp.bak_20260528` + - [x] SubTask E1.2: 删除 `SpiOta.cpp.bak_20260529_txdebug` + +- [x] Task E2: 处理 RTE 冲突副本文件 + - [x] SubTask E2.1: 列出 `RTE\_Debug_GD32E508VE\` 目录所有 `RTE_Components*.h` 文件（共两份：正式版 + 冲突副本） + - [x] SubTask E2.2: 决策为**删除冲突副本**：冲突副本文件名带「冲突副本」字样，明显为 IDE 合并冲突产物；csolution 只引用 `RTE_Components.h`，副本不会被编译器引用；两份内容差异（CTC/SPI vs FWDGT）反映配置演进，正式版代表当前生效配置。删除 `RTE_Components(CBB-003的冲突副本1_2026-05-06 00-26-28).h` + +## Group F — C 风格包装层审计（仅审计，不强制删除） + +- [x] Task F1: 审计 systick.h C 包装是否有调用方 + - [x] SubTask F1.1: Grep `systick_config`、`delay_1ms`、`delay_1us`、`delay_1s`、`delay_decrement`、`system_soft_reset` 全工程 + - [x] SubTask F1.2: 删除无调用方的 C 包装：`systick_config`、`delay_1s`、`delay_decrement` + - [x] SubTask F1.3: 保留有调用方的 C 包装：`delay_1ms`（main.cpp/OTA.cpp/initial.cpp 调用）、`delay_1us`（OTA.cpp 调用）、`system_soft_reset`（initial.cpp 调用） + +- [x] Task F2: 审计 initial.h 中除 Group C 外剩余 C 包装是否有调用方 + - [x] SubTask F2.1: Grep `USART0_init`、`enter_deep_sleep`、`check_sleep`、`EXTI5_9_IRQHandler`、`EXTI10_15_IRQHandler` 全工程 + - [x] SubTask F2.2: 删除无调用方的 C 包装：`USART0_init`、`enter_deep_sleep`、`check_sleep` + - [x] SubTask F2.3: 保留有调用方的 C 包装：`EXTI5_9_IRQHandler`、`EXTI10_15_IRQHandler`（extern "C" IRQ Handler，启动文件 [WEAK] 引用） + +- [x] Task F3: 审计 SpiOta.h C 包装是否有调用方 + - [x] SubTask F3.1: Grep 全部 11 个 `spi_ota_*` C 包装（含 `spi_ota_cs_exti_handler`）全工程 + - [x] SubTask F3.2: 删除无调用方的 C 包装：`spi_ota_init`、`spi_ota_poll`、`spi_ota_service_pending_tx`、`spi_ota_set_sender`、`spi_ota_on_cs_assert`、`spi_ota_on_cs_release`、`spi_ota_gpio_init`、`spi_ota_spi0_init`、`spi_ota_exti_init`、`spi_ota_irq_handler`、`spi_ota_has_tx_frame`、`spi_ota_cs_exti_handler`（initial.cpp 中的引用仅为注释，实际调用走 `SpiOta::onCsExtiInterrupt()` C++ 静态方法） + - [x] SubTask F3.3: 删除空的 `#ifdef __cplusplus extern "C" { ... } #endif` 块；同步更新 initial.cpp 中提到 `spi_ota_cs_exti_handler` 的注释 + +- [x] Task F4: 审计 Can.h C 包装是否有调用方 + - [x] SubTask F4.1: Grep `can_gpio_init`、`canfd_config`、`CAN0_RX0_IRQHandler`、`CAN0_TX_IRQHandler` 全工程 + - [x] SubTask F4.2: `can_gpio_init` / `canfd_config` 仅在 Can.cpp 内被同类方法调用，但删除会破坏调用点（需内联修改），违反"严格只做删除"约束 → **保留** + - [x] SubTask F4.3: `CAN0_RX0_IRQHandler` / `CAN0_TX_IRQHandler` 为 extern "C" IRQ Handler，启动文件 [WEAK] 引用 → **保留** + +## Group G — 注释同步与最终验证 + +- [x] Task G1: 注释一致性扫描 + - [x] SubTask G1.1: 全工程 Grep 19 个已删符号名 + - [x] SubTask G1.2: 所有 19 个符号命中数为 0（`SPI0_IRQHandler` 仅在启动文件 [WEAK] 处出现，属预期） + +- [x] Task G2: 编译验证 + - [x] SubTask G2.1: 检查编译工具链可用性：`cbuild` / `csolution` / `cbuildgen` 均不在 PATH；`CMSIS_COMPILER_ROOT` / `CMSIS_PACK_ROOT` 未设置；ARM 编译器（armcc/armclang/arm-none-eabi-gcc）均未找到 + - [ ] SubTask G2.2: **编译验证未执行**——当前环境编译工具链不可用，需用户在装有 CMSIS-Toolbox 的本地 IDE 中手动编译验证。验证清单已提供给用户 + - [x] SubTask G2.3: 注释一致性扫描通过，无遗漏引用 + +# Task Dependencies + +- Group A、B、C、E 之间无依赖，可并行执行。✅ 已全部完成 +- Task D 依赖对 SpiOta.cpp 中 NVIC 配置的调研。✅ 已完成（决策为删除） +- Group F（C 包装审计）依赖 Group A/B/C 完成后才能得到「剩余待审计」的精确清单。✅ 已完成 +- Task G1 依赖所有删除任务完成。✅ 已完成 +- Task G2 依赖 G1 完成。⚠️ 注释扫描已完成；编译验证因工具链不可用未执行，需用户本地验证 + +# 用户应在本地 IDE 执行的编译验证清单 + +1. 确认 `cbuild.exe` / `csolution.exe` 在 PATH，或 `CMSIS_COMPILER_ROOT` / `CMSIS_PACK_ROOT` 已设置 +2. （可选）清理上次构建产物：`out/`、`*.cbuild-idx.yml`、`*.cbuild-pack.yml`、`*.cgen.yml` +3. 执行编译： + ``` + cd g:\Robot_v2\DCU\GD32E50X_BOOT + cbuild GD32E50X_BOOT.csolution.yml + ``` + 或 + ``` + csolution convert -s GD32E50X_BOOT.csolution.yml + cbuild GD32E50X_BOOT+GD32E508VE.Debug.cgen.yml + ``` +4. 验证通过标准：编译 0 error；无新增 warning；链接成功生成 `out\GD32E50X_BOOT+GD32E508VE\Debug\GD32E50X_BOOT.axf`（或 `.elf`）及 `.hex`/`.bin` +5. 若出现 `undefined reference to XXX`：对照 G1 表核实是否仍有调用点（已确认全工程 0 命中，理论上不会出现） diff --git a/GD32E50X_BOOT/RTE/_Debug_GD32E508VE/RTE_Components(CBB-003的冲突副本1_2026-05-06 00-26-28).h b/GD32E50X_BOOT/RTE/_Debug_GD32E508VE/RTE_Components(CBB-003的冲突副本1_2026-05-06 00-26-28).h deleted file mode 100644 index 295dafd..0000000 --- a/GD32E50X_BOOT/RTE/_Debug_GD32E508VE/RTE_Components(CBB-003的冲突副本1_2026-05-06 00-26-28).h +++ /dev/null @@ -1,48 +0,0 @@ -/* - * CSOLUTION generated file: DO NOT EDIT! - * Generated by: csolution version 2.13.0+p3-g40b6564e - * - * Project: 'GD32E50X_BOOT.Debug+GD32E508VE' - * Target: 'Debug+GD32E508VE' - */ - -#ifndef RTE_COMPONENTS_H -#define RTE_COMPONENTS_H - - -/* - * Define the Device Header File: - */ -#define CMSIS_device_header "gd32e50x.h" - -/* GigaDevice::Device:GD32E50x_StdPeripherals:ADC@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_ADC -/* GigaDevice::Device:GD32E50x_StdPeripherals:CAN@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_CAN -/* GigaDevice::Device:GD32E50x_StdPeripherals:CRC@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_CRC -/* GigaDevice::Device:GD32E50x_StdPeripherals:DBG@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_DBG -/* GigaDevice::Device:GD32E50x_StdPeripherals:DMA@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_DMA -/* GigaDevice::Device:GD32E50x_StdPeripherals:EXTI@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_EXTI -/* GigaDevice::Device:GD32E50x_StdPeripherals:FMC@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_FMC -/* GigaDevice::Device:GD32E50x_StdPeripherals:FWDGT@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_FWDGT -/* GigaDevice::Device:GD32E50x_StdPeripherals:GPIO@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_GPIO -/* GigaDevice::Device:GD32E50x_StdPeripherals:MISC@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_MISC -/* GigaDevice::Device:GD32E50x_StdPeripherals:PMU@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_PMU -/* GigaDevice::Device:GD32E50x_StdPeripherals:RCU@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_RCU -/* GigaDevice::Device:GD32E50x_StdPeripherals:TIMER@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_TIMER -/* GigaDevice::Device:GD32E50x_StdPeripherals:USART@1.6.0 */ -#define RTE_DEVICE_STDPERIPHERALS_USART - - -#endif /* RTE_COMPONENTS_H */ diff --git a/GD32E50X_BOOT/device/Can.cpp b/GD32E50X_BOOT/device/Can.cpp index 2ef3d6c..e22e203 100644 --- a/GD32E50X_BOOT/device/Can.cpp +++ b/GD32E50X_BOOT/device/Can.cpp @@ -53,18 +53,6 @@ u8 can_gpio_init(void) { return 0; } -/** - * @brief CAN DMA 初始化占位函数。 - * - * @return 无。 - */ -void can_dma_init(void) { - /** - * 预留接口：当前接收路径直接从 FIFO 取数， - * 这样实现更简单，也避免 DMA 与中断抢数据的时序问题。 - */ -} - /** * @brief 配置 CAN-FD 控制器、滤波器和中断。 * @@ -77,7 +65,6 @@ void canfd_config(void) { can_filter_parameter_struct can_filter; can_gpio_init(); - can_dma_init(); can_deinit(CAN0); /** @@ -303,28 +290,6 @@ u8 CAN_transmiter::CANFD_AsyncSend(u32 id, u8 length, u8 *data) { return 1; } -/** - * @brief 发送带 4 字节业务数据的 OTA ACK。 - * - * @param packet_num 分包序号，当前实现未使用。 - * @param data 指向业务数据缓冲区的指针。 - * @return 1 表示发送成功或成功入队，0 表示失败。 - */ -u8 CAN_transmiter::sendDATA_ACK(u32 packet_num, u8 *data) { - /** - * 兼容旧协议格式： - * 前两个字节固定为 0xBA 0xC0，后四字节带上包号/状态信息。 - */ - (void)packet_num; - memset(transmit_data, 0, sizeof(transmit_data)); - transmit_data[0] = 0xBA; - transmit_data[1] = 0xC0; - for (int i = 2; i < 6; ++i) { - transmit_data[i] = data[i - 2]; - } - return CANFD_AsyncSend(CANFD_OTA_ACK_ID, 6, transmit_data); -} - /** * @brief 发送备份恢复提示帧。 * @@ -338,28 +303,6 @@ u8 CAN_transmiter::sendBackup() { return CANFD_AsyncSend(CANFD_OTA_ACK_ID, 0, transmit_data); } -/** - * @brief 发送简化 OTA ACK。 - * - * @param packet_num 分包序号，当前实现未使用。 - * @param if_success 是否成功。 - * @return 1 表示发送成功或成功入队，0 表示失败。 - */ -u8 CAN_transmiter::sendDATA_ACK(u32 packet_num, bool if_success) { - /** - * 简化 ACK 版本： - * if_success 为真时写入 0x0001； - * 为假则发送全 0，保留给对端区分处理。 - */ - (void)packet_num; - memset(transmit_data, 0, 4); - if (if_success) { - transmit_data[0] = 0x01; - transmit_data[1] = 0x00; - } - return CANFD_AsyncSend(CANFD_OTA_ACK_ID, 4, transmit_data); -} - /** * @brief CAN RX0 中断实际处理逻辑。 * diff --git a/GD32E50X_BOOT/device/Can.h b/GD32E50X_BOOT/device/Can.h index a78b4d2..ea7e5e4 100644 --- a/GD32E50X_BOOT/device/Can.h +++ b/GD32E50X_BOOT/device/Can.h @@ -32,15 +32,6 @@ u8 can_gpio_init(void); */ void canfd_config(void); -/** - * @brief 初始化 CAN DMA 通道。 - * - * @note 当前实现未启用 DMA,该函数作为保留占位接口。 - * - * @return 无。 - */ -void can_dma_init(void); - /** * @brief CAN0 FIFO0 接收中断服务函数。 * @@ -118,22 +109,6 @@ class CAN_transmiter { */ u8 CANFD_AsyncSend(u32 ID, u8 length, u8 *data); - /** @brief 发送简化 OTA ACK 帧。 - * - * @param packet_num 分包序号，当前实现未实际使用。 - * @param if_success 是否成功，true 表示成功。 - * @return 1 表示发送成功或成功入队，0 表示失败。 - */ - u8 sendDATA_ACK(u32 packet_num, bool if_success); - - /** @brief 发送带 4 字节有效载荷的 OTA ACK 帧。 - * - * @param packet_num 分包序号，当前实现未实际使用。 - * @param data 指向 4 字节业务数据的缓冲区指针。 - * @return 1 表示发送成功或成功入队，0 表示失败。 - */ - u8 sendDATA_ACK(u32 packet_num, u8 *data); - /** @brief 发送「备份恢复」提示帧。 * * @return 1 表示发送成功或成功入队，0 表示失败。 diff --git a/GD32E50X_BOOT/device/SpiOta.cpp b/GD32E50X_BOOT/device/SpiOta.cpp index 6ffc4a0..1fbe3f1 100644 --- a/GD32E50X_BOOT/device/SpiOta.cpp +++ b/GD32E50X_BOOT/device/SpiOta.cpp @@ -895,24 +895,6 @@ void SpiOta::onCsRelease() { updateDataReady(); } -/* ============================================================ - * SPI0 中断处理 — RX 已改 DMA，保留空壳兼容旧 IRQ 向量 - * ============================================================ */ - -void SpiOta::onSpiInterrupt() { - /* 遗留 ERR 标志清理（RBNE 已禁用） */ - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RBNE) != RESET) { - (void)spi_i2s_data_receive(SPI0); - } - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RXORERR) != RESET) { - (void)spi_i2s_data_receive(SPI0); - (void)SPI_STAT(SPI0); - } - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_FERR) != RESET) { - spi_i2s_format_error_clear(SPI0, SPI_FLAG_FERR); - } -} - void SpiOta::onCsExtiInterrupt() { if (exti_interrupt_flag_get(EXTI_15) != RESET) { exti_interrupt_flag_clear(EXTI_15); @@ -1006,23 +988,3 @@ bool SpiOta::hasTxFrame() { return s_txPending; } u8 SpiOta::pendingDataCount() { return s_otaDataCnt; } -/* ============================================================ - * C 风格兼容包装 - * ============================================================ */ - -void spi_ota_init(void) { SpiOta::init(); } -void spi_ota_poll(void) { SpiOta::poll(); } -void spi_ota_service_pending_tx(void) { SpiOta::servicePendingTx(); } -void spi_ota_set_sender(OTAHandler *handler) { SpiOta::setSender(handler); } -void spi_ota_on_cs_assert(void) { SpiOta::onCsAssert(); } -void spi_ota_on_cs_release(void) { SpiOta::onCsRelease(); } -void spi_ota_gpio_init(void) { SpiOta::configureGpio(); } -void spi_ota_spi0_init(void) { SpiOta::configureSpi0(); } -void spi_ota_exti_init(void) { SpiOta::configureExti(); } -void spi_ota_irq_handler(void) { SpiOta::onSpiInterrupt(); } -void spi_ota_cs_exti_handler(void) { SpiOta::onCsExtiInterrupt(); } -bool spi_ota_has_tx_frame(void) { return SpiOta::hasTxFrame(); } - -extern "C" void SPI0_IRQHandler(void) { - SpiOta::onSpiInterrupt(); -} diff --git a/GD32E50X_BOOT/device/SpiOta.cpp.bak_20260528 b/GD32E50X_BOOT/device/SpiOta.cpp.bak_20260528 deleted file mode 100644 index a0b6753..0000000 --- a/GD32E50X_BOOT/device/SpiOta.cpp.bak_20260528 +++ /dev/null @@ -1,608 +0,0 @@ -#include "SpiOta.h" -#include "core_cm33.h" -#include "gd32e50x_gpio.h" -#include "gd32e50x_exti.h" -#include "gd32e50x_rcu.h" -#include "gd32e50x_misc.h" -#include "gd32e50x_spi.h" -#include "gd32e50x_dma.h" -#include -#include - -/** - * @file SpiOta.cpp - * @brief Bootloader SPI OTA 从机模块实现(DMA 发送版)。 - * - * CS 释放后,EXTI ISR 直接完成帧校验+处理+响应准备, - * 响应帧写入 s_txBuf,DMA0_CH2 自动推到 SPI0 TX 寄存器。 - * 硬件 DMA 替代 TBE 中断逐字节发送,消除字节错位问题。 - * - * GD32E50x DMA 通道映射(固定): - * DMA0_CH0: ADC0, DMA0_CH1: SPI0_RX, DMA0_CH2: SPI0_TX - * DMA0_CH3: SPI1_RX, DMA0_CH4: SPI1_TX - */ - -/* SPI0_TX = DMA0_CH2 (GD32E50x 固定映射) */ -#define SPI_TX_DMA_PERIPH DMA0 -#define SPI_TX_DMA_CH DMA_CH2 - -/* 调试：多缓冲日志（8 条 ISR 日志缓冲区，避免关键事件丢失） */ -static constexpr u16 kLogSlotCount = 8U; -static constexpr u16 kLogSlotSize = 80U; -static u8 s_logSlots[kLogSlotCount][kLogSlotSize] = {}; -static u16 s_logSlotLens[kLogSlotCount] = {}; -static volatile u8 s_logWriteIdx = 0U; -static volatile u8 s_logReadIdx = 0U; -static volatile u8 s_logCount = 0U; - -static void isrLogFrameEx(const char *tag, const u8 *data, u16 len); - - -OTAHandler *SpiOta::s_otaHandler = nullptr; -volatile bool SpiOta::s_csActive = false; -volatile u16 SpiOta::s_rxStreamLen = 0U; -volatile u16 SpiOta::s_txStreamPos = 0U; -u8 SpiOta::s_rxStream[SpiOta::kRxBufSize] = {}; -u8 SpiOta::s_txBuf[SpiOta::kTxBufSize] = {}; -volatile u16 SpiOta::s_txLen = 0U; -volatile bool SpiOta::s_txPending = false; -volatile bool SpiOta::s_isrResponded = false; -volatile bool SpiOta::s_isrLogPending = false; -volatile bool SpiOta::s_otaSessionActive = false; -u8 SpiOta::s_isrLogBuf[kIsrLogBufSize] = {}; -u16 SpiOta::s_isrLogLen = 0U; - -/* ============================================================ - * CRC8 / 帧构建 / 帧解析 - * ============================================================ */ - -u8 SpiOta::crc8(const u8 *data, u16 len) { - u8 crc = 0x00U; - for (u16 i = 0; i < len; ++i) { crc ^= data[i]; } - return crc; -} - -bool SpiOta::buildFrame(spi_ota_frame_type_t type, const u8 *payload, u16 payload_len, - u8 *out, u16 *out_len) { - if (!out || !out_len) { return false; } - u16 total_len = (u16)(kFrameOverhead + payload_len); - if (total_len > kTxBufSize) { return false; } - out[0] = kSof; - out[1] = (u8)type; - out[2] = (u8)(payload_len & 0xFFU); - out[3] = (u8)((payload_len >> 8) & 0xFFU); - if (payload && payload_len > 0U) { memcpy(&out[4], payload, payload_len); } - out[4 + payload_len] = crc8(out, (u16)(4U + payload_len)); - *out_len = total_len; - return true; -} - -bool SpiOta::tryParseFrame(const u8 *frame, u16 frame_len, - spi_ota_frame_type_t *type, - const u8 **payload, - u16 *payload_len) { - if (!frame || frame_len < kFrameOverhead) { return false; } - if (frame[0] != kSof) { return false; } - u16 len = (u16)(frame[2] | ((u16)frame[3] << 8)); - u16 expected = (u16)(kFrameOverhead + len); - if (expected != frame_len) { return false; } - if (crc8(frame, (u16)(4U + len)) != frame[4U + len]) { return false; } - if (type) { *type = (spi_ota_frame_type_t)frame[1]; } - if (payload) { *payload = &frame[4]; } - if (payload_len){ *payload_len = len; } - return true; -} - -ota_link_msg_type_t SpiOta::mapMsgType(spi_ota_frame_type_t type) { - switch (type) { - case SPI_OTA_CMD_REQ: return OTA_LINK_MSG_REQ; - case SPI_OTA_CMD_DATA: return OTA_LINK_MSG_DATA; - default: return OTA_LINK_MSG_CANCEL; - } -} - -/* ============================================================ - * 回包发送适配器(ISR 安全) - * ============================================================ */ - -void SpiOta::sender(ota_link_tx_type_t type, u8 len, const u8 *data) { - spi_ota_frame_type_t rsp_type = SPI_OTA_RSP_ACK; - if (type == OTA_LINK_TX_WAIT) { rsp_type = SPI_OTA_RSP_WAIT; } - u8 frame_buf[kTxBufSize] = {}; - u16 built_len = 0U; - if (buildFrame(rsp_type, data, len, frame_buf, &built_len)) { - queueTxFrame(frame_buf, built_len); - } -} - -uint32_t SpiOta::enterCritical() { - uint32_t primask = __get_PRIMASK(); - __disable_irq(); - return primask; -} - -void SpiOta::exitCritical(uint32_t primask) { - __set_PRIMASK(primask); -} - -/** @brief 启动 DMA 传输 s_txBuf → SPI0_DATA */ -static void startSpiTxDma(const u8 *buf, u16 len) { - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) &= ~DMA_CHXCTL_CHEN; - DMA_INTC(SPI_TX_DMA_PERIPH) |= DMA_FLAG_ADD(DMA_CHINTF_RESET_VALUE, SPI_TX_DMA_CH); - DMA_CHMADDR(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) = (uint32_t)buf; - DMA_CHCNT(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) = len; - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) |= DMA_CHXCTL_CHEN; -} - -void SpiOta::queueTxFrame(const u8 *frame, u16 len) { - if (!frame || len == 0U || len > kTxBufSize) { return; } - uint32_t primask = enterCritical(); - memcpy(s_txBuf, frame, len); - s_txLen = len; - s_txPending = true; - s_txStreamPos = 0U; - if (len < kTxBufSize) { memset(&s_txBuf[len], 0xFF, kTxBufSize - len); } - exitCritical(primask); - spi_dma_enable(SPI0, SPI_DMA_TRANSMIT); - startSpiTxDma(s_txBuf, s_txLen); -} - -/* ============================================================ - * RBNE 中断内即时帧检测与处理 - * ============================================================ */ - -void SpiOta::tryProcessCompleteFrame() { - u16 rx_len = s_rxStreamLen; - if (rx_len == 0U) { return; } - - if (s_rxStream[0] == SPI_OTA_CMD_BOOT) { - isrProcessFrame(); - return; - } - - if (s_rxStream[0] != kSof) { return; } - if (rx_len < kFrameOverhead) { return; } - - u16 payload_len = (u16)(s_rxStream[2] | ((u16)s_rxStream[3] << 8)); - u16 expected = (u16)(kFrameOverhead + payload_len); - if (expected > kRxBufSize) { return; } - if (rx_len < expected) { return; } - - isrProcessFrame(); -} - -/* ============================================================ - * CS 释放后 ISR 直接帧处理 - * ============================================================ */ - -void SpiOta::isrProcessFrame() { - u16 rx_len = s_rxStreamLen; - if (rx_len == 0U) { return; } - - if (rx_len >= 1U && s_rxStream[0] == SPI_OTA_CMD_BOOT) { - u8 ack[4] = {0xB0U, 0xA5U, 0x4FU, 0x54U}; - queueTxFrame(ack, sizeof(ack)); - if (s_otaHandler != nullptr) { s_otaHandler->resetToIdle(); } - isrLogFrameEx("[BOOT-CMD]", s_txBuf, s_txLen); - s_isrResponded = true; - return; - } - - if (s_rxStream[0] != kSof) { - isrLogFrameEx("[NO-SOF]", s_rxStream, (rx_len < 8U) ? rx_len : 8U); - return; - } - if (rx_len < kFrameOverhead) { return; } - - u16 payload_len = (u16)(s_rxStream[2] | ((u16)s_rxStream[3] << 8)); - u16 expected = (u16)(kFrameOverhead + payload_len); - if (expected > rx_len) { return; } - - if (crc8(s_rxStream, (u16)(4U + payload_len)) != s_rxStream[4U + payload_len]) { - u8 err = 0xEEU; - u8 err_frame[kTxBufSize] = {}; - u16 built = 0U; - if (buildFrame(SPI_OTA_RSP_ERROR, &err, 1U, err_frame, &built)) { - queueTxFrame(err_frame, built); - } - isrLogFrameEx("[CRC-ERR]", s_rxStream, (rx_len < 8U) ? rx_len : 8U); - return; - } - - spi_ota_frame_type_t frame_type; - const u8 *payload = nullptr; - u16 p_len = 0U; - if (!tryParseFrame(s_rxStream, expected, &frame_type, &payload, &p_len)) { - isrLogFrameEx("[PARSE-ERR]", s_rxStream, (rx_len < 8U) ? rx_len : 8U); - return; - } - - if (frame_type == SPI_OTA_CMD_BOOT) { - u8 ack_ok = 0xA5U; - u8 ack_frame[kTxBufSize] = {}; - u16 ack_len = 0U; - if (buildFrame(SPI_OTA_RSP_ACK, &ack_ok, 1U, ack_frame, &ack_len)) { - queueTxFrame(ack_frame, ack_len); - } - if (s_otaHandler != nullptr) { s_otaHandler->resetToIdle(); } - isrLogFrameEx("[BOOT-FRAME]", s_txBuf, s_txLen); - s_isrResponded = true; - return; - } - - if (s_otaHandler != nullptr) { - s_otaSessionActive = true; - s_otaHandler->setTransportSender(sender); - ota_link_msg_type_t msg_type = mapMsgType(frame_type); - isrLogFrameEx("[OTA-MSG]", (const u8*)&frame_type, 1); - s_otaHandler->handleLinkMessage(msg_type, const_cast(payload), (u8)p_len); - } - - isrLogFrameEx("[ISR-DONE]", s_txBuf, (s_txLen < 16U) ? s_txLen : 16U); - s_isrResponded = true; -} - -/* ============================================================ - * ISR 内延迟日志 - * ============================================================ */ - -void SpiOta::isrLogFrame(const char *tag, const u8 *data, u16 len) { - isrLogFrameEx(tag, data, len); -} - -static void isrLogFrameEx(const char *tag, const u8 *data, u16 len) { - if (s_logCount >= kLogSlotCount) { return; } - u8 *buf = s_logSlots[s_logWriteIdx]; - u16 pos = 0U; - if (tag) { - while (*tag && pos < kLogSlotSize - 3U) { buf[pos++] = (u8)*tag++; } - buf[pos++] = ' '; - } - const u16 preview = (len < 24U) ? len : 24U; - for (u16 i = 0U; i < preview && pos + 2U < kLogSlotSize; ++i) { - static const char hx[] = "0123456789ABCDEF"; - buf[pos++] = (u8)hx[data[i] >> 4]; - buf[pos++] = (u8)hx[data[i] & 0x0FU]; - } - buf[pos] = 0U; - s_logSlotLens[s_logWriteIdx] = pos; - s_logWriteIdx = (s_logWriteIdx + 1U) % kLogSlotCount; - s_logCount++; -} - -/* ============================================================ - * 硬件初始化 - * ============================================================ */ - -void SpiOta::configureGpio() { - rcu_periph_clock_enable(RCU_GPIOA); - rcu_periph_clock_enable(RCU_GPIOB); - rcu_periph_clock_enable(RCU_AF); - gpio_pin_remap_config(GPIO_SPI0_REMAP, ENABLE); - gpio_pin_remap_config(GPIO_SWJ_SWDPENABLE_REMAP, ENABLE); - gpio_init(GPIOB, GPIO_MODE_IN_FLOATING, GPIO_OSPEED_50MHZ, GPIO_PIN_3 | GPIO_PIN_5); - gpio_init(GPIOB, GPIO_MODE_AF_PP, GPIO_OSPEED_50MHZ, GPIO_PIN_4); - gpio_init(GPIOB, GPIO_MODE_IN_FLOATING, GPIO_OSPEED_50MHZ, GPIO_PIN_6 | GPIO_PIN_7); - gpio_init(GPIOA, GPIO_MODE_IPU, GPIO_OSPEED_50MHZ, GPIO_PIN_15); -} - -void SpiOta::configureSpi0() { - rcu_periph_clock_enable(RCU_SPI0); - spi_i2s_deinit(SPI0); - (void)spi_i2s_data_receive(SPI0); /* 清空 SPI_DATA 残留值 */ - spi_parameter_struct spi_init_struct; - spi_struct_para_init(&spi_init_struct); - spi_init_struct.device_mode = SPI_SLAVE; - spi_init_struct.trans_mode = SPI_TRANSMODE_FULLDUPLEX; - spi_init_struct.frame_size = SPI_FRAMESIZE_8BIT; - spi_init_struct.nss = SPI_NSS_SOFT; - spi_init_struct.endian = SPI_ENDIAN_MSB; - /* SPI NSS 由 ESP32 的 CS 线 (GPIO4→PA15) 控制。\外部 CS 高时： - * spi_nss_internal_high(SPI0) 让 SPI 外设进入未选中状态， - * 这样 SPI0_DATA 输出 0xFF（而非 0x00）， - * DMA 只在 CS 低时才被 SPI 外设允许推数据到 MISO。 */ - spi_init_struct.clock_polarity_phase = SPI_CK_PL_LOW_PH_1EDGE; - spi_init_struct.prescale = SPI_PSC_8; - spi_init(SPI0, &spi_init_struct); - /* 关键修正：内部 NSS 始终保持 LOW，使 SPI0 从机一直处于“已选中”状态。 - * - * APP 侧的稳定实现就是这样做的：PA15 只作为 EXTI 事务边界信号， - * 不再依赖 EXTI ISR 去“赶在首个 SCK 之前”切 internal NSS。 - * - * 之前 BOOT 的问题正出在这里： - * - 空闲时 spi_nss_internal_high(SPI0) - * - CS↓ 后靠 EXTI ISR 再切 internal_low - * - 但 ESP32 在 CS↓ 后几乎立刻出时钟 - * - 于是 GD32 常常在 internal NSS 真正拉低前丢掉首字节/首几位 - * → 请求帧首字节 0xA5 被吃掉，看到 0x80 / 0xFF / 随机 0x07 - * - * 现在改为 internal NSS 常低： - * - SPI 从机始终就绪，不会因为 EXTI 延迟丢首字节 - * - PA15 仍用于 onCsAssert/onCsRelease 做缓冲区清零和 DMA 重装 - */ - spi_nss_internal_low(SPI0); - /* 初始化时清空 SPI_DATA 寄存器，防止残留值影响首次传输 */ - (void)spi_i2s_data_receive(SPI0); - spi_i2s_interrupt_enable(SPI0, SPI_I2S_INT_RBNE); - spi_i2s_interrupt_enable(SPI0, SPI_I2S_INT_ERR); - spi_i2s_interrupt_disable(SPI0, SPI_I2S_INT_TBE); - /* SPI0 RBNE 中断优先级设为 (2, 0)，低于 EXTI CS 中断 (1, 1)。 - * 这确保 CS 下降沿的 onCsAssert() 在任何 RBNE 之前执行， - * 否则 s_csActive 仍为 false，RBNE 中断丢弃接收字节。 */ - nvic_irq_enable(SPI0_IRQn, 2U, 0U); - spi_enable(SPI0); -} - -void SpiOta::configureDma() { - rcu_periph_clock_enable(RCU_DMA0); - dma_deinit(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH); - dma_parameter_struct dma_init_struct; - dma_struct_para_init(&dma_init_struct); - dma_init_struct.periph_addr = (uint32_t)(SPI0 + 0x0CU); - dma_init_struct.periph_width = DMA_PERIPHERAL_WIDTH_8BIT; - dma_init_struct.memory_addr = (uint32_t)s_txBuf; - dma_init_struct.memory_width = DMA_MEMORY_WIDTH_8BIT; - dma_init_struct.number = 0U; - dma_init_struct.priority = DMA_PRIORITY_HIGH; - dma_init_struct.periph_inc = DMA_PERIPH_INCREASE_DISABLE; - dma_init_struct.memory_inc = DMA_MEMORY_INCREASE_ENABLE; - dma_init_struct.direction = DMA_MEMORY_TO_PERIPHERAL; - dma_init(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH, &dma_init_struct); - dma_circulation_disable(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH); - dma_channel_disable(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH); -} - -void SpiOta::configureExti() { - gpio_exti_source_select(GPIO_PORT_SOURCE_GPIOA, GPIO_PIN_SOURCE_15); - exti_init(EXTI_15, EXTI_INTERRUPT, EXTI_TRIG_BOTH); - exti_interrupt_enable(EXTI_15); - exti_interrupt_flag_clear(EXTI_15); - nvic_irq_enable(EXTI10_15_IRQn, 1U, 1U); -} - -void SpiOta::init() { - s_csActive = false; - s_rxStreamLen = 0U; - s_txStreamPos = 0U; - s_txLen = 0U; - s_txPending = false; - s_isrLogPending = false; - memset(s_rxStream, 0, sizeof(s_rxStream)); - memset(s_txBuf, 0xFF, sizeof(s_txBuf)); - configureGpio(); - configureDma(); - configureSpi0(); - configureExti(); -} - -/* ============================================================ - * CS 边沿处理 - * ============================================================ */ - -void SpiOta::onCsAssert() { - if (s_csActive) { return; } - s_csActive = true; - s_rxStreamLen = 0U; - s_txStreamPos = 0U; - s_isrResponded = false; - - /* 内部 NSS 已在初始化时常置 LOW，这里不再依赖 EXTI 时序切 NSS。 - * PA15 仅作为“事务开始”的边界信号，用于清接收缓冲和重装 DMA。 */ - - /* 关键修复：先彻底停止 DMA，再重新配置并启动。 - * - * 上一版预填 SPI_DATA 方案失败原因： - * spi_i2s_data_transmit() 写入的字节进入 TBE 发送缓冲区。 - * 当 ESP32 第一个 SCK 边沿到来时，TBE 的预填字节被移入移位 - * 寄存器推到 MISO，同时移位寄存器中的 MOSI 数据被推入 RBNE。 - * 但 GD32E50x SPI 的 RBNE 标志在 SPI_DATA 写入后可能被误触发， - * 导致 isrLogFrame 报出预填值（如 0x80）而非 MOSI 真实数据。 - * - * 正确方案：不预填 SPI_DATA，而是确保 DMA 在 CS↓ 后、ESP32 - * 第一个 SCK 边沿之前就绪。DMA 第一拍自动把 s_txBuf[0] 写入 - * SPI_DATA，不会产生 RBNE 误触发。 - * - * 操作顺序： - * 1. 禁 DMA 通道 + 禁 SPI DMA 请求（彻底停止旧传输） - * 2. 清 RBNE / RXORERR / FERR 等残留标志 - * 3. 配置并启动 DMA（此时 RBNE 仍关闭，DMA 写 SPI_DATA 不会触发中断） - * 4. 重新使能 RBNE 中断，开始接收 MOSI 数据 - */ - - /* 1. 停止旧 DMA */ - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) &= ~DMA_CHXCTL_CHEN; - spi_dma_disable(SPI0, SPI_DMA_TRANSMIT); - - /* 2. 清残留错误标志 - * - * 关键：这里绝不能在 CS↓ 时“顺手读掉 RBNE”。 - * - * 因为 internal NSS 现在常 LOW，ESP32 在 CS↓ 后很快就会发出首字节 A5。 - * 如果 EXTI ISR 略晚于 SPI 接收，A5 可能已经先进了 RX 缓冲并置位 RBNE。 - * 这时若在 onCsAssert() 里执行 spi_i2s_data_receive()，就会把真实首字节 A5 - * 当成“残留垃圾”吞掉，导致后续看到的首字节变成 0x01。 - * - * 所以这里仅清真正的错误标志，不主动消费 RBNE。 - * 已到达的首字节留给 onSpiInterrupt() 正常读走。 */ - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RXORERR) != RESET) { - (void)spi_i2s_data_receive(SPI0); - } - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_FERR) != RESET) { - spi_i2s_format_error_clear(SPI0, SPI_FLAG_FERR); - } - - /* 3. 配置并启动 DMA（RBNE 仍关闭，DMA 写 SPI_DATA 不会触发中断） */ - if (s_txPending && s_txLen > 0U) { - spi_dma_enable(SPI0, SPI_DMA_TRANSMIT); - startSpiTxDma(s_txBuf, s_txLen); - } else { - s_txPending = false; - s_txLen = kTxBufSize; - memset(s_txBuf, 0xFF, kTxBufSize); - spi_dma_enable(SPI0, SPI_DMA_TRANSMIT); - startSpiTxDma(s_txBuf, s_txLen); - } - - /* 4. 重新使能 RBNE 中断，开始接收 MOSI 数据 */ - spi_i2s_interrupt_enable(SPI0, SPI_I2S_INT_RBNE); -} -void SpiOta::onCsRelease() { - if (!s_csActive) { return; } - s_csActive = false; - - /* 内部 NSS 保持常 LOW，不在 CS↑ 时切回 HIGH。 - * 这样 SPI 从机始终处于就绪态，下一次 CS↓ 不会因 EXTI 延迟丢首字节。 - * - * 这里只停止 DMA，避免 CS 高期间继续空转消耗 s_txBuf 内容。 - * s_txPending 保持不变——如果 ISR 已准备好 ACK 帧， - * 下次 CS↓ 时 onCsAssert 会重启 DMA 推出。 */ - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) &= ~DMA_CHXCTL_CHEN; - spi_dma_disable(SPI0, SPI_DMA_TRANSMIT); - - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RBNE) != RESET) { - u8 last_byte = (u8)(spi_i2s_data_receive(SPI0) & 0xFFU); - if (s_rxStreamLen < kRxBufSize) { - s_rxStream[s_rxStreamLen++] = last_byte; - } - } - - /* 关键调试日志：记录本次 CS 周期结束时到底收到了多少字节。 - * 这样能直接判断： - * 1) 是不是完整收到了 ESP32 发来的整帧 - * 2) 还是只收到了首字节 A5 / 少数字节，导致永远无法解析回包 */ - if (s_rxStreamLen > 0U) { - isrLogFrame("[CS-RX]", s_rxStream, (s_rxStreamLen < 12U) ? s_rxStreamLen : 12U); - } - - if (s_rxStreamLen > 0U && !s_isrResponded) { - isrProcessFrame(); - } -} - -/* ============================================================ - * SPI0 中断处理 — 只处理 RBNE（接收）和错误，TBE 已由 DMA 替代 - * ============================================================ */ - -void SpiOta::onSpiInterrupt() { - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RBNE) != RESET) { - /* 与 APP 侧稳定实现保持一致： - * 如果 EXTI 下降沿因为中断时序略晚到达，但此时 PA15 已经为低， - * 则在首个 RBNE 到来时立即补做 onCsAssert()。 - * - * 关键限制：只能在“本事务尚未收过任何字节”时补做。 - * 否则若 EXTI / SPI 时序偶发抖动，重复 onCsAssert() 会把 - * s_rxStreamLen 清零，导致已经收到的半帧被冲掉，最终永远拼不成完整帧。 */ - if (!s_csActive && s_rxStreamLen == 0U && gpio_input_bit_get(GPIOA, GPIO_PIN_15) == RESET) { - onCsAssert(); - } - - u8 byte = (u8)(spi_i2s_data_receive(SPI0) & 0xFFU); - if (s_rxStreamLen < kRxBufSize) { - s_rxStream[s_rxStreamLen++] = byte; - } - /* 调试阶段先关闭 RBNE 首字节日志。 - * 原因：ISR 日志缓冲是单槽的，RBNE 一旦先占位， - * onCsRelease() 里的 [CS-RX] 摘要日志就会被丢弃。 - * 目前排查重点是“本次 CS 周期总共收到了哪些字节”， - * 所以优先保留 [CS-RX]，暂时关闭 [RBNE]。 */ - tryProcessCompleteFrame(); - } - /* TBE 已由 DMA 替代，不再处理 */ - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RXORERR) != RESET) { - (void)spi_i2s_data_receive(SPI0); - s_rxStreamLen = 0U; - isrLogFrame("[RXOR]", (const u8*)"OVR", 3); - } - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_FERR) != RESET) { - spi_i2s_format_error_clear(SPI0, SPI_FLAG_FERR); - } -} - -void SpiOta::onCsExtiInterrupt() { - if (exti_interrupt_flag_get(EXTI_15) != RESET) { - exti_interrupt_flag_clear(EXTI_15); - if (gpio_input_bit_get(GPIOA, GPIO_PIN_15) == RESET) { onCsAssert(); } - else { onCsRelease(); } - } -} - -/* ============================================================ - * 主循环接口 - * ============================================================ */ - -void SpiOta::poll() { - if (s_isrResponded) { - s_isrResponded = false; - } - /* 打印 ISR 日志缓冲区 */ - while (s_logCount > 0U) { - __disable_irq(); - if (s_logCount == 0U) { __enable_irq(); break; } - u8 *buf = s_logSlots[s_logReadIdx]; - u8 nextRead = (s_logReadIdx + 1U) % kLogSlotCount; - s_logReadIdx = nextRead; - s_logCount--; - __enable_irq(); - printf("[SPI] %s\r\n", buf); - } - if (s_otaHandler != nullptr) { s_otaHandler->process(); } - - /* 调试:定期打印 SPI 从机状态 */ - static u32 dbg_tick = 0; - dbg_tick++; - if ((dbg_tick % 5000U) == 0U) { - printf("[SPI] csActive=%u rxLen=%u txPend=%u txLen=%u isrResp=%u otaState=%u\r\n", - (unsigned)s_csActive, (unsigned)s_rxStreamLen, - (unsigned)s_txPending, (unsigned)s_txLen, - (unsigned)s_isrResponded, - (unsigned)(s_otaHandler ? s_otaHandler->getState() : 0xFF)); - } -} - -void SpiOta::setSender(OTAHandler *handler) { s_otaHandler = handler; } - -void SpiOta::feedRxBytes(const u8 *data, u16 len) { - if (!data || len == 0U || !s_csActive) { return; } - if (len > kRxBufSize) { data += (len - kRxBufSize); len = kRxBufSize; } - memcpy(s_rxStream, data, len); - s_rxStreamLen = len; -} - -bool SpiOta::hasTxFrame() { return s_txPending; } - -bool SpiOta::hasActiveSession() { return s_otaSessionActive; } - -u16 SpiOta::copyTxFrame(u8 *out, u16 max_len) { - if (!out || max_len == 0U) { return 0U; } - uint32_t primask = enterCritical(); - if (!s_txPending || s_txLen == 0U) { exitCritical(primask); return 0U; } - u16 copy_len = (s_txLen < max_len) ? s_txLen : max_len; - memcpy(out, s_txBuf, copy_len); - s_txPending = false; - s_txLen = 0U; - s_txStreamPos = 0U; - exitCritical(primask); - return copy_len; -} - -/* ============================================================ - * C 风格兼容包装 - * ============================================================ */ - -void spi_ota_init(void) { SpiOta::init(); } -void spi_ota_poll(void) { SpiOta::poll(); } -void spi_ota_set_sender(OTAHandler *handler) { SpiOta::setSender(handler); } -void spi_ota_on_cs_assert(void) { SpiOta::onCsAssert(); } -void spi_ota_on_cs_release(void) { SpiOta::onCsRelease(); } -void spi_ota_gpio_init(void) { SpiOta::configureGpio(); } -void spi_ota_spi0_init(void) { SpiOta::configureSpi0(); } -void spi_ota_exti_init(void) { SpiOta::configureExti(); } -void spi_ota_irq_handler(void) { SpiOta::onSpiInterrupt(); } -void spi_ota_cs_exti_handler(void) { SpiOta::onCsExtiInterrupt(); } -void spi_ota_feed_rx_bytes(const u8 *data, u16 len) { SpiOta::feedRxBytes(data, len); } -bool spi_ota_has_tx_frame(void) { return SpiOta::hasTxFrame(); } -u16 spi_ota_copy_tx_frame(u8 *out, u16 max_len) { return SpiOta::copyTxFrame(out, max_len); } - -extern "C" void SPI0_IRQHandler(void) { - SpiOta::onSpiInterrupt(); -} diff --git a/GD32E50X_BOOT/device/SpiOta.cpp.bak_20260529_txdebug b/GD32E50X_BOOT/device/SpiOta.cpp.bak_20260529_txdebug deleted file mode 100644 index dc775a8..0000000 --- a/GD32E50X_BOOT/device/SpiOta.cpp.bak_20260529_txdebug +++ /dev/null @@ -1,626 +0,0 @@ -#include "SpiOta.h" -#include "core_cm33.h" -#include "gd32e50x_gpio.h" -#include "gd32e50x_exti.h" -#include "gd32e50x_rcu.h" -#include "gd32e50x_misc.h" -#include "gd32e50x_spi.h" -#include "gd32e50x_dma.h" -#include -#include - -/** - * @file SpiOta.cpp - * @brief Bootloader SPI OTA 从机模块实现(DMA 发送版)。 - * - * CS 释放后,EXTI ISR 直接完成帧校验+处理+响应准备, - * 响应帧写入 s_txBuf,DMA0_CH2 自动推到 SPI0 TX 寄存器。 - * 硬件 DMA 替代 TBE 中断逐字节发送,消除字节错位问题。 - * - * GD32E50x DMA 通道映射(固定): - * DMA0_CH0: ADC0, DMA0_CH1: SPI0_RX, DMA0_CH2: SPI0_TX - * DMA0_CH3: SPI1_RX, DMA0_CH4: SPI1_TX - */ - -/* SPI0_TX = DMA0_CH2 (GD32E50x 固定映射) */ -#define SPI_TX_DMA_PERIPH DMA0 -#define SPI_TX_DMA_CH DMA_CH2 - -/* 调试：多缓冲日志（8 条 ISR 日志缓冲区，避免关键事件丢失） */ -static constexpr u16 kLogSlotCount = 8U; -static constexpr u16 kLogSlotSize = 80U; -static u8 s_logSlots[kLogSlotCount][kLogSlotSize] = {}; -static u16 s_logSlotLens[kLogSlotCount] = {}; -static volatile u8 s_logWriteIdx = 0U; -static volatile u8 s_logReadIdx = 0U; -static volatile u8 s_logCount = 0U; - -static void isrLogFrameEx(const char *tag, const u8 *data, u16 len); - - -OTAHandler *SpiOta::s_otaHandler = nullptr; -volatile bool SpiOta::s_csActive = false; -volatile u16 SpiOta::s_rxStreamLen = 0U; -volatile u16 SpiOta::s_txStreamPos = 0U; -u8 SpiOta::s_rxStream[SpiOta::kRxBufSize] = {}; -u8 SpiOta::s_txBuf[SpiOta::kTxBufSize] = {}; -volatile u16 SpiOta::s_txLen = 0U; -volatile bool SpiOta::s_txPending = false; -volatile bool SpiOta::s_isrResponded = false; -volatile bool SpiOta::s_isrLogPending = false; -volatile bool SpiOta::s_otaSessionActive = false; -u8 SpiOta::s_isrLogBuf[kIsrLogBufSize] = {}; -u16 SpiOta::s_isrLogLen = 0U; - -/* ============================================================ - * CRC8 / 帧构建 / 帧解析 - * ============================================================ */ - -u8 SpiOta::crc8(const u8 *data, u16 len) { - u8 crc = 0x00U; - for (u16 i = 0; i < len; ++i) { crc ^= data[i]; } - return crc; -} - -bool SpiOta::buildFrame(spi_ota_frame_type_t type, const u8 *payload, u16 payload_len, - u8 *out, u16 *out_len) { - if (!out || !out_len) { return false; } - u16 total_len = (u16)(kFrameOverhead + payload_len); - if (total_len > kTxBufSize) { return false; } - out[0] = kSof; - out[1] = (u8)type; - out[2] = (u8)(payload_len & 0xFFU); - out[3] = (u8)((payload_len >> 8) & 0xFFU); - if (payload && payload_len > 0U) { memcpy(&out[4], payload, payload_len); } - out[4 + payload_len] = crc8(out, (u16)(4U + payload_len)); - *out_len = total_len; - return true; -} - -bool SpiOta::tryParseFrame(const u8 *frame, u16 frame_len, - spi_ota_frame_type_t *type, - const u8 **payload, - u16 *payload_len) { - if (!frame || frame_len < kFrameOverhead) { return false; } - if (frame[0] != kSof) { return false; } - u16 len = (u16)(frame[2] | ((u16)frame[3] << 8)); - u16 expected = (u16)(kFrameOverhead + len); - if (expected != frame_len) { return false; } - if (crc8(frame, (u16)(4U + len)) != frame[4U + len]) { return false; } - if (type) { *type = (spi_ota_frame_type_t)frame[1]; } - if (payload) { *payload = &frame[4]; } - if (payload_len){ *payload_len = len; } - return true; -} - -ota_link_msg_type_t SpiOta::mapMsgType(spi_ota_frame_type_t type) { - switch (type) { - case SPI_OTA_CMD_REQ: return OTA_LINK_MSG_REQ; - case SPI_OTA_CMD_DATA: return OTA_LINK_MSG_DATA; - default: return OTA_LINK_MSG_CANCEL; - } -} - -/* ============================================================ - * 回包发送适配器(ISR 安全) - * ============================================================ */ - -void SpiOta::sender(ota_link_tx_type_t type, u8 len, const u8 *data) { - spi_ota_frame_type_t rsp_type = SPI_OTA_RSP_ACK; - if (type == OTA_LINK_TX_WAIT) { rsp_type = SPI_OTA_RSP_WAIT; } - u8 frame_buf[kTxBufSize] = {}; - u16 built_len = 0U; - if (buildFrame(rsp_type, data, len, frame_buf, &built_len)) { - queueTxFrame(frame_buf, built_len); - } -} - -uint32_t SpiOta::enterCritical() { - uint32_t primask = __get_PRIMASK(); - __disable_irq(); - return primask; -} - -void SpiOta::exitCritical(uint32_t primask) { - __set_PRIMASK(primask); -} - -/** @brief 启动 DMA 传输 s_txBuf → SPI0_DATA */ -static void startSpiTxDma(const u8 *buf, u16 len) { - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) &= ~DMA_CHXCTL_CHEN; - DMA_INTC(SPI_TX_DMA_PERIPH) |= DMA_FLAG_ADD(DMA_CHINTF_RESET_VALUE, SPI_TX_DMA_CH); - DMA_CHMADDR(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) = (uint32_t)buf; - DMA_CHCNT(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) = len; - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) |= DMA_CHXCTL_CHEN; -} - -void SpiOta::queueTxFrame(const u8 *frame, u16 len) { - if (!frame || len == 0U || len > kTxBufSize) { return; } - uint32_t primask = enterCritical(); - memcpy(s_txBuf, frame, len); - s_txLen = len; - s_txPending = true; - s_txStreamPos = 0U; - if (len < kTxBufSize) { memset(&s_txBuf[len], 0xFF, kTxBufSize - len); } - exitCritical(primask); - spi_dma_enable(SPI0, SPI_DMA_TRANSMIT); - startSpiTxDma(s_txBuf, s_txLen); -} - -/* ============================================================ - * RBNE 中断内即时帧检测与处理 - * ============================================================ */ - -void SpiOta::tryProcessCompleteFrame() { - u16 rx_len = s_rxStreamLen; - if (rx_len == 0U) { return; } - - if (s_rxStream[0] == SPI_OTA_CMD_BOOT) { - isrProcessFrame(); - return; - } - - if (s_rxStream[0] != kSof) { return; } - if (rx_len < kFrameOverhead) { return; } - - u16 payload_len = (u16)(s_rxStream[2] | ((u16)s_rxStream[3] << 8)); - u16 expected = (u16)(kFrameOverhead + payload_len); - if (expected > kRxBufSize) { return; } - if (rx_len < expected) { return; } - - isrProcessFrame(); -} - -/* ============================================================ - * CS 释放后 ISR 直接帧处理 - * ============================================================ */ - -void SpiOta::isrProcessFrame() { - u16 rx_len = s_rxStreamLen; - if (rx_len == 0U) { return; } - - /* 关键修复：一旦开始处理本次 CS 周期中的完整帧，立刻标记“已响应/已处理”。 - * - * 否则会出现这样的竞态： - * 1) SPI0 RBNE IRQ 在收到最后一个字节后调用 isrProcessFrame() - * 2) 还没执行到函数尾部的 s_isrResponded = true - * 3) CS 已经上升，优先级更高的 EXTI IRQ 抢占进来执行 onCsRelease() - * 4) onCsRelease() 看到 s_isrResponded 仍为 false，于是再次调用 isrProcessFrame() - * 5) 同一帧被重复处理一次，状态机被错误推进（如 challenge 又被当 confirm） - * - * 所以必须在函数入口就先置位，防止 REQ/DATA 被双重分发。 */ - s_isrResponded = true; - - if (rx_len >= 1U && s_rxStream[0] == SPI_OTA_CMD_BOOT) { - u8 ack[4] = {0xB0U, 0xA5U, 0x4FU, 0x54U}; - queueTxFrame(ack, sizeof(ack)); - if (s_otaHandler != nullptr) { s_otaHandler->resetToIdle(); } - isrLogFrameEx("[BOOT-CMD]", s_txBuf, s_txLen); - s_isrResponded = true; - return; - } - - if (s_rxStream[0] != kSof) { - isrLogFrameEx("[NO-SOF]", s_rxStream, (rx_len < 8U) ? rx_len : 8U); - return; - } - if (rx_len < kFrameOverhead) { return; } - - u16 payload_len = (u16)(s_rxStream[2] | ((u16)s_rxStream[3] << 8)); - u16 expected = (u16)(kFrameOverhead + payload_len); - if (expected > rx_len) { return; } - - if (crc8(s_rxStream, (u16)(4U + payload_len)) != s_rxStream[4U + payload_len]) { - u8 err = 0xEEU; - u8 err_frame[kTxBufSize] = {}; - u16 built = 0U; - if (buildFrame(SPI_OTA_RSP_ERROR, &err, 1U, err_frame, &built)) { - queueTxFrame(err_frame, built); - } - isrLogFrameEx("[CRC-ERR]", s_rxStream, (rx_len < 8U) ? rx_len : 8U); - return; - } - - spi_ota_frame_type_t frame_type; - const u8 *payload = nullptr; - u16 p_len = 0U; - if (!tryParseFrame(s_rxStream, expected, &frame_type, &payload, &p_len)) { - isrLogFrameEx("[PARSE-ERR]", s_rxStream, (rx_len < 8U) ? rx_len : 8U); - return; - } - - if (frame_type == SPI_OTA_CMD_BOOT) { - u8 ack_ok = 0xA5U; - u8 ack_frame[kTxBufSize] = {}; - u16 ack_len = 0U; - if (buildFrame(SPI_OTA_RSP_ACK, &ack_ok, 1U, ack_frame, &ack_len)) { - queueTxFrame(ack_frame, ack_len); - } - if (s_otaHandler != nullptr) { s_otaHandler->resetToIdle(); } - isrLogFrameEx("[BOOT-FRAME]", s_txBuf, s_txLen); - s_isrResponded = true; - return; - } - - if (s_otaHandler != nullptr) { - s_otaSessionActive = true; - s_otaHandler->setTransportSender(sender); - ota_link_msg_type_t msg_type = mapMsgType(frame_type); - isrLogFrameEx("[OTA-MSG]", (const u8*)&frame_type, 1); - /* 诊断: 记录 OTA 状态机当前状态和帧信息 */ - { - u8 diag[4] = {(u8)frame_type, (u8)p_len, (u8)(p_len >> 8), - (u8)(s_otaHandler ? s_otaHandler->getState() : 0xFFU)}; - isrLogFrameEx("[OTA-DIAG]", diag, 4); - } - s_otaHandler->handleLinkMessage(msg_type, const_cast(payload), (u8)p_len); - } - - isrLogFrameEx("[ISR-DONE]", s_txBuf, (s_txLen < 16U) ? s_txLen : 16U); - s_isrResponded = true; -} - -/* ============================================================ - * ISR 内延迟日志 - * ============================================================ */ - -void SpiOta::isrLogFrame(const char *tag, const u8 *data, u16 len) { - isrLogFrameEx(tag, data, len); -} - -static void isrLogFrameEx(const char *tag, const u8 *data, u16 len) { - if (s_logCount >= kLogSlotCount) { return; } - u8 *buf = s_logSlots[s_logWriteIdx]; - u16 pos = 0U; - if (tag) { - while (*tag && pos < kLogSlotSize - 3U) { buf[pos++] = (u8)*tag++; } - buf[pos++] = ' '; - } - const u16 preview = (len < 24U) ? len : 24U; - for (u16 i = 0U; i < preview && pos + 2U < kLogSlotSize; ++i) { - static const char hx[] = "0123456789ABCDEF"; - buf[pos++] = (u8)hx[data[i] >> 4]; - buf[pos++] = (u8)hx[data[i] & 0x0FU]; - } - buf[pos] = 0U; - s_logSlotLens[s_logWriteIdx] = pos; - s_logWriteIdx = (s_logWriteIdx + 1U) % kLogSlotCount; - s_logCount++; -} - -/* ============================================================ - * 硬件初始化 - * ============================================================ */ - -void SpiOta::configureGpio() { - rcu_periph_clock_enable(RCU_GPIOA); - rcu_periph_clock_enable(RCU_GPIOB); - rcu_periph_clock_enable(RCU_AF); - gpio_pin_remap_config(GPIO_SPI0_REMAP, ENABLE); - gpio_pin_remap_config(GPIO_SWJ_SWDPENABLE_REMAP, ENABLE); - gpio_init(GPIOB, GPIO_MODE_IN_FLOATING, GPIO_OSPEED_50MHZ, GPIO_PIN_3 | GPIO_PIN_5); - gpio_init(GPIOB, GPIO_MODE_AF_PP, GPIO_OSPEED_50MHZ, GPIO_PIN_4); - gpio_init(GPIOB, GPIO_MODE_IN_FLOATING, GPIO_OSPEED_50MHZ, GPIO_PIN_6 | GPIO_PIN_7); - gpio_init(GPIOA, GPIO_MODE_IPU, GPIO_OSPEED_50MHZ, GPIO_PIN_15); -} - -void SpiOta::configureSpi0() { - rcu_periph_clock_enable(RCU_SPI0); - spi_i2s_deinit(SPI0); - (void)spi_i2s_data_receive(SPI0); /* 清空 SPI_DATA 残留值 */ - spi_parameter_struct spi_init_struct; - spi_struct_para_init(&spi_init_struct); - spi_init_struct.device_mode = SPI_SLAVE; - spi_init_struct.trans_mode = SPI_TRANSMODE_FULLDUPLEX; - spi_init_struct.frame_size = SPI_FRAMESIZE_8BIT; - spi_init_struct.nss = SPI_NSS_SOFT; - spi_init_struct.endian = SPI_ENDIAN_MSB; - /* SPI NSS 由 ESP32 的 CS 线 (GPIO4→PA15) 控制。\外部 CS 高时： - * spi_nss_internal_high(SPI0) 让 SPI 外设进入未选中状态， - * 这样 SPI0_DATA 输出 0xFF（而非 0x00）， - * DMA 只在 CS 低时才被 SPI 外设允许推数据到 MISO。 */ - spi_init_struct.clock_polarity_phase = SPI_CK_PL_LOW_PH_1EDGE; - spi_init_struct.prescale = SPI_PSC_8; - spi_init(SPI0, &spi_init_struct); - /* 关键修正：内部 NSS 始终保持 LOW，使 SPI0 从机一直处于“已选中”状态。 - * - * APP 侧的稳定实现就是这样做的：PA15 只作为 EXTI 事务边界信号， - * 不再依赖 EXTI ISR 去“赶在首个 SCK 之前”切 internal NSS。 - * - * 之前 BOOT 的问题正出在这里： - * - 空闲时 spi_nss_internal_high(SPI0) - * - CS↓ 后靠 EXTI ISR 再切 internal_low - * - 但 ESP32 在 CS↓ 后几乎立刻出时钟 - * - 于是 GD32 常常在 internal NSS 真正拉低前丢掉首字节/首几位 - * → 请求帧首字节 0xA5 被吃掉，看到 0x80 / 0xFF / 随机 0x07 - * - * 现在改为 internal NSS 常低： - * - SPI 从机始终就绪，不会因为 EXTI 延迟丢首字节 - * - PA15 仍用于 onCsAssert/onCsRelease 做缓冲区清零和 DMA 重装 - */ - spi_nss_internal_low(SPI0); - /* 初始化时清空 SPI_DATA 寄存器，防止残留值影响首次传输 */ - (void)spi_i2s_data_receive(SPI0); - spi_i2s_interrupt_enable(SPI0, SPI_I2S_INT_RBNE); - spi_i2s_interrupt_enable(SPI0, SPI_I2S_INT_ERR); - spi_i2s_interrupt_disable(SPI0, SPI_I2S_INT_TBE); - /* SPI0 RBNE 中断优先级设为 (2, 0)，低于 EXTI CS 中断 (1, 1)。 - * 这确保 CS 下降沿的 onCsAssert() 在任何 RBNE 之前执行， - * 否则 s_csActive 仍为 false，RBNE 中断丢弃接收字节。 */ - nvic_irq_enable(SPI0_IRQn, 2U, 0U); - spi_enable(SPI0); -} - -void SpiOta::configureDma() { - rcu_periph_clock_enable(RCU_DMA0); - dma_deinit(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH); - dma_parameter_struct dma_init_struct; - dma_struct_para_init(&dma_init_struct); - dma_init_struct.periph_addr = (uint32_t)(SPI0 + 0x0CU); - dma_init_struct.periph_width = DMA_PERIPHERAL_WIDTH_8BIT; - dma_init_struct.memory_addr = (uint32_t)s_txBuf; - dma_init_struct.memory_width = DMA_MEMORY_WIDTH_8BIT; - dma_init_struct.number = 0U; - dma_init_struct.priority = DMA_PRIORITY_HIGH; - dma_init_struct.periph_inc = DMA_PERIPH_INCREASE_DISABLE; - dma_init_struct.memory_inc = DMA_MEMORY_INCREASE_ENABLE; - dma_init_struct.direction = DMA_MEMORY_TO_PERIPHERAL; - dma_init(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH, &dma_init_struct); - dma_circulation_disable(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH); - dma_channel_disable(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH); -} - -void SpiOta::configureExti() { - gpio_exti_source_select(GPIO_PORT_SOURCE_GPIOA, GPIO_PIN_SOURCE_15); - exti_init(EXTI_15, EXTI_INTERRUPT, EXTI_TRIG_BOTH); - exti_interrupt_enable(EXTI_15); - exti_interrupt_flag_clear(EXTI_15); - nvic_irq_enable(EXTI10_15_IRQn, 1U, 1U); -} - -void SpiOta::init() { - s_csActive = false; - s_rxStreamLen = 0U; - s_txStreamPos = 0U; - s_txLen = 0U; - s_txPending = false; - s_isrLogPending = false; - memset(s_rxStream, 0, sizeof(s_rxStream)); - memset(s_txBuf, 0xFF, sizeof(s_txBuf)); - configureGpio(); - configureDma(); - configureSpi0(); - configureExti(); -} - -/* ============================================================ - * CS 边沿处理 - * ============================================================ */ - -void SpiOta::onCsAssert() { - if (s_csActive) { return; } - s_csActive = true; - s_rxStreamLen = 0U; - s_txStreamPos = 0U; - s_isrResponded = false; - - /* 内部 NSS 已在初始化时常置 LOW，这里不再依赖 EXTI 时序切 NSS。 - * PA15 仅作为“事务开始”的边界信号，用于清接收缓冲和重装 DMA。 */ - - /* 关键修复：先彻底停止 DMA，再重新配置并启动。 - * - * 上一版预填 SPI_DATA 方案失败原因： - * spi_i2s_data_transmit() 写入的字节进入 TBE 发送缓冲区。 - * 当 ESP32 第一个 SCK 边沿到来时，TBE 的预填字节被移入移位 - * 寄存器推到 MISO，同时移位寄存器中的 MOSI 数据被推入 RBNE。 - * 但 GD32E50x SPI 的 RBNE 标志在 SPI_DATA 写入后可能被误触发， - * 导致 isrLogFrame 报出预填值（如 0x80）而非 MOSI 真实数据。 - * - * 正确方案：不预填 SPI_DATA，而是确保 DMA 在 CS↓ 后、ESP32 - * 第一个 SCK 边沿之前就绪。DMA 第一拍自动把 s_txBuf[0] 写入 - * SPI_DATA，不会产生 RBNE 误触发。 - * - * 操作顺序： - * 1. 禁 DMA 通道 + 禁 SPI DMA 请求（彻底停止旧传输） - * 2. 清 RBNE / RXORERR / FERR 等残留标志 - * 3. 配置并启动 DMA（此时 RBNE 仍关闭，DMA 写 SPI_DATA 不会触发中断） - * 4. 重新使能 RBNE 中断，开始接收 MOSI 数据 - */ - - /* 1. 停止旧 DMA */ - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) &= ~DMA_CHXCTL_CHEN; - spi_dma_disable(SPI0, SPI_DMA_TRANSMIT); - - /* 2. 清残留错误标志 - * - * 关键：这里绝不能在 CS↓ 时“顺手读掉 RBNE”。 - * - * 因为 internal NSS 现在常 LOW，ESP32 在 CS↓ 后很快就会发出首字节 A5。 - * 如果 EXTI ISR 略晚于 SPI 接收，A5 可能已经先进了 RX 缓冲并置位 RBNE。 - * 这时若在 onCsAssert() 里执行 spi_i2s_data_receive()，就会把真实首字节 A5 - * 当成“残留垃圾”吞掉，导致后续看到的首字节变成 0x01。 - * - * 所以这里仅清真正的错误标志，不主动消费 RBNE。 - * 已到达的首字节留给 onSpiInterrupt() 正常读走。 */ - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RXORERR) != RESET) { - (void)spi_i2s_data_receive(SPI0); - } - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_FERR) != RESET) { - spi_i2s_format_error_clear(SPI0, SPI_FLAG_FERR); - } - - /* 3. 配置并启动 DMA（RBNE 仍关闭，DMA 写 SPI_DATA 不会触发中断） */ - if (s_txPending && s_txLen > 0U) { - spi_dma_enable(SPI0, SPI_DMA_TRANSMIT); - startSpiTxDma(s_txBuf, s_txLen); - } else { - s_txPending = false; - s_txLen = kTxBufSize; - memset(s_txBuf, 0xFF, kTxBufSize); - spi_dma_enable(SPI0, SPI_DMA_TRANSMIT); - startSpiTxDma(s_txBuf, s_txLen); - } - - /* 4. 重新使能 RBNE 中断，开始接收 MOSI 数据 */ - spi_i2s_interrupt_enable(SPI0, SPI_I2S_INT_RBNE); -} -void SpiOta::onCsRelease() { - if (!s_csActive) { return; } - s_csActive = false; - - /* 内部 NSS 保持常 LOW，不在 CS↑ 时切回 HIGH。 - * 这样 SPI 从机始终处于就绪态，下一次 CS↓ 不会因 EXTI 延迟丢首字节。 - * - * 这里只停止 DMA，避免 CS 高期间继续空转消耗 s_txBuf 内容。 - * s_txPending 保持不变——如果 ISR 已准备好 ACK 帧， - * 下次 CS↓ 时 onCsAssert 会重启 DMA 推出。 */ - DMA_CHCTL(SPI_TX_DMA_PERIPH, SPI_TX_DMA_CH) &= ~DMA_CHXCTL_CHEN; - spi_dma_disable(SPI0, SPI_DMA_TRANSMIT); - - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RBNE) != RESET) { - u8 last_byte = (u8)(spi_i2s_data_receive(SPI0) & 0xFFU); - if (s_rxStreamLen < kRxBufSize) { - s_rxStream[s_rxStreamLen++] = last_byte; - } - } - - /* 关键调试日志：记录本次 CS 周期结束时到底收到了多少字节。 - * 这样能直接判断： - * 1) 是不是完整收到了 ESP32 发来的整帧 - * 2) 还是只收到了首字节 A5 / 少数字节，导致永远无法解析回包 */ - if (s_rxStreamLen > 0U) { - isrLogFrame("[CS-RX]", s_rxStream, (s_rxStreamLen < 12U) ? s_rxStreamLen : 12U); - } - - if (s_rxStreamLen > 0U && !s_isrResponded) { - isrProcessFrame(); - } -} - -/* ============================================================ - * SPI0 中断处理 — 只处理 RBNE（接收）和错误，TBE 已由 DMA 替代 - * ============================================================ */ - -void SpiOta::onSpiInterrupt() { - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RBNE) != RESET) { - /* 与 APP 侧稳定实现保持一致： - * 如果 EXTI 下降沿因为中断时序略晚到达，但此时 PA15 已经为低， - * 则在首个 RBNE 到来时立即补做 onCsAssert()。 - * - * 关键限制：只能在“本事务尚未收过任何字节”时补做。 - * 否则若 EXTI / SPI 时序偶发抖动，重复 onCsAssert() 会把 - * s_rxStreamLen 清零，导致已经收到的半帧被冲掉，最终永远拼不成完整帧。 */ - if (!s_csActive && s_rxStreamLen == 0U && gpio_input_bit_get(GPIOA, GPIO_PIN_15) == RESET) { - onCsAssert(); - } - - u8 byte = (u8)(spi_i2s_data_receive(SPI0) & 0xFFU); - if (s_rxStreamLen < kRxBufSize) { - s_rxStream[s_rxStreamLen++] = byte; - } - /* 调试阶段先关闭 RBNE 首字节日志。 - * 原因：ISR 日志缓冲是单槽的，RBNE 一旦先占位， - * onCsRelease() 里的 [CS-RX] 摘要日志就会被丢弃。 - * 目前排查重点是“本次 CS 周期总共收到了哪些字节”， - * 所以优先保留 [CS-RX]，暂时关闭 [RBNE]。 */ - tryProcessCompleteFrame(); - } - /* TBE 已由 DMA 替代，不再处理 */ - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_RXORERR) != RESET) { - (void)spi_i2s_data_receive(SPI0); - s_rxStreamLen = 0U; - isrLogFrame("[RXOR]", (const u8*)"OVR", 3); - } - if (spi_i2s_interrupt_flag_get(SPI0, SPI_I2S_INT_FLAG_FERR) != RESET) { - spi_i2s_format_error_clear(SPI0, SPI_FLAG_FERR); - } -} - -void SpiOta::onCsExtiInterrupt() { - if (exti_interrupt_flag_get(EXTI_15) != RESET) { - exti_interrupt_flag_clear(EXTI_15); - if (gpio_input_bit_get(GPIOA, GPIO_PIN_15) == RESET) { onCsAssert(); } - else { onCsRelease(); } - } -} - -/* ============================================================ - * 主循环接口 - * ============================================================ */ - -void SpiOta::poll() { - if (s_isrResponded) { - s_isrResponded = false; - } - /* 打印 ISR 日志缓冲区 */ - while (s_logCount > 0U) { - __disable_irq(); - if (s_logCount == 0U) { __enable_irq(); break; } - u8 *buf = s_logSlots[s_logReadIdx]; - u8 nextRead = (s_logReadIdx + 1U) % kLogSlotCount; - s_logReadIdx = nextRead; - s_logCount--; - __enable_irq(); - printf("[SPI] %s\r\n", buf); - } - if (s_otaHandler != nullptr) { s_otaHandler->process(); } - - /* 调试:定期打印 SPI 从机状态 */ - static u32 dbg_tick = 0; - dbg_tick++; - if ((dbg_tick % 5000U) == 0U) { - printf("[SPI] csActive=%u rxLen=%u txPend=%u txLen=%u isrResp=%u otaState=%u\r\n", - (unsigned)s_csActive, (unsigned)s_rxStreamLen, - (unsigned)s_txPending, (unsigned)s_txLen, - (unsigned)s_isrResponded, - (unsigned)(s_otaHandler ? s_otaHandler->getState() : 0xFF)); - } -} - -void SpiOta::setSender(OTAHandler *handler) { s_otaHandler = handler; } - -void SpiOta::feedRxBytes(const u8 *data, u16 len) { - if (!data || len == 0U || !s_csActive) { return; } - if (len > kRxBufSize) { data += (len - kRxBufSize); len = kRxBufSize; } - memcpy(s_rxStream, data, len); - s_rxStreamLen = len; -} - -bool SpiOta::hasTxFrame() { return s_txPending; } - -bool SpiOta::hasActiveSession() { return s_otaSessionActive; } - -u16 SpiOta::copyTxFrame(u8 *out, u16 max_len) { - if (!out || max_len == 0U) { return 0U; } - uint32_t primask = enterCritical(); - if (!s_txPending || s_txLen == 0U) { exitCritical(primask); return 0U; } - u16 copy_len = (s_txLen < max_len) ? s_txLen : max_len; - memcpy(out, s_txBuf, copy_len); - s_txPending = false; - s_txLen = 0U; - s_txStreamPos = 0U; - exitCritical(primask); - return copy_len; -} - -/* ============================================================ - * C 风格兼容包装 - * ============================================================ */ - -void spi_ota_init(void) { SpiOta::init(); } -void spi_ota_poll(void) { SpiOta::poll(); } -void spi_ota_set_sender(OTAHandler *handler) { SpiOta::setSender(handler); } -void spi_ota_on_cs_assert(void) { SpiOta::onCsAssert(); } -void spi_ota_on_cs_release(void) { SpiOta::onCsRelease(); } -void spi_ota_gpio_init(void) { SpiOta::configureGpio(); } -void spi_ota_spi0_init(void) { SpiOta::configureSpi0(); } -void spi_ota_exti_init(void) { SpiOta::configureExti(); } -void spi_ota_irq_handler(void) { SpiOta::onSpiInterrupt(); } -void spi_ota_cs_exti_handler(void) { SpiOta::onCsExtiInterrupt(); } -void spi_ota_feed_rx_bytes(const u8 *data, u16 len) { SpiOta::feedRxBytes(data, len); } -bool spi_ota_has_tx_frame(void) { return SpiOta::hasTxFrame(); } -u16 spi_ota_copy_tx_frame(u8 *out, u16 max_len) { return SpiOta::copyTxFrame(out, max_len); } - -extern "C" void SPI0_IRQHandler(void) { - SpiOta::onSpiInterrupt(); -} diff --git a/GD32E50X_BOOT/device/SpiOta.h b/GD32E50X_BOOT/device/SpiOta.h index f5c45f2..9fd2916 100644 --- a/GD32E50X_BOOT/device/SpiOta.h +++ b/GD32E50X_BOOT/device/SpiOta.h @@ -56,7 +56,6 @@ public: static void configureSpi0(); static void configureDma(); static void configureExti(); - static void onSpiInterrupt(); static void onCsExtiInterrupt(); static bool hasTxFrame(); @@ -125,25 +124,4 @@ private: /* OTA.cpp 通过此变量更新 SPI 响应帧的 CREDIT 字段 */ extern volatile u8 g_localCredit; -#ifdef __cplusplus -extern "C" { -#endif - -void spi_ota_init(void); -void spi_ota_poll(void); -void spi_ota_service_pending_tx(void); -void spi_ota_set_sender(OTAHandler *handler); -void spi_ota_on_cs_assert(void); -void spi_ota_on_cs_release(void); -void spi_ota_gpio_init(void); -void spi_ota_spi0_init(void); -void spi_ota_exti_init(void); -void spi_ota_irq_handler(void); -void spi_ota_cs_exti_handler(void); -bool spi_ota_has_tx_frame(void); - -#ifdef __cplusplus -} -#endif - #endif diff --git a/GD32E50X_BOOT/sys/OTA.cpp b/GD32E50X_BOOT/sys/OTA.cpp index 43f05ab..9ae04f8 100644 --- a/GD32E50X_BOOT/sys/OTA.cpp +++ b/GD32E50X_BOOT/sys/OTA.cpp @@ -1060,25 +1060,6 @@ void OTAHandler::handleLinkMessage(ota_link_msg_type_t type, u8 *data, u8 len) { } } -/** - * @brief 兼容旧接口:按 CAN ID 分发 OTA 消息。 - * - * @param id CAN 扩展帧 ID。 - * @param data 指向消息数据缓冲区的指针。 - * @param len 消息数据长度。 - * @return 无。 - */ -void OTAHandler::handleOTAMessage(u32 id, u8 *data, u8 len) { - /** 兼容旧的"按 CAN ID 分发"调用方式。 */ - if (id == CANFD_OTA_CANCEL_ID) { - handleLinkMessage(OTA_LINK_MSG_CANCEL, data, len); - } else if (id == CANFD_OTA_DATA_ID) { - handleLinkMessage(OTA_LINK_MSG_DATA, data, len); - } else if (id == CANFD_OTA_REQ_ID) { - handleLinkMessage(OTA_LINK_MSG_REQ, data, len); - } -} - /** * @brief 发送 OTA ACK 帧。 * diff --git a/GD32E50X_BOOT/sys/OTA.h b/GD32E50X_BOOT/sys/OTA.h index 659f88b..14d2792 100644 --- a/GD32E50X_BOOT/sys/OTA.h +++ b/GD32E50X_BOOT/sys/OTA.h @@ -1,4 +1,4 @@ -#ifndef __OTA_H +#ifndef __OTA_H #define __OTA_H #include "device/Can.h" @@ -46,7 +46,6 @@ typedef void (*ota_send_fn_t)(ota_link_tx_type_t type, u8 len, const u8 *data); */ class OTAHandler { public: - u32 ota_stateota_time; /**< 历史遗留字段，当前实现未实际使用。 */ u32 received_size; /**< 已接收总字节数。 */ u32 expected_size; /**< 预期固件总字节数。 */ @@ -142,16 +141,6 @@ class OTAHandler { */ void handleLinkMessage(ota_link_msg_type_t type, u8 *data, u8 len); - /** - * @brief 兼容旧接口：按 CAN ID 处理 OTA 消息。 - * - * @param id CAN 扩展帧 ID。 - * @param data 指向消息数据缓冲区的指针。 - * @param len 消息数据长度。 - * @return 无。 - */ - void handleOTAMessage(u32 id, u8 *data, u8 len); - /** * @brief 关闭 Bootloader 运行环境并跳转执行主应用。 * diff --git a/GD32E50X_BOOT/sys/config.h b/GD32E50X_BOOT/sys/config.h index 8517df4..4238e43 100644 --- a/GD32E50X_BOOT/sys/config.h +++ b/GD32E50X_BOOT/sys/config.h @@ -14,9 +14,9 @@ typedef uint32_t u32; /* BOOT 固件版本号 */ #define BOOT_FW_VERSION_MAJOR 1U -#define BOOT_FW_VERSION_MINOR 0U -#define BOOT_FW_VERSION_PATCH 2U -#define BOOT_FW_VERSION_BUILD 1U +#define BOOT_FW_VERSION_MINOR 1U +#define BOOT_FW_VERSION_PATCH 0U +#define BOOT_FW_VERSION_BUILD 0U /** * Flash 布局说明（GD32E508VE，512KB） diff --git a/GD32E50X_BOOT/sys/initial.cpp b/GD32E50X_BOOT/sys/initial.cpp index 31363a7..2f7c66e 100644 --- a/GD32E50X_BOOT/sys/initial.cpp +++ b/GD32E50X_BOOT/sys/initial.cpp @@ -62,89 +62,6 @@ int BootDebugUart::putChar(int ch, FILE *f) */ int fputc(int ch, FILE *f) throw() { return BootDebugUart::putChar(ch, f); } -/** - * @brief Initialize USART0 through the boot debug UART helper. - * @return 1 when initialization succeeds. - */ -u8 USART0_init(void) { return BootDebugUart::init(); } - -/** - * @brief 获取复位原因。 - * - * @return 复位原因编码。 - */ -u16 BootPlatform::getResetCause(void) -{ - /** 预留：如需区分上电/看门狗/软复位等原因，可在此补齐。 */ - return 0; -} - -/** - * @brief Get reset cause through the platform helper. - * @return Reset-cause code. - */ -u16 get_reset_cause(void) { return BootPlatform::getResetCause(); } - -/** - * @brief 初始化未使用引脚。 - * - * @return 无。 - */ -void BootPlatform::initUnusedPins(void) -{ - /** - * 预留：如果需要降低悬空功耗，可把未使用 GPIO 统一初始化。 - * 当前只保证时钟可用，未进一步配置具体引脚。 - */ - rcu_periph_clock_enable(RCU_GPIOA); - rcu_periph_clock_enable(RCU_GPIOB); - rcu_periph_clock_enable(RCU_GPIOC); -} - -/** - * @brief Initialize unused pins through the platform helper. - * @return none - */ -void unused_pin_init(void) { BootPlatform::initUnusedPins(); } - -/** - * @brief 打印系统时钟信息。 - * - * @return 无。 - */ -void BootPlatform::printSystemClockInfo(void) -{ - /** 预留：可在此打印系统时钟源与频率。 */ -} - -/** - * @brief Print system clock info through the platform helper. - * @return none - */ -void print_system_clock_info(void) { BootPlatform::printSystemClockInfo(); } - -/** - * @brief 扫描主应用区 Flash 内容。 - * - * @param print_content 是否打印内容。 - * @param page_count 需要扫描的页数。 - * @return 无。 - */ -void BootPlatform::scanAppFlashContent(int print_content, u32 page_count) -{ - /** 预留：调试用，后续可打印主应用区内容。 */ - (void)print_content; - (void)page_count; -} - -/** - * @brief Scan application flash content through the platform helper. - * @param print_content Whether to print flash payload. - * @param page_count Number of pages to scan. - * @return none - */ -void scanAppFlashContent(int print_content, u32 page_count) { BootPlatform::scanAppFlashContent(print_content, page_count); } - /** * @brief 向指定 Flash 地址写入一个 32 位字。 * @@ -268,9 +185,8 @@ void BootSleepManager::onExtiInterrupt(void) } /* 注意：SPI OTA 的 CS 边沿信号 PA15 映射到 EXTI_15，属于 EXTI10_15_IRQn， - * 由 EXTI10_15_IRQHandler 直接分发到 SpiOta::onCsExtiInterrupt()， - * 不应在此处（EXTI5_9_Handler）调用 spi_ota_cs_exti_handler()。 - * 该调用属于死代码，会导致 PA15 边沿在错误的 ISR 中被重复处理。 */ + * 由 EXTI10_15_IRQHandler 直接分发到 SpiOta::onCsExtiInterrupt()。 + * 本函数（EXTI5_9）只处理 PB8 深睡唤醒，不处理 SPI OTA CS 信号。 */ } /** @@ -307,18 +223,6 @@ int BootSleepManager::check(void) return 2; } -/** - * @brief Enter deep sleep through the sleep manager. - * @return none - */ -void enter_deep_sleep(void) { BootSleepManager::enterDeepSleep(); } - -/** - * @brief Check sleep configuration through the sleep manager. - * @return Status code from BootSleepManager::check(). - */ -int check_sleep(void) { return BootSleepManager::check(); } - /** * @brief External interrupt handler for EXTI line group 5 to 9. * @return none diff --git a/GD32E50X_BOOT/sys/initial.h b/GD32E50X_BOOT/sys/initial.h index c213ac8..3bf9ecd 100644 --- a/GD32E50X_BOOT/sys/initial.h +++ b/GD32E50X_BOOT/sys/initial.h @@ -42,19 +42,6 @@ private: static fmc_state_enum writeAddress(u32 address, u32 data); }; -/** - * @brief Bootloader 平台辅助模块。 - * - * 负责复位原因读取、未使用引脚初始化、系统时钟信息打印与应用区扫描等平台杂项。 - */ -class BootPlatform { -public: - static u16 getResetCause(); - static void initUnusedPins(); - static void printSystemClockInfo(); - static void scanAppFlashContent(int print_content, u32 page_count); -}; - /** * @brief printf 重定向输出函数。 * @@ -64,52 +51,6 @@ public: */ int fputc(int ch, FILE *f) throw(); -/** - * @brief 初始化 USART0 调试串口。 - * - * @return 1 表示初始化完成。 - */ -u8 USART0_init(void); - -/** - * @brief 获取复位原因。 - * - * @note 当前实现为占位实现。 - * - * @return 复位原因编码。 - */ -u16 get_reset_cause(void); - -/** - * @brief 初始化未使用引脚。 - * - * @return 无。 - */ -void unused_pin_init(void); - -/** - * @brief 打印系统时钟信息。 - * - * @note 当前实现为占位实现。 - * - * @return 无。 - */ -void print_system_clock_info(void); - -/** - * @brief 进入深度睡眠模式。 - * - * @return 无。 - */ -void enter_deep_sleep(void); - -/** - * @brief 检查睡眠配置并决定是否进入深睡。 - * - * @return 0 表示已有有效配置；1 表示写默认配置失败；2 表示已写入默认禁用配置。 - */ -int check_sleep(void); - /** * @brief EXTI5_9 中断服务函数。 * @@ -123,15 +64,6 @@ void EXTI5_9_IRQHandler(void); */ void EXTI10_15_IRQHandler(void); -/** - * @brief 扫描主应用区 Flash 内容。 - * - * @param print_content 是否打印内容。 - * @param page_count 需要扫描的页数。 - * @return 无。 - */ -void scanAppFlashContent(int print_content, u32 page_count); - #ifdef __cplusplus } #endif diff --git a/GD32E50X_BOOT/sys/systick.cpp b/GD32E50X_BOOT/sys/systick.cpp index 4c24962..94e3d73 100644 --- a/GD32E50X_BOOT/sys/systick.cpp +++ b/GD32E50X_BOOT/sys/systick.cpp @@ -127,12 +127,6 @@ uint32_t SysTickTimer::getMs() return s_msTicks; } -/** - * @brief Configure SysTick via the timer helper. - * @return none - */ -void systick_config(void) { SysTickTimer::init(); } - /** * @brief Delay for microseconds via the timer helper. * @param count Delay time in microseconds. @@ -147,19 +141,6 @@ void delay_1us(uint32_t count) { SysTickTimer::delayUs(count); } */ void delay_1ms(uint32_t count) { SysTickTimer::delayMs(count); } -/** - * @brief Delay for seconds via the timer helper. - * @param count Delay time in seconds. - * @return none - */ -void delay_1s(uint32_t count) { SysTickTimer::delaySeconds(count); } - -/** - * @brief Decrement the global millisecond delay counter. - * @return none - */ -void delay_decrement(void) { SysTickTimer::decrement(); } - /** * @brief Trigger a software reset via the timer helper. * @return none diff --git a/GD32E50X_BOOT/sys/systick.h b/GD32E50X_BOOT/sys/systick.h index c399c2d..5548f5f 100644 --- a/GD32E50X_BOOT/sys/systick.h +++ b/GD32E50X_BOOT/sys/systick.h @@ -1,4 +1,4 @@ -#ifndef SYSTICK_H +#ifndef SYSTICK_H #define SYSTICK_H #ifdef __cplusplus @@ -37,13 +37,6 @@ private: static volatile uint32_t s_msTicks; }; -/** - * @brief 配置 SysTick 为 1ms 周期中断。 - * - * @return 无。 - */ -void systick_config(void); - /** * @brief 执行毫秒级阻塞延时。 * @@ -60,21 +53,6 @@ void delay_1ms(uint32_t count); */ void delay_1us(uint32_t count); -/** - * @brief 执行秒级阻塞延时。 - * - * @param count 延时秒数。 - * @return 无。 - */ -void delay_1s(uint32_t count); - -/** - * @brief 递减 SysTick 维护的毫秒倒计时。 - * - * @return 无。 - */ -void delay_decrement(void); - /** * @brief 触发 MCU 软件复位。 *