serial: 8250_bcm2835aux: Switch to DEFINE_SIMPLE_DEV_PM_OPS()

andy-shev · gregkh · commit 98e24a58b597 · 2024-08-14T08:22:50.000+02:00
The definition of the PM operations opens code the existing macro, replace it with the DEFINE_SIMPLE_DEV_PM_OPS() for setting the driver's PM routines. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Link: https://lore.kernel.org/r/20240813081954.1408792-1-andriy.shevchenko@linux.intel.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
diff --git a/drivers/tty/serial/8250/8250_bcm2835aux.c b/drivers/tty/serial/8250/8250_bcm2835aux.c
@@ -245,9 +245,7 @@ static int bcm2835aux_resume(struct device *dev)
 	return 0;
 }
 
-static const struct dev_pm_ops bcm2835aux_dev_pm_ops = {
-	SYSTEM_SLEEP_PM_OPS(bcm2835aux_suspend, bcm2835aux_resume)
-};
+static DEFINE_SIMPLE_DEV_PM_OPS(bcm2835aux_dev_pm_ops, bcm2835aux_suspend, bcm2835aux_resume);
 
 static struct platform_driver bcm2835aux_serial_driver = {
 	.driver = {

Original file line number	Diff line number	Diff line change
`@@ -245,9 +245,7 @@ static int bcm2835aux_resume(struct device *dev)`
`245`	`245`	`return 0;`
`246`	`246`	`}`
`247`	`247`
`248`		`-static const struct dev_pm_ops bcm2835aux_dev_pm_ops = {`
`249`		`- SYSTEM_SLEEP_PM_OPS(bcm2835aux_suspend, bcm2835aux_resume)`
`250`		`-};`
	`248`	`+static DEFINE_SIMPLE_DEV_PM_OPS(bcm2835aux_dev_pm_ops, bcm2835aux_suspend, bcm2835aux_resume);`
`251`	`249`
`252`	`250`	`static struct platform_driver bcm2835aux_serial_driver = {`
`253`	`251`	`.driver = {`