wifi: ath: Convert sprintf/snprintf to sysfs_emit

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

coccinelle complains that there are still a couple of functions that use
snprintf(). Convert them to sysfs_emit().

sprintf() will be converted as weel if they have.

Generally, this patch is generated by
make coccicheck M=<path/to/file> MODE=patch \
COCCI=scripts/coccinelle/api/device_attr_show.cocci

No functional change intended

CC: Kalle Valo <kvalo@kernel.org>
CC: Jeff Johnson <jjohnson@kernel.org>
CC: linux-wireless@vger.kernel.org
CC: ath11k@lists.infradead.org
CC: ath10k@lists.infradead.org
Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com>
Link: https://msgid.link/20240315055211.1347548-1-lizhijian@fujitsu.com

diff --git a/drivers/net/wireless/ath/ath10k/thermal.c b/drivers/net/wireless/ath/ath10k/thermal.c
index 31c8d7fbb0955b3b85fb53ba119c67d3d6994a06..8b15ec07b1071573cda9a4a2567e411ffd15a7ca 100644 (file)
--- a/drivers/net/wireless/ath/ath10k/thermal.c
+++ b/drivers/net/wireless/ath/ath10k/thermal.c
@@ -100,7 +100,7 @@ static ssize_t ath10k_thermal_show_temp(struct device *dev,
        spin_unlock_bh(&ar->data_lock);
 
        /* display in millidegree celsius */
-       ret = snprintf(buf, PAGE_SIZE, "%d\n", temperature * 1000);
+       ret = sysfs_emit(buf, "%d\n", temperature * 1000);
 out:
        mutex_unlock(&ar->conf_mutex);
        return ret;

diff --git a/drivers/net/wireless/ath/ath11k/thermal.c b/drivers/net/wireless/ath/ath11k/thermal.c
index 41e7499f075f52f341cb8c0486c2a8586457bbe3..18d6eab5cce3c7382f80964365b523377e38f3f3 100644 (file)
--- a/drivers/net/wireless/ath/ath11k/thermal.c
+++ b/drivers/net/wireless/ath/ath11k/thermal.c
@@ -101,7 +101,7 @@ static ssize_t ath11k_thermal_show_temp(struct device *dev,
        spin_unlock_bh(&ar->data_lock);
 
        /* display in millidegree Celsius */
-       ret = snprintf(buf, PAGE_SIZE, "%d\n", temperature * 1000);
+       ret = sysfs_emit(buf, "%d\n", temperature * 1000);
 out:
        mutex_unlock(&ar->conf_mutex);
        return ret;