[PATCH v6 09/13] gpio: mxc: use cleanup guard for pm_runtime_get_noresume() balance
Andy Shevchenko
andy.shevchenko at gmail.com
Mon Oct 12 03:41:40 PDT 2026
On Mon, Oct 12, 2026 at 1:06 PM Peng Fan <peng.fan at oss.nxp.com> wrote:
> On Mon, Oct 12, 2026 at 12:38:37PM +0300, Andy Shevchenko wrote:
> >On Sun, Oct 11, 2026 at 2:54 PM Peng Fan (OSS) <peng.fan at oss.nxp.com> wrote:
...
> >> +DEFINE_FREE(pm_runtime_put_noidle, struct device *, if (_T) pm_runtime_put_noidle(_T))
> >
> >Please, split this to a separate patch and move it directly to pm_runtime.h.
>
> Frank raised same suggestion in V5.
>
> I tend to avoid introducing cross-subsystem dependency and use a follow-up
> patch to move it to pm_runtime.h
Alternatively you may not use it, just carefully put a goto:s and error labels.
> But since you both vote for pm_runtime.h, let me separate it in V7.
Also think about splitting the series to three. Do not rush! It anyway
missed v7.4 (except, perhaps, the first fixes part).
--
With Best Regards,
Andy Shevchenko
More information about the linux-arm-kernel
mailing list