summaryrefslogtreecommitdiff
path: root/drivers/pinctrl/pinctrl-gemini.c
diff options
context:
space:
mode:
authorLinus Walleij <linus.walleij@linaro.org>2018-10-09 11:11:53 +0300
committerLinus Walleij <linus.walleij@linaro.org>2018-10-10 14:55:04 +0300
commitd17f477c5bc6b4a5dd9f51ae263870da132a8e89 (patch)
tree06f1d81d57eca0b7fda8adc95c1e9b6b0b84441a /drivers/pinctrl/pinctrl-gemini.c
parent71a9d395aa12279e53d411ef3345fddc840347c6 (diff)
downloadlinux-d17f477c5bc6b4a5dd9f51ae263870da132a8e89.tar.xz
pinctrl: gemini: Mask and set properly
The code was written under the assumption that the regmap_update_bits() would mask the bits in the mask and set the bits in the value. It missed the points that it will not set bits in the value unless these are also masked in the mask. Set value bits that are not in the mask will simply be ignored. Fixes: 06351d133dea ("pinctrl: add a Gemini SoC pin controller") Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Diffstat (limited to 'drivers/pinctrl/pinctrl-gemini.c')
-rw-r--r--drivers/pinctrl/pinctrl-gemini.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinctrl-gemini.c b/drivers/pinctrl/pinctrl-gemini.c
index fa7d998e1d5a..1e484a36ff07 100644
--- a/drivers/pinctrl/pinctrl-gemini.c
+++ b/drivers/pinctrl/pinctrl-gemini.c
@@ -2184,7 +2184,8 @@ static int gemini_pmx_set_mux(struct pinctrl_dev *pctldev,
func->name, grp->name);
regmap_read(pmx->map, GLOBAL_MISC_CTRL, &before);
- regmap_update_bits(pmx->map, GLOBAL_MISC_CTRL, grp->mask,
+ regmap_update_bits(pmx->map, GLOBAL_MISC_CTRL,
+ grp->mask | grp->value,
grp->value);
regmap_read(pmx->map, GLOBAL_MISC_CTRL, &after);