]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
Input: stmfts - use sysfs_emit() instead of sprintf()
authorPetr Hodina <petr.hodina@protonmail.com>
Mon, 15 Dec 2025 14:30:03 +0000 (15:30 +0100)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Tue, 16 Dec 2025 05:57:02 +0000 (21:57 -0800)
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: Petr Hodina <petr.hodina@protonmail.com>
Signed-off-by: David Heidelberg <david@ixit.cz>
Link: https://patch.msgid.link/20251215-fts-fixes-v1-3-8c1e3a63ebf1@ixit.cz
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/stmfts.c

index ad363b3c344cd28e0b8da8e59007583a581c7287..4b166b0a9a5a6c440029fe3e60e01e100fbabc08 100644 (file)
@@ -410,7 +410,7 @@ static ssize_t stmfts_sysfs_chip_id(struct device *dev,
 {
        struct stmfts_data *sdata = dev_get_drvdata(dev);
 
-       return sprintf(buf, "%#x\n", sdata->chip_id);
+       return sysfs_emit(buf, "%#x\n", sdata->chip_id);
 }
 
 static ssize_t stmfts_sysfs_chip_version(struct device *dev,
@@ -418,7 +418,7 @@ static ssize_t stmfts_sysfs_chip_version(struct device *dev,
 {
        struct stmfts_data *sdata = dev_get_drvdata(dev);
 
-       return sprintf(buf, "%u\n", sdata->chip_ver);
+       return sysfs_emit(buf, "%u\n", sdata->chip_ver);
 }
 
 static ssize_t stmfts_sysfs_fw_ver(struct device *dev,
@@ -426,7 +426,7 @@ static ssize_t stmfts_sysfs_fw_ver(struct device *dev,
 {
        struct stmfts_data *sdata = dev_get_drvdata(dev);
 
-       return sprintf(buf, "%u\n", sdata->fw_ver);
+       return sysfs_emit(buf, "%u\n", sdata->fw_ver);
 }
 
 static ssize_t stmfts_sysfs_config_id(struct device *dev,
@@ -434,7 +434,7 @@ static ssize_t stmfts_sysfs_config_id(struct device *dev,
 {
        struct stmfts_data *sdata = dev_get_drvdata(dev);
 
-       return sprintf(buf, "%#x\n", sdata->config_id);
+       return sysfs_emit(buf, "%#x\n", sdata->config_id);
 }
 
 static ssize_t stmfts_sysfs_config_version(struct device *dev,
@@ -442,7 +442,7 @@ static ssize_t stmfts_sysfs_config_version(struct device *dev,
 {
        struct stmfts_data *sdata = dev_get_drvdata(dev);
 
-       return sprintf(buf, "%u\n", sdata->config_ver);
+       return sysfs_emit(buf, "%u\n", sdata->config_ver);
 }
 
 static ssize_t stmfts_sysfs_read_status(struct device *dev,
@@ -457,7 +457,7 @@ static ssize_t stmfts_sysfs_read_status(struct device *dev,
        if (err)
                return err;
 
-       return sprintf(buf, "%#02x\n", status[0]);
+       return sysfs_emit(buf, "%#02x\n", status[0]);
 }
 
 static ssize_t stmfts_sysfs_hover_enable_read(struct device *dev,
@@ -465,7 +465,7 @@ static ssize_t stmfts_sysfs_hover_enable_read(struct device *dev,
 {
        struct stmfts_data *sdata = dev_get_drvdata(dev);
 
-       return sprintf(buf, "%u\n", sdata->hover_enabled);
+       return sysfs_emit(buf, "%u\n", sdata->hover_enabled);
 }
 
 static ssize_t stmfts_sysfs_hover_enable_write(struct device *dev,