summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTimothy Pearson <tpearson@raptorengineering.com>2017-01-09 14:10:25 -0600
committerTimothy Pearson <tpearson@raptorengineering.com>2017-01-10 21:07:55 +0100
commit5153cbfeb3498b75efdde6c7aa312d5920541702 (patch)
tree65d1db9dace5ffd17f8e837a1f7860c2cd68080d
parentcf1cb5b2d4f528e7eab55ee9393cf72016bac888 (diff)
downloadcoreboot-5153cbfeb3498b75efdde6c7aa312d5920541702.tar.xz
amd/mct/ddr3: Allow critical delay delta to go negative
The critical delay delta was incorrectly specified as an unsigned short. Use a signed short instead. Found-by: Coverity Scan #1347355 Change-Id: I37d769afb8c8af85a0375ae459e9d4ab0adcca74 Signed-off-by: Timothy Pearson <tpearson@raptorengineering.com> Reviewed-on: https://review.coreboot.org/18071 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth <martinroth@google.com> Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net> Tested-by: Raptor Engineering Automated Test Stand <noreply@raptorengineeringinc.com>
-rw-r--r--src/northbridge/amd/amdmct/mct_ddr3/mct_d.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
index 0892857e68..7b9fa6555a 100644
--- a/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
+++ b/src/northbridge/amd/amdmct/mct_ddr3/mct_d.c
@@ -3028,7 +3028,7 @@ void fam15EnableTrainingMode(struct MCTStatStruc *pMCTstat,
uint8_t lane;
uint8_t dimm;
uint16_t max_cdd_we_delta;
- uint16_t cdd_trwtto_we_delta;
+ int16_t cdd_trwtto_we_delta;
uint8_t receiver;
uint8_t lane_count;
uint8_t x4_present = 0;