]> git.baikalelectronics.ru Git - kernel.git/commit
ALSA: hda - hdmi: Set converter channel count even without sink
authorAnssi Hannula <anssi.hannula@iki.fi>
Sun, 4 May 2014 23:38:43 +0000 (02:38 +0300)
committerTakashi Iwai <tiwai@suse.de>
Mon, 5 May 2014 14:28:10 +0000 (16:28 +0200)
commit2ac2268a9848ef039fedb5251c6e30fe6ebb2e79
tree5f39fcbc3ca8c1e7b112c51e48d32465b49c66a9
parent2b8dd2fd978ebe1e992b506751c77ef7cb56b0ef
ALSA: hda - hdmi: Set converter channel count even without sink

Since commit 844df56f ("ALSA: hda - hdmi: Fix programmed active channel
count") channel count is no longer being set if monitor_present is 0.
This is because setting the count was moved after the CA value is
determined, which is only after the monitor_present check in
hdmi_setup_audio_infoframe().

Unfortunately, in some cases, such as with a non-spec-compliant codec or
with a problematic video driver, monitor_present is always 0. As a
specific example, this seems to happen with gen1 ATV (SiI1390 codec),
causing left-channel-only stereo playback (multi-channel playback has
apparently never worked with this codec despite it reporting 8 channels,
reason unknown).

Simply setting converter channel count without setting the pin infoframe
and channel mapping as well does not theoretically make much sense as
this will just mean they are out-of-sync and multichannel playback will
have a wrong channel mapping.

However, adding back just setting the converter channel count even in
no-monitor case is the safest change which at least fixes the stereo
playback regression on SiI1390 codec. Do that.

Signed-off-by: Anssi Hannula <anssi.hannula@iki.fi>
Reported-by: Stephan Raue <stephan@openelec.tv>
Tested-by: Stephan Raue <stephan@openelec.tv>
Cc: <stable@vger.kernel.org> # 3.12+
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/patch_hdmi.c