From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: ACPI Devel Maling List <linux-acpi@vger.kernel.org>
Cc: Linux PM list <linux-pm@vger.kernel.org>,
LKML <linux-kernel@vger.kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Len Brown <lenb@kernel.org>
Subject: [PATCH 3/3] ACPI / PM: Export power states of ACPI devices via sysfs
Date: Wed, 05 Dec 2012 01:42:35 +0100 [thread overview]
Message-ID: <1711926.9E8yF37X23@vostro.rjw.lan> (raw)
In-Reply-To: <1464990.iyrX0DeZyT@vostro.rjw.lan>
From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Make it possible to retrieve the current power state of an ACPI
device from user space via sysfs by adding a new attribute
power_state to the power subdirectory of the sysfs directory
associated with the struct acpi_device representing the device's
ACPI node.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---
Documentation/ABI/testing/sysfs-devices-power | 13 ++++++++
drivers/acpi/scan.c | 41 ++++++++++++++++++++++++++
2 files changed, 54 insertions(+)
Index: linux/drivers/acpi/scan.c
===================================================================
--- linux.orig/drivers/acpi/scan.c
+++ linux/drivers/acpi/scan.c
@@ -174,6 +174,43 @@ err_out:
}
EXPORT_SYMBOL(acpi_bus_hot_remove_device);
+#ifdef CONFIG_PM
+static ssize_t power_state_show(struct device *dev,
+ struct device_attribute *attr, char *buf)
+{
+ struct acpi_device *adev = to_acpi_device(dev);
+ int state;
+ int ret;
+
+ ret = acpi_device_get_power(adev, &state);
+ return ret ? ret : sprintf(buf, "%s\n", acpi_power_state_string(state));
+}
+
+static DEVICE_ATTR(power_state, 0444, power_state_show, NULL);
+
+static struct attribute *acpi_dev_pm_attrs[] = {
+ &dev_attr_power_state.attr,
+ NULL,
+};
+static struct attribute_group acpi_dev_pm_attr_group = {
+ .name = power_group_name,
+ .attrs = acpi_dev_pm_attrs,
+};
+
+static void acpi_dev_pm_sysfs_add(struct device *dev)
+{
+ sysfs_merge_group(&dev->kobj, &acpi_dev_pm_attr_group);
+}
+
+static void acpi_dev_pm_sysfs_remove(struct device *dev)
+{
+ sysfs_unmerge_group(&dev->kobj, &acpi_dev_pm_attr_group);
+}
+#else /* !CONFIG_PM */
+static inline void acpi_dev_pm_sysfs_add(struct device *dev) {}
+static inline void acpi_dev_pm_sysfs_remove(struct device *dev) {}
+#endif /* !CONFIG_PM */
+
static ssize_t
acpi_eject_store(struct device *d, struct device_attribute *attr,
const char *buf, size_t count)
@@ -367,6 +404,9 @@ static int acpi_device_setup_files(struc
status = acpi_get_handle(dev->handle, "_EJ0", &temp);
if (ACPI_SUCCESS(status))
result = device_create_file(&dev->dev, &dev_attr_eject);
+
+ acpi_dev_pm_sysfs_add(&dev->dev);
+
end:
return result;
}
@@ -376,6 +416,7 @@ static void acpi_device_remove_files(str
acpi_status status;
acpi_handle temp;
+ acpi_dev_pm_sysfs_remove(&dev->dev);
/*
* If device has _STR, remove 'description' file
*/
Index: linux/Documentation/ABI/testing/sysfs-devices-power
===================================================================
--- linux.orig/Documentation/ABI/testing/sysfs-devices-power
+++ linux/Documentation/ABI/testing/sysfs-devices-power
@@ -235,3 +235,16 @@ Description:
This attribute has no effect on system-wide suspend/resume and
hibernation.
+
+What: /sys/devices/.../power/power_state
+Date: December 2012
+Contact: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
+Description:
+ The /sys/devices/.../power/power_state attribute is only present
+ for ACPI device nodes (i.e. objects of type struct acpi_device).
+
+ If present, it contains the string representation of the current
+ ACPI power state of the device represented by the given ACPI
+ device node.
+
+ This attribute is read-only.
prev parent reply other threads:[~2012-12-05 0:42 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-12-05 0:39 [PATCH 0/3] ACPI / PM: Export ACPI power states of devices via sysfs Rafael J. Wysocki
2012-12-05 0:40 ` [PATCH 1/3] ACPI / PM: More visible function for retrieving device power states Rafael J. Wysocki
2012-12-05 0:41 ` [PATCH 2/3] ACPI / PM: Common string representations of " Rafael J. Wysocki
2012-12-05 0:42 ` Rafael J. Wysocki [this message]
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=1711926.9E8yF37X23@vostro.rjw.lan \
--to=rjw@sisk.pl \
--cc=gregkh@linuxfoundation.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox