Hi Bernhard,
On 4/5/25 12:28, Bernhard Beschow wrote:
Am 5. April 2025 17:26:14 UTC schrieb Guenter Roeck <li...@roeck-us.net>:
On 4/5/25 07:25, Philippe Mathieu-Daudé wrote:
Hi Guenter,
On 5/4/25 16:00, Guenter Roeck wrote:
This series is needed to support the USB interface on imx8mp-evk when
booting the Linux kernel.
According to the XHCI specification, ERSTBA should be written in Low-High
order. The Linux kernel writes the high word first. This results in an
initialization failure.
The following information is found in the Linux kernel commit log.
[Synopsys]- The host controller was design to support ERST setting
during the RUN state. But since there is a limitation in controller
in supporting separate ERSTBA_HI and ERSTBA_LO programming,
It is supported when the ERSTBA is programmed in 64bit,
or in 32 bit mode ERSTBA_HI before ERSTBA_LO
[Synopsys]- The internal initialization of event ring fetches
the "Event Ring Segment Table Entry" based on the indication of
ERSTBA_LO written.
Add property to support writing the high word first. Enable it
for dwc3.
----------------------------------------------------------------
Guenter Roeck (2):
hw: usb: xhci: Add property to support writing ERSTBA in high-low order
hw/usb/hcd-dwc3: Set erstba-hi-lo property
What about using .impl.min_access_size = 8 instead?
Could you try this patch, or provide me with a reproducer?
You should find everything you need to reproduce the problem at
Hi Guenter,
thanks for testing the new board, much appreciated.
http://server.roeck-us.net/qemu/xhci/
I like your approach of pruning the device tree automatically:
<http://server.roeck-us.net/qemu/xhci/patches/0002-imx8mp-evk-Remove-unimplemented-properties-and-nodes.patch>
Would you mind if I upstream it with a few modifications like in
<https://github.com/shentok/qemu/commits/upstream/imx8mp-test/> ? If yes, please let
me know if I attribute your work there correctly.
Please feel free to go ahead. Attribution is fine.
Note that I had submitted a similar patch a couple of years ago; that
was rejected with the argument that I should not use the dts file
from the Linux kernel in the first place. That is why I did not bother
to submit the patch myself.
Anyway, just for reference: the inspiration for this change is from
commit bf1da4b308 ("hw/arm/raspi4b: Temporarily disable unimplemented
rpi4b devices").
Thanks,
Guenter
