On Tue, Jul 18, 2023 at 10:53:22AM +0100, Jonathan Cameron wrote: > On Mon, 17 Jul 2023 20:28:13 +0300 > Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > > > Cleaning up the driver to use pm_ptr() and *_PM_OPS() macros that > > make it simpler and allows the compiler to remove those functions > > if built without CONFIG_PM and CONFIG_PM_SLEEP support. > > > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > > LGTM but why is it in a set that claims to be about NOIRQ PM helper? Have you chance to read a cover letter? I explained that I added them as they are "kinda" to the topic (PM related). I can apply them and drop for next version. Thank you for the review! -- With Best Regards, Andy Shevchenko