Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-11 Thread Markus Armbruster
Thomas Huth writes: > On 09/11/2020 13.50, Markus Armbruster wrote: >> Alex Chen writes: >> >>> On 2020/11/9 15:57, Markus Armbruster wrote: Thomas Huth writes: > On 06/11/2020 15.18, Philippe Mathieu-Daudé wrote: >> On 11/6/20 7:33 AM, Markus Armbruster wrote: >>> Thomas

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-10 Thread Thomas Huth
On 09/11/2020 13.50, Markus Armbruster wrote: > Alex Chen writes: > >> On 2020/11/9 15:57, Markus Armbruster wrote: >>> Thomas Huth writes: >>> On 06/11/2020 15.18, Philippe Mathieu-Daudé wrote: > On 11/6/20 7:33 AM, Markus Armbruster wrote: >> Thomas Huth writes: >> >>> On

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-09 Thread Markus Armbruster
Alex Chen writes: > On 2020/11/9 15:57, Markus Armbruster wrote: >> Thomas Huth writes: >> >>> On 06/11/2020 15.18, Philippe Mathieu-Daudé wrote: On 11/6/20 7:33 AM, Markus Armbruster wrote: > Thomas Huth writes: > >> On 05/11/2020 06.14, AlexChen wrote: >>> On 2020/11/4 1

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-09 Thread Alex Chen
On 2020/11/9 15:57, Markus Armbruster wrote: > Thomas Huth writes: > >> On 06/11/2020 15.18, Philippe Mathieu-Daudé wrote: >>> On 11/6/20 7:33 AM, Markus Armbruster wrote: Thomas Huth writes: > On 05/11/2020 06.14, AlexChen wrote: >> On 2020/11/4 18:44, Thomas Huth wrote: >

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-08 Thread Markus Armbruster
Thomas Huth writes: > On 06/11/2020 15.18, Philippe Mathieu-Daudé wrote: >> On 11/6/20 7:33 AM, Markus Armbruster wrote: >>> Thomas Huth writes: >>> On 05/11/2020 06.14, AlexChen wrote: > On 2020/11/4 18:44, Thomas Huth wrote: >> On 04/11/2020 11.23, AlexChen wrote: >>> We shoul

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-07 Thread Thomas Huth
On 06/11/2020 15.18, Philippe Mathieu-Daudé wrote: > On 11/6/20 7:33 AM, Markus Armbruster wrote: >> Thomas Huth writes: >> >>> On 05/11/2020 06.14, AlexChen wrote: On 2020/11/4 18:44, Thomas Huth wrote: > On 04/11/2020 11.23, AlexChen wrote: >> We should use printf format specifier "

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-07 Thread Thomas Huth
On 05/11/2020 09.19, Markus Armbruster wrote: > Thomas Huth writes: > >> On 04/11/2020 11.23, AlexChen wrote: >>> We should use printf format specifier "%u" instead of "%d" for >>> argument of type "unsigned int". >>> >>> Reported-by: Euler Robot >>> Signed-off-by: Alex Chen >>> --- >>> tests/

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-06 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > On 11/6/20 7:33 AM, Markus Armbruster wrote: [...] >> In other words "%" PRIu32 is just a less legible alias for "%u" in all >> cases that matter. > > Can we add a checkpatch rule to avoid using 'PRI[dux]32' format, > so it is clear for everyone? I guess we could

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-06 Thread Philippe Mathieu-Daudé
On 11/6/20 7:33 AM, Markus Armbruster wrote: > Thomas Huth writes: > >> On 05/11/2020 06.14, AlexChen wrote: >>> On 2020/11/4 18:44, Thomas Huth wrote: On 04/11/2020 11.23, AlexChen wrote: > We should use printf format specifier "%u" instead of "%d" for > argument of type "unsigned i

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-05 Thread Markus Armbruster
Thomas Huth writes: > On 05/11/2020 06.14, AlexChen wrote: >> On 2020/11/4 18:44, Thomas Huth wrote: >>> On 04/11/2020 11.23, AlexChen wrote: We should use printf format specifier "%u" instead of "%d" for argument of type "unsigned int". Reported-by: Euler Robot Signed-o

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-05 Thread Markus Armbruster
Thomas Huth writes: > On 04/11/2020 11.23, AlexChen wrote: >> We should use printf format specifier "%u" instead of "%d" for >> argument of type "unsigned int". >> >> Reported-by: Euler Robot >> Signed-off-by: Alex Chen >> --- >> tests/qtest/arm-cpu-features.c | 8 >> 1 file changed,

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-04 Thread Thomas Huth
On 05/11/2020 06.14, AlexChen wrote: > On 2020/11/4 18:44, Thomas Huth wrote: >> On 04/11/2020 11.23, AlexChen wrote: >>> We should use printf format specifier "%u" instead of "%d" for >>> argument of type "unsigned int". >>> >>> Reported-by: Euler Robot >>> Signed-off-by: Alex Chen >>> --- >>>

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-04 Thread AlexChen
On 2020/11/4 18:44, Thomas Huth wrote: > On 04/11/2020 11.23, AlexChen wrote: >> We should use printf format specifier "%u" instead of "%d" for >> argument of type "unsigned int". >> >> Reported-by: Euler Robot >> Signed-off-by: Alex Chen >> --- >> tests/qtest/arm-cpu-features.c | 8 >>

Re: [PATCH] qtest: Fix bad printf format specifiers

2020-11-04 Thread Thomas Huth
On 04/11/2020 11.23, AlexChen wrote: > We should use printf format specifier "%u" instead of "%d" for > argument of type "unsigned int". > > Reported-by: Euler Robot > Signed-off-by: Alex Chen > --- > tests/qtest/arm-cpu-features.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-)

[PATCH] qtest: Fix bad printf format specifiers

2020-11-04 Thread AlexChen
We should use printf format specifier "%u" instead of "%d" for argument of type "unsigned int". Reported-by: Euler Robot Signed-off-by: Alex Chen --- tests/qtest/arm-cpu-features.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/qtest/arm-cpu-features.c b/tests