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 18:40:53 +0100 [thread overview]
Message-ID: <s5hd2weacbe.wl%tiwai@suse.de> (raw)
In-Reply-To: <s5hobfyafeg.wl%tiwai@suse.de>
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
next prev parent reply other threads:[~2013-02-05 17:40 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
2013-02-05 16:29 ` Daniel J Blueman
2013-02-05 16:34 ` Takashi Iwai
2013-02-05 17:40 ` Takashi Iwai [this message]
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=s5hd2weacbe.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