hollalinux
/
linux-sg2042
mirror of https://github.com/sophgo/linux-riscv.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

usb: gadged: pch_udc: PCI core handles power state for us 

There is no need to repeat the work that is already done in the PCI
driver core. The patch removes excerpts from suspend and resume
callbacks.

Note that there is no more calls performed to enable or disable a PCI
device during suspend-resume cycle. Nowadays they seems to be
superfluous. Someone can read more in [1].

While here, convert PM ops to use modern API.

[1] https://www.kernel.org/doc/ols/2009/ols2009-pages-319-330.pdf

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

[felipe.balbi@linux.intel.com: fixed build break and checkpatch error ]

Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
This commit is contained in:
Andy Shevchenko 2016-03-18 16:55:34 +02:00 committed by Felipe Balbi
parent badf6d47f8
commit 5e3bd45f17
1 changed files with 12 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

39
drivers/usb/gadget/udc/pch_udc.c
View File

@ -3096,44 +3096,28 @@ static void pch_udc_remove(struct pci_dev *pdev)
kfree(dev); kfree(dev);
} }
#ifdef CONFIG_PM #ifdef CONFIG_PM_SLEEP
static int pch_udc_suspend(struct pci_dev *pdev, pm_message_t state) static int pch_udc_suspend(struct device *d)
{ {
struct pci_dev *pdev = to_pci_dev(d);
struct pch_udc_dev *dev = pci_get_drvdata(pdev); struct pch_udc_dev *dev = pci_get_drvdata(pdev);
pch_udc_disable_interrupts(dev, UDC_DEVINT_MSK); pch_udc_disable_interrupts(dev, UDC_DEVINT_MSK);
pch_udc_disable_ep_interrupts(dev, UDC_EPINT_MSK_DISABLE_ALL); pch_udc_disable_ep_interrupts(dev, UDC_EPINT_MSK_DISABLE_ALL);
pci_disable_device(pdev);
pci_enable_wake(pdev, PCI_D3hot, 0);
if (pci_save_state(pdev)) {
dev_err(&pdev->dev,
"%s: could not save PCI config state\n", __func__);
return -ENOMEM;
}
pci_set_power_state(pdev, pci_choose_state(pdev, state));
return 0; return 0;
} }
static int pch_udc_resume(struct pci_dev *pdev) static int pch_udc_resume(struct device *d)
{ {
int ret;
pci_set_power_state(pdev, PCI_D0);
pci_restore_state(pdev);
ret = pci_enable_device(pdev);
if (ret) {
dev_err(&pdev->dev, "%s: pci_enable_device failed\n", __func__);
return ret;
}
pci_enable_wake(pdev, PCI_D3hot, 0);
return 0; return 0;
} }
static SIMPLE_DEV_PM_OPS(pch_udc_pm, pch_udc_suspend, pch_udc_resume);
#define PCH_UDC_PM_OPS (&pch_udc_pm)
#else #else
#define pch_udc_suspend NULL #define PCH_UDC_PM_OPS NULL
#define pch_udc_resume NULL #endif /* CONFIG_PM_SLEEP */
#endif /* CONFIG_PM */
static int pch_udc_probe(struct pci_dev *pdev, static int pch_udc_probe(struct pci_dev *pdev,
const struct pci_device_id *id) const struct pci_device_id *id)
@ -3262,9 +3246,10 @@ static struct pci_driver pch_udc_driver = {
.id_table = pch_udc_pcidev_id, .id_table = pch_udc_pcidev_id,
.probe = pch_udc_probe, .probe = pch_udc_probe,
.remove = pch_udc_remove, .remove = pch_udc_remove,
.suspend = pch_udc_suspend,
.resume = pch_udc_resume,
.shutdown = pch_udc_shutdown, .shutdown = pch_udc_shutdown,
.driver = {
.pm = PCH_UDC_PM_OPS,
},
}; };
module_pci_driver(pch_udc_driver); module_pci_driver(pch_udc_driver);