[lkml]   [2011]   [Aug]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 5/5 v4] mfd: omap: usb: Runtime PM support
    On Tue, Aug 30, 2011 at 2:17 AM, Kevin Hilman <> wrote:
    > Kevin Hilman <> writes:
    >> Keshava Munegowda <> writes:
    >>> From: Keshava Munegowda <>
    >>> 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 <>
    >> 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 <>
    > 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]

    Thanks for review; yes, its reworks of complete runtime pm of usbhs as
    suggested by Jassi Brar.

    balbi, sameo
    this patch series available at

    the branch kmg-usbhs-pm

    of code repository : git://

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2011-08-30 08:19    [W:0.023 / U:9.288 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site