linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: haibo.chen@freescale.com (Haibo Chen)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 5/6] mmc: sdhci-esdhc-imx: change default watermark level and burst length
Date: Wed, 5 Aug 2015 18:38:41 +0800	[thread overview]
Message-ID: <1438771122-8601-6-git-send-email-haibo.chen@freescale.com> (raw)
In-Reply-To: <1438771122-8601-1-git-send-email-haibo.chen@freescale.com>

By default, for all imx SoC types, the watermark level is 16, and the
burst length is 8. But if the SDIO/SD/MMC I/O speed is fast enough,
this default watermark level and burst length will be the performance
bottleneck.

For example, i.MX7D support eMMC HS400 mode, this mode can run in 8 bit,
200MHZ DDR mode. So the I/O speed improve a lot compare to SD3.0.
The default burst length is 8, if we don't change this value, in
HS400 mode, when we do eMMC read operation, we can find that the
clock signal will stop for a period of time. This means the speed
of data moving on AHB bus is slower than I/O speed. So we should
improve the speed of data moving on AHB bus.

This patch set the default burst length as 16, and set the default
watermark level as 64. The test result is the clock signal has
no stop during the eMMC HS400 operation.

Signed-off-by: Haibo Chen <haibo.chen@freescale.com>
---
 drivers/mmc/host/sdhci-esdhc-imx.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/mmc/host/sdhci-esdhc-imx.c b/drivers/mmc/host/sdhci-esdhc-imx.c
index 803d24f..97aa944 100644
--- a/drivers/mmc/host/sdhci-esdhc-imx.c
+++ b/drivers/mmc/host/sdhci-esdhc-imx.c
@@ -1160,7 +1160,8 @@ static int sdhci_esdhc_imx_probe(struct platform_device *pdev)
 	 * to something insane.  Change it back here.
 	 */
 	if (esdhc_is_usdhc(imx_data)) {
-		writel(0x08100810, host->ioaddr + ESDHC_WTMK_LVL);
+		writel(0x10401040, host->ioaddr + ESDHC_WTMK_LVL);
+
 		host->quirks2 |= SDHCI_QUIRK2_PRESET_VALUE_BROKEN;
 		host->mmc->caps |= MMC_CAP_1_8V_DDR;
 
-- 
1.9.1

  parent reply	other threads:[~2015-08-05 10:38 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-08-05 10:38 [PATCH v4 0/6] mmc: imx: a few fixes and new feature Haibo Chen
2015-08-05 10:38 ` [PATCH v4 1/6] mmc: sdhci-esdhc-imx: add imx7d support and support HS400 Haibo Chen
2015-08-07  7:38   ` Dong Aisheng
2015-08-07  9:53     ` Chen Bough
2015-08-07 11:19       ` Dong Aisheng
2015-08-05 10:38 ` [PATCH v4 2/6] mmc: sdhci-esdhc-imx: add tuning-step seting support Haibo Chen
2015-08-07  7:40   ` Dong Aisheng
2015-08-07 10:04     ` Chen Bough
2015-08-05 10:38 ` [PATCH v4 3/6] mmc: sdhci-esdhc-imx: add compatible string in bingding doc Haibo Chen
2015-08-05 10:38 ` [PATCH v4 4/6] ARM: dts: imx7d-sdb: add eMMC5.0 support Haibo Chen
2015-08-05 10:38 ` Haibo Chen [this message]
2015-08-05 10:38 ` [PATCH v4 6/6] mmc: sdhci-esdhc-imx: set back the burst_length_enable bit to 1 Haibo Chen
2015-08-07  7:50   ` Dong Aisheng
2015-08-07 10:11     ` Chen Bough

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=1438771122-8601-6-git-send-email-haibo.chen@freescale.com \
    --to=haibo.chen@freescale.com \
    --cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).