[alsa-devel] Wrong channel order with multichannel HDMI on MCP7A

Hi all!
I have a motherboard with NVIDIA MCP7A HDMI audio. However, multichannel audio is not mapped properly into the HDMI order.
On 5.1, I get RL in FC, RR in LFE, FC in RL, LFE in RR.
Interestingly, this is not consistent with the ALSA channel order (FL FR RL RR FC LFE) being passed directly to HDMI (FL FR LFE FC RL RR). Instead it looks like there has been a waveformatex (FL FR FC LFE RL RR; windows?) => HDMI conversion instead of ALSA => HDMI.
Same happens for 7.1.
Is it possible to set the hardware to do an ALSA => HDMI conversion instead? If not, we should manually compensate for this somewhere, right?
Somewhat relatedly, trying to output 4 channels results in silence only. Trying to output 3 or 5 channels triggers a timeout with I/O error, with the following in the kernel log: "ALSA pcm_lib.c:1757: playback write error (DMA or IRQ trouble?)"
I used a git alsa-kmirror from few days ago on 2.6.35-rc4 for testing. Speaker-test and alsa-lib were 1.0.23.

At Sun, 25 Jul 2010 19:51:39 +0300, Anssi Hannula wrote:
Does the patch below change anything?
Somewhat relatedly, trying to output 4 channels results in silence only.
Although 5.1/7.1 work?
HD-audio doesn't support the odd number of channels in general, so they should be played as 4 or 6 channels. But, it looks like something wrong in the setup...
thanks,
Takashi
--- diff --git a/sound/pci/hda/patch_nvhdmi.c b/sound/pci/hda/patch_nvhdmi.c index 3c10c0b..7355c60 100644 --- a/sound/pci/hda/patch_nvhdmi.c +++ b/sound/pci/hda/patch_nvhdmi.c @@ -511,6 +511,8 @@ static int patch_nvhdmi_8ch_7x(struct hda_codec *codec)
codec->patch_ops = nvhdmi_patch_ops_8ch_7x;
+ init_channel_allocations(); + return 0; }

Takashi Iwai kirjoitti maanantai, 26. heinäkuuta 2010 12:14:17:
AFAICS no stuff from patch_hdmi.c is used for MCP7x, so it has no effect. Tested anyway, it doesn't change anything.
Somewhat relatedly, trying to output 4 channels results in silence only.
Although 5.1/7.1 work?
Yep, with wrong channel order.

Hi, Takashi MCP7x use old HW design in HDMI, they don't support channel mapping like MCP89. It use nvhdmi_dig_playback_pcm_prepare_8ch() to prepare channel mapping and steam id,format.
Anssi Hannula Please try the following changes in patch_nvhdmi.c static hda_nid_t nvhdmi_con_nids_7x[4] = { /*front, rear, clfe, rear_surr */ - 0x6, 0x8, 0xa, 0xc, + 0x6, 0xa, 0x8, 0xc, };
Thanks Wei.
-----Original Message----- From: Takashi Iwai [mailto:tiwai@suse.de] Sent: Monday, July 26, 2010 5:35 PM To: Anssi Hannula Cc: alsa-devel@alsa-project.org; Wei Ni Subject: Re: [alsa-devel] Wrong channel order with multichannel HDMI on MCP7A
At Mon, 26 Jul 2010 11:14:17 +0200, 私 wrote:
Ah crap, it should be irrelevant. Wei, don't MCP 7x chips support the channel mapping as MCP 89?
thanks,
Takashi
----------------------------------------------------------------------------------- This email message is for the sole use of the intended recipient(s) and may contain confidential information. Any unauthorized review, use, disclosure or distribution is prohibited. If you are not the intended recipient, please contact the sender by reply email and destroy all copies of the original message. -----------------------------------------------------------------------------------

Wei Ni kirjoitti tiistai, 27. heinäkuuta 2010 10:14:37:
I actually tried that already, but for some reason changing the order in nvhdmi_con_nids_7x doesn't have any effect.
Actually, disabling the whole for loop that assigns the channels in nvhdmi_dig_playback_pcm_prepare_8ch() has no effect, leading me to believe that the commands are not reaching the hardware or they are being overridden.
I also just noticed that the HDMI output is the only detected output, even though this board has analog audio and S/PDIF outputs as well. I'm not sure if this is related or not.

Anssi Hannula kirjoitti keskiviikko, 28. heinäkuuta 2010 05:42:09:
Well, I confirmed with AC_VERB_GET_CONV that they are getting through, and they are still the same in nvhdmi_dig_playback_pcm_close_8ch_7x().
Any idea why they are having no effect?
This was not related, updating to latest git fixed it.

Anssi Hannula kirjoitti torstai, 29. heinäkuuta 2010 17:41:15:
If this can't be fixed (either due to hardware or missing documentation), how can this be workarounded in alsa?
Only idea I can come up with is reserving a special device index in hda-intel for use with "broken hdmi", which can then be handled in HDA-Intel.conf with an appropriate ttable.. But that would be rather ugly.
Also, do we know if this affects all MCP7x devices or a subset of them?

At Tue, 3 Aug 2010 22:01:47 +0300, Anssi Hannula wrote:
Well, this is one of my looong-standing TODOs. Originally I wanted to implement a new API to get/set the channel-mapping, mainly for HDMI. But, I've been recently too busy for other tasks (I can spend little time for ALSA), thus it's pending forever.
Takashi

You would only need an API to get the channel mapping. The HDMI sink will report its speaker configuration through a CEA code (section 7.4 of the spec), and if we parsed the EDID info we could present the relevant channel order to user-space apps.

pl bossart kirjoitti tiistai, 3. elokuuta 2010 23:15:28:
I'm not sure I see how that will help with the problem of this thread, though, i.e. NVIDIA MCP7A hdmi codec apparently assuming audio being fed in windows channel order instead of alsa channel order.

At Tue, 3 Aug 2010 23:32:45 +0300, Anssi Hannula wrote:
If we get the channel order, the channels could be remapped either by the app itself or an alsa-lib plugin. The channel-map API itself is just a bridge for that...
Takashi

At Tue, 3 Aug 2010 13:15:28 -0700, pl bossart wrote:
But if the hardware is capable of channel-remapping (the intel hdmi code actually does even now), it's better as it works without any software-side channel swaps. Ditto for analog surrounds.
Of course, setting the channel-map is optional. The first step is to provide the API to get the map.
Takashi
participants (4)
-
Anssi Hannula
-
pl bossart
-
Takashi Iwai
-
Wei Ni