pinctrl: sh-pfc: Return pinconf with arguments in packed format
authorNiklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Tue, 6 Sep 2016 15:14:14 +0000 (17:14 +0200)
committerGeert Uytterhoeven <geert+renesas@glider.be>
Mon, 12 Sep 2016 08:58:23 +0000 (10:58 +0200)
The pinconf-generic code expects configurations with arguments to be
returned in a packed format in order to be displayed properly by
pinconf_generic_dump_one().

Reading /sys/kernel/debug/pinctrl/e6060000.pfc/pinconf-pins on
r8a7795/salvator-x now shows:

    pin 101 (GP_3_5): output drive strength (9 mA), pin power source (3300 selector)

Instead of:

    pin 101 (GP_3_5): output drive strength (0 mA), pin power source (0 selector)

Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Tested-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
drivers/pinctrl/sh-pfc/pinctrl.c

index e208ee04a9f45482044606bc509c70ec5170c4cb..c5772584594ce3c48686f5df0315586dc2e9f8b2 100644 (file)
@@ -596,6 +596,7 @@ static int sh_pfc_pinconf_get(struct pinctrl_dev *pctldev, unsigned _pin,
        struct sh_pfc *pfc = pmx->pfc;
        enum pin_config_param param = pinconf_to_config_param(*config);
        unsigned long flags;
+       unsigned int arg;
 
        if (!sh_pfc_pinconf_validate(pfc, _pin, param))
                return -ENOTSUPP;
@@ -616,7 +617,7 @@ static int sh_pfc_pinconf_get(struct pinctrl_dev *pctldev, unsigned _pin,
                if (bias != param)
                        return -EINVAL;
 
-               *config = 0;
+               arg = 0;
                break;
        }
 
@@ -627,7 +628,7 @@ static int sh_pfc_pinconf_get(struct pinctrl_dev *pctldev, unsigned _pin,
                if (ret < 0)
                        return ret;
 
-               *config = ret;
+               arg = ret;
                break;
        }
 
@@ -646,7 +647,7 @@ static int sh_pfc_pinconf_get(struct pinctrl_dev *pctldev, unsigned _pin,
                val = sh_pfc_read_reg(pfc, pocctrl, 32);
                spin_unlock_irqrestore(&pfc->lock, flags);
 
-               *config = (val & BIT(bit)) ? 3300 : 1800;
+               arg = (val & BIT(bit)) ? 3300 : 1800;
                break;
        }
 
@@ -654,6 +655,7 @@ static int sh_pfc_pinconf_get(struct pinctrl_dev *pctldev, unsigned _pin,
                return -ENOTSUPP;
        }
 
+       *config = pinconf_to_config_packed(param, arg);
        return 0;
 }