summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTimothy Pearson <tpearson@raptorengineeringinc.com>2015-01-23 20:27:26 -0600
committerAlexandru Gagniuc <mr.nuke.me@gmail.com>2015-01-28 01:03:44 +0100
commitc139c42f3f37d8910893e8421bc3a665d99c8bdc (patch)
treebf0453443d110e3297b25019744aa96b427eebbc
parent8d9492e5364eb8a77b0f6024ead14a443d57d44d (diff)
downloadcoreboot-c139c42f3f37d8910893e8421bc3a665d99c8bdc.tar.xz
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@raptorengineeringinc.com> Reviewed-on: http://review.coreboot.org/8267 Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Reviewed-by: Alexandru Gagniuc <mr.nuke.me@gmail.com> Tested-by: build bot (Jenkins)
-rw-r--r--src/northbridge/amd/amdht/h3finit.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/northbridge/amd/amdht/h3finit.c b/src/northbridge/amd/amdht/h3finit.c
index ff57033031..88d5b6a43b 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@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)