All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adrian Hunter <adrian.hunter@intel.com>
To: Chris Ball <cjb@laptop.org>
Cc: linux-mmc <linux-mmc@vger.kernel.org>
Subject: Re: [PATCH 2/2] mmc: sdhci-pci: add runtime pm support
Date: Mon, 17 Oct 2011 11:19:37 +0300	[thread overview]
Message-ID: <4E9BE519.2020908@intel.com> (raw)
In-Reply-To: <m262jpekox.fsf@bob.laptop.org>

On 16/10/11 05:26, Chris Ball wrote:
> Hi,
> 
> On Sat, Oct 15 2011, Chris Ball wrote:
>> Card insertion detection after boot no longer works (nothing new appears
>> in dmesg), so I'm unable to use the reader.  Are you able to try to
>> figure this out?  I'm happy to try any patches you suggest.
> 
> If I boot with a card inserted, I still see repeated "irq while runtime
> suspended" messages for 80 seconds after boot, but the card *does* work.
> I don't see further "irq while runtime suspended" messages during or
> after I/O to the card at that point.
> 
> So, we have a problem of unexpected irqs while runtime suspended, and a
> problem of broken card detection when booted without a card inserted
> (on my laptop).

Unfortunately I do not have any information about the unexpected irqs.

Card detect will not work because the interrupt is masked while the
host controller is runtime suspended, on the grounds that the host
controller may be powered off and so cannot handle interrupts.  An
alternate mechanism is needed to enable the host controller to
wake-up from a card detect signal.

So Runtime PM needs to be opt-in.  I have sent a patch.

      reply	other threads:[~2011-10-17  8:19 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-03 12:33 [PATCH 0/2] mmc: sdhci-pci: add runtime pm support Adrian Hunter
2011-10-03 12:33 ` [PATCH 1/2] mmc: core: move ->request() call from atomic context Adrian Hunter
2011-10-05 13:56   ` Ulf Hansson
2011-10-10 22:05     ` Chris Ball
2011-10-03 12:33 ` [PATCH 2/2] mmc: sdhci-pci: add runtime pm support Adrian Hunter
2011-10-04  4:44   ` Jaehoon Chung
2011-10-04  6:22     ` Adrian Hunter
2011-10-05 14:59   ` Ulf Hansson
2011-10-06  7:47     ` Adrian Hunter
2011-10-06 14:39       ` Ulf Hansson
2011-10-10 22:07   ` Chris Ball
2011-10-16  1:06   ` Chris Ball
2011-10-16  2:26     ` Chris Ball
2011-10-17  8:19       ` Adrian Hunter [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=4E9BE519.2020908@intel.com \
    --to=adrian.hunter@intel.com \
    --cc=cjb@laptop.org \
    --cc=linux-mmc@vger.kernel.org \
    /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.