Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
MIME-Version: 1.0
References: <20210624025647.101387-1-reniuschengl@gmail.com> <CAPDyKFoQNDTe8__eUmmbE=98HgjERYudzjyRu1fr2ZCyWXuqew@mail.gmail.com>
In-Reply-To: <CAPDyKFoQNDTe8__eUmmbE=98HgjERYudzjyRu1fr2ZCyWXuqew@mail.gmail.com>
From:   Renius Chen <reniuschengl@gmail.com>
Date:   Thu, 1 Jul 2021 11:41:41 +0800
Message-ID: <CAJU4x8tAoHebcynA1+UdAiqF4kRYgzEpu2qpzcikfUSKxjL3AQ@mail.gmail.com>
Subject: Re: [PATCH] mmc: sdhci-pci-gli: Finetune GL9763E L1 Entry Delay
To:     Ulf Hansson <ulf.hansson@linaro.org>
Cc:     Adrian Hunter <adrian.hunter@intel.com>,
        linux-mmc <linux-mmc@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Ben Chuang <Ben.Chuang@genesyslogic.com.tw>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Ulf Hansson <ulf.hansson@linaro.org> =E6=96=BC 2021=E5=B9=B46=E6=9C=8830=E6=
=97=A5 =E9=80=B1=E4=B8=89 =E4=B8=8B=E5=8D=8811:16=E5=AF=AB=E9=81=93=EF=BC=
=9A
>
> On Thu, 24 Jun 2021 at 04:56, Renius Chen <reniuschengl@gmail.com> wrote:
> >
> > Finetune the L1 entry delay to 20us for better balance of performance a=
nd
> > battery life.
> >
> > Signed-off-by: Renius Chen <reniuschengl@gmail.com>
> > ---
> >  drivers/mmc/host/sdhci-pci-gli.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/mmc/host/sdhci-pci-gli.c b/drivers/mmc/host/sdhci-=
pci-gli.c
> > index 302a7579a9b3..4e3c0561354d 100644
> > --- a/drivers/mmc/host/sdhci-pci-gli.c
> > +++ b/drivers/mmc/host/sdhci-pci-gli.c
> > @@ -90,7 +90,7 @@
> >
> >  #define PCIE_GLI_9763E_CFG2      0x8A4
> >  #define   GLI_9763E_CFG2_L1DLY     GENMASK(28, 19)
> > -#define   GLI_9763E_CFG2_L1DLY_MID 0x54
> > +#define   GLI_9763E_CFG2_L1DLY_MID 0x50
>
> We just changed from 21us to 20us. Really, how big of a difference can
> this make?
>
> Moreover, I suppose the difference is related to the running use case. No=
?
>
> If you really want this, at least I want an ack from Ben for it, then
> let's be done with it.
>
> Kind regards
> Uffe
>
Yes, according to the result of our customer's PLT test for battery life,
It will pass the test with 20us and will not pass the test with 21us.
I'll ask Ben for acking this, thank you.

> >
> >  #define PCIE_GLI_9763E_MMC_CTRL  0x960
> >  #define   GLI_9763E_HS400_SLOW     BIT(3)
> > @@ -810,7 +810,7 @@ static void gli_set_gl9763e(struct sdhci_pci_slot *=
slot)
> >
> >         pci_read_config_dword(pdev, PCIE_GLI_9763E_CFG2, &value);
> >         value &=3D ~GLI_9763E_CFG2_L1DLY;
> > -       /* set ASPM L1 entry delay to 21us */
> > +       /* set ASPM L1 entry delay to 20us */
> >         value |=3D FIELD_PREP(GLI_9763E_CFG2_L1DLY, GLI_9763E_CFG2_L1DL=
Y_MID);
> >         pci_write_config_dword(pdev, PCIE_GLI_9763E_CFG2, value);
> >
> > --
> > 2.27.0
> >