[Intel-gfx] [PATCH] Revert "drm/i915: Check live status before reading edid"
Jani Nikula
jani.nikula at linux.intel.com
Wed Aug 17 13:43:36 UTC 2016
On Wed, 17 Aug 2016, Chris Wilson <chris at chris-wilson.co.uk> 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 at linux.intel.com>
>
> 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 at chris-wilson.co.uk>
> Cc: stable at vger.kernel.org
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
More information about the Intel-gfx
mailing list