backlight: s6e63m0: remove redundant variable 'before_power'
'before_power' was used to check the previous status when resume() is called. However, FB_BLANK_POWERDOWN was used in suspend() all the time, so there is no need to check the previous status. Also, redundant return variables are removed to reduce the code. 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>
This commit is contained in:
parent
81c650e3ac
commit
b6bef463f5
1 changed files with 3 additions and 21 deletions
|
@ -827,44 +827,26 @@ static int s6e63m0_remove(struct spi_device *spi)
|
||||||
}
|
}
|
||||||
|
|
||||||
#if defined(CONFIG_PM)
|
#if defined(CONFIG_PM)
|
||||||
static unsigned int before_power;
|
|
||||||
|
|
||||||
static int s6e63m0_suspend(struct spi_device *spi, pm_message_t mesg)
|
static int s6e63m0_suspend(struct spi_device *spi, pm_message_t mesg)
|
||||||
{
|
{
|
||||||
int ret = 0;
|
|
||||||
struct s6e63m0 *lcd = dev_get_drvdata(&spi->dev);
|
struct s6e63m0 *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);
|
||||||
|
|
||||||
before_power = lcd->power;
|
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* 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 = s6e63m0_power(lcd, FB_BLANK_POWERDOWN);
|
return s6e63m0_power(lcd, FB_BLANK_POWERDOWN);
|
||||||
|
|
||||||
return ret;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int s6e63m0_resume(struct spi_device *spi)
|
static int s6e63m0_resume(struct spi_device *spi)
|
||||||
{
|
{
|
||||||
int ret = 0;
|
|
||||||
struct s6e63m0 *lcd = dev_get_drvdata(&spi->dev);
|
struct s6e63m0 *lcd = dev_get_drvdata(&spi->dev);
|
||||||
|
|
||||||
/*
|
|
||||||
* after suspended, if lcd panel status is FB_BLANK_UNBLANK
|
|
||||||
* (at that time, before_power is FB_BLANK_UNBLANK) then
|
|
||||||
* it changes that status to FB_BLANK_POWERDOWN to get lcd on.
|
|
||||||
*/
|
|
||||||
if (before_power == FB_BLANK_UNBLANK)
|
|
||||||
lcd->power = FB_BLANK_POWERDOWN;
|
lcd->power = FB_BLANK_POWERDOWN;
|
||||||
|
|
||||||
dev_dbg(&spi->dev, "before_power = %d\n", before_power);
|
return s6e63m0_power(lcd, FB_BLANK_UNBLANK);
|
||||||
|
|
||||||
ret = s6e63m0_power(lcd, before_power);
|
|
||||||
|
|
||||||
return ret;
|
|
||||||
}
|
}
|
||||||
#else
|
#else
|
||||||
#define s6e63m0_suspend NULL
|
#define s6e63m0_suspend NULL
|
||||||
|
|
Loading…
Reference in a new issue