Skip to contentSkip to conversation

staging: greybus: Replace sscanf with kstrto* to single variable conversion.

PATCH In review Elise Lennion 1 participant 1 message Oct 11, 2016

Subsystems (3)
Staging Greybus Greybus audio protocols drivers
lore
Share & tools
lore

Elise Lennion

author
#0 lore raw

PATCH 7/7 staging: greybus: Replace sscanf with kstrto* to single variable conversion.

Fix checkpatch warning:

WARNING: Prefer kstrto<type> to single variable sscanf

kstrto* is designed to convert string to numerical value and makes it easier to understand what the code does.

Signed-off-by Elise Lennion <[email protected]>
---
 drivers/staging/greybus/audio_manager_sysfs.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/staging/greybus/audio_manager_sysfs.c +2 -2
index d8bf859..469a922 100644
@@ -44,7 +44,7 @@ static ssize_t manager_sysfs_remove_store(
 {
 	int id;
 
-	int num = sscanf(buf, "%d", &id);
+	int num = kstrtoint(buf, 10, &id);
 
 	if (num != 1)
 		return -EINVAL;

@@ -65,7 +65,7 @@ static ssize_t manager_sysfs_dump_store(
 {
 	int id;
 
-	int num = sscanf(buf, "%d", &id);
+	int num = kstrtoint(buf, 10, &id);
 
 	if (num == 1) {
 		num = gb_audio_manager_dump_module(id);
Signature · 1 line
2.7.4