On Fri, Apr 13, 2012 at 18:03, Chris Wilson <chris at chris-wilson.co.uk>wrote: > On Fri, 13 Apr 2012 17:08:53 -0300, Eugeni Dodonov < > eugeni.dodonov at intel.com> wrote: > > void intel_modeset_init(struct drm_device *dev) > > { > > struct drm_i915_private *dev_priv = dev->dev_private; > > @@ -9652,6 +9680,9 @@ void intel_modeset_init(struct drm_device *dev) > > > > intel_init_quirks(dev); > > > > + if (IS_HASWELL(dev)) > > + intel_init_power_wells(dev); > > + > > Perform the gen checks in intel_init_power_wells() so that we keep the > initialisation sequence readable. > Yes, makes total sense, I'll do this. Thanks! -- Eugeni Dodonov <http://eugeni.dodonov.net/> -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20120413/f580dfee/attachment.html>