[coreboot-gerrit] Patch set updated for coreboot: 3c0a128 northbridge/amd/amdht: Initialize variable `currentBUID`

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Sat Jan 24 21:50:38 CET 2015


Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/8267

-gerrit

commit 3c0a128247ba038c48f881ce862778e470c1da09
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Fri Jan 23 20:27:26 2015 -0600

    northbridge/amd/amdht: Initialize variable `currentBUID`
    
    Fix uninitialized variable when manual non-coherent BUID
    selection is used.
    
    Change-Id: Id19745b29486aef5297fdbb3324ae36bf9b8f466
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/northbridge/amd/amdht/h3finit.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/northbridge/amd/amdht/h3finit.c b/src/northbridge/amd/amdht/h3finit.c
index ff57033..88d5b6a 100644
--- a/src/northbridge/amd/amdht/h3finit.c
+++ b/src/northbridge/amd/amdht/h3finit.c
@@ -1,6 +1,7 @@
 /*
  * This file is part of the coreboot project.
  *
+ * Copyright (C) 2015 Timothy Pearson <tpearson at raptorengineeringinc.com>, Raptor Engineering
  * Copyright (C) 2007 Advanced Micro Devices, Inc.
  *
  * This program is free software; you can redistribute it and/or modify
@@ -963,6 +964,7 @@ static void processLink(u8 node, u8 link, sMainData *pDat)
 	 && pDat->HtBlock->AMD_CB_ManualBUIDSwapList(node, link, &pSwapPtr))
 	{
 		/* Manual non-coherent BUID assignment */
+		currentBUID = 1;
 
 		/* Assign BUID's per manual override */
 		while (*pSwapPtr != 0xFF)



More information about the coreboot-gerrit mailing list