From: Eric A. <er...@an...> - 2009-04-01 19:29:23
|
On Fri, 2009-03-27 at 22:01 -0700, Arjan van de Ven wrote: > On Fri, 27 Mar 2009 16:08:18 -0700 > Jesse Barnes <jb...@vi...> wrote: > > > > > > return ret; > > > > I definitely like the idea of caching the LVDS EDID, but rather than > > adding it to the intel_output struct I'd rather see it stored with the > > rest of the LVDS stuff in i915_private (though that stuff should be > > pulled out into an output private at some point). We could also just > > open code the intel_ddc_get_modes stuff in intel_lvds_init, getting > > the EDID just once, storing the property once, and adding the mode > > list once. intel_lvds_get_modes would need to be fixed too... > > > > that gets a bit messy though, esp if other output types decide to also > want to cache. Remember that some of this code gets called from DRM or > even userland, so the edid will end up being stored in a generic > structure.... Ack. SDVO LVDS and eDP would also want to do this, and i915_private then seems like the wrong place. -- Eric Anholt er...@an... eri...@in... |