From: Takashi Iwai <tiwai@suse.de>
Date: Wed, 4 May 2011 16:28:50 +0000 (+0200)
Subject: ALSA: hda - Use position_fix=3 as default for AMD chipsets
X-Git-Tag: 2.1b_release~3866^2~12^2~37
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=447ee6a7cbbfb5ae7ab8f9b8b058b4a04fe398bf;p=platform%2Fkernel%2Fkernel-mfld-blackbay.git

ALSA: hda - Use position_fix=3 as default for AMD chipsets

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>
---

diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index f95ff6e..010fd31 100644
--- a/sound/pci/hda/hda_intel.c
+++ b/sound/pci/hda/hda_intel.c
@@ -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;