From: Fabio Estevam <fabio.estevam@xxxxxxx>
Add suspend/resume pm hooks, so that the pinctrl state could be changed.
The motivation for doing this was to solve a problem on a imx6-cubox-i
board, where there is a GPIO controlled via pwm-leds that keeps turned on
when the system goes into suspend. Such behaviour is not what we expect
from a system going into suspend.
In order to solve this problem add the suspend/resume pm hooks where
the pinctrl state can be changed accordingly.
This allows to add an extra 'sleep' entry into the pinctrl node, where
the pinctrl can be changed from PWM to GPIO and pull up/pull down can be
configured to turn off the LED during suspend mode.
Signed-off-by: Fabio Estevam <fabio.estevam@xxxxxxx>
---
drivers/leds/leds-pwm.c | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
diff --git a/drivers/leds/leds-pwm.c b/drivers/leds/leds-pwm.c
index 8d456dc6..ff21aa2 100644
--- a/drivers/leds/leds-pwm.c
+++ b/drivers/leds/leds-pwm.c
@@ -14,6 +14,7 @@
#include <linux/module.h>
#include <linux/kernel.h>
+#include <linux/pinctrl/consumer.h>
#include <linux/platform_device.h>
#include <linux/of_platform.h>
#include <linux/fb.h>
@@ -210,6 +211,20 @@ static int led_pwm_remove(struct platform_device *pdev)
return 0;
}
+static int __maybe_unused led_pwm_suspend(struct device *dev)
+{
+ return pinctrl_pm_select_sleep_state(dev);
+}
+
+static int __maybe_unused led_pwm_resume(struct device *dev)
+{
+ return pinctrl_pm_select_default_state(dev);
+}
+
+static const struct dev_pm_ops led_pwm_pm_ops = {
+ SET_SYSTEM_SLEEP_PM_OPS(led_pwm_suspend, led_pwm_resume)
+};
+
static const struct of_device_id of_pwm_leds_match[] = {
{ .compatible = "pwm-leds", },
{},
@@ -222,6 +237,7 @@ static struct platform_driver led_pwm_driver = {
.driver = {
.name = "leds_pwm",
.of_match_table = of_pwm_leds_match,
+ .pm = &led_pwm_pm_ops,
},
};
--
2.7.4
