Re: [PATCH] PM: runtime: add system sleep callbacks to EXPORT_*_RUNTIME_DEV_PM_OPS()

[Rafael J. Wysocki]

On Sun, Mar 1, 2026 at 2:22 AM Neel Bullywon <neelb2403@xxxxxxxxx> wrote:
>
> DEFINE_RUNTIME_DEV_PM_OPS() sets pm_runtime_force_suspend() and
> pm_runtime_force_resume() as system sleep callbacks via
> _DEFINE_DEV_PM_OPS(),

Isn't that a mistake?

I'm not actually sure if it would work for everybody.  Stuff like USB
may want to use the exact same callbacks for runtime PM and
system-wide PM.

> but the exported variants
> (EXPORT_RUNTIME_DEV_PM_OPS, EXPORT_GPL_RUNTIME_DEV_PM_OPS,
> EXPORT_NS_RUNTIME_DEV_PM_OPS, EXPORT_NS_GPL_RUNTIME_DEV_PM_OPS)
> only set RUNTIME_PM_OPS() and omit SYSTEM_SLEEP_PM_OPS(). This
> means drivers using the export macros do not get system sleep
> support, contrary to what the documentation suggests.
>
> Add SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
> pm_runtime_force_resume) to all four export variants to match the
> behavior of DEFINE_RUNTIME_DEV_PM_OPS().
>
> All current users of these macros are IIO drivers (bme680, bmp280,
> kxsd9, mpl115, bmi088-accel) and none implement custom system sleep
> callbacks, so this change is safe for all existing callers.

So maybe let IIO have their own set of macros?

> Suggested-by: Jonathan Cameron <jic23@xxxxxxxxxx>
> Suggested-by: Paul Cercueil <paul@xxxxxxxxxxxxxxx>
> Signed-off-by: Neel Bullywon <neelb2403@xxxxxxxxx>
> ---
>  include/linux/pm_runtime.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/include/linux/pm_runtime.h b/include/linux/pm_runtime.h
> index 41037c513f06..c6a6a0569b11 100644
> --- a/include/linux/pm_runtime.h
> +++ b/include/linux/pm_runtime.h
> @@ -42,18 +42,26 @@
>
>  #define EXPORT_RUNTIME_DEV_PM_OPS(name, suspend_fn, resume_fn, idle_fn) \
>         EXPORT_DEV_PM_OPS(name) = { \
> +               SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, \
> +                                   pm_runtime_force_resume) \
>                 RUNTIME_PM_OPS(suspend_fn, resume_fn, idle_fn) \
>         }
>  #define EXPORT_GPL_RUNTIME_DEV_PM_OPS(name, suspend_fn, resume_fn, idle_fn) \
>         EXPORT_GPL_DEV_PM_OPS(name) = { \
> +               SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, \
> +                                   pm_runtime_force_resume) \
>                 RUNTIME_PM_OPS(suspend_fn, resume_fn, idle_fn) \
>         }
>  #define EXPORT_NS_RUNTIME_DEV_PM_OPS(name, suspend_fn, resume_fn, idle_fn, ns) \
>         EXPORT_NS_DEV_PM_OPS(name, ns) = { \
> +               SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, \
> +                                   pm_runtime_force_resume) \
>                 RUNTIME_PM_OPS(suspend_fn, resume_fn, idle_fn) \
>         }
>  #define EXPORT_NS_GPL_RUNTIME_DEV_PM_OPS(name, suspend_fn, resume_fn, idle_fn, ns) \
>         EXPORT_NS_GPL_DEV_PM_OPS(name, ns) = { \
> +               SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend, \
> +                                   pm_runtime_force_resume) \
>                 RUNTIME_PM_OPS(suspend_fn, resume_fn, idle_fn) \
>         }
>
> --
> 2.44.0
>