Skip to content

Commit

Permalink
ALSA: hda - Use position_fix=3 as default for AMD chipsets
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Takashi Iwai committed May 4, 2011
1 parent ce85c9a commit 447ee6a
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions sound/pci/hda/hda_intel.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down

0 comments on commit 447ee6a

Please sign in to comment.