[coreboot-gerrit] New patch to review for coreboot: [Not For Merge] K8 northbridge: validate CONFIG_CDB value

Jonathan A. Kollasch (jakllsch@kollasch.net) gerrit at coreboot.org
Tue Jul 14 15:01:14 CEST 2015


Jonathan A. Kollasch (jakllsch at kollasch.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10915

-gerrit

commit a70caf9603047b071969e76f04168c1db42ada10
Author: Jonathan A. Kollasch <jakllsch at kollasch.net>
Date:   Tue Jul 14 07:59:45 2015 -0500

    [Not For Merge] K8 northbridge: validate CONFIG_CDB value
    
    Change-Id: Iecdb3891997149b2c76763214967b49c7bdf7104
    Signed-off-by: Jonathan A. Kollasch <jakllsch at kollasch.net>
---
 src/northbridge/amd/amdk8/northbridge.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/northbridge/amd/amdk8/northbridge.c b/src/northbridge/amd/amdk8/northbridge.c
index c4fd3fd..2514143 100644
--- a/src/northbridge/amd/amdk8/northbridge.c
+++ b/src/northbridge/amd/amdk8/northbridge.c
@@ -207,6 +207,10 @@ static void amdk8_scan_chain(struct bus *link)
 		sysconf.hcdn_reg[index] = link->hcdn_reg;
 }
 
+#if CONFIG_CDB != 0x18
+#error CONFIG_CDB is not 0x18
+#endif
+
 /* Do sb ht chain at first, in case s2885 put sb chain
  * (8131/8111) on link2, but put 8151 on link0.
  */



More information about the coreboot-gerrit mailing list