diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2025-11-03 13:08:09 +0100 |
|---|---|---|
| committer | Lee Jones <lee@kernel.org> | 2025-11-13 15:22:24 +0000 |
| commit | 92f4b016dcb955cc0bb3942ac217cff6fe450a57 (patch) | |
| tree | 45ea0e4964d8b5ea5204292e1a1dafe0a09bb75a | |
| parent | c42b91c426a12523b8d02bc6041f7931f199a301 (diff) | |
leds: led-class: Replace sprintf() with sysfs_emit() in sysfs show functions
Replace sprintf() with sysfs_emit() in sysfs show functions.
sysfs_emit() is preferred to format sysfs output as it provides better
bounds checking. No functional changes.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20251103120809.32834-1-thorsten.blum@linux.dev
Signed-off-by: Lee Jones <lee@kernel.org>
| -rw-r--r-- | drivers/leds/led-class.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/leds/led-class.c b/drivers/leds/led-class.c index f3faf37f9a08..885399ed0776 100644 --- a/drivers/leds/led-class.c +++ b/drivers/leds/led-class.c @@ -38,7 +38,7 @@ static ssize_t brightness_show(struct device *dev, brightness = led_cdev->brightness; mutex_unlock(&led_cdev->led_access); - return sprintf(buf, "%u\n", brightness); + return sysfs_emit(buf, "%u\n", brightness); } static ssize_t brightness_store(struct device *dev, @@ -80,7 +80,7 @@ static ssize_t max_brightness_show(struct device *dev, max_brightness = led_cdev->max_brightness; mutex_unlock(&led_cdev->led_access); - return sprintf(buf, "%u\n", max_brightness); + return sysfs_emit(buf, "%u\n", max_brightness); } static DEVICE_ATTR_RO(max_brightness); @@ -122,7 +122,7 @@ static ssize_t brightness_hw_changed_show(struct device *dev, if (led_cdev->brightness_hw_changed == -1) return -ENODATA; - return sprintf(buf, "%u\n", led_cdev->brightness_hw_changed); + return sysfs_emit(buf, "%u\n", led_cdev->brightness_hw_changed); } static DEVICE_ATTR_RO(brightness_hw_changed); |