rockchip: backport mmc fixes from kernel 6.4
Signed-off-by: Tianling Shen <cnsztl@immortalwrt.org>
This commit is contained in:
parent
dae139a209
commit
7be1e397c7
@ -0,0 +1,60 @@
|
||||
From b75a52b0dda353aeefb4830a320589a363f49579 Mon Sep 17 00:00:00 2001
|
||||
From: Shawn Lin <shawn.lin@rock-chips.com>
|
||||
Date: Thu, 2 Feb 2023 08:35:16 +0800
|
||||
Subject: [PATCH] mmc: sdhci-of-dwcmshc: Update DLL and pre-change delay for
|
||||
rockchip platform
|
||||
|
||||
For Rockchip platform, DLL bypass bit and start bit need to be set if
|
||||
DLL is not locked. And adjust pre-change delay to 0x3 for better signal
|
||||
test result.
|
||||
|
||||
Signed-off-by: Shawn Lin <shawn.lin@rock-chips.com>
|
||||
Link: https://lore.kernel.org/r/1675298118-64243-2-git-send-email-shawn.lin@rock-chips.com
|
||||
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
|
||||
---
|
||||
drivers/mmc/host/sdhci-of-dwcmshc.c | 13 +++++++++----
|
||||
1 file changed, 9 insertions(+), 4 deletions(-)
|
||||
|
||||
--- a/drivers/mmc/host/sdhci-of-dwcmshc.c
|
||||
+++ b/drivers/mmc/host/sdhci-of-dwcmshc.c
|
||||
@@ -48,6 +48,7 @@
|
||||
#define DWCMSHC_EMMC_DLL_RXCLK_SRCSEL 29
|
||||
#define DWCMSHC_EMMC_DLL_START_POINT 16
|
||||
#define DWCMSHC_EMMC_DLL_INC 8
|
||||
+#define DWCMSHC_EMMC_DLL_BYPASS BIT(24)
|
||||
#define DWCMSHC_EMMC_DLL_DLYENA BIT(27)
|
||||
#define DLL_TXCLK_TAPNUM_DEFAULT 0x10
|
||||
#define DLL_TXCLK_TAPNUM_90_DEGREES 0xA
|
||||
@@ -60,6 +61,7 @@
|
||||
#define DLL_RXCLK_NO_INVERTER 1
|
||||
#define DLL_RXCLK_INVERTER 0
|
||||
#define DLL_CMDOUT_TAPNUM_90_DEGREES 0x8
|
||||
+#define DLL_RXCLK_ORI_GATE BIT(31)
|
||||
#define DLL_CMDOUT_TAPNUM_FROM_SW BIT(24)
|
||||
#define DLL_CMDOUT_SRC_CLK_NEG BIT(28)
|
||||
#define DLL_CMDOUT_EN_SRC_CLK_NEG BIT(29)
|
||||
@@ -234,9 +236,12 @@ static void dwcmshc_rk3568_set_clock(str
|
||||
sdhci_writel(host, extra, reg);
|
||||
|
||||
if (clock <= 52000000) {
|
||||
- /* Disable DLL and reset both of sample and drive clock */
|
||||
- sdhci_writel(host, 0, DWCMSHC_EMMC_DLL_CTRL);
|
||||
- sdhci_writel(host, 0, DWCMSHC_EMMC_DLL_RXCLK);
|
||||
+ /*
|
||||
+ * Disable DLL and reset both of sample and drive clock.
|
||||
+ * The bypass bit and start bit need to be set if DLL is not locked.
|
||||
+ */
|
||||
+ sdhci_writel(host, DWCMSHC_EMMC_DLL_BYPASS | DWCMSHC_EMMC_DLL_START, DWCMSHC_EMMC_DLL_CTRL);
|
||||
+ sdhci_writel(host, DLL_RXCLK_ORI_GATE, DWCMSHC_EMMC_DLL_RXCLK);
|
||||
sdhci_writel(host, 0, DWCMSHC_EMMC_DLL_TXCLK);
|
||||
sdhci_writel(host, 0, DECMSHC_EMMC_DLL_CMDOUT);
|
||||
/*
|
||||
@@ -279,7 +284,7 @@ static void dwcmshc_rk3568_set_clock(str
|
||||
}
|
||||
|
||||
extra = 0x1 << 16 | /* tune clock stop en */
|
||||
- 0x2 << 17 | /* pre-change delay */
|
||||
+ 0x3 << 17 | /* pre-change delay */
|
||||
0x3 << 19; /* post-change delay */
|
||||
sdhci_writel(host, extra, dwc_priv->vendor_specific_area1 + DWCMSHC_EMMC_ATCTRL);
|
||||
|
||||
@ -0,0 +1,52 @@
|
||||
From 49502408007b77ff290ce62e6218cefaeedcb31a Mon Sep 17 00:00:00 2001
|
||||
From: Vasily Khoruzhick <anarsoul@gmail.com>
|
||||
Date: Thu, 9 Mar 2023 17:03:49 -0800
|
||||
Subject: [PATCH] mmc: sdhci-of-dwcmshc: properly determine max clock on
|
||||
Rockchip
|
||||
|
||||
Currently .get_max_clock returns the current clock rate for cclk_emmc
|
||||
on rk35xx, thus max clock gets set to whatever bootloader set it to.
|
||||
|
||||
In case of u-boot, it is intentionally reset to 50 MHz if it boots
|
||||
from eMMC, see mmc_deinit() in u-boot sources. As a result, HS200 and
|
||||
HS400 modes are never selected by Linux, because dwcmshc_rk35xx_postinit
|
||||
clears appropriate caps if host->mmc->f_max is < 52MHz
|
||||
|
||||
cclk_emmc is not a fixed clock on rk35xx, so using
|
||||
sdhci_pltfm_clk_get_max_clock is not appropriate here.
|
||||
|
||||
Implement rk35xx_get_max_clock that returns actual max clock for cclk_emmc.
|
||||
|
||||
Signed-off-by: Vasily Khoruzhick <anarsoul@gmail.com>
|
||||
Acked-by: Adrian Hunter <adrian.hunter@intel.com>
|
||||
Link: https://lore.kernel.org/r/20230310010349.509132-1-anarsoul@gmail.com
|
||||
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
|
||||
---
|
||||
drivers/mmc/host/sdhci-of-dwcmshc.c | 9 ++++++++-
|
||||
1 file changed, 8 insertions(+), 1 deletion(-)
|
||||
|
||||
--- a/drivers/mmc/host/sdhci-of-dwcmshc.c
|
||||
+++ b/drivers/mmc/host/sdhci-of-dwcmshc.c
|
||||
@@ -126,6 +126,13 @@ static unsigned int dwcmshc_get_max_cloc
|
||||
return pltfm_host->clock;
|
||||
}
|
||||
|
||||
+static unsigned int rk35xx_get_max_clock(struct sdhci_host *host)
|
||||
+{
|
||||
+ struct sdhci_pltfm_host *pltfm_host = sdhci_priv(host);
|
||||
+
|
||||
+ return clk_round_rate(pltfm_host->clk, ULONG_MAX);
|
||||
+}
|
||||
+
|
||||
static void dwcmshc_check_auto_cmd23(struct mmc_host *mmc,
|
||||
struct mmc_request *mrq)
|
||||
{
|
||||
@@ -343,7 +350,7 @@ static const struct sdhci_ops sdhci_dwcm
|
||||
.set_clock = dwcmshc_rk3568_set_clock,
|
||||
.set_bus_width = sdhci_set_bus_width,
|
||||
.set_uhs_signaling = dwcmshc_set_uhs_signaling,
|
||||
- .get_max_clock = sdhci_pltfm_clk_get_max_clock,
|
||||
+ .get_max_clock = rk35xx_get_max_clock,
|
||||
.reset = rk35xx_sdhci_reset,
|
||||
.adma_write_desc = dwcmshc_adma_write_desc,
|
||||
};
|
||||
Loading…
Reference in New Issue
Block a user