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. -Chris -- Chris Wilson, Intel Open Source Technology Centre