From: Takashi Iwai <tiwai@suse.de>
Date: Fri, 20 May 2011 14:27:02 +0000 (+0200)
Subject: Revert "ALSA: hda - Use position_fix=3 as default for AMD chipsets"
X-Git-Tag: 2.1b_release~3866^2~12^2~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=314c3ff47643c7c1f11317f81a80902af80795e3;p=platform%2Fkernel%2Fkernel-mfld-blackbay.git

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 <tiwai@suse.de>
---

diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index ecadb5b..0c1996d 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;