GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

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)
This commit is contained in:
Timothy Pearson 2015-01-23 20:27:26 -06:00 committed by Alexandru Gagniuc
parent 8d9492e536
commit c139c42f3f
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/northbridge/amd/amdht/h3finit.c
View File

@ -1,6 +1,7 @@
/* /*
* This file is part of the coreboot project. * 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. * Copyright (C) 2007 Advanced Micro Devices, Inc.
* *
* This program is free software; you can redistribute it and/or modify * 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)) && pDat->HtBlock->AMD_CB_ManualBUIDSwapList(node, link, &pSwapPtr))
{ {
/* Manual non-coherent BUID assignment */ /* Manual non-coherent BUID assignment */
currentBUID = 1;
/* Assign BUID's per manual override */ /* Assign BUID's per manual override */
while (*pSwapPtr != 0xFF) while (*pSwapPtr != 0xFF)