From: Vincent Yang <Vincent.Yang@tw.fujitsu.com> To: linux-mmc@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: andy.green@linaro.org, patches@linaro.org, jaswinder.singh@linaro.org, arnd@arndb.de, olof@lixom.net, linux@arm.linux.org.uk, chris@printf.net, anton@enomsg.org, t.takinishi@jp.fujitsu.com, mollie.wu@linaro.org, Vincent.Yang@tw.fujitsu.com Subject: [PATCH 4/4] mmc: core: hold SD Clock before CMD11 during Signal Date: Sun, 13 Jul 2014 13:21:10 +0800 [thread overview] Message-ID: <1405228870-5088-5-git-send-email-Vincent.Yang@tw.fujitsu.com> (raw) In-Reply-To: <1405228870-5088-1-git-send-email-Vincent.Yang@tw.fujitsu.com> Voltage Switch Procedure This patch is to fix an issue found on mb86s7x platforms. [symptom] There are some UHS-1 SD memory cards sometimes cannot be detected correctly, e.g., Transcend 600x SDXC 64GB UHS-1 memory card. During Signal Voltage Switch Procedure, failure to switch is indicated by the card holding DAT[3:0] low. [analysis] According to SD Host Controller Simplified Specification Version 3.00 chapter 3.6.1, the Signal Voltage Switch Procedure should be: (1) Check S18A; (2) Issue CMD11; (3) Check CMD 11 response; (4) Stop providing SD clock; (5) Check DAT[3:0] should be 0000b; (6) Set 1.8V Signal Enable; (7) Wait 5ms; (8) Check 1.8V Signal Enable; (9) Provide SD Clock; (10) Wait 1ms; (11) Check DAT[3:0] should be 1111b; (12) error handling With CONFIG_MMC_CLKGATE=y, sometimes there is one more gating/un-gating SD clock between (2) and (3). In this case, some UHS-1 SD cards will hold DAT[3:0] 0000b at (11) and thus fails Signal Voltage Switch Procedure. [solution] By mmc_host_clk_hold() before CMD11, the additional gating/un-gating SD clock between (2) and (3) can be prevented and thus no failure at (11). It has been verified with many UHS-1 SD cards on mb86s7x platforms and works correctly. Signed-off-by: Vincent Yang <Vincent.Yang@tw.fujitsu.com> --- drivers/mmc/core/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c index 7dc0c85..764af63 100644 --- a/drivers/mmc/core/core.c +++ b/drivers/mmc/core/core.c @@ -1428,6 +1428,8 @@ int mmc_set_signal_voltage(struct mmc_host *host, int signal_voltage, u32 ocr) pr_warning("%s: cannot verify signal voltage switch\n", mmc_hostname(host)); + mmc_host_clk_hold(host); + cmd.opcode = SD_SWITCH_VOLTAGE; cmd.arg = 0; cmd.flags = MMC_RSP_R1 | MMC_CMD_AC; @@ -1438,8 +1440,6 @@ int mmc_set_signal_voltage(struct mmc_host *host, int signal_voltage, u32 ocr) if (!mmc_host_is_spi(host) && (cmd.resp[0] & R1_ERROR)) return -EIO; - - mmc_host_clk_hold(host); /* * The card should drive cmd and dat[0:3] low immediately * after the response of cmd11, but wait 1 ms to be sure -- 1.9.0
WARNING: multiple messages have this Message-ID (diff)
From: Vincent.Yang@tw.fujitsu.com (Vincent Yang) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 4/4] mmc: core: hold SD Clock before CMD11 during Signal Date: Sun, 13 Jul 2014 13:21:10 +0800 [thread overview] Message-ID: <1405228870-5088-5-git-send-email-Vincent.Yang@tw.fujitsu.com> (raw) In-Reply-To: <1405228870-5088-1-git-send-email-Vincent.Yang@tw.fujitsu.com> Voltage Switch Procedure This patch is to fix an issue found on mb86s7x platforms. [symptom] There are some UHS-1 SD memory cards sometimes cannot be detected correctly, e.g., Transcend 600x SDXC 64GB UHS-1 memory card. During Signal Voltage Switch Procedure, failure to switch is indicated by the card holding DAT[3:0] low. [analysis] According to SD Host Controller Simplified Specification Version 3.00 chapter 3.6.1, the Signal Voltage Switch Procedure should be: (1) Check S18A; (2) Issue CMD11; (3) Check CMD 11 response; (4) Stop providing SD clock; (5) Check DAT[3:0] should be 0000b; (6) Set 1.8V Signal Enable; (7) Wait 5ms; (8) Check 1.8V Signal Enable; (9) Provide SD Clock; (10) Wait 1ms; (11) Check DAT[3:0] should be 1111b; (12) error handling With CONFIG_MMC_CLKGATE=y, sometimes there is one more gating/un-gating SD clock between (2) and (3). In this case, some UHS-1 SD cards will hold DAT[3:0] 0000b at (11) and thus fails Signal Voltage Switch Procedure. [solution] By mmc_host_clk_hold() before CMD11, the additional gating/un-gating SD clock between (2) and (3) can be prevented and thus no failure at (11). It has been verified with many UHS-1 SD cards on mb86s7x platforms and works correctly. Signed-off-by: Vincent Yang <Vincent.Yang@tw.fujitsu.com> --- drivers/mmc/core/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c index 7dc0c85..764af63 100644 --- a/drivers/mmc/core/core.c +++ b/drivers/mmc/core/core.c @@ -1428,6 +1428,8 @@ int mmc_set_signal_voltage(struct mmc_host *host, int signal_voltage, u32 ocr) pr_warning("%s: cannot verify signal voltage switch\n", mmc_hostname(host)); + mmc_host_clk_hold(host); + cmd.opcode = SD_SWITCH_VOLTAGE; cmd.arg = 0; cmd.flags = MMC_RSP_R1 | MMC_CMD_AC; @@ -1438,8 +1440,6 @@ int mmc_set_signal_voltage(struct mmc_host *host, int signal_voltage, u32 ocr) if (!mmc_host_is_spi(host) && (cmd.resp[0] & R1_ERROR)) return -EIO; - - mmc_host_clk_hold(host); /* * The card should drive cmd and dat[0:3] low immediately * after the response of cmd11, but wait 1 ms to be sure -- 1.9.0
next prev parent reply other threads:[~2014-07-13 5:23 UTC|newest] Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-07-13 5:21 [PATCH 0/4] mmc: sdhci: adding support for a new Fujitsu sdhci IP Vincent Yang 2014-07-13 5:21 ` Vincent Yang 2014-07-13 5:21 ` [PATCH 1/4] mmc: sdhci: add quirk for voltage switch callback Vincent Yang 2014-07-13 5:21 ` Vincent Yang 2014-07-13 8:21 ` Russell King - ARM Linux 2014-07-13 8:21 ` Russell King - ARM Linux 2014-07-13 9:44 ` Vincent.Yang 2014-07-13 9:44 ` Vincent.Yang 2014-07-13 5:21 ` [PATCH 2/4] mmc: sdhci: add quirk for tuning work around Vincent Yang 2014-07-13 5:21 ` Vincent Yang 2014-07-13 5:21 ` [PATCH 3/4] mmc: sdhci: add quirk for single block transactions Vincent Yang 2014-07-13 5:21 ` Vincent Yang 2014-07-13 5:21 ` Vincent Yang [this message] 2014-07-13 5:21 ` [PATCH 4/4] mmc: core: hold SD Clock before CMD11 during Signal Vincent Yang 2014-07-13 8:30 ` Russell King - ARM Linux 2014-07-13 8:30 ` Russell King - ARM Linux 2014-07-13 9:49 ` Vincent.Yang 2014-07-13 9:49 ` Vincent.Yang 2014-11-21 0:51 [PATCH 0/4] mmc: sdhci: adding support for a new Fujitsu sdhci IP Vincent Yang 2014-11-21 0:51 ` [PATCH 4/4] mmc: core: hold SD Clock before CMD11 during Signal Vincent Yang 2014-11-21 0:51 ` Vincent Yang 2014-11-21 7:23 ` Johan Rudholm 2014-11-21 7:23 ` Johan Rudholm 2014-11-24 9:56 ` Ulf Hansson 2014-11-24 9:56 ` Ulf Hansson
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=1405228870-5088-5-git-send-email-Vincent.Yang@tw.fujitsu.com \ --to=vincent.yang@tw.fujitsu.com \ --cc=andy.green@linaro.org \ --cc=anton@enomsg.org \ --cc=arnd@arndb.de \ --cc=chris@printf.net \ --cc=jaswinder.singh@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-mmc@vger.kernel.org \ --cc=linux@arm.linux.org.uk \ --cc=mollie.wu@linaro.org \ --cc=olof@lixom.net \ --cc=patches@linaro.org \ --cc=t.takinishi@jp.fujitsu.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: linkBe 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.