Re: [PATCH v2 16/46] drm/mediatek: mtk_hdmi_v2: Don't warn on RPM active during detach

[CK Hu (胡俊光)]

On Tue, 2026-07-14 at 13:43 +0200, AngeloGioacchino Del Regno wrote:
> The Runtime PM may be active during bridge .detach() and this
> happens especially at boot, when probe deferrals in the display
> controller driver happen.
> 
> Even though it looks safe to assume that RPM has to be inactive
> at that time, this happens to be true only if the DDC was to be
> handled internally (like before .hpd_{en,dis}able() existed).
> 
> At this point in time, this driver uses HPD callbacks, and for
> HPD to actually work it is necessary to raise the RPM refcount
> to wake up the entire HDMI IP: since it's not granted that the
> HPD or other children of the HDMI node are disabled during the
> display controller probe sequence, the WARN at bridge .detach()
> time may trigger (and will do that when mediatek-drm defers),
> unnecessarily bloating the kernel log, as the RPM refcount will
> be decreased anyway when removing the driver.
> 
> This is why the WARN(pm_runtime_active()) check during detach
> is simply a development stage leftover: remove it!

Reviewed-by: CK Hu <ck.hu@xxxxxxxxxxxx>

> 
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
> ---