On Tue, May 20, 2014 at 9:58 PM, Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> 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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
