[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