Revert "ALSA: hda - Use position_fix=3 as default for AMD chipsets"
authorTakashi Iwai <tiwai@suse.de>
Fri, 20 May 2011 14:27:02 +0000 (16:27 +0200)
committerTakashi Iwai <tiwai@suse.de>
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 ecadb5b7d647b0d65fdac7086fc30a7e3c57883a..0c1996dd137beacfc3c161add24246f3f98a303c 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;