* [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode
@ 2019-01-29 5:40 marek.vasut
2019-01-29 8:01 ` Wolfram Sang
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: marek.vasut @ 2019-01-29 5:40 UTC (permalink / raw)
To: linux-mmc
Cc: Takeshi Saito, Marek Vasut, Niklas Söderlund, Simon Horman,
Ulf Hansson, Wolfram Sang, linux-renesas-soc
From: Takeshi Saito <takeshi.saito.xv@renesas.com>
This fixes card initialization failure in high speed mode.
If U-Boot uses SDR or HS200/400 mode before starting Linux and Linux
DT does not enable SDR/HS200/HS400 mode, card initialization fails in
high speed mode.
It is necessary to initialize SCC registers during card initialization
phase. HW reset function is registered only for a port with either of
SDR/HS200/HS400 properties in device tree. If SDR/HS200/HS400 properties
are not present in device tree, SCC registers will not be reset. In SoC
that support SCC registers, HW reset function should be registered
regardless of the configuration of device tree.
Reproduction procedure:
- Use U-Boot that support MMC HS200/400 mode.
- Delete HS200/HS400 properties in device tree.
(Delete mmc-hs200-1_8v and mmc-hs400-1_8v)
- MMC port works high speed mode and all commands fail.
Signed-off-by: Takeshi Saito <takeshi.saito.xv@renesas.com>
Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
Cc: Simon Horman <horms+renesas@verge.net.au>
Cc: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
Cc: linux-renesas-soc@vger.kernel.org
To: linux-mmc@vger.kernel.org
--
NOTE: Marek: - Reworded commit message
- Updated patch to next/master
---
drivers/mmc/host/renesas_sdhi_core.c | 11 +++++++----
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/drivers/mmc/host/renesas_sdhi_core.c b/drivers/mmc/host/renesas_sdhi_core.c
index 31a351a20dc0..7e2a75c4f36f 100644
--- a/drivers/mmc/host/renesas_sdhi_core.c
+++ b/drivers/mmc/host/renesas_sdhi_core.c
@@ -723,6 +723,13 @@ int renesas_sdhi_probe(struct platform_device *pdev,
host->ops.start_signal_voltage_switch =
renesas_sdhi_start_signal_voltage_switch;
host->sdcard_irq_setbit_mask = TMIO_STAT_ALWAYS_SET_27;
+
+ /* SDR and HS200/400 registers requires HW reset */
+ if (of_data && of_data->scc_offset) {
+ priv->scc_ctl = host->ctl + of_data->scc_offset;
+ host->mmc->caps |= MMC_CAP_HW_RESET;
+ host->hw_reset = renesas_sdhi_hw_reset;
+ }
}
/* Orginally registers were 16 bit apart, could be 32 or 64 nowadays */
@@ -775,8 +782,6 @@ int renesas_sdhi_probe(struct platform_device *pdev,
const struct renesas_sdhi_scc *taps = of_data->taps;
bool hit = false;
- host->mmc->caps |= MMC_CAP_HW_RESET;
-
for (i = 0; i < of_data->taps_num; i++) {
if (taps[i].clk_rate == 0 ||
taps[i].clk_rate == host->mmc->f_max) {
@@ -789,12 +794,10 @@ int renesas_sdhi_probe(struct platform_device *pdev,
if (!hit)
dev_warn(&host->pdev->dev, "Unknown clock rate for SDR104\n");
- priv->scc_ctl = host->ctl + of_data->scc_offset;
host->init_tuning = renesas_sdhi_init_tuning;
host->prepare_tuning = renesas_sdhi_prepare_tuning;
host->select_tuning = renesas_sdhi_select_tuning;
host->check_scc_error = renesas_sdhi_check_scc_error;
- host->hw_reset = renesas_sdhi_hw_reset;
host->prepare_hs400_tuning =
renesas_sdhi_prepare_hs400_tuning;
host->hs400_downgrade = renesas_sdhi_disable_scc;
--
2.19.2
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode
2019-01-29 5:40 [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode marek.vasut
@ 2019-01-29 8:01 ` Wolfram Sang
2019-01-29 8:05 ` Ulf Hansson
2019-01-30 9:20 ` Ulf Hansson
2 siblings, 0 replies; 6+ messages in thread
From: Wolfram Sang @ 2019-01-29 8:01 UTC (permalink / raw)
To: marek.vasut
Cc: linux-mmc, Takeshi Saito, Marek Vasut, Niklas Söderlund,
Simon Horman, Ulf Hansson, Wolfram Sang, linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 1504 bytes --]
On Tue, Jan 29, 2019 at 06:40:39AM +0100, marek.vasut@gmail.com wrote:
> From: Takeshi Saito <takeshi.saito.xv@renesas.com>
>
> This fixes card initialization failure in high speed mode.
>
> If U-Boot uses SDR or HS200/400 mode before starting Linux and Linux
> DT does not enable SDR/HS200/HS400 mode, card initialization fails in
> high speed mode.
>
> It is necessary to initialize SCC registers during card initialization
> phase. HW reset function is registered only for a port with either of
> SDR/HS200/HS400 properties in device tree. If SDR/HS200/HS400 properties
> are not present in device tree, SCC registers will not be reset. In SoC
> that support SCC registers, HW reset function should be registered
> regardless of the configuration of device tree.
>
> Reproduction procedure:
> - Use U-Boot that support MMC HS200/400 mode.
> - Delete HS200/HS400 properties in device tree.
> (Delete mmc-hs200-1_8v and mmc-hs400-1_8v)
> - MMC port works high speed mode and all commands fail.
>
> Signed-off-by: Takeshi Saito <takeshi.saito.xv@renesas.com>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> Cc: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> Cc: Simon Horman <horms+renesas@verge.net.au>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Cc: linux-renesas-soc@vger.kernel.org
> To: linux-mmc@vger.kernel.org
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode
2019-01-29 5:40 [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode marek.vasut
2019-01-29 8:01 ` Wolfram Sang
@ 2019-01-29 8:05 ` Ulf Hansson
2019-01-29 9:56 ` Marek Vasut
2019-01-30 9:20 ` Ulf Hansson
2 siblings, 1 reply; 6+ messages in thread
From: Ulf Hansson @ 2019-01-29 8:05 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-mmc, Takeshi Saito, Marek Vasut, Niklas Söderlund,
Simon Horman, Wolfram Sang, Linux-Renesas
On Tue, 29 Jan 2019 at 06:40, <marek.vasut@gmail.com> wrote:
>
> From: Takeshi Saito <takeshi.saito.xv@renesas.com>
>
> This fixes card initialization failure in high speed mode.
>
> If U-Boot uses SDR or HS200/400 mode before starting Linux and Linux
> DT does not enable SDR/HS200/HS400 mode, card initialization fails in
> high speed mode.
>
> It is necessary to initialize SCC registers during card initialization
> phase. HW reset function is registered only for a port with either of
> SDR/HS200/HS400 properties in device tree. If SDR/HS200/HS400 properties
> are not present in device tree, SCC registers will not be reset. In SoC
> that support SCC registers, HW reset function should be registered
> regardless of the configuration of device tree.
>
> Reproduction procedure:
> - Use U-Boot that support MMC HS200/400 mode.
> - Delete HS200/HS400 properties in device tree.
> (Delete mmc-hs200-1_8v and mmc-hs400-1_8v)
> - MMC port works high speed mode and all commands fail.
>
> Signed-off-by: Takeshi Saito <takeshi.saito.xv@renesas.com>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> Cc: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> Cc: Simon Horman <horms+renesas@verge.net.au>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Cc: linux-renesas-soc@vger.kernel.org
> To: linux-mmc@vger.kernel.org
Is this a regression? Should it be tagged for stable?
Kind regards
Uffe
> --
> NOTE: Marek: - Reworded commit message
> - Updated patch to next/master
> ---
> drivers/mmc/host/renesas_sdhi_core.c | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mmc/host/renesas_sdhi_core.c b/drivers/mmc/host/renesas_sdhi_core.c
> index 31a351a20dc0..7e2a75c4f36f 100644
> --- a/drivers/mmc/host/renesas_sdhi_core.c
> +++ b/drivers/mmc/host/renesas_sdhi_core.c
> @@ -723,6 +723,13 @@ int renesas_sdhi_probe(struct platform_device *pdev,
> host->ops.start_signal_voltage_switch =
> renesas_sdhi_start_signal_voltage_switch;
> host->sdcard_irq_setbit_mask = TMIO_STAT_ALWAYS_SET_27;
> +
> + /* SDR and HS200/400 registers requires HW reset */
> + if (of_data && of_data->scc_offset) {
> + priv->scc_ctl = host->ctl + of_data->scc_offset;
> + host->mmc->caps |= MMC_CAP_HW_RESET;
> + host->hw_reset = renesas_sdhi_hw_reset;
> + }
> }
>
> /* Orginally registers were 16 bit apart, could be 32 or 64 nowadays */
> @@ -775,8 +782,6 @@ int renesas_sdhi_probe(struct platform_device *pdev,
> const struct renesas_sdhi_scc *taps = of_data->taps;
> bool hit = false;
>
> - host->mmc->caps |= MMC_CAP_HW_RESET;
> -
> for (i = 0; i < of_data->taps_num; i++) {
> if (taps[i].clk_rate == 0 ||
> taps[i].clk_rate == host->mmc->f_max) {
> @@ -789,12 +794,10 @@ int renesas_sdhi_probe(struct platform_device *pdev,
> if (!hit)
> dev_warn(&host->pdev->dev, "Unknown clock rate for SDR104\n");
>
> - priv->scc_ctl = host->ctl + of_data->scc_offset;
> host->init_tuning = renesas_sdhi_init_tuning;
> host->prepare_tuning = renesas_sdhi_prepare_tuning;
> host->select_tuning = renesas_sdhi_select_tuning;
> host->check_scc_error = renesas_sdhi_check_scc_error;
> - host->hw_reset = renesas_sdhi_hw_reset;
> host->prepare_hs400_tuning =
> renesas_sdhi_prepare_hs400_tuning;
> host->hs400_downgrade = renesas_sdhi_disable_scc;
> --
> 2.19.2
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode
2019-01-29 8:05 ` Ulf Hansson
@ 2019-01-29 9:56 ` Marek Vasut
2019-01-29 16:31 ` Wolfram Sang
0 siblings, 1 reply; 6+ messages in thread
From: Marek Vasut @ 2019-01-29 9:56 UTC (permalink / raw)
To: Ulf Hansson
Cc: linux-mmc, Takeshi Saito, Marek Vasut, Niklas Söderlund,
Simon Horman, Wolfram Sang, Linux-Renesas
On 1/29/19 9:05 AM, Ulf Hansson wrote:
> On Tue, 29 Jan 2019 at 06:40, <marek.vasut@gmail.com> wrote:
>>
>> From: Takeshi Saito <takeshi.saito.xv@renesas.com>
>>
>> This fixes card initialization failure in high speed mode.
>>
>> If U-Boot uses SDR or HS200/400 mode before starting Linux and Linux
>> DT does not enable SDR/HS200/HS400 mode, card initialization fails in
>> high speed mode.
>>
>> It is necessary to initialize SCC registers during card initialization
>> phase. HW reset function is registered only for a port with either of
>> SDR/HS200/HS400 properties in device tree. If SDR/HS200/HS400 properties
>> are not present in device tree, SCC registers will not be reset. In SoC
>> that support SCC registers, HW reset function should be registered
>> regardless of the configuration of device tree.
>>
>> Reproduction procedure:
>> - Use U-Boot that support MMC HS200/400 mode.
>> - Delete HS200/HS400 properties in device tree.
>> (Delete mmc-hs200-1_8v and mmc-hs400-1_8v)
>> - MMC port works high speed mode and all commands fail.
>>
>> Signed-off-by: Takeshi Saito <takeshi.saito.xv@renesas.com>
>> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
>> Cc: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
>> Cc: Simon Horman <horms+renesas@verge.net.au>
>> Cc: Ulf Hansson <ulf.hansson@linaro.org>
>> Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
>> Cc: linux-renesas-soc@vger.kernel.org
>> To: linux-mmc@vger.kernel.org
>
> Is this a regression?
No, because this was broken since the beginning. But U-Boot HS200/HS400
support is now, so this bug was hidden.
> Should it be tagged for stable?
I am tempted to say yes.
--
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode
2019-01-29 9:56 ` Marek Vasut
@ 2019-01-29 16:31 ` Wolfram Sang
0 siblings, 0 replies; 6+ messages in thread
From: Wolfram Sang @ 2019-01-29 16:31 UTC (permalink / raw)
To: Marek Vasut
Cc: Ulf Hansson, linux-mmc, Takeshi Saito, Marek Vasut,
Niklas Söderlund, Simon Horman, Wolfram Sang, Linux-Renesas
[-- Attachment #1: Type: text/plain, Size: 85 bytes --]
> > Should it be tagged for stable?
>
> I am tempted to say yes.
I agree.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode
2019-01-29 5:40 [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode marek.vasut
2019-01-29 8:01 ` Wolfram Sang
2019-01-29 8:05 ` Ulf Hansson
@ 2019-01-30 9:20 ` Ulf Hansson
2 siblings, 0 replies; 6+ messages in thread
From: Ulf Hansson @ 2019-01-30 9:20 UTC (permalink / raw)
To: Marek Vasut
Cc: linux-mmc, Takeshi Saito, Marek Vasut, Niklas Söderlund,
Simon Horman, Wolfram Sang, Linux-Renesas
On Tue, 29 Jan 2019 at 06:40, <marek.vasut@gmail.com> wrote:
>
> From: Takeshi Saito <takeshi.saito.xv@renesas.com>
>
> This fixes card initialization failure in high speed mode.
>
> If U-Boot uses SDR or HS200/400 mode before starting Linux and Linux
> DT does not enable SDR/HS200/HS400 mode, card initialization fails in
> high speed mode.
>
> It is necessary to initialize SCC registers during card initialization
> phase. HW reset function is registered only for a port with either of
> SDR/HS200/HS400 properties in device tree. If SDR/HS200/HS400 properties
> are not present in device tree, SCC registers will not be reset. In SoC
> that support SCC registers, HW reset function should be registered
> regardless of the configuration of device tree.
>
> Reproduction procedure:
> - Use U-Boot that support MMC HS200/400 mode.
> - Delete HS200/HS400 properties in device tree.
> (Delete mmc-hs200-1_8v and mmc-hs400-1_8v)
> - MMC port works high speed mode and all commands fail.
>
> Signed-off-by: Takeshi Saito <takeshi.saito.xv@renesas.com>
> Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
> Cc: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> Cc: Simon Horman <horms+renesas@verge.net.au>
> Cc: Ulf Hansson <ulf.hansson@linaro.org>
> Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>
> Cc: linux-renesas-soc@vger.kernel.org
> To: linux-mmc@vger.kernel.org
Applied for next and added a stable tag, thanks!
Kind regards
Uffe
> --
> NOTE: Marek: - Reworded commit message
> - Updated patch to next/master
> ---
> drivers/mmc/host/renesas_sdhi_core.c | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mmc/host/renesas_sdhi_core.c b/drivers/mmc/host/renesas_sdhi_core.c
> index 31a351a20dc0..7e2a75c4f36f 100644
> --- a/drivers/mmc/host/renesas_sdhi_core.c
> +++ b/drivers/mmc/host/renesas_sdhi_core.c
> @@ -723,6 +723,13 @@ int renesas_sdhi_probe(struct platform_device *pdev,
> host->ops.start_signal_voltage_switch =
> renesas_sdhi_start_signal_voltage_switch;
> host->sdcard_irq_setbit_mask = TMIO_STAT_ALWAYS_SET_27;
> +
> + /* SDR and HS200/400 registers requires HW reset */
> + if (of_data && of_data->scc_offset) {
> + priv->scc_ctl = host->ctl + of_data->scc_offset;
> + host->mmc->caps |= MMC_CAP_HW_RESET;
> + host->hw_reset = renesas_sdhi_hw_reset;
> + }
> }
>
> /* Orginally registers were 16 bit apart, could be 32 or 64 nowadays */
> @@ -775,8 +782,6 @@ int renesas_sdhi_probe(struct platform_device *pdev,
> const struct renesas_sdhi_scc *taps = of_data->taps;
> bool hit = false;
>
> - host->mmc->caps |= MMC_CAP_HW_RESET;
> -
> for (i = 0; i < of_data->taps_num; i++) {
> if (taps[i].clk_rate == 0 ||
> taps[i].clk_rate == host->mmc->f_max) {
> @@ -789,12 +794,10 @@ int renesas_sdhi_probe(struct platform_device *pdev,
> if (!hit)
> dev_warn(&host->pdev->dev, "Unknown clock rate for SDR104\n");
>
> - priv->scc_ctl = host->ctl + of_data->scc_offset;
> host->init_tuning = renesas_sdhi_init_tuning;
> host->prepare_tuning = renesas_sdhi_prepare_tuning;
> host->select_tuning = renesas_sdhi_select_tuning;
> host->check_scc_error = renesas_sdhi_check_scc_error;
> - host->hw_reset = renesas_sdhi_hw_reset;
> host->prepare_hs400_tuning =
> renesas_sdhi_prepare_hs400_tuning;
> host->hs400_downgrade = renesas_sdhi_disable_scc;
> --
> 2.19.2
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-01-30 9:21 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-29 5:40 [PATCH] mmc: renesas_sdhi: Fix card initialization failure in high speed mode marek.vasut
2019-01-29 8:01 ` Wolfram Sang
2019-01-29 8:05 ` Ulf Hansson
2019-01-29 9:56 ` Marek Vasut
2019-01-29 16:31 ` Wolfram Sang
2019-01-30 9:20 ` Ulf Hansson
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).