pinctrl: lpc18xx: create pin cap lookup helper
authorJoachim Eastwood <manabian@gmail.com>
Wed, 6 May 2015 17:11:21 +0000 (19:11 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Tue, 12 May 2015 11:17:48 +0000 (13:17 +0200)
Both pconf_get_pin and pconf_set_pin needs to lookup pin cap based
on the pin number. Create a common helper function that both
functions can use that also handles the case where no pin number is
found in the pins array.

This also fixes a small bug in pconf_get_pin where pconf_get_i2c0
would use the pins array index rather than the pin number.

Signed-off-by: Joachim Eastwood <manabian@gmail.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/pinctrl-lpc18xx.c

index 15fe1e2535c25c2bd68ae613d8f37ff6d8fb9c23..0facb7e64fef59fe271629811bbf42184a03427c 100644 (file)
@@ -729,6 +729,18 @@ static int lpc18xx_pconf_get_pin(enum pin_config_param param, int *arg, u32 reg,
        return 0;
 }
 
+static struct lpc18xx_pin_caps *lpc18xx_get_pin_caps(unsigned pin)
+{
+       int i;
+
+       for (i = 0; i < ARRAY_SIZE(lpc18xx_pins); i++) {
+               if (lpc18xx_pins[i].number == pin)
+                       return lpc18xx_pins[i].drv_data;
+       }
+
+       return NULL;
+}
+
 static int lpc18xx_pconf_get(struct pinctrl_dev *pctldev, unsigned pin,
                             unsigned long *config)
 {
@@ -737,14 +749,11 @@ static int lpc18xx_pconf_get(struct pinctrl_dev *pctldev, unsigned pin,
        struct lpc18xx_pin_caps *pin_cap;
        int ret, arg = 0;
        u32 reg;
-       int i;
 
-       for (i = 0; i < ARRAY_SIZE(lpc18xx_pins); i++) {
-               if (lpc18xx_pins[i].number == pin)
-                       pin = i;
-       }
+       pin_cap = lpc18xx_get_pin_caps(pin);
+       if (!pin_cap)
+               return -EINVAL;
 
-       pin_cap = lpc18xx_pins[pin].drv_data;
        reg = readl(scu->base + pin_cap->offset);
 
        if (pin_cap->type == TYPE_I2C0)
@@ -905,12 +914,10 @@ static int lpc18xx_pconf_set(struct pinctrl_dev *pctldev, unsigned pin,
        int ret;
        int i;
 
-       for (i = 0; i < ARRAY_SIZE(lpc18xx_pins); i++) {
-               if (lpc18xx_pins[i].number == pin)
-                       break;
-       }
+       pin_cap = lpc18xx_get_pin_caps(pin);
+       if (!pin_cap)
+               return -EINVAL;
 
-       pin_cap = lpc18xx_pins[i].drv_data;
        reg = readl(scu->base + pin_cap->offset);
 
        for (i = 0; i < num_configs; i++) {