* [PATCH] mmc: sprd: Fix eMMC init failure after hw reset
@ 2023-12-04 6:49 Wenchao Chen
2023-12-07 13:58 ` Ulf Hansson
0 siblings, 1 reply; 6+ messages in thread
From: Wenchao Chen @ 2023-12-04 6:49 UTC (permalink / raw)
To: ulf.hansson, zhang.lyra, orsonzhai, baolin.wang
Cc: linux-mmc, linux-kernel, wenchao.chen666, zhenxiong.lai,
yuelin.tang, Wenchao Chen
Some eMMC devices that do not close the auto clk gate
after hw reset will cause eMMC initialization to fail.
Signed-off-by: Wenchao Chen <wenchao.chen@unisoc.com>
---
drivers/mmc/host/sdhci-sprd.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/mmc/host/sdhci-sprd.c b/drivers/mmc/host/sdhci-sprd.c
index 6b8a57e2d20f..bed57a1c64b5 100644
--- a/drivers/mmc/host/sdhci-sprd.c
+++ b/drivers/mmc/host/sdhci-sprd.c
@@ -239,15 +239,19 @@ static inline void _sdhci_sprd_set_clock(struct sdhci_host *host,
div = ((div & 0x300) >> 2) | ((div & 0xFF) << 8);
sdhci_enable_clk(host, div);
+ val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
+ mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN | SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
/* Enable CLK_AUTO when the clock is greater than 400K. */
if (clk > 400000) {
- val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
- mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN |
- SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
if (mask != (val & mask)) {
val |= mask;
sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
}
+ } else {
+ if (val & mask) {
+ val &= ~mask;
+ sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
+ }
}
}
--
2.17.1
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] mmc: sprd: Fix eMMC init failure after hw reset
2023-12-04 6:49 [PATCH] mmc: sprd: Fix eMMC init failure after hw reset Wenchao Chen
@ 2023-12-07 13:58 ` Ulf Hansson
2023-12-08 11:52 ` Wenchao Chen
0 siblings, 1 reply; 6+ messages in thread
From: Ulf Hansson @ 2023-12-07 13:58 UTC (permalink / raw)
To: Wenchao Chen
Cc: zhang.lyra, orsonzhai, baolin.wang, linux-mmc, linux-kernel,
wenchao.chen666, zhenxiong.lai, yuelin.tang
On Mon, 4 Dec 2023 at 07:50, Wenchao Chen <wenchao.chen@unisoc.com> wrote:
>
> Some eMMC devices that do not close the auto clk gate
> after hw reset will cause eMMC initialization to fail.
>
> Signed-off-by: Wenchao Chen <wenchao.chen@unisoc.com>
I assume we want this tagged for stable kernels too, but do we have a
corresponding fixes commit that we can point out?
Kind regards
Uffe
> ---
> drivers/mmc/host/sdhci-sprd.c | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci-sprd.c b/drivers/mmc/host/sdhci-sprd.c
> index 6b8a57e2d20f..bed57a1c64b5 100644
> --- a/drivers/mmc/host/sdhci-sprd.c
> +++ b/drivers/mmc/host/sdhci-sprd.c
> @@ -239,15 +239,19 @@ static inline void _sdhci_sprd_set_clock(struct sdhci_host *host,
> div = ((div & 0x300) >> 2) | ((div & 0xFF) << 8);
> sdhci_enable_clk(host, div);
>
> + val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> + mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN | SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> /* Enable CLK_AUTO when the clock is greater than 400K. */
> if (clk > 400000) {
> - val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> - mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN |
> - SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> if (mask != (val & mask)) {
> val |= mask;
> sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> }
> + } else {
> + if (val & mask) {
> + val &= ~mask;
> + sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> + }
> }
> }
>
> --
> 2.17.1
>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] mmc: sprd: Fix eMMC init failure after hw reset
2023-12-07 13:58 ` Ulf Hansson
@ 2023-12-08 11:52 ` Wenchao Chen
2023-12-12 2:57 ` Baolin Wang
0 siblings, 1 reply; 6+ messages in thread
From: Wenchao Chen @ 2023-12-08 11:52 UTC (permalink / raw)
To: Ulf Hansson
Cc: Wenchao Chen, zhang.lyra, orsonzhai, baolin.wang, linux-mmc,
linux-kernel, zhenxiong.lai, yuelin.tang
On Thu, 7 Dec 2023 at 21:59, Ulf Hansson <ulf.hansson@linaro.org> wrote:
>
> On Mon, 4 Dec 2023 at 07:50, Wenchao Chen <wenchao.chen@unisoc.com> wrote:
> >
> > Some eMMC devices that do not close the auto clk gate
> > after hw reset will cause eMMC initialization to fail.
> >
> > Signed-off-by: Wenchao Chen <wenchao.chen@unisoc.com>
>
> I assume we want this tagged for stable kernels too, but do we have a
> corresponding fixes commit that we can point out?
>
> Kind regards
> Uffe
>
Hi Uffe
Sorry, I forgot to add fixes commit.
Fixes: ff874dbc4f86 ("mmc: sdhci-sprd: Disable CLK_AUTO when the clock
is less than 400K")
Thanks
> > ---
> > drivers/mmc/host/sdhci-sprd.c | 10 +++++++---
> > 1 file changed, 7 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/mmc/host/sdhci-sprd.c b/drivers/mmc/host/sdhci-sprd.c
> > index 6b8a57e2d20f..bed57a1c64b5 100644
> > --- a/drivers/mmc/host/sdhci-sprd.c
> > +++ b/drivers/mmc/host/sdhci-sprd.c
> > @@ -239,15 +239,19 @@ static inline void _sdhci_sprd_set_clock(struct sdhci_host *host,
> > div = ((div & 0x300) >> 2) | ((div & 0xFF) << 8);
> > sdhci_enable_clk(host, div);
> >
> > + val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> > + mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN | SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> > /* Enable CLK_AUTO when the clock is greater than 400K. */
> > if (clk > 400000) {
> > - val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> > - mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN |
> > - SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> > if (mask != (val & mask)) {
> > val |= mask;
> > sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> > }
> > + } else {
> > + if (val & mask) {
> > + val &= ~mask;
> > + sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> > + }
> > }
> > }
> >
> > --
> > 2.17.1
> >
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] mmc: sprd: Fix eMMC init failure after hw reset
2023-12-08 11:52 ` Wenchao Chen
@ 2023-12-12 2:57 ` Baolin Wang
2024-01-03 1:40 ` Wenchao Chen
0 siblings, 1 reply; 6+ messages in thread
From: Baolin Wang @ 2023-12-12 2:57 UTC (permalink / raw)
To: Wenchao Chen, Ulf Hansson
Cc: Wenchao Chen, zhang.lyra, orsonzhai, linux-mmc, linux-kernel,
zhenxiong.lai, yuelin.tang
On 12/8/2023 7:52 PM, Wenchao Chen wrote:
> On Thu, 7 Dec 2023 at 21:59, Ulf Hansson <ulf.hansson@linaro.org> wrote:
>>
>> On Mon, 4 Dec 2023 at 07:50, Wenchao Chen <wenchao.chen@unisoc.com> wrote:
>>>
>>> Some eMMC devices that do not close the auto clk gate
>>> after hw reset will cause eMMC initialization to fail.
>>>
>>> Signed-off-by: Wenchao Chen <wenchao.chen@unisoc.com>
>>
>> I assume we want this tagged for stable kernels too, but do we have a
>> corresponding fixes commit that we can point out?
>>
>> Kind regards
>> Uffe
>>
>
> Hi Uffe
> Sorry, I forgot to add fixes commit.
>
> Fixes: ff874dbc4f86 ("mmc: sdhci-sprd: Disable CLK_AUTO when the clock
> is less than 400K")
With the Fixes tag, looks goo to me.
Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
>>> ---
>>> drivers/mmc/host/sdhci-sprd.c | 10 +++++++---
>>> 1 file changed, 7 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/drivers/mmc/host/sdhci-sprd.c b/drivers/mmc/host/sdhci-sprd.c
>>> index 6b8a57e2d20f..bed57a1c64b5 100644
>>> --- a/drivers/mmc/host/sdhci-sprd.c
>>> +++ b/drivers/mmc/host/sdhci-sprd.c
>>> @@ -239,15 +239,19 @@ static inline void _sdhci_sprd_set_clock(struct sdhci_host *host,
>>> div = ((div & 0x300) >> 2) | ((div & 0xFF) << 8);
>>> sdhci_enable_clk(host, div);
>>>
>>> + val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
>>> + mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN | SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
>>> /* Enable CLK_AUTO when the clock is greater than 400K. */
>>> if (clk > 400000) {
>>> - val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
>>> - mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN |
>>> - SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
>>> if (mask != (val & mask)) {
>>> val |= mask;
>>> sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
>>> }
>>> + } else {
>>> + if (val & mask) {
>>> + val &= ~mask;
>>> + sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
>>> + }
>>> }
>>> }
>>>
>>> --
>>> 2.17.1
>>>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] mmc: sprd: Fix eMMC init failure after hw reset
2023-12-12 2:57 ` Baolin Wang
@ 2024-01-03 1:40 ` Wenchao Chen
2024-01-03 10:27 ` Ulf Hansson
0 siblings, 1 reply; 6+ messages in thread
From: Wenchao Chen @ 2024-01-03 1:40 UTC (permalink / raw)
To: Baolin Wang
Cc: Ulf Hansson, Wenchao Chen, zhang.lyra, orsonzhai, linux-mmc,
linux-kernel, zhenxiong.lai, yuelin.tang
Gentle ping.
Thanks,
Wenchao.Chen
On Tue, 12 Dec 2023 at 10:57, Baolin Wang <baolin.wang@linux.alibaba.com> wrote:
>
>
>
> On 12/8/2023 7:52 PM, Wenchao Chen wrote:
> > On Thu, 7 Dec 2023 at 21:59, Ulf Hansson <ulf.hansson@linaro.org> wrote:
> >>
> >> On Mon, 4 Dec 2023 at 07:50, Wenchao Chen <wenchao.chen@unisoc.com> wrote:
> >>>
> >>> Some eMMC devices that do not close the auto clk gate
> >>> after hw reset will cause eMMC initialization to fail.
> >>>
> >>> Signed-off-by: Wenchao Chen <wenchao.chen@unisoc.com>
> >>
> >> I assume we want this tagged for stable kernels too, but do we have a
> >> corresponding fixes commit that we can point out?
> >>
> >> Kind regards
> >> Uffe
> >>
> >
> > Hi Uffe
> > Sorry, I forgot to add fixes commit.
> >
> > Fixes: ff874dbc4f86 ("mmc: sdhci-sprd: Disable CLK_AUTO when the clock
> > is less than 400K")
>
> With the Fixes tag, looks goo to me.
> Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
>
> >>> ---
> >>> drivers/mmc/host/sdhci-sprd.c | 10 +++++++---
> >>> 1 file changed, 7 insertions(+), 3 deletions(-)
> >>>
> >>> diff --git a/drivers/mmc/host/sdhci-sprd.c b/drivers/mmc/host/sdhci-sprd.c
> >>> index 6b8a57e2d20f..bed57a1c64b5 100644
> >>> --- a/drivers/mmc/host/sdhci-sprd.c
> >>> +++ b/drivers/mmc/host/sdhci-sprd.c
> >>> @@ -239,15 +239,19 @@ static inline void _sdhci_sprd_set_clock(struct sdhci_host *host,
> >>> div = ((div & 0x300) >> 2) | ((div & 0xFF) << 8);
> >>> sdhci_enable_clk(host, div);
> >>>
> >>> + val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> >>> + mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN | SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> >>> /* Enable CLK_AUTO when the clock is greater than 400K. */
> >>> if (clk > 400000) {
> >>> - val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> >>> - mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN |
> >>> - SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> >>> if (mask != (val & mask)) {
> >>> val |= mask;
> >>> sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> >>> }
> >>> + } else {
> >>> + if (val & mask) {
> >>> + val &= ~mask;
> >>> + sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> >>> + }
> >>> }
> >>> }
> >>>
> >>> --
> >>> 2.17.1
> >>>
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH] mmc: sprd: Fix eMMC init failure after hw reset
2024-01-03 1:40 ` Wenchao Chen
@ 2024-01-03 10:27 ` Ulf Hansson
0 siblings, 0 replies; 6+ messages in thread
From: Ulf Hansson @ 2024-01-03 10:27 UTC (permalink / raw)
To: Wenchao Chen
Cc: Baolin Wang, Wenchao Chen, zhang.lyra, orsonzhai, linux-mmc,
linux-kernel, zhenxiong.lai, yuelin.tang
On Wed, 3 Jan 2024 at 02:41, Wenchao Chen <wenchao.chen666@gmail.com> wrote:
>
> Gentle ping.
Thanks for pinging and sorry for the delay!
Patch applied for fixes and by adding a stable tag, thanks!
Kind regards
Uffe
>
> Thanks,
> Wenchao.Chen
>
> On Tue, 12 Dec 2023 at 10:57, Baolin Wang <baolin.wang@linux.alibaba.com> wrote:
> >
> >
> >
> > On 12/8/2023 7:52 PM, Wenchao Chen wrote:
> > > On Thu, 7 Dec 2023 at 21:59, Ulf Hansson <ulf.hansson@linaro.org> wrote:
> > >>
> > >> On Mon, 4 Dec 2023 at 07:50, Wenchao Chen <wenchao.chen@unisoc.com> wrote:
> > >>>
> > >>> Some eMMC devices that do not close the auto clk gate
> > >>> after hw reset will cause eMMC initialization to fail.
> > >>>
> > >>> Signed-off-by: Wenchao Chen <wenchao.chen@unisoc.com>
> > >>
> > >> I assume we want this tagged for stable kernels too, but do we have a
> > >> corresponding fixes commit that we can point out?
> > >>
> > >> Kind regards
> > >> Uffe
> > >>
> > >
> > > Hi Uffe
> > > Sorry, I forgot to add fixes commit.
> > >
> > > Fixes: ff874dbc4f86 ("mmc: sdhci-sprd: Disable CLK_AUTO when the clock
> > > is less than 400K")
> >
> > With the Fixes tag, looks goo to me.
> > Reviewed-by: Baolin Wang <baolin.wang@linux.alibaba.com>
> >
> > >>> ---
> > >>> drivers/mmc/host/sdhci-sprd.c | 10 +++++++---
> > >>> 1 file changed, 7 insertions(+), 3 deletions(-)
> > >>>
> > >>> diff --git a/drivers/mmc/host/sdhci-sprd.c b/drivers/mmc/host/sdhci-sprd.c
> > >>> index 6b8a57e2d20f..bed57a1c64b5 100644
> > >>> --- a/drivers/mmc/host/sdhci-sprd.c
> > >>> +++ b/drivers/mmc/host/sdhci-sprd.c
> > >>> @@ -239,15 +239,19 @@ static inline void _sdhci_sprd_set_clock(struct sdhci_host *host,
> > >>> div = ((div & 0x300) >> 2) | ((div & 0xFF) << 8);
> > >>> sdhci_enable_clk(host, div);
> > >>>
> > >>> + val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> > >>> + mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN | SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> > >>> /* Enable CLK_AUTO when the clock is greater than 400K. */
> > >>> if (clk > 400000) {
> > >>> - val = sdhci_readl(host, SDHCI_SPRD_REG_32_BUSY_POSI);
> > >>> - mask = SDHCI_SPRD_BIT_OUTR_CLK_AUTO_EN |
> > >>> - SDHCI_SPRD_BIT_INNR_CLK_AUTO_EN;
> > >>> if (mask != (val & mask)) {
> > >>> val |= mask;
> > >>> sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> > >>> }
> > >>> + } else {
> > >>> + if (val & mask) {
> > >>> + val &= ~mask;
> > >>> + sdhci_writel(host, val, SDHCI_SPRD_REG_32_BUSY_POSI);
> > >>> + }
> > >>> }
> > >>> }
> > >>>
> > >>> --
> > >>> 2.17.1
> > >>>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-01-03 10:27 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-04 6:49 [PATCH] mmc: sprd: Fix eMMC init failure after hw reset Wenchao Chen
2023-12-07 13:58 ` Ulf Hansson
2023-12-08 11:52 ` Wenchao Chen
2023-12-12 2:57 ` Baolin Wang
2024-01-03 1:40 ` Wenchao Chen
2024-01-03 10:27 ` Ulf Hansson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®