From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756208AbZEGKpZ (ORCPT ); Thu, 7 May 2009 06:45:25 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752775AbZEGKpK (ORCPT ); Thu, 7 May 2009 06:45:10 -0400 Received: from cantor.suse.de ([195.135.220.2]:50419 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751094AbZEGKpI (ORCPT ); Thu, 7 May 2009 06:45:08 -0400 Date: Thu, 07 May 2009 12:45:08 +0200 Message-ID: From: Takashi Iwai To: Tino Keitel Cc: linux-kernel@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: 2.6.29 regression: left audio channel broken after resume from suspend with Intel HDA In-Reply-To: <20090507072352.GA6513@mac.home> References: <20090420194659.GA11399@dose.home.local> <20090421073219.GA5866@dose.home.local> <20090421183936.GA21935@dose.home.local> <20090424142113.GA18668@dose.home.local> <20090428002309.GA29572@x61> <20090506222753.GA6728@mac.home> <20090507072352.GA6513@mac.home> User-Agent: Wanderlust/2.12.0 (Your Wildest Dreams) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 Emacs/22.3 (x86_64-suse-linux-gnu) MULE/5.0 (SAKAKI) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Thu, 7 May 2009 09:23:53 +0200, Tino Keitel wrote: > > On Thu, May 07, 2009 at 00:27:53 +0200, Tino Keitel wrote: > > On Tue, Apr 28, 2009 at 02:23:09 +0200, Tino Keitel wrote: > > > > [...] > > > > > I just saw another, major problem: line in doesn't work anymore with > > > 2.6.29 and also 2.6.30-rc3 (no usable input level). It works with > > > 2.6.27.20. > > > > FYI: I just tried 2.6.28 and line-in was still broken. So the last > > working kernel is 2.6.27. > > I tried this: > > git bisect start v2.6.28 v2.6.27 sound/pci/hda/patch_sigmatel.c > > The result is commit 4f1e6bc3646ab50b8181555ab7e6eeab68b8632a. Could you try the patch below instead? The problem is that BIOS on your machine sets this pin both for input and output. Since it's set as input, the driver respects the BIOS setup and skips to override. So, it's a BIOS problem, as usual :) thanks, Takashi --- diff --git a/sound/pci/hda/patch_sigmatel.c b/sound/pci/hda/patch_sigmatel.c index 76487de..e58e008 100644 --- a/sound/pci/hda/patch_sigmatel.c +++ b/sound/pci/hda/patch_sigmatel.c @@ -4084,7 +4084,9 @@ static int stac92xx_init(struct hda_codec *codec) pinctl = snd_hda_codec_read(codec, nid, 0, AC_VERB_GET_PIN_WIDGET_CONTROL, 0); /* if PINCTL already set then skip */ - if (!(pinctl & AC_PINCTL_IN_EN)) { + if (!(pinctl & AC_PINCTL_IN_EN) || + (pinctl & AC_PINCTL_OUT_EN)) { + pinctl &= ~AC_PINCTL_OUT_EN; pinctl |= AC_PINCTL_IN_EN; stac92xx_auto_set_pinctl(codec, nid, pinctl);