From: "Joe Woodward" <jw@terrafix.co.uk>
To: linux-mmc@vger.kernel.org
Subject: mmc: sdio: runtime PM and 8686 problems
Date: Thu, 17 Nov 2011 09:16:17 +0000 [thread overview]
Message-ID: <WC20111117091617.420182@terrafix.co.uk> (raw)
In-Reply-To: <004901cca4cc$f18d6180$d4a82480$%jun@samsung.com>
I have a GUMSTIX Overo (OMAP35xx) with a Marvel 8686 WiFi attached via SDIO (ok, I think it's actually a Wi2Wi W2CBW003 module containing an 8686).
Up to the 3.1 mainline kernel WiFi worked just fine! I could boot the kernel and load the libertas module successfully:
(during boot)
[ 3.193176] mmc1: new SDIO card at address 0001
(when loading module)
# modprobe libertas_sdio
[ 30.817749] lib80211: common routines for IEEE802.11 drivers
[ 31.143951] cfg80211: Calling CRDA to update world regulatory domain
[ 31.375823] libertas_sdio: Libertas SDIO driver
[ 31.380737] libertas_sdio: Copyright Pierre Ossman
[ 32.035186] libertas_sdio mmc1:0001:1: (unregistered net_device): 00:19:88:3e:4f:f9, fw 9.70.3p36, cap 0x00000303
[ 32.129943] libertas_sdio mmc1:0001:1: wlan0: Marvell WLAN 802.11 adapter
However it seems that runtime PM has now been turned on by default (by setting MMC_CAP_POWER_OFF_CARD in drivers/mmc/core/host.c), and the WiFi
has stopped working!
So, if I run the 3.2-rc2 (or rc1 for that matter) I get the following:
(during boot)
[ 3.486541] mmc1: new SDIO card at address 0001
(when loading module)
# modprobe libertas_sdio
[ 34.015350] lib80211: common routines for IEEE802.11 drivers
[ 34.345092] cfg80211: Calling CRDA to update world regulatory domain
[ 34.574096] libertas_sdio: Libertas SDIO driver
[ 34.578979] libertas_sdio: Copyright Pierre Ossman
[ 34.637481] libertas_sdio: probe of mmc1:0001:1 failed with error -16
Looking at arch/arm/mach-omap2/overo.c you can see that GPIO16 (OVERO_GPIO_W2W_NRESET) is the nRESET line for the 8686. This is held low for
10usecs during boot (see overo.c line 530 to 533).
I'm not 100% sure how power is connected to the SD8686 as GUMSTIX don't release schematics for their modules.
If I disable runtime PM (by commenting out the setting of MMC_CAP_POWER_OFF_CARD) everything works as before.
I can see plenty of threads relating to getting the SD8686 working for OLPC, and that some workarounds have made it in to the mainline kernel. From what
I understand the problem is that runtime PM removes power from the 8686 after boot and before the loading of the libertas module.
Are these workarounds expected to be enough for runtime PM to just "work"?
Is there anything special I should be doing with the nRESET line other than that which is done by default in the board file (holding nRESET low for 10usecs)?
Cheers,
Joe
next prev parent reply other threads:[~2011-11-17 10:10 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-10 13:41 [PATCH 2/2] mmc: core: Support packed command for eMMC4.5 device merez
2011-11-11 7:26 ` Seungwon Jeon
2011-11-11 9:38 ` S, Venkatraman
2011-11-11 19:01 ` merez
2011-11-13 13:04 ` merez
2011-11-14 9:46 ` Seungwon Jeon
2011-11-15 12:48 ` merez
2011-11-17 2:02 ` Seungwon Jeon
2011-11-17 9:16 ` Joe Woodward [this message]
2011-11-14 9:44 ` Seungwon Jeon
2011-11-15 13:27 ` merez
2011-11-17 2:21 ` Seungwon Jeon
2011-11-17 13:45 ` merez
-- strict thread matches above, loose matches on Subject: below --
2011-11-18 8:53 mmc: sdio: runtime PM and 8686 problems Joe Woodward
2011-11-18 9:06 ` Ohad Ben-Cohen
2011-11-18 10:00 ` Joe Woodward
2011-11-18 12:15 ` Ohad Ben-Cohen
2011-11-29 11:17 ` Joe Woodward
2011-11-29 12:43 ` Ohad Ben-Cohen
2011-11-29 21:42 ` Daniel Drake
2011-11-30 10:36 ` Ohad Ben-Cohen
2011-11-30 14:12 ` Daniel Drake
2011-11-30 14:29 ` Ohad Ben-Cohen
2011-11-30 14:57 ` Daniel Drake
2011-12-01 7:50 ` Ohad Ben-Cohen
2011-11-30 15:00 ` Joe Woodward
2011-12-01 8:07 ` Ohad Ben-Cohen
2011-12-01 11:28 ` Joe Woodward
2011-12-03 16:44 ` Daniel Drake
2011-12-03 16:45 ` Daniel Drake
2011-12-07 12:28 ` Joe Woodward
2011-12-19 9:10 ` Ohad Ben-Cohen
2011-12-19 9:22 ` Ohad Ben-Cohen
2012-04-26 23:51 ` Steve Sakoman
2012-04-27 8:26 ` Joe Woodward
2012-04-28 9:51 ` NeilBrown
2011-12-01 11:29 Joe Woodward
2011-12-01 11:29 Joe Woodward
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=WC20111117091617.420182@terrafix.co.uk \
--to=jw@terrafix.co.uk \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox