[PATCH v2 15/25] usb: penwell_otg: add runtime pm support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Add runtime pm support in penwell_otg transceiver driver. Penwell_otg
transceiver driver will invoke host/peripheral runtime_pm functions 
according to work state.

Signed-off-by: Hao Wu <hao.wu@xxxxxxxxx>
---
 drivers/usb/otg/penwell_otg.c     |  167 ++++++++++++++++++++++++++++++++-----
 include/linux/usb/intel_mid_otg.h |    7 ++
 2 files changed, 153 insertions(+), 21 deletions(-)

diff --git a/drivers/usb/otg/penwell_otg.c b/drivers/usb/otg/penwell_otg.c
index e09406f..1c6212e 100644
--- a/drivers/usb/otg/penwell_otg.c
+++ b/drivers/usb/otg/penwell_otg.c
@@ -35,6 +35,7 @@
 #include <linux/usb/otg.h>
 #include <linux/notifier.h>
 #include <linux/delay.h>
+#include <linux/pm_runtime.h>
 #include <asm/intel_scu_ipc.h>
 #include "../core/usb.h"
 
@@ -63,27 +64,6 @@ static int penwell_otg_set_peripheral(struct otg_transceiver *otg,
 static int penwell_otg_start_srp(struct otg_transceiver *otg);
 static int penwell_otg_msic_write(u16 addr, u8 data);
 
-static DEFINE_PCI_DEVICE_TABLE(pci_ids) = {{
-	.class =        ((PCI_CLASS_SERIAL_USB << 8) | 0x20),
-	.class_mask =   ~0,
-	.vendor =	0x8086,
-	.device =	0x0829,
-	.subvendor =	PCI_ANY_ID,
-	.subdevice =	PCI_ANY_ID,
-}, { /* end: all zeroes */ }
-};
-
-static struct pci_driver otg_pci_driver = {
-	.name =		(char *) driver_name,
-	.id_table =	pci_ids,
-
-	.probe =	penwell_otg_probe,
-	.remove =	penwell_otg_remove,
-
-	.suspend =	penwell_otg_suspend,
-	.resume =	penwell_otg_resume,
-};
-
 static const char *charger_string(enum usb_charger_type charger)
 {
 	switch (charger) {
@@ -1071,6 +1051,9 @@ static irqreturn_t otg_irq(int irq, void *_dev)
 	int_en = (int_sts & OTGSC_INTEN_MASK) >> 8;
 	int_mask = int_sts & int_en;
 
+	if (int_mask == 0)
+		return IRQ_NONE;
+
 	if (int_mask) {
 		dev_dbg(pnw->dev,
 			"OTGSC = 0x%x, mask =0x%x\n", int_sts, int_mask);
@@ -1271,6 +1254,8 @@ static void penwell_otg_work(struct work_struct *work)
 	dev_dbg(pnw->dev,
 		"old state = %s\n", otg_state_string(iotg->otg.state));
 
+	pm_runtime_get_sync(pnw->dev);
+
 	switch (iotg->otg.state) {
 	case OTG_STATE_UNDEFINED:
 	case OTG_STATE_B_IDLE:
@@ -2103,6 +2088,8 @@ static void penwell_otg_work(struct work_struct *work)
 		;
 	}
 
+	pm_runtime_put_sync(pnw->dev);
+
 	dev_dbg(pnw->dev,
 			"new state = %s\n", otg_state_string(iotg->otg.state));
 }
@@ -2826,6 +2813,144 @@ error:
 	return ret;
 }
 
+#ifdef CONFIG_PM_RUNTIME
+/* Runtime PM */
+static int penwell_otg_runtime_suspend(struct device *dev)
+{
+	struct penwell_otg	*pnw = the_transceiver;
+	struct pci_dev		*pdev;
+	int			ret = 0;
+
+	dev_dbg(dev, "%s --->\n", __func__);
+
+	pdev = to_pci_dev(dev);
+
+	switch (pnw->iotg.otg.state) {
+	case OTG_STATE_A_IDLE:
+	case OTG_STATE_B_IDLE:
+		/* Transceiver handle it itself */
+		penwell_otg_phy_low_power(1);
+		break;
+	case OTG_STATE_A_WAIT_BCON:
+	case OTG_STATE_A_HOST:
+	case OTG_STATE_A_SUSPEND:
+		if (pnw->iotg.runtime_suspend_host)
+			ret = pnw->iotg.runtime_suspend_host(&pnw->iotg);
+		break;
+	case OTG_STATE_A_PERIPHERAL:
+	case OTG_STATE_B_PERIPHERAL:
+		if (pnw->iotg.runtime_suspend_peripheral)
+			ret = pnw->iotg.runtime_suspend_peripheral(&pnw->iotg);
+		break;
+	default:
+		break;
+	}
+
+	dev_dbg(dev, "%s <---\n", __func__);
+	return ret;
+}
+
+static int penwell_otg_runtime_resume(struct device *dev)
+{
+	struct penwell_otg	*pnw = the_transceiver;
+	struct pci_dev		*pdev;
+	int			ret = 0;
+
+	dev_dbg(dev, "%s --->\n", __func__);
+
+	pdev = to_pci_dev(dev);
+
+	penwell_otg_intr(1);
+	penwell_otg_phy_low_power(0);
+
+	switch (pnw->iotg.otg.state) {
+	case OTG_STATE_A_IDLE:
+	case OTG_STATE_B_IDLE:
+		penwell_otg_phy_low_power(0);
+		break;
+	case OTG_STATE_A_WAIT_BCON:
+	case OTG_STATE_A_HOST:
+	case OTG_STATE_A_SUSPEND:
+		if (pnw->iotg.runtime_resume_host)
+			ret = pnw->iotg.runtime_resume_host(&pnw->iotg);
+		break;
+	case OTG_STATE_A_PERIPHERAL:
+	case OTG_STATE_B_PERIPHERAL:
+		if (pnw->iotg.runtime_resume_peripheral)
+			ret = pnw->iotg.runtime_resume_peripheral(&pnw->iotg);
+		break;
+	default:
+		break;
+	}
+
+	dev_dbg(dev, "%s <---\n", __func__);
+	return 0;
+}
+
+static int penwell_otg_runtime_idle(struct device *dev)
+{
+	struct penwell_otg	*pnw = the_transceiver;
+
+	dev_dbg(dev, "%s --->\n", __func__);
+
+	switch (pnw->iotg.otg.state) {
+	case OTG_STATE_A_WAIT_VRISE:
+	case OTG_STATE_A_WAIT_VFALL:
+	case OTG_STATE_A_VBUS_ERR:
+	case OTG_STATE_B_WAIT_ACON:
+	case OTG_STATE_B_HOST:
+		dev_dbg(dev, "Keep in active\n");
+		return -EBUSY;
+	default:
+		break;
+	}
+
+	dev_dbg(dev, "%s <---\n", __func__);
+
+	return 0;
+}
+
+#else
+
+#define penwell_otg_runtime_suspend NULL
+#define penwell_otg_runtime_resume NULL
+#define penwell_otg_runtime_idle NULL
+
+#endif
+
+/*----------------------------------------------------------*/
+
+static DEFINE_PCI_DEVICE_TABLE(pci_ids) = {{
+	.class =        ((PCI_CLASS_SERIAL_USB << 8) | 0x20),
+	.class_mask =   ~0,
+	.vendor =	0x8086,
+	.device =	0x0829,
+	.subvendor =	PCI_ANY_ID,
+	.subdevice =	PCI_ANY_ID,
+}, { /* end: all zeroes */ }
+};
+
+static const struct dev_pm_ops penwell_otg_pm_ops = {
+	.runtime_suspend = penwell_otg_runtime_suspend,
+	.runtime_resume = penwell_otg_runtime_resume,
+	.runtime_idle = penwell_otg_runtime_idle,
+};
+
+static struct pci_driver otg_pci_driver = {
+	.name =		(char *) driver_name,
+	.id_table =	pci_ids,
+
+	.probe =	penwell_otg_probe,
+	.remove =	penwell_otg_remove,
+
+	.suspend =	penwell_otg_suspend,
+	.resume =	penwell_otg_resume,
+
+	.driver = {
+		.pm =	&penwell_otg_pm_ops
+	},
+};
+
 static int __init penwell_otg_init(void)
 {
 	return pci_register_driver(&otg_pci_driver);
diff --git a/include/linux/usb/intel_mid_otg.h b/include/linux/usb/intel_mid_otg.h
index c9ec416..62db841 100644
--- a/include/linux/usb/intel_mid_otg.h
+++ b/include/linux/usb/intel_mid_otg.h
@@ -143,6 +143,13 @@ struct intel_mid_otg_xceiv {
 	int	(*suspend_peripheral)(struct intel_mid_otg_xceiv *iotg,
 					pm_message_t message);
 	int	(*resume_peripheral)(struct intel_mid_otg_xceiv *iotg);
+
+	/* runtime suspend/resume */
+	int	(*runtime_suspend_host)(struct intel_mid_otg_xceiv *iotg);
+	int	(*runtime_resume_host)(struct intel_mid_otg_xceiv *iotg);
+	int	(*runtime_suspend_peripheral)(struct intel_mid_otg_xceiv *iotg);
+	int	(*runtime_resume_peripheral)(struct intel_mid_otg_xceiv *iotg);
+
 #endif
 
 };
-- 
1.6.0.6

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux