| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 13 Feb 2012 21:49:15 +0100
From: "Rafael J. Wysocki" <rjw@...k.pl>
To: Lin Ming <ming.m.lin@...el.com>
Cc: Zhang Rui <rui.zhang@...el.com>, Jeff Garzik <jgarzik@...ox.com>,
Alan Stern <stern@...land.harvard.edu>,
Tejun Heo <tj@...nel.org>, Len Brown <lenb@...nel.org>,
linux-kernel@...r.kernel.org, linux-ide@...r.kernel.org,
linux-scsi@...r.kernel.org, linux-pm@...r.kernel.org
Subject: Re: [RFC PATCH 5/6] PCI: Move acpi_dev_run_wake to acpi core
On Monday, February 13, 2012, Lin Ming wrote:
> acpi_dev_run_wake is a generic function which can be used by
> other subsystem too. Rename it to acpi_pm_device_run_wake, to be
> consistent with acpi_pm_device_sleep_wake.
>
> Then move it to acpi core.
>
> Signed-off-by: Lin Ming <ming.m.lin@...el.com>
OK, why not.
Acked-by: Rafael J. Wysocki <rjw@...k.pl>
> ---
> drivers/acpi/sleep.c | 35 +++++++++++++++++++++++++++++++++++
> drivers/pci/pci-acpi.c | 40 +++-------------------------------------
> include/acpi/acpi_bus.h | 5 +++++
> 3 files changed, 43 insertions(+), 37 deletions(-)
>
> diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
> index ca191ff..00fa664 100644
> --- a/drivers/acpi/sleep.c
> +++ b/drivers/acpi/sleep.c
> @@ -17,6 +17,7 @@
> #include <linux/suspend.h>
> #include <linux/reboot.h>
> #include <linux/acpi.h>
> +#include <linux/pm_runtime.h>
>
> #include <asm/io.h>
>
> @@ -730,6 +731,40 @@ int acpi_pm_device_sleep_state(struct device *dev, int *d_min_p)
>
> #ifdef CONFIG_PM_SLEEP
> /**
> + * acpi_pm_device_run_wake - Enable/disable wake-up for given device.
> + * @phys_dev: Device to enable/disable the platform to wake-up the system for.
> + * @enable: Whether enable or disable the wake-up functionality.
> + *
> + * Find the ACPI device object corresponding to @pci_dev and try to
> + * enable/disable the GPE associated with it.
> + */
> +int acpi_pm_device_run_wake(struct device *phys_dev, bool enable)
> +{
> + struct acpi_device *dev;
> + acpi_handle handle;
> +
> + if (!device_run_wake(phys_dev))
> + return -EINVAL;
> +
> + handle = DEVICE_ACPI_HANDLE(phys_dev);
> + if (!handle || ACPI_FAILURE(acpi_bus_get_device(handle, &dev))) {
> + dev_dbg(phys_dev, "ACPI handle has no context in %s!\n",
> + __func__);
> + return -ENODEV;
> + }
> +
> + if (enable) {
> + acpi_enable_wakeup_device_power(dev, ACPI_STATE_S0);
> + acpi_enable_gpe(dev->wakeup.gpe_device, dev->wakeup.gpe_number);
> + } else {
> + acpi_disable_gpe(dev->wakeup.gpe_device, dev->wakeup.gpe_number);
> + acpi_disable_wakeup_device_power(dev);
> + }
> +
> + return 0;
> +}
> +
> +/**
> * acpi_pm_device_sleep_wake - enable or disable the system wake-up
> * capability of given device
> * @dev: device to handle
> diff --git a/drivers/pci/pci-acpi.c b/drivers/pci/pci-acpi.c
> index 060fd22..0f150f2 100644
> --- a/drivers/pci/pci-acpi.c
> +++ b/drivers/pci/pci-acpi.c
> @@ -277,40 +277,6 @@ static int acpi_pci_sleep_wake(struct pci_dev *dev, bool enable)
> return 0;
> }
>
> -/**
> - * acpi_dev_run_wake - Enable/disable wake-up for given device.
> - * @phys_dev: Device to enable/disable the platform to wake-up the system for.
> - * @enable: Whether enable or disable the wake-up functionality.
> - *
> - * Find the ACPI device object corresponding to @pci_dev and try to
> - * enable/disable the GPE associated with it.
> - */
> -static int acpi_dev_run_wake(struct device *phys_dev, bool enable)
> -{
> - struct acpi_device *dev;
> - acpi_handle handle;
> -
> - if (!device_run_wake(phys_dev))
> - return -EINVAL;
> -
> - handle = DEVICE_ACPI_HANDLE(phys_dev);
> - if (!handle || ACPI_FAILURE(acpi_bus_get_device(handle, &dev))) {
> - dev_dbg(phys_dev, "ACPI handle has no context in %s!\n",
> - __func__);
> - return -ENODEV;
> - }
> -
> - if (enable) {
> - acpi_enable_wakeup_device_power(dev, ACPI_STATE_S0);
> - acpi_enable_gpe(dev->wakeup.gpe_device, dev->wakeup.gpe_number);
> - } else {
> - acpi_disable_gpe(dev->wakeup.gpe_device, dev->wakeup.gpe_number);
> - acpi_disable_wakeup_device_power(dev);
> - }
> -
> - return 0;
> -}
> -
> static void acpi_pci_propagate_run_wake(struct pci_bus *bus, bool enable)
> {
> while (bus->parent) {
> @@ -318,14 +284,14 @@ static void acpi_pci_propagate_run_wake(struct pci_bus *bus, bool enable)
>
> if (bridge->pme_interrupt)
> return;
> - if (!acpi_dev_run_wake(&bridge->dev, enable))
> + if (!acpi_pm_device_run_wake(&bridge->dev, enable))
> return;
> bus = bus->parent;
> }
>
> /* We have reached the root bus. */
> if (bus->bridge)
> - acpi_dev_run_wake(bus->bridge, enable);
> + acpi_pm_device_run_wake(bus->bridge, enable);
> }
>
> static int acpi_pci_run_wake(struct pci_dev *dev, bool enable)
> @@ -333,7 +299,7 @@ static int acpi_pci_run_wake(struct pci_dev *dev, bool enable)
> if (dev->pme_interrupt)
> return 0;
>
> - if (!acpi_dev_run_wake(&dev->dev, enable))
> + if (!acpi_pm_device_run_wake(&dev->dev, enable))
> return 0;
>
> acpi_pci_propagate_run_wake(dev->bus, enable);
> diff --git a/include/acpi/acpi_bus.h b/include/acpi/acpi_bus.h
> index 32fb899..2d556e3 100644
> --- a/include/acpi/acpi_bus.h
> +++ b/include/acpi/acpi_bus.h
> @@ -394,8 +394,13 @@ static inline int acpi_pm_device_sleep_state(struct device *d, int *p)
> #endif
>
> #ifdef CONFIG_PM_SLEEP
> +int acpi_pm_device_run_wake(struct device *, bool);
> int acpi_pm_device_sleep_wake(struct device *, bool);
> #else
> +static inline int acpi_pm_device_run_wake(struct device *dev, bool enable)
> +{
> + return -ENODEV;
> +}
> static inline int acpi_pm_device_sleep_wake(struct device *dev, bool enable)
> {
> return -ENODEV;
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists