Re: [PATCH] HID: wacom: convert sysfs sprintf/snprintf family to sysfs_emit

From: Jiri Kosina
Date: Fri Feb 05 2021 - 04:22:00 EST


On Tue, 2 Feb 2021, Jiapeng Chong wrote:

> Fix the following coccicheck warning:
>
> ./drivers/hid/wacom_sys.c:1828:8-16: WARNING: use scnprintf or sprintf.
>
> Reported-by: Abaci Robot<abaci@xxxxxxxxxxxxxxxxx>
> Signed-off-by: Jiapeng Chong <jiapeng.chong@xxxxxxxxxxxxxxxxx>
> ---
> drivers/hid/wacom_sys.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/hid/wacom_sys.c b/drivers/hid/wacom_sys.c
> index aa9e488..8328ef1 100644
> --- a/drivers/hid/wacom_sys.c
> +++ b/drivers/hid/wacom_sys.c
> @@ -1825,7 +1825,7 @@ static ssize_t wacom_show_speed(struct device *dev,
> struct hid_device *hdev = to_hid_device(dev);
> struct wacom *wacom = hid_get_drvdata(hdev);
>
> - return snprintf(buf, PAGE_SIZE, "%i\n", wacom->wacom_wac.bt_high_speed);
> + return sysfs_emit(buf, "%i\n", wacom->wacom_wac.bt_high_speed);
> }

Applied, thanks.

--
Jiri Kosina
SUSE Labs