summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTushar Behera2012-11-16 07:50:39 +0100
committerLinus Walleij2012-11-23 08:33:41 +0100
commit7d8dd20e56be0196fd015feb19ca38449c607794 (patch)
tree4e3952e33e11ae4923cf49dd4695ef7c9b774bb1
parentMerge branch 'at91' into devel (diff)
downloadkernel-qcow2-linux-7d8dd20e56be0196fd015feb19ca38449c607794.tar.gz
kernel-qcow2-linux-7d8dd20e56be0196fd015feb19ca38449c607794.tar.xz
kernel-qcow2-linux-7d8dd20e56be0196fd015feb19ca38449c607794.zip
pinctrl: SPEAr: Update error check for unsigned variables
Checking '< 0' for unsigned variables always returns false. For error codes, use IS_ERR_VALUE() instead. Signed-off-by: Tushar Behera <tushar.behera@linaro.org> Acked-by: Viresh Kumar <viresh.kumar@linaro.org> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
-rw-r--r--drivers/pinctrl/spear/pinctrl-plgpio.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/pinctrl/spear/pinctrl-plgpio.c b/drivers/pinctrl/spear/pinctrl-plgpio.c
index 5aebbf780fb5..4c045053bbdd 100644
--- a/drivers/pinctrl/spear/pinctrl-plgpio.c
+++ b/drivers/pinctrl/spear/pinctrl-plgpio.c
@@ -284,7 +284,7 @@ static int plgpio_to_irq(struct gpio_chip *chip, unsigned offset)
{
struct plgpio *plgpio = container_of(chip, struct plgpio, chip);
- if (plgpio->irq_base < 0)
+ if (IS_ERR_VALUE(plgpio->irq_base))
return -EINVAL;
return irq_find_mapping(plgpio->irq_domain, offset);