HID: hid-logitech: use sysfs_emit_at() instead of scnprintf()
authorTangDongxing <tang.dongxing@zte.com.cn>
Mon, 17 Mar 2025 06:49:03 +0000 (14:49 +0800)
committerJiri Kosina <jkosina@suse.com>
Thu, 24 Apr 2025 09:33:48 +0000 (11:33 +0200)
Follow the advice in Documentation/filesystems/sysfs.rst:
show() should only use sysfs_emit() or sysfs_emit_at() when formatting
the value to be returned to user space.

Signed-off-by: TangDongxing <tang.dongxing@zte.com.cn>
Signed-off-by: Jiri Kosina <jkosina@suse.com>
drivers/hid/hid-lg4ff.c

index c0a138f21ca49996d8ad4d76fc548f5cf31f606e..445623dd1bd6c19a0b0f1acced3b13307fc4c6c8 100644 (file)
@@ -823,7 +823,7 @@ static ssize_t lg4ff_alternate_modes_show(struct device *dev, struct device_attr
        for (i = 0; i < LG4FF_MODE_MAX_IDX; i++) {
                if (entry->wdata.alternate_modes & BIT(i)) {
                        /* Print tag and full name */
-                       count += scnprintf(buf + count, PAGE_SIZE - count, "%s: %s",
+                       count += sysfs_emit_at(buf, count, "%s: %s",
                                           lg4ff_alternate_modes[i].tag,
                                           !lg4ff_alternate_modes[i].product_id ? entry->wdata.real_name : lg4ff_alternate_modes[i].name);
                        if (count >= PAGE_SIZE - 1)
@@ -832,9 +832,9 @@ static ssize_t lg4ff_alternate_modes_show(struct device *dev, struct device_attr
                        /* Mark the currently active mode with an asterisk */
                        if (lg4ff_alternate_modes[i].product_id == entry->wdata.product_id ||
                            (lg4ff_alternate_modes[i].product_id == 0 && entry->wdata.product_id == entry->wdata.real_product_id))
-                               count += scnprintf(buf + count, PAGE_SIZE - count, " *\n");
+                               count += sysfs_emit_at(buf, count, " *\n");
                        else
-                               count += scnprintf(buf + count, PAGE_SIZE - count, "\n");
+                               count += sysfs_emit_at(buf, count, "\n");
 
                        if (count >= PAGE_SIZE - 1)
                                return count;