linux-kernel - Re: [v1,1/1] mmc: core: Add HS400es support in retune and initialization flow

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <62ea1a73aab2d58fb0ca610cc92db9e18920f022.camel@mediatek.com>
Date: Thu, 21 Dec 2023 09:00:51 +0000
From: Mengqi Zhang (张梦琦) <Mengqi.Zhang@...iatek.com>
To: "ulf.hansson@...aro.org" <ulf.hansson@...aro.org>
CC: "linux-mediatek@...ts.infradead.org" <linux-mediatek@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-mmc@...r.kernel.org" <linux-mmc@...r.kernel.org>,
	Wenbin Mei (梅文彬) <Wenbin.Mei@...iatek.com>,
	"avri.altman@....com" <avri.altman@....com>,
	Chaotian Jing (井朝天)
	<Chaotian.Jing@...iatek.com>, "linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "matthias.bgg@...il.com"
	<matthias.bgg@...il.com>, "vincent.whitchurch@...s.com"
	<vincent.whitchurch@...s.com>, "adrian.hunter@...el.com"
	<adrian.hunter@...el.com>, "yangyingliang@...wei.com"
	<yangyingliang@...wei.com>
Subject: Re: [v1,1/1] mmc: core: Add HS400es support in retune and
 initialization flow

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@...iatek.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@...iatek.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
> >