From: Kurt Kanzenbach <kurt@linutronix.de>
To: adrian.hunter@intel.com
Cc: ulf.hansson@linaro.org, linux-mmc@vger.kernel.org,
linux-kernel@vger.kernel.org, tglx@linutronix.de,
kurt@linutronix.de
Subject: [PATCH 0/1] eMMC controller issue on Intel Baytrail SoC
Date: Tue, 19 Jun 2018 08:31:18 +0200 [thread overview]
Message-ID: <20180619063119.3955-1-kurt@linutronix.de> (raw)
Hi,
I've encountered a problem on an Intel Atom E3825. When performing lots of
reboots (10, 50, 100, ...) the eMMC controller stops working. The reset commands
won't work anymore and you get error messages such as:
|mmc1: Reset 0x1 never completed.
|sdhci: =========== REGISTER DUMP (mmc1)===========
|sdhci: Sys addr: 0xffffffff | Version: 0x0000ffff
|sdhci: Blk size: 0x0000ffff | Blk cnt: 0x0000ffff
|sdhci: Argument: 0xffffffff | Trn mode: 0x0000ffff
|sdhci: Present: 0xffffffff | Host ctl: 0x000000ff
|sdhci: Power: 0x000000ff | Blk gap: 0x000000ff
|sdhci: Wake-up: 0x000000ff | Clock: 0x0000ffff
|sdhci: Timeout: 0x000000ff | Int stat: 0xffffffff
|sdhci: Int enab: 0xffffffff | Sig enab: 0xffffffff
|sdhci: AC12 err: 0x0000ffff | Slot int: 0x0000ffff
|sdhci: Caps: 0xffffffff | Caps_1: 0xffffffff
|sdhci: Cmd: 0x0000ffff | Max curr: 0xffffffff
|sdhci: Host ctl2: 0x0000ffff
|sdhci: ADMA Err: 0xffffffff | ADMA Ptr: 0xffffffff
After using ftrace, I've discovered that this issue happens when runtime power
management is utilized. So after searching a bit, I've found the errata list for
the E3825:
https://www.intel.ca/content/dam/www/public/us/en/documents/specification-updates/atom-e3800-family-spec-update.pdf
Erratum VLI10 basically states, that suspend/resume shouldn't be used. Otherwise
wrong data between memory the device may be transferred. Therefore, I've
disabled runtime power management and the issue disappeared. That's what the
following patch does.
This patch is tested against v4.17 and v4.9.
Any suggestions?
Kurt Kanzenbach (1):
mmc: sdhci-pci: fix eMMC controller issue on Intel Baytrail SoCs
drivers/mmc/host/sdhci-pci-core.c | 17 ++++++++++++++++-
1 file changed, 16 insertions(+), 1 deletion(-)
--
2.11.0
next reply other threads:[~2018-06-19 6:31 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-19 6:31 Kurt Kanzenbach [this message]
2018-06-19 6:31 ` [PATCH 1/1] mmc: sdhci-pci: fix eMMC controller issue on Intel Baytrail SoCs Kurt Kanzenbach
2018-06-19 7:03 ` Adrian Hunter
2018-06-20 13:15 ` Kurt Kanzenbach
2018-06-20 15:50 ` Adrian Hunter
2018-06-25 14:36 ` Kurt Kanzenbach
2018-07-10 12:51 ` Adrian Hunter
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=20180619063119.3955-1-kurt@linutronix.de \
--to=kurt@linutronix.de \
--cc=adrian.hunter@intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=tglx@linutronix.de \
--cc=ulf.hansson@linaro.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