Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c9023492 authored by Jingoo Han's avatar Jingoo Han Committed by Linus Torvalds
Browse files

backlight: ams369fg06: 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: 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 1d7976b2

drivers/video/backlight/ams369fg06.c

+3 −21
Original line number Diff line number Diff line
@@ -545,44 +545,26 @@ static int ams369fg06_remove(struct spi_device *spi) 
}



#if defined(CONFIG_PM)

static unsigned int before_power;



static int ams369fg06_suspend(struct spi_device *spi, pm_message_t mesg)

{

	int ret = 0;

	struct ams369fg06 *lcd = dev_get_drvdata(&spi->dev);



	dev_dbg(&spi->dev, "lcd->power = %d\n", lcd->power);



	before_power = lcd->power;



	/*

	 * when lcd panel is suspend, lcd panel becomes off

	 * regardless of status.

	 */

	ret = ams369fg06_power(lcd, FB_BLANK_POWERDOWN);



	return ret;

	return ams369fg06_power(lcd, FB_BLANK_POWERDOWN);

}



static int ams369fg06_resume(struct spi_device *spi)

{

	int ret = 0;

	struct ams369fg06 *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;



	dev_dbg(&spi->dev, "before_power = %d\n", before_power);



	ret = ams369fg06_power(lcd, before_power);



	return ret;

	return ams369fg06_power(lcd, FB_BLANK_UNBLANK);

}

#else

#define ams369fg06_suspend	NULL