Re: [PATCH v2] i2c: xiic: restore runtime PM teardown in remove to fix clk WARN flood

From: Andi Shyti

Date: Tue Aug 18 2026 - 06:18:37 EST


Hi Abdurraham,

On Tue, Aug 18, 2026 at 12:44:27AM -0700, Abdurrahman Hussain wrote:
> On Mon Aug 17, 2026 at 12:20 PM PDT, Andi Shyti wrote:
> > A solution would be to drop the managed runtime-PM helper and
> > restore the non managed runtime PM setup and teardown. Otherwise,
> > this ordering issue needs to be addressed in the runtime-PM/clock
> > framework.
> >
> > Does that work?
> >
> > Andi
>
> Agreed — that's cleaner than special-casing the ordering. v3 drops
> devm_pm_runtime_set_active_enabled() and restores the non-managed
> setup/teardown.
>
> Let me try it on hardware before sending v3.

Please, also write a nice comment explaining why we won't use the
non managed runtime PM, I don't want to receive in the future a
patch adding the managed runtime PM.

Thanks,
Andi