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

amd/mct/ddr3: Fix unintended sign extension warning 

An unintended sign extension warning was thrown by Coverity.
Explicitly state the length of the constant multiplier.

Found-by: Coverity Scan #1347342
Change-Id: Icd42eec13be04fc5fd2ffc85320cbadafc852148
Signed-off-by: Timothy Pearson <tpearson@raptorengineering.com>
Reviewed-on: https://review.coreboot.org/18077
Reviewed-by: Jonathan Neuschäfer <j.neuschaefer@gmx.net>
Tested-by: build bot (Jenkins)
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
This commit is contained in:
Timothy Pearson 2017-01-09 17:40:21 -06:00 committed by Martin Roth
parent 590a3e1f6c
commit 88a2e3b3bf
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
View File

@ -2474,10 +2474,10 @@ static void set_up_cc6_storage_fam15(struct MCTStatStruc *pMCTstat,
if (interleaved)
/* Move upper limit down by 16M * the number of nodes */
max_range_limit -= (0x1000000 * num_nodes);
max_range_limit -= (0x1000000ULL * num_nodes);
else
/* Move upper limit down by 16M */
max_range_limit -= 0x1000000;
max_range_limit -= 0x1000000ULL;
printk(BIOS_INFO, "%s:\tnew max_range_limit: %16llx\n",
__func__, max_range_limit);