[coreboot-gerrit] New patch to review for coreboot: 3dc51dc AMD Steppe Eagle: Disable "No Snoop Enable" to stop HDMI audio stutter
Bruce Griffith (Bruce.Griffith@se-eng.com)
gerrit at coreboot.org
Mon Sep 1 07:29:06 CEST 2014
Bruce Griffith (Bruce.Griffith at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6822
-gerrit
commit 3dc51dca2474f233569fe846e29078bbbc7ffd1f
Author: Bruce Griffith <Bruce.Griffith at se-eng.com>
Date: Sun Aug 31 22:40:57 2014 -0600
AMD Steppe Eagle: Disable "No Snoop Enable" to stop HDMI audio stutter
Ubuntu's HDMI audio has noise and echo. Disable NoSnoopEnable can
resolve this issue. The posted amd_late_init.c northbridge code
is missing a test for Steppe Eagle northbridges. See coreboot Gerrit
change 3934, commit ID 4ca721399c.
Change-Id: I89894d0ce4ad72ea16d61b445edb9e67920bca24
Signed-off-by: Bruce Griffith <Bruce.Griffith at se-eng.com>
---
src/cpu/amd/agesa/amd_late_init.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/cpu/amd/agesa/amd_late_init.c b/src/cpu/amd/agesa/amd_late_init.c
index faba328..c7927dc 100644
--- a/src/cpu/amd/agesa/amd_late_init.c
+++ b/src/cpu/amd/agesa/amd_late_init.c
@@ -20,8 +20,11 @@
#include <arch/acpi.h>
#include <bootstate.h>
#include <console/console.h>
+#include <device/device.h>
+#include <device/pci_def.h>
+#include <device/pci_ops.h>
-#include "agesawrapper.h"
+#include <agesawrapper.h>
#include <northbridge/amd/agesa/agesawrapper_call.h>
#if IS_ENABLED(CONFIG_HUDSON_IMC_FWM)
@@ -31,6 +34,9 @@
#include <sb_cimx.h>
#endif
+#define NORTHBRIDGE_00700F00 IS_ENABLED(CONFIG_NORTHBRIDGE_AMD_AGESA_FAMILY_16KB)
+#define NORTHBRIDGE_00730F01 IS_ENABLED(CONFIG_NORTHBRIDGE_AMD_AGESA_00730F01)
+
static void agesawrapper_post_device(void *unused)
{
if (acpi_is_wakeup_s3())
@@ -38,7 +44,7 @@ static void agesawrapper_post_device(void *unused)
AGESAWRAPPER(amdinitlate);
-#if IS_ENABLED(CONFIG_NORTHBRIDGE_AMD_AGESA_FAMILY_16KB)
+#if (NORTHBRIDGE_00700F00) || (NORTHBRIDGE_00730F01)
device_t dev;
u32 value;
dev = dev_find_slot(0, PCI_DEVFN(0, 0)); /* clear IoapicSbFeatureEn */
More information about the coreboot-gerrit
mailing list