Commit d6b81be1 authored by Uwe Kleine-König's avatar Uwe Kleine-König Committed by Thierry Reding
Browse files

pwm: brcmstb: Use DEFINE_SIMPLE_DEV_PM_OPS for PM functions 



This macro has the advantage over SIMPLE_DEV_PM_OPS that we don't have to
care about when the functions are actually used, so the corresponding
#ifdef can be dropped.

Also make use of pm_ptr() to discard all PM related stuff if CONFIG_PM
isn't enabled.

Acked-by: default avatarFlorian Fainelli <florian.fainelli@broadcom.com>
Reviewed-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
Signed-off-by: default avatarUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: default avatarThierry Reding <thierry.reding@gmail.com>
parent fac37751

drivers/pwm/pwm-brcmstb.c

+3 −5
Original line number Diff line number Diff line
@@ -259,7 +259,6 @@ static int brcmstb_pwm_probe(struct platform_device *pdev) 
	return 0;

}



#ifdef CONFIG_PM_SLEEP

static int brcmstb_pwm_suspend(struct device *dev)

{

	struct brcmstb_pwm *p = dev_get_drvdata(dev);
@@ -275,9 +274,8 @@ static int brcmstb_pwm_resume(struct device *dev) 


	return clk_prepare_enable(p->clk);

}

#endif



static SIMPLE_DEV_PM_OPS(brcmstb_pwm_pm_ops, brcmstb_pwm_suspend,

static DEFINE_SIMPLE_DEV_PM_OPS(brcmstb_pwm_pm_ops, brcmstb_pwm_suspend,

				brcmstb_pwm_resume);



static struct platform_driver brcmstb_pwm_driver = {
@@ -285,7 +283,7 @@ static struct platform_driver brcmstb_pwm_driver = { 
	.driver = {

		.name = "pwm-brcmstb",

		.of_match_table = brcmstb_pwm_of_match,

		.pm = &brcmstb_pwm_pm_ops,

		.pm = pm_ptr(&brcmstb_pwm_pm_ops),

	},

};

module_platform_driver(brcmstb_pwm_driver);