* [PATCH] PM / Runtime: Add new helper function: pm_runtime_active()
@ 2013-01-23 8:54 shuox.liu
2013-01-24 23:00 ` Rafael J. Wysocki
0 siblings, 1 reply; 2+ messages in thread
From: shuox.liu @ 2013-01-23 8:54 UTC (permalink / raw)
To: linux-kernel, linux-pm
Cc: pavel, rjw, len.brown, yanmin_zhang, ShuoX Liu, Yanmin Zhang
From: ShuoX Liu <shuox.liu@intel.com>
This boolean function simply returns whether or not the runtime
status of the device is 'active'. The typical scenario is driver
calls pm_runtime_get firstly, then check pm_runtime_active in
atomic environment.
Also add entry to Documentation/power/runtime.txt
Signed-off-by: Yanmin Zhang <yanmin.zhang@intel.com>
Signed-off-by: ShuoX Liu <shuox.liu@intel.com>
---
Documentation/power/runtime_pm.txt | 4 ++++
include/linux/pm_runtime.h | 7 +++++++
2 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/Documentation/power/runtime_pm.txt b/Documentation/power/runtime_pm.txt
index 03591a7..6c9f5d9 100644
--- a/Documentation/power/runtime_pm.txt
+++ b/Documentation/power/runtime_pm.txt
@@ -426,6 +426,10 @@ drivers/base/power/runtime.c and include/linux/pm_runtime.h:
'power.runtime_error' is set or 'power.disable_depth' is greater than
zero)
+ bool pm_runtime_active(struct device *dev);
+ - return true if the device's runtime PM status is 'active' or its
+ 'power.disable_depth' field is not equal to zero, or false otherwise
+
bool pm_runtime_suspended(struct device *dev);
- return true if the device's runtime PM status is 'suspended' and its
'power.disable_depth' field is equal to zero, or false otherwise
diff --git a/include/linux/pm_runtime.h b/include/linux/pm_runtime.h
index f271860..c785c21 100644
--- a/include/linux/pm_runtime.h
+++ b/include/linux/pm_runtime.h
@@ -80,6 +80,12 @@ static inline bool pm_runtime_suspended(struct device *dev)
&& !dev->power.disable_depth;
}
+static inline bool pm_runtime_active(struct device *dev)
+{
+ return dev->power.runtime_status == RPM_ACTIVE
+ || dev->power.disable_depth;
+}
+
static inline bool pm_runtime_status_suspended(struct device *dev)
{
return dev->power.runtime_status == RPM_SUSPENDED;
@@ -132,6 +138,7 @@ static inline void pm_runtime_put_noidle(struct device *dev) {}
static inline bool device_run_wake(struct device *dev) { return false; }
static inline void device_set_run_wake(struct device *dev, bool enable) {}
static inline bool pm_runtime_suspended(struct device *dev) { return false; }
+static inline bool pm_runtime_active(struct device *dev) { return true; }
static inline bool pm_runtime_status_suspended(struct device *dev) { return false; }
static inline bool pm_runtime_enabled(struct device *dev) { return false; }
--
1.7.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] PM / Runtime: Add new helper function: pm_runtime_active()
2013-01-23 8:54 [PATCH] PM / Runtime: Add new helper function: pm_runtime_active() shuox.liu
@ 2013-01-24 23:00 ` Rafael J. Wysocki
0 siblings, 0 replies; 2+ messages in thread
From: Rafael J. Wysocki @ 2013-01-24 23:00 UTC (permalink / raw)
To: shuox.liu
Cc: linux-kernel, linux-pm, pavel, len.brown, yanmin_zhang, Yanmin Zhang
On Wednesday, January 23, 2013 04:54:01 PM shuox.liu@intel.com wrote:
> From: ShuoX Liu <shuox.liu@intel.com>
>
> This boolean function simply returns whether or not the runtime
> status of the device is 'active'. The typical scenario is driver
> calls pm_runtime_get firstly, then check pm_runtime_active in
> atomic environment.
>
> Also add entry to Documentation/power/runtime.txt
Well, why not if that helps.
Applied to linux-pm.git/linux-next.
Thanks,
Rafael
> Signed-off-by: Yanmin Zhang <yanmin.zhang@intel.com>
> Signed-off-by: ShuoX Liu <shuox.liu@intel.com>
> ---
> Documentation/power/runtime_pm.txt | 4 ++++
> include/linux/pm_runtime.h | 7 +++++++
> 2 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/Documentation/power/runtime_pm.txt b/Documentation/power/runtime_pm.txt
> index 03591a7..6c9f5d9 100644
> --- a/Documentation/power/runtime_pm.txt
> +++ b/Documentation/power/runtime_pm.txt
> @@ -426,6 +426,10 @@ drivers/base/power/runtime.c and include/linux/pm_runtime.h:
> 'power.runtime_error' is set or 'power.disable_depth' is greater than
> zero)
>
> + bool pm_runtime_active(struct device *dev);
> + - return true if the device's runtime PM status is 'active' or its
> + 'power.disable_depth' field is not equal to zero, or false otherwise
> +
> bool pm_runtime_suspended(struct device *dev);
> - return true if the device's runtime PM status is 'suspended' and its
> 'power.disable_depth' field is equal to zero, or false otherwise
> diff --git a/include/linux/pm_runtime.h b/include/linux/pm_runtime.h
> index f271860..c785c21 100644
> --- a/include/linux/pm_runtime.h
> +++ b/include/linux/pm_runtime.h
> @@ -80,6 +80,12 @@ static inline bool pm_runtime_suspended(struct device *dev)
> && !dev->power.disable_depth;
> }
>
> +static inline bool pm_runtime_active(struct device *dev)
> +{
> + return dev->power.runtime_status == RPM_ACTIVE
> + || dev->power.disable_depth;
> +}
> +
> static inline bool pm_runtime_status_suspended(struct device *dev)
> {
> return dev->power.runtime_status == RPM_SUSPENDED;
> @@ -132,6 +138,7 @@ static inline void pm_runtime_put_noidle(struct device *dev) {}
> static inline bool device_run_wake(struct device *dev) { return false; }
> static inline void device_set_run_wake(struct device *dev, bool enable) {}
> static inline bool pm_runtime_suspended(struct device *dev) { return false; }
> +static inline bool pm_runtime_active(struct device *dev) { return true; }
> static inline bool pm_runtime_status_suspended(struct device *dev) { return false; }
> static inline bool pm_runtime_enabled(struct device *dev) { return false; }
>
>
--
I speak only for myself.
Rafael J. Wysocki, Intel Open Source Technology Center.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-01-24 22:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-01-23 8:54 [PATCH] PM / Runtime: Add new helper function: pm_runtime_active() shuox.liu
2013-01-24 23:00 ` Rafael J. Wysocki
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.