diff options
| author | Stanley Jhu <stanleyjhu@google.com> | 2026-11-04 23:16:51 +0800 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-11-04 20:09:24 +0100 |
| commit | 7a93f7e30a3f7c8dd4f5fe4cde54d01c8018e2bc (patch) | |
| tree | 6f4efb6eb3cd59087f280cd0886da28014dff1ae /include/linux | |
| parent | bd56424ce808fb1f92a4dc5bdeedc45b85c5d6cd (diff) | |
| download | linux-7a93f7e30a3f7c8dd4f5fe4cde54d01c8018e2bc.tar.gz linux-7a93f7e30a3f7c8dd4f5fe4cde54d01c8018e2bc.zip | |
PM: runtime: Update runtime_pm.rst for non-fatal resume errors
Since commit eef1d74bf88f ("PM: runtime: Only set runtime_error on
suspend callback failures"), rpm_callback() no longer sets
dev->power.runtime_error when ->runtime_resume() returns an error.
However, Documentation/power/runtime_pm.rst still states in both the
resume callback rules and the ->runtime_idle() comparison that the PM
core treats ->runtime_resume() errors as fatal device errors and blocks
runtime PM helper functions until the status is manually reset.
Update the resume callback semantics bullet to state that on error the
PM core resets the runtime PM status to 'suspended', returns the error
code from synchronous resume helpers, and unlike with the suspend
callback never treats resume failures as fatal, leaving the Section 4
runtime PM helpers unblocked so a subsequent resume request can run
->runtime_resume() again. Also drop ->runtime_resume() from the
->runtime_idle() fatal-error comparison.
Fixes: eef1d74bf88f ("PM: runtime: Only set runtime_error on suspend callback failures")
Fixes: aedc51762a0d ("PM: runtime: Clarify ->runtime_idle() callback return value handling")
Signed-off-by: Stanley Jhu <stanleyjhu@google.com>
Reviewed-by: Brian Norris <briannorris@chromium.org>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
[ rjw: Adjustment of the resume callback errors bullet point ]
Link: https://patch.msgid.link/20261104151651.3035537-1-stanleyjhu@google.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'include/linux')
0 files changed, 0 insertions, 0 deletions
