Commit 95cd8806 authored by yunshui's avatar yunshui Committed by Hans de Goede
Browse files

platform/x86: thinkpad_acpi: change sprintf() to sysfs_emit() 



As Documentation/filesystems/sysfs.rst suggested,
show() should only use sysfs_emit() or sysfs_emit_at() when formatting
the value to be returned to user space.

Signed-off-by: default avataryunshui <jiangyunshui@kylinos.cn>
Reviewed-by: default avatarAi Chao <aichao@kylinos.cn>
Link: https://lore.kernel.org/r/20240417092055.1170586-1-jiangyunshui@kylinos.cn


Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent f1f663eb

drivers/platform/x86/thinkpad_acpi.c

+2 −2
Original line number Diff line number Diff line
@@ -2679,7 +2679,7 @@ static ssize_t hotkey_bios_enabled_show(struct device *dev, 
			   struct device_attribute *attr,

			   char *buf)

{

	return sprintf(buf, "0\n");

	return sysfs_emit(buf, "0\n");

}



static DEVICE_ATTR_RO(hotkey_bios_enabled);
@@ -9789,7 +9789,7 @@ static ssize_t tpacpi_battery_show(int what, 
		battery = BAT_PRIMARY;

	if (tpacpi_battery_get(what, battery, &ret))

		return -ENODEV;

	return sprintf(buf, "%d\n", ret);

	return sysfs_emit(buf, "%d\n", ret);

}



static ssize_t charge_control_start_threshold_show(struct device *device,