On Mon, 2 Mar 2015 22:03:59 +0100 Pavel Machek wrote:
> Hi!
>
> > + status_changed =
> > + (twl->linkstat == OMAP_MUSB_VBUS_VALID ||
> > +twl->linkstat == OMAP_MUSB_ID_GROUND)
> > + !=
> > + (status == OMAP_MUSB_
Hi!
> + status_changed =
> + (twl->linkstat == OMAP_MUSB_VBUS_VALID ||
> + twl->linkstat == OMAP_MUSB_ID_GROUND)
> + !=
> + (status == OMAP_MUSB_VBUS_VALID ||
> + status == OMAP_MUSB_I
* NeilBrown [150223 19:45]:
> A construct like:
>
> if (pm_runtime_suspended(twl->dev))
>pm_runtime_get_sync(twl->dev);
>
> is against the spirit of the runtime_pm interface as it
> makes the internal refcounting useless.
>
> In this case it is also racy, particularly as
A construct like:
if (pm_runtime_suspended(twl->dev))
pm_runtime_get_sync(twl->dev);
is against the spirit of the runtime_pm interface as it
makes the internal refcounting useless.
In this case it is also racy, particularly as 'put_autosuspend'
is use to drop a reference.
4 matches
Mail list logo