]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commitdiff
usb: dwc2: host: Fix transaction errors in host mode
authorMinas Harutyunyan <hminas@synopsys.com>
Fri, 19 Jan 2018 10:44:20 +0000 (14:44 +0400)
committerStefan Bader <stefan.bader@canonical.com>
Tue, 14 Aug 2018 10:28:45 +0000 (12:28 +0200)
BugLink: http://bugs.launchpad.net/bugs/1783418
[ Upstream commit 92a8dd26464e1f21f1d869ec53717bd2c1200d63 ]

Added missing GUSBCFG programming in host mode, which fixes
transaction errors issue on HiKey and Altera Cyclone V boards.

These field even if was programmed in device mode (in function
dwc2_hsotg_core_init_disconnected()) will be resetting to POR values
after core soft reset applied.
So, each time when switching to host mode required to set this field
to correct value.

Acked-by: John Youn <johnyoun@synopsys.com>
Signed-off-by: Minas Harutyunyan <hminas@synopsys.com>
Signed-off-by: Grigor Tovmasyan <tovmasya@synopsys.com>
Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Kamal Mostafa <kamal@canonical.com>
Signed-off-by: Khalid Elmously <khalid.elmously@canonical.com>
drivers/usb/dwc2/hcd.c

index eb5ea58cd23b02053085aa8c33a358b897d048dc..6c13fd9f78a8ee5b46a377708f49332f8d242110 100644 (file)
@@ -2335,10 +2335,22 @@ static int dwc2_core_init(struct dwc2_hsotg *hsotg, bool initial_setup)
  */
 static void dwc2_core_host_init(struct dwc2_hsotg *hsotg)
 {
-       u32 hcfg, hfir, otgctl;
+       u32 hcfg, hfir, otgctl, usbcfg;
 
        dev_dbg(hsotg->dev, "%s(%p)\n", __func__, hsotg);
 
+       /* Set HS/FS Timeout Calibration to 7 (max available value).
+        * The number of PHY clocks that the application programs in
+        * this field is added to the high/full speed interpacket timeout
+        * duration in the core to account for any additional delays
+        * introduced by the PHY. This can be required, because the delay
+        * introduced by the PHY in generating the linestate condition
+        * can vary from one PHY to another.
+        */
+       usbcfg = dwc2_readl(hsotg->regs + GUSBCFG);
+       usbcfg |= GUSBCFG_TOUTCAL(7);
+       dwc2_writel(usbcfg, hsotg->regs + GUSBCFG);
+
        /* Restart the Phy Clock */
        dwc2_writel(0, hsotg->regs + PCGCTL);