Re: [PATCH v6 03/15] drm/rockchip: vop2: Disable core clocks before dropping runtime PM reference
From: Daniel Stone
Date: Thu Nov 05 2026 - 07:03:30 EST
On Thu, 5 Nov 2026 at 01:01, Cristian Ciocaltea
<cristian.ciocaltea@xxxxxxxxxxxxx> wrote:
> vop2_enable() acquires the runtime PM reference before enabling the core
> clocks, and its error path releases them in the reverse order. However,
> vop2_disable() drops the runtime PM reference first, potentially
> powering off the VOP power domain while the core clocks are still
> running.
>
> Make the teardown mirror the bring-up sequence by disabling the core
> clocks before calling pm_runtime_put_sync().
Reviewed-by: Daniel Stone <daniels@xxxxxxxxxxxxx>