[PATCH] tools/gpio:Fix the wrong format specifier

Zhu Jun posted 1 patch 1 year, 6 months ago
tools/gpio/gpio-hammer.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[PATCH] tools/gpio:Fix the wrong format specifier
Posted by Zhu Jun 1 year, 6 months ago
The unsigned int should use "%u" instead of "%d".

Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com>
---
 tools/gpio/gpio-hammer.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/gpio/gpio-hammer.c b/tools/gpio/gpio-hammer.c
index 54fdf59dd320..ba0866eb3581 100644
--- a/tools/gpio/gpio-hammer.c
+++ b/tools/gpio/gpio-hammer.c
@@ -54,7 +54,7 @@ int hammer_device(const char *device_name, unsigned int *lines, int num_lines,
 
 	fprintf(stdout, "Hammer lines [");
 	for (i = 0; i < num_lines; i++) {
-		fprintf(stdout, "%d", lines[i]);
+		fprintf(stdout, "%u", lines[i]);
 		if (i != (num_lines - 1))
 			fprintf(stdout, ", ");
 	}
@@ -89,7 +89,7 @@ int hammer_device(const char *device_name, unsigned int *lines, int num_lines,
 
 		fprintf(stdout, "[");
 		for (i = 0; i < num_lines; i++) {
-			fprintf(stdout, "%d: %d", lines[i],
+			fprintf(stdout, "%u: %d", lines[i],
 				gpiotools_test_bit(values.bits, i));
 			if (i != (num_lines - 1))
 				fprintf(stdout, ", ");
-- 
2.17.1
Re: [PATCH] tools/gpio:Fix the wrong format specifier
Posted by Bartosz Golaszewski 1 year, 6 months ago
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>


On Tue, 23 Jul 2024 19:46:36 -0700, Zhu Jun wrote:
> The unsigned int should use "%u" instead of "%d".
> 
> 

Applied, thanks!

[1/1] tools/gpio:Fix the wrong format specifier
      commit: ac93ca125b5409df56c5139648bbe10fd1ea989b

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>