Emulation/switch-coreboot
1
0
Fork 0
mirror of https://github.com/fail0verflow/switch-coreboot.git synced 2025-05-04 01:39:18 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

UPSTREAM: amd/mct/ddr3: Fix unintended sign extension warning

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

BUG=none
BRANCH=none
TEST=none

Change-Id: I27d4c25248395d9595df285b2e9e99e16f92dc18
Signed-off-by: Patrick Georgi <pgeorgi@chromium.org>
Original-Commit-Id: 88a2e3b3bf
Original-Found-by: Coverity Scan #1347342
Original-Change-Id: Icd42eec13be04fc5fd2ffc85320cbadafc852148
Original-Signed-off-by: Timothy Pearson <tpearson@raptorengineering.com>
Original-Reviewed-on: https://review.coreboot.org/18077
Original-Reviewed-by: Jonathan Neuschfer <j.neuschaefer@gmx.net>
Original-Tested-by: build bot (Jenkins)
Original-Reviewed-by: Nico Huber <nico.h@gmx.de>
Original-Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
Reviewed-on: https://chromium-review.googlesource.com/428250
Commit-Ready: Aaron Durbin <adurbin@chromium.org>
This commit is contained in:
Timothy Pearson 2017-01-09 17:40:21 -06:00 • committed by chrome-bot
parent 303afe6a03
commit aa6270a1d5
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);