From: Grygorii Strashko <grygorii.strashko@ti.com> To: Kishon Vijay Abraham I <kishon@ti.com>, <bhelgaas@google.com>, <jingoohan1@gmail.com>, <pratyush.anand@gmail.com>, <linux-omap@vger.kernel.org>, <linux-pci@vger.kernel.org>, <linux-kernel@vger.kernel.org> Cc: <nsekhar@ti.com> Subject: Re: [PATCH 3/3] PCI: host: pci-dra7xx: add pm support to pci dra7xx Date: Fri, 3 Jul 2015 15:04:10 +0300 [thread overview] Message-ID: <55967A3A.7000907@ti.com> (raw) In-Reply-To: <1435921425-15121-4-git-send-email-kishon@ti.com> 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@ti.com> > --- > 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? > +}; > +#define DEV_PM_OPS (&dra7xx_pcie_pm_ops) > +#else > +#define DEV_PM_OPS NULL > +#endif > + > static const struct of_device_id of_dra7xx_pcie_match[] = { > { .compatible = "ti,dra7-pcie", }, > {}, > @@ -444,6 +517,7 @@ static struct platform_driver dra7xx_pcie_driver = { > .driver = { > .name = "dra7-pcie", > .of_match_table = of_dra7xx_pcie_match, > + .pm = DEV_PM_OPS, > }, > }; > > -- regards, -grygorii
WARNING: multiple messages have this Message-ID (diff)
From: Grygorii Strashko <grygorii.strashko@ti.com> To: Kishon Vijay Abraham I <kishon@ti.com>, bhelgaas@google.com, jingoohan1@gmail.com, pratyush.anand@gmail.com, linux-omap@vger.kernel.org, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Cc: nsekhar@ti.com Subject: Re: [PATCH 3/3] PCI: host: pci-dra7xx: add pm support to pci dra7xx Date: Fri, 3 Jul 2015 15:04:10 +0300 [thread overview] Message-ID: <55967A3A.7000907@ti.com> (raw) In-Reply-To: <1435921425-15121-4-git-send-email-kishon@ti.com> 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@ti.com> > --- > 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? > +}; > +#define DEV_PM_OPS (&dra7xx_pcie_pm_ops) > +#else > +#define DEV_PM_OPS NULL > +#endif > + > static const struct of_device_id of_dra7xx_pcie_match[] = { > { .compatible = "ti,dra7-pcie", }, > {}, > @@ -444,6 +517,7 @@ static struct platform_driver dra7xx_pcie_driver = { > .driver = { > .name = "dra7-pcie", > .of_match_table = of_dra7xx_pcie_match, > + .pm = DEV_PM_OPS, > }, > }; > > -- regards, -grygorii
next prev parent reply other threads:[~2015-07-03 12:04 UTC|newest] Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-07-03 11:03 [PATCH 0/3] J6/J6Eco: Add PM support to PCIe Kishon Vijay Abraham I 2015-07-03 11:03 ` Kishon Vijay Abraham I 2015-07-03 11:03 ` [PATCH 1/3] PCI: host: pci-dra7xx: Disable pm_runtime on get_sync failure Kishon Vijay Abraham I 2015-07-03 11:03 ` Kishon Vijay Abraham I 2015-07-03 11:03 ` [PATCH 2/3] PCI: host: pcie-designware: add support for suspend and resume Kishon Vijay Abraham I 2015-07-03 11:03 ` Kishon Vijay Abraham I 2015-07-10 13:22 ` Pratyush Anand 2015-07-10 14:49 ` Kishon Vijay Abraham I 2015-07-10 14:49 ` Kishon Vijay Abraham I 2015-07-10 15:30 ` Pratyush Anand 2015-07-10 15:41 ` Marek Vasut 2015-07-12 10:31 ` Jingoo Han 2015-07-12 10:31 ` Jingoo Han 2015-07-23 12:50 ` Kishon Vijay Abraham I 2015-07-23 12:50 ` Kishon Vijay Abraham I 2015-07-03 11:03 ` [PATCH 3/3] PCI: host: pci-dra7xx: add pm support to pci dra7xx Kishon Vijay Abraham I 2015-07-03 11:03 ` Kishon Vijay Abraham I 2015-07-03 12:04 ` Grygorii Strashko [this message] 2015-07-03 12:04 ` Grygorii Strashko 2015-07-10 14:50 ` Kishon Vijay Abraham I 2015-07-10 14:50 ` Kishon Vijay Abraham I
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=55967A3A.7000907@ti.com \ --to=grygorii.strashko@ti.com \ --cc=bhelgaas@google.com \ --cc=jingoohan1@gmail.com \ --cc=kishon@ti.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-omap@vger.kernel.org \ --cc=linux-pci@vger.kernel.org \ --cc=nsekhar@ti.com \ --cc=pratyush.anand@gmail.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.