Patch set updated for coreboot: amd/mct/ddr3: Fix unintended sign extension warning

[Timothy Pearson]

Timothy Pearson ([e-mail]) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18077

-gerrit

commit 6700c9fd63053e49c4089aa96d7825b2867dc672
Author: Timothy Pearson 
Date:   Mon Jan 9 17:40:21 2017 -0600

amd/mct/ddr3: Fix unintended sign extension warning

An unintended sign extension warning was thrown by Coverity.
    Explicitly state the unsigned nature of num_nodes.

Found-by: Coverity Scan #1347342
    Change-Id: Icd42eec13be04fc5fd2ffc85320cbadafc852148
    Signed-off-by: Timothy Pearson 
---
 src/northbridge/amd/amdmct/mct_ddr3/mct_d.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
index f726345..8769a7d 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
@@ -2474,7 +2474,7 @@ 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 -= (0x1000000 * (uint8_t)num_nodes);
    	else
    		/* Move upper limit down by 16M */
    		max_range_limit -= 0x1000000;