public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Daniel J Blueman <daniel@quora.org>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>, alsa-devel@alsa-project.org
Subject: Re: 3.8-rc1 patch_cirrus 4.0 regression...
Date: Tue, 05 Feb 2013 17:16:32 +0100	[thread overview]
Message-ID: <s5hsj5aag7z.wl%tiwai@suse.de> (raw)
In-Reply-To: <CAMVG2suCiLak3CbeKZtEgjeJqw3=1fD3k9FMsm=tYE7h3cuVjQ@mail.gmail.com>

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
> 

  reply	other threads:[~2013-02-05 16:16 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-05 16:10 3.8-rc1 patch_cirrus 4.0 regression Daniel J Blueman
2013-02-05 16:16 ` Takashi Iwai [this message]
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

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=s5hsj5aag7z.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=daniel@quora.org \
    --cc=linux-kernel@vger.kernel.org \
    /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