On Wed, 17 Aug 2016, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote:
> On Wed, Aug 17, 2016 at 03:47:48PM +0300, David Weinehall wrote:
>> This reverts commit 237ed86c693d8a8e4db476976aeb30df4deac74b.
>>
>> Our current implementation of live status check (repeat 9 times
>> with 10ms delays between each attempt as a workaround for
>> buggy displays) imposes a rather serious penalty, time wise,
>> on intel_hdmi_detect(). Since we we already skip live status
>> checks on platforms before gen 7, and since we seem to have
>> coped quite well before the live status check was introduced
>> for newer platforms too, the previous behaviour is probably
>> preferable, at least unless someone can point to a use-case
>> that the live status check improves (apart from "Bspec says so".
>>
>> Signed-off-by: David Weinehall <david.weinehall@xxxxxxxxxxxxxxx>
>
> Fixes: 237ed86c693d ("drm/i915: Check live status before reading edid")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97139
> Acked-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
> Cc: stable@xxxxxxxxxxxxxxx
And we've come full circle on live status. Again.
References: https://upload.wikimedia.org/wikipedia/commons/3/3b/Paris_Tuileries_Garden_Facepalm_statue.jpg
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
