aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2010-01-16 18:08:13 +0100
committerMax Kellermann <max@duempel.org>2010-01-16 23:44:52 +0100
commitda47afe7d1aa6b59bf04764d0bd7d0b91dfac94b (patch)
treef43e589ffed29d4c7b8efd9f23d84cf4ae74a184
parent96546c1a8a319f2c549bea20b7ebb6aceb85bdd3 (diff)
downloadmpd-da47afe7d1aa6b59bf04764d0bd7d0b91dfac94b.tar.gz
mpd-da47afe7d1aa6b59bf04764d0bd7d0b91dfac94b.tar.xz
mpd-da47afe7d1aa6b59bf04764d0bd7d0b91dfac94b.zip
output/alsa: probe all sample formats in a loop
More code simplification. Probe all formats, no matter which input format.
Diffstat (limited to '')
-rw-r--r--src/output/alsa_plugin.c70
1 files changed, 34 insertions, 36 deletions
diff --git a/src/output/alsa_plugin.c b/src/output/alsa_plugin.c
index d90be7815..e157609db 100644
--- a/src/output/alsa_plugin.c
+++ b/src/output/alsa_plugin.c
@@ -260,55 +260,53 @@ alsa_output_try_reverse(snd_pcm_t *pcm, snd_pcm_hw_params_t *hwparams,
}
/**
+ * Attempts to configure the specified sample format, and tries the
+ * reversed host byte order if was not supported.
+ */
+static int
+alsa_output_try_format_both(snd_pcm_t *pcm, snd_pcm_hw_params_t *hwparams,
+ struct audio_format *audio_format,
+ enum sample_format sample_format)
+{
+ int err = alsa_output_try_format(pcm, hwparams, audio_format,
+ sample_format);
+ if (err == -EINVAL)
+ err = alsa_output_try_reverse(pcm, hwparams, audio_format,
+ sample_format);
+
+ return err;
+}
+
+/**
* Configure a sample format, and probe other formats if that fails.
*/
static int
alsa_output_setup_format(snd_pcm_t *pcm, snd_pcm_hw_params_t *hwparams,
struct audio_format *audio_format)
{
- snd_pcm_format_t bitformat = get_bitformat(audio_format->format);
- if (bitformat == SND_PCM_FORMAT_UNKNOWN) {
- /* sample format is not supported by this plugin -
- fall back to 16 bit samples */
-
- audio_format->format = SAMPLE_FORMAT_S16;
- bitformat = SND_PCM_FORMAT_S16;
- }
+ /* try the input format first */
- int err = snd_pcm_hw_params_set_format(pcm, hwparams, bitformat);
+ int err = alsa_output_try_format_both(pcm, hwparams, audio_format,
+ audio_format->format);
if (err != -EINVAL)
return err;
- err = alsa_output_try_reverse(pcm, hwparams, audio_format,
- audio_format->format);
- if (err != -EINVAL)
- return err;
+ /* if unsupported by the hardware, try other formats */
- if (audio_format->format == SAMPLE_FORMAT_S24_P32 ||
- audio_format->format == SAMPLE_FORMAT_S16) {
- /* fall back to 32 bit, let pcm_convert.c do the conversion */
+ static const enum sample_format probe_formats[] = {
+ SAMPLE_FORMAT_S24_P32,
+ SAMPLE_FORMAT_S32,
+ SAMPLE_FORMAT_S16,
+ SAMPLE_FORMAT_S8,
+ SAMPLE_FORMAT_UNDEFINED,
+ };
- err = alsa_output_try_format(pcm, hwparams, audio_format,
- SAMPLE_FORMAT_S24_P32);
- if (err != -EINVAL)
- return err;
-
- err = alsa_output_try_reverse(pcm, hwparams, audio_format,
- SAMPLE_FORMAT_S24_P32);
- if (err != -EINVAL)
- return err;
- }
-
- if (audio_format->format != SAMPLE_FORMAT_S16) {
- /* fall back to 16 bit, let pcm_convert.c do the conversion */
-
- err = alsa_output_try_format(pcm, hwparams, audio_format,
- SAMPLE_FORMAT_S16);
- if (err != -EINVAL)
- return err;
+ for (unsigned i = 0; probe_formats[i] != SAMPLE_FORMAT_UNDEFINED; ++i) {
+ if (probe_formats[i] == audio_format->format)
+ continue;
- err = alsa_output_try_reverse(pcm, hwparams, audio_format,
- SAMPLE_FORMAT_S16);
+ err = alsa_output_try_format_both(pcm, hwparams, audio_format,
+ probe_formats[i]);
if (err != -EINVAL)
return err;
}