On 29 June 2017 at 10:09, Arvind Yadav <arvind.yadav.cs@xxxxxxxxx> wrote: > dev_pm_ops are not supposed to change at runtime. All functions > working with dev_pm_ops provided by <linux/device.h> 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 <arvind.yadav.cs@xxxxxxxxx> Thanks, applied for next! Kind regards Uffe > --- > drivers/mmc/host/omap_hsmmc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c > index 8c39dcc..5b391dc 100644 > --- a/drivers/mmc/host/omap_hsmmc.c > +++ b/drivers/mmc/host/omap_hsmmc.c > @@ -2342,7 +2342,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, > -- > 1.9.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html