Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752433Ab1D1FuP (ORCPT ); Thu, 28 Apr 2011 01:50:15 -0400 Received: from mail-qw0-f46.google.com ([209.85.216.46]:43862 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751878Ab1D1FuN convert rfc822-to-8bit (ORCPT ); Thu, 28 Apr 2011 01:50:13 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=Q20ajOC9roP1MmlGyuNxKkigYQuWh2pkS21Ug67ns64XM0u62FVmITUS2cWPSQOw3+ IqEQeuSiefQJz2FXo6Ai7JODQYyoTOCHyuzDsUhU9niuZSZXzKRdP3CgxG5/cZtsGzgo Yzr0D7eISldxiFEE2yBbxF64KrjXrXdeUow+Y= MIME-Version: 1.0 In-Reply-To: References: <20110424113037.11af0b04@omega.digital-domain.net> <20110425000126.GA18681@suse.de> <20110425173409.GA32047@suse.de> Date: Thu, 28 Apr 2011 12:50:12 +0700 Message-ID: Subject: Re: 2.6.38.4 broke my audio From: Emmanuel Benisty To: Takashi Iwai Cc: Greg KH , Andrew Clayton , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, stable@kernel.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 6747 Lines: 154 On Thu, Apr 28, 2011 at 12:37 PM, Takashi Iwai wrote: > At Thu, 28 Apr 2011 08:07:23 +0700, > Emmanuel Benisty wrote: >> >> [1  ] >> On Thu, Apr 28, 2011 at 2:35 AM, Takashi Iwai wrote: >> > At Wed, 27 Apr 2011 23:17:20 +0700, >> > Emmanuel Benisty wrote: >> >> >> >> [1  ] >> >> On Wed, Apr 27, 2011 at 10:47 PM, Takashi Iwai wrote: >> >> > At Wed, 27 Apr 2011 21:57:32 +0700, >> >> > Emmanuel Benisty wrote: >> >> >> >> >> >> On Wed, Apr 27, 2011 at 3:36 PM, Takashi Iwai wrote: >> >> >> > At Wed, 27 Apr 2011 08:15:28 +0700, >> >> >> > Emmanuel Benisty wrote: >> >> >> >> >> >> >> >> [1  ] >> >> >> >> On Tue, Apr 26, 2011 at 7:49 PM, Takashi Iwai wrote: >> >> >> >> > At Tue, 26 Apr 2011 08:42:59 +0700, >> >> >> >> > Emmanuel Benisty wrote: >> >> >> >> >> >> >> >> >> >> On Tue, Apr 26, 2011 at 3:25 AM, Takashi Iwai wrote: >> >> >> >> >> > At Mon, 25 Apr 2011 10:34:09 -0700, >> >> >> >> >> > Greg KH wrote: >> >> >> >> >> >> >> >> >> >> >> >> On Mon, Apr 25, 2011 at 11:25:29AM +0200, Takashi Iwai wrote: >> >> >> >> >> >> > At Sun, 24 Apr 2011 17:01:26 -0700, >> >> >> >> >> >> > Greg KH wrote: >> >> >> >> >> >> > > >> >> >> >> >> >> > > On Sun, Apr 24, 2011 at 11:30:37AM +0100, Andrew Clayton wrote: >> >> >> >> >> >> > > > Hi guys. >> >> >> >> >> >> > > > >> >> >> >> >> >> > > > Just upgraded to .4 from .3 and found my audio was _really_ quiet. All >> >> >> >> >> >> > > > the mixers looked sane. >> >> >> >> >> >> > > > >> >> >> >> >> >> > > > A quick check of the changelog and a revert of: >> >> >> >> >> >> > > > >> >> >> >> >> >> > > > commit 58541cc27531727f7120683a7cb123ee3ada4bd4 >> >> >> >> >> >> > > > Author: Takashi Iwai >> >> >> >> >> >> > > > Date:   Mon Mar 28 12:05:31 2011 +0200 >> >> >> >> >> >> > > > >> >> >> >> >> >> > > >     ALSA: hda - Fix pin-config of Gigabyte mobo >> >> >> >> >> >> > > > >> >> >> >> >> >> > > >     commit c6b358748e19ce7e230b0926ac42696bc485a562 upstream. >> >> >> >> >> >> > > > >> >> >> >> >> >> > > >     Use pin-fix instead of the static quirk for Gigabyte mobos 1458:a002. >> >> >> >> >> >> > > > >> >> >> >> >> >> > > >     Bugzilla: https://bugzilla.novell.com/show_bug.cgi?id=677256 >> >> >> >> >> >> > > >     Signed-off-by: Takashi Iwai >> >> >> >> >> >> > > >     Signed-off-by: Greg Kroah-Hartman >> >> >> >> >> >> > > > >> >> >> >> >> >> > > > made it normal again. >> >> >> >> >> >> > > >> >> >> >> >> >> > > Takashi, do you have a fix for this, or should I revert this from the >> >> >> >> >> >> > > .38 stable tree?  It looks like we now have 2 reports of this problem. >> >> >> >> >> >> > >> >> >> >> >> >> > If we need to revert, I'll do it in the upstream first, as it must hit >> >> >> >> >> >> > to 2.6.39, too. >> >> >> >> >> >> >> >> >> >> >> >> True. >> >> >> >> >> >> >> >> >> >> >> >> > But let me check this breakage first.  I'll take a look at it more >> >> >> >> >> >> > deeply tomorrow. >> >> >> >> >> >> >> >> >> >> >> >> Ok, that sounds good. >> >> >> >> >> > >> >> >> >> >> > Thanks. >> >> >> >> >> > >> >> >> >> >> > Andrew, could you give alsa-info.sh outputs on your machine? >> >> >> >> >> > Preferably, the outputs both before and after the kernel update would >> >> >> >> >> > be really helpful. >> >> >> >> >> >> >> >> >> >> In case it might help, mine are attached too. >> >> >> >> > >> >> >> >> > Thanks.  But the output after update (alsa-info.txt.bad) looks weird. >> >> >> >> > It contains unexpected pin-update for NID 0x15 (found in >> >> >> >> > /sys/class/sound/hwC0D0/driver_pin_configs), and thus the mixer >> >> >> >> > elements created don't match with the results on HD-audio emulator. >> >> >> >> > >> >> >> >> > Could you double-check whether you have no modification over 2.6.38.4 >> >> >> >> > driver? >> >> >> >> >> >> >> >> Hi Takashi, >> >> >> >> >> >> >> >> To double check, I have rebuilt .3 and .4 straight from kernel.org >> >> >> >> sources with the same config (which I attached, just in case). >> >> >> >> alsa-info.sh outputs are attached. >> >> >> > >> >> >> > Thanks.  The bad output still shows the strange pin-config for NID >> >> >> > 0x15 while it's not found in the good output. >> >> >> > >> >> >> > Could you check the following? >> >> >> > >> >> >> > 1. Set model=auto option to snd-hda-intel module and boot 2.6.38.3. >> >> >> >   Get alsa-info.sh output. >> >> >> > >> >> >> > 2. Apply fix1.diff patch below to 2.6.38.4, and boot, get alsa-info.sh >> >> >> >   again. >> >> >> > >> >> >> > 3. Reviert fix1.diff, and apply fix2.diff to 2.6.38.4.  The same >> >> >> >   procedure. >> >> >> > >> >> >> > The point to be checked is the widget 0x15.  In a good case, it should >> >> >> > have the pin config such as: >> >> >> >  Pin Default 0x411111f0: [N/A] Speaker at Ext Rear >> >> >> > >> >> >> > In a bad case, the driver modifies this by some reason to >> >> >> >  Pin Default 0x99130112: [Fixed] Speaker at Int ATAPI >> >> >> >> >> >> Hi Takashi, >> >> >> >> >> >> alsa-info.sh outputs are attached. fix1 works, fix2 is still broken. >> >> > >> >> > Thanks.  Hm, so it happens only with the patch.  Puzzling. >> >> > >> >> > Could you check the messages triggered by the patch below? >> >> > Does it include 0x15? >> >> >> >> attached is what it gives. >> > >> > OK, so it's chained wrongly by some reason.  Maybe using bool type >> > causes a problem for some compiler? >> > >> > Could you try the patch below instead?  If it works as expected, >> > you'll see _no_ Surround volume control. >> >> Sorry Takashi, still no luck with this patch. > > Please double-check whether the patch was really applied. > Now there is no debug message seen.  It should still put some > 'XXX hda_codec: ' lines.  This looks odd. my bad, I have reverted this patch (the debug one, habit of keeping the sources unpatched) but the latest one has been applied, sorry. >> > And, what is your configuration, i.e. which output are you testing? >> > The output from the speaker and/of from the headphone jack? >> >> Ouput from speakers only until now. > > You don't plug into the headphone or line-out jack, right? > Otherwise the speaker output is muted automatically by the jack. in fact, it's a desktop machine, not a laptop, so speakers are plugged into the line-out jack. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/