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 <[email protected]> > > Reviewed-by: Eugeni Dodonov <[email protected]>
Merged the first two patches of this series to dinq, thanks for the review. -Daniel -- Daniel Vetter Mail: [email protected] Mobile: +41 (0)79 365 57 48 _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
