Patch "kselftest/alsa - mixer-test: fix the number of parameters to ksft_exit_fail_msg()" has been added to the 6.1-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    kselftest/alsa - mixer-test: fix the number of parameters to ksft_exit_fail_msg()

to the 6.1-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     kselftest-alsa-mixer-test-fix-the-number-of-paramete.patch
and it can be found in the queue-6.1 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 362d5896a6c6b9dc71e488bbb18a468138ec5a17
Author: Mirsad Todorovac <mirsad.todorovac@xxxxxxxxxxxx>
Date:   Sun Jan 7 18:37:02 2024 +0100

    kselftest/alsa - mixer-test: fix the number of parameters to ksft_exit_fail_msg()
    
    [ Upstream commit 8c51c13dc63d46e754c44215eabc0890a8bd9bfb ]
    
    Minor fix in the number of arguments to error reporting function in the
    test program as reported by GCC 13.2.0 warning.
    
    mixer-test.c: In function ‘find_controls’:
    mixer-test.c:169:44: warning: too many arguments for format [-Wformat-extra-args]
      169 |                         ksft_exit_fail_msg("snd_ctl_poll_descriptors() failed for %d\n",
          |                                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    The number of arguments in call to ksft_exit_fail_msg() doesn't correspond
    to the format specifiers, so this is adjusted resembling the sibling calls
    to the error function.
    
    Fixes: b1446bda56456 ("kselftest: alsa: Check for event generation when we write to controls")
    Cc: Mark Brown <broonie@xxxxxxxxxx>
    Cc: Jaroslav Kysela <perex@xxxxxxxx>
    Cc: Takashi Iwai <tiwai@xxxxxxxx>
    Cc: Shuah Khan <shuah@xxxxxxxxxx>
    Cc: linux-sound@xxxxxxxxxxxxxxx
    Cc: linux-kselftest@xxxxxxxxxxxxxxx
    Cc: linux-kernel@xxxxxxxxxxxxxxx
    Signed-off-by: Mirsad Todorovac <mirsad.todorovac@xxxxxxxxxxxx>
    Acked-by: Mark Brown <broonie@xxxxxxxxxx>
    Link: https://lore.kernel.org/r/20240107173704.937824-2-mirsad.todorovac@xxxxxxxxxxxx
    Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/tools/testing/selftests/alsa/mixer-test.c b/tools/testing/selftests/alsa/mixer-test.c
index 37da902545a4..d59910658c8c 100644
--- a/tools/testing/selftests/alsa/mixer-test.c
+++ b/tools/testing/selftests/alsa/mixer-test.c
@@ -205,7 +205,7 @@ static void find_controls(void)
 		err = snd_ctl_poll_descriptors(card_data->handle,
 					       &card_data->pollfd, 1);
 		if (err != 1) {
-			ksft_exit_fail_msg("snd_ctl_poll_descriptors() failed for %d\n",
+			ksft_exit_fail_msg("snd_ctl_poll_descriptors() failed for card %d: %d\n",
 				       card, err);
 		}
 




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux