pinctrl: remove double pin validity check.

Function pin_is_valid just call pin_desc_get which is in pin_request
call some line below. Remove pin_is_valid() check.

Acked-by: Stephen Warren <swarren@nvidia.com>
Signed-off-by: Marek Belisko <marek.belisko@open-nandra.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

diff --git a/drivers/pinctrl/pinmux.c b/drivers/pinctrl/pinmux.c
index 72b760bc880a38acbb4fd31ce7863f4f14a5a1ef..17c3931d5ef0b65ef7d772ed6e0faba0a703e64a 100644 (file)
--- a/drivers/pinctrl/pinmux.c
+++ b/drivers/pinctrl/pinmux.c
@@ -108,16 +108,6 @@ static int pin_request(struct pinctrl_dev *pctldev,
 
        dev_dbg(&pctldev->dev, "request pin %d for %s\n", pin, function);
 
-       if (!pin_is_valid(pctldev, pin)) {
-               dev_err(&pctldev->dev, "pin is invalid\n");
-               return -EINVAL;
-       }
-
-       if (!function) {
-               dev_err(&pctldev->dev, "no function name given\n");
-               return -EINVAL;
-       }
-
        desc = pin_desc_get(pctldev, pin);
        if (desc == NULL) {
                dev_err(&pctldev->dev,
@@ -125,6 +115,11 @@ static int pin_request(struct pinctrl_dev *pctldev,
                goto out;
        }
 
+       if (!function) {
+               dev_err(&pctldev->dev, "no function name given\n");
+               return -EINVAL;
+       }
+
        spin_lock(&desc->lock);
        if (desc->mux_function) {
                spin_unlock(&desc->lock);