From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915: cache the EDID for eDP panels Date: Wed, 6 Jun 2012 09:23:18 +0200 Message-ID: <20120606072318.GA4699@phenom.ffwll.local> References: <1338929667-6814-1-git-send-email-jbarnes@virtuousgeek.org> <1338930190_513452@CP5-2952> <20120605142611.02f8eafb@jbarnes-desktop> <1338932330_514046@CP5-2952> <20120605144115.31ca3da7@jbarnes-desktop> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wi0-f171.google.com (mail-wi0-f171.google.com [209.85.212.171]) by gabe.freedesktop.org (Postfix) with ESMTP id 27CC5A0983 for ; Wed, 6 Jun 2012 00:21:53 -0700 (PDT) Received: by wibhm14 with SMTP id hm14so3490245wib.12 for ; Wed, 06 Jun 2012 00:21:53 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20120605144115.31ca3da7@jbarnes-desktop> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Jesse Barnes Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Jun 05, 2012 at 02:41:15PM -0700, Jesse Barnes wrote: > On Tue, 05 Jun 2012 22:38:02 +0100 > Chris Wilson wrote: > > > On Tue, 5 Jun 2012 14:26:11 -0700, Jesse Barnes wrote: > > > Yeah that makes sense. Not sure how much this conflicts with Daniel's > > > hot plug rework either; there's likely some overlap. > > > > We could have made that Daniel's problem.... ;) > > http://lists.freedesktop.org/archives/intel-gfx/2011-April/010401.html > > Hey look I even reviewed it. > > Pick up the pieces, Daniel! :) I agree looks nice. Can I ask for a resend? Afaics all the stuff got acks/r-bs safe for patch 6 (maybe just drop that one). And I agree with Jesse's bikeshed on patch 8, is_panel sounds more like what we want. Dunno about patch 10, killing that pointless flicker would certainly be nice. I guess we could just merge it and see what happens, then claim innonce when getting caught with our hands in the cookie jar ;-) Oh, and I don't think there's much overlap with the hpd stuff of mine, that only caches edids at ->detect time and relies on core drm changes to make calls to ->detect happen less often. -Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48