New patch to review for coreboot: 3dc51dc AMD Steppe Eagle: Disable "No Snoop Enable" to stop HDMI audio stutter

[Bruce Griffith]

Bruce Griffith ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6822

-gerrit

commit 3dc51dca2474f233569fe846e29078bbbc7ffd1f
Author: Bruce Griffith 
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 
---
 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 */