Hi, On Friday 03 July 2015 05:34 PM, Grygorii Strashko wrote: > Hi Kishon, > > On 07/03/2015 02:03 PM, Kishon Vijay Abraham I wrote: >> Add PM support to pci-dra7xx so that PCI clocks can be disabled >> during suspend and enabled back during resume without affecting >> PCI functionality. >> >> Signed-off-by: Kishon Vijay Abraham I <kishon@xxxxxx> >> --- >> drivers/pci/host/pci-dra7xx.c | 74 +++++++++++++++++++++++++++++++++++++++++ >> 1 file changed, 74 insertions(+) >> >> diff --git a/drivers/pci/host/pci-dra7xx.c b/drivers/pci/host/pci-dra7xx.c >> index d8b6d66..1f5c039 100644 >> --- a/drivers/pci/host/pci-dra7xx.c >> +++ b/drivers/pci/host/pci-dra7xx.c >> @@ -433,6 +433,79 @@ static int __exit dra7xx_pcie_remove(struct platform_device *pdev) >> return 0; >> } >> >> +#ifdef CONFIG_PM_SLEEP > > [...] > >> + >> +static const struct dev_pm_ops dra7xx_pcie_pm_ops = { >> + .suspend_noirq = dra7xx_pcie_suspend_noirq, >> + .suspend = dra7xx_pcie_suspend, >> + .resume_noirq = dra7xx_pcie_resume_noirq, >> + .resume = dra7xx_pcie_resume, > > Could you use here SET_SYSTEM_SLEEP_PM_OPS() > and SET_NOIRQ_SYSTEM_SLEEP_PM_OPS() macro, pls? sure! Thanks Kishon -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html