Re: [PATCH 3/4] drm/i915: enable VT switchless resume v3

Daniel Vetter
2014-05-20T20:15:45Z

On Tue, May 20, 2014 at 9:58 PM, Jesse Barnes <jbarnes@virtuousgeek.org> wrote:
> Ah, poll_enable is false until after _thaw finishes, so
> our hotplug_resume call of hpd_irq_event does nothing. So aside from
> the encoder hot_plug callbacks (which really just check dp link status,
> which ought to be a no-op) our resume_hotplug function doesn't do
> anything right now. May as well kill it and just send an unconditional
> uevent?

That's expensive since the reprobe will likely result in a few dropped
frames (if the edid reading takes a long time). Better to do that in
the kernel and avoid sending the uevent if nothing changed. Which iirc
hpd_irq_event does for us.
-Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch