diff options
author | Jingoo Han <jg1.han@samsung.com> | 2014-08-27 03:12:37 +0200 |
---|---|---|
committer | Lee Jones <lee.jones@linaro.org> | 2014-08-29 09:25:33 +0200 |
commit | b50ffb8a5f7f7dedd92a3bc2733605b3888fae8c (patch) | |
tree | 43e818bc2149d1f4bceb65845ad703bfcccd941d /drivers/video | |
parent | backlight: lm3639: Remove unnecessary return statements (diff) | |
download | linux-b50ffb8a5f7f7dedd92a3bc2733605b3888fae8c.tar.xz linux-b50ffb8a5f7f7dedd92a3bc2733605b3888fae8c.zip |
backlight: lms501kf03: Remove 'else' after a return
Fixed the following checkpatch warning.
WARNING: else is not generally useful after a break or return
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Lee Jones <lee.jones@linaro.org>
Diffstat (limited to 'drivers/video')
-rw-r--r-- | drivers/video/backlight/lms501kf03.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/video/backlight/lms501kf03.c b/drivers/video/backlight/lms501kf03.c index 77258b7b04be..7e3810308c3e 100644 --- a/drivers/video/backlight/lms501kf03.c +++ b/drivers/video/backlight/lms501kf03.c @@ -232,19 +232,19 @@ static int lms501kf03_power_on(struct lms501kf03 *lcd) if (!pd->power_on) { dev_err(lcd->dev, "power_on is NULL.\n"); return -EINVAL; - } else { - pd->power_on(lcd->ld, 1); - msleep(pd->power_on_delay); } + pd->power_on(lcd->ld, 1); + msleep(pd->power_on_delay); + if (!pd->reset) { dev_err(lcd->dev, "reset is NULL.\n"); return -EINVAL; - } else { - pd->reset(lcd->ld); - msleep(pd->reset_delay); } + pd->reset(lcd->ld); + msleep(pd->reset_delay); + ret = lms501kf03_ldi_init(lcd); if (ret) { dev_err(lcd->dev, "failed to initialize ldi.\n"); |