Messages in this thread | | | From | Kevin Hilman <> | Subject | Re: [PATCH 5/5 v4] mfd: omap: usb: Runtime PM support | Date | Mon, 29 Aug 2011 13:47:39 -0700 |
| |
Kevin Hilman <khilman@ti.com> writes:
> Keshava Munegowda <keshava_mgowda@ti.com> writes: > >> From: Keshava Munegowda <Keshava_mgowda@ti.com> >> >> The usbhs core driver does not enable/disable the intefrace and >> fucntional clocks; These clocks are handled by hwmod and runtime pm, >> hence insted of the clock enable/disable, the runtime pm APIS are >> used. however,the port clocks are handled by the usbhs core. >> >> Signed-off-by: Keshava Munegowda <keshava_mgowda@ti.com> > > General comment: all usage of pm_runtime_put_sync() can likely be > replaced by the asynchronous versions. I don't currently see why the > synchronous verions are needed here. > > Other than that, the runtime PM parts of this look OK to me. After > changing to asynchonous puts, feel free to add: > > Reviewed-by: Kevin Hilman <khilman@ti.com>
oops, this should've been in response to your v8 version.
Kevin
> > Also, after a quick glance, it looks like this version of the series > addresses the problems seen by Jassi Brar with the TLL reset[1]. Please > confirm. > > Speaking of which, it's helpful to Cc folks who have had comments on > previous versions of your series so they are sure they're previous > issues are addressed. I've Cc'd Jassi Brar. > > Thanks, > > Kevin > > [1] http://marc.info/?l=linux-omap&m=130921260703865&w=2
| |