Commit 2ca8b90a authored by Jingoo Han's avatar Jingoo Han Committed by Linus Torvalds

backlight: ld9040: remove redundant return variables

Redundant return variables are removed to reduce the code.
Signed-off-by: default avatarJingoo Han <jg1.han@samsung.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 30f085ca
...@@ -513,14 +513,9 @@ static int _ld9040_gamma_ctl(struct ld9040 *lcd, const unsigned int *gamma) ...@@ -513,14 +513,9 @@ static int _ld9040_gamma_ctl(struct ld9040 *lcd, const unsigned int *gamma)
static int ld9040_gamma_ctl(struct ld9040 *lcd, int gamma) static int ld9040_gamma_ctl(struct ld9040 *lcd, int gamma)
{ {
int ret = 0; return _ld9040_gamma_ctl(lcd, gamma_table.gamma_22_table[gamma]);
ret = _ld9040_gamma_ctl(lcd, gamma_table.gamma_22_table[gamma]);
return ret;
} }
static int ld9040_ldi_init(struct ld9040 *lcd) static int ld9040_ldi_init(struct ld9040 *lcd)
{ {
int ret, i; int ret, i;
...@@ -549,11 +544,7 @@ static int ld9040_ldi_init(struct ld9040 *lcd) ...@@ -549,11 +544,7 @@ static int ld9040_ldi_init(struct ld9040 *lcd)
static int ld9040_ldi_enable(struct ld9040 *lcd) static int ld9040_ldi_enable(struct ld9040 *lcd)
{ {
int ret = 0; return ld9040_panel_send_sequence(lcd, seq_display_on);
ret = ld9040_panel_send_sequence(lcd, seq_display_on);
return ret;
} }
static int ld9040_ldi_disable(struct ld9040 *lcd) static int ld9040_ldi_disable(struct ld9040 *lcd)
...@@ -696,7 +687,6 @@ static const struct backlight_ops ld9040_backlight_ops = { ...@@ -696,7 +687,6 @@ static const struct backlight_ops ld9040_backlight_ops = {
.update_status = ld9040_set_brightness, .update_status = ld9040_set_brightness,
}; };
static int ld9040_probe(struct spi_device *spi) static int ld9040_probe(struct spi_device *spi)
{ {
int ret = 0; int ret = 0;
...@@ -802,7 +792,6 @@ static int ld9040_remove(struct spi_device *spi) ...@@ -802,7 +792,6 @@ static int ld9040_remove(struct spi_device *spi)
#if defined(CONFIG_PM) #if defined(CONFIG_PM)
static int ld9040_suspend(struct spi_device *spi, pm_message_t mesg) static int ld9040_suspend(struct spi_device *spi, pm_message_t mesg)
{ {
int ret = 0;
struct ld9040 *lcd = dev_get_drvdata(&spi->dev); struct ld9040 *lcd = dev_get_drvdata(&spi->dev);
dev_dbg(&spi->dev, "lcd->power = %d\n", lcd->power); dev_dbg(&spi->dev, "lcd->power = %d\n", lcd->power);
...@@ -811,21 +800,16 @@ static int ld9040_suspend(struct spi_device *spi, pm_message_t mesg) ...@@ -811,21 +800,16 @@ static int ld9040_suspend(struct spi_device *spi, pm_message_t mesg)
* when lcd panel is suspend, lcd panel becomes off * when lcd panel is suspend, lcd panel becomes off
* regardless of status. * regardless of status.
*/ */
ret = ld9040_power(lcd, FB_BLANK_POWERDOWN); return ld9040_power(lcd, FB_BLANK_POWERDOWN);
return ret;
} }
static int ld9040_resume(struct spi_device *spi) static int ld9040_resume(struct spi_device *spi)
{ {
int ret = 0;
struct ld9040 *lcd = dev_get_drvdata(&spi->dev); struct ld9040 *lcd = dev_get_drvdata(&spi->dev);
lcd->power = FB_BLANK_POWERDOWN; lcd->power = FB_BLANK_POWERDOWN;
ret = ld9040_power(lcd, FB_BLANK_UNBLANK); return ld9040_power(lcd, FB_BLANK_UNBLANK);
return ret;
} }
#else #else
#define ld9040_suspend NULL #define ld9040_suspend NULL
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment