[Intel-gfx] [PATCH 2/5] drm/i915: make enable/disable_gt_powersave locking consistent

Daniel Vetter daniel at ffwll.ch
Mon Jun 25 21:08:18 CEST 2012


On Sun, Jun 24, 2012 at 08:03:37PM -0300, Eugeni Dodonov wrote:
> On 06/24/2012 11:42 AM, Daniel Vetter wrote:
> > 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.
> > 
> > Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> 
> Reviewed-by: Eugeni Dodonov <eugeni.dodonov at intel.com>

Merged the first two patches of this series to dinq, thanks for the
review.
-Daniel
-- 
Daniel Vetter
Mail: daniel at ffwll.ch
Mobile: +41 (0)79 365 57 48



More information about the Intel-gfx mailing list