lkml.org 
[lkml]   [2013]   [Jul]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Subject[PATCH 4/4] drivers: Add pinctrl handling for dynamic pin states
From
Date
We want to have static pin states handled separately from
dynamic pin states, so let's add optional state_active.

Then if state_active is defined, let's check and make sure
state_idle and state_sleep match state_active for the
pin groups to avoid checking them during runtime as the
active and idle pins may need to be toggled for many
devices every time we enter and exit idle.

Cc: Stephen Warren <swarren@wwwdotorg.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Tony Lindgren <tony@atomide.com>
---
drivers/base/pinctrl.c | 39 +++++++++++++++++++++++++++++++++++++--
1 file changed, 37 insertions(+), 2 deletions(-)

diff --git a/drivers/base/pinctrl.c b/drivers/base/pinctrl.c
index 5fb74b4..49644ed 100644
--- a/drivers/base/pinctrl.c
+++ b/drivers/base/pinctrl.c
@@ -34,6 +34,7 @@ int pinctrl_bind_pins(struct device *dev)
goto cleanup_alloc;
}

+ /* Default static pins that don't need to change */
dev->pins->default_state = pinctrl_lookup_state(dev->pins->p,
PINCTRL_STATE_DEFAULT);
if (IS_ERR(dev->pins->default_state)) {
@@ -48,23 +49,57 @@ int pinctrl_bind_pins(struct device *dev)
goto cleanup_get;
}

+ /* Optional runtime dynamic pins in addition to the static pins */
+ dev->pins->active_state = pinctrl_lookup_state(dev->pins->p,
+ PINCTRL_STATE_ACTIVE);
+ if (IS_ERR(dev->pins->active_state)) {
+ /* Not supplying this state is perfectly legal */
+ dev_dbg(dev, "no active pinctrl state\n");
+ } else {
+ ret = pinctrl_select_dynamic(dev->pins->p,
+ dev->pins->active_state);
+ if (ret) {
+ dev_dbg(dev, "failed to select active pinctrl state\n");
+ goto cleanup_get;
+ }
+ }
+
#ifdef CONFIG_PM
/*
* If power management is enabled, we also look for the optional
* sleep and idle pin states, with semantics as defined in
* <linux/pinctrl/pinctrl-state.h>
+ *
+ * Note that if active state is defined, sleep and idle states must
+ * cover the same pin groups as active state.
*/
dev->pins->sleep_state = pinctrl_lookup_state(dev->pins->p,
PINCTRL_STATE_SLEEP);
- if (IS_ERR(dev->pins->sleep_state))
+ if (IS_ERR(dev->pins->sleep_state)) {
/* Not supplying this state is perfectly legal */
dev_dbg(dev, "no sleep pinctrl state\n");
+ } else if (!IS_ERR(dev->pins->active_state)) {
+ ret = pinctrl_check_dynamic(dev, dev->pins->active_state,
+ dev->pins->sleep_state);
+ if (ret) {
+ dev_err(dev, "sleep state groups do not match active state\n");
+ dev->pins->sleep_state = ERR_PTR(-EINVAL);
+ }
+ }

dev->pins->idle_state = pinctrl_lookup_state(dev->pins->p,
PINCTRL_STATE_IDLE);
- if (IS_ERR(dev->pins->idle_state))
+ if (IS_ERR(dev->pins->idle_state)) {
/* Not supplying this state is perfectly legal */
dev_dbg(dev, "no idle pinctrl state\n");
+ } else if (!IS_ERR(dev->pins->active_state)) {
+ ret = pinctrl_check_dynamic(dev, dev->pins->active_state,
+ dev->pins->idle_state);
+ if (ret) {
+ dev_err(dev, "idle state groups do not match active state\n");
+ dev->pins->idle_state = ERR_PTR(-EINVAL);
+ }
+ }
#endif

return 0;


\
 
 \ /
  Last update: 2013-07-16 11:41    [W:0.196 / U:0.824 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site