2016-03-16 10:04:12

by Jisheng Zhang

Subject: [PATCH 0/2] PCI: mvebu: trivial cleanup

trivial clean ups to pci-mvebu.c

Jisheng Zhang (2):
PCI: mvebu: constify mvebu_pcie_pm_ops structure
PCI: mvebu: use SET_NOIRQ_SYSTEM_SLEEP_PM_OPS for mvebu_pcie_pm_ops

drivers/pci/host/pci-mvebu.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)

--
2.7.0

2016-03-16 10:04:07

by Jisheng Zhang

Subject: [PATCH 2/2] PCI: mvebu: use SET_NOIRQ_SYSTEM_SLEEP_PM_OPS for mvebu_pcie_pm_ops

use SET_NOIRQ_SYSTEM_SLEEP_PM_OPS helper macro for mvebu_pcie_pm_ops.
The macro also setups freeze_noirq, thaw_noirq and poweroff_noirq,
restore_noirq accordingly.

Signed-off-by: Jisheng Zhang <[email protected]>
---
drivers/pci/host/pci-mvebu.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/pci/host/pci-mvebu.c b/drivers/pci/host/pci-mvebu.c
index 6c43cb8..6b451df 100644
--- a/drivers/pci/host/pci-mvebu.c
+++ b/drivers/pci/host/pci-mvebu.c
@@ -1003,6 +1003,7 @@ static void mvebu_pcie_msi_enable(struct mvebu_pcie *pcie)
pcie->msi->dev = &pcie->pdev->dev;
}

+#ifdef CONFIG_PM_SLEEP
static int mvebu_pcie_suspend(struct device *dev)
{
struct mvebu_pcie *pcie;
@@ -1031,6 +1032,7 @@ static int mvebu_pcie_resume(struct device *dev)

return 0;
}
+#endif

static void mvebu_pcie_port_clk_put(void *data)
{
@@ -1299,8 +1301,7 @@ static const struct of_device_id mvebu_pcie_of_match_table[] = {
MODULE_DEVICE_TABLE(of, mvebu_pcie_of_match_table);

static const struct dev_pm_ops mvebu_pcie_pm_ops = {
- .suspend_noirq = mvebu_pcie_suspend,
- .resume_noirq = mvebu_pcie_resume,
+ SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(mvebu_pcie_suspend, mvebu_pcie_resume)
};

static struct platform_driver mvebu_pcie_driver = {
--
2.7.0

2016-03-16 10:04:43

by Jisheng Zhang

Subject: [PATCH 1/2] PCI: mvebu: constify mvebu_pcie_pm_ops structure

The mvebu_pcie_pm_ops structure is never modified, so declare it as
const.

Signed-off-by: Jisheng Zhang <[email protected]>
---
drivers/pci/host/pci-mvebu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/host/pci-mvebu.c b/drivers/pci/host/pci-mvebu.c
index 53b79c5..6c43cb8 100644
--- a/drivers/pci/host/pci-mvebu.c
+++ b/drivers/pci/host/pci-mvebu.c
@@ -1298,7 +1298,7 @@ static const struct of_device_id mvebu_pcie_of_match_table[] = {
};
MODULE_DEVICE_TABLE(of, mvebu_pcie_of_match_table);

-static struct dev_pm_ops mvebu_pcie_pm_ops = {
+static const struct dev_pm_ops mvebu_pcie_pm_ops = {
.suspend_noirq = mvebu_pcie_suspend,
.resume_noirq = mvebu_pcie_resume,
};
--
2.7.0

2016-03-16 10:07:29

by Thomas Petazzoni

Subject: Re: [PATCH 1/2] PCI: mvebu: constify mvebu_pcie_pm_ops structure

Dear Jisheng Zhang,

On Wed, 16 Mar 2016 17:59:40 +0800, Jisheng Zhang wrote:
> The mvebu_pcie_pm_ops structure is never modified, so declare it as
> const.
>
> Signed-off-by: Jisheng Zhang <[email protected]>
> ---
> drivers/pci/host/pci-mvebu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Thomas Petazzoni <[email protected]>

Thanks!

Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

2016-03-16 10:09:03

by Thomas Petazzoni

Subject: Re: [PATCH 2/2] PCI: mvebu: use SET_NOIRQ_SYSTEM_SLEEP_PM_OPS for mvebu_pcie_pm_ops

Dear Jisheng Zhang,

On Wed, 16 Mar 2016 17:59:41 +0800, Jisheng Zhang wrote:
> use SET_NOIRQ_SYSTEM_SLEEP_PM_OPS helper macro for mvebu_pcie_pm_ops.
> The macro also setups freeze_noirq, thaw_noirq and poweroff_noirq,
> restore_noirq accordingly.
>
> Signed-off-by: Jisheng Zhang <[email protected]>
> ---
> drivers/pci/host/pci-mvebu.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)

Reviewed-by: Thomas Petazzoni <[email protected]>

Thanks!

Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

2016-04-07 16:54:15

by Bjorn Helgaas

Subject: Re: [PATCH 0/2] PCI: mvebu: trivial cleanup

On Wed, Mar 16, 2016 at 05:59:39PM +0800, Jisheng Zhang wrote:
> trivial clean ups to pci-mvebu.c
>
> Jisheng Zhang (2):
> PCI: mvebu: constify mvebu_pcie_pm_ops structure
> PCI: mvebu: use SET_NOIRQ_SYSTEM_SLEEP_PM_OPS for mvebu_pcie_pm_ops

Applied with Thomas' ack to pci/host-mvebu for v4.7, thanks!