On Sat, Dec 12, 2015 at 09:40:45PM +0200, Imre Deak wrote:
> On Sat, 2015-12-12 at 02:51 +0100, Rafael J. Wysocki wrote:
> > +bool pm_runtime_get_if_in_use(struct device *dev)
> > +{
> > +   unsigned long flags;
> > +   bool retval;
> > +
> > +   spin_lock_irqsave(&dev->power.lock, flags);
> > +   retval = dev->power.runtime_status == RPM_ACTIVE ?
> > +           !!atomic_inc_not_zero(&dev->power.usage_count) :
> > false;
> > +   spin_unlock_irqrestore(&dev->power.lock, flags);
> > +   return retval;
> > +}
> > +EXPORT_SYMBOL_GPL(pm_runtime_get_if_in_use);
> > +
> 
> To me this looks ok:
> Acked-by: Imre Deak <imre.d...@intel.com>

Pendant says
retval = (dev->power.runtime_status == RPM_ACTIVE &&
          atomic_inc_not_zero(&dev->power.usage_count);
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to