* [v1,1/1] mmc: core: Add HS400es support in retune and initialization flow
@ 2023-12-01 3:05 Mengqi Zhang
2023-12-07 13:37 ` Ulf Hansson
0 siblings, 1 reply; 3+ messages in thread
From: Mengqi Zhang @ 2023-12-01 3:05 UTC (permalink / raw)
To: ulf.hansson, chaotian.jing, matthias.bgg, mengqi.zhang,
wenbin.mei
Cc: yangyingliang, adrian.hunter, avri.altman, vincent.whitchurch,
linux-mmc, linux-arm-kernel, linux-mediatek, linux-kernel
In HS400es mode, when we enter mmc_retune(), we will be forced to
the HS400 mode instead of HS400es mode. So we decided to add support
for HS400es in the mmc_retune() process.
Additionally, during the initialization to the HS400es stage, add
hs400 tuning as an optional process.
Signed-off-by: Mengqi Zhang <mengqi.zhang@mediatek.com>
---
drivers/mmc/core/host.c | 11 ++++++++++-
drivers/mmc/core/mmc.c | 10 +++++++++-
2 files changed, 19 insertions(+), 2 deletions(-)
diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
index 096093f7be00..4bddbbb3d539 100644
--- a/drivers/mmc/core/host.c
+++ b/drivers/mmc/core/host.c
@@ -179,7 +179,7 @@ EXPORT_SYMBOL(mmc_retune_release);
int mmc_retune(struct mmc_host *host)
{
bool return_to_hs400 = false;
- int err;
+ int err = 0;
if (host->retune_now)
host->retune_now = 0;
@@ -194,6 +194,15 @@ int mmc_retune(struct mmc_host *host)
host->doing_retune = 1;
if (host->ios.timing == MMC_TIMING_MMC_HS400) {
+ if (host->ios.enhanced_strobe) {
+ if (host->ops->execute_hs400_tuning) {
+ mmc_retune_disable(host);
+ err = host->ops->execute_hs400_tuning(host, host->card);
+ mmc_retune_enable(host);
+ }
+ goto out;
+ }
+
err = mmc_hs400_to_hs200(host->card);
if (err)
goto out;
diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
index 705942edacc6..9760eea2c104 100644
--- a/drivers/mmc/core/mmc.c
+++ b/drivers/mmc/core/mmc.c
@@ -1823,7 +1823,15 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
if (err)
goto free_card;
- } else if (!mmc_card_hs400es(card)) {
+ } else if (mmc_card_hs400es(card)) {
+ if (host->ops->execute_hs400_tuning) {
+ mmc_retune_disable(host);
+ err = host->ops->execute_hs400_tuning(host, card);
+ mmc_retune_enable(host);
+ if (err)
+ goto free_card;
+ }
+ } else {
/* Select the desired bus width optionally */
err = mmc_select_bus_width(card);
if (err > 0 && mmc_card_hs(card)) {
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [v1,1/1] mmc: core: Add HS400es support in retune and initialization flow
2023-12-01 3:05 [v1,1/1] mmc: core: Add HS400es support in retune and initialization flow Mengqi Zhang
@ 2023-12-07 13:37 ` Ulf Hansson
2023-12-21 9:00 ` Mengqi Zhang (张梦琦)
0 siblings, 1 reply; 3+ messages in thread
From: Ulf Hansson @ 2023-12-07 13:37 UTC (permalink / raw)
To: Mengqi Zhang
Cc: chaotian.jing, matthias.bgg, wenbin.mei, yangyingliang,
adrian.hunter, avri.altman, vincent.whitchurch, linux-mmc,
linux-arm-kernel, linux-mediatek, linux-kernel
On Fri, 1 Dec 2023 at 04:06, Mengqi Zhang <mengqi.zhang@mediatek.com> wrote:
>
> In HS400es mode, when we enter mmc_retune(), we will be forced to
> the HS400 mode instead of HS400es mode. So we decided to add support
> for HS400es in the mmc_retune() process.
Hmm, I was under the impression that we really don't need to do
re-tuning at all, when using HS400es mode.
Are we calling mmc_retune_enable() from a place that we shouldn't?
>
> Additionally, during the initialization to the HS400es stage, add
> hs400 tuning as an optional process.
>
> Signed-off-by: Mengqi Zhang <mengqi.zhang@mediatek.com>
Kind regards
Uffe
> ---
> drivers/mmc/core/host.c | 11 ++++++++++-
> drivers/mmc/core/mmc.c | 10 +++++++++-
> 2 files changed, 19 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
> index 096093f7be00..4bddbbb3d539 100644
> --- a/drivers/mmc/core/host.c
> +++ b/drivers/mmc/core/host.c
> @@ -179,7 +179,7 @@ EXPORT_SYMBOL(mmc_retune_release);
> int mmc_retune(struct mmc_host *host)
> {
> bool return_to_hs400 = false;
> - int err;
> + int err = 0;
>
> if (host->retune_now)
> host->retune_now = 0;
> @@ -194,6 +194,15 @@ int mmc_retune(struct mmc_host *host)
> host->doing_retune = 1;
>
> if (host->ios.timing == MMC_TIMING_MMC_HS400) {
> + if (host->ios.enhanced_strobe) {
> + if (host->ops->execute_hs400_tuning) {
> + mmc_retune_disable(host);
> + err = host->ops->execute_hs400_tuning(host, host->card);
> + mmc_retune_enable(host);
> + }
> + goto out;
> + }
> +
> err = mmc_hs400_to_hs200(host->card);
> if (err)
> goto out;
> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> index 705942edacc6..9760eea2c104 100644
> --- a/drivers/mmc/core/mmc.c
> +++ b/drivers/mmc/core/mmc.c
> @@ -1823,7 +1823,15 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
> if (err)
> goto free_card;
>
> - } else if (!mmc_card_hs400es(card)) {
> + } else if (mmc_card_hs400es(card)) {
> + if (host->ops->execute_hs400_tuning) {
> + mmc_retune_disable(host);
> + err = host->ops->execute_hs400_tuning(host, card);
> + mmc_retune_enable(host);
> + if (err)
> + goto free_card;
> + }
> + } else {
> /* Select the desired bus width optionally */
> err = mmc_select_bus_width(card);
> if (err > 0 && mmc_card_hs(card)) {
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [v1,1/1] mmc: core: Add HS400es support in retune and initialization flow
2023-12-07 13:37 ` Ulf Hansson
@ 2023-12-21 9:00 ` Mengqi Zhang (张梦琦)
0 siblings, 0 replies; 3+ messages in thread
From: Mengqi Zhang (张梦琦) @ 2023-12-21 9:00 UTC (permalink / raw)
To: ulf.hansson@linaro.org
Cc: linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-mmc@vger.kernel.org, Wenbin Mei (梅文彬),
avri.altman@wdc.com, Chaotian Jing (井朝天),
linux-arm-kernel@lists.infradead.org, matthias.bgg@gmail.com,
vincent.whitchurch@axis.com, adrian.hunter@intel.com,
yangyingliang@huawei.com
On Thu, 2023-12-07 at 14:37 +0100, Ulf Hansson wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> On Fri, 1 Dec 2023 at 04:06, Mengqi Zhang <mengqi.zhang@mediatek.com
> > wrote:
> >
> > In HS400es mode, when we enter mmc_retune(), we will be forced to
> > the HS400 mode instead of HS400es mode. So we decided to add
> support
> > for HS400es in the mmc_retune() process.
>
> Hmm, I was under the impression that we really don't need to do
> re-tuning at all, when using HS400es mode.
>
> Are we calling mmc_retune_enable() from a place that we shouldn't?
>
> >
> > Additionally, during the initialization to the HS400es stage, add
> > hs400 tuning as an optional process.
> >
> > Signed-off-by: Mengqi Zhang <mengqi.zhang@mediatek.com>
>
> Kind regards
> Uffe
>
Thanks for your review.
I will follow your comment in next change.
> > ---
> > drivers/mmc/core/host.c | 11 ++++++++++-
> > drivers/mmc/core/mmc.c | 10 +++++++++-
> > 2 files changed, 19 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
> > index 096093f7be00..4bddbbb3d539 100644
> > --- a/drivers/mmc/core/host.c
> > +++ b/drivers/mmc/core/host.c
> > @@ -179,7 +179,7 @@ EXPORT_SYMBOL(mmc_retune_release);
> > int mmc_retune(struct mmc_host *host)
> > {
> > bool return_to_hs400 = false;
> > - int err;
> > + int err = 0;
> >
> > if (host->retune_now)
> > host->retune_now = 0;
> > @@ -194,6 +194,15 @@ int mmc_retune(struct mmc_host *host)
> > host->doing_retune = 1;
> >
> > if (host->ios.timing == MMC_TIMING_MMC_HS400) {
> > + if (host->ios.enhanced_strobe) {
> > + if (host->ops->execute_hs400_tuning) {
> > + mmc_retune_disable(host);
> > + err = host->ops-
> >execute_hs400_tuning(host, host->card);
> > + mmc_retune_enable(host);
> > + }
> > + goto out;
> > + }
> > +
> > err = mmc_hs400_to_hs200(host->card);
> > if (err)
> > goto out;
> > diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> > index 705942edacc6..9760eea2c104 100644
> > --- a/drivers/mmc/core/mmc.c
> > +++ b/drivers/mmc/core/mmc.c
> > @@ -1823,7 +1823,15 @@ static int mmc_init_card(struct mmc_host
> *host, u32 ocr,
> > if (err)
> > goto free_card;
> >
> > - } else if (!mmc_card_hs400es(card)) {
> > + } else if (mmc_card_hs400es(card)) {
> > + if (host->ops->execute_hs400_tuning) {
> > + mmc_retune_disable(host);
> > + err = host->ops->execute_hs400_tuning(host,
> card);
> > + mmc_retune_enable(host);
> > + if (err)
> > + goto free_card;
> > + }
> > + } else {
> > /* Select the desired bus width optionally */
> > err = mmc_select_bus_width(card);
> > if (err > 0 && mmc_card_hs(card)) {
> > --
> > 2.25.1
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-12-21 9:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-12-01 3:05 [v1,1/1] mmc: core: Add HS400es support in retune and initialization flow Mengqi Zhang
2023-12-07 13:37 ` Ulf Hansson
2023-12-21 9:00 ` Mengqi Zhang (张梦琦)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox