mei: me: use runtime PG pm domain for non wakeable devices

For non wakeable devices we can't use pci runtime framework
as we are not able to wakeup from D3 states.
Instead we create new pg runtime domain that only drives ME power
gating protocol to reduce the power consumption.

Signed-off-by: Tomas Winkler <tomas.winkler@intel.com>
Signed-off-by: Alexander Usyskin <alexander.usyskin@intel.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/misc/mei/mei_dev.h b/drivers/misc/mei/mei_dev.h
index fe76e5b4cd2a5d5aff5989fefb37734609d77237..beff9ef319f8c93592a98d90ca479d6d604b5631 100644 (file)
--- a/drivers/misc/mei/mei_dev.h
+++ b/drivers/misc/mei/mei_dev.h
@@ -419,6 +419,9 @@ struct mei_device {
         * Power Gating support
         */
        enum mei_pg_event pg_event;
+#ifdef CONFIG_PM_RUNTIME
+       struct dev_pm_domain pg_domain;
+#endif /* CONFIG_PM_RUNTIME */
 
        unsigned char rd_msg_buf[MEI_RD_MSG_BUF_SIZE];  /* control messages */
        u32 rd_msg_hdr;

diff --git a/drivers/misc/mei/pci-me.c b/drivers/misc/mei/pci-me.c
index fcbf270c2c2a355669490b5f4f34b807addf3245..88516b02c6855c9e43a6b1a09396bdbaed67c8bf 100644 (file)
--- a/drivers/misc/mei/pci-me.c
+++ b/drivers/misc/mei/pci-me.c
@@ -87,6 +87,14 @@ static const struct pci_device_id mei_me_pci_tbl[] = {
 
 MODULE_DEVICE_TABLE(pci, mei_me_pci_tbl);
 
+#ifdef CONFIG_PM_RUNTIME
+static inline void mei_me_set_pm_domain(struct mei_device *dev);
+static inline void mei_me_unset_pm_domain(struct mei_device *dev);
+#else
+static inline void mei_me_set_pm_domain(struct mei_device *dev) {}
+static inline void mei_me_unset_pm_domain(struct mei_device *dev) {}
+#endif /* CONFIG_PM_RUNTIME */
+
 /**
  * mei_quirk_probe - probe for devices that doesn't valid ME interface
  *
@@ -225,6 +233,14 @@ static int mei_me_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
 
        schedule_delayed_work(&dev->timer_work, HZ);
 
+       /*
+       * For not wake-able HW runtime pm framework
+       * can't be used on pci device level.
+       * Use domain runtime pm callbacks instead.
+       */
+       if (!pci_dev_run_wake(pdev))
+               mei_me_set_pm_domain(dev);
+
        if (mei_pg_is_enabled(dev))
                pm_runtime_put_noidle(&pdev->dev);
 
@@ -276,6 +292,9 @@ static void mei_me_remove(struct pci_dev *pdev)
        dev_dbg(&pdev->dev, "stop\n");
        mei_stop(dev);
 
+       if (!pci_dev_run_wake(pdev))
+               mei_me_unset_pm_domain(dev);
+
        /* disable interrupts */
        mei_disable_interrupts(dev);
 
@@ -421,6 +440,37 @@ static int mei_me_pm_runtime_resume(struct device *device)
 
        return ret;
 }
+
+/**
+ * mei_me_set_pm_domain - fill and set pm domian stucture for device
+ *
+ * @dev: mei_device
+ */
+static inline void mei_me_set_pm_domain(struct mei_device *dev)
+{
+       struct pci_dev *pdev  = dev->pdev;
+
+       if (pdev->dev.bus && pdev->dev.bus->pm) {
+               dev->pg_domain.ops = *pdev->dev.bus->pm;
+
+               dev->pg_domain.ops.runtime_suspend = mei_me_pm_runtime_suspend;
+               dev->pg_domain.ops.runtime_resume = mei_me_pm_runtime_resume;
+               dev->pg_domain.ops.runtime_idle = mei_me_pm_runtime_idle;
+
+               pdev->dev.pm_domain = &dev->pg_domain;
+       }
+}
+
+/**
+ * mei_me_unset_pm_domain - clean pm domian stucture for device
+ *
+ * @dev: mei_device
+ */
+static inline void mei_me_unset_pm_domain(struct mei_device *dev)
+{
+       /* stop using pm callbacks if any */
+       dev->pdev->dev.pm_domain = NULL;
+}
 #endif /* CONFIG_PM_RUNTIME */
 
 #ifdef CONFIG_PM