Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Anssi Hannula <anssi.hannula@iki.fi>
To: alsa-devel@alsa-project.org
Cc: "Takashi Iwai" <tiwai@suse.de>,
	"Rafał Miłecki" <zajec5@gmail.com>,
	"Peter Frühberger" <fritsch@xbmc.org>
Subject: Re: [RFC/RFT PATCH] ALSA: hda - hdmi: Add ATI/AMD multi-channel audio support
Date: Mon, 23 Sep 2013 00:14:11 +0300	[thread overview]
Message-ID: <523F5DA3.6070103@iki.fi> (raw)
In-Reply-To: <1379872233-32210-1-git-send-email-anssi.hannula@iki.fi>

Just a small update on a few things.

22.09.2013 20:50, Anssi Hannula kirjoitti:
[...]
> Since there is no ELD support (unless out-of-band communication with the
> graphics driver is added so that they can provide it to us directly),
> ELD is currently emulated with only the basic information populated
> (speaker mask, SADs, connection type). There are some bits missing on
> the radeon driver side (the SADs etc are not populated) so currently
> the ELD emulation only works on fglrx. ELD emulation is not needed for
> multi-channel playback.

Actually, it seems this support is already there in radeon open-source
driver. It was added in 3.11 for some chips and 3.12 for others. I
haven't tested yet, though.

[...]
> HBR mode (needed for DTS-HD/TrueHD passthrough) is not working yet, and
> the released documentation does not mention it.

I'm hearing we may get HBR docs soon. If so, I'll update the patch
accordingly at that time.

[...]
> +				/* in paired mode we need to take into account
> +				 * the occupied channel pairs instead of just the
> +				 * channel count */
> +				for (c = 0; c < 8; c++) {
> +					if (cap->speakers[c] || cap->speakers[c+1])
> +						chanpairs++;
> +				}

s/c++/c += 2/

That part is still untested though.

-- 
Anssi Hannula

  reply	other threads:[~2013-09-22 21:14 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-22 17:50 [RFC/RFT PATCH] ALSA: hda - hdmi: Add ATI/AMD multi-channel audio support Anssi Hannula
2013-09-22 21:14 ` Anssi Hannula [this message]
2013-09-25  4:22 ` Olivier Langlois
2013-09-25 14:10   ` Anssi Hannula
2013-09-25 15:17     ` LANGLOIS Olivier PIS -EXT
2013-09-25 22:21       ` Anssi Hannula
2013-09-26  5:54         ` Olivier Langlois
2013-09-26  9:59 ` Takashi Iwai
2013-09-27  9:27 ` Rafał Miłecki
2013-09-27 10:19 ` Rafał Miłecki
2013-09-27 13:10   ` Anssi Hannula

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=523F5DA3.6070103@iki.fi \
    --to=anssi.hannula@iki.fi \
    --cc=alsa-devel@alsa-project.org \
    --cc=fritsch@xbmc.org \
    --cc=tiwai@suse.de \
    --cc=zajec5@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox