On Wed, 01 Apr 2009 12:38:17 -0700
Eric Anholt <[email protected]> wrote:

> 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.

for me the caching patch obsoletes this one, so absolutely fine by me.

> 


-- 
Arjan van de Ven        Intel Open Source Technology Centre
For development, discussion and tips for power savings, 
visit http://www.lesswatts.org

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

Reply via email to