 AskaEth
|
f9c2ea42a4
|
fix(boot): M21 r1 评审 P1 — bl_cap 把 cur=0 当「无有效值」照写 20（+P2 两处）
P1 根因（评审给的真机驱动证据链，已按其最小修法改）：
/module_driver/pwm_backlight.sh 是 `insmod pwm_backlight.ko ... default_brightness=0
max_brightness=101`，模块 init 把 props.brightness 置 0 再 update_status（PWM 0 分支）→ 上电即灭屏；
该模块由 S11module_driver_default 加载，rcS 自然序排在 S11jpeg_display_shell 之后。于是：
- 函数入口那次 bl_cap：背光节点还不存在，写必然失败（no-op）；
- fb0 出现后画图前那次 bl_cap：读到 cur=0，旧条件 `cur <= cap` 命中「只降不升」跳过分支，
20 从未写入 → 画图时刻亮度 0。
物理后果两头都是缺陷：面板仍亮则 logo 依旧亮（原问题没修）；面板灭则 S11~S21 无可见 logo，
settings.conf 缺失时直到 launcherd 才亮屏（相对 stock「0.1s 后写 60」是回归）。
修改：
- bl_cap 判定改为「仅 1 <= cur <= cap 才视为已有有效值跳过；0 / 空 / 非数字一律照写」。
S22 抢写 1 的竞态保护不变（1 落在 1..20 → 仍跳过，不会被顶回 20）。
- 删掉函数入口那次 bl_cap（节点未建，必然 no-op，且是重定向报错的来源）；只保留 fb0 就绪后、
画 logo 前那一次 —— 同一驱动脚本里 pwm_backlight.ko 先于 soc_fb.ko 加载，fb0 出现即节点就绪。
- 写入成功时打一行 `backlight: -> 20 (was N)` 便于刷机后复核；`2>/dev/null` 移到 `>` 之前，
节点未就绪时连重定向报错一起吞掉（P2-1）。
- S22：失败路径日志由「保持 S11 低亮度」改为「不改背光（当前 brightness=N）」（P2-2）；写入失败
日志同样附当前读数；写入处的 2>/dev/null 次序一并改正。
- 头部注释按时序真相重写（驱动默认 0 = 灭屏，不是「上电很亮」）。
实测（沙箱重写 sysfs 路径 + stub cmd_jpeg_display/nanddump/sleep，dash 与 busybox 各一遍）：
cur=0 → 画图时刻 20、结束 20（修复前 0/0）；cur=1 → 1/1；cur=60、cur=101 → 20/20；
竞态（fb0 就绪前外部写 1）→ 画图时刻 1、结束 1（未被顶回）；节点不可写时 stderr 0 行、仍画图、rc=0。
S22 19 用例全 rc=0（仅 usage rc=1，rcS 不可达）、sleep 调用 0、行为与改文案前一致。
S11 与 S22 内容均变 → fw_content.conf 两条 md5 同步更新为
3ad33301fb9f0884102a47e17a1985ed / 1a0aa33d6f806f3f1648168cc1d8b953
（未改动 meow_watchdog 那条：main tip dfde95b 仍是 553840cd，与本单无关，已在报告中提示。）
|
2026-11-05 22:04:16 +08:00 |
|