On Saturday 15 August 2009, Magnus Damm wrote:
> From: Magnus Damm <damm@xxxxxxxxxx>
>
> This patch adds default Runtime PM callbacks to the dev_pm_ops
> belonging to the platform bus. The callbacks are weak symbols
> that architecture specific code can override.
>
> Allows Runtime PM even though CONFIG_PM_SLEEP=n.
>
> Signed-off-by: Magnus Damm <damm@xxxxxxxxxx>
Thanks for the patch.
Greg, is it fine with you?
Rafael
> ---
>
> Needed by the SuperH Mobile Runtime PM implementation. Depends on
> "PM: Introduce core framework for run-time PM of I/O devices (rev. 17)"
>
> Changes since 20090812 (v15 -> v17):
> - platform_pm_runtime_idle() now returns int
>
> Changes since 20090807:
> - removed pm_runtime_enable()
>
> Changes since 20090731:
> - Based on "Driver Core: Runtime PM callbacks for the Platform Bus V2"
> - Reworked to handle CONFIG_PM_SLEEP=n case
>
> drivers/base/platform.c | 46 +++++++++++++++++++++++++++++++++++++---------
> 1 file changed, 37 insertions(+), 9 deletions(-)
>
> --- 0001/drivers/base/platform.c
> +++ work/drivers/base/platform.c 2009-08-15 12:15:38.000000000 +0900
> @@ -17,6 +17,7 @@
> #include <linux/bootmem.h>
> #include <linux/err.h>
> #include <linux/slab.h>
> +#include <linux/pm_runtime.h>
>
> #include "base.h"
>
> @@ -680,6 +681,13 @@ static void platform_pm_complete(struct
> drv->pm->complete(dev);
> }
>
> +#else /* !CONFIG_PM_SLEEP */
> +
> +#define platform_pm_prepare NULL
> +#define platform_pm_complete NULL
> +
> +#endif /* !CONFIG_PM_SLEEP */
> +
> #ifdef CONFIG_SUSPEND
>
> static int platform_pm_suspend(struct device *dev)
> @@ -922,6 +930,31 @@ static int platform_pm_restore_noirq(str
>
> #endif /* !CONFIG_HIBERNATION */
>
> +#ifdef CONFIG_PM_RUNTIME
> +
> +int __weak platform_pm_runtime_suspend(struct device *dev)
> +{
> + return -ENOSYS;
> +};
> +
> +int __weak platform_pm_runtime_resume(struct device *dev)
> +{
> + return -ENOSYS;
> +};
> +
> +int __weak platform_pm_runtime_idle(struct device *dev)
> +{
> + return -ENOSYS;
> +};
> +
> +#else /* !CONFIG_PM_RUNTIME */
> +
> +#define platform_pm_runtime_suspend NULL
> +#define platform_pm_runtime_resume NULL
> +#define platform_pm_runtime_idle NULL
> +
> +#endif /* !CONFIG_PM_RUNTIME */
> +
> static struct dev_pm_ops platform_dev_pm_ops = {
> .prepare = platform_pm_prepare,
> .complete = platform_pm_complete,
> @@ -937,22 +970,17 @@ static struct dev_pm_ops platform_dev_pm
> .thaw_noirq = platform_pm_thaw_noirq,
> .poweroff_noirq = platform_pm_poweroff_noirq,
> .restore_noirq = platform_pm_restore_noirq,
> + .runtime_suspend = platform_pm_runtime_suspend,
> + .runtime_resume = platform_pm_runtime_resume,
> + .runtime_idle = platform_pm_runtime_idle,
> };
>
> -#define PLATFORM_PM_OPS_PTR (&platform_dev_pm_ops)
> -
> -#else /* !CONFIG_PM_SLEEP */
> -
> -#define PLATFORM_PM_OPS_PTR NULL
> -
> -#endif /* !CONFIG_PM_SLEEP */
> -
> struct bus_type platform_bus_type = {
> .name = "platform",
> .dev_attrs = platform_dev_attrs,
> .match = platform_match,
> .uevent = platform_uevent,
> - .pm = PLATFORM_PM_OPS_PTR,
> + .pm = &platform_dev_pm_ops,
> };
> EXPORT_SYMBOL_GPL(platform_bus_type);
_______________________________________________
linux-pm mailing list
linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linux-foundation.org/mailman/listinfo/linux-pm
