On Fri, Apr 13, 2012 at 18:03, Chris Wilson <chris@chris-wilson.co.uk> wrote:
On Fri, 13 Apr 2012 17:08:53 -0300, Eugeni Dodonov <eugeni.dodonov@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