[PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[David Henningsson]

[-- Attachment #1: Type: text/plain, Size: 282 bytes --]

Since "Front" controls headphones as well on these two codecs,
they should be labeled "Master Front" in order to be handled correct
by PulseAudio.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

[-- Attachment #2: 0002-ALSA-HDA-Rename-Front-to-Master-Front-on-ALC861VD-an.patch --]
[-- Type: text/x-patch, Size: 1159 bytes --]

>From daf075d2301e9d2105ab3a5a6517ad72dbba7c7f Mon Sep 17 00:00:00 2001
From: David Henningsson <david.henningsson@canonical.com>
Date: Tue, 16 Nov 2010 09:32:53 +0100
Subject: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

Since "Front" controls headphones as well on these two codecs,
they should be labeled "Master Front" in order to be handled correct
by PulseAudio.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>
---
 sound/pci/hda/patch_realtek.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index ce4017e..4cc7bb2 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -16966,7 +16966,7 @@ static void alc861vd_auto_init_analog_input(struct hda_codec *codec)
 static int alc861vd_auto_create_multi_out_ctls(struct alc_spec *spec,
 					     const struct auto_pin_cfg *cfg)
 {
-	static const char *chname[4] = {"Front", "Surround", "CLFE", "Side"};
+	static const char *chname[4] = {"Master Front", "Surround", "CLFE", "Side"};
 	hda_nid_t nid_v, nid_s;
 	int i, err;
 
-- 
1.7.1


[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[Takashi Iwai]

At Fri, 19 Nov 2010 10:29:37 +0100,
David Henningsson wrote:
> 
> Since "Front" controls headphones as well on these two codecs,
> they should be labeled "Master Front" in order to be handled correct
> by PulseAudio.

Well, "Front" doesn't mean the physical output but rather the output
channels.  So, it doesn't conflict with the fact that this control
actually changes all outputs, as long as the output jacks are of front
channels.  In that sense, even "Surround" is also a sort of Master
control.

That being said, I'm not happy with this rename.  If any, we should
try to get rid of unnecessary front volume control.  But, just because
PA gets confused doesn't sound like a right reason.

Could you give the alsa-info.sh output for checking what mixer elements
are created?


thanks,

Takashi


> 
> Signed-off-by: David Henningsson <david.henningsson@canonical.com>
> 
> -- 
> David Henningsson, Canonical Ltd.
> http://launchpad.net/~diwic

Re: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[David Henningsson]

On 2010-11-22 07:50, Takashi Iwai wrote:
> At Fri, 19 Nov 2010 10:29:37 +0100,
> David Henningsson wrote:
>>
>> Since "Front" controls headphones as well on these two codecs,
>> they should be labeled "Master Front" in order to be handled correct
>> by PulseAudio.
>
> Well, "Front" doesn't mean the physical output but rather the output
> channels.  So, it doesn't conflict with the fact that this control
> actually changes all outputs, as long as the output jacks are of front
> channels.  In that sense, even "Surround" is also a sort of Master
> control.
>
> That being said, I'm not happy with this rename.  If any, we should
> try to get rid of unnecessary front volume control.  But, just because
> PA gets confused doesn't sound like a right reason.
>
> Could you give the alsa-info.sh output for checking what mixer elements
> are created?

As you know PulseAudio needs consistent naming of volume controls, and 
this is not expressed clearly enough in the docs, so, what would you 
call a volume control that controls a front-line-out, but does not 
control a front-speaker or a front-headphone? PulseAudio assumes that if 
a control is named "Front", it does just that. So that's why I would 
prefer a rename to "Master Front".

As for alsa-info output, I believe anything in hda-emu/codecs/alc861vd-* 
would do, as they have nsteps=0 on NIDs 0x0c - 0x0f.

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

Re: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[Takashi Iwai]

At Mon, 22 Nov 2010 09:42:46 +0100,
David Henningsson wrote:
> 
> On 2010-11-22 07:50, Takashi Iwai wrote:
> > At Fri, 19 Nov 2010 10:29:37 +0100,
> > David Henningsson wrote:
> >>
> >> Since "Front" controls headphones as well on these two codecs,
> >> they should be labeled "Master Front" in order to be handled correct
> >> by PulseAudio.
> >
> > Well, "Front" doesn't mean the physical output but rather the output
> > channels.  So, it doesn't conflict with the fact that this control
> > actually changes all outputs, as long as the output jacks are of front
> > channels.  In that sense, even "Surround" is also a sort of Master
> > control.
> >
> > That being said, I'm not happy with this rename.  If any, we should
> > try to get rid of unnecessary front volume control.  But, just because
> > PA gets confused doesn't sound like a right reason.
> >
> > Could you give the alsa-info.sh output for checking what mixer elements
> > are created?
> 
> As you know PulseAudio needs consistent naming of volume controls, and 
> this is not expressed clearly enough in the docs, so, what would you 
> call a volume control that controls a front-line-out, but does not 
> control a front-speaker or a front-headphone?

What do you mean "front" in this context?  Is it the front 2 channels
out of 4.0 or 5.1 output?  Or do you mean the physical location of
the output?  Front/Surround/Center/LFE are the former case.  They are
controls independent from the physical output locations (per
definition).

I guess this confusion came up because of the mix-up with front
channel and front panel.  We've used "Front Mic" in some places, for
example, which increases the confusion.

> PulseAudio assumes that if 
> a control is named "Front", it does just that. So that's why I would 
> prefer a rename to "Master Front".
> 
> As for alsa-info output, I believe anything in hda-emu/codecs/alc861vd-* 
> would do, as they have nsteps=0 on NIDs 0x0c - 0x0f.

Yeah, the output with your first patch would be helpful.


thanks,

Takashi

Re: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[David Henningsson]

[-- Attachment #1: Type: text/plain, Size: 2965 bytes --]

On 2010-11-22 09:51, Takashi Iwai wrote:
> At Mon, 22 Nov 2010 09:42:46 +0100,
> David Henningsson wrote:
>>
>> On 2010-11-22 07:50, Takashi Iwai wrote:
>>> At Fri, 19 Nov 2010 10:29:37 +0100,
>>> David Henningsson wrote:
>>>>
>>>> Since "Front" controls headphones as well on these two codecs,
>>>> they should be labeled "Master Front" in order to be handled correct
>>>> by PulseAudio.
>>>
>>> Well, "Front" doesn't mean the physical output but rather the output
>>> channels.  So, it doesn't conflict with the fact that this control
>>> actually changes all outputs, as long as the output jacks are of front
>>> channels.  In that sense, even "Surround" is also a sort of Master
>>> control.
>>>
>>> That being said, I'm not happy with this rename.  If any, we should
>>> try to get rid of unnecessary front volume control.  But, just because
>>> PA gets confused doesn't sound like a right reason.
>>>
>>> Could you give the alsa-info.sh output for checking what mixer elements
>>> are created?
>>
>> As you know PulseAudio needs consistent naming of volume controls, and
>> this is not expressed clearly enough in the docs, so, what would you
>> call a volume control that controls a front-line-out, but does not
>> control a front-speaker or a front-headphone?
>
> What do you mean "front" in this context?  Is it the front 2 channels
> out of 4.0 or 5.1 output?  Or do you mean the physical location of
> the output?  Front/Surround/Center/LFE are the former case.  They are
> controls independent from the physical output locations (per
> definition).
>
> I guess this confusion came up because of the mix-up with front
> channel and front panel.  We've used "Front Mic" in some places, for
> example, which increases the confusion.

Good point. I think the reasoning so far is to refer to channels for 
output and location for input.

Anyway, what I meant was "Front" channel (not location), so let me 
rephrase my question into two:

1) If I find a volume control (or mute switch) labeled "Front", should I 
assume that it controls, or does not control, headphone output? How 
about internal speakers?

2) Assume I have a volume control (or mute switch) which controls only 
the front line-out, not headphones, and not speakers. How should I name it?

>> PulseAudio assumes that if
>> a control is named "Front", it does just that. So that's why I would
>> prefer a rename to "Master Front".
>>
>> As for alsa-info output, I believe anything in hda-emu/codecs/alc861vd-*
>> would do, as they have nsteps=0 on NIDs 0x0c - 0x0f.
>
> Yeah, the output with your first patch would be helpful.

Unfortunately I don't have access to the actual hardware at the moment, 
so I'm attaching a codec dump from the hardware before the patch. 
Running it against current HEAD, with today's patch from Kailang in, 
probably makes hda-emu take the wrong code path (since hda-emu can't 
read proc coefs).

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

[-- Attachment #2: a887.txt --]
[-- Type: text/plain, Size: 15302 bytes --]

Codec: Realtek ALC887
Address: 0
Function Id: 0x1
Vendor Id: 0x10ec0887
Subsystem Id: 0x102804aa
Revision Id: 0x100302
No Modem Function Group found
Default PCM:
    rates [0x5f0]: 32000 44100 48000 88200 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
Default Amp-In caps: N/A
Default Amp-Out caps: N/A
GPIO: io=2, o=0, i=0, unsolicited=1, wake=0
  IO[0]: enable=0, dir=0, wake=0, sticky=0, data=0, unsol=0
  IO[1]: enable=0, dir=0, wake=0, sticky=0, data=0, unsol=0
Node 0x02 [Audio Output] wcaps 0x41d: Stereo Amp-Out
  Device: name="ALC887 Analog", type="Audio", device=0
  Amp-Out caps: ofs=0x40, nsteps=0x40, stepsize=0x03, mute=0
  Amp-Out vals:  [0x40 0x40]
  Converter: stream=0, channel=0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x03 [Audio Output] wcaps 0x41d: Stereo Amp-Out
  Amp-Out caps: ofs=0x40, nsteps=0x40, stepsize=0x03, mute=0
  Amp-Out vals:  [0x40 0x40]
  Converter: stream=0, channel=0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x04 [Audio Output] wcaps 0x41d: Stereo Amp-Out
  Amp-Out caps: ofs=0x40, nsteps=0x40, stepsize=0x03, mute=0
  Amp-Out vals:  [0x40 0x40]
  Converter: stream=0, channel=0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x05 [Audio Output] wcaps 0x41d: Stereo Amp-Out
  Amp-Out caps: ofs=0x40, nsteps=0x40, stepsize=0x03, mute=0
  Amp-Out vals:  [0x40 0x40]
  Converter: stream=0, channel=0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x06 [Audio Output] wcaps 0x611: Stereo Digital
  Control: name="IEC958 Playback Con Mask", index=0, device=0
  Control: name="IEC958 Playback Pro Mask", index=0, device=0
  Control: name="IEC958 Playback Default", index=0, device=0
  Control: name="IEC958 Playback Switch", index=0, device=0
  Control: name="IEC958 Default PCM Playback Switch", index=0, device=0
  Device: name="ALC887 Digital", type="SPDIF", device=1
  Converter: stream=0, channel=0
  Digital: Enabled
  Digital category: 0x0
  PCM:
    rates [0x5f0]: 32000 44100 48000 88200 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x07 [Vendor Defined Widget] wcaps 0xf00000: Mono
Node 0x08 [Audio Input] wcaps 0x10051b: Stereo Amp-In
  Control: name="Capture Switch", index=0, device=0
  Control: name="Capture Volume", index=0, device=0
  Device: name="ALC887 Analog", type="Audio", device=0
  Amp-In caps: ofs=0x10, nsteps=0x2e, stepsize=0x03, mute=1
  Amp-In vals:  [0x0a 0x0a]
  Converter: stream=0, channel=0
  SDI-Select: 0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x23
Node 0x09 [Audio Input] wcaps 0x10051b: Stereo Amp-In
  Control: name="Capture Switch", index=1, device=0
  Control: name="Capture Volume", index=1, device=0
  Device: name="ALC887 Analog", type="Audio", device=2
  Amp-In caps: ofs=0x10, nsteps=0x2e, stepsize=0x03, mute=1
  Amp-In vals:  [0x00 0x00]
  Converter: stream=0, channel=0
  SDI-Select: 0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x22
Node 0x0a [Audio Input] wcaps 0x100711: Stereo Digital
  Converter: stream=0, channel=0
  SDI-Select: 0
  Digital:
  Digital category: 0x0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x1f
Node 0x0b [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Control: name="Mic Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=0, ofs=0
  Control: name="Mic Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=0, ofs=0
  Control: name="Front Mic Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=1, ofs=0
  Control: name="Front Mic Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=1, ofs=0
  Control: name="Line Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=2, ofs=0
  Control: name="Line Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=2, ofs=0
  Amp-In caps: ofs=0x17, nsteps=0x1f, stepsize=0x05, mute=1
  Amp-In vals:  [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80]
  Connection: 10
     0x18 0x19 0x1a 0x1b 0x1c 0x1d 0x14 0x15 0x16 0x17
Node 0x0c [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Control: name="Front Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Control: name="Front Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=2, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x80 0x80] [0x80 0x80]
  Connection: 2
     0x02 0x0b
Node 0x0d [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Control: name="Surround Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Control: name="Surround Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=2, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x80 0x80] [0x80 0x80]
  Connection: 2
     0x03 0x0b
Node 0x0e [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Control: name="Center Playback Volume", index=0, device=0
    ControlAmp: chs=1, dir=Out, idx=0, ofs=0
  Control: name="LFE Playback Volume", index=0, device=0
    ControlAmp: chs=2, dir=Out, idx=0, ofs=0
  Control: name="Center Playback Switch", index=0, device=0
    ControlAmp: chs=1, dir=In, idx=2, ofs=0
  Control: name="LFE Playback Switch", index=0, device=0
    ControlAmp: chs=2, dir=In, idx=2, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x80 0x80] [0x80 0x80]
  Connection: 2
     0x04 0x0b
Node 0x0f [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Control: name="Side Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Control: name="Side Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=2, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x80 0x80] [0x80 0x80]
  Connection: 2
     0x05 0x0b
Node 0x10 [Audio Output] wcaps 0x611: Stereo Digital
  Converter: stream=0, channel=0
  Digital:
  Digital category: 0x0
  PCM:
    rates [0x5f0]: 32000 44100 48000 88200 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x11 [Pin Complex] wcaps 0x400781: Stereo Digital
  Pincap 0x00000014: OUT Detect
  Pin Default 0x411111f0: [N/A] Speaker at Ext Rear
    Conn = 1/8, Color = Black
    DefAssociation = 0xf, Sequence = 0x0
    Misc = NO_PRESENCE
  Pin-ctls: 0x40: OUT
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x10
Node 0x12 [Pin Complex] wcaps 0x400401: Stereo
  Pincap 0x00000020: IN
  Pin Default 0x411111f0: [N/A] Speaker at Ext Rear
    Conn = 1/8, Color = Black
    DefAssociation = 0xf, Sequence = 0x0
    Misc = NO_PRESENCE
  Pin-ctls: 0x00:
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x13 [Vendor Defined Widget] wcaps 0xf00000: Mono
Node 0x14 [Pin Complex] wcaps 0x40058d: Stereo Amp-Out
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x00 0x00]
  Pincap 0x0001003e: IN OUT HP EAPD Detect Trigger
  EAPD 0x0:
  Pin Default 0x01014010: [Jack] Line Out at Ext Rear
    Conn = 1/8, Color = Green
    DefAssociation = 0x1, Sequence = 0x0
  Pin-ctls: 0x40: OUT
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x0c
Node 0x15 [Pin Complex] wcaps 0x40058d: Stereo Amp-Out
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x00 0x00]
  Pincap 0x00000036: IN OUT Detect Trigger
  Pin Default 0x01011012: [Jack] Line Out at Ext Rear
    Conn = 1/8, Color = Black
    DefAssociation = 0x1, Sequence = 0x2
  Pin-ctls: 0x40: OUT
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x0d
Node 0x16 [Pin Complex] wcaps 0x40058d: Stereo Amp-Out
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x00 0x00]
  Pincap 0x00000036: IN OUT Detect Trigger
  Pin Default 0x01016011: [Jack] Line Out at Ext Rear
    Conn = 1/8, Color = Orange
    DefAssociation = 0x1, Sequence = 0x1
  Pin-ctls: 0x40: OUT
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x0e
Node 0x17 [Pin Complex] wcaps 0x40058d: Stereo Amp-Out
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x00 0x00]
  Pincap 0x00000036: IN OUT Detect Trigger
  Pin Default 0x01012014: [Jack] Line Out at Ext Rear
    Conn = 1/8, Color = Grey
    DefAssociation = 0x1, Sequence = 0x4
  Pin-ctls: 0x40: OUT
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x0f
Node 0x18 [Pin Complex] wcaps 0x40058f: Stereo Amp-In Amp-Out
  Control: name="Mic Boost", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=0, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x80 0x80]
  Pincap 0x00003736: IN OUT Detect Trigger
    Vref caps: HIZ 50 GRD 80 100
  Pin Default 0x01a19830: [Jack] Mic at Ext Rear
    Conn = 1/8, Color = Pink
    DefAssociation = 0x3, Sequence = 0x0
  Pin-ctls: 0x24: IN VREF_80
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 5
     0x0c* 0x0d 0x0e 0x0f 0x26
Node 0x19 [Pin Complex] wcaps 0x40058f: Stereo Amp-In Amp-Out
  Control: name="Front Mic Boost", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=0, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x80 0x80]
  Pincap 0x0000373e: IN OUT HP Detect Trigger
    Vref caps: HIZ 50 GRD 80 100
  Pin Default 0x02a19831: [Jack] Mic at Ext Front
    Conn = 1/8, Color = Pink
    DefAssociation = 0x3, Sequence = 0x1
  Pin-ctls: 0x24: IN VREF_80
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 5
     0x0c* 0x0d 0x0e 0x0f 0x26
Node 0x1a [Pin Complex] wcaps 0x40058f: Stereo Amp-In Amp-Out
  Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x80 0x80]
  Pincap 0x00003736: IN OUT Detect Trigger
    Vref caps: HIZ 50 GRD 80 100
  Pin Default 0x0181303f: [Jack] Line In at Ext Rear
    Conn = 1/8, Color = Blue
    DefAssociation = 0x3, Sequence = 0xf
  Pin-ctls: 0x20: IN VREF_HIZ
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 5
     0x0c* 0x0d 0x0e 0x0f 0x26
Node 0x1b [Pin Complex] wcaps 0x40058f: Stereo Amp-In Amp-Out
  Control: name="Headphone Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x80 0x80]
  Pincap 0x0001373e: IN OUT HP EAPD Detect Trigger
    Vref caps: HIZ 50 GRD 80 100
  EAPD 0x0:
  Pin Default 0x0221401f: [Jack] HP Out at Ext Front
    Conn = 1/8, Color = Green
    DefAssociation = 0x1, Sequence = 0xf
  Pin-ctls: 0xc0: OUT HP VREF_HIZ
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 5
     0x0c* 0x0d 0x0e 0x0f 0x26
Node 0x1c [Pin Complex] wcaps 0x400481: Stereo
  Pincap 0x00000024: IN Detect
  Pin Default 0x411111f0: [N/A] Speaker at Ext Rear
    Conn = 1/8, Color = Black
    DefAssociation = 0xf, Sequence = 0x0
    Misc = NO_PRESENCE
  Pin-ctls: 0x20: IN
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x1d [Pin Complex] wcaps 0x400400: Mono
  Pincap 0x00000020: IN
  Pin Default 0x4005e601: [N/A] Line Out at Ext N/A
    Conn = Optical, Color = White
    DefAssociation = 0x0, Sequence = 0x1
  Pin-ctls: 0x20: IN
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x1e [Pin Complex] wcaps 0x400781: Stereo Digital
  Pincap 0x00000014: OUT Detect
  Pin Default 0x01451120: [Jack] SPDIF Out at Ext Rear
    Conn = Optical, Color = Black
    DefAssociation = 0x2, Sequence = 0x0
    Misc = NO_PRESENCE
  Pin-ctls: 0x40: OUT
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 1
     0x06
Node 0x1f [Pin Complex] wcaps 0x400681: Stereo Digital
  Pincap 0x00000024: IN Detect
  Pin Default 0x411111f0: [N/A] Speaker at Ext Rear
    Conn = 1/8, Color = Black
    DefAssociation = 0xf, Sequence = 0x0
    Misc = NO_PRESENCE
  Pin-ctls: 0x20: IN
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x20 [Vendor Defined Widget] wcaps 0xf00040: Mono
  Processing caps: benign=0, ncoeff=24
Node 0x21 [Vendor Defined Widget] wcaps 0xf00000: Mono
Node 0x22 [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Control: name="Input Source", index=1, device=0
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x00 0x00] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80]
  Connection: 12
     0x18 0x19 0x1a 0x1b 0x1c 0x1d 0x14 0x15 0x16 0x17 0x0b 0x12
Node 0x23 [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Control: name="Input Source", index=0, device=0
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x80 0x80] [0x00 0x00] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80] [0x80 0x80]
  Connection: 11
     0x18 0x19 0x1a 0x1b 0x1c 0x1d 0x14 0x15 0x16 0x17 0x0b
Node 0x24 [Vendor Defined Widget] wcaps 0xf00000: Mono
Node 0x25 [Audio Output] wcaps 0x41d: Stereo Amp-Out
  Amp-Out caps: ofs=0x40, nsteps=0x40, stepsize=0x03, mute=0
  Amp-Out vals:  [0x40 0x40]
  Converter: stream=0, channel=0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x26 [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x00 0x00] [0x00 0x00]
  Connection: 2
     0x25 0x0b


[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[Raymond Yau]

2010/11/22 David Henningsson <david.henningsson@canonical.com>

> Unfortunately I don't have access to the actual hardware at the moment, so
> I'm attaching a codec dump from the hardware before the patch. Running it
> against current HEAD, with today's patch from Kailang in, probably makes
> hda-emu take the wrong code path (since hda-emu can't read proc coefs).
>
>
>
How about the volume control at node 0x25 ?

Is it "Independent headphone" for multi stream playback when 0x1b connected
to 0x26 ?

alc887 has 5 [Audio Output], 0x02, 0x3, 0x04 ,0x05 and 0x25 and they are
connected to 0x0c, 0x0d , 0x0e , 0x0f and 0x26 [audio mixer] mute switches


Node 0x25 [Audio Output] wcaps 0x41d: Stereo Amp-Out
  Amp-Out caps: ofs=0x40, nsteps=0x40, stepsize=0x03, mute=0
  Amp-Out vals:  [0x40 0x40]
  Converter: stream=0, channel=0
  PCM:
    rates [0x560]: 44100 48000 96000 192000
    bits [0xe]: 16 20 24
    formats [0x1]: PCM
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
Node 0x26 [Audio Mixer] wcaps 0x20010b: Stereo Amp-In
  Amp-In caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-In vals:  [0x00 0x00] [0x00 0x00]
  Connection: 2
     0x25 0x0b


Node 0x1b [Pin Complex] wcaps 0x40058f: Stereo Amp-In Amp-Out
  Control: name="Headphone Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Amp-Out caps: ofs=0x00, nsteps=0x00, stepsize=0x00, mute=1
  Amp-Out vals:  [0x80 0x80]
  Pincap 0x0001373e: IN OUT HP EAPD Detect Trigger
    Vref caps: HIZ 50 GRD 80 100
  EAPD 0x0:
  Pin Default 0x0221401f: [Jack] HP Out at Ext Front
    Conn = 1/8, Color = Green
    DefAssociation = 0x1, Sequence = 0xf
  Pin-ctls: 0xc0: OUT HP VREF_HIZ
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D1 D2 D3 EPSS
  Power: setting=D0, actual=D0
  Connection: 5
     0x0c* 0x0d 0x0e 0x0f 0x26

Re: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[Takashi Iwai]

At Mon, 22 Nov 2010 15:59:58 +0100,
David Henningsson wrote:
> 
> On 2010-11-22 09:51, Takashi Iwai wrote:
> > At Mon, 22 Nov 2010 09:42:46 +0100,
> > David Henningsson wrote:
> >>
> >> On 2010-11-22 07:50, Takashi Iwai wrote:
> >>> At Fri, 19 Nov 2010 10:29:37 +0100,
> >>> David Henningsson wrote:
> >>>>
> >>>> Since "Front" controls headphones as well on these two codecs,
> >>>> they should be labeled "Master Front" in order to be handled correct
> >>>> by PulseAudio.
> >>>
> >>> Well, "Front" doesn't mean the physical output but rather the output
> >>> channels.  So, it doesn't conflict with the fact that this control
> >>> actually changes all outputs, as long as the output jacks are of front
> >>> channels.  In that sense, even "Surround" is also a sort of Master
> >>> control.
> >>>
> >>> That being said, I'm not happy with this rename.  If any, we should
> >>> try to get rid of unnecessary front volume control.  But, just because
> >>> PA gets confused doesn't sound like a right reason.
> >>>
> >>> Could you give the alsa-info.sh output for checking what mixer elements
> >>> are created?
> >>
> >> As you know PulseAudio needs consistent naming of volume controls, and
> >> this is not expressed clearly enough in the docs, so, what would you
> >> call a volume control that controls a front-line-out, but does not
> >> control a front-speaker or a front-headphone?
> >
> > What do you mean "front" in this context?  Is it the front 2 channels
> > out of 4.0 or 5.1 output?  Or do you mean the physical location of
> > the output?  Front/Surround/Center/LFE are the former case.  They are
> > controls independent from the physical output locations (per
> > definition).
> >
> > I guess this confusion came up because of the mix-up with front
> > channel and front panel.  We've used "Front Mic" in some places, for
> > example, which increases the confusion.
> 
> Good point. I think the reasoning so far is to refer to channels for 
> output and location for input.
> 
> Anyway, what I meant was "Front" channel (not location), so let me 
> rephrase my question into two:
> 
> 1) If I find a volume control (or mute switch) labeled "Front", should I 
> assume that it controls, or does not control, headphone output? How 
> about internal speakers?

Yes, this should be applied to all outputs in general.
(There might be some misuses, though...)

> 2) Assume I have a volume control (or mute switch) which controls only 
> the front line-out, not headphones, and not speakers. How should I name it?

A good question.  We should have a unified prefix for the location.
IIRC, "Front Line-Out" or "Front XXX" are used indeed, so far.

> >> PulseAudio assumes that if
> >> a control is named "Front", it does just that. So that's why I would
> >> prefer a rename to "Master Front".
> >>
> >> As for alsa-info output, I believe anything in hda-emu/codecs/alc861vd-*
> >> would do, as they have nsteps=0 on NIDs 0x0c - 0x0f.
> >
> > Yeah, the output with your first patch would be helpful.
> 
> Unfortunately I don't have access to the actual hardware at the moment, 
> so I'm attaching a codec dump from the hardware before the patch. 
> Running it against current HEAD, with today's patch from Kailang in, 
> probably makes hda-emu take the wrong code path (since hda-emu can't 
> read proc coefs).

Indeed, something is still fishy there.
OK, we need to fix it again...


thanks,

Takashi

Re: [PATCH 2/3] ALSA: HDA: Rename "Front" to "Master Front" on ALC861VD and ALC887

[Takashi Iwai]

At Tue, 23 Nov 2010 09:05:09 +0100,
Takashi Iwai wrote:
> 
> At Mon, 22 Nov 2010 15:59:58 +0100,
> David Henningsson wrote:
> > 
> > On 2010-11-22 09:51, Takashi Iwai wrote:
> > > At Mon, 22 Nov 2010 09:42:46 +0100,
> > > David Henningsson wrote:
> > >>
> > >> On 2010-11-22 07:50, Takashi Iwai wrote:
> > >>> At Fri, 19 Nov 2010 10:29:37 +0100,
> > >>> David Henningsson wrote:
> > >>>>
> > >>>> Since "Front" controls headphones as well on these two codecs,
> > >>>> they should be labeled "Master Front" in order to be handled correct
> > >>>> by PulseAudio.
> > >>>
> > >>> Well, "Front" doesn't mean the physical output but rather the output
> > >>> channels.  So, it doesn't conflict with the fact that this control
> > >>> actually changes all outputs, as long as the output jacks are of front
> > >>> channels.  In that sense, even "Surround" is also a sort of Master
> > >>> control.
> > >>>
> > >>> That being said, I'm not happy with this rename.  If any, we should
> > >>> try to get rid of unnecessary front volume control.  But, just because
> > >>> PA gets confused doesn't sound like a right reason.
> > >>>
> > >>> Could you give the alsa-info.sh output for checking what mixer elements
> > >>> are created?
> > >>
> > >> As you know PulseAudio needs consistent naming of volume controls, and
> > >> this is not expressed clearly enough in the docs, so, what would you
> > >> call a volume control that controls a front-line-out, but does not
> > >> control a front-speaker or a front-headphone?
> > >
> > > What do you mean "front" in this context?  Is it the front 2 channels
> > > out of 4.0 or 5.1 output?  Or do you mean the physical location of
> > > the output?  Front/Surround/Center/LFE are the former case.  They are
> > > controls independent from the physical output locations (per
> > > definition).
> > >
> > > I guess this confusion came up because of the mix-up with front
> > > channel and front panel.  We've used "Front Mic" in some places, for
> > > example, which increases the confusion.
> > 
> > Good point. I think the reasoning so far is to refer to channels for 
> > output and location for input.
> > 
> > Anyway, what I meant was "Front" channel (not location), so let me 
> > rephrase my question into two:
> > 
> > 1) If I find a volume control (or mute switch) labeled "Front", should I 
> > assume that it controls, or does not control, headphone output? How 
> > about internal speakers?
> 
> Yes, this should be applied to all outputs in general.
> (There might be some misuses, though...)
> 
> > 2) Assume I have a volume control (or mute switch) which controls only 
> > the front line-out, not headphones, and not speakers. How should I name it?
> 
> A good question.  We should have a unified prefix for the location.
> IIRC, "Front Line-Out" or "Front XXX" are used indeed, so far.
> 
> > >> PulseAudio assumes that if
> > >> a control is named "Front", it does just that. So that's why I would
> > >> prefer a rename to "Master Front".
> > >>
> > >> As for alsa-info output, I believe anything in hda-emu/codecs/alc861vd-*
> > >> would do, as they have nsteps=0 on NIDs 0x0c - 0x0f.
> > >
> > > Yeah, the output with your first patch would be helpful.
> > 
> > Unfortunately I don't have access to the actual hardware at the moment, 
> > so I'm attaching a codec dump from the hardware before the patch. 
> > Running it against current HEAD, with today's patch from Kailang in, 
> > probably makes hda-emu take the wrong code path (since hda-emu can't 
> > read proc coefs).
> 
> Indeed, something is still fishy there.
> OK, we need to fix it again...

I guess it's because the driver checks the COEF to identify the chip
model, but hda-emu doesn't provide it properly.


Takashi