<div class="gmail_quote">On Fri, Apr 13, 2012 at 18:03, Chris Wilson <span dir="ltr"><<a href="mailto:chris@chris-wilson.co.uk">chris@chris-wilson.co.uk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">On Fri, 13 Apr 2012 17:08:53 -0300, Eugeni Dodonov <<a href="mailto:eugeni.dodonov@intel.com">eugeni.dodonov@intel.com</a>> wrote:<br>
>  void intel_modeset_init(struct drm_device *dev)<br>
>  {<br>
>       struct drm_i915_private *dev_priv = dev->dev_private;<br>
> @@ -9652,6 +9680,9 @@ void intel_modeset_init(struct drm_device *dev)<br>
><br>
>       intel_init_quirks(dev);<br>
><br>
> +     if (IS_HASWELL(dev))<br>
> +             intel_init_power_wells(dev);<br>
> +<br>
<br>
</div>Perform the gen checks in intel_init_power_wells() so that we keep the<br>
initialisation sequence readable.<br></blockquote><div><br></div><div>Yes, makes total sense, I'll do this.</div><div><br></div><div>Thanks!</div><div><br></div></div>-- <br>Eugeni Dodonov<a href="http://eugeni.dodonov.net/" target="_blank"><br>

</a><br>