From 314c3ff47643c7c1f11317f81a80902af80795e3 Mon Sep 17 00:00:00 2001 From: Takashi Iwai Date: Fri, 20 May 2011 16:27:02 +0200 Subject: Revert "ALSA: hda - Use position_fix=3 as default for AMD chipsets" This reverts commit 447ee6a7cbbfb5ae7ab8f9b8b058b4a04fe398bf. The workaround introduced by this commit seems bogus. The AMD chipsets don't provide proper position-buffer nor FIFO value required by VIACOMBO fix. Signed-off-by: Takashi Iwai --- sound/pci/hda/hda_intel.c | 5 ----- 1 file changed, 5 deletions(-) (limited to 'sound/pci/hda/hda_intel.c') diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c index ecadb5b7d647..0c1996dd137b 100644 --- a/sound/pci/hda/hda_intel.c +++ b/sound/pci/hda/hda_intel.c @@ -2370,11 +2370,6 @@ static int __devinit check_position_fix(struct azx *chip, int fix) case AZX_DRIVER_ATI: /* Use link position directly, avoid any transfer problem. */ return POS_FIX_VIACOMBO; - case AZX_DRIVER_GENERIC: - /* AMD chipsets behave often badly, too */ - if (chip->pci->vendor == PCI_VENDOR_ID_AMD) - return POS_FIX_VIACOMBO; - break; } return POS_FIX_AUTO; -- cgit v1.2.3