linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "WeitaoWang-oc@zhaoxin.com" <WeitaoWang-oc@zhaoxin.com>
To: <stern@rowland.harvard.edu>, <gregkh@linuxfoundation.org>,
	<linux-usb@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Cc: <CobeChen@zhaoxin.com>, <TimGuo@zhaoxin.com>,
	<tonywwang@zhaoxin.com>, <weitaowang@zhaoxin.com>
Subject: [PATCH] USB:Fix ehci infinite suspend-resume loop issue in zhaoxin
Date: Mon, 14 Mar 2022 15:35:37 +0800	[thread overview]
Message-ID: <3d0ae3ca-9dad-bb8f-5c41-45bdcb07b9cd@zhaoxin.com> (raw)

In zhaoxin platform, some ehci projects will latch a wakeup signal
internal when plug in a device on port during system S0. This wakeup
signal will turn on when ehci runtime suspend, which will trigger a
system control interrupt that will resume ehci back to D0. As no
device connect, ehci will be set to runtime suspend and turn on the
internal latched wakeup signal again. It will cause a suspend-resume
loop and generate system control interrupt continuously.

Fixed this issue by clear wakeup signal latched in ehci internal when
ehci resume callback is called.

Signed-off-by: Weitao Wang <WeitaoWang-oc@zhaoxin.com>
---
  drivers/usb/host/ehci-hcd.c | 26 ++++++++++++++++++++++++++
  drivers/usb/host/ehci-pci.c |  4 ++++
  drivers/usb/host/ehci.h     |  1 +
  3 files changed, 31 insertions(+)

diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c
index 3d82e0b..e4840ef 100644
--- a/drivers/usb/host/ehci-hcd.c
+++ b/drivers/usb/host/ehci-hcd.c
@@ -1103,6 +1103,30 @@ static void ehci_remove_device(struct usb_hcd 
*hcd, struct usb_device *udev)

  #ifdef CONFIG_PM

+/* Clear wakeup signal locked in zhaoxin platform when device plug in. */
+static void ehci_zx_wakeup_clear(struct ehci_hcd *ehci)
+{
+       u32 __iomem     *reg = &ehci->regs->port_status[4];
+       u32     t1 = ehci_readl(ehci, reg);
+
+       t1 &= (u32)~0xf0000;
+       t1 |= PORT_TEST_FORCE;
+       ehci_writel(ehci, t1, reg);
+       t1 = ehci_readl(ehci, reg);
+       msleep(1);
+       t1 &= (u32)~0xf0000;
+       ehci_writel(ehci, t1, reg);
+       ehci_readl(ehci, reg);
+       msleep(1);
+       t1 = ehci_readl(ehci, reg);
+       ehci_writel(ehci, t1 | PORT_CSC, reg);
+       ehci_readl(ehci, reg);
+       udelay(500);
+       t1 = ehci_readl(ehci, &ehci->regs->status);
+       ehci_writel(ehci, t1 & STS_PCD, &ehci->regs->status);
+       ehci_readl(ehci, &ehci->regs->status);
+}
+
  /* suspend/resume, section 4.3 */

  /* These routines handle the generic parts of controller suspend/resume */
@@ -1154,6 +1178,8 @@ int ehci_resume(struct usb_hcd *hcd, bool force_reset)
         if (ehci->shutdown)
                 return 0;               /* Controller is dead */

+       if (ehci->zx_wakeup_clear == 1)
+               ehci_zx_wakeup_clear(ehci);
         /*
          * If CF is still set and reset isn't forced
          * then we maintained suspend power.
diff --git a/drivers/usb/host/ehci-pci.c b/drivers/usb/host/ehci-pci.c
index e87cf3a..a5e27de 100644
--- a/drivers/usb/host/ehci-pci.c
+++ b/drivers/usb/host/ehci-pci.c
@@ -222,6 +222,10 @@ static int ehci_pci_setup(struct usb_hcd *hcd)
                         ehci->has_synopsys_hc_bug = 1;
                 }
                 break;
+       case PCI_VENDOR_ID_ZHAOXIN:
+               if (pdev->device == 0x3104 && (pdev->revision & 0xf0) == 
0x90)
+                       ehci->zx_wakeup_clear = 1;
+               break;
         }

         /* optional debug port, normally in the first BAR */
diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h
index fdd073c..712fdd0 100644
--- a/drivers/usb/host/ehci.h
+++ b/drivers/usb/host/ehci.h
@@ -220,6 +220,7 @@ struct ehci_hcd {                   /* one per 
controller */
         unsigned                imx28_write_fix:1; /* For Freescale 
i.MX28 */
         unsigned                spurious_oc:1;
         unsigned                is_aspeed:1;
+       unsigned                zx_wakeup_clear:1;

         /* required for usb32 quirk */
         #define OHCI_CTRL_HCFS          (3 << 6)
-- 
2.7.4

             reply	other threads:[~2022-03-14  7:35 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-14  7:35 WeitaoWang-oc [this message]
2022-03-14  7:45 ` [PATCH] USB:Fix ehci infinite suspend-resume loop issue in zhaoxin Greg KH
2022-03-14 14:24 ` Alan Stern
2022-03-15 12:39   ` WeitaoWang-oc
2022-03-15 15:18     ` Alan Stern
2022-03-16  2:18       ` WeitaoWang-oc
2022-03-22 13:52         ` WeitaoWang-oc
2022-04-05 16:02         ` Alan Stern
2022-04-06  2:38           ` WeitaoWang-oc
2022-04-06 16:20             ` Alan Stern
2022-04-07  6:15               ` WeitaoWang-oc
2022-04-07 14:22                 ` Alan Stern

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3d0ae3ca-9dad-bb8f-5c41-45bdcb07b9cd@zhaoxin.com \
    --to=weitaowang-oc@zhaoxin.com \
    --cc=CobeChen@zhaoxin.com \
    --cc=TimGuo@zhaoxin.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=stern@rowland.harvard.edu \
    --cc=tonywwang@zhaoxin.com \
    --cc=weitaowang@zhaoxin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).