wandering1/STM32F4-Base
1
0
Fork 0
Code Issues 30 Pull Requests Actions Packages Projects 2 Releases Wiki Activity

fix(BSP): SPI1 完成标志收回 GD5F 私有，HAL 回调改契约转调

Browse Source 
推荐清单第 4a 步（结构债 S-2 剩余面里唯一带"静默失效"属性的那条）。

**问题不是风格**：HAL 的 SPI 完成/错误回调是**全局唯一符号**，整个工程只能有一份
实现，当前由 CH395F 代管并按 `hspi->Instance` 分发——于是 GD5F 的完成标志
`g_spi1_dma_done` 被 CH395F 跨模块裸 `extern` 直接写，**GD5F 出错时的"释放等待者"
寄居在别人的 .c 里**。谁哪天做"CH395F 清理"，把那两个 `else if (SPI1)` 分支当无关
代码删掉：编译零报错，GD5F 的 DMA 等待在出错时只能干等 `GD5F_SPI_TIMEOUT` 兜底。

**改法（时序一字不动，只换所有权）**：
· 标志收为 GD5F 文件内 `static volatile uint8_t s_spi1_dma_done`（6 处引用同步改名）
· GD5F 导出 `gd5f_spi1_notify_done()` / `gd5f_spi1_notify_error()`，头文件里写明
  "SPI1 分支必须转调这两个函数，否则本模块的 DMA 等待在出错时只能靠超时兜底"
· CH395F 删除裸 `extern`、include `gd5f2gq5ue.h`、两个 SPI1 分支改为转调并就地注明
  "勿删：少了这行 GD5F 的 DMA 出错只能靠超时退出"

`HAL_SPI_AbortCpltCallback`（`HAL_SPI_DMAStop()` 会触发）本次**刻意不做**——它会改
变中止语义、需要真机专项，当前超时兜底可用（已在报告 §0.4 记为观察项）。

**体量代价**：`ch395f.o` −8 B、`gd5f2gq5ue.o` +24 B → `Code` 213,188 → **213,204 B
（+16 B）**，RO/RW/ZI 全不变，`ER_IROM1` 231,768 → **231,784 B（22.1%）**。16 B 换
掉一个静默失效类，划算。

**真机 `@test` PASS**：无 `[ERR]`/Fault，复位原因 software；`[STACK] heap
free=15800 min_ever=8464` 与改前**逐项相同**；各任务实测峰值 default 2152 / net
1116 / ftp 764 / ui 4052 / timesync 616 / adc 952 / iec104 540 / protect 624，
**静态最坏链全部未变**（3024/1912/1456/4052/1264/1320/1456/872）→ 9/9 PASS。

⚠ 记一条读数陷阱：测试脚本打印的 "boot to net init: OK = 7,907 ms" 是采集窗口
锚点造成的度量偏差，真机日志里 `boot: syslog init OK → net init: OK` 实测 ≈699 ms，
与改前一致，**不是性能回退**。

**数字联动**：`data/ram_report_20261022.md`、`data/stack_report_20261022.md` 重跑；
`ram_baseline.py --update/--write`；`check_mem_numbers.py --write`（RAM 方案受管区 +
台账）；`gen_memory_svg.py` 常量更新后重新生成 FLASH 与任务栈两张配图（逐文本按
字号与锚点复查越界 = 0；RAM 图无需变更）；报告 §1.3/§5.1/§5.4、部署文档 §3.3 的
Flash 数字同步；§0.4/§9 的 S-2 口径改为"24 处手写已消 23 处，只剩
`net_socket.c:77`"。

门禁：`@build` 0 错 0 警告；`cubemx-survival --check`、`mem-numbers`、`mem-budget`
（57.4% / 87.9%）、`task-table`、`doc-links --all --sections-strict`（146 文档 0 问题）、
`id-check --strict`（增量拦截通过、可判定 82.83%）全绿；字节卫生 BOM/CR/U+FFFD/尾
空格全清；未提交 UV4 抖动的 `.uvprojx`（本次未被改写）。未推送。
This commit is contained in:
Wanderingss
2026-10-22 19:21:08 +08:00
parent 36a3c3449c
commit 7e77af08a0
13 changed files with 131 additions and 71 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Drivers/BSP/CH395F/ch395f.c
+3 -5
View File
@@ -20,6 +20,7 @@
#include <stdio.h>
#include "ch395f.h"
#include "spi.h"
#include "gd5f2gq5ue.h" /* SPI1 分支转调 gd5f_spi1_notify_*（HAL SPI 回调全局唯一，本文件代管分发） */
#include "usart.h"
/*
@@ -173,14 +174,11 @@ static int ch395f_spi_dma_transfer(uint16_t len) {
* 入口参数：hspi - SPI句柄指针 SPI_HandleTypeDef*
* 函数说明：由HAL SPI驱动在DMA TX+RX完成后调用
*/
/* SPI1 DMA完成标志（定义在gd5f2gq5ue.c中） */
extern volatile uint8_t g_spi1_dma_done;
void HAL_SPI_TxRxCpltCallback(SPI_HandleTypeDef *hspi) {
if (hspi->Instance == SPI2) {
s_spi2_dma_done = 1U;
} else if (hspi->Instance == SPI1) {
g_spi1_dma_done = 1U;
gd5f_spi1_notify_done(); /* 归 GD5F 所有：本文件只代 HAL 分发，不碰它的标志 */
}
}
@@ -193,7 +191,7 @@ void HAL_SPI_ErrorCallback(SPI_HandleTypeDef *hspi) {
if (hspi->Instance == SPI2) {
s_spi2_dma_done = 1U;
} else if (hspi->Instance == SPI1) {
g_spi1_dma_done = 1U;
gd5f_spi1_notify_error(); /* 勿删：少了这行 GD5F 的 DMA 出错只能靠超时退出 */
}
}
Drivers/BSP/GD5F2GQ5UE/gd5f2gq5ue.c
+37 -9
View File
@@ -45,8 +45,14 @@ static uint8_t s_bbt_best[GD5F_BBT_SIZE]; /* 最高版本位图缓存 */
static const uint8_t s_bbt_magic[4] = { 'G', 'B', 'B', 'T' }; /* BBT 存储魔数 "GBBT" */
/* DMA 完成标志（ch395f.c 中的 HAL_SPI_TxRxCpltCallback 会设置此标志） */
volatile uint8_t g_spi1_dma_done;
/* DMA 完成标志（本模块私有，2026-10-22 收编）：
* HAL 的 SPI 完成/错误回调是**全局唯一符号**（整个工程只能有一份实现），当前由
* Drivers/BSP/CH395F/ch395f.c 实现并按 hspi->Instance 分发——SPI1 分支必须转调本
* 模块导出的 gd5f_spi1_notify_done() / gd5f_spi1_notify_error()，不得反过来由
* CH395F 直接写本模块的变量（原先是跨模块裸 extern，删掉那两行不会报错，只会让
* 下面的 DMA 等待循环在出错时退化成干等 GD5F_SPI_TIMEOUT）。
*/
static volatile uint8_t s_spi1_dma_done;
/* DMA 阈值（字节数超过此值使用 DMA 传输）：取值定义在头文件，理由如下——
* 阈值以下走 CPU 轮询。存在把**栈上小缓冲**直接传进本驱动做收发的大小
@@ -68,11 +74,11 @@ volatile uint8_t g_spi1_dma_done;
* 入口参数：hspi - 触发回调的 SPI 句柄 SPI_HandleTypeDef*
* 返回值：无
* 限定条件：由 HAL DMA 中断调用
* 函数说明：仅当 SPI1 传输完成时置位 g_spi1_dma_done 标志
* 函数说明：仅当 SPI1 传输完成时置位 s_spi1_dma_done 标志
*/
void HAL_SPI_TxCpltCallback(SPI_HandleTypeDef *hspi) {
if (hspi->Instance == SPI1) {
g_spi1_dma_done = 1U;
s_spi1_dma_done = 1U;
}
}
@@ -84,10 +90,32 @@ void HAL_SPI_TxCpltCallback(SPI_HandleTypeDef *hspi) {
*/
void HAL_SPI_RxCpltCallback(SPI_HandleTypeDef *hspi) {
if (hspi->Instance == SPI1) {
g_spi1_dma_done = 1U;
s_spi1_dma_done = 1U;
}
}
/*
* 函数功能：SPI1 DMA 完成通知（供实际注册 HAL 回调的一方转调）
* 入口参数：无
* 返回值：无
* 限定条件：由 HAL SPI 回调（中断上下文）调用；只写一个标志，不阻塞
* 函数说明：HAL 的 SPI 回调在整个工程里只能有一份实现，本模块不抢占 Error 回调
* 的所有权，只导出这两个通知入口。**转调方不得省略本调用**——少了它，
* 出错路径只能靠 GD5F_SPI_TIMEOUT 超时退出（能自保但会静默变慢）。
*/
void gd5f_spi1_notify_done(void) {
s_spi1_dma_done = 1U;
}
/*
* 函数功能：SPI1 DMA 错误通知（释放等待者；语义与 notify_done 相同）
* 入口参数：无
* 返回值：无
* 限定条件：由 HAL SPI 错误回调（中断上下文）调用
*/
void gd5f_spi1_notify_error(void) {
s_spi1_dma_done = 1U;
}
/* ======================== 私有函数声明 ======================== */
static int gd5f_page_program(uint32_t page_addr, uint16_t column,
@@ -318,12 +346,12 @@ int32_t gd5f_read_from_cache(uint16_t column, uint8_t *p_buf,
if (use_dma != 0U) {
uint32_t tick_start = HAL_GetTick();
g_spi1_dma_done = 0U;
s_spi1_dma_done = 0U;
if (HAL_SPI_Receive_DMA(&hspi1, p_buf, (uint16_t)size) != HAL_OK) {
GD5F_CS_HIGH();
return GD5F_ERROR;
}
while (g_spi1_dma_done == 0U) {
while (s_spi1_dma_done == 0U) {
if ((HAL_GetTick() - tick_start) >= GD5F_SPI_TIMEOUT) {
HAL_SPI_DMAStop(&hspi1);
GD5F_CS_HIGH();
@@ -399,13 +427,13 @@ int32_t gd5f_program_load(uint16_t column, const uint8_t *p_buf, size_t size) {
if (use_dma != 0U) {
uint32_t tick_start = HAL_GetTick();
g_spi1_dma_done = 0U;
s_spi1_dma_done = 0U;
if (HAL_SPI_Transmit_DMA(&hspi1, (uint8_t *)p_buf, (uint16_t)size)
!= HAL_OK) {
GD5F_CS_HIGH();
return GD5F_ERROR;
}
while (g_spi1_dma_done == 0U) {
while (s_spi1_dma_done == 0U) {
if ((HAL_GetTick() - tick_start) >= GD5F_SPI_TIMEOUT) {
HAL_SPI_DMAStop(&hspi1);
GD5F_CS_HIGH();
Drivers/BSP/GD5F2GQ5UE/gd5f2gq5ue.h
+13
View File
@@ -324,6 +324,19 @@ void nand_ftl_wd_beat(void);
*/
void nand_ftl_wd_feed(void);
/*
* 函数功能：SPI1 DMA 完成 / 错误通知（由实际注册 HAL SPI 回调的模块转调）
* 入口参数：无
* 返回值：无
* 限定条件：中断上下文（HAL SPI 回调内），两者只写一个完成标志
* 函数说明：HAL 的 SPI 完成/错误回调是全局唯一符号，本模块只实现 TxCplt/RxCplt
* 两个（供自己的收发用），Error 回调的所有权留给回调的实际注册方
* （当前 = Drivers/BSP/CH395F/ch395f.c，按 hspi->Instance 分发）。**SPI1
* 分支必须转调这两个函数**，否则本模块的 DMA 等待在出错时只能靠超时
* 兜底。2026-10-22：取代原先被 CH395F 跨模块裸 extern 直接写的全局变量。
*/
void gd5f_spi1_notify_done(void);
void gd5f_spi1_notify_error(void);
#ifdef __cplusplus
}
#endif