All of lore.kernel.org
 help / color / mirror / Atom feed
From: Li Zhijian <lizhijian@fujitsu.com>
To: linux-kernel@vger.kernel.org
Cc: Li Zhijian <lizhijian@fujitsu.com>,
	Jiri Kosina <jikos@kernel.org>,
	Benjamin Tissoires <benjamin.tissoires@redhat.com>,
	linux-input@vger.kernel.org
Subject: [PATCH 09/42] drivers/hid/hid-lenovo: Convert snprintf to sysfs_emit
Date: Tue, 16 Jan 2024 12:51:18 +0800	[thread overview]
Message-ID: <20240116045151.3940401-7-lizhijian@fujitsu.com> (raw)
In-Reply-To: <20240116045151.3940401-1-lizhijian@fujitsu.com>

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().

> ./drivers/hid/hid-lenovo.c:558:8-16: WARNING: please use sysfs_emit
> ./drivers/hid/hid-lenovo.c:602:8-16: WARNING: please use sysfs_emit
> ./drivers/hid/hid-lenovo.c:792:8-16: WARNING: please use sysfs_emit
> ./drivers/hid/hid-lenovo.c:822:8-16: WARNING: please use sysfs_emit
> ./drivers/hid/hid-lenovo.c:852:8-16: WARNING: please use sysfs_emit
> ./drivers/hid/hid-lenovo.c:882:8-16: WARNING: please use sysfs_emit
> ./drivers/hid/hid-lenovo.c:912:8-16: WARNING: please use sysfs_emit
> ./drivers/hid/hid-lenovo.c:941:8-16: WARNING: please use sysfs_emit

No functional change intended

CC: Jiri Kosina <jikos@kernel.org>
CC: Benjamin Tissoires <benjamin.tissoires@redhat.com>
CC: linux-input@vger.kernel.org
Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
---
 drivers/hid/hid-lenovo.c | 19 ++++++++-----------
 1 file changed, 8 insertions(+), 11 deletions(-)

diff --git a/drivers/hid/hid-lenovo.c b/drivers/hid/hid-lenovo.c
index 149a3c74346b..26768c319361 100644
--- a/drivers/hid/hid-lenovo.c
+++ b/drivers/hid/hid-lenovo.c
@@ -555,7 +555,7 @@ static ssize_t attr_fn_lock_show(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *data = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n", data->fn_lock);
+	return sysfs_emit(buf, "%u\n", data->fn_lock);
 }
 
 static ssize_t attr_fn_lock_store(struct device *dev,
@@ -599,8 +599,7 @@ static ssize_t attr_sensitivity_show_cptkbd(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *cptkbd_data = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n",
-		cptkbd_data->sensitivity);
+	return sysfs_emit(buf, "%u\n", cptkbd_data->sensitivity);
 }
 
 static ssize_t attr_sensitivity_store_cptkbd(struct device *dev,
@@ -789,7 +788,7 @@ static ssize_t attr_press_to_select_show_tpkbd(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *data_pointer = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n", data_pointer->press_to_select);
+	return sysfs_emit(buf, "%u\n", data_pointer->press_to_select);
 }
 
 static ssize_t attr_press_to_select_store_tpkbd(struct device *dev,
@@ -819,7 +818,7 @@ static ssize_t attr_dragging_show_tpkbd(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *data_pointer = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n", data_pointer->dragging);
+	return sysfs_emit(buf, "%u\n", data_pointer->dragging);
 }
 
 static ssize_t attr_dragging_store_tpkbd(struct device *dev,
@@ -849,7 +848,7 @@ static ssize_t attr_release_to_select_show_tpkbd(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *data_pointer = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n", data_pointer->release_to_select);
+	return sysfs_emit(buf, "%u\n", data_pointer->release_to_select);
 }
 
 static ssize_t attr_release_to_select_store_tpkbd(struct device *dev,
@@ -879,7 +878,7 @@ static ssize_t attr_select_right_show_tpkbd(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *data_pointer = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n", data_pointer->select_right);
+	return sysfs_emit(buf, "%u\n", data_pointer->select_right);
 }
 
 static ssize_t attr_select_right_store_tpkbd(struct device *dev,
@@ -909,8 +908,7 @@ static ssize_t attr_sensitivity_show_tpkbd(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *data_pointer = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n",
-		data_pointer->sensitivity);
+	return sysfs_emit(buf, "%u\n", data_pointer->sensitivity);
 }
 
 static ssize_t attr_sensitivity_store_tpkbd(struct device *dev,
@@ -938,8 +936,7 @@ static ssize_t attr_press_speed_show_tpkbd(struct device *dev,
 	struct hid_device *hdev = to_hid_device(dev);
 	struct lenovo_drvdata *data_pointer = hid_get_drvdata(hdev);
 
-	return snprintf(buf, PAGE_SIZE, "%u\n",
-		data_pointer->press_speed);
+	return sysfs_emit(buf, "%u\n", data_pointer->press_speed);
 }
 
 static ssize_t attr_press_speed_store_tpkbd(struct device *dev,
-- 
2.29.2


  parent reply	other threads:[~2024-01-16  4:53 UTC|newest]

Thread overview: 78+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-16  4:10 [PATCH 00/42] Fix coccicheck warnings Li Zhijian
2024-01-16  4:10 ` [cocci] " Li Zhijian
2024-01-16  4:10 ` Li Zhijian
2024-01-16  4:10 ` Li Zhijian
2024-01-16  4:10 ` [PATCH 01/42] coccinelle: device_attr_show.cocci: update description and warning message Li Zhijian
2024-01-16  4:10   ` [cocci] " Li Zhijian
2024-01-18 20:26   ` Julia Lawall
2024-01-18 20:26     ` Julia Lawall
2024-01-19  2:53     ` Zhijian Li (Fujitsu)
2024-01-19  2:53       ` [cocci] " Zhijian Li (Fujitsu)
2024-01-19  6:25       ` Zhijian Li (Fujitsu)
2024-01-19  6:25         ` [cocci] " Zhijian Li (Fujitsu)
2024-01-16  4:10 ` [PATCH 02/42] arch/arm/mm: Convert snprintf to sysfs_emit Li Zhijian
2024-01-16  4:10   ` Li Zhijian
2024-01-16  4:51 ` [PATCH 03/42] arch/x86/events/amd: " Li Zhijian
2024-01-16  4:51   ` [PATCH 04/42] arch/x86/events/core: " Li Zhijian
2024-01-16  7:01     ` Adrian Hunter
2024-01-16  7:35       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 05/42] arch/x86/events/intel: " Li Zhijian
2024-01-19 22:00     ` Namhyung Kim
2024-01-22  2:59       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 06/42] drivers/bus/fsl-mc: " Li Zhijian
2024-01-16  4:51   ` [PATCH 07/42] drivers/edac: " Li Zhijian
2024-01-16  4:51   ` [PATCH 08/42] drivers/fsi: " Li Zhijian
2024-01-17 14:19     ` Eddie James
2024-01-16  4:51   ` Li Zhijian [this message]
2024-01-16  4:51   ` [PATCH 10/42] drivers/hid/hid-roccat-*: " Li Zhijian
2024-01-16  4:51   ` [PATCH 11/42] drivers/hid: " Li Zhijian
2024-02-24 16:59     ` Bruno Prémont
2024-02-28  9:56       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 12/42] drivers/message/fusion: " Li Zhijian
2024-01-16  4:51   ` [PATCH 13/42] drivers/most: " Li Zhijian
2024-01-16  4:51   ` [PATCH 14/42] drivers/platform/mellanox: " Li Zhijian
2024-01-24 11:01     ` Ilpo Järvinen
2024-01-16  4:51   ` [PATCH 15/42] drivers/ptp: " Li Zhijian
2024-01-16  5:01     ` Rahul Rameshbabu
2024-01-16  5:52       ` Zhijian Li (Fujitsu)
2024-01-16 15:33     ` Jakub Kicinski
2024-01-17  5:39       ` Zhijian Li (Fujitsu)
2024-01-16  4:51   ` [PATCH 16/42] drivers/scsi/53c700: " Li Zhijian
2024-01-16  4:51   ` [PATCH 17/42] drivers/scsi/aacraid: " Li Zhijian
2024-01-16  4:51   ` [PATCH 18/42] drivers/scsi/aic94xx: " Li Zhijian
2024-01-16  4:51   ` [PATCH 19/42] drivers/scsi/arcmsr: " Li Zhijian
2024-01-16  4:51   ` [PATCH 20/42] drivers/scsi/be2iscsi: " Li Zhijian
2024-01-16  4:51   ` [PATCH 21/42] drivers/scsi/fcoe: " Li Zhijian
2024-01-16  4:51   ` [PATCH 22/42] drivers/scsi/fnic: " Li Zhijian
2024-01-18 23:24     ` Karan Tilak Kumar (kartilak)
2024-01-16  4:51   ` [PATCH 23/42] drivers/scsi/hpsa: " Li Zhijian
2024-01-16  4:51   ` [PATCH 24/42] drivers/scsi/hptiop: " Li Zhijian
2024-01-16  4:51   ` [PATCH 25/42] drivers/scsi/ibmvscsi: " Li Zhijian
2024-01-16  4:51     ` Li Zhijian
2024-01-16 21:38     ` Tyrel Datwyler
2024-01-16 21:38       ` Tyrel Datwyler
2024-01-16  4:51   ` [PATCH 26/42] drivers/scsi/ibmvscsi_tgt: " Li Zhijian
2024-01-16 21:38     ` Tyrel Datwyler
2024-01-16  4:51   ` [PATCH 27/42] drivers/scsi/isci: " Li Zhijian
2024-01-16  7:24     ` Artur Paszkiewicz
2024-01-16  4:51   ` [PATCH 28/42] drivers/scsi/megaraid: " Li Zhijian
2024-01-16  4:51   ` [PATCH 29/42] drivers/scsi/mpt3sas: " Li Zhijian
2024-01-16  4:51   ` [PATCH 30/42] drivers/scsi/myrb: " Li Zhijian
2024-01-16  4:51   ` [PATCH 31/42] drivers/scsi/myrs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 32/42] drivers/scsi/ncr53c8xx: " Li Zhijian
2024-01-16  4:51   ` [PATCH 33/42] drivers/scsi/pcmcia/sym53c500_cs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 34/42] drivers/scsi/pm8001: " Li Zhijian
2024-01-16  5:52     ` Jinpu Wang
2024-01-16  4:51   ` [PATCH 35/42] drivers/scsi/pmcraid: " Li Zhijian
2024-01-16  4:51   ` [PATCH 36/42] drivers/scsi/qla4xxx/ql4_attr: " Li Zhijian
2024-01-16  4:51   ` [PATCH 37/42] drivers/scsi/scsi_sysfs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 38/42] drivers/scsi/scsi_transport_fc: " Li Zhijian
2024-01-16  4:51   ` [PATCH 39/42] drivers/scsi/scsi_transport_sas: " Li Zhijian
2024-01-16  4:51   ` [PATCH 40/42] drivers/scsi/snic/snic_attrs: " Li Zhijian
2024-01-16  4:51   ` [PATCH 41/42] drivers/video/fbdev/uvesafb: " Li Zhijian
2024-01-16  4:51     ` Li Zhijian
2024-01-16  4:51   ` [PATCH 42/42] kernel/cpu: " Li Zhijian
2024-01-26 17:32     ` [tip: smp/core] kernel/cpu: Convert snprintf() to sysfs_emit() tip-bot2 for Li Zhijian
2024-02-06  2:07 ` (subset) [PATCH 00/42] Fix coccicheck warnings Martin K. Petersen
2024-02-06  2:07   ` Martin K. Petersen
2024-02-06  2:07   ` [cocci] " Martin K. Petersen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240116045151.3940401-7-lizhijian@fujitsu.com \
    --to=lizhijian@fujitsu.com \
    --cc=benjamin.tissoires@redhat.com \
    --cc=jikos@kernel.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.