| Message ID | 20240626084859.4350-1-zhujun2@cmss.chinamobile.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 5b707581c072bcef086ef4c122a2ea81470a1027 |
| Headers | show |
| Series | selftests/alsa:Fix printf format string in pcm-test.c | expand |
On Wed, Jun 26, 2024 at 01:48:59AM -0700, Zhu Jun wrote: > Inside of test_pcm_time() arguments are printed via printf > but '%d' is used to print @flags (of type unsigned int). > Use '%u' instead, just like we do everywhere else. > > Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com> Reviewed-by: Mark Brown <broonie@kernel.org>
On Wed, 26 Jun 2024 10:48:59 +0200, Zhu Jun wrote: > > Inside of test_pcm_time() arguments are printed via printf > but '%d' is used to print @flags (of type unsigned int). > Use '%u' instead, just like we do everywhere else. > > Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com> Applied now to for-next branch. Thanks. Takashi
diff --git a/tools/testing/selftests/alsa/pcm-test.c b/tools/testing/selftests/alsa/pcm-test.c index de664dedb541..914efcdce1ec 100644 --- a/tools/testing/selftests/alsa/pcm-test.c +++ b/tools/testing/selftests/alsa/pcm-test.c @@ -383,7 +383,7 @@ static void test_pcm_time(struct pcm_data *data, enum test_class class, goto __close; } if (rrate != rate) { - snprintf(msg, sizeof(msg), "rate mismatch %ld != %d", rate, rrate); + snprintf(msg, sizeof(msg), "rate mismatch %ld != %u", rate, rrate); goto __close; } rperiod_size = period_size;
Inside of test_pcm_time() arguments are printed via printf but '%d' is used to print @flags (of type unsigned int). Use '%u' instead, just like we do everywhere else. Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com> --- tools/testing/selftests/alsa/pcm-test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)