From: Anssi Hannula Date: Sun, 10 Nov 2013 18:29:18 +0000 (+0200) Subject: speaker-test: Always show chmap channel names if available X-Git-Tag: v1.0.28~27 X-Git-Url: https://git.alsa-project.org/?a=commitdiff_plain;h=5b0c6da0d8166cd404379740c5f75fda44ca5806;p=alsa-utils.git speaker-test: Always show chmap channel names if available Currently speaker-test only uses channel names retrieved by snd_pcm_chmap_long_name() when a channel map has been manually set. However, the device may provide a default (or fixed) channel map that differs from the traditional ALSA map, in which case wrong channel names are shown. Fix that by always using the name from the channel map when a channel map is present. Note that the names retrieved by snd_pcm_chmap_long_name() are not currently localized via gettext. Also note that Linux kernel HDMI driver reported wrong default channel maps before 56cac413dd6d43af8355f5d1f90a199b540f73fc ("ALSA: hda - hdmi: Fix reported channel map on common default layouts"). Signed-off-by: Anssi Hannula Signed-off-by: Takashi Iwai --- diff --git a/speaker-test/speaker-test.c b/speaker-test/speaker-test.c index 25b08dc..101bb0d 100644 --- a/speaker-test/speaker-test.c +++ b/speaker-test/speaker-test.c @@ -267,7 +267,7 @@ static int get_speaker_channel(int chn) static const char *get_channel_name(int chn) { #ifdef CONFIG_SUPPORT_CHMAP - if (channel_map_set && chn < channel_map->channels) { + if (channel_map && chn < channel_map->channels) { const char *name = snd_pcm_chmap_long_name(channel_map->pos[chn]); return name ? name : "Unknown"; }