On Mon, 2009-03-16 at 19:36 -0700, Arjan van de Ven wrote:
> >From 8ad1bd63c097f9f6948439c1ce7c0b17b8caa64a Mon Sep 17 00:00:00 2001
> From: Arjan van de Ven <[email protected]>
> Date: Mon, 16 Mar 2009 19:31:39 -0700
> Subject: [PATCH] drm: Cache the EDID value for a short time in i915
> 
> during the boot process we have several places that want to make sure we have
> EDID information in a short time. An EDID probe for me takes 0.23 seconds, so
> doing multiple of them is not very nice.
> 
> This patch caches the EDID result for upto 1 second to avoid repeated
> delays.
> 
> Signed-off-by: Arjan van de Ven <[email protected]>

I'm holding off on this one.  I'm applying the LVDS caching (makes
complete sense) but I think someone needs to take a serious look at the
requests for modes on outputs by the kernel and be sure that we've
separated "I want you go to reprobe and then tell me the current data"
from "tell me what the current set of modes is."

Once that's done, I expect this patch goes away.

-- 
Eric Anholt
[email protected]                         [email protected]


Attachment: signature.asc
Description: This is a digitally signed message part

------------------------------------------------------------------------------
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to