diff options
| author | Tristan Corrick <tristancorrick86@gmail.com> | 2017-05-10 22:31:14 +1200 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2017-05-11 16:52:25 +0200 |
| commit | 267d086a0835f8fae8414cd91350fd19627564dc (patch) | |
| tree | 898543e6f2b8eff6fa83dc43cf07ca0926da2a90 /src/northbridge/intel/gm45/raminit_receive_enable_calibration.c | |
| parent | ca39df81957c45ae215e5423aabadf1e60299120 (diff) | |
| download | coreboot-267d086a0835f8fae8414cd91350fd19627564dc.tar.gz coreboot-267d086a0835f8fae8414cd91350fd19627564dc.tar.bz2 coreboot-267d086a0835f8fae8414cd91350fd19627564dc.zip | |
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 <tristancorrick86@gmail.com>
Reviewed-on: https://review.coreboot.org/19651
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Diffstat (limited to 'src/northbridge/intel/gm45/raminit_receive_enable_calibration.c')
| -rw-r--r-- | src/northbridge/intel/gm45/raminit_receive_enable_calibration.c | 11 |
1 files 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 0d346bffe20d..c5614e119286 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] = { |