]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
staging: iio: ad7816: Replace sprintf() with sysfs_emit()
authorBruno Martins <ehanoc@protonmail.com>
Sun, 1 Mar 2026 21:28:26 +0000 (21:28 +0000)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Mon, 2 Mar 2026 21:01:23 +0000 (21:01 +0000)
As stated in Documentation/filesystems/sysfs.rst:
'New implementations of show() methods should only use sysfs_emit() or
sysfs_emit_at() when formatting the value to be returned to user space.'

Replace sprintf with sysfs_emit in the following sysfs show functions:
- ad7816_show_mode()
- ad7816_show_available_modes()
- ad7816_show_channel()
- ad7816_show_value()
- ad7816_show_oti()

Signed-off-by: Bruno Martins <ehanoc@protonmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/staging/iio/adc/ad7816.c

index 172acf135f3bd001c7da6d57fb68a496c869a120..0e32a2295990dfa023b94a9f685b6f24e64a8824 100644 (file)
@@ -124,8 +124,8 @@ static ssize_t ad7816_show_mode(struct device *dev,
        struct ad7816_chip_info *chip = iio_priv(indio_dev);
 
        if (chip->mode)
-               return sprintf(buf, "power-save\n");
-       return sprintf(buf, "full\n");
+               return sysfs_emit(buf, "power-save\n");
+       return sysfs_emit(buf, "full\n");
 }
 
 static ssize_t ad7816_store_mode(struct device *dev,
@@ -156,7 +156,7 @@ static ssize_t ad7816_show_available_modes(struct device *dev,
                                           struct device_attribute *attr,
                                           char *buf)
 {
-       return sprintf(buf, "full\npower-save\n");
+       return sysfs_emit(buf, "full\npower-save\n");
 }
 
 static IIO_DEVICE_ATTR(available_modes, 0444, ad7816_show_available_modes,
@@ -169,7 +169,7 @@ static ssize_t ad7816_show_channel(struct device *dev,
        struct iio_dev *indio_dev = dev_to_iio_dev(dev);
        struct ad7816_chip_info *chip = iio_priv(indio_dev);
 
-       return sprintf(buf, "%d\n", chip->channel_id);
+       return sysfs_emit(buf, "%d\n", chip->channel_id);
 }
 
 static ssize_t ad7816_store_channel(struct device *dev,
@@ -231,9 +231,9 @@ static ssize_t ad7816_show_value(struct device *dev,
                data &= AD7816_TEMP_FLOAT_MASK;
                if (value < 0)
                        data = BIT(AD7816_TEMP_FLOAT_OFFSET) - data;
-               return sprintf(buf, "%d.%.2d\n", value, data * 25);
+               return sysfs_emit(buf, "%d.%.2d\n", value, data * 25);
        }
-       return sprintf(buf, "%u\n", data);
+       return sysfs_emit(buf, "%u\n", data);
 }
 
 static IIO_DEVICE_ATTR(value, 0444, ad7816_show_value, NULL, 0);
@@ -281,9 +281,9 @@ static ssize_t ad7816_show_oti(struct device *dev,
                value = AD7816_BOUND_VALUE_MIN +
                        (chip->oti_data[chip->channel_id] -
                        AD7816_BOUND_VALUE_BASE);
-               return sprintf(buf, "%d\n", value);
+               return sysfs_emit(buf, "%d\n", value);
        }
-       return sprintf(buf, "%u\n", chip->oti_data[chip->channel_id]);
+       return sysfs_emit(buf, "%u\n", chip->oti_data[chip->channel_id]);
 }
 
 static inline ssize_t ad7816_set_oti(struct device *dev,