PM / Domains: Rename struct dev_power_domain to struct dev_pm_domain

The naming convention used by commit 7538e3db6e015e890825fbd9f86599b
(PM: Add support for device power domains), which introduced the
struct dev_power_domain type for representing device power domains,
evidently confuses some developers who tend to think that objects
of this type must correspond to "power domains" as defined by
hardware, which is not the case.  Namely, at the kernel level, a
struct dev_power_domain object can represent arbitrary set of devices
that are mutually dependent power management-wise and need not belong
to one hardware power domain.  To avoid that confusion, rename struct
dev_power_domain to struct dev_pm_domain and rename the related
pointers in struct device and struct pm_clk_notifier_block from
pwr_domain to pm_domain.

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: Kevin Hilman <khilman@ti.com>

diff --git a/Documentation/power/devices.txt b/Documentation/power/devices.txt
index 64565aac6e4009b1bca8f963cf03f900e319d538..85c6f980b6428a00ad07322cdf89514a2f394fd5 100644 (file)
--- a/Documentation/power/devices.txt
+++ b/Documentation/power/devices.txt
@@ -506,8 +506,8 @@ routines.  Nevertheless, different callback pointers are used in case there is a
 situation where it actually matters.
 
 
-Device Power Domains
---------------------
+Device Power Management Domains
+-------------------------------
 Sometimes devices share reference clocks or other power resources.  In those
 cases it generally is not possible to put devices into low-power states
 individually.  Instead, a set of devices sharing a power resource can be put
@@ -516,8 +516,8 @@ power resource.  Of course, they also need to be put into the full-power state
 together, by turning the shared power resource on.  A set of devices with this
 property is often referred to as a power domain.
 
-Support for power domains is provided through the pwr_domain field of struct
-device.  This field is a pointer to an object of type struct dev_power_domain,
+Support for power domains is provided through the pm_domain field of struct
+device.  This field is a pointer to an object of type struct dev_pm_domain,
 defined in include/linux/pm.h, providing a set of power management callbacks
 analogous to the subsystem-level and device driver callbacks that are executed
 for the given device during all power transitions, instead of the respective

diff --git a/arch/arm/mach-omap1/pm_bus.c b/arch/arm/mach-omap1/pm_bus.c
index 334fb8871bc319348f9edb152e5d611fb539aa6f..212f331037122ab65f0459ae2cba0ed54e189d49 100644 (file)
--- a/arch/arm/mach-omap1/pm_bus.c
+++ b/arch/arm/mach-omap1/pm_bus.c
@@ -49,20 +49,20 @@ static int omap1_pm_runtime_resume(struct device *dev)
        return pm_generic_runtime_resume(dev);
 }
 
-static struct dev_power_domain default_power_domain = {
+static struct dev_pm_domain default_pm_domain = {
        .ops = {
                .runtime_suspend = omap1_pm_runtime_suspend,
                .runtime_resume = omap1_pm_runtime_resume,
                USE_PLATFORM_PM_SLEEP_OPS
        },
 };
-#define OMAP1_PWR_DOMAIN (&default_power_domain)
+#define OMAP1_PM_DOMAIN (&default_pm_domain)
 #else
-#define OMAP1_PWR_DOMAIN NULL
+#define OMAP1_PM_DOMAIN NULL
 #endif /* CONFIG_PM_RUNTIME */
 
 static struct pm_clk_notifier_block platform_bus_notifier = {
-       .pwr_domain = OMAP1_PWR_DOMAIN,
+       .pm_domain = OMAP1_PM_DOMAIN,
        .con_ids = { "ick", "fck", NULL, },
 };
 

diff --git a/arch/arm/mach-shmobile/pm_runtime.c b/arch/arm/mach-shmobile/pm_runtime.c
index 2d1b67a59e4ae4318a888736680ef9dc31fcfdd6..99802d28e5d3177b07730a36b1abc57f79d23bdf 100644 (file)
--- a/arch/arm/mach-shmobile/pm_runtime.c
+++ b/arch/arm/mach-shmobile/pm_runtime.c
@@ -28,7 +28,7 @@ static int default_platform_runtime_idle(struct device *dev)
        return pm_runtime_suspend(dev);
 }
 
-static struct dev_power_domain default_power_domain = {
+static struct dev_pm_domain default_pm_domain = {
        .ops = {
                .runtime_suspend = pm_runtime_clk_suspend,
                .runtime_resume = pm_runtime_clk_resume,
@@ -37,16 +37,16 @@ static struct dev_power_domain default_power_domain = {
        },
 };
 
-#define DEFAULT_PWR_DOMAIN_PTR (&default_power_domain)
+#define DEFAULT_PM_DOMAIN_PTR  (&default_pm_domain)
 
 #else
 
-#define DEFAULT_PWR_DOMAIN_PTR NULL
+#define DEFAULT_PM_DOMAIN_PTR  NULL
 
 #endif /* CONFIG_PM_RUNTIME */
 
 static struct pm_clk_notifier_block platform_bus_notifier = {
-       .pwr_domain = DEFAULT_PWR_DOMAIN_PTR,
+       .pm_domain = DEFAULT_PM_DOMAIN_PTR,
        .con_ids = { NULL, },
 };
 

diff --git a/arch/arm/plat-omap/omap_device.c b/arch/arm/plat-omap/omap_device.c
index 49fc0df0c21f58be364d6da00e14d73eb8c32344..d21579b2c11eea80e82cc49f418552991eb7daf6 100644 (file)
--- a/arch/arm/plat-omap/omap_device.c
+++ b/arch/arm/plat-omap/omap_device.c
@@ -564,7 +564,7 @@ static int _od_runtime_resume(struct device *dev)
        return pm_generic_runtime_resume(dev);
 }
 
-static struct dev_power_domain omap_device_power_domain = {
+static struct dev_pm_domain omap_device_pm_domain = {
        .ops = {
                .runtime_suspend = _od_runtime_suspend,
                .runtime_idle = _od_runtime_idle,
@@ -586,7 +586,7 @@ int omap_device_register(struct omap_device *od)
        pr_debug("omap_device: %s: registering\n", od->pdev.name);
 
        od->pdev.dev.parent = &omap_device_parent;
-       od->pdev.dev.pwr_domain = &omap_device_power_domain;
+       od->pdev.dev.pm_domain = &omap_device_pm_domain;
        return platform_device_register(&od->pdev);
 }
 

diff --git a/arch/sh/kernel/cpu/shmobile/pm_runtime.c b/arch/sh/kernel/cpu/shmobile/pm_runtime.c
index 64c807c392087c600fde506ffc36563e88532518..bf280c812d2f238cbe8044e28a930377419ba66d 100644 (file)
--- a/arch/sh/kernel/cpu/shmobile/pm_runtime.c
+++ b/arch/sh/kernel/cpu/shmobile/pm_runtime.c
@@ -256,7 +256,7 @@ out:
        return ret;
 }
 
-static struct dev_power_domain default_power_domain = {
+static struct dev_pm_domain default_pm_domain = {
        .ops = {
                .runtime_suspend = default_platform_runtime_suspend,
                .runtime_resume = default_platform_runtime_resume,
@@ -285,7 +285,7 @@ static int platform_bus_notify(struct notifier_block *nb,
                hwblk_disable(hwblk_info, hwblk);
                /* make sure driver re-inits itself once */
                __set_bit(PDEV_ARCHDATA_FLAG_INIT, &pdev->archdata.flags);
-               dev->pwr_domain = &default_power_domain;
+               dev->pm_domain = &default_pm_domain;
                break;
        /* TODO: add BUS_NOTIFY_BIND_DRIVER and increase idle count */
        case BUS_NOTIFY_BOUND_DRIVER:
@@ -299,7 +299,7 @@ static int platform_bus_notify(struct notifier_block *nb,
                __set_bit(PDEV_ARCHDATA_FLAG_INIT, &pdev->archdata.flags);
                break;
        case BUS_NOTIFY_DEL_DEVICE:
-               dev->pwr_domain = NULL;
+               dev->pm_domain = NULL;
                break;
        }
        return 0;

diff --git a/drivers/base/power/clock_ops.c b/drivers/base/power/clock_ops.c
index ad367c4139b14ead6102ed2e31d03f52c63d1fcf..c5624818259e1715f3944343dd5809d8c54c8642 100644 (file)
--- a/drivers/base/power/clock_ops.c
+++ b/drivers/base/power/clock_ops.c
@@ -278,11 +278,11 @@ int pm_runtime_clk_resume(struct device *dev)
  *
  * For this function to work, @nb must be a member of an object of type
  * struct pm_clk_notifier_block containing all of the requisite data.
- * Specifically, the pwr_domain member of that object is copied to the device's
- * pwr_domain field and its con_ids member is used to populate the device's list
+ * Specifically, the pm_domain member of that object is copied to the device's
+ * pm_domain field and its con_ids member is used to populate the device's list
  * of runtime PM clocks, depending on @action.
  *
- * If the device's pwr_domain field is already populated with a value different
+ * If the device's pm_domain field is already populated with a value different
  * from the one stored in the struct pm_clk_notifier_block object, the function
  * does nothing.
  */
@@ -300,14 +300,14 @@ static int pm_runtime_clk_notify(struct notifier_block *nb,
 
        switch (action) {
        case BUS_NOTIFY_ADD_DEVICE:
-               if (dev->pwr_domain)
+               if (dev->pm_domain)
                        break;
 
                error = pm_runtime_clk_init(dev);
                if (error)
                        break;
 
-               dev->pwr_domain = clknb->pwr_domain;
+               dev->pm_domain = clknb->pm_domain;
                if (clknb->con_ids[0]) {
                        for (con_id = clknb->con_ids; *con_id; con_id++)
                                pm_runtime_clk_add(dev, *con_id);
@@ -317,10 +317,10 @@ static int pm_runtime_clk_notify(struct notifier_block *nb,
 
                break;
        case BUS_NOTIFY_DEL_DEVICE:
-               if (dev->pwr_domain != clknb->pwr_domain)
+               if (dev->pm_domain != clknb->pm_domain)
                        break;
 
-               dev->pwr_domain = NULL;
+               dev->pm_domain = NULL;
                pm_runtime_clk_destroy(dev);
                break;
        }

diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
index 06f09bf89cb2bab3319d800ca2dd7939dfb59a3b..85b591a5429ac8195b3ba9d6eae97e938d6b726c 100644 (file)
--- a/drivers/base/power/main.c
+++ b/drivers/base/power/main.c
@@ -425,9 +425,9 @@ static int device_resume_noirq(struct device *dev, pm_message_t state)
        TRACE_DEVICE(dev);
        TRACE_RESUME(0);
 
-       if (dev->pwr_domain) {
+       if (dev->pm_domain) {
                pm_dev_dbg(dev, state, "EARLY power domain ");
-               error = pm_noirq_op(dev, &dev->pwr_domain->ops, state);
+               error = pm_noirq_op(dev, &dev->pm_domain->ops, state);
        } else if (dev->type && dev->type->pm) {
                pm_dev_dbg(dev, state, "EARLY type ");
                error = pm_noirq_op(dev, dev->type->pm, state);
@@ -521,9 +521,9 @@ static int device_resume(struct device *dev, pm_message_t state, bool async)
        if (!dev->power.is_suspended)
                goto Unlock;
 
-       if (dev->pwr_domain) {
+       if (dev->pm_domain) {
                pm_dev_dbg(dev, state, "power domain ");
-               error = pm_op(dev, &dev->pwr_domain->ops, state);
+               error = pm_op(dev, &dev->pm_domain->ops, state);
                goto End;
        }
 
@@ -641,10 +641,10 @@ static void device_complete(struct device *dev, pm_message_t state)
 {
        device_lock(dev);
 
-       if (dev->pwr_domain) {
+       if (dev->pm_domain) {
                pm_dev_dbg(dev, state, "completing power domain ");
-               if (dev->pwr_domain->ops.complete)
-                       dev->pwr_domain->ops.complete(dev);
+               if (dev->pm_domain->ops.complete)
+                       dev->pm_domain->ops.complete(dev);
        } else if (dev->type && dev->type->pm) {
                pm_dev_dbg(dev, state, "completing type ");
                if (dev->type->pm->complete)
@@ -744,9 +744,9 @@ static int device_suspend_noirq(struct device *dev, pm_message_t state)
 {
        int error;
 
-       if (dev->pwr_domain) {
+       if (dev->pm_domain) {
                pm_dev_dbg(dev, state, "LATE power domain ");
-               error = pm_noirq_op(dev, &dev->pwr_domain->ops, state);
+               error = pm_noirq_op(dev, &dev->pm_domain->ops, state);
                if (error)
                        return error;
        } else if (dev->type && dev->type->pm) {
@@ -853,9 +853,9 @@ static int __device_suspend(struct device *dev, pm_message_t state, bool async)
                goto Unlock;
        }
 
-       if (dev->pwr_domain) {
+       if (dev->pm_domain) {
                pm_dev_dbg(dev, state, "power domain ");
-               error = pm_op(dev, &dev->pwr_domain->ops, state);
+               error = pm_op(dev, &dev->pm_domain->ops, state);
                goto End;
        }
 
@@ -982,11 +982,11 @@ static int device_prepare(struct device *dev, pm_message_t state)
 
        device_lock(dev);
 
-       if (dev->pwr_domain) {
+       if (dev->pm_domain) {
                pm_dev_dbg(dev, state, "preparing power domain ");
-               if (dev->pwr_domain->ops.prepare)
-                       error = dev->pwr_domain->ops.prepare(dev);
-               suspend_report_result(dev->pwr_domain->ops.prepare, error);
+               if (dev->pm_domain->ops.prepare)
+                       error = dev->pm_domain->ops.prepare(dev);
+               suspend_report_result(dev->pm_domain->ops.prepare, error);
                if (error)
                        goto End;
        } else if (dev->type && dev->type->pm) {

diff --git a/drivers/base/power/runtime.c b/drivers/base/power/runtime.c
index 0d4587b15c5599372e7b9f917fcea636404bf8d9..5f5c4236f006e1608ef3fda4dd66fe4fbc99f40f 100644 (file)
--- a/drivers/base/power/runtime.c
+++ b/drivers/base/power/runtime.c
@@ -213,8 +213,8 @@ static int rpm_idle(struct device *dev, int rpmflags)
 
        dev->power.idle_notification = true;
 
-       if (dev->pwr_domain)
-               callback = dev->pwr_domain->ops.runtime_idle;
+       if (dev->pm_domain)
+               callback = dev->pm_domain->ops.runtime_idle;
        else if (dev->type && dev->type->pm)
                callback = dev->type->pm->runtime_idle;
        else if (dev->class && dev->class->pm)
@@ -374,8 +374,8 @@ static int rpm_suspend(struct device *dev, int rpmflags)
 
        __update_runtime_status(dev, RPM_SUSPENDING);
 
-       if (dev->pwr_domain)
-               callback = dev->pwr_domain->ops.runtime_suspend;
+       if (dev->pm_domain)
+               callback = dev->pm_domain->ops.runtime_suspend;
        else if (dev->type && dev->type->pm)
                callback = dev->type->pm->runtime_suspend;
        else if (dev->class && dev->class->pm)
@@ -573,8 +573,8 @@ static int rpm_resume(struct device *dev, int rpmflags)
 
        __update_runtime_status(dev, RPM_RESUMING);
 
-       if (dev->pwr_domain)
-               callback = dev->pwr_domain->ops.runtime_resume;
+       if (dev->pm_domain)
+               callback = dev->pm_domain->ops.runtime_resume;
        else if (dev->type && dev->type->pm)
                callback = dev->type->pm->runtime_resume;
        else if (dev->class && dev->class->pm)

diff --git a/include/linux/device.h b/include/linux/device.h
index e4f62d8896b7c9adf3cfb745cbf4c79ec5603d4d..160d4ddb2499c8736cb05e0952703775b94fbb0a 100644 (file)
--- a/include/linux/device.h
+++ b/include/linux/device.h
@@ -516,7 +516,7 @@ struct device_dma_parameters {
  *             minimizes board-specific #ifdefs in drivers.
  * @power:     For device power management.
  *             See Documentation/power/devices.txt for details.
- * @pwr_domain:        Provide callbacks that are executed during system suspend,
+ * @pm_domain: Provide callbacks that are executed during system suspend,
  *             hibernation, system resume and during runtime PM transitions
  *             along with subsystem-level and driver-level callbacks.
  * @numa_node: NUMA node this device is close to.
@@ -567,7 +567,7 @@ struct device {
        void            *platform_data; /* Platform specific data, device
                                           core doesn't touch it */
        struct dev_pm_info      power;
-       struct dev_power_domain *pwr_domain;
+       struct dev_pm_domain    *pm_domain;
 
 #ifdef CONFIG_NUMA
        int             numa_node;      /* NUMA node this device is close to */

diff --git a/include/linux/pm.h b/include/linux/pm.h
index 411e4f4be52b984fa22f9a36502a8c2b79b18b4a..e3963208aa93f2a96d14346eefbafc5f21c13c9c 100644 (file)
--- a/include/linux/pm.h
+++ b/include/linux/pm.h
@@ -472,7 +472,7 @@ extern void update_pm_runtime_accounting(struct device *dev);
  * hibernation, system resume and during runtime PM transitions along with
  * subsystem-level and driver-level callbacks.
  */
-struct dev_power_domain {
+struct dev_pm_domain {
        struct dev_pm_ops       ops;
 };
 

diff --git a/include/linux/pm_runtime.h b/include/linux/pm_runtime.h
index 878cf84baeb1c3f89087e5106e292d32536f68d4..ef91904c711050aa8bc1381b5c64994e5a423c5e 100644 (file)
--- a/include/linux/pm_runtime.h
+++ b/include/linux/pm_runtime.h
@@ -247,7 +247,7 @@ static inline void pm_runtime_dont_use_autosuspend(struct device *dev)
 
 struct pm_clk_notifier_block {
        struct notifier_block nb;
-       struct dev_power_domain *pwr_domain;
+       struct dev_pm_domain *pm_domain;
        char *con_ids[];
 };