pinctrl: Ingenic: Add support for read the pin configuration of X1830.
author周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>
Sun, 18 Apr 2021 14:44:23 +0000 (22:44 +0800)
committerLinus Walleij <linus.walleij@linaro.org>
Wed, 21 Apr 2021 23:53:01 +0000 (01:53 +0200)
Add X1830 support in "ingenic_pinconf_get()", so that it can read the
configuration of X1830 SoC correctly.

Fixes: d7da2a1e4e08 ("pinctrl: Ingenic: Add pinctrl driver for X1830.")
Cc: <stable@vger.kernel.org>
Signed-off-by: 周琰杰 (Zhou Yanjie) <zhouyanjie@wanyeetech.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Reviewed-by: Paul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/1618757073-1724-3-git-send-email-zhouyanjie@wanyeetech.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-ingenic.c

index f15ef81..02729da 100644 (file)
@@ -2109,26 +2109,48 @@ static int ingenic_pinconf_get(struct pinctrl_dev *pctldev,
        enum pin_config_param param = pinconf_to_config_param(*config);
        unsigned int idx = pin % PINS_PER_GPIO_CHIP;
        unsigned int offt = pin / PINS_PER_GPIO_CHIP;
-       bool pull;
+       unsigned int bias;
+       bool pull, pullup, pulldown;
 
-       if (jzpc->info->version >= ID_JZ4770)
-               pull = !ingenic_get_pin_config(jzpc, pin, JZ4770_GPIO_PEN);
-       else
-               pull = !ingenic_get_pin_config(jzpc, pin, JZ4740_GPIO_PULL_DIS);
+       if (jzpc->info->version >= ID_X1830) {
+               unsigned int half = PINS_PER_GPIO_CHIP / 2;
+               unsigned int idxh = (pin % half) * 2;
+
+               if (idx < half)
+                       regmap_read(jzpc->map, offt * jzpc->info->reg_offset +
+                                       X1830_GPIO_PEL, &bias);
+               else
+                       regmap_read(jzpc->map, offt * jzpc->info->reg_offset +
+                                       X1830_GPIO_PEH, &bias);
+
+               bias = (bias >> idxh) & (GPIO_PULL_UP | GPIO_PULL_DOWN);
+
+               pullup = (bias == GPIO_PULL_UP) && (jzpc->info->pull_ups[offt] & BIT(idx));
+               pulldown = (bias == GPIO_PULL_DOWN) && (jzpc->info->pull_downs[offt] & BIT(idx));
+
+       } else {
+               if (jzpc->info->version >= ID_JZ4770)
+                       pull = !ingenic_get_pin_config(jzpc, pin, JZ4770_GPIO_PEN);
+               else
+                       pull = !ingenic_get_pin_config(jzpc, pin, JZ4740_GPIO_PULL_DIS);
+
+               pullup = pull && (jzpc->info->pull_ups[offt] & BIT(idx));
+               pulldown = pull && (jzpc->info->pull_downs[offt] & BIT(idx));
+       }
 
        switch (param) {
        case PIN_CONFIG_BIAS_DISABLE:
-               if (pull)
+               if (pullup || pulldown)
                        return -EINVAL;
                break;
 
        case PIN_CONFIG_BIAS_PULL_UP:
-               if (!pull || !(jzpc->info->pull_ups[offt] & BIT(idx)))
+               if (!pullup)
                        return -EINVAL;
                break;
 
        case PIN_CONFIG_BIAS_PULL_DOWN:
-               if (!pull || !(jzpc->info->pull_downs[offt] & BIT(idx)))
+               if (!pulldown)
                        return -EINVAL;
                break;
 
@@ -2146,7 +2168,7 @@ static void ingenic_set_bias(struct ingenic_pinctrl *jzpc,
        if (jzpc->info->version >= ID_X1830) {
                unsigned int idx = pin % PINS_PER_GPIO_CHIP;
                unsigned int half = PINS_PER_GPIO_CHIP / 2;
-               unsigned int idxh = pin % half * 2;
+               unsigned int idxh = (pin % half) * 2;
                unsigned int offt = pin / PINS_PER_GPIO_CHIP;
 
                if (idx < half) {