On Tue, Jul 18, 2023 at 12:47 PM AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx> wrote: > Il 17/07/23 19:28, Andy Shevchenko ha scritto: ... > > -static int mtk_paris_pinctrl_suspend(struct device *device) > > +static int mtk_paris_suspend(struct device *device) > > -static int mtk_paris_pinctrl_resume(struct device *device) > > +static int mtk_paris_resume(struct device *device) > > What's the reason why you changed the suspend/resume function names? > I don't really mind, but please at least mention that in the commit description. To put it on a single line. I will amend the commit message, thank you for review! ... > > +DEFINE_NOIRQ_DEV_PM_OPS(mtk_paris_pinctrl_pm_ops, mtk_paris_suspend, mtk_paris_resume); ...here ^^^ -- With Best Regards, Andy Shevchenko