* [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration
@ 2020-03-25 8:25 Baolin Wang
2020-03-25 8:25 ` [PATCH 2/2] pinctrl: sprd: Add pin high impedance mode support Baolin Wang
2020-03-27 21:07 ` [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration Linus Walleij
0 siblings, 2 replies; 4+ messages in thread
From: Baolin Wang @ 2020-03-25 8:25 UTC (permalink / raw)
To: linus.walleij
Cc: linhua.xu, orsonzhai, zhang.lyra, baolin.wang7, linux-gpio, linux-kernel
From: Linhua Xu <linhua.xu@unisoc.com>
The Spreadtrum pin controller did not supply registers to set high level
or low level for output mode, instead we should let the pin controller
current configuration drive values on the line. So we should use the
PIN_CONFIG_OUTPUT_ENABLE configuration to enable or disable the output
mode.
[Baolin Wang changes the commit message]
Fixes: 41d32cfce1ae ("pinctrl: sprd: Add Spreadtrum pin control driver")
Signed-off-by: Linhua Xu <linhua.xu@unisoc.com>
Signed-off-by: Baolin Wang <baolin.wang7@gmail.com>
---
drivers/pinctrl/sprd/pinctrl-sprd.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/pinctrl/sprd/pinctrl-sprd.c b/drivers/pinctrl/sprd/pinctrl-sprd.c
index ea04bac..8e39610 100644
--- a/drivers/pinctrl/sprd/pinctrl-sprd.c
+++ b/drivers/pinctrl/sprd/pinctrl-sprd.c
@@ -464,7 +464,7 @@ static int sprd_pinconf_get(struct pinctrl_dev *pctldev, unsigned int pin_id,
case PIN_CONFIG_INPUT_ENABLE:
arg = (reg >> SLEEP_INPUT_SHIFT) & SLEEP_INPUT_MASK;
break;
- case PIN_CONFIG_OUTPUT:
+ case PIN_CONFIG_OUTPUT_ENABLE:
arg = reg & SLEEP_OUTPUT_MASK;
break;
case PIN_CONFIG_DRIVE_STRENGTH:
@@ -635,9 +635,13 @@ static int sprd_pinconf_set(struct pinctrl_dev *pctldev, unsigned int pin_id,
shift = SLEEP_INPUT_SHIFT;
}
break;
- case PIN_CONFIG_OUTPUT:
+ case PIN_CONFIG_OUTPUT_ENABLE:
if (is_sleep_config == true) {
- val |= SLEEP_OUTPUT;
+ if (arg > 0)
+ val |= SLEEP_OUTPUT;
+ else
+ val &= ~SLEEP_OUTPUT;
+
mask = SLEEP_OUTPUT_MASK;
shift = SLEEP_OUTPUT_SHIFT;
}
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PATCH 2/2] pinctrl: sprd: Add pin high impedance mode support
2020-03-25 8:25 [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration Baolin Wang
@ 2020-03-25 8:25 ` Baolin Wang
2020-03-27 21:11 ` Linus Walleij
2020-03-27 21:07 ` [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration Linus Walleij
1 sibling, 1 reply; 4+ messages in thread
From: Baolin Wang @ 2020-03-25 8:25 UTC (permalink / raw)
To: linus.walleij
Cc: linhua.xu, orsonzhai, zhang.lyra, baolin.wang7, linux-gpio, linux-kernel
From: Linhua Xu <linhua.xu@unisoc.com>
For Spreadtrum pin controller, it will be the high impedance
mode if disable input and output mode for a pin. Thus add
PIN_CONFIG_BIAS_HIGH_IMPEDANCE configuration to support it.
Signed-off-by: Linhua Xu <linhua.xu@unisoc.com>
Signed-off-by: Baolin Wang <baolin.wang7@gmail.com>
---
drivers/pinctrl/sprd/pinctrl-sprd.c | 12 ++++++++++++
1 file changed, 12 insertions(+)
diff --git a/drivers/pinctrl/sprd/pinctrl-sprd.c b/drivers/pinctrl/sprd/pinctrl-sprd.c
index 8e39610..48cbf2a 100644
--- a/drivers/pinctrl/sprd/pinctrl-sprd.c
+++ b/drivers/pinctrl/sprd/pinctrl-sprd.c
@@ -467,6 +467,12 @@ static int sprd_pinconf_get(struct pinctrl_dev *pctldev, unsigned int pin_id,
case PIN_CONFIG_OUTPUT_ENABLE:
arg = reg & SLEEP_OUTPUT_MASK;
break;
+ case PIN_CONFIG_BIAS_HIGH_IMPEDANCE:
+ if ((reg & SLEEP_OUTPUT) || (reg & SLEEP_INPUT))
+ return -EINVAL;
+
+ arg = 1;
+ break;
case PIN_CONFIG_DRIVE_STRENGTH:
arg = (reg >> DRIVE_STRENGTH_SHIFT) &
DRIVE_STRENGTH_MASK;
@@ -646,6 +652,12 @@ static int sprd_pinconf_set(struct pinctrl_dev *pctldev, unsigned int pin_id,
shift = SLEEP_OUTPUT_SHIFT;
}
break;
+ case PIN_CONFIG_BIAS_HIGH_IMPEDANCE:
+ if (is_sleep_config == true) {
+ val = shift = 0;
+ mask = SLEEP_OUTPUT | SLEEP_INPUT;
+ }
+ break;
case PIN_CONFIG_DRIVE_STRENGTH:
if (arg < 2 || arg > 60)
return -EINVAL;
--
1.9.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH 2/2] pinctrl: sprd: Add pin high impedance mode support
2020-03-25 8:25 ` [PATCH 2/2] pinctrl: sprd: Add pin high impedance mode support Baolin Wang
@ 2020-03-27 21:11 ` Linus Walleij
0 siblings, 0 replies; 4+ messages in thread
From: Linus Walleij @ 2020-03-27 21:11 UTC (permalink / raw)
To: Baolin Wang
Cc: linhua.xu, Orson Zhai, Lyra Zhang, open list:GPIO SUBSYSTEM,
linux-kernel
On Wed, Mar 25, 2020 at 9:25 AM Baolin Wang <baolin.wang7@gmail.com> wrote:
> From: Linhua Xu <linhua.xu@unisoc.com>
>
> For Spreadtrum pin controller, it will be the high impedance
> mode if disable input and output mode for a pin. Thus add
> PIN_CONFIG_BIAS_HIGH_IMPEDANCE configuration to support it.
>
> Signed-off-by: Linhua Xu <linhua.xu@unisoc.com>
> Signed-off-by: Baolin Wang <baolin.wang7@gmail.com>
Patch applied! Nice engineering here, figuring out the right
fit for pin control configs, thanks folks!
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration
2020-03-25 8:25 [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration Baolin Wang
2020-03-25 8:25 ` [PATCH 2/2] pinctrl: sprd: Add pin high impedance mode support Baolin Wang
@ 2020-03-27 21:07 ` Linus Walleij
1 sibling, 0 replies; 4+ messages in thread
From: Linus Walleij @ 2020-03-27 21:07 UTC (permalink / raw)
To: Baolin Wang
Cc: linhua.xu, Orson Zhai, Lyra Zhang, open list:GPIO SUBSYSTEM,
linux-kernel
On Wed, Mar 25, 2020 at 9:25 AM Baolin Wang <baolin.wang7@gmail.com> wrote:
> From: Linhua Xu <linhua.xu@unisoc.com>
>
> The Spreadtrum pin controller did not supply registers to set high level
> or low level for output mode, instead we should let the pin controller
> current configuration drive values on the line. So we should use the
> PIN_CONFIG_OUTPUT_ENABLE configuration to enable or disable the output
> mode.
>
> [Baolin Wang changes the commit message]
> Fixes: 41d32cfce1ae ("pinctrl: sprd: Add Spreadtrum pin control driver")
> Signed-off-by: Linhua Xu <linhua.xu@unisoc.com>
> Signed-off-by: Baolin Wang <baolin.wang7@gmail.com>
Patch applied.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-03-27 21:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-25 8:25 [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration Baolin Wang
2020-03-25 8:25 ` [PATCH 2/2] pinctrl: sprd: Add pin high impedance mode support Baolin Wang
2020-03-27 21:11 ` Linus Walleij
2020-03-27 21:07 ` [PATCH 1/2] pinctrl: sprd: Use the correct pin output configuration Linus Walleij
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).