From 267d086a0835f8fae8414cd91350fd19627564dc Mon Sep 17 00:00:00 2001 From: Tristan Corrick Date: Wed, 10 May 2017 22:31:14 +1200 Subject: [PATCH] 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. Change-Id: I8d2f4f1fe2f17aa44c0a7090c178eee418defe78 Signed-off-by: Tristan Corrick Reviewed-on: https://review.coreboot.org/19651 Tested-by: build bot (Jenkins) Reviewed-by: Arthur Heymans Reviewed-by: Nico Huber Reviewed-by: Paul Menzel --- .../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] = {