Yes we had the issue of incorrect edid read. But as of now you can go ahead with the revert. I have moved to a different group, so I am not working on this anymore. Regards, Sonika -----Original Message----- From: Chris Wilson [mailto:chris@xxxxxxxxxxxxxxxxxx] Sent: Tuesday, August 2, 2016 8:02 PM To: Daniel Vetter <daniel@xxxxxxxx>; Jindal, Sonika <sonika.jindal@xxxxxxxxx>; intel-gfx@xxxxxxxxxxxxxxxxxxxxx Subject: Re: [PATCH 4/5] drm/i915: Check live status before reading edid On Tue, Jul 12, 2016 at 02:39:47PM +0300, David Weinehall wrote: > I'm feeling sorely tempted to treat this as a proper regression, and > simply submit a revert patch, seeing as it slows down resume by > something like 200ms, but seeing as there was mention of a case where > incorrect EDID-information might end up being read after resume on > some Chromebooks, I'll just try to open a discussion instead. Wrt https://bugs.freedesktop.org/show_bug.cgi?id=97139 this is a regression and I'll ack the revert. Yes, we need to resolve exactly how we get a stall between intel_hdmi_detect() and pageflip/cursor, but the onus is on the submitter of the patch to fix existing issues first to prevent such regressions (if possible, or else mitigate them). -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
