Re: [PATCH v2 0/2] usb: dwc2: fix USB peripheral role in dual-role mode on PX30

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Minas,

On 12/20/22 07:02, Minas Harutyunyan wrote:
Hi Quentin,

On 12/16/2022 8:29 PM, Quentin Schulz <foss+kernel@xxxxxxxxx> wrote:
From: Quentin Schulz <foss+kernel@xxxxxxxxx>
Sent: Friday, December 16, 2022 8:29 PM
To: Minas Harutyunyan <hminas@xxxxxxxxxxxx>; Greg Kroah-Hartman
<gregkh@xxxxxxxxxxxxxxxxxxx>
Cc: Quentin Schulz <foss+kernel@xxxxxxxxx>; linux-usb@xxxxxxxxxxxxxxx;
linux-kernel@xxxxxxxxxxxxxxx; Bin Yang <yangbin@xxxxxxxxxxxxxx>; Quentin
Schulz <quentin.schulz@xxxxxxxxxxxxxxxxxxxxx>
Subject: [PATCH v2 0/2] usb: dwc2: fix USB peripheral role in dual-role mode
on PX30

On Theobroma Ringneck SoM, USB peripheral role when in dual-role mode does
not work and displays the following error message:
dwc2 ff300000.usb: dwc2_core_reset: HANG! Soft Reset timeout GRSTCTL_CSFTRST


This mostly happen if PHY not initialized correctly.


Any way to know which part of the PHY wouldn't be correctly initialized? or is this just a whack-a-mole game with register bits?

The USB sniffer shows nothing and dumping the host registers is stuck on
HCDMA(0) register.

Note that for some reason it works "fine" on PX30-EVB (there's another issue
but not related to/fixed in this patch series).

Since there's no documentation available for this IP, this patch series is
basically just slightly adapted downstream BSP vendor kernel patches and I
cannot unfortunately give more information than what I have.

This patch series was tested on Theobroma Ringneck SoM on Haikou devkit and
PX30 EVB. It fixes Ringneck support and does not break PX30-EVB's.

For reference, the content of those commits can be found in tag linux-5.10-
gen-rkr1, and the following commits have been used:
964d50060bf53a8defd1fc561b9261424f25ddad
ad81c375602819a538ad68d979906c05663046e2
6e6adab8f735bc4fe27a67bdc3144d8fa89250d4
7c3a4e60247fd7f7b04d95d15cb12c63a5c20408

Note that Rockchip kernel called a slightly different implementation of
__dwc2_lowlevel_hw_enable instead of dwc2_lowlevel_hw_enable but it seemed
more right to me to call dwc2_lowlevel_hw_enable as done for the forced
peripheral mode.

Note that it is still not flawlessly working but the support is improved
(somewhat working vs not working at all).

If "still not flawlessly working" maybe required deeper investigate issue
and submit "flawlessly working" patch.


I would love to honestly. But no documentation and the vendor kernel works differently but not better. I get the point, not sure I'll be able to provide any meaningful improvement to this patch though. Let's see if I'll have time to dig into this sometime soon (I expect not :/).

I'm all ears if someone wants to share how they would start debugging this though.

Cheers,
Quentin



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux