* [char-misc-next] mei: bus: use simple sprintf for sysfs
@ 2019-11-24 10:15 Tomas Winkler
0 siblings, 0 replies; 5+ messages in thread
From: Tomas Winkler @ 2019-11-24 10:15 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Alexander Usyskin, linux-kernel, Tomas Winkler
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@intel.com>
---
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [char-misc-next] mei: bus: use simple sprintf for sysfs
@ 2019-11-26 12:30 Tomas Winkler
2020-01-09 8:28 ` Winkler, Tomas
0 siblings, 1 reply; 5+ messages in thread
From: Tomas Winkler @ 2019-11-26 12:30 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Alexander Usyskin, linux-kernel, Tomas Winkler
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@intel.com>
---
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
^ permalink raw reply related [flat|nested] 5+ messages in thread
* RE: [char-misc-next] mei: bus: use simple sprintf for sysfs
2019-11-26 12:30 Tomas Winkler
@ 2020-01-09 8:28 ` Winkler, Tomas
2020-01-09 8:53 ` Greg Kroah-Hartman
0 siblings, 1 reply; 5+ messages in thread
From: Winkler, Tomas @ 2020-01-09 8:28 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Usyskin, Alexander, linux-kernel@vger.kernel.org
>
> 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@intel.com>
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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [char-misc-next] mei: bus: use simple sprintf for sysfs
2020-01-09 8:28 ` Winkler, Tomas
@ 2020-01-09 8:53 ` Greg Kroah-Hartman
2020-01-09 9:03 ` Winkler, Tomas
0 siblings, 1 reply; 5+ messages in thread
From: Greg Kroah-Hartman @ 2020-01-09 8:53 UTC (permalink / raw)
To: Winkler, Tomas; +Cc: Usyskin, Alexander, linux-kernel@vger.kernel.org
On Thu, Jan 09, 2020 at 08:28:56AM +0000, Winkler, Tomas wrote:
> >
> > 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@intel.com>
>
> Hi Greg, hope you have great holiday, has anything changed in the misc-char dev process,
> no patches were staged including this one?
$ mdfrm -c ~/mail/todo
1430 messages in /home/gregkh/mail/todo
I am way behind due to the holidays, please give me a chance to catch
up, especially for trivial stuff like this :)
thanks,
greg k-h
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [char-misc-next] mei: bus: use simple sprintf for sysfs
2020-01-09 8:53 ` Greg Kroah-Hartman
@ 2020-01-09 9:03 ` Winkler, Tomas
0 siblings, 0 replies; 5+ messages in thread
From: Winkler, Tomas @ 2020-01-09 9:03 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: Usyskin, Alexander, linux-kernel@vger.kernel.org
>
> On Thu, Jan 09, 2020 at 08:28:56AM +0000, Winkler, Tomas wrote:
> > >
> > > 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@intel.com>
> >
> > Hi Greg, hope you have great holiday, has anything changed in the
> > misc-char dev process, no patches were staged including this one?
>
> $ mdfrm -c ~/mail/todo
> 1430 messages in /home/gregkh/mail/todo
>
> I am way behind due to the holidays, please give me a chance to catch up,
> especially for trivial stuff like this :)
No problem, just wanted to know.
Thanks.
Tomas.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-01-09 9:03 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-11-24 10:15 [char-misc-next] mei: bus: use simple sprintf for sysfs Tomas Winkler
-- strict thread matches above, loose matches on Subject: below --
2019-11-26 12:30 Tomas Winkler
2020-01-09 8:28 ` Winkler, Tomas
2020-01-09 8:53 ` Greg Kroah-Hartman
2020-01-09 9:03 ` Winkler, Tomas
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox