diff options
author | Axel Lin <axel.lin@gmail.com> | 2010-05-26 23:42:19 +0200 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-05-27 18:12:41 +0200 |
commit | c1cc9b9775c931a0015178daf92d41aec00580de (patch) | |
tree | 750cd8b9dc72d1e68597e8848cdffe8273ce1701 /drivers/gpio | |
parent | gpiolib: document that names can contain printk format specifiers (diff) | |
download | linux-c1cc9b9775c931a0015178daf92d41aec00580de.tar.xz linux-c1cc9b9775c931a0015178daf92d41aec00580de.zip |
pl061: fix offset value range checking
The valid offset value is 0..PL061_GPIO_NR-1, this patch corrects the
offset value range checking.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
Acked-by: Baruch Siach <baruch@tkos.co.il>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/gpio')
-rw-r--r-- | drivers/gpio/pl061.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpio/pl061.c b/drivers/gpio/pl061.c index 105701a1f05b..ee568c8fcbd0 100644 --- a/drivers/gpio/pl061.c +++ b/drivers/gpio/pl061.c @@ -164,7 +164,7 @@ static int pl061_irq_type(unsigned irq, unsigned trigger) unsigned long flags; u8 gpiois, gpioibe, gpioiev; - if (offset < 0 || offset > PL061_GPIO_NR) + if (offset < 0 || offset >= PL061_GPIO_NR) return -EINVAL; spin_lock_irqsave(&chip->irq_lock, flags); |