Revert "ALSA: hda - Use position_fix=3 as default for AMD chipsets"
Takashi Iwai [Fri, 20 May 2011 14:27:02 +0000 (16:27 +0200)]
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>

sound/pci/hda/hda_intel.c

index ecadb5b..0c1996d 100644 (file)
@@ -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;