From 60a8f8df41eb99371e3f6c93533cd66931192129 Mon Sep 17 00:00:00 2001 From: Tristan Corrick Date: Wed, 10 May 2017 22:31:14 +1200 Subject: [PATCH] UPSTREAM: nb/intel/gm45: Fix some errors/warnings given by checkpatch This results in raminit_receive_enable_calibration.c producing no errors or warnings with checkpatch. The issues fixed are: ERROR: that open brace { should be on the previous line WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Tested by compiling after making the changes. BUG=none BRANCH=none TEST=none Change-Id: I4aec07bcaba5721297f432194310163d75925d7d Signed-off-by: Patrick Georgi Original-Commit-Id: 267d086a0835f8fae8414cd91350fd19627564dc Original-Change-Id: I8d2f4f1fe2f17aa44c0a7090c178eee418defe78 Original-Signed-off-by: Tristan Corrick Original-Reviewed-on: https://review.coreboot.org/19651 Original-Tested-by: build bot (Jenkins) Original-Reviewed-by: Arthur Heymans Original-Reviewed-by: Nico Huber Original-Reviewed-by: Paul Menzel Reviewed-on: https://chromium-review.googlesource.com/506209 Commit-Ready: Patrick Georgi Tested-by: Patrick Georgi Reviewed-by: Patrick Georgi --- .../intel/gm45/raminit_receive_enable_calibration.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/northbridge/intel/gm45/raminit_receive_enable_calibration.c b/src/northbridge/intel/gm45/raminit_receive_enable_calibration.c index 0d346bffe2..c5614e1192 100644 --- a/src/northbridge/intel/gm45/raminit_receive_enable_calibration.c +++ b/src/northbridge/intel/gm45/raminit_receive_enable_calibration.c @@ -209,11 +209,14 @@ static void receive_enable_calibration(const timings_t *const timings, /* F */{ { 0, 0 }, { 3, 3 }, { 6, 6 }, { 5, 5 } }, }; - const int cardF[] = - { dimms[0].card_type == 0xf, dimms[0].card_type == 0xf }; - const unsigned t_bound = + const int cardF[] = { + dimms[0].card_type == 0xf, + dimms[0].card_type == 0xf, + }; + + const unsigned int t_bound = (timings->mem_clock == MEM_CLOCK_1067MT) ? 9 : 12; - const unsigned p_bound = + const unsigned int p_bound = (timings->mem_clock == MEM_CLOCK_1067MT) ? 8 : 1; rec_timing_t rec_timings[2][4] = {