diff options
author | Axel Lin <axel.lin@ingics.com> | 2013-08-30 10:31:25 +0200 |
---|---|---|
committer | Linus Walleij <linus.walleij@linaro.org> | 2013-08-30 10:37:36 +0200 |
commit | c9e3b2d8f75d84c7b333761471f6cef98ec4429a (patch) | |
tree | a50f04f05983c912277a6f999c10522a564ff5a8 | |
parent | pinctrl: sunxi: drop lock on error path (diff) | |
download | linux-c9e3b2d8f75d84c7b333761471f6cef98ec4429a.tar.xz linux-c9e3b2d8f75d84c7b333761471f6cef98ec4429a.zip |
pinctrl: sunxi: Fix off-by-one for valid offset range checking
The valid offset range should be 0 ... chip->ngpio - 1.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
-rw-r--r-- | drivers/pinctrl/pinctrl-sunxi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pinctrl/pinctrl-sunxi.c b/drivers/pinctrl/pinctrl-sunxi.c index 4432e5e062c5..119d2ddedfe7 100644 --- a/drivers/pinctrl/pinctrl-sunxi.c +++ b/drivers/pinctrl/pinctrl-sunxi.c @@ -521,7 +521,7 @@ static int sunxi_pinctrl_gpio_to_irq(struct gpio_chip *chip, unsigned offset) struct sunxi_pinctrl *pctl = dev_get_drvdata(chip->dev); struct sunxi_desc_function *desc; - if (offset > chip->ngpio) + if (offset >= chip->ngpio) return -ENXIO; desc = sunxi_pinctrl_desc_find_function_by_pin(pctl, offset, "irq"); |