Commit 65f44679 authored by Greg KH's avatar Greg KH Committed by Greg Kroah-Hartman

ACPI: bgrt: take advantage of binary sysfs groups

Attribute groups now can handle binary sysfs attributes, so clean up the
code here by using a binary attribute array.  This saves us the extra
call to create the binary attribute at saves 6 lines overall.

Cc: Len Brown <lenb@kernel.org>
Acked-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>

--
 I can take this in my driver-core tree if someone from ACPI acks it,
 otherwise, feel free to take it through the ACPI trees instead, just
 let me know.

 drivers/acpi/bgrt.c |   26 ++++++++++----------------
 1 file changed, 10 insertions(+), 16 deletions(-)
parent a7492451
...@@ -51,20 +51,14 @@ static ssize_t show_yoffset(struct device *dev, ...@@ -51,20 +51,14 @@ static ssize_t show_yoffset(struct device *dev,
} }
static DEVICE_ATTR(yoffset, S_IRUGO, show_yoffset, NULL); static DEVICE_ATTR(yoffset, S_IRUGO, show_yoffset, NULL);
static ssize_t show_image(struct file *file, struct kobject *kobj, static ssize_t image_read(struct file *file, struct kobject *kobj,
struct bin_attribute *attr, char *buf, loff_t off, size_t count) struct bin_attribute *attr, char *buf, loff_t off, size_t count)
{ {
memcpy(buf, attr->private + off, count); memcpy(buf, attr->private + off, count);
return count; return count;
} }
static struct bin_attribute image_attr = { static BIN_ATTR_RO(image, 0); /* size gets filled in later */
.attr = {
.name = "image",
.mode = S_IRUGO,
},
.read = show_image,
};
static struct attribute *bgrt_attributes[] = { static struct attribute *bgrt_attributes[] = {
&dev_attr_version.attr, &dev_attr_version.attr,
...@@ -75,8 +69,14 @@ static struct attribute *bgrt_attributes[] = { ...@@ -75,8 +69,14 @@ static struct attribute *bgrt_attributes[] = {
NULL, NULL,
}; };
static struct bin_attribute *bgrt_bin_attributes[] = {
&bin_attr_image,
NULL,
};
static struct attribute_group bgrt_attribute_group = { static struct attribute_group bgrt_attribute_group = {
.attrs = bgrt_attributes, .attrs = bgrt_attributes,
.bin_attrs = bgrt_bin_attributes,
}; };
static int __init bgrt_init(void) static int __init bgrt_init(void)
...@@ -87,8 +87,8 @@ static int __init bgrt_init(void) ...@@ -87,8 +87,8 @@ static int __init bgrt_init(void)
return -ENODEV; return -ENODEV;
sysfs_bin_attr_init(&image_attr); sysfs_bin_attr_init(&image_attr);
image_attr.private = bgrt_image; bin_attr_image.private = bgrt_image;
image_attr.size = bgrt_image_size; bin_attr_image.size = bgrt_image_size;
bgrt_kobj = kobject_create_and_add("bgrt", acpi_kobj); bgrt_kobj = kobject_create_and_add("bgrt", acpi_kobj);
if (!bgrt_kobj) if (!bgrt_kobj)
...@@ -98,14 +98,8 @@ static int __init bgrt_init(void) ...@@ -98,14 +98,8 @@ static int __init bgrt_init(void)
if (ret) if (ret)
goto out_kobject; goto out_kobject;
ret = sysfs_create_bin_file(bgrt_kobj, &image_attr);
if (ret)
goto out_group;
return 0; return 0;
out_group:
sysfs_remove_group(bgrt_kobj, &bgrt_attribute_group);
out_kobject: out_kobject:
kobject_put(bgrt_kobj); kobject_put(bgrt_kobj);
return ret; return ret;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment