On Tue, Feb 15, 2011 at 6:12 AM, Kevin Hilman <khilman@xxxxxx> wrote: > Hema Kalliguddi <hemahk@xxxxxx> writes: > > [...] > >>>On Thu, Feb 10, 2011 at 07:38:01PM +0530, Hema HK wrote: >>>> Calling runtime pm APIs pm_runtime_put_sync() and >>>pm_runtime_get_sync() >>>> for enabling/disabling the clocks, sysconfig settings. >>>> >>>> Enable clock, configure no-idle/standby when active and >>>configure force idle/standby >>>> and disable clock when idled. This is taken care by the >>>runtime framework when >>>> driver calls the pm_runtime_get_sync and pm_runtime_put_sync APIs. >>> >>>does it have to be the _sync() ?? >> Yes. Because immediately after this I access the registers. >> > > What about the put? Put can be async. I will send the patch with changes. Regards, Hema > > Kevin > > -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html