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

[Heinrich Schuchardt]

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.

The problem already existed in v2021.01.
Do you know of any QEMU/U-Boot combination where USB ever worked on the 
board?

Best regards

Heinrich