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
 <http://eugeni.dodonov.net/>