diff options
| author |   Jingoo Han <jg1.han@samsung.com> | 2014-08-27 10:13:30 +0900 |
|---|---|---|
| committer |   Lee Jones <lee.jones@linaro.org> | 2014-08-29 08:26:02 +0100 |
| commit | 3bf3b79fab16b97b0a5a1232af760e8d7a2d3a78 (patch) | |
| tree | 9668e85eb68f17caf8eedb160f635899ec41e691 /drivers/video/backlight | |
| parent | backlight: pcf50633: Add blank line after declarations (diff) | |
| download | linux-dev-3bf3b79fab16b97b0a5a1232af760e8d7a2d3a78.tar.xz linux-dev-3bf3b79fab16b97b0a5a1232af760e8d7a2d3a78.zip | |
backlight: s6e63m0: 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/backlight')
| -rw-r--r-- | drivers/video/backlight/s6e63m0.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/video/backlight/s6e63m0.c b/drivers/video/backlight/s6e63m0.c index f3a65c8940ed..28bfa127fee4 100644 --- a/drivers/video/backlight/s6e63m0.c +++ b/drivers/video/backlight/s6e63m0.c @@ -507,19 +507,19 @@ static int s6e63m0_power_on(struct s6e63m0 *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 = s6e63m0_ldi_init(lcd); if (ret) { dev_err(lcd->dev, "failed to initialize ldi.\n"); |