At Mon, 1 Jun 2009 15:36:35 +0200, Kacper Szczesniak wrote:

On Mon, Jun 1, 2009 at 10:54 AM, Takashi Iwai tiwai@suse.de wrote:

...

Hm, this patch changes the channel mapping completely. Is it intentional?

yes, 0xc/0xd/0xe for Front/LFE/Surr

...

...

static struct snd_kcontrol_new alc885_mb5_mixer[] = {

• HDA_CODEC_VOLUME("Front Playback Volume", 0x0d, 0x00, HDA_OUTPUT),
• HDA_BIND_MUTE   ("Front Playback Switch", 0x0d, 0x02, HDA_INPUT),
• HDA_CODEC_VOLUME("Line-Out Playback Volume", 0x0c, 0x00, HDA_OUTPUT),
• HDA_BIND_MUTE   ("Line-Out Playback Switch", 0x0c, 0x02, HDA_INPUT),

• HDA_CODEC_VOLUME("Front Playback Volume", 0x0c, 0x00, HDA_OUTPUT),
• HDA_BIND_MUTE   ("Front Playback Switch", 0x0c, 0x02, HDA_INPUT),
• HDA_CODEC_VOLUME("Surround Playback Volume", 0x0d, 0x00, HDA_OUTPUT),
• HDA_BIND_MUTE   ("Surround Playback Switch", 0x0d, 0x02, HDA_INPUT),
• HDA_CODEC_VOLUME("LFE Playback Volume", 0x0e, 0x00, HDA_OUTPUT),
• HDA_BIND_MUTE   ("LFE Playback Switch", 0x0e, 0x02, HDA_INPUT),
• HDA_CODEC_VOLUME("HP Playback Volume", 0x0f, 0x00, HDA_OUTPUT),
• HDA_BIND_MUTE   ("HP Playback Switch", 0x0f, 0x02, HDA_INPUT),

HDA_CODEC_VOLUME("Line Playback Volume", 0x0b, 0x02, HDA_INPUT),       HDA_CODEC_MUTE  ("Line Playback Switch", 0x0b, 0x02, HDA_INPUT),

I'm wondering whether this breaks MB5, not MB5.1.  If it's MB5.1 specific, please create another model instead of breaking the old one.

it's confirmed to work on MB5.1 MBP5.1 MBP52

Hrm, then I wonder why the old mb5 model works as is although the mapping is totally different...

Takashi