spi: bcm63xx: use macro DEFINE_SIMPLE_DEV_PM_OPS

DhruvaG2000 · broonie · commit cc5f6fa4f659 · 2023-04-25T16:55:45.000+01:00
Using this macro makes the code more readable. It also inits the members of dev_pm_ops in the following manner without us explicitly needing to: .suspend = bcm63xx_spi_suspend, \ .resume = bcm63xx_spi_resume, \ .freeze = bcm63xx_spi_suspend, \ .thaw = bcm63xx_spi_resume, \ .poweroff = bcm63xx_spi_suspend, \ .restore = bcm63xx_spi_resume Signed-off-by: Dhruva Gole <d-gole@ti.com> Link: https://lore.kernel.org/r/20230424102546.1604484-1-d-gole@ti.com Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/drivers/spi/spi-bcm63xx.c b/drivers/spi/spi-bcm63xx.c
@@ -644,9 +644,7 @@ static int bcm63xx_spi_resume(struct device *dev)
 	return 0;
 }
 
-static const struct dev_pm_ops bcm63xx_spi_pm_ops = {
-	SET_SYSTEM_SLEEP_PM_OPS(bcm63xx_spi_suspend, bcm63xx_spi_resume)
-};
+static DEFINE_SIMPLE_DEV_PM_OPS(bcm63xx_spi_pm_ops, bcm63xx_spi_suspend, bcm63xx_spi_resume);
 
 static struct platform_driver bcm63xx_spi_driver = {
 	.driver = {

Original file line number	Diff line number	Diff line change
`@@ -644,9 +644,7 @@ static int bcm63xx_spi_resume(struct device *dev)`
`644`	`644`	`return 0;`
`645`	`645`	`}`
`646`	`646`
`647`		`-static const struct dev_pm_ops bcm63xx_spi_pm_ops = {`
`648`		`- SET_SYSTEM_SLEEP_PM_OPS(bcm63xx_spi_suspend, bcm63xx_spi_resume)`
`649`		`-};`
	`647`	`+static DEFINE_SIMPLE_DEV_PM_OPS(bcm63xx_spi_pm_ops, bcm63xx_spi_suspend, bcm63xx_spi_resume);`
`650`	`648`
`651`	`649`	`static struct platform_driver bcm63xx_spi_driver = {`
`652`	`650`	`.driver = {`