diff options
| author | Timothy Pearson <tpearson@raptorengineering.com> | 2017-01-09 17:40:21 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2017-01-11 17:14:54 +0100 |
| commit | 88a2e3b3bf7b622cd3bef941a3e3f71337644a39 (patch) | |
| tree | f3eab026408e848ce02ef086b2461a1ad8de8b3c /src/northbridge/amd | |
| parent | 590a3e1f6ccce873bb8f2129dc3680cab12e5a42 (diff) | |
| download | coreboot-88a2e3b3bf7b622cd3bef941a3e3f71337644a39.tar.xz | |
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>
Diffstat (limited to 'src/northbridge/amd')
| -rw-r--r-- | src/northbridge/amd/amdmct/mct_ddr3/mct_d.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c index f726345f00..b64ad54711 100644 --- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c +++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c @@ -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); |