The "temp" will be used later. So it has to be calculated correctly.
Comment by Peter, The variable name "temp" unfortunately does not explain what the value is. The commit message also does not have hints. Hopefully in the future it's possible to also use a brief moment to improve the clarity of the code, while it is already being fixed for some other reason. Ie. fixing up variable names, writing particularly informative commit messages, or of course both at the same time! :) Signed-off-by: Zheng Bao <zheng.bao@amd.com> Acked-by: Marc Jones <marcj303@gmail.com> git-svn-id: svn://svn.coreboot.org/coreboot/trunk@6517 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
parent
52ffb2b66d
commit
b18f9b0ff4
|
@ -565,7 +565,8 @@ static u8 fam10GetNumCoresOnNode(u8 node, cNorthBridge *nb)
|
||||||
|
|
||||||
/* bits[15,13,12] specify the cores */
|
/* bits[15,13,12] specify the cores */
|
||||||
/* Support Downcoring */
|
/* Support Downcoring */
|
||||||
cores = ((temp & 8) >> 1) + (temp & 3) + 1;
|
temp = ((temp & 8) >> 1) + (temp & 3);
|
||||||
|
cores = temp + 1;
|
||||||
AmdPCIReadBits (MAKE_SBDFO(makePCISegmentFromNode(node),
|
AmdPCIReadBits (MAKE_SBDFO(makePCISegmentFromNode(node),
|
||||||
makePCIBusFromNode(node),
|
makePCIBusFromNode(node),
|
||||||
makePCIDeviceFromNode(node),
|
makePCIDeviceFromNode(node),
|
||||||
|
|
Loading…
Reference in New Issue