diff options
author | Johan Hovold <johan@kernel.org> | 2015-01-12 17:20:51 +0100 |
---|---|---|
committer | Luis Henriques <luis.henriques@canonical.com> | 2015-02-04 10:57:48 +0000 |
commit | 1d3695b3687222cebf05eeab2c32d1c65ccac765 (patch) | |
tree | ec4b6cfe8c6d0dd923b12e73844793081af2a4ff | |
parent | f2e727848d2fd8c4daf55cd658688a784505c3ca (diff) |
pinctrl: lantiq: remove bogus of_gpio_chip_add
commit 41f632fe177bc4822c2e8236fe7c291e6e9eb6f8 upstream.
Remove bogus call to of_gpiochip_add (and of_gpio_chip remove in error
path) which is also called when adding the gpio chip.
This prevents adding the same pinctrl range twice.
Fixes: 3f8c50c9b110 ("OF: pinctrl: MIPS: lantiq: implement lantiq/xway
pinctrl support")
Signed-off-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Luis Henriques <luis.henriques@canonical.com>
-rw-r--r-- | drivers/pinctrl/pinctrl-xway.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/pinctrl/pinctrl-xway.c b/drivers/pinctrl/pinctrl-xway.c index 37040ab42890..fdff39ff5021 100644 --- a/drivers/pinctrl/pinctrl-xway.c +++ b/drivers/pinctrl/pinctrl-xway.c @@ -798,10 +798,8 @@ static int pinmux_xway_probe(struct platform_device *pdev) /* load the gpio chip */ xway_chip.dev = &pdev->dev; - of_gpiochip_add(&xway_chip); ret = gpiochip_add(&xway_chip); if (ret) { - of_gpiochip_remove(&xway_chip); dev_err(&pdev->dev, "Failed to register gpio chip\n"); return ret; } |