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

PATCH Superseded Elise Lennion 1 participant 1 message Oct 14, 2016

Subsystems (3)

Staging Greybus Greybus audio protocols drivers

lore

Share & tools

lore

Elise Lennion

author

2016-10-14 21:54 UTC #0 lore raw

PATCH v3 1/1 staging: greybus: audio_manager_sysfs: 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 376e17c..e74fdcf 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