Skip to content

drivers/timers/watchdog: add missing parameters to watchdog_automonitor_timeout#19426

Closed
Zepp-Hanzj wants to merge 1 commit into
apache:masterfrom
Zepp-Hanzj:fix/watchdog-notifier-params
Closed

drivers/timers/watchdog: add missing parameters to watchdog_automonitor_timeout#19426
Zepp-Hanzj wants to merge 1 commit into
apache:masterfrom
Zepp-Hanzj:fix/watchdog-notifier-params

Conversation

@Zepp-Hanzj

Copy link
Copy Markdown
Contributor

Summary

Fix incomplete implementation of watchdog timeout notifier chain.

The function watchdog_automonitor_timeout() was declared with void parameters but used undefined action and data variables in its body when calling atomic_notifier_call_chain().

Add action and data parameters to the function signature in both the implementation and header file.

Impact

  • Bug fix only, completes the notifier chain feature
  • No functional changes (function has no callers yet)
  • Enables future use of watchdog timeout notifications

Testing

  • Verified by code inspection against master
  • checkpatch passes
  • Builds successfully with sim:nsh + CONFIG_WATCHDOG_TIMEOUT_NOTIFIER=y

Signed-off-by: hanzhijian hanzhijian@zepp.com

…or_timeout

Fix incomplete implementation of watchdog timeout notifier chain.
The function watchdog_automonitor_timeout() was declared with void
parameters but used undefined 'action' and 'data' variables in its
body when calling atomic_notifier_call_chain().

Add 'action' and 'data' parameters to the function signature in both
the implementation and header file.

Fixes: 43797ea (drivers/timers/watchdog: add watchdog timer notifier chain)

Signed-off-by: hanzhijian <hanzhijian@zepp.com>
Author: hanzhijian <hanzhijian@zepp.com>
@github-actions github-actions Bot added Area: Drivers Drivers issues Size: XS The size of the change in this PR is very small labels Jul 13, 2026
@Zepp-Hanzj

Copy link
Copy Markdown
Contributor Author

Closing as requested.

@Zepp-Hanzj Zepp-Hanzj closed this Jul 13, 2026
@github-actions

Copy link
Copy Markdown

MemBrowse Memory Report

No memory changes detected for:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: Drivers Drivers issues Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant