diff options
author | 2013-02-21 16:43:29 -0800 | |
---|---|---|
committer | 2013-02-21 17:22:22 -0800 | |
commit | 1d7976b24d11f1ba971b95382735b300be80f3df (patch) | |
tree | 34ee245520ea604e2ff6784a9fd0fbd53dec9e28 /drivers/video | |
parent | backlight: ams369fg06: remove unnecessary NULL deference check (diff) | |
download | linux-1d7976b24d11f1ba971b95382735b300be80f3df.tar.gz linux-1d7976b24d11f1ba971b95382735b300be80f3df.tar.bz2 linux-1d7976b24d11f1ba971b95382735b300be80f3df.zip |
backlight: ams369fg06: replace EFAULT with EINVAL
Replace EFAULT with EINVAL, because EFAULT tends to be for the invalid
memory addresses.
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/backlight/ams369fg06.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/video/backlight/ams369fg06.c b/drivers/video/backlight/ams369fg06.c index b4bf130bc203..22756dda8682 100644 --- a/drivers/video/backlight/ams369fg06.c +++ b/drivers/video/backlight/ams369fg06.c @@ -328,7 +328,7 @@ static int ams369fg06_power_on(struct ams369fg06 *lcd) if (!pd->power_on) { dev_err(lcd->dev, "power_on is NULL.\n"); - return -EFAULT; + return -EINVAL; } else { pd->power_on(lcd->ld, 1); msleep(pd->power_on_delay); @@ -336,7 +336,7 @@ static int ams369fg06_power_on(struct ams369fg06 *lcd) if (!pd->reset) { dev_err(lcd->dev, "reset is NULL.\n"); - return -EFAULT; + return -EINVAL; } else { pd->reset(lcd->ld); msleep(pd->reset_delay); @@ -485,7 +485,7 @@ static int ams369fg06_probe(struct spi_device *spi) lcd->lcd_pd = spi->dev.platform_data; if (!lcd->lcd_pd) { dev_err(&spi->dev, "platform data is NULL\n"); - return -EFAULT; + return -EINVAL; } ld = lcd_device_register("ams369fg06", &spi->dev, lcd, |