drm/i915: make enable/disable_gt_powersave locking consistent
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Sun, 24 Jun 2012 14:42:33 +0000 (16:42 +0200)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Mon, 25 Jun 2012 19:07:29 +0000 (21:07 +0200)
commit79f5b2c7599270aa3dcfffb445f8f520c05a7fc5
tree7136bcd85119075fc624f8a6d55ddb6ac2dc870e
parent8090c6b9daa04dda649ac0a2209601042abfb0a4
drm/i915: make enable/disable_gt_powersave locking consistent

The enable functions grabbed dev->struct_mutex themselves, whereas
the disable functions expected dev->struct_mutex to be held by the
caller. Move the locking out to the (currently only) callsite of
intel_enable_gt_powersave to make this more consistent.

Originally this was prep work for future patches, but I've chased down
a totally wrong alley. Still, I think this is a sensible
clarification.

Reviewed-by: Eugeni Dodonov <eugeni.dodonov@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c
drivers/gpu/drm/i915/intel_pm.c