diff options
author | Takashi Iwai <tiwai@suse.de> | 2011-05-20 10:27:02 -0400 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2011-05-20 10:27:02 -0400 |
commit | 314c3ff47643c7c1f11317f81a80902af80795e3 (patch) | |
tree | 2f3884ca9efc373f0a47245356de47d32cca48fd | |
parent | 591e610d651079117bd71c625605a3ebd868b1a1 (diff) |
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>
-rw-r--r-- | sound/pci/hda/hda_intel.c | 5 |
1 files changed, 0 insertions, 5 deletions
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) | |||
2370 | case AZX_DRIVER_ATI: | 2370 | case AZX_DRIVER_ATI: |
2371 | /* Use link position directly, avoid any transfer problem. */ | 2371 | /* Use link position directly, avoid any transfer problem. */ |
2372 | return POS_FIX_VIACOMBO; | 2372 | return POS_FIX_VIACOMBO; |
2373 | case AZX_DRIVER_GENERIC: | ||
2374 | /* AMD chipsets behave often badly, too */ | ||
2375 | if (chip->pci->vendor == PCI_VENDOR_ID_AMD) | ||
2376 | return POS_FIX_VIACOMBO; | ||
2377 | break; | ||
2378 | } | 2373 | } |
2379 | 2374 | ||
2380 | return POS_FIX_AUTO; | 2375 | return POS_FIX_AUTO; |