diff options
author | Christian Marangi <ansuelsmth@gmail.com> | 2024-06-27 00:15:12 +0200 |
---|---|---|
committer | Lee Jones <lee@kernel.org> | 2024-07-04 16:44:45 +0100 |
commit | 8eac0379d3bd9d048b1144d74d9309a198fd3f40 (patch) | |
tree | 7c45b2f6536885025fd7f8149c5b62bdd9db8bfd /drivers/leds | |
parent | 6f2fdde9096f3c4d35a7711c91a78c086be66aed (diff) | |
download | lwn-8eac0379d3bd9d048b1144d74d9309a198fd3f40.tar.gz lwn-8eac0379d3bd9d048b1144d74d9309a198fd3f40.zip |
leds: leds-lp5523: Convert to sysfs_emit API
Convert sprintf to the much safer sysfs_emit API to handle output for
sysfs.
Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
Link: https://lore.kernel.org/r/20240626221520.2846-2-ansuelsmth@gmail.com
Signed-off-by: Lee Jones <lee@kernel.org>
Diffstat (limited to 'drivers/leds')
-rw-r--r-- | drivers/leds/leds-lp5523.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/leds/leds-lp5523.c b/drivers/leds/leds-lp5523.c index 57df920192d2..095060533d1a 100644 --- a/drivers/leds/leds-lp5523.c +++ b/drivers/leds/leds-lp5523.c @@ -245,8 +245,8 @@ static ssize_t lp5523_selftest(struct device *dev, goto fail; if (adc >= vdd || adc < LP5523_ADC_SHORTCIRC_LIM) - pos += sprintf(buf + pos, "LED %d FAIL\n", - led->chan_nr); + pos += sysfs_emit_at(buf, pos, "LED %d FAIL\n", + led->chan_nr); lp55xx_write(chip, LP5523_REG_LED_PWM_BASE + led->chan_nr, 0x00); @@ -257,10 +257,10 @@ static ssize_t lp5523_selftest(struct device *dev, led++; } if (pos == 0) - pos = sprintf(buf, "OK\n"); + pos = sysfs_emit(buf, "OK\n"); goto release_lock; fail: - pos = sprintf(buf, "FAIL\n"); + pos = sysfs_emit(buf, "FAIL\n"); release_lock: mutex_unlock(&chip->lock); |