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
