mmc: omap_hsmmc: constify dev_pm_ops structures

ArvindYadavCs · storulf · commit 6bba406445e6 · 2017-08-30T14:01:20.000+02:00
dev_pm_ops are not supposed to change at runtime. All functions
working with dev_pm_ops provided by &lt;linux/device.h&gt; work with const
dev_pm_ops. So mark the non-const structs as const.

File size before:
   text	   data	    bss	    dec	    hex	filename
  11586	    624	      0	  12210	   2fb2	drivers/mmc/host/omap_hsmmc.o

File size After adding 'const':
   text	   data	    bss	    dec	    hex	filename
  11778	    432	      0	  12210	   2fb2	drivers/mmc/host/omap_hsmmc.o

Signed-off-by: Arvind Yadav &lt;arvind.yadav.cs@gmail.com&gt;
Signed-off-by: Ulf Hansson &lt;ulf.hansson@linaro.org&gt;
diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
@@ -2322,7 +2322,7 @@ static int omap_hsmmc_runtime_resume(struct device *dev)
 	return 0;
 }
 
-static struct dev_pm_ops omap_hsmmc_dev_pm_ops = {
+static const struct dev_pm_ops omap_hsmmc_dev_pm_ops = {
 	SET_SYSTEM_SLEEP_PM_OPS(omap_hsmmc_suspend, omap_hsmmc_resume)
 	.runtime_suspend = omap_hsmmc_runtime_suspend,
 	.runtime_resume = omap_hsmmc_runtime_resume,

Original file line number	Diff line number	Diff line change
`@@ -2322,7 +2322,7 @@ static int omap_hsmmc_runtime_resume(struct device *dev)`
`2322`	`2322`	`return 0;`
`2323`	`2323`	`}`
`2324`	`2324`
`2325`		`-static struct dev_pm_ops omap_hsmmc_dev_pm_ops = {`
	`2325`	`+static const struct dev_pm_ops omap_hsmmc_dev_pm_ops = {`
`2326`	`2326`	`SET_SYSTEM_SLEEP_PM_OPS(omap_hsmmc_suspend, omap_hsmmc_resume)`
`2327`	`2327`	`.runtime_suspend = omap_hsmmc_runtime_suspend,`
`2328`	`2328`	`.runtime_resume = omap_hsmmc_runtime_resume,`