All of lore.kernel.org
 help / color / mirror / Atom feed
From: Roger Quadros <rogerq@ti.com>
To: Chao Xu <caesarxuchao@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	<stern@rowland.harvard.edu>
Subject: Re: USB: OMAP: questions on patch "Implement runtime idling and remote wakeup for OMAP EHCI controller"
Date: Wed, 9 Apr 2014 11:01:37 +0300	[thread overview]
Message-ID: <5344FE61.4030306@ti.com> (raw)
In-Reply-To: <CAKZq+reJuhKE6L67RfZL7Zyd6W=cU-ehK3faBKmj3stVg5ukdA@mail.gmail.com>

Hi,

On 04/09/2014 07:12 AM, Chao Xu wrote:
> Hi,
> 
> I have two questions about this patch series. Any help is appreciated.
> 1. in the cover letter(https://lkml.org/lkml/2013/7/10/355), it states
> "the series implements suspend/resume for the OMAP EHCI host
> controller during runtime idle". I don't understand when is the EHCI
> host controller "runtime idle"? Does "runtime idle" refers to when all
> attached USB devices are suspended? or does it refer to whenever there
> is no tx/rx ongoing in the EHCI host controller?

It refers to when all USB devices are auto-suspended.

> 
> 2. I think the purpose of the patch is to allow the USB power domain
> (namely, L3INIT power domain) to enter low power state. So assuming

Right.

> the power domain is now in RETENTION state, will the EHCI host be able
> to detect remote wakeup in this case? i.e. will the ULPI DATA0, DATA1
> and DATA3 reflect the Linestate and Interrupt when the L3INIT power
> domain is in RETENTION? Put it in another way, when the power domain
> in is a low power state defined in OMAP spec, is the USB host and PHY
> in  "low power mode" defined in USB spec?

The OMAP EHCI controller is not running and the OMAP USB_HS module is
idle as well and so it can't detect any changes on the ULPI lines.
The PHY is in low power state and should reflect the line status on the DAT0-1
lines.

During the suspended state, we use the IOring mechanism to detect a changed on DAT lines
and wake up the controller/system.

cheers,
-roger


      reply	other threads:[~2014-04-09  8:01 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-09  4:12 USB: OMAP: questions on patch "Implement runtime idling and remote wakeup for OMAP EHCI controller" Chao Xu
2014-04-09  8:01 ` Roger Quadros [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5344FE61.4030306@ti.com \
    --to=rogerq@ti.com \
    --cc=caesarxuchao@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=stern@rowland.harvard.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.