[PATCH v2 3/6] configs: xilinx_versal_virt: disable USB_DWC3

[Michal Simek]

On 11/25/24 10:21, Heinrich Schuchardt wrote:
> On 11/25/24 09:32, Michal Simek wrote:
>>
>>
>> On 11/23/24 22:45, Heinrich Schuchardt wrote:
>>> The CI uses the following command to launch xilinx_versal_virt_defconfig:
>>>
>>>      qemu-system-aarch64 -M xlnx-versal-virt \
>>>      -display none -m 4G -serial mon:stdio \
>>>      -device loader,file=u-boot,cpu-num=0
>>>
>>> 'usb start' or invoking eth_bootdev_hunt leads to a crash when function
>>> dwc3_core_init() tries to access a register at offset 0xc704 (DWC3_DCTL)
>>> relative to the register start address 0xfe20c100.
>>>
>>> Disable CONFIG_USB_DWC3 until the driver problem is fixed.
>>>
>>> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
>>> ---
>>> v2:
>>>     new patch
>>> ---
>>>   configs/xilinx_versal_virt_defconfig | 2 --
>>>   1 file changed, 2 deletions(-)
>>>
>>> diff --git a/configs/xilinx_versal_virt_defconfig b/configs/ 
>>> xilinx_versal_virt_defconfig
>>> index c8f166c1221..06a240173ba 100644
>>> --- a/configs/xilinx_versal_virt_defconfig
>>> +++ b/configs/xilinx_versal_virt_defconfig
>>> @@ -153,8 +153,6 @@ CONFIG_USB=y
>>>   CONFIG_DM_USB_GADGET=y
>>>   CONFIG_USB_XHCI_HCD=y
>>>   CONFIG_USB_XHCI_DWC3=y
>>> -CONFIG_USB_DWC3=y
>>> -CONFIG_USB_DWC3_GENERIC=y
>>>   CONFIG_USB_ULPI_VIEWPORT=y
>>>   CONFIG_USB_ULPI=y
>>>   CONFIG_USB_GADGET=y
>>
>> NACK. That's not the way to go. If one test is failing in CI disable that test 
>> not really support which is for HW too.
>>
>> Thanks,
>> Michal
> 
> Hello Michal,
> 
> It is not that a single test fails. USB is completely unusable on the board. 
> 'usb start' leads to an immediate crash.

I am not applying any patches on SOM with HEAD and usb is partially working.
I can agree that it is not super stable with upstream only patches but that's 
different story.

> 
> The problem already existed in v2021.01.

Any log?

> Do you know of any QEMU/U-Boot combination where USB ever worked on the board?

Why was this visible in CI in past and why this pops up now?
What has changed? I don't think there was any change in QEMu regarding USB but I 
can double check.

Thanks,
Michal