From: Fengguang Wu <fengguang.wu@intel.com>
To: Wang Xingchao <xingchao.wang@intel.com>
Cc: tiwai@suse.de, anssi.hannula@iki.fi, alsa-devel@alsa-project.org,
alanwww1@xbmc.org
Subject: Re: [PATCH] HDA-Intel: Fix wrong CTLINDEX entries for HBR Passthrough
Date: Mon, 10 Sep 2012 11:46:01 +0800 [thread overview]
Message-ID: <20120910034601.GA31239@localhost> (raw)
In-Reply-To: <1346898800-13951-1-git-send-email-xingchao.wang@intel.com>
On Thu, Sep 06, 2012 at 10:33:20AM +0800, Wang Xingchao wrote:
> >From Anssi Hannula:
> "The CTLINDEX=x values in /usr/share/alsa/cards/HDA-Intel.conf are off
> when you have S/PDIF output in the same card as HDMI, which is often the case
> on Intel.
>
> The workaround is to bump all the CTLINDEX=x values one step up in the
> HDA-Intel.pcm.hdmi.foo entries, i.e.
> CTLINDEX=0 => CTLINDEX=1
> CTLINDEX=1 => CTLINDEX=2
> etc, in the /usr/share/alsa/cards/HDA-Intel.conf."
Hmm.. my T420 does not have SPDIF and could possibly be broken by this
change.
wfg ~% amixer controls
numid=2,iface=MIXER,name='Master Playback Switch'
numid=1,iface=MIXER,name='Master Playback Volume'
numid=23,iface=MIXER,name='PCM Playback Volume'
numid=5,iface=MIXER,name='Capture Switch'
numid=4,iface=MIXER,name='Capture Volume'
numid=8,iface=MIXER,name='IEC958 Playback Con Mask'
numid=13,iface=MIXER,name='IEC958 Playback Con Mask',index=1
numid=18,iface=MIXER,name='IEC958 Playback Con Mask',index=2
numid=9,iface=MIXER,name='IEC958 Playback Pro Mask'
numid=14,iface=MIXER,name='IEC958 Playback Pro Mask',index=1
numid=19,iface=MIXER,name='IEC958 Playback Pro Mask',index=2
==> numid=10,iface=MIXER,name='IEC958 Playback Default'
==> numid=15,iface=MIXER,name='IEC958 Playback Default',index=1
==> numid=20,iface=MIXER,name='IEC958 Playback Default',index=2
numid=11,iface=MIXER,name='IEC958 Playback Switch'
numid=16,iface=MIXER,name='IEC958 Playback Switch',index=1
numid=21,iface=MIXER,name='IEC958 Playback Switch',index=2
numid=3,iface=MIXER,name='Analog Mic Boost Capture Enum'
numid=7,iface=MIXER,name='Beep Playback Switch'
numid=6,iface=MIXER,name='Beep Playback Volume'
numid=24,iface=MIXER,name='Digital Capture Volume'
numid=12,iface=PCM,name='ELD',device=3
numid=17,iface=PCM,name='ELD',device=7
numid=22,iface=PCM,name='ELD',device=8
Thanks,
Fengguang
> Signed-off-by: Wang Xingchao <xingchao.wang@intel.com>
> ---
> src/conf/cards/HDA-Intel.conf | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/conf/cards/HDA-Intel.conf b/src/conf/cards/HDA-Intel.conf
> index d4f2667..7d6de8f 100644
> --- a/src/conf/cards/HDA-Intel.conf
> +++ b/src/conf/cards/HDA-Intel.conf
> @@ -211,7 +211,7 @@ HDA-Intel.pcm.hdmi.0 {
> "cards.HDA-Intel.pcm.hdmi.common:"
> "CARD=" $CARD ","
> "DEVICE=3,"
> - "CTLINDEX=0,"
> + "CTLINDEX=1,"
> "AES0=" $AES0 ","
> "AES1=" $AES1 ","
> "AES2=" $AES2 ","
> @@ -234,7 +234,7 @@ HDA-Intel.pcm.hdmi.1 {
> "cards.HDA-Intel.pcm.hdmi.common:"
> "CARD=" $CARD ","
> "DEVICE=7,"
> - "CTLINDEX=1,"
> + "CTLINDEX=2,"
> "AES0=" $AES0 ","
> "AES1=" $AES1 ","
> "AES2=" $AES2 ","
> @@ -257,7 +257,7 @@ HDA-Intel.pcm.hdmi.2 {
> "cards.HDA-Intel.pcm.hdmi.common:"
> "CARD=" $CARD ","
> "DEVICE=8,"
> - "CTLINDEX=2,"
> + "CTLINDEX=3,"
> "AES0=" $AES0 ","
> "AES1=" $AES1 ","
> "AES2=" $AES2 ","
> @@ -280,7 +280,7 @@ HDA-Intel.pcm.hdmi.3 {
> "cards.HDA-Intel.pcm.hdmi.common:"
> "CARD=" $CARD ","
> "DEVICE=9,"
> - "CTLINDEX=3,"
> + "CTLINDEX=4,"
> "AES0=" $AES0 ","
> "AES1=" $AES1 ","
> "AES2=" $AES2 ","
> --
> 1.7.9.5
prev parent reply other threads:[~2012-09-10 3:46 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-09-06 2:33 [PATCH] HDA-Intel: Fix wrong CTLINDEX entries for HBR Passthrough Wang Xingchao
2012-09-06 6:59 ` Takashi Iwai
2012-09-10 3:46 ` Fengguang Wu [this message]
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=20120910034601.GA31239@localhost \
--to=fengguang.wu@intel.com \
--cc=alanwww1@xbmc.org \
--cc=alsa-devel@alsa-project.org \
--cc=anssi.hannula@iki.fi \
--cc=tiwai@suse.de \
--cc=xingchao.wang@intel.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;
as well as URLs for NNTP newsgroup(s).