* [PATCH] usb: dwc2: skip polling for the soft reset clear bit and add after reset delay
@ 2026-06-23 14:23 chalianis1
2026-06-29 10:56 ` Sascha Hauer
0 siblings, 1 reply; 4+ messages in thread
From: chalianis1 @ 2026-06-23 14:23 UTC (permalink / raw)
To: s.hauer; +Cc: barebox, Chali Anis
From: Chali Anis <chalianis1@gmail.com>
The RPi4 SoC do not deassert GRSTCTL_CSFTRST within the expected
window but continue to operate correctly, causing probe to fail
with ETIMEDOUT.
Per the datasheet, GRSTCTL_CSFTRST is self-clearing but requires at least
3 PHY clocks after reset before any PHY domain access. Add a 1us delay to
satisfy this requirement.
Signed-off-by: Chali Anis <chalianis1@gmail.com>
---
drivers/usb/dwc2/core.c | 13 +++++++++----
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/drivers/usb/dwc2/core.c b/drivers/usb/dwc2/core.c
index 60cc690fdbc0..6d4b7930da9e 100644
--- a/drivers/usb/dwc2/core.c
+++ b/drivers/usb/dwc2/core.c
@@ -809,11 +809,16 @@ int dwc2_core_reset(struct dwc2 *dwc2)
greset |= GRSTCTL_CSFTRST;
dwc2_writel(dwc2, greset, GRSTCTL);
- ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
- if (ret) {
- dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
+ /* Wait for at least 3 PHY Clocks */
+ udelay(1);
+
+ if (!of_machine_is_compatible("brcm,bcm2711")) {
+ ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
+ if (ret) {
+ dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
__func__);
- return ret;
+ return ret;
+ }
}
if (wait_for_host_mode)
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] usb: dwc2: skip polling for the soft reset clear bit and add after reset delay
2026-06-23 14:23 [PATCH] usb: dwc2: skip polling for the soft reset clear bit and add after reset delay chalianis1
@ 2026-06-29 10:56 ` Sascha Hauer
2026-08-26 11:54 ` chalianis1
0 siblings, 1 reply; 4+ messages in thread
From: Sascha Hauer @ 2026-06-29 10:56 UTC (permalink / raw)
To: chalianis1; +Cc: Chali Anis, barebox
On 2026-06-23 10:23, chalianis1 wrote:
> From: Chali Anis <chalianis1@gmail.com>
>
> The RPi4 SoC do not deassert GRSTCTL_CSFTRST within the expected
> window but continue to operate correctly, causing probe to fail
> with ETIMEDOUT.
Does "within the expected window" imply that there is a bigger window in
which it is de-asserted?
>
> Per the datasheet, GRSTCTL_CSFTRST is self-clearing but requires at least
> 3 PHY clocks after reset before any PHY domain access. Add a 1us delay to
> satisfy this requirement.
Which datasheet, the SoC datasheet or a DWC2 datasheet?
I'm just asking if there is a chance to not need a SoC check here.
Sascha
>
> Signed-off-by: Chali Anis <chalianis1@gmail.com>
> ---
> drivers/usb/dwc2/core.c | 13 +++++++++----
> 1 file changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/usb/dwc2/core.c b/drivers/usb/dwc2/core.c
> index 60cc690fdbc0..6d4b7930da9e 100644
> --- a/drivers/usb/dwc2/core.c
> +++ b/drivers/usb/dwc2/core.c
> @@ -809,11 +809,16 @@ int dwc2_core_reset(struct dwc2 *dwc2)
> greset |= GRSTCTL_CSFTRST;
> dwc2_writel(dwc2, greset, GRSTCTL);
>
> - ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
> - if (ret) {
> - dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
> + /* Wait for at least 3 PHY Clocks */
> + udelay(1);
> +
> + if (!of_machine_is_compatible("brcm,bcm2711")) {
> + ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
> + if (ret) {
> + dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
> __func__);
> - return ret;
> + return ret;
> + }
> }
>
> if (wait_for_host_mode)
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] usb: dwc2: skip polling for the soft reset clear bit and add after reset delay
2026-06-29 10:56 ` Sascha Hauer
@ 2026-08-26 11:54 ` chalianis1
2026-08-26 12:12 ` chalianis1
0 siblings, 1 reply; 4+ messages in thread
From: chalianis1 @ 2026-08-26 11:54 UTC (permalink / raw)
To: Sascha Hauer; +Cc: chalianis1, barebox
On 2026-06-29 10:56 +0000, Sascha Hauer wrote:
> On 2026-06-23 10:23, chalianis1 wrote:
> > From: Chali Anis <chalianis1@gmail.com>
> >
> > The RPi4 SoC do not deassert GRSTCTL_CSFTRST within the expected
> > window but continue to operate correctly, causing probe to fail
> > with ETIMEDOUT.
>
> Does "within the expected window" imply that there is a bigger window in
> which it is de-asserted?
>
What I mean is within the expected window according to the SoC datasheet
of RK3588 which I just realized that I wrongly thought that it has a DWC2.
Rockchip RK3588 TRM V1.0-Part2 20220309 page 2221
> >
> > Per the datasheet, GRSTCTL_CSFTRST is self-clearing but requires at least
> > 3 PHY clocks after reset before any PHY domain access. Add a 1us delay to
> > satisfy this requirement.
>
> Which datasheet, the SoC datasheet or a DWC2 datasheet?
>
the SOC one.
> I'm just asking if there is a chance to not need a SoC check here.
>
> Sascha
>
> >
> > Signed-off-by: Chali Anis <chalianis1@gmail.com>
> > ---
> > drivers/usb/dwc2/core.c | 13 +++++++++----
> > 1 file changed, 9 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/usb/dwc2/core.c b/drivers/usb/dwc2/core.c
> > index 60cc690fdbc0..6d4b7930da9e 100644
> > --- a/drivers/usb/dwc2/core.c
> > +++ b/drivers/usb/dwc2/core.c
> > @@ -809,11 +809,16 @@ int dwc2_core_reset(struct dwc2 *dwc2)
> > greset |= GRSTCTL_CSFTRST;
> > dwc2_writel(dwc2, greset, GRSTCTL);
> >
> > - ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
> > - if (ret) {
> > - dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
> > + /* Wait for at least 3 PHY Clocks */
> > + udelay(1);
> > +
> > + if (!of_machine_is_compatible("brcm,bcm2711")) {
> > + ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
> > + if (ret) {
> > + dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
> > __func__);
> > - return ret;
> > + return ret;
> > + }
> > }
> >
> > if (wait_for_host_mode)
> >
>
> --
> Pengutronix e.K. | |
> Steuerwalder Str. 21 | http://www.pengutronix.de/ |
> 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
> Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
>
>
>
Best regards.
Anis
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] usb: dwc2: skip polling for the soft reset clear bit and add after reset delay
2026-08-26 11:54 ` chalianis1
@ 2026-08-26 12:12 ` chalianis1
0 siblings, 0 replies; 4+ messages in thread
From: chalianis1 @ 2026-08-26 12:12 UTC (permalink / raw)
To: chalianis1; +Cc: barebox
On 2026-08-26 13:54 +0200, chalianis1@gmail.com wrote:
> On 2026-06-29 10:56 +0000, Sascha Hauer wrote:
> > On 2026-06-23 10:23, chalianis1 wrote:
> > > From: Chali Anis <chalianis1@gmail.com>
> > >
> > > The RPi4 SoC do not deassert GRSTCTL_CSFTRST within the expected
> > > window but continue to operate correctly, causing probe to fail
> > > with ETIMEDOUT.
> >
> > Does "within the expected window" imply that there is a bigger window in
> > which it is de-asserted?
> >
> What I mean is within the expected window according to the SoC datasheet
> of RK3588 which I just realized that I wrongly thought that it has a DWC2.
> Rockchip RK3588 TRM V1.0-Part2 20220309 page 2221
> > >
> > > Per the datasheet, GRSTCTL_CSFTRST is self-clearing but requires at least
> > > 3 PHY clocks after reset before any PHY domain access. Add a 1us delay to
> > > satisfy this requirement.
> >
> > Which datasheet, the SoC datasheet or a DWC2 datasheet?
> >
> the SOC one.
> > I'm just asking if there is a chance to not need a SoC check here.
> >
> > Sascha
> >
> > >
> > > Signed-off-by: Chali Anis <chalianis1@gmail.com>
> > > ---
> > > drivers/usb/dwc2/core.c | 13 +++++++++----
> > > 1 file changed, 9 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/drivers/usb/dwc2/core.c b/drivers/usb/dwc2/core.c
> > > index 60cc690fdbc0..6d4b7930da9e 100644
> > > --- a/drivers/usb/dwc2/core.c
> > > +++ b/drivers/usb/dwc2/core.c
> > > @@ -809,11 +809,16 @@ int dwc2_core_reset(struct dwc2 *dwc2)
> > > greset |= GRSTCTL_CSFTRST;
> > > dwc2_writel(dwc2, greset, GRSTCTL);
> > >
> > > - ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
> > > - if (ret) {
> > > - dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
> > > + /* Wait for at least 3 PHY Clocks */
> > > + udelay(1);
> > > +
> > > + if (!of_machine_is_compatible("brcm,bcm2711")) {
> > > + ret = dwc2_wait_bit_clear(dwc2, GRSTCTL, GRSTCTL_CSFTRST, 10000);
> > > + if (ret) {
> > > + dwc2_warn(dwc2, "%s: Timeout! Waiting for Core Soft Reset\n",
> > > __func__);
> > > - return ret;
> > > + return ret;
> > > + }
> > > }
> > >
> > > if (wait_for_host_mode)
> > >
> >
> > --
> > Pengutronix e.K. | |
> > Steuerwalder Str. 21 | http://www.pengutronix.de/ |
> > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
> > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
> >
> >
> >
>
> Best regards.
> Anis
>
>
>
>
I reply again on the thread because I tested something that works
by adding a delay on the the top of dwc2_core_reset the issue is
resolved and corresponds more for what the SoC even if it is not
the good datasheet stipulates see below.
diff --git a/drivers/usb/dwc2/core.c b/drivers/usb/dwc2/core.c
index 60cc690fdb..1fb8e2c6ac 100644
--- a/drivers/usb/dwc2/core.c
+++ b/drivers/usb/dwc2/core.c
@@ -776,6 +776,7 @@ int dwc2_core_reset(struct dwc2 *dwc2)
dwc2_dbg(dwc2, "%s(%p)\n", __func__, dwc2);
+ udelay(10);
/* Wait for AHB master IDLE state. */
ret = dwc2_wait_bit_set(dwc2, GRSTCTL, GRSTCTL_AHBIDLE, 10000);
if (ret) {
Best regards,
Anis
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-08-26 12:12 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-06-23 14:23 [PATCH] usb: dwc2: skip polling for the soft reset clear bit and add after reset delay chalianis1
2026-06-29 10:56 ` Sascha Hauer
2026-08-26 11:54 ` chalianis1
2026-08-26 12:12 ` chalianis1
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox