[PATCH 2/2] pinctrl: single: Allow indicating loss of pin states during low-power

From: Florian Fainelli
Date: Wed Sep 20 2017 - 21:04:55 EST


Some platforms (e.g: Broadcom STB: BMIPS_GENERIC/ARCH_BRCMSTB) will lose
their register contents when entering their lower power state. In such a
case, the pinctrl-single driver that is used will not be able to restore
the power states without telling the core about it and having
pinctrl_select_state() check for that.

This patch adds a new optional boolean property that Device Tree can
define in order to obtain exactly that and having the core pinctrl code
take that into account.

Signed-off-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
---
Documentation/devicetree/bindings/pinctrl/pinctrl-single.txt | 4 ++++
drivers/pinctrl/pinctrl-single.c | 3 +++
2 files changed, 7 insertions(+)

diff --git a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.txt b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.txt
index e705acd3612c..e71967f6a1a7 100644
--- a/Documentation/devicetree/bindings/pinctrl/pinctrl-single.txt
+++ b/Documentation/devicetree/bindings/pinctrl/pinctrl-single.txt
@@ -82,6 +82,10 @@ Optional properties:
/* pin base, nr pins & gpio function */
pinctrl-single,gpio-range = <&range 0 3 0 &range 3 9 1>;

+- pinctrl-single,low-power-state-loss : indicates that the pins lose their
+ state during low power modes and therefore need to be restored upon
+ system resumption.
+
- interrupt-controller : standard interrupt controller binding if using
interrupts for wake-up events for example. In this case pinctrl-single
is set up as a chained interrupt controller and the wake-up interrupts
diff --git a/drivers/pinctrl/pinctrl-single.c b/drivers/pinctrl/pinctrl-single.c
index b8b3d932cd73..d69d20b8247a 100644
--- a/drivers/pinctrl/pinctrl-single.c
+++ b/drivers/pinctrl/pinctrl-single.c
@@ -1749,6 +1749,9 @@ static int pcs_probe(struct platform_device *pdev)
goto free;
}

+ if (of_property_read_bool(np, "pinctrl-single,low-power-state-loss"))
+ pcs->pctl->flags |= PINCTRL_FLG_FORCE_STATE;
+
ret = pcs_add_gpio_func(np, pcs);
if (ret < 0)
goto free;
--
2.14.1