lkml.org 
[lkml]   [2019]   [Dec]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] PCI/PM: Report runtime wakeup is not supported if bridge isn't bound to driver
Date
We have a Pericom USB add-on card that has three USB controller
functions 06:00.[0-2], connected to bridge device 05:03.0, which is
connected to another bridge device 04:00.0:

-[0000:00]-+-00.0
+-1c.6-[04-06]----00.0-[05-06]----03.0-[06]--+-00.0
| +-00.1
| \-00.2

When bridge device (05:03.0) and all three USB controller functions
(06:00.[0-2]) are runtime suspended, they don't get woken up by plugging
USB devices into the add-on card.

This is because the pcieport driver failed to probe on 04:00.0, since
the device supports neither legacy IRQ, MSI nor MSI-X. Because of that,
there's no native PCIe PME can work for devices connected to it.

So let's correctly report runtime wakeup isn't supported when any of
PCIe bridges isn't bound to pcieport driver.

Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=205981
Signed-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
---
drivers/pci/pci.c | 12 ++++++++++++
1 file changed, 12 insertions(+)

diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index 951099279192..ca686cfbd65e 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -2493,6 +2493,18 @@ bool pci_dev_run_wake(struct pci_dev *dev)
if (!pci_pme_capable(dev, pci_target_state(dev, true)))
return false;

+ /* If any upstream PCIe bridge isn't bound to pcieport driver, there's
+ * no IRQ for PME.
+ */
+ if (pci_is_pcie(dev)) {
+ while (bus->parent) {
+ if (!bus->self->driver)
+ return false;
+
+ bus = bus->parent;
+ }
+ }
+
if (device_can_wakeup(&dev->dev))
return true;

--
2.17.1
\
 
 \ /
  Last update: 2019-12-27 10:25    [W:0.050 / U:14.240 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site