xiaoxiang781216 commented on code in PR #19433:
URL: https://github.com/apache/nuttx/pull/19433#discussion_r3579620373
##########
docs/watchdog_automonitor_validation.md:
##########
@@ -0,0 +1,208 @@
+# watchdog_automonitor 验证归档
+
+本文档只记录已经执行的验证和可复核证据。没有日志或截图支持的项目统一标为“待验证”。
Review Comment:
why use Chinese
##########
docs/validation-evidence/watchdog/04-stm32-reset-cycle-0714.log:
##########
@@ -0,0 +1,55 @@
+WWDG reset test armed; starting in 20 seconds
+
+NuttShell (NSH) NuttX-13.0.0
Review Comment:
why add log file
##########
drivers/timers/watchdog.c:
##########
@@ -162,15 +170,47 @@ static int watchdog_automonitor_capture(int irq, FAR void
*context,
FAR void *arg)
{
FAR struct watchdog_upperhalf_s *upper = arg;
+
+ /* Some lower-half drivers (including STM32 WWDG) invoke the capture
Review Comment:
should fix the lower half driver instead
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]