On 20-10-27 10:33:52, Mathias Nyman wrote:
> On 27.10.2020 3.50, Peter Chen wrote:
> >
> >>> Cc: Pawel Laszczak <pawell@xxxxxxxxxxx>
> >>> Cc: Roger Quadros <rogerq@xxxxxx>
> >>> Signed-off-by: Peter Chen <peter.chen@xxxxxxx>
> >>> ---
> >>> drivers/usb/host/xhci.c | 2 +-
> >>> drivers/usb/host/xhci.h | 1 +
> >>> 2 files changed, 2 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/drivers/usb/host/xhci.c b/drivers/usb/host/xhci.c index
> >>> 482fe8c5e3b4..fc72a03dc27f 100644
> >>> --- a/drivers/usb/host/xhci.c
> >>> +++ b/drivers/usb/host/xhci.c
> >>> @@ -193,7 +193,7 @@ int xhci_reset(struct xhci_hcd *xhci)
> >>> * Without this delay, the subsequent HC register access,
> >>> * may result in a system hang very rarely.
> >>> */
> >>> - if (xhci->quirks & XHCI_INTEL_HOST)
> >>> + if (xhci->quirks & (XHCI_INTEL_HOST | XHCI_CDNS_HOST))
> >>> udelay(1000);
> >>>
> >>> ret = xhci_handshake(&xhci->op_regs->command,
> >>> diff --git a/drivers/usb/host/xhci.h b/drivers/usb/host/xhci.h index
> >>> 8be88379c0fb..4b7275c73ea5 100644
> >>> --- a/drivers/usb/host/xhci.h
> >>> +++ b/drivers/usb/host/xhci.h
> >>> @@ -1877,6 +1877,7 @@ struct xhci_hcd {
> >>> #define XHCI_SNPS_BROKEN_SUSPEND BIT_ULL(35)
> >>> #define XHCI_RENESAS_FW_QUIRK BIT_ULL(36)
> >>> #define XHCI_SKIP_PHY_INIT BIT_ULL(37)
> >>> +#define XHCI_CDNS_HOST BIT_ULL(38)
> >>>
> >>> unsigned int num_active_eps;
> >>> unsigned int limit_active_eps;
> >>>
> >>
> >> Is the XHCI_CDNS_HOST quirk bit set in some other patchseries?
> >>
> >
> > Currently, no, only at the function of xhci_reset in this commit.
>
> I can only see it being checked, not set. Am I missing something?
>
> -Mathias
>
Oh, there is a missing patch for cdns3 host change, the code is
located at drivers/usb/cdns3/host.c. I will merge them together,
and send v2.
--
Thanks,
Peter Chen
