]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
HID: core: use utility macros to define sysfs attributes
authorThomas Weißschuh <linux@weissschuh.net>
Mon, 2 Dec 2024 19:01:32 +0000 (20:01 +0100)
committerJiri Kosina <jkosina@suse.com>
Wed, 11 Dec 2024 14:11:04 +0000 (15:11 +0100)
Save some lines of code and prepare for the constification of
'struct bin_attribute'.
Also align the definitions within the source file.

Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
Signed-off-by: Jiri Kosina <jkosina@suse.com>
drivers/hid/hid-core.c

index 33a19197332488bd232dcee510d9da9ffec770bc..e35cf426c16f755bf2f1a5af8079553567dbf4f8 100644 (file)
@@ -2174,9 +2174,9 @@ static bool hid_hiddev(struct hid_device *hdev)
 
 
 static ssize_t
-read_report_descriptor(struct file *filp, struct kobject *kobj,
-               struct bin_attribute *attr,
-               char *buf, loff_t off, size_t count)
+report_descriptor_read(struct file *filp, struct kobject *kobj,
+                      struct bin_attribute *attr,
+                      char *buf, loff_t off, size_t count)
 {
        struct device *dev = kobj_to_dev(kobj);
        struct hid_device *hdev = to_hid_device(dev);
@@ -2193,24 +2193,17 @@ read_report_descriptor(struct file *filp, struct kobject *kobj,
 }
 
 static ssize_t
-show_country(struct device *dev, struct device_attribute *attr,
-               char *buf)
+country_show(struct device *dev, struct device_attribute *attr,
+            char *buf)
 {
        struct hid_device *hdev = to_hid_device(dev);
 
        return sprintf(buf, "%02x\n", hdev->country & 0xff);
 }
 
-static struct bin_attribute dev_bin_attr_report_desc = {
-       .attr = { .name = "report_descriptor", .mode = 0444 },
-       .read = read_report_descriptor,
-       .size = HID_MAX_DESCRIPTOR_SIZE,
-};
+static BIN_ATTR_RO(report_descriptor, HID_MAX_DESCRIPTOR_SIZE);
 
-static const struct device_attribute dev_attr_country = {
-       .attr = { .name = "country", .mode = 0444 },
-       .show = show_country,
-};
+static const DEVICE_ATTR_RO(country);
 
 int hid_connect(struct hid_device *hdev, unsigned int connect_mask)
 {
@@ -2801,7 +2794,7 @@ static struct attribute *hid_dev_attrs[] = {
        NULL,
 };
 static struct bin_attribute *hid_dev_bin_attrs[] = {
-       &dev_bin_attr_report_desc,
+       &bin_attr_report_descriptor,
        NULL
 };
 static const struct attribute_group hid_dev_group = {