ALSA: hda - Use position_fix=3 as default for AMD chipsets
authorTakashi Iwai <tiwai@suse.de>
Wed, 4 May 2011 16:28:50 +0000 (18:28 +0200)
committerTakashi Iwai <tiwai@suse.de>
Wed, 4 May 2011 16:28:50 +0000 (18:28 +0200)
AMD chipsets often behave pretty badly regarding the DMA position
reporting.  It results in the bad quality audio recording.
Using position_fix=3 works well in general for them, so let's enable
it as default for AMD.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/hda/hda_intel.c

index f95ff6e..010fd31 100644 (file)
@@ -2364,6 +2364,11 @@ 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;