[igt-dev] [PATCH i-g-t 3/8] tests/kms_chamelium: rename do_test_display_audio and test_audio_configuration
Simon Ser
simon.ser at intel.com
Fri May 24 08:07:31 UTC 2019
- Rename do_test_display_audio to test_audio_frequencies to prepare for a
future amplitude/delay test
- Rename test_audio_configuration to check_audio_configuration because this
function doesn't execute any test, it just checks whether we can perform
audio tests using a particular configuration
Signed-off-by: Simon Ser <simon.ser at intel.com>
---
tests/kms_chamelium.c | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
index 3a78aa150bef..0324d81b81e3 100644
--- a/tests/kms_chamelium.c
+++ b/tests/kms_chamelium.c
@@ -1003,7 +1003,7 @@ static void audio_state_stop(struct audio_state *state, bool success)
}
static int
-audio_output_callback(void *data, void *buffer, int samples)
+audio_output_frequencies_callback(void *data, void *buffer, int samples)
{
struct audio_state *state = data;
@@ -1024,7 +1024,7 @@ audio_output_callback(void *data, void *buffer, int samples)
return state->run ? 0 : -1;
}
-static bool do_test_display_audio(struct audio_state *state)
+static bool test_audio_frequencies(struct audio_state *state)
{
int freq, step;
int32_t *recv, *buf;
@@ -1058,7 +1058,8 @@ static bool do_test_display_audio(struct audio_state *state)
}
audio_signal_synthesize(state->signal);
- alsa_register_output_callback(state->alsa, audio_output_callback, state,
+ alsa_register_output_callback(state->alsa,
+ audio_output_frequencies_callback, state,
PLAYBACK_SAMPLES);
audio_state_start(state);
@@ -1133,8 +1134,8 @@ static bool do_test_display_audio(struct audio_state *state)
return success;
}
-static bool test_audio_configuration(struct alsa *alsa, snd_pcm_format_t format,
- int channels, int sampling_rate)
+static bool check_audio_configuration(struct alsa *alsa, snd_pcm_format_t format,
+ int channels, int sampling_rate)
{
if (!alsa_test_output_configuration(alsa, format, channels,
sampling_rate)) {
@@ -1219,15 +1220,15 @@ test_display_audio(data_t *data, struct chamelium_port *port,
channels = PLAYBACK_CHANNELS;
sampling_rate = test_sampling_rates[i];
- if (!test_audio_configuration(alsa, format, channels,
- sampling_rate))
+ if (!check_audio_configuration(alsa, format, channels,
+ sampling_rate))
continue;
run = true;
audio_state_init(&state, data, alsa, port,
format, channels, sampling_rate);
- success &= do_test_display_audio(&state);
+ success &= test_audio_frequencies(&state);
audio_state_fini(&state);
alsa_close_output(alsa);
--
2.21.0
More information about the igt-dev
mailing list