linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* 3.8-rc1 patch_cirrus 4.0 regression...
@ 2013-02-05 16:10 Daniel J Blueman
  2013-02-05 16:16 ` Takashi Iwai
  0 siblings, 1 reply; 9+ messages in thread
From: Daniel J Blueman @ 2013-02-05 16:10 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: Linux Kernel, alsa-devel

Hi Takashi,

The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
arrangement on my Macbook Pro 10,1 - only the higher-frequency
speakers work despite the front and rear channels being exposed in the
mixer.

Reverting f37bc7 [1] restored the correct behaviour ([2] was reverted
to compile, but the problem still occurs with only [2] reverted).

How can I help to debug this and find the right approach?

Thanks,
  Daniel

--- [1]

commit f37bc7a88d374448a1f4bba9267d308606d78bf2
Author: Takashi Iwai <tiwai@suse.de>
Date:   Thu Nov 8 15:59:23 2012 +0100

    ALSA: hda - Give standard "Bass Speaker" mixer for 2.1 speakers

    When two built-in speakers are found on the machine, we can suppose
    it's rather a 2.1 speaker system with a bass output instead of
    front/surround channels.

    Signed-off-by: Takashi Iwai <tiwai@suse.de>

--- [2]

commit ee81abb623cb5e03c182d16871bb4fb34fdc9b4f
Author: Takashi Iwai <tiwai@suse.de>
Date:   Thu Nov 8 17:12:10 2012 +0100

    ALSA: hda - Apply a proper chmap for built-in 2.1 speakers

    When 2.1 speakers are detected, use the corresponding channel map
    instead of the standard map with front+rear surrounds.

    Signed-off-by: Takashi Iwai <tiwai@suse.de>
-- 
Daniel J Blueman

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-05 16:10 3.8-rc1 patch_cirrus 4.0 regression Daniel J Blueman
@ 2013-02-05 16:16 ` Takashi Iwai
  2013-02-05 16:29   ` Daniel J Blueman
  2013-02-05 17:34   ` Takashi Iwai
  0 siblings, 2 replies; 9+ messages in thread
From: Takashi Iwai @ 2013-02-05 16:16 UTC (permalink / raw)
  To: Daniel J Blueman; +Cc: Linux Kernel, alsa-devel

At Wed, 6 Feb 2013 00:10:30 +0800,
Daniel J Blueman wrote:
> 
> Hi Takashi,
> 
> The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
> arrangement on my Macbook Pro 10,1 - only the higher-frequency
> speakers work despite the front and rear channels being exposed in the
> mixer.

Interesting.  So you have a machine with 4.0 built-in speaker instead
of 2.1?  Then we need to add a device-specific flag for it.  Currently
the driver assumes 2.1 system blindly because majority of machines
have that.

FWIW, the codec parser code has been totally rewritten for 3.9, so any
patch to 3.8 won't be applied to 3.9 (and vice versa)...

Could you give alsa-info.sh output on 3.8 kernel?  Then I'll try to
cook it for 3.9 (and maybe backport to 3.8).


Takashi

> 
> Reverting f37bc7 [1] restored the correct behaviour ([2] was reverted
> to compile, but the problem still occurs with only [2] reverted).
> 
> How can I help to debug this and find the right approach?
> 
> Thanks,
>   Daniel
> 
> --- [1]
> 
> commit f37bc7a88d374448a1f4bba9267d308606d78bf2
> Author: Takashi Iwai <tiwai@suse.de>
> Date:   Thu Nov 8 15:59:23 2012 +0100
> 
>     ALSA: hda - Give standard "Bass Speaker" mixer for 2.1 speakers
> 
>     When two built-in speakers are found on the machine, we can suppose
>     it's rather a 2.1 speaker system with a bass output instead of
>     front/surround channels.
> 
>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> 
> --- [2]
> 
> commit ee81abb623cb5e03c182d16871bb4fb34fdc9b4f
> Author: Takashi Iwai <tiwai@suse.de>
> Date:   Thu Nov 8 17:12:10 2012 +0100
> 
>     ALSA: hda - Apply a proper chmap for built-in 2.1 speakers
> 
>     When 2.1 speakers are detected, use the corresponding channel map
>     instead of the standard map with front+rear surrounds.
> 
>     Signed-off-by: Takashi Iwai <tiwai@suse.de>
> -- 
> Daniel J Blueman
> 

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-05 16:16 ` Takashi Iwai
@ 2013-02-05 16:29   ` Daniel J Blueman
  2013-02-05 16:34     ` Takashi Iwai
  2013-02-05 17:34   ` Takashi Iwai
  1 sibling, 1 reply; 9+ messages in thread
From: Daniel J Blueman @ 2013-02-05 16:29 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: Linux Kernel, alsa-devel

On 6 February 2013 00:16, Takashi Iwai <tiwai@suse.de> wrote:
> At Wed, 6 Feb 2013 00:10:30 +0800,
> Daniel J Blueman wrote:
>>
>> Hi Takashi,
>>
>> The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
>> arrangement on my Macbook Pro 10,1 - only the higher-frequency
>> speakers work despite the front and rear channels being exposed in the
>> mixer.
>
> Interesting.  So you have a machine with 4.0 built-in speaker instead
> of 2.1?  Then we need to add a device-specific flag for it.  Currently
> the driver assumes 2.1 system blindly because majority of machines
> have that.
>
> FWIW, the codec parser code has been totally rewritten for 3.9, so any
> patch to 3.8 won't be applied to 3.9 (and vice versa)...
>
> Could you give alsa-info.sh output on 3.8 kernel?  Then I'll try to
> cook it for 3.9 (and maybe backport to 3.8).

Here's the output from the current alsa-info.sh on 3.8-rc6 with the
two cited patches reverted [1]; let me know if you'd like 3.8-rc6
pure.

The sound is pretty impressive for a laptop when the low-frequecy
speakers are enabled.

Thanks, Takashi!
  Daniel

--- [1]

upload=true&script=true&cardinfo=
!!################################
!!ALSA Information Script v 0.4.61
!!################################

!!Script ran on: Tue Feb  5 16:22:01 UTC 2013


!!Linux Distribution
!!------------------

Ubuntu 12.10 \n \l DISTRIB_ID=Ubuntu DISTRIB_DESCRIPTION="Ubuntu
12.10" NAME="Ubuntu" ID=ubuntu ID_LIKE=debian PRETTY_NAME="Ubuntu
quantal (12.10)"


!!DMI Information
!!---------------

Manufacturer:      Apple Inc.
Product Name:      MacBookPro10,1
Product Version:   1.0
Firmware Version:  MBP101.88Z.00EE.B02.1208081132


!!Kernel Information
!!------------------

Kernel release:    3.8.0-rc6-ninja+
Operating System:  GNU/Linux
Architecture:      x86_64
Processor:         x86_64
SMP Enabled:       Yes


!!ALSA Version
!!------------

Driver version:     k3.8.0-rc6-ninja+
Library version:    1.0.25
Utilities version:  1.0.25


!!Loaded ALSA modules
!!-------------------

snd_hda_intel
snd_hda_intel


!!Sound Servers on this system
!!----------------------------

Pulseaudio:
      Installed - Yes (/usr/bin/pulseaudio)
      Running - Yes


!!Soundcards recognised by ALSA
!!-----------------------------

 0 [PCH            ]: HDA-Intel - HDA Intel PCH
                      HDA Intel PCH at 0xc1c10000 irq 50
 1 [NVidia         ]: HDA-Intel - HDA NVidia
                      HDA NVidia at 0xc1080000 irq 51


!!PCI Soundcards installed in the system
!!--------------------------------------

00:1b.0 Audio device: Intel Corporation 7 Series/C210 Series Chipset
Family High Definition Audio Controller (rev 04)
01:00.1 Audio device: NVIDIA Corporation Device 0e1b (rev ff)


!!Advanced information - PCI Vendor/Device/Subsystem ID's
!!-------------------------------------------------------

00:1b.0 0403: 8086:1e20 (rev 04)
	Subsystem: 8086:7270
--
01:00.1 0403: 10de:0e1b (rev ff) (prog-if ff)
	!!! Unknown header type 7f


!!Modprobe options (Sound related)
!!--------------------------------

snd-atiixp-modem: index=-2
snd-intel8x0m: index=-2
snd-via82xx-modem: index=-2
snd-usb-audio: index=-2
snd-usb-caiaq: index=-2
snd-usb-ua101: index=-2
snd-usb-us122l: index=-2
snd-usb-usx2y: index=-2
snd-cmipci: mpu_port=0x330 fm_port=0x388
snd-pcsp: index=-2
snd-usb-audio: index=-2
snd_hda_intel: enable_msi=1 power_save=1


!!Loaded sound module options
!!---------------------------

!!Module: snd_hda_intel
	align_buffer_size : -1
	bdl_pos_adj : 1,32,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	enable : Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y
	enable_msi : 1
	id : (null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null)
	index : -1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	jackpoll_ms : 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0
	model : (null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null)
	patch : (null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null)
	position_fix :
-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	power_save : 0
	power_save_controller : Y
	probe_mask : -1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	probe_only : 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0
	single_cmd : N
	snoop : Y

!!Module: snd_hda_intel
	align_buffer_size : -1
	bdl_pos_adj : 1,32,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	enable : Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y,Y
	enable_msi : 1
	id : (null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null)
	index : -1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	jackpoll_ms : 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0
	model : (null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null)
	patch : (null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null),(null)
	position_fix :
-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	power_save : 0
	power_save_controller : Y
	probe_mask : -1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1,-1
	probe_only : 0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0
	single_cmd : N
	snoop : Y


!!HDA-Intel Codec information
!!---------------------------
--startcollapse--

Codec: Cirrus Logic CS4206
Address: 0
AFG Function Id: 0x1 (unsol 0)
Vendor Id: 0x10134206
Subsystem Id: 0x106b2800
Revision Id: 0x100302
No Modem Function Group found
Default PCM:
    rates [0x7f0]: 32000 44100 48000 88200 96000 176400 192000
    bits [0x1e]: 16 20 24 32
    formats [0x3]: PCM FLOAT
Default Amp-In caps: N/A
Default Amp-Out caps: N/A
GPIO: io=4, o=0, i=0, unsolicited=0, wake=0
  IO[0]: enable=0, dir=0, wake=0, sticky=0, data=0, unsol=0
  IO[1]: enable=1, dir=1, wake=0, sticky=0, data=0, unsol=0
  IO[2]: enable=0, dir=0, wake=0, sticky=0, data=0, unsol=0
  IO[3]: enable=1, dir=1, wake=0, sticky=0, data=1, unsol=0
Node 0x02 [Audio Output] wcaps 0xd041d: Stereo Amp-Out
  Control: name="Master Playback Switch", index=0, device=0
  Control: name="Master Playback Volume", index=0, device=0
  Control: name="Headphone Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Control: name="Headphone Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Amp-Out caps: ofs=0x73, nsteps=0x7f, stepsize=0x01, mute=1
  Amp-Out vals:  [0x42 0x42]
  Converter: stream=8, channel=0
  PCM:
    rates [0x7f0]: 32000 44100 48000 88200 96000 176400 192000
    bits [0x1e]: 16 20 24 32
    formats [0x3]: PCM FLOAT
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 13 samples
Node 0x03 [Audio Output] wcaps 0xd041d: Stereo Amp-Out
  Control: name="Surround Speaker Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Control: name="Surround Speaker Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Amp-Out caps: ofs=0x73, nsteps=0x7f, stepsize=0x01, mute=1
  Amp-Out vals:  [0x42 0x42]
  Converter: stream=8, channel=2
  PCM:
    rates [0x7f0]: 32000 44100 48000 88200 96000 176400 192000
    bits [0x1e]: 16 20 24 32
    formats [0x3]: PCM FLOAT
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 13 samples
Node 0x04 [Audio Output] wcaps 0xd041d: Stereo Amp-Out
  Control: name="Front Speaker Playback Switch", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Control: name="Front Speaker Playback Volume", index=0, device=0
    ControlAmp: chs=3, dir=Out, idx=0, ofs=0
  Device: name="Cirrus Analog", type="Audio", device=0
  Amp-Out caps: ofs=0x73, nsteps=0x7f, stepsize=0x01, mute=1
  Amp-Out vals:  [0x42 0x42]
  Converter: stream=8, channel=0
  PCM:
    rates [0x7f0]: 32000 44100 48000 88200 96000 176400 192000
    bits [0x1e]: 16 20 24 32
    formats [0x3]: PCM FLOAT
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 13 samples
Node 0x05 [Audio Input] wcaps 0x18051b: Stereo Amp-In
  Amp-In caps: ofs=0x33, nsteps=0x3f, stepsize=0x03, mute=1
  Amp-In vals:  [0xb3 0xb3] [0xb3 0xb3]
  Converter: stream=0, channel=0
  SDI-Select: 0
  PCM:
    rates [0x1f5]: 8000 16000 32000 44100 48000 88200 96000
    bits [0x1e]: 16 20 24 32
    formats [0x3]: PCM FLOAT
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 8 samples
  Connection: 2
     0x0c* 0x12
Node 0x06 [Audio Input] wcaps 0x18051b: Stereo Amp-In
  Control: name="Capture Switch", index=0, device=0
  Control: name="Capture Volume", index=0, device=0
  Device: name="Cirrus Analog", type="Audio", device=0
  Amp-In caps: ofs=0x33, nsteps=0x3f, stepsize=0x03, mute=1
  Amp-In vals:  [0x37 0x37] [0x37 0x37]
  Converter: stream=4, channel=0
  SDI-Select: 0
  PCM:
    rates [0x1f5]: 8000 16000 32000 44100 48000 88200 96000
    bits [0x1e]: 16 20 24 32
    formats [0x3]: PCM FLOAT
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 8 samples
  Connection: 2
     0x0d 0x0e*
Node 0x07 [Audio Input] wcaps 0x180791: Stereo Digital
  Converter: stream=0, channel=0
  SDI-Select: 0
  Digital: Preemphasis Non-Copyright
  Digital category: 0x0
  IEC Coding Type: 0x0
  PCM:
    rates [0x570]: 32000 44100 48000 96000 192000
    bits [0x1e]: 16 20 24 32
    formats [0x7]: PCM FLOAT AC3
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 8 samples
  Connection: 1
     0x0f
Node 0x08 [Audio Output] wcaps 0x40611: 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="Cirrus Digital", type="SPDIF", device=1
  Converter: stream=0, channel=0
  Digital:
  Digital category: 0x0
  IEC Coding Type: 0x0
  PCM:
    rates [0x7f0]: 32000 44100 48000 88200 96000 176400 192000
    bits [0x1e]: 16 20 24 32
    formats [0x7]: PCM FLOAT AC3
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 4 samples
Node 0x09 [Pin Complex] wcaps 0x410581: Stereo
  Control: name="Headphone Jack", index=0, device=0
  Pincap 0x0000001c: OUT HP Detect
  Pin Default 0x002b4020: [Jack] HP Out at Ext N/A
    Conn = Comb, Color = Green
    DefAssociation = 0x2, Sequence = 0x0
  Pin-ctls: 0xc0: OUT HP
  Unsolicited: tag=01, enabled=1
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 1 samples
  Connection: 1
     0x02
Node 0x0a [Pin Complex] wcaps 0x410581: Stereo
  Control: name="Speaker Surround Phantom Jack", index=0, device=0
  Pincap 0x00000054: OUT Detect Balanced
  Pin Default 0x90100112: [Fixed] Speaker at Int N/A
    Conn = Unknown, Color = Unknown
    DefAssociation = 0x1, Sequence = 0x2
    Misc = NO_PRESENCE
  Pin-ctls: 0x40: OUT
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 1 samples
  Connection: 1
     0x03
Node 0x0b [Pin Complex] wcaps 0x410101: Stereo
  Control: name="Speaker Front Phantom Jack", index=0, device=0
  Pincap 0x00000050: OUT Balanced
  Pin Default 0x90100110: [Fixed] Speaker at Int N/A
    Conn = Unknown, Color = Unknown
    DefAssociation = 0x1, Sequence = 0x0
    Misc = NO_PRESENCE
  Pin-ctls: 0x40: OUT
  Delay: 1 samples
  Connection: 1
     0x04
Node 0x0c [Pin Complex] wcaps 0x41048b: Stereo Amp-In
  Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Pincap 0x00000024: IN Detect
  Pin Default 0x400000f0: [N/A] Line Out at Ext N/A
    Conn = Unknown, Color = Unknown
    DefAssociation = 0xf, Sequence = 0x0
  Pin-ctls: 0x00:
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 1 samples
Node 0x0d [Pin Complex] wcaps 0x41048b: Stereo Amp-In
  Amp-In caps: ofs=0x00, nsteps=0x03, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Pincap 0x00001764: IN Detect Balanced
    Vref caps: HIZ 50 GRD 80
  Pin Default 0x400000f0: [N/A] Line Out at Ext N/A
    Conn = Unknown, Color = Unknown
    DefAssociation = 0xf, Sequence = 0x0
  Pin-ctls: 0x00: VREF_HIZ
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 1 samples
Node 0x0e [Pin Complex] wcaps 0x41000b: Stereo Amp-In
  Control: name="Mic Capture Volume", index=0, device=0
    ControlAmp: chs=3, dir=In, idx=0, ofs=0
  Control: name="Mic Phantom Jack", index=0, device=0
  Amp-In caps: ofs=0x00, nsteps=0x02, stepsize=0x27, mute=0
  Amp-In vals:  [0x02 0x02]
  Pincap 0x00000020: IN
  Pin Default 0x90a60100: [Fixed] Mic at Int N/A
    Conn = Digital, Color = Unknown
    DefAssociation = 0x0, Sequence = 0x0
    Misc = NO_PRESENCE
  Pin-ctls: 0x20: IN
  Delay: 1 samples
Node 0x0f [Pin Complex] wcaps 0x410681: Stereo Digital
  Pincap 0x00000024: IN Detect
  Pin Default 0x400000f0: [N/A] Line Out at Ext N/A
    Conn = Unknown, Color = Unknown
    DefAssociation = 0xf, Sequence = 0x0
  Pin-ctls: 0x00:
  Unsolicited: tag=00, enabled=0
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 1 samples
Node 0x10 [Pin Complex] wcaps 0x410301: Stereo Digital
  Control: name="SPDIF Phantom Jack", index=0, device=0
  Pincap 0x00000010: OUT
  Pin Default 0x004be030: [Jack] SPDIF Out at Ext N/A
    Conn = Comb, Color = White
    DefAssociation = 0x3, Sequence = 0x0
  Pin-ctls: 0x00:
  Delay: 1 samples
  Connection: 1
     0x08
Node 0x11 [Vendor Defined Widget] wcaps 0xf00040: Mono
  Processing caps: benign=0, ncoeff=22
Node 0x12 [Pin Complex] wcaps 0x41000b: Stereo Amp-In
  Amp-In caps: ofs=0x00, nsteps=0x02, stepsize=0x27, mute=0
  Amp-In vals:  [0x00 0x00]
  Pincap 0x00000020: IN
  Pin Default 0x400000f0: [N/A] Line Out at Ext N/A
    Conn = Unknown, Color = Unknown
    DefAssociation = 0xf, Sequence = 0x0
  Pin-ctls: 0x00:
  Delay: 1 samples
Node 0x13 [Beep Generator Widget] wcaps 0x700000: Mono
Node 0x14 [Audio Output] wcaps 0x40611: Stereo Digital
  Converter: stream=0, channel=0
  Digital:
  Digital category: 0x0
  IEC Coding Type: 0x0
  PCM:
    rates [0x7f0]: 32000 44100 48000 88200 96000 176400 192000
    bits [0x1e]: 16 20 24 32
    formats [0x7]: PCM FLOAT AC3
  Power states:  D0 D3 EPSS
  Power: setting=D0, actual=D0
  Delay: 4 samples
Node 0x15 [Pin Complex] wcaps 0x410301: Stereo Digital
  Pincap 0x00000010: OUT
  Pin Default 0x400000f0: [N/A] Line Out at Ext N/A
    Conn = Unknown, Color = Unknown
    DefAssociation = 0xf, Sequence = 0x0
  Pin-ctls: 0x00:
  Delay: 1 samples
  Connection: 1
     0x14
Codec: Nvidia GPU 42 HDMI/DP
Address: 0
AFG Function Id: 0x1 (unsol 0)
Vendor Id: 0x10de0042
Subsystem Id: 0x106b00f2
Revision Id: 0x100100
No Modem Function Group found
Default PCM:
    rates [0x0]:
    bits [0x0]:
    formats [0x0]:
Default Amp-In caps: N/A
Default Amp-Out caps: N/A
Invalid AFG subtree
--endcollapse--


!!ALSA Device nodes
!!-----------------

crw-rw---T+ 1 root audio 116,  6 Feb  6 00:20 /dev/snd/controlC0
crw-rw---T+ 1 root audio 116, 11 Feb  6 00:20 /dev/snd/controlC1
crw-rw---T+ 1 root audio 116,  5 Feb  6 00:20 /dev/snd/hwC0D0
crw-rw---T+ 1 root audio 116, 10 Feb  6 00:20 /dev/snd/hwC1D0
crw-rw---T+ 1 root audio 116,  4 Feb  6 00:21 /dev/snd/pcmC0D0c
crw-rw---T+ 1 root audio 116,  3 Feb  6 00:21 /dev/snd/pcmC0D0p
crw-rw---T+ 1 root audio 116,  2 Feb  6 00:21 /dev/snd/pcmC0D1p
crw-rw---T+ 1 root audio 116,  9 Feb  6 00:20 /dev/snd/pcmC1D3p
crw-rw---T+ 1 root audio 116,  8 Feb  6 00:20 /dev/snd/pcmC1D7p
crw-rw---T+ 1 root audio 116,  7 Feb  6 00:20 /dev/snd/pcmC1D8p
crw-rw---T+ 1 root audio 116, 33 Feb  6 00:20 /dev/snd/timer

/dev/snd/by-path:
total 0
drwxr-xr-x 2 root root  80 Feb  6 00:20 .
drwxr-xr-x 3 root root 280 Feb  6 00:20 ..
lrwxrwxrwx 1 root root  12 Feb  6 00:20 pci-0000:00:1b.0 -> ../controlC0
lrwxrwxrwx 1 root root  12 Feb  6 00:20 pci-0000:01:00.1 -> ../controlC1


!!Aplay/Arecord output
!!--------------------

APLAY

**** List of PLAYBACK Hardware Devices ****
card 0: PCH [HDA Intel PCH], device 0: Cirrus Analog [Cirrus Analog]
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 0: PCH [HDA Intel PCH], device 1: Cirrus Digital [Cirrus Digital]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

ARECORD

**** List of CAPTURE Hardware Devices ****
card 0: PCH [HDA Intel PCH], device 0: Cirrus Analog [Cirrus Analog]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

!!Amixer output
!!-------------

!!-------Mixer controls for card 0 [PCH]

Card hw:0 'PCH'/'HDA Intel PCH at 0xc1c10000 irq 50'
  Mixer name	: 'Cirrus Logic CS4206'
  Components	: 'HDA:10134206,106b2800,00100302'
  Controls      : 25
  Simple ctrls  : 9
Simple mixer control 'Master',0
  Capabilities: pvolume pvolume-joined pswitch pswitch-joined penum
  Playback channels: Mono
  Limits: Playback 0 - 115
  Mono: Playback 66 [57%] [-24.50dB] [on]
Simple mixer control 'Headphone',0
  Capabilities: pvolume pswitch penum
  Playback channels: Front Left - Front Right
  Limits: Playback 0 - 115
  Mono:
  Front Left: Playback 115 [100%] [0.00dB] [on]
  Front Right: Playback 115 [100%] [0.00dB] [on]
Simple mixer control 'PCM',0
  Capabilities: pvolume penum
  Playback channels: Front Left - Front Right
  Limits: Playback 0 - 255
  Mono:
  Front Left: Playback 255 [100%] [0.00dB]
  Front Right: Playback 255 [100%] [0.00dB]
Simple mixer control 'Front Speaker',0
  Capabilities: pvolume pswitch penum
  Playback channels: Front Left - Front Right
  Limits: Playback 0 - 115
  Mono:
  Front Left: Playback 115 [100%] [0.00dB] [on]
  Front Right: Playback 115 [100%] [0.00dB] [on]
Simple mixer control 'Surround Speaker',0
  Capabilities: pvolume pswitch penum
  Playback channels: Front Left - Front Right
  Limits: Playback 0 - 115
  Mono:
  Front Left: Playback 115 [100%] [0.00dB] [on]
  Front Right: Playback 115 [100%] [0.00dB] [on]
Simple mixer control 'Mic',0
  Capabilities: cvolume penum
  Capture channels: Front Left - Front Right
  Limits: Capture 0 - 2
  Front Left: Capture 2 [100%] [20.00dB]
  Front Right: Capture 2 [100%] [20.00dB]
Simple mixer control 'IEC958',0
  Capabilities: pswitch pswitch-joined penum
  Playback channels: Mono
  Mono: Playback [off]
Simple mixer control 'IEC958 Default PCM',0
  Capabilities: pswitch pswitch-joined penum
  Playback channels: Mono
  Mono: Playback [on]
Simple mixer control 'Capture',0
  Capabilities: cvolume cswitch penum
  Capture channels: Front Left - Front Right
  Limits: Capture 0 - 63
  Front Left: Capture 55 [87%] [4.00dB] [on]
  Front Right: Capture 55 [87%] [4.00dB] [on]

!!-------Mixer controls for card 1 [NVidia]

Invalid card number.
Usage: amixer <options> [command]

Available options:
  -h,--help       this help
  -c,--card N     select the card
  -D,--device N   select the device, default 'default'
  -d,--debug      debug mode
  -n,--nocheck    do not perform range checking
  -v,--version    print version of this program
  -q,--quiet      be quiet
  -i,--inactive   show also inactive controls
  -a,--abstract L select abstraction level (none or basic)
  -s,--stdin      Read and execute commands from stdin sequentially

Available commands:
  scontrols       show all mixer simple controls
  scontents	  show contents of all mixer simple controls (default command)
  sset sID P      set contents for one mixer simple control
  sget sID        get contents for one mixer simple control
  controls        show all controls for given card
  contents        show contents of all controls for given card
  cset cID P      set control contents for one control
  cget cID        get control contents for one control
Invalid card number.
Usage: amixer <options> [command]

Available options:
  -h,--help       this help
  -c,--card N     select the card
  -D,--device N   select the device, default 'default'
  -d,--debug      debug mode
  -n,--nocheck    do not perform range checking
  -v,--version    print version of this program
  -q,--quiet      be quiet
  -i,--inactive   show also inactive controls
  -a,--abstract L select abstraction level (none or basic)
  -s,--stdin      Read and execute commands from stdin sequentially

Available commands:
  scontrols       show all mixer simple controls
  scontents	  show contents of all mixer simple controls (default command)
  sset sID P      set contents for one mixer simple control
  sget sID        get contents for one mixer simple control
  controls        show all controls for given card
  contents        show contents of all controls for given card
  cset cID P      set control contents for one control
  cget cID        get control contents for one control


!!Alsactl output
!!--------------

--startcollapse--
state.PCH {
	control.1 {
		iface MIXER
		name 'Master Playback Switch'
		value true
		comment {
			access 'read write'
			type BOOLEAN
			count 1
		}
	}
	control.2 {
		iface MIXER
		name 'Master Playback Volume'
		value 66
		comment {
			access 'read write'
			type INTEGER
			count 1
			range '0 - 115'
			dbmin -5750
			dbmax 0
			dbvalue.0 -2450
		}
	}
	control.3 {
		iface MIXER
		name 'Headphone Playback Switch'
		value.0 true
		value.1 true
		comment {
			access 'read write'
			type BOOLEAN
			count 2
		}
	}
	control.4 {
		iface MIXER
		name 'Headphone Playback Volume'
		value.0 115
		value.1 115
		comment {
			access 'read write'
			type INTEGER
			count 2
			range '0 - 115'
			dbmin -5750
			dbmax 0
			dbvalue.0 0
			dbvalue.1 0
		}
	}
	control.5 {
		iface MIXER
		name 'Front Speaker Playback Switch'
		value.0 true
		value.1 true
		comment {
			access 'read write'
			type BOOLEAN
			count 2
		}
	}
	control.6 {
		iface MIXER
		name 'Front Speaker Playback Volume'
		value.0 115
		value.1 115
		comment {
			access 'read write'
			type INTEGER
			count 2
			range '0 - 115'
			dbmin -5750
			dbmax 0
			dbvalue.0 0
			dbvalue.1 0
		}
	}
	control.7 {
		iface MIXER
		name 'Surround Speaker Playback Switch'
		value.0 true
		value.1 true
		comment {
			access 'read write'
			type BOOLEAN
			count 2
		}
	}
	control.8 {
		iface MIXER
		name 'Surround Speaker Playback Volume'
		value.0 115
		value.1 115
		comment {
			access 'read write'
			type INTEGER
			count 2
			range '0 - 115'
			dbmin -5750
			dbmax 0
			dbvalue.0 0
			dbvalue.1 0
		}
	}
	control.9 {
		iface MIXER
		name 'Capture Switch'
		value.0 true
		value.1 true
		comment {
			access 'read write'
			type BOOLEAN
			count 2
		}
	}
	control.10 {
		iface MIXER
		name 'Capture Volume'
		value.0 55
		value.1 55
		comment {
			access 'read write'
			type INTEGER
			count 2
			range '0 - 63'
			dbmin -5100
			dbmax 1200
			dbvalue.0 400
			dbvalue.1 400
		}
	}
	control.11 {
		iface MIXER
		name 'Mic Capture Volume'
		value.0 2
		value.1 2
		comment {
			access 'read write'
			type INTEGER
			count 2
			range '0 - 2'
			dbmin 0
			dbmax 2000
			dbvalue.0 2000
			dbvalue.1 2000
		}
	}
	control.12 {
		iface MIXER
		name 'IEC958 Playback Con Mask'
		value '0fff000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
		comment {
			access read
			type IEC958
			count 1
		}
	}
	control.13 {
		iface MIXER
		name 'IEC958 Playback Pro Mask'
		value '0f00000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
		comment {
			access read
			type IEC958
			count 1
		}
	}
	control.14 {
		iface MIXER
		name 'IEC958 Playback Default'
		value '0400000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000'
		comment {
			access 'read write'
			type IEC958
			count 1
		}
	}
	control.15 {
		iface MIXER
		name 'IEC958 Playback Switch'
		value false
		comment {
			access 'read write'
			type BOOLEAN
			count 1
		}
	}
	control.16 {
		iface MIXER
		name 'IEC958 Default PCM Playback Switch'
		value true
		comment {
			access 'read write'
			type BOOLEAN
			count 1
		}
	}
	control.17 {
		iface CARD
		name 'Headphone Jack'
		value false
		comment {
			access read
			type BOOLEAN
			count 1
		}
	}
	control.18 {
		iface CARD
		name 'Speaker Front Phantom Jack'
		value true
		comment {
			access read
			type BOOLEAN
			count 1
		}
	}
	control.19 {
		iface CARD
		name 'Speaker Surround Phantom Jack'
		value true
		comment {
			access read
			type BOOLEAN
			count 1
		}
	}
	control.20 {
		iface CARD
		name 'Mic Phantom Jack'
		value true
		comment {
			access read
			type BOOLEAN
			count 1
		}
	}
	control.21 {
		iface CARD
		name 'SPDIF Phantom Jack'
		value true
		comment {
			access read
			type BOOLEAN
			count 1
		}
	}
	control.22 {
		iface PCM
		name 'Playback Channel Map'
		value.0 0
		value.1 0
		value.2 0
		value.3 0
		comment {
			access read
			type INTEGER
			count 4
			range '0 - 36'
		}
	}
	control.23 {
		iface PCM
		name 'Capture Channel Map'
		value.0 0
		value.1 0
		comment {
			access read
			type INTEGER
			count 2
			range '0 - 36'
		}
	}
	control.24 {
		iface PCM
		device 1
		name 'Playback Channel Map'
		value.0 0
		value.1 0
		comment {
			access read
			type INTEGER
			count 2
			range '0 - 36'
		}
	}
	control.25 {
		iface MIXER
		name 'PCM Playback Volume'
		value.0 255
		value.1 255
		comment {
			access 'read write user'
			type INTEGER
			count 2
			range '0 - 255'
			tlv '0000000100000008ffffec1400000014'
			dbmin -5100
			dbmax 0
			dbvalue.0 0
			dbvalue.1 0
		}
	}
}
--endcollapse--


!!All Loaded Modules
!!------------------

Module
fuse
snd_hda_codec_hdmi
snd_hda_codec_cirrus
joydev
hid_apple
bcm5974
uvcvideo
videobuf2_core
videobuf2_vmalloc
videobuf2_memops
coretemp
kvm_intel
kvm
ghash_clmulni_intel
ax88179_178a
usbnet
mii
b43
ssb
applesmc
input_polldev
microcode
lpc_ich
bnep
bcma
mfd_core
rfcomm
apple_gmux
snd_hda_intel
snd_hda_codec
snd_hwdep
snd_pcm
nouveau
snd_timer
ttm
hwmon
mxm_wmi
snd
snd_page_alloc
binfmt_misc
i915
apple_bl
drm_kms_helper
video
nls_iso8859_1


!!Sysfs Files
!!-----------

/sys/class/sound/hwC0D0/init_pin_configs:
0x09 0x002b4020
0x0a 0x90100112
0x0b 0x90100110
0x0c 0x400000f0
0x0d 0x400000f0
0x0e 0x90a60100
0x0f 0x400000f0
0x10 0x004be030
0x12 0x400000f0
0x15 0x400000f0

/sys/class/sound/hwC0D0/driver_pin_configs:
0x0d 0x40ab90f0
0x0e 0x90a600f0
0x12 0x50a600f0

/sys/class/sound/hwC0D0/user_pin_configs:

/sys/class/sound/hwC0D0/init_verbs:

/sys/class/sound/hwC1D0/init_pin_configs:
0x04 0x185600f0
0x05 0x185600f0
0x06 0x585600f0
0x07 0x185600f0

/sys/class/sound/hwC1D0/driver_pin_configs:

/sys/class/sound/hwC1D0/user_pin_configs:

/sys/class/sound/hwC1D0/init_verbs:


!!ALSA/HDA dmesg
!!--------------

[    7.844761] [drm] Initialized i915 1.6.0 20080730 for 0000:00:02.0 on minor 0
[    7.845045] snd_hda_intel 0000:00:1b.0: enabling device (0000 -> 0002)
[    7.845130] snd_hda_intel 0000:00:1b.0: irq 50 for MSI/MSI-X
[    7.845864] nouveau  [  DEVICE][0000:01:00.0] BOOT0  : 0x0e7150a2
--
[    7.848918] nouveau  [   VBIOS][0000:01:00.0] checking PRAMIN for image...
[    7.878753] input: HDA Intel PCH Headphone as
/devices/pci0000:00/0000:00:1b.0/sound/card0/input11
[    7.880014] ACPI Warning: 0x0000000000000428-0x000000000000042f
SystemIO conflicts with Region \PMIO 1 (20121018/utaddress-251)
--
[    8.101675] [drm] Initialized nouveau 1.1.0 20120801 for
0000:01:00.0 on minor 1
[    8.104209] snd_hda_intel 0000:01:00.1: enabling device (0000 -> 0002)
[    8.107687] hda-intel 0000:01:00.1: Handle VGA-switcheroo audio client
[    8.113735] snd_hda_intel 0000:01:00.1: irq 51 for MSI/MSI-X
[    8.441961] b43-phy0: Loading firmware version 666.2 (2011-02-23 01:15:07)
[    9.201709] input: HDA NVidia HDMI/DP,pcm=8 as
/devices/pci0000:00/0000:00:01.0/0000:01:00.1/sound/card1/input12
[    9.202150] input: HDA NVidia HDMI/DP,pcm=7 as
/devices/pci0000:00/0000:00:01.0/0000:01:00.1/sound/card1/input13
[    9.202567] input: HDA NVidia HDMI/DP,pcm=3 as
/devices/pci0000:00/0000:00:01.0/0000:01:00.1/sound/card1/input14
[    9.935412] hda-intel 0000:01:00.1: Disabling via VGA-switcheroo
[   10.011120] VGA switcheroo: switched nouveau off
-- 
Daniel J Blueman

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-05 16:29   ` Daniel J Blueman
@ 2013-02-05 16:34     ` Takashi Iwai
  2013-02-05 17:40       ` Takashi Iwai
  0 siblings, 1 reply; 9+ messages in thread
From: Takashi Iwai @ 2013-02-05 16:34 UTC (permalink / raw)
  To: Daniel J Blueman; +Cc: Linux Kernel, alsa-devel

At Wed, 6 Feb 2013 00:29:54 +0800,
Daniel J Blueman wrote:
> 
> On 6 February 2013 00:16, Takashi Iwai <tiwai@suse.de> wrote:
> > At Wed, 6 Feb 2013 00:10:30 +0800,
> > Daniel J Blueman wrote:
> >>
> >> Hi Takashi,
> >>
> >> The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
> >> arrangement on my Macbook Pro 10,1 - only the higher-frequency
> >> speakers work despite the front and rear channels being exposed in the
> >> mixer.
> >
> > Interesting.  So you have a machine with 4.0 built-in speaker instead
> > of 2.1?  Then we need to add a device-specific flag for it.  Currently
> > the driver assumes 2.1 system blindly because majority of machines
> > have that.
> >
> > FWIW, the codec parser code has been totally rewritten for 3.9, so any
> > patch to 3.8 won't be applied to 3.9 (and vice versa)...
> >
> > Could you give alsa-info.sh output on 3.8 kernel?  Then I'll try to
> > cook it for 3.9 (and maybe backport to 3.8).
> 
> Here's the output from the current alsa-info.sh on 3.8-rc6 with the
> two cited patches reverted [1]; let me know if you'd like 3.8-rc6
> pure.

Thanks.

> The sound is pretty impressive for a laptop when the low-frequecy
> speakers are enabled.

Which program are you using for testing the surrounds?
I'm interested in it because the commit you reverted is basically
providing only an additional information for the channel map, and it
doesn't change anything else.  It implies that some applications are
really referring to the chmap info.


Takashi

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-05 16:16 ` Takashi Iwai
  2013-02-05 16:29   ` Daniel J Blueman
@ 2013-02-05 17:34   ` Takashi Iwai
  1 sibling, 0 replies; 9+ messages in thread
From: Takashi Iwai @ 2013-02-05 17:34 UTC (permalink / raw)
  To: Daniel J Blueman; +Cc: Linux Kernel, alsa-devel

At Tue, 05 Feb 2013 17:16:32 +0100,
Takashi Iwai wrote:
> 
> At Wed, 6 Feb 2013 00:10:30 +0800,
> Daniel J Blueman wrote:
> > 
> > Hi Takashi,
> > 
> > The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
> > arrangement on my Macbook Pro 10,1 - only the higher-frequency
> > speakers work despite the front and rear channels being exposed in the
> > mixer.
> 
> Interesting.  So you have a machine with 4.0 built-in speaker instead
> of 2.1?  Then we need to add a device-specific flag for it.  Currently
> the driver assumes 2.1 system blindly because majority of machines
> have that.
> 
> FWIW, the codec parser code has been totally rewritten for 3.9, so any
> patch to 3.8 won't be applied to 3.9 (and vice versa)...
> 
> Could you give alsa-info.sh output on 3.8 kernel?  Then I'll try to
> cook it for 3.9 (and maybe backport to 3.8).

The patch below is a quick fix for 3.8-rc6.
Could you check whether this works alone?


Takashi

---
diff --git a/sound/pci/hda/patch_cirrus.c b/sound/pci/hda/patch_cirrus.c
index a2537b2..d74e06c 100644
--- a/sound/pci/hda/patch_cirrus.c
+++ b/sound/pci/hda/patch_cirrus.c
@@ -448,7 +448,8 @@ static int parse_output(struct hda_codec *codec)
 	spec->multiout.max_channels = i * 2;
 
 	if (cfg->line_out_type == AUTO_PIN_SPEAKER_OUT && i == 2)
-		spec->speaker_2_1 = 1; /* assume 2.1 speakers */
+		if (spec->gen.fixup_id != CS420X_MBP101)
+			spec->speaker_2_1 = 1; /* assume 2.1 speakers */
 
 	/* add HP and speakers */
 	extra_nids = 0;

^ permalink raw reply related	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-05 16:34     ` Takashi Iwai
@ 2013-02-05 17:40       ` Takashi Iwai
  2013-02-06  0:47         ` Daniel J Blueman
  0 siblings, 1 reply; 9+ messages in thread
From: Takashi Iwai @ 2013-02-05 17:40 UTC (permalink / raw)
  To: Daniel J Blueman; +Cc: Linux Kernel, alsa-devel

At Tue, 05 Feb 2013 17:34:15 +0100,
Takashi Iwai wrote:
> 
> At Wed, 6 Feb 2013 00:29:54 +0800,
> Daniel J Blueman wrote:
> > 
> > On 6 February 2013 00:16, Takashi Iwai <tiwai@suse.de> wrote:
> > > At Wed, 6 Feb 2013 00:10:30 +0800,
> > > Daniel J Blueman wrote:
> > >>
> > >> Hi Takashi,
> > >>
> > >> The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
> > >> arrangement on my Macbook Pro 10,1 - only the higher-frequency
> > >> speakers work despite the front and rear channels being exposed in the
> > >> mixer.
> > >
> > > Interesting.  So you have a machine with 4.0 built-in speaker instead
> > > of 2.1?  Then we need to add a device-specific flag for it.  Currently
> > > the driver assumes 2.1 system blindly because majority of machines
> > > have that.
> > >
> > > FWIW, the codec parser code has been totally rewritten for 3.9, so any
> > > patch to 3.8 won't be applied to 3.9 (and vice versa)...
> > >
> > > Could you give alsa-info.sh output on 3.8 kernel?  Then I'll try to
> > > cook it for 3.9 (and maybe backport to 3.8).
> > 
> > Here's the output from the current alsa-info.sh on 3.8-rc6 with the
> > two cited patches reverted [1]; let me know if you'd like 3.8-rc6
> > pure.
> 
> Thanks.
> 
> > The sound is pretty impressive for a laptop when the low-frequecy
> > speakers are enabled.
> 
> Which program are you using for testing the surrounds?
> I'm interested in it because the commit you reverted is basically
> providing only an additional information for the channel map, and it
> doesn't change anything else.  It implies that some applications are
> really referring to the chmap info.

Or, it might be that the mixer value is simply not set correct.

To be sure, could you try again 3.8-rc6 without reversing patches,
adjust "Speaker" and "Bass Speaker" volumes properly, and retest?
If it still doesn't work, please take alsa-info.sh snapshot at this
state for comparing with the previous result.


thanks,

Takashi

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-05 17:40       ` Takashi Iwai
@ 2013-02-06  0:47         ` Daniel J Blueman
  2013-02-06  7:05           ` Takashi Iwai
  0 siblings, 1 reply; 9+ messages in thread
From: Daniel J Blueman @ 2013-02-06  0:47 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: Linux Kernel, alsa-devel

On 6 February 2013 01:40, Takashi Iwai <tiwai@suse.de> wrote:
> At Tue, 05 Feb 2013 17:34:15 +0100,
> Takashi Iwai wrote:
>>
>> At Wed, 6 Feb 2013 00:29:54 +0800,
>> Daniel J Blueman wrote:
>> >
>> > On 6 February 2013 00:16, Takashi Iwai <tiwai@suse.de> wrote:
>> > > At Wed, 6 Feb 2013 00:10:30 +0800,
>> > > Daniel J Blueman wrote:
>> > >>
>> > >> Hi Takashi,
>> > >>
>> > >> The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
>> > >> arrangement on my Macbook Pro 10,1 - only the higher-frequency
>> > >> speakers work despite the front and rear channels being exposed in the
>> > >> mixer.
>> > >
>> > > Interesting.  So you have a machine with 4.0 built-in speaker instead
>> > > of 2.1?  Then we need to add a device-specific flag for it.  Currently
>> > > the driver assumes 2.1 system blindly because majority of machines
>> > > have that.
>> > >
>> > > FWIW, the codec parser code has been totally rewritten for 3.9, so any
>> > > patch to 3.8 won't be applied to 3.9 (and vice versa)...
>> > >
>> > > Could you give alsa-info.sh output on 3.8 kernel?  Then I'll try to
>> > > cook it for 3.9 (and maybe backport to 3.8).
>> >
>> > Here's the output from the current alsa-info.sh on 3.8-rc6 with the
>> > two cited patches reverted [1]; let me know if you'd like 3.8-rc6
>> > pure.
>>
>> Thanks.
>>
>> > The sound is pretty impressive for a laptop when the low-frequecy
>> > speakers are enabled.
>>
>> Which program are you using for testing the surrounds?
>> I'm interested in it because the commit you reverted is basically
>> providing only an additional information for the channel map, and it
>> doesn't change anything else.  It implies that some applications are
>> really referring to the chmap info.
>
> Or, it might be that the mixer value is simply not set correct.
>
> To be sure, could you try again 3.8-rc6 without reversing patches,
> adjust "Speaker" and "Bass Speaker" volumes properly, and retest?
> If it still doesn't work, please take alsa-info.sh snapshot at this
> state for comparing with the previous result.

My apologies! We do now have "Bass Speaker" which affects both bass
speakers. It was always being restored to level 0 and works when set
up.

There is a "Subwoofer" slider in addition to "Balance" and "Fade" in
the GNOME mixer UI which is greyed out; presumably this is intended as
the same mixer control?

Dan
-- 
Daniel J Blueman

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-06  0:47         ` Daniel J Blueman
@ 2013-02-06  7:05           ` Takashi Iwai
  2013-02-06 12:10             ` David Henningsson
  0 siblings, 1 reply; 9+ messages in thread
From: Takashi Iwai @ 2013-02-06  7:05 UTC (permalink / raw)
  To: Daniel J Blueman; +Cc: David Henningsson, Linux Kernel, alsa-devel

At Wed, 6 Feb 2013 08:47:41 +0800,
Daniel J Blueman wrote:
> 
> On 6 February 2013 01:40, Takashi Iwai <tiwai@suse.de> wrote:
> > At Tue, 05 Feb 2013 17:34:15 +0100,
> > Takashi Iwai wrote:
> >>
> >> At Wed, 6 Feb 2013 00:29:54 +0800,
> >> Daniel J Blueman wrote:
> >> >
> >> > On 6 February 2013 00:16, Takashi Iwai <tiwai@suse.de> wrote:
> >> > > At Wed, 6 Feb 2013 00:10:30 +0800,
> >> > > Daniel J Blueman wrote:
> >> > >>
> >> > >> Hi Takashi,
> >> > >>
> >> > >> The v3.8-rc kernels have regressed from v3.7 with the quad-speaker
> >> > >> arrangement on my Macbook Pro 10,1 - only the higher-frequency
> >> > >> speakers work despite the front and rear channels being exposed in the
> >> > >> mixer.
> >> > >
> >> > > Interesting.  So you have a machine with 4.0 built-in speaker instead
> >> > > of 2.1?  Then we need to add a device-specific flag for it.  Currently
> >> > > the driver assumes 2.1 system blindly because majority of machines
> >> > > have that.
> >> > >
> >> > > FWIW, the codec parser code has been totally rewritten for 3.9, so any
> >> > > patch to 3.8 won't be applied to 3.9 (and vice versa)...
> >> > >
> >> > > Could you give alsa-info.sh output on 3.8 kernel?  Then I'll try to
> >> > > cook it for 3.9 (and maybe backport to 3.8).
> >> >
> >> > Here's the output from the current alsa-info.sh on 3.8-rc6 with the
> >> > two cited patches reverted [1]; let me know if you'd like 3.8-rc6
> >> > pure.
> >>
> >> Thanks.
> >>
> >> > The sound is pretty impressive for a laptop when the low-frequecy
> >> > speakers are enabled.
> >>
> >> Which program are you using for testing the surrounds?
> >> I'm interested in it because the commit you reverted is basically
> >> providing only an additional information for the channel map, and it
> >> doesn't change anything else.  It implies that some applications are
> >> really referring to the chmap info.
> >
> > Or, it might be that the mixer value is simply not set correct.
> >
> > To be sure, could you try again 3.8-rc6 without reversing patches,
> > adjust "Speaker" and "Bass Speaker" volumes properly, and retest?
> > If it still doesn't work, please take alsa-info.sh snapshot at this
> > state for comparing with the previous result.
> 
> My apologies! We do now have "Bass Speaker" which affects both bass
> speakers. It was always being restored to level 0 and works when set
> up.

OK, good to know.

> There is a "Subwoofer" slider in addition to "Balance" and "Fade" in
> the GNOME mixer UI which is greyed out; presumably this is intended as
> the same mixer control?

I have no idea about GNOME stuff... David, do you know of this?


Takashi

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: 3.8-rc1 patch_cirrus 4.0 regression...
  2013-02-06  7:05           ` Takashi Iwai
@ 2013-02-06 12:10             ` David Henningsson
  0 siblings, 0 replies; 9+ messages in thread
From: David Henningsson @ 2013-02-06 12:10 UTC (permalink / raw)
  To: Takashi Iwai; +Cc: Daniel J Blueman, Linux Kernel, alsa-devel

On 02/06/2013 08:05 AM, Takashi Iwai wrote:
> At Wed, 6 Feb 2013 08:47:41 +0800,
> Daniel J Blueman wrote:
>>

>>> To be sure, could you try again 3.8-rc6 without reversing patches,
>>> adjust "Speaker" and "Bass Speaker" volumes properly, and retest?
>>> If it still doesn't work, please take alsa-info.sh snapshot at this
>>> state for comparing with the previous result.
>>
>> My apologies! We do now have "Bass Speaker" which affects both bass
>> speakers. It was always being restored to level 0 and works when set
>> up.
>
> OK, good to know.
>
>> There is a "Subwoofer" slider in addition to "Balance" and "Fade" in
>> the GNOME mixer UI which is greyed out; presumably this is intended as
>> the same mixer control?
>
> I have no idea about GNOME stuff... David, do you know of this?

I haven't followed this discussion closely, but what sliders are enabled 
in Gnome sound settings is related to what profile is selected for that 
card. So if you select the 2.1 profile, "Balance" and "Subwoofer" will 
be enabled (and "Fade" disabled), but in a 4.0 profile, "Balance" and 
"Fade" will be enabled (and "Subwoofer" disabled). Is that answering 
your question?

For the record, PulseAudio does not yet use the new channel mapping API.


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

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2013-02-06 12:10 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-02-05 16:10 3.8-rc1 patch_cirrus 4.0 regression Daniel J Blueman
2013-02-05 16:16 ` Takashi Iwai
2013-02-05 16:29   ` Daniel J Blueman
2013-02-05 16:34     ` Takashi Iwai
2013-02-05 17:40       ` Takashi Iwai
2013-02-06  0:47         ` Daniel J Blueman
2013-02-06  7:05           ` Takashi Iwai
2013-02-06 12:10             ` David Henningsson
2013-02-05 17:34   ` Takashi Iwai

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).