RE: [char-misc-next] mei: bus: use simple sprintf for sysfs
From: Winkler, Tomas
Date: Thu Jan 09 2020 - 03:29:03 EST
>
> Replace scnprintf with simple sprintf for sysfs files.
> it is implicitly known that the buffer is big enough for the variables to fit in.
>
> Signed-off-by: Tomas Winkler <tomas.winkler@xxxxxxxxx>
Hi Greg, hope you have great holiday, has anything changed in the misc-char dev process,
no patches were staged including this one?
Thanks
Tomas
> ---
> drivers/misc/mei/bus.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/misc/mei/bus.c b/drivers/misc/mei/bus.c index
> a0a495c95e3c..8d468e0a950a 100644
> --- a/drivers/misc/mei/bus.c
> +++ b/drivers/misc/mei/bus.c
> @@ -765,7 +765,7 @@ static ssize_t uuid_show(struct device *dev, struct
> device_attribute *a,
> struct mei_cl_device *cldev = to_mei_cl_device(dev);
> const uuid_le *uuid = mei_me_cl_uuid(cldev->me_cl);
>
> - return scnprintf(buf, PAGE_SIZE, "%pUl", uuid);
> + return sprintf(buf, "%pUl", uuid);
> }
> static DEVICE_ATTR_RO(uuid);
>
> @@ -775,7 +775,7 @@ static ssize_t version_show(struct device *dev, struct
> device_attribute *a,
> struct mei_cl_device *cldev = to_mei_cl_device(dev);
> u8 version = mei_me_cl_ver(cldev->me_cl);
>
> - return scnprintf(buf, PAGE_SIZE, "%02X", version);
> + return sprintf(buf, "%02X", version);
> }
> static DEVICE_ATTR_RO(version);
>
> @@ -797,7 +797,7 @@ static ssize_t max_conn_show(struct device *dev,
> struct device_attribute *a,
> struct mei_cl_device *cldev = to_mei_cl_device(dev);
> u8 maxconn = mei_me_cl_max_conn(cldev->me_cl);
>
> - return scnprintf(buf, PAGE_SIZE, "%d", maxconn);
> + return sprintf(buf, "%d", maxconn);
> }
> static DEVICE_ATTR_RO(max_conn);
>
> @@ -807,7 +807,7 @@ static ssize_t fixed_show(struct device *dev, struct
> device_attribute *a,
> struct mei_cl_device *cldev = to_mei_cl_device(dev);
> u8 fixed = mei_me_cl_fixed(cldev->me_cl);
>
> - return scnprintf(buf, PAGE_SIZE, "%d", fixed);
> + return sprintf(buf, "%d", fixed);
> }
> static DEVICE_ATTR_RO(fixed);
>
> @@ -817,7 +817,7 @@ static ssize_t max_len_show(struct device *dev, struct
> device_attribute *a,
> struct mei_cl_device *cldev = to_mei_cl_device(dev);
> u32 maxlen = mei_me_cl_max_len(cldev->me_cl);
>
> - return scnprintf(buf, PAGE_SIZE, "%u", maxlen);
> + return sprintf(buf, "%u", maxlen);
> }
> static DEVICE_ATTR_RO(max_len);
>
> --
> 2.21.0