From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757737Ab3AIKAb (ORCPT ); Wed, 9 Jan 2013 05:00:31 -0500 Received: from cantor2.suse.de ([195.135.220.15]:37556 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757699Ab3AIKA2 (ORCPT ); Wed, 9 Jan 2013 05:00:28 -0500 Date: Wed, 09 Jan 2013 11:00:26 +0100 Message-ID: From: Takashi Iwai To: Vincent Blut Cc: Maurizio Avogadro , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [3.6.9 -> 3.7.1 regression] sound: snd_hda_intel codec probing issue? In-Reply-To: <1357676721.3776.3.camel@lamella> References: <1356704740.3953.26.camel@lamella> <1357421043.3912.13.camel@lamella> <1357648135.3849.7.camel@lamella> <1357676721.3776.3.camel@lamella> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.2 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") 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 At Tue, 08 Jan 2013 21:25:21 +0100, Vincent Blut wrote: > > Le mardi 08 janvier 2013 à 13:56 +0100, Takashi Iwai a écrit : > > At Tue, 08 Jan 2013 13:28:55 +0100, > > From: Takashi Iwai > > Subject: [PATCH] ALSA: hda - Disable runtime D3 for Intel CPT & co > > > > We've got a few bug reports that the runtime D3 results in the dead > > HD-audio controller. It seems that the problem is in a deeper level > > than the sound driver itself, so as a temporal solution, disable the > > feature for these controllers again. > > > > Reported-by: Vincent Blut > > Reported-by: Maurizio Avogadro > > Cc: [v3.7] > > Signed-off-by: Takashi Iwai > > --- > > sound/pci/hda/hda_intel.c | 13 ++++++++----- > > 1 file changed, 8 insertions(+), 5 deletions(-) > > > > diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c > > index cca8727..0b6aeba 100644 > > --- a/sound/pci/hda/hda_intel.c > > +++ b/sound/pci/hda/hda_intel.c > > @@ -573,9 +573,12 @@ enum { > > #define AZX_DCAPS_PM_RUNTIME (1 << 26) /* runtime PM support */ > > > > /* quirks for Intel PCH */ > > -#define AZX_DCAPS_INTEL_PCH \ > > +#define AZX_DCAPS_INTEL_PCH_NOPM \ > > (AZX_DCAPS_SCH_SNOOP | AZX_DCAPS_BUFSIZE | \ > > - AZX_DCAPS_COUNT_LPIB_DELAY | AZX_DCAPS_PM_RUNTIME) > > + AZX_DCAPS_COUNT_LPIB_DELAY) > > + > > +#define AZX_DCAPS_INTEL_PCH \ > > + (AZX_DCAPS_INTEL_PCH_NOPM | AZX_DCAPS_PM_RUNTIME) > > > > /* quirks for ATI SB / AMD Hudson */ > > #define AZX_DCAPS_PRESET_ATI_SB \ > > @@ -3586,13 +3589,13 @@ static void azx_remove(struct pci_dev *pci) > > static DEFINE_PCI_DEVICE_TABLE(azx_ids) = { > > /* CPT */ > > { PCI_DEVICE(0x8086, 0x1c20), > > - .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH }, > > + .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH_NOPM }, > > /* PBG */ > > { PCI_DEVICE(0x8086, 0x1d20), > > - .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH }, > > + .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH_NOPM }, > > /* Panther Point */ > > { PCI_DEVICE(0x8086, 0x1e20), > > - .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH }, > > + .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH_NOPM }, > > /* Lynx Point */ > > { PCI_DEVICE(0x8086, 0x8c20), > > .driver_data = AZX_DRIVER_PCH | AZX_DCAPS_INTEL_PCH }, > > Well, my sound controller behaves nicely with the above quirk. > > Tested-by: Vincent Blut OK, I queued the patch now. thanks, Takashi