On Thu, Jul 28, 2016 at 04:47:07PM +0100, Chris Wilson wrote: > On Thu, Jul 28, 2016 at 05:50:37PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > Currently we try to check the lid status to see whether eDP is connected > > or not. However we ignore the result anyway, and we have no lid notifier > > to fire off uevents, so let's just ignore the lid stuff for eDP > > entirely. > > You mean the OpRegion no longer contains lid_state? I don't see it being > disabled... The lid notifier is from acpi/button.c. I haven't looked at > why the result would be ignored, but you haven't convinced me why eDP is > special. We don't register a lid notifier for eDP. So if the initial detect would happen with lid closed we'd mark eDP as disconnected, and when the lid gets opened we won't even notice, and thus the connector never becomes connected. IIRC on my ILK the lid state in OpRegion didn't work anyway. But I should probably double check. -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
