From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753266Ab3AHUZp (ORCPT ); Tue, 8 Jan 2013 15:25:45 -0500 Received: from smtp3-g21.free.fr ([212.27.42.3]:55729 "EHLO smtp3-g21.free.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751031Ab3AHUZo (ORCPT ); Tue, 8 Jan 2013 15:25:44 -0500 Message-ID: <1357676721.3776.3.camel@lamella> Subject: Re: [3.6.9 -> 3.7.1 regression] sound: snd_hda_intel codec probing issue? From: Vincent Blut To: Takashi Iwai Cc: Maurizio Avogadro , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Date: Tue, 08 Jan 2013 21:25:21 +0100 In-Reply-To: References: <1356704740.3953.26.camel@lamella> <1357421043.3912.13.camel@lamella> <1357648135.3849.7.camel@lamella> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.4.4-1 Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 Thanks Takashi for your support, Vincent