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:34:15 +0100	[thread overview]
Message-ID: <s5hobfyafeg.wl%tiwai@suse.de> (raw)
In-Reply-To: <CAMVG2stEPHuk-cNwZHjtnOP=PeXwF7SMxMSp=HVaOy9SzW2Dgw@mail.gmail.com>

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

  reply	other threads:[~2013-02-05 16:34 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 [this message]
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=s5hobfyafeg.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