linux-next: Tree for January 18 (pci + pm)

Prev: Matter of Urgency (GET BACK TO ME)
Next: [PATCH] lib/dma-debug.c: mark file-local struct symbol static.

From: Randy Dunlap on 18 Jan 2010 18:50

---

On Mon, 18 Jan 2010 19:57:43 +1100 Stephen Rothwell wrote:

> Hi all,
>
> Changes since 20100114:


drivers/pci/pci-driver.c: In function 'pci_pm_runtime_resume':
drivers/pci/pci-driver.c:991: error: implicit declaration of function 'pci_pm_default_resume_early'
drivers/pci/pci-driver.c: At top level:
drivers/pci/pci-driver.c:1027: error: 'pci_pm_prepare' undeclared here (not in a function)
drivers/pci/pci-driver.c:1028: error: 'pci_pm_complete' undeclared here (not in a function)
drivers/pci/pci-driver.c:1029: error: 'pci_pm_suspend' undeclared here (not in a function)
drivers/pci/pci-driver.c:1030: error: 'pci_pm_resume' undeclared here (not in a function)
drivers/pci/pci-driver.c:1031: error: 'pci_pm_freeze' undeclared here (not in a function)
drivers/pci/pci-driver.c:1032: error: 'pci_pm_thaw' undeclared here (not in a function)
drivers/pci/pci-driver.c:1033: error: 'pci_pm_poweroff' undeclared here (not in a function)
drivers/pci/pci-driver.c:1034: error: 'pci_pm_restore' undeclared here (not in a function)
drivers/pci/pci-driver.c:1035: error: 'pci_pm_suspend_noirq' undeclared here (not in a function)
drivers/pci/pci-driver.c:1036: error: 'pci_pm_resume_noirq' undeclared here (not in a function)
drivers/pci/pci-driver.c:1037: error: 'pci_pm_freeze_noirq' undeclared here (not in a function)
drivers/pci/pci-driver.c:1038: error: 'pci_pm_thaw_noirq' undeclared here (not in a function)
drivers/pci/pci-driver.c:1039: error: 'pci_pm_poweroff_noirq' undeclared here (not in a function)
drivers/pci/pci-driver.c:1040: error: 'pci_pm_restore_noirq' undeclared here (not in a function)


partial config:

CONFIG_PM=y
# CONFIG_PM_DEBUG is not set
# CONFIG_SUSPEND is not set
# CONFIG_HIBERNATION is not set
CONFIG_PM_RUNTIME=y
CONFIG_PM_OPS=y


Full config is attached.

---
~Randy

From: Rafael J. Wysocki on 19 Jan 2010 18:10

---

On Tuesday 19 January 2010, Randy Dunlap wrote:
> On Mon, 18 Jan 2010 19:57:43 +1100 Stephen Rothwell wrote:
>
> > Hi all,
> >
> > Changes since 20100114:
>
>
> drivers/pci/pci-driver.c: In function 'pci_pm_runtime_resume':
> drivers/pci/pci-driver.c:991: error: implicit declaration of function 'pci_pm_default_resume_early'
> drivers/pci/pci-driver.c: At top level:
> drivers/pci/pci-driver.c:1027: error: 'pci_pm_prepare' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1028: error: 'pci_pm_complete' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1029: error: 'pci_pm_suspend' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1030: error: 'pci_pm_resume' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1031: error: 'pci_pm_freeze' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1032: error: 'pci_pm_thaw' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1033: error: 'pci_pm_poweroff' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1034: error: 'pci_pm_restore' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1035: error: 'pci_pm_suspend_noirq' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1036: error: 'pci_pm_resume_noirq' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1037: error: 'pci_pm_freeze_noirq' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1038: error: 'pci_pm_thaw_noirq' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1039: error: 'pci_pm_poweroff_noirq' undeclared here (not in a function)
> drivers/pci/pci-driver.c:1040: error: 'pci_pm_restore_noirq' undeclared here (not in a function)
>
>
> partial config:
>
> CONFIG_PM=y
> # CONFIG_PM_DEBUG is not set
> # CONFIG_SUSPEND is not set
> # CONFIG_HIBERNATION is not set
> CONFIG_PM_RUNTIME=y
> CONFIG_PM_OPS=y
>
>
> Full config is attached.

Ah, too many ifdefs.

Thanks, the patch below fixes that for me.

Rafael

---
From: Rafael J. Wysocki <rjw(a)sisk.pl>
Subject: PCI / PM: Fix compilation for set CONFIG_PM_RUNTIME and CONFIG_PM_SLEEP unset

Fix failure to compile drivers/pci/pci-driver.c for CONFIG_PM_RUNTIME
set and CONFIG_PM_SLEEP is unset, introduced by the PCI run-time
power management patches.

Signed-off-by: Rafael J. Wysocki <rjw(a)sisk.pl>
Reported-by: Randy Dunlap <randy.dunlap(a)oracle.com>
---
drivers/pci/pci-driver.c | 61 +++++++++++++++++++++++++++--------------------
1 file changed, 36 insertions(+), 25 deletions(-)

Index: linux-2.6/drivers/pci/pci-driver.c
===================================================================
--- linux-2.6.orig/drivers/pci/pci-driver.c
+++ linux-2.6/drivers/pci/pci-driver.c
@@ -405,6 +405,35 @@ static void pci_device_shutdown(struct d
pci_msix_shutdown(pci_dev);
}

+#ifdef CONFIG_PM_OPS
+
+/* Auxiliary functions used for system resume and run-time resume. */
+
+/**
+ * pci_restore_standard_config - restore standard config registers of PCI device
+ * @pci_dev: PCI device to handle
+ */
+static int pci_restore_standard_config(struct pci_dev *pci_dev)
+{
+ pci_update_current_state(pci_dev, PCI_UNKNOWN);
+
+ if (pci_dev->current_state != PCI_D0) {
+ int error = pci_set_power_state(pci_dev, PCI_D0);
+ if (error)
+ return error;
+ }
+
+ return pci_restore_state(pci_dev);
+}
+
+static void pci_pm_default_resume_early(struct pci_dev *pci_dev)
+{
+ pci_restore_standard_config(pci_dev);
+ pci_fixup_device(pci_fixup_resume_early, pci_dev);
+}
+
+#endif
+
#ifdef CONFIG_PM_SLEEP

/*
@@ -521,29 +550,6 @@ static int pci_legacy_resume(struct devi

/* Auxiliary functions used by the new power management framework */

-/**
- * pci_restore_standard_config - restore standard config registers of PCI device
- * @pci_dev: PCI device to handle
- */
-static int pci_restore_standard_config(struct pci_dev *pci_dev)
-{
- pci_update_current_state(pci_dev, PCI_UNKNOWN);
-
- if (pci_dev->current_state != PCI_D0) {
- int error = pci_set_power_state(pci_dev, PCI_D0);
- if (error)
- return error;
- }
-
- return pci_restore_state(pci_dev);
-}
-
-static void pci_pm_default_resume_early(struct pci_dev *pci_dev)
-{
- pci_restore_standard_config(pci_dev);
- pci_fixup_device(pci_fixup_resume_early, pci_dev);
-}
-
static void pci_pm_default_resume(struct pci_dev *pci_dev)
{
pci_fixup_device(pci_fixup_resume, pci_dev);
@@ -607,6 +613,13 @@ static void pci_pm_complete(struct devic
drv->pm->complete(dev);
}

+#else /* !CONFIG_PM_SLEEP */
+
+#define pci_pm_prepare NULL
+#define pci_pm_complete NULL
+
+#endif /* !CONFIG_PM_SLEEP */
+
#ifdef CONFIG_SUSPEND

static int pci_pm_suspend(struct device *dev)
@@ -943,8 +956,6 @@ static int pci_pm_restore(struct device

#endif /* !CONFIG_HIBERNATION */

-#endif /* !CONFIG_PM_SLEEP */
-
#ifdef CONFIG_PM_RUNTIME

static int pci_pm_runtime_suspend(struct device *dev)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

From: Jesse Barnes on 19 Jan 2010 23:50

---

On Wed, 20 Jan 2010 00:04:07 +0100
"Rafael J. Wysocki" <rjw(a)sisk.pl> wrote:

> On Tuesday 19 January 2010, Randy Dunlap wrote:
> > On Mon, 18 Jan 2010 19:57:43 +1100 Stephen Rothwell wrote:
> >
> > > Hi all,
> > >
> > > Changes since 20100114:
> >
> >
> > drivers/pci/pci-driver.c: In function 'pci_pm_runtime_resume':
> > drivers/pci/pci-driver.c:991: error: implicit declaration of
> > function 'pci_pm_default_resume_early' drivers/pci/pci-driver.c: At
> > top level: drivers/pci/pci-driver.c:1027: error: 'pci_pm_prepare'
> > undeclared here (not in a function) drivers/pci/pci-driver.c:1028:
> > error: 'pci_pm_complete' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1029: error: 'pci_pm_suspend' undeclared
> > here (not in a function) drivers/pci/pci-driver.c:1030: error:
> > 'pci_pm_resume' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1031: error: 'pci_pm_freeze' undeclared
> > here (not in a function) drivers/pci/pci-driver.c:1032: error:
> > 'pci_pm_thaw' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1033: error: 'pci_pm_poweroff' undeclared
> > here (not in a function) drivers/pci/pci-driver.c:1034: error:
> > 'pci_pm_restore' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1035: error: 'pci_pm_suspend_noirq'
> > undeclared here (not in a function) drivers/pci/pci-driver.c:1036:
> > error: 'pci_pm_resume_noirq' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1037: error: 'pci_pm_freeze_noirq'
> > undeclared here (not in a function) drivers/pci/pci-driver.c:1038:
> > error: 'pci_pm_thaw_noirq' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1039: error: 'pci_pm_poweroff_noirq'
> > undeclared here (not in a function) drivers/pci/pci-driver.c:1040:
> > error: 'pci_pm_restore_noirq' undeclared here (not in a function)
> >
> >
> > partial config:
> >
> > CONFIG_PM=y
> > # CONFIG_PM_DEBUG is not set
> > # CONFIG_SUSPEND is not set
> > # CONFIG_HIBERNATION is not set
> > CONFIG_PM_RUNTIME=y
> > CONFIG_PM_OPS=y
> >
> >
> > Full config is attached.
>
> Ah, too many ifdefs.
>
> Thanks, the patch below fixes that for me.
>
> Rafael
>
> ---
> From: Rafael J. Wysocki <rjw(a)sisk.pl>
> Subject: PCI / PM: Fix compilation for set CONFIG_PM_RUNTIME and
> CONFIG_PM_SLEEP unset
>
> Fix failure to compile drivers/pci/pci-driver.c for CONFIG_PM_RUNTIME
> set and CONFIG_PM_SLEEP is unset, introduced by the PCI run-time
> power management patches.
>
> Signed-off-by: Rafael J. Wysocki <rjw(a)sisk.pl>
> Reported-by: Randy Dunlap <randy.dunlap(a)oracle.com>
> ---
> drivers/pci/pci-driver.c | 61
> +++++++++++++++++++++++++++-------------------- 1 file changed, 36
> insertions(+), 25 deletions(-)
>
> Index: linux-2.6/drivers/pci/pci-driver.c
> ===================================================================
> --- linux-2.6.orig/drivers/pci/pci-driver.c
> +++ linux-2.6/drivers/pci/pci-driver.c
> @@ -405,6 +405,35 @@ static void pci_device_shutdown(struct d
> pci_msix_shutdown(pci_dev);
> }
>
> +#ifdef CONFIG_PM_OPS
> +
> +/* Auxiliary functions used for system resume and run-time resume. */
> +
> +/**
> + * pci_restore_standard_config - restore standard config registers
> of PCI device
> + * @pci_dev: PCI device to handle
> + */
> +static int pci_restore_standard_config(struct pci_dev *pci_dev)
> +{
> + pci_update_current_state(pci_dev, PCI_UNKNOWN);
> +
> + if (pci_dev->current_state != PCI_D0) {
> + int error = pci_set_power_state(pci_dev, PCI_D0);
> + if (error)
> + return error;
> + }
> +
> + return pci_restore_state(pci_dev);
> +}
> +
> +static void pci_pm_default_resume_early(struct pci_dev *pci_dev)
> +{
> + pci_restore_standard_config(pci_dev);
> + pci_fixup_device(pci_fixup_resume_early, pci_dev);
> +}
> +
> +#endif
> +
> #ifdef CONFIG_PM_SLEEP
>
> /*
> @@ -521,29 +550,6 @@ static int pci_legacy_resume(struct devi
>
> /* Auxiliary functions used by the new power management framework */
>
> -/**
> - * pci_restore_standard_config - restore standard config registers
> of PCI device
> - * @pci_dev: PCI device to handle
> - */
> -static int pci_restore_standard_config(struct pci_dev *pci_dev)
> -{
> - pci_update_current_state(pci_dev, PCI_UNKNOWN);
> -
> - if (pci_dev->current_state != PCI_D0) {
> - int error = pci_set_power_state(pci_dev, PCI_D0);
> - if (error)
> - return error;
> - }
> -
> - return pci_restore_state(pci_dev);
> -}
> -
> -static void pci_pm_default_resume_early(struct pci_dev *pci_dev)
> -{
> - pci_restore_standard_config(pci_dev);
> - pci_fixup_device(pci_fixup_resume_early, pci_dev);
> -}
> -
> static void pci_pm_default_resume(struct pci_dev *pci_dev)
> {
> pci_fixup_device(pci_fixup_resume, pci_dev);
> @@ -607,6 +613,13 @@ static void pci_pm_complete(struct devic
> drv->pm->complete(dev);
> }
>
> +#else /* !CONFIG_PM_SLEEP */
> +
> +#define pci_pm_prepare NULL
> +#define pci_pm_complete NULL
> +
> +#endif /* !CONFIG_PM_SLEEP */
> +
> #ifdef CONFIG_SUSPEND
>
> static int pci_pm_suspend(struct device *dev)
> @@ -943,8 +956,6 @@ static int pci_pm_restore(struct device
>
> #endif /* !CONFIG_HIBERNATION */
>
> -#endif /* !CONFIG_PM_SLEEP */
> -
> #ifdef CONFIG_PM_RUNTIME
>
> static int pci_pm_runtime_suspend(struct device *dev)
>

Pushed this one into linux-next, thanks.

Jesse
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

From: Stephen Rothwell on 20 Jan 2010 00:30

---

Hi Jesse,

On Wed, 20 Jan 2010 06:39:45 +0200 Jesse Barnes <jbarnes(a)virtuousgeek.org> wrote:
>
> Pushed this one into linux-next, thanks.

Great, thanks.

--
Cheers,
Stephen Rothwell sfr(a)canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

From: Randy Dunlap on 20 Jan 2010 12:00

---

On Wed, 20 Jan 2010 00:04:07 +0100 Rafael J. Wysocki wrote:

> On Tuesday 19 January 2010, Randy Dunlap wrote:
> > On Mon, 18 Jan 2010 19:57:43 +1100 Stephen Rothwell wrote:
> >
> > > Hi all,
> > >
> > > Changes since 20100114:
> >
> >
> > drivers/pci/pci-driver.c: In function 'pci_pm_runtime_resume':
> > drivers/pci/pci-driver.c:991: error: implicit declaration of function 'pci_pm_default_resume_early'
> > drivers/pci/pci-driver.c: At top level:
> > drivers/pci/pci-driver.c:1027: error: 'pci_pm_prepare' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1028: error: 'pci_pm_complete' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1029: error: 'pci_pm_suspend' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1030: error: 'pci_pm_resume' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1031: error: 'pci_pm_freeze' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1032: error: 'pci_pm_thaw' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1033: error: 'pci_pm_poweroff' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1034: error: 'pci_pm_restore' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1035: error: 'pci_pm_suspend_noirq' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1036: error: 'pci_pm_resume_noirq' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1037: error: 'pci_pm_freeze_noirq' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1038: error: 'pci_pm_thaw_noirq' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1039: error: 'pci_pm_poweroff_noirq' undeclared here (not in a function)
> > drivers/pci/pci-driver.c:1040: error: 'pci_pm_restore_noirq' undeclared here (not in a function)
> >
> >
> > partial config:
> >
> > CONFIG_PM=y
> > # CONFIG_PM_DEBUG is not set
> > # CONFIG_SUSPEND is not set
> > # CONFIG_HIBERNATION is not set
> > CONFIG_PM_RUNTIME=y
> > CONFIG_PM_OPS=y
> >
> >
> > Full config is attached.
>
> Ah, too many ifdefs.
>
> Thanks, the patch below fixes that for me.

Acked-by: Randy Dunlap <randy.dunlap(a)oracle.com>

Thanks.


> Rafael
>
> ---
> From: Rafael J. Wysocki <rjw(a)sisk.pl>
> Subject: PCI / PM: Fix compilation for set CONFIG_PM_RUNTIME and CONFIG_PM_SLEEP unset
>
> Fix failure to compile drivers/pci/pci-driver.c for CONFIG_PM_RUNTIME
> set and CONFIG_PM_SLEEP is unset, introduced by the PCI run-time
> power management patches.
>
> Signed-off-by: Rafael J. Wysocki <rjw(a)sisk.pl>
> Reported-by: Randy Dunlap <randy.dunlap(a)oracle.com>
> ---
> drivers/pci/pci-driver.c | 61 +++++++++++++++++++++++++++--------------------
> 1 file changed, 36 insertions(+), 25 deletions(-)
>
> Index: linux-2.6/drivers/pci/pci-driver.c
> ===================================================================
> --- linux-2.6.orig/drivers/pci/pci-driver.c
> +++ linux-2.6/drivers/pci/pci-driver.c
> @@ -405,6 +405,35 @@ static void pci_device_shutdown(struct d
> pci_msix_shutdown(pci_dev);
> }
>
> +#ifdef CONFIG_PM_OPS
> +
> +/* Auxiliary functions used for system resume and run-time resume. */
> +
> +/**
> + * pci_restore_standard_config - restore standard config registers of PCI device
> + * @pci_dev: PCI device to handle
> + */
> +static int pci_restore_standard_config(struct pci_dev *pci_dev)
> +{
> + pci_update_current_state(pci_dev, PCI_UNKNOWN);
> +
> + if (pci_dev->current_state != PCI_D0) {
> + int error = pci_set_power_state(pci_dev, PCI_D0);
> + if (error)
> + return error;
> + }
> +
> + return pci_restore_state(pci_dev);
> +}
> +
> +static void pci_pm_default_resume_early(struct pci_dev *pci_dev)
> +{
> + pci_restore_standard_config(pci_dev);
> + pci_fixup_device(pci_fixup_resume_early, pci_dev);
> +}
> +
> +#endif
> +
> #ifdef CONFIG_PM_SLEEP
>
> /*
> @@ -521,29 +550,6 @@ static int pci_legacy_resume(struct devi
>
> /* Auxiliary functions used by the new power management framework */
>
> -/**
> - * pci_restore_standard_config - restore standard config registers of PCI device
> - * @pci_dev: PCI device to handle
> - */
> -static int pci_restore_standard_config(struct pci_dev *pci_dev)
> -{
> - pci_update_current_state(pci_dev, PCI_UNKNOWN);
> -
> - if (pci_dev->current_state != PCI_D0) {
> - int error = pci_set_power_state(pci_dev, PCI_D0);
> - if (error)
> - return error;
> - }
> -
> - return pci_restore_state(pci_dev);
> -}
> -
> -static void pci_pm_default_resume_early(struct pci_dev *pci_dev)
> -{
> - pci_restore_standard_config(pci_dev);
> - pci_fixup_device(pci_fixup_resume_early, pci_dev);
> -}
> -
> static void pci_pm_default_resume(struct pci_dev *pci_dev)
> {
> pci_fixup_device(pci_fixup_resume, pci_dev);
> @@ -607,6 +613,13 @@ static void pci_pm_complete(struct devic
> drv->pm->complete(dev);
> }
>
> +#else /* !CONFIG_PM_SLEEP */
> +
> +#define pci_pm_prepare NULL
> +#define pci_pm_complete NULL
> +
> +#endif /* !CONFIG_PM_SLEEP */
> +
> #ifdef CONFIG_SUSPEND
>
> static int pci_pm_suspend(struct device *dev)
> @@ -943,8 +956,6 @@ static int pci_pm_restore(struct device
>
> #endif /* !CONFIG_HIBERNATION */
>
> -#endif /* !CONFIG_PM_SLEEP */
> -
> #ifdef CONFIG_PM_RUNTIME
>
> static int pci_pm_runtime_suspend(struct device *dev)


---
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

 |  Next  |  Last
Pages: 1 2
Prev: Matter of Urgency (GET BACK TO ME)
Next: [PATCH] lib/dma-debug.c: mark file-local struct symbol static.