Re: [PATCH v8 6/9] usb: dwc3: qcom: Add multiport controller support for qcom wrapper

From: Krishna Kurapati PSSNV
Date: Thu Jun 22 2023 - 00:43:33 EST




On 6/21/2023 1:08 PM, Johan Hovold wrote:
On Thu, Jun 15, 2023 at 09:08:01PM +0000, Thinh Nguyen wrote:
On Thu, Jun 15, 2023, Krishna Kurapati PSSNV wrote:

How about we add compatible data indicating the number of usb2/usb3 ports.
That way we needn't parse the DT or read xhci registers atleast as a
temporary solution to unblock other patches. Once this series is merged, we
can get back to fixing the port count calculation. Does it seem fine ?


Temporary solution should not involve DT as it's not easily reverted or
changed. Just include xhci-ext-caps.h and use the inline function. I
think Johan is fine with that. If not, he can provide more feedback.

Yes, I already suggested that as a quick way forward since it is already
used this way by the xhci debug driver.

Johan

Hi Johan, Thinh,

Pushed a v9 following the above suggestion.

Thanks,
Krishna,