summaryrefslogtreecommitdiffstats
path: root/drivers/video
diff options
context:
space:
mode:
authorJingoo Han <jg1.han@samsung.com>2014-08-27 03:12:37 +0200
committerLee Jones <lee.jones@linaro.org>2014-08-29 09:25:33 +0200
commitb50ffb8a5f7f7dedd92a3bc2733605b3888fae8c (patch)
tree43e818bc2149d1f4bceb65845ad703bfcccd941d /drivers/video
parentbacklight: lm3639: Remove unnecessary return statements (diff)
downloadlinux-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.c12
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");