All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anders Roxell <anders.roxell@linaro.org>
To: Quentin Schulz <quentin.schulz@cherry.de>
Cc: u-boot@lists.u-boot-project.org, marek.vasut@mailbox.org,
	trini@konsulko.com, ilias.apalodimas@linaro.org,
	michal.simek@amd.com, lukma@denx.de, alchark@flipper.net,
	jerome.forissier@arm.com, dlechner@baylibre.com, anshuld@ti.com,
	sjg@chromium.org, Thinh.Nguyen@synopsys.com,
	mkorpershoek@kernel.org, love.kumar@amd.com
Subject: Re: [PATCH v9 00/26] usb: dwc3: part 2, make the resynced code work
Date: Tue, 13 Oct 2026 15:02:42 +0200	[thread overview]
Message-ID: <as4r8p_70JTOl068@monster> (raw)
In-Reply-To: <80e9e892-24de-4cba-b84a-35bdcb6a903c@cherry.de>

On 2026-10-02 16:17, Quentin Schulz wrote:
> Hi Anders,

Hi Quentin,

> 
> On 9/16/26 2:38 PM, Anders Roxell wrote:
> > Hi,
> > 

[ ... ]

> > 
> > Older changes are listed in v7 [6] and v5 [1].
> > 
> 
> I applied both series on top of current main (c92d8c3c431e "board: beacon:
> Remove i.MX8MP Beacon EmbeddedWorks Devkit support") and tested on RK3588
> Tiger (three DWC3 controllers exposed) and RK3399 Puma (two DWC3 controllers
> exposed). For each board, one of the DWC3 is routed to a micro-USB OTG port
> which doesn't seem to work (before or after your patch). Considering it's
> historically finicky in Linux, I wouldn't read too much into this. For
> RK3588 Tiger, the third controller doesn't even appear in the usb tree
> output. On RK3399 Puma, it does but no USB stick detected (either via a
> micro USB-OTG USB2 or USB3 adapter).

Thanks for testing on both boards, the logs help a lot.

You say the OTG port fails with and without my patches, so I will
leave that one out of this series.

With the defconfigs the OTG controller is not bound on either board.
dr_mode is "otg" and there is no gadget support, so dwc3-generic skips
it. That fits Tiger, but you say it shows up on Puma. Did you use
another config there?

> 
> Tests are performed like this: I insert one stick in a port and power reset
> the board and run the commands. Tests for different ports for the same
> boards are run with a reset in-between tries. If you want some other tests
> or a different test procedure, I can do that too, but only in a month as I
> will not have physical access to my devices for that amount of time.
> 

[ ... ]

> 
> usb write: device 0 block # 0, count 64 ... WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> Resetting EP 3...
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> Resetting EP 3...
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> Resetting EP 3...
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> WARN endpoint is halted
> 0 blocks written: ERROR

Do you see the halted warnings and this write error without my patches
too? And does the Kingston stick fail every time on that port?

> 
> # RK3588 Tiger: Q7 USB P0 lower retry with another USB stick
> =>  usb reset && usb tree && random $loadaddr 0x8000 && mw.q $kernel_addr_r
> 0xffffffffffffffff 0x1000 && usb dev 0 && usb write $loadaddr 0 0x40 && usb
> read $kernel_addr_r 0 0x40 && cmp.q $loadaddr $kernel_addr_r 0x1000
> resetting USB...

[ ... ]

> Total of 4096 double word(s) were the same
> 
> I also have two SanDisk Corp. Ultra Flair that are USB3 but they are
> consistently detected as USB2 in U-Boot for some reason.

Same without my patches?

No rush, I know you are away for a month.

Cheers,
Anders

      reply	other threads:[~2026-10-13 14:24 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-16 12:38 [PATCH v9 00/26] usb: dwc3: part 2, make the resynced code work Anders Roxell
2026-09-16 12:38 ` [PATCH v9 01/26] x86: asm: io.h: add rmb() and wmb() Anders Roxell
2026-09-16 12:38 ` [PATCH v9 02/26] dm: add helpers needed by the resynced DWC3 code Anders Roxell
2026-09-16 12:38 ` [PATCH v9 03/26] usb: phy: add the usb_phy " Anders Roxell
2026-09-16 12:38 ` [PATCH v9 04/26] usb: gadget: adapt the gadget API after the resync Anders Roxell
2026-09-16 12:38 ` [PATCH v9 05/26] usb: gadget: udc: make udc-core build in U-Boot Anders Roxell
2026-09-16 12:38 ` [PATCH v9 06/26] usb: dwc3: make the core driver " Anders Roxell
2026-09-16 12:38 ` [PATCH v9 07/26] usb: dwc3: bring back the samsung usb phy driver Anders Roxell
2026-09-16 12:38 ` [PATCH v9 08/26] usb: dwc3: make gadget and ep0 build in U-Boot Anders Roxell
2026-09-16 12:38 ` [PATCH v9 09/26] usb: gadget: udc: enable async callbacks on bind Anders Roxell
2026-09-16 12:38 ` [PATCH v9 10/26] usb: dwc3: ep0: flush and invalidate cache around ep0 transfers Anders Roxell
2026-09-16 12:38 ` [PATCH v9 11/26] usb: dwc3: gadget: flush and invalidate cache around transfers Anders Roxell
2026-09-16 12:38 ` [PATCH v9 12/26] usb: dwc3: make the am62 glue build in U-Boot Anders Roxell
2026-09-16 12:38 ` [PATCH v9 13/26] usb: gadget: ci_udc: move to the udc-core registration Anders Roxell
2026-09-16 12:38 ` [PATCH v9 14/26] usb: gadget: ci_udc: set endpoint capabilities Anders Roxell
2026-09-16 12:38 ` [PATCH v9 15/26] usb: gadget: dwc2_udc_otg: fix up after the resync Anders Roxell
2026-09-16 12:38 ` [PATCH v9 16/26] usb: gadget: fix up the function drivers " Anders Roxell
2026-09-16 12:38 ` [PATCH v9 17/26] usb: gadget: fix up the last UDC " Anders Roxell
2026-09-16 12:38 ` [PATCH v9 18/26] usb: gadget: atmel_usba_udc: set endpoint capabilities Anders Roxell
2026-09-16 12:38 ` [PATCH v9 19/26] usb: gadget: at91_udc: register with udc-core Anders Roxell
2026-09-16 12:38 ` [PATCH v9 20/26] usb: gadget: at91_udc: set endpoint capabilities Anders Roxell
2026-09-16 12:38 ` [PATCH v9 21/26] usb: musb-new: fix up after the resync Anders Roxell
2026-09-16 12:38 ` [PATCH v9 22/26] usb: musb-new: set endpoint capabilities Anders Roxell
2026-09-16 12:38 ` [PATCH v9 23/26] usb: mtu3: fix up after the resync Anders Roxell
2026-09-16 12:38 ` [PATCH v9 24/26] usb: mtu3: set endpoint capabilities Anders Roxell
2026-09-16 12:38 ` [PATCH v9 25/26] usb: host: xhci: fix up after the resync Anders Roxell
2026-09-16 12:38 ` [PATCH v9 26/26] usb: cdns3: " Anders Roxell
2026-09-17 12:43 ` [PATCH v9 00/26] usb: dwc3: part 2, make the resynced code work Anshul Dalal
2026-09-25  9:59   ` Anders Roxell
2026-09-25 10:39     ` Siddharth Vadapalli
2026-09-29 10:07       ` Anders Roxell
2026-09-29 13:05         ` Siddharth Vadapalli
2026-09-30 12:10           ` Anders Roxell
2026-10-02 14:17 ` Quentin Schulz
2026-10-13 13:02   ` Anders Roxell [this message]

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=as4r8p_70JTOl068@monster \
    --to=anders.roxell@linaro.org \
    --cc=Thinh.Nguyen@synopsys.com \
    --cc=alchark@flipper.net \
    --cc=anshuld@ti.com \
    --cc=dlechner@baylibre.com \
    --cc=ilias.apalodimas@linaro.org \
    --cc=jerome.forissier@arm.com \
    --cc=love.kumar@amd.com \
    --cc=lukma@denx.de \
    --cc=marek.vasut@mailbox.org \
    --cc=michal.simek@amd.com \
    --cc=mkorpershoek@kernel.org \
    --cc=quentin.schulz@cherry.de \
    --cc=sjg@chromium.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.u-boot-project.org \
    /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.