All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: anis chali <chalianis1@gmail.com>
Cc: Ahmad Fatoum <a.fatoum@pengutronix.de>,
	BAREBOX <barebox@lists.infradead.org>,
	"Claude Sonnet 4.6" <noreply@anthropic.com>
Subject: Re: [PATCH 1/2] usb: dwc2: handle NAK when CHHLTD does not fire
Date: Tue, 21 Apr 2026 09:54:43 +0200	[thread overview]
Message-ID: <aectQyiyowJc1DCH@pengutronix.de> (raw)
In-Reply-To: <CAL+1fyCYo3ZATq8d3Tv5fy_OcGVo2J3q3853-LqxgLqXNEahiQ@mail.gmail.com>

Hi,

On Mon, Apr 20, 2026 at 10:28:58PM -0400, anis chali wrote:
> Hi.
> 
> On Raspberry pi cm 4, with only the patch usb: dwc2: handle NAK when
> CHHLTD does not fire,
> the usb controller does not work and displays indefinitely the message
> below even if no usb storage plugged in
> ERROR: dwc2 fe980000.usb@7e980000.of: Timeout on bulk endpoint
> I did the test with the patch series and I get the same behaviour,
> console continuously displaying the timeout error.

I just tried with only your patch applied and it doesn't work for me. It
seems there is some timing issue involved, because when I add debugging
output at the right places then it works.

I haven't gotten further so far.

Sascha

-- 
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 |



  reply	other threads:[~2026-04-21  7:55 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-04-20 11:20 [PATCH 0/2] USB: dwc2: Fix handling NAK Sascha Hauer
2026-04-20 11:20 ` [PATCH 1/2] usb: dwc2: handle NAK when CHHLTD does not fire Sascha Hauer
2026-04-20 11:32   ` Ahmad Fatoum
2026-04-20 12:18     ` Sascha Hauer
2026-04-20 14:11       ` Ahmad Fatoum
2026-04-20 15:40         ` anis chali
2026-04-21  2:28           ` anis chali
2026-04-21  7:54             ` Sascha Hauer [this message]
2026-04-21 13:48               ` anis chali
2026-04-20 11:20 ` [PATCH 2/2] usb: dwc2: fix data toggle reset direction on ClearFeature(ENDPOINT_HALT) Sascha Hauer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aectQyiyowJc1DCH@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=chalianis1@gmail.com \
    --cc=noreply@anthropic.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.