On Fri, 12 Aug 2011, Pandita, Vikram wrote:

> On Fri, Aug 12, 2011 at 8:50 AM, Alan Stern <[email protected]> wrote:
> > On Fri, 12 Aug 2011, Vikram Pandita wrote:
> >
> >> From: Vikram Pandita <[email protected]>
> >>
> >> musb pm_runtime_get_sync call happens in intrrupt context on cable attach 
> >> case
> >> That can result in re-enabling the interrupts and cause side affects.
> >>
> >> So move the code to a work queue.
> >
> > Instead of creating your own, new work queue, just use the pre-existing
> > PM work queue. �In other words, replace pm_runtime_get_sync() with
> > pm_runtime_get().
> 
> In this case it may not work, because immediately after a
> pm_runtime_get_sync(),
> we are going to access MUSB registers and xxx_sync()  call ensures we
> will not abort.

I don't understand.  Why not just access those registers from within 
the runtime_resume callback?

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to