All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alan Stern <stern@rowland.harvard.edu>
To: Pierre Tardy <tardyp@gmail.com>
Cc: linux-kernel@vger.kernel.org,
	<linux-pm@lists.linux-foundation.org>,
	<linux-mmc@vger.kernel.org>
Subject: Re: [linux-pm] [RFC,PATCHv3 0/3] sdhci runtime_pm implementation
Date: Tue, 1 Mar 2011 14:57:04 -0500 (EST)	[thread overview]
Message-ID: <Pine.LNX.4.44L0.1103011454250.2034-100000@iolanthe.rowland.org> (raw)
In-Reply-To: <AANLkTi=ogThxB_2bxANV4hUsLGZs04gzHSCyH90Bm87U@mail.gmail.com>

On Tue, 1 Mar 2011, Pierre Tardy wrote:

> On Tue, Mar 1, 2011 at 8:33 PM, Alan Stern <stern@rowland.harvard.edu> wrote:
> > On Tue, 1 Mar 2011, Pierre Tardy wrote:
> >
> >> Please find sdhci runtime_pm implementation.
> >>
> >> It uses clock gating fw as a tip to know when our chip is idle.
> >> It implements wake up from card insertion/removal.
> >>
> >> This is RFC, please dont merge yet. I really would like to have deep review
> >> from PCI linux-pm guys.
> >>
> >> Opens are:
> >>
> >> 1/ Not sure if the pci configs in the driver in rpm_suspend/resume flow
> >>  are not duplicate from what the core is doing.
> >
> > There may be one or two small errors.
> >
> >> 2/ Wakeup from D3hot: I cannot find any driver that is implementing it in current upstream,
> >
> > Other drivers do it, but they use PCI PME# instead of interrupts.
> Could you please elaborate?
> My understanding is that PCI PME will generate MSI, which translate in
> interrupt.

It depends on the platform.  On systems with ACPI, PCI PME generates an 
ACPI I/O event, which is handled by the ACPI and PM cores.  It does not 
invoke the device driver's interrupt handler.

Alan Stern


WARNING: multiple messages have this Message-ID (diff)
From: Alan Stern <stern@rowland.harvard.edu>
To: Pierre Tardy <tardyp@gmail.com>
Cc: linux-kernel@vger.kernel.org,
	linux-pm@lists.linux-foundation.org, linux-mmc@vger.kernel.org
Subject: Re: [linux-pm] [RFC,PATCHv3 0/3] sdhci runtime_pm implementation
Date: Tue, 1 Mar 2011 14:57:04 -0500 (EST)	[thread overview]
Message-ID: <Pine.LNX.4.44L0.1103011454250.2034-100000@iolanthe.rowland.org> (raw)
In-Reply-To: <AANLkTi=ogThxB_2bxANV4hUsLGZs04gzHSCyH90Bm87U@mail.gmail.com>

On Tue, 1 Mar 2011, Pierre Tardy wrote:

> On Tue, Mar 1, 2011 at 8:33 PM, Alan Stern <stern@rowland.harvard.edu> wrote:
> > On Tue, 1 Mar 2011, Pierre Tardy wrote:
> >
> >> Please find sdhci runtime_pm implementation.
> >>
> >> It uses clock gating fw as a tip to know when our chip is idle.
> >> It implements wake up from card insertion/removal.
> >>
> >> This is RFC, please dont merge yet. I really would like to have deep review
> >> from PCI linux-pm guys.
> >>
> >> Opens are:
> >>
> >> 1/ Not sure if the pci configs in the driver in rpm_suspend/resume flow
> >>  are not duplicate from what the core is doing.
> >
> > There may be one or two small errors.
> >
> >> 2/ Wakeup from D3hot: I cannot find any driver that is implementing it in current upstream,
> >
> > Other drivers do it, but they use PCI PME# instead of interrupts.
> Could you please elaborate?
> My understanding is that PCI PME will generate MSI, which translate in
> interrupt.

It depends on the platform.  On systems with ACPI, PCI PME generates an 
ACPI I/O event, which is handled by the ACPI and PM cores.  It does not 
invoke the device driver's interrupt handler.

Alan Stern

  parent reply	other threads:[~2011-03-01 19:57 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-01 18:15 [RFC,PATCHv3 0/3] sdhci runtime_pm implementation Pierre Tardy
2011-03-01 18:15 ` [RFC, PATCHv3 1/3] mmc: sdhci-pci: Enable runtime PM support Pierre Tardy
2011-03-01 18:15 ` [RFC,PATCHv3 " Pierre Tardy
2011-03-01 19:46   ` [RFC, PATCHv3 " Alan Stern
2011-03-01 19:46   ` [linux-pm] " Alan Stern
2011-03-01 19:46     ` Alan Stern
2011-03-01 18:15 ` [RFC,PATCHv3 2/3] mmc: sdhci: use ios->clock to know when sdhci is idle Pierre Tardy
2011-03-01 18:15 ` [RFC, PATCHv3 " Pierre Tardy
2011-03-01 18:15 ` [RFC,PATCHv3 3/3] mmc: sdhci: handle wake-up from runtime_pm Pierre Tardy
2011-03-01 19:53   ` [RFC, PATCHv3 " Alan Stern
2011-03-01 19:53   ` [linux-pm] " Alan Stern
2011-03-01 19:53     ` Alan Stern
2011-03-01 20:01     ` Pierre Tardy
2011-03-01 20:01     ` [linux-pm] " Pierre Tardy
2011-03-01 20:27       ` Alan Stern
2011-03-01 20:27       ` [linux-pm] " Alan Stern
2011-03-01 20:27         ` Alan Stern
2011-03-01 20:48         ` Pierre Tardy
2011-03-01 20:48         ` [linux-pm] " Pierre Tardy
2011-03-01 18:15 ` Pierre Tardy
2011-03-01 19:33 ` [linux-pm] [RFC,PATCHv3 0/3] sdhci runtime_pm implementation Alan Stern
2011-03-01 19:33   ` Alan Stern
2011-03-01 19:36   ` Pierre Tardy
2011-03-01 19:57     ` Alan Stern
2011-03-01 19:57     ` Alan Stern [this message]
2011-03-01 19:57       ` [linux-pm] " Alan Stern
2011-03-01 20:06       ` Pierre Tardy
2011-03-01 20:06       ` [linux-pm] " Pierre Tardy
2011-03-01 20:30         ` Alan Stern
2011-03-01 20:30         ` [linux-pm] " Alan Stern
2011-03-01 20:30           ` Alan Stern
2011-03-01 21:07     ` Rafael J. Wysocki
2011-03-01 21:07     ` [linux-pm] " Rafael J. Wysocki
2011-03-01 23:40       ` Pierre Tardy
2011-03-01 23:40       ` [linux-pm] " Pierre Tardy
2011-03-01 23:49         ` Rafael J. Wysocki
2011-03-02 15:12           ` Alan Stern
2011-03-02 15:12           ` [linux-pm] " Alan Stern
2011-03-02 15:12             ` Alan Stern
2011-03-01 23:49         ` Rafael J. Wysocki
2011-03-01 19:36   ` Pierre Tardy
2011-03-01 19:33 ` Alan Stern

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=Pine.LNX.4.44L0.1103011454250.2034-100000@iolanthe.rowland.org \
    --to=stern@rowland.harvard.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=tardyp@gmail.com \
    /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.