On 06/29/2012 06:32 PM, Daniel Vetter wrote: > @@ -7270,6 +7270,8 @@ void intel_modeset_cleanup(struct drm_device *dev) > > intel_disable_gt_powersave(dev); > > + ironlake_teardown_rc6(dev); > + Couldn't we pass a true/false parameter to the intel_disable_gt_powersave for tearing down ironlake RC6 directly instead? Something like intel_disable_gt_powersave(struct drm_device *dev, bool teardown_context); maybe? Apart from that, Reviewed-by: Eugeni Dodonov <eugeni.dodonov at intel.com> Eugeni
