diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2009-07-22 00:33:06 +0200 |
---|---|---|
committer | Ben Dooks <ben-linux@fluff.org> | 2009-07-30 00:47:14 +0200 |
commit | 0c997c0eaac8c68fa23719617484dae29bddaedd (patch) | |
tree | 4b12117ebb165056909cfd5033e545460671c7b4 /arch | |
parent | kmemleak: Protect the seq start/next/stop sequence by rcu_read_lock() (diff) | |
download | linux-0c997c0eaac8c68fa23719617484dae29bddaedd.tar.xz linux-0c997c0eaac8c68fa23719617484dae29bddaedd.zip |
S3C24XX: GPIO: Fix pin range check in s3c_gpiolib_getchip
In the s3c_gpiolib_getchip implementation for s3c24xx the check whether a pin is
in the gpio banks range is reversed. Thus the function returns NULL for valid
pins and the gpio chip if its not valid.
As a result gpio states are not saved/restored properly during suspend/resume.
Signed-off-by: Ben Dooks <ben-linux@fluff.org>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/arm/mach-s3c2410/include/mach/gpio-core.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm/mach-s3c2410/include/mach/gpio-core.h b/arch/arm/mach-s3c2410/include/mach/gpio-core.h index 8fe192081d3a..f8b879a7973c 100644 --- a/arch/arm/mach-s3c2410/include/mach/gpio-core.h +++ b/arch/arm/mach-s3c2410/include/mach/gpio-core.h @@ -28,7 +28,7 @@ static inline struct s3c_gpio_chip *s3c_gpiolib_getchip(unsigned int pin) return NULL; chip = &s3c24xx_gpios[pin/32]; - return (S3C2410_GPIO_OFFSET(pin) > chip->chip.ngpio) ? chip : NULL; + return (S3C2410_GPIO_OFFSET(pin) < chip->chip.ngpio) ? chip : NULL; } #endif /* __ASM_ARCH_GPIO_CORE_H */ |