diff options
| author | Elyes HAOUAS <ehaouas@noos.fr> | 2018-05-09 19:06:46 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-05-14 22:25:02 +0000 |
| commit | 9749a85cb0aea99818a7b0047cbb04fa0a60e07c (patch) | |
| tree | d7b9ad711b9c3ba9e42aeb72c6d7e526857628b0 | |
| parent | 60fd684698e7da894d539936307aea94d0c83bd1 (diff) | |
nb/intel/i945/raminit.c: Remove not necessary braces {}
Braces {} are not necessary for single statement blocks.
Change-Id: I2a2d8672fe3f53450dcfa53dc127b89b4aa6b75e
Signed-off-by: Elyes HAOUAS <ehaouas@noos.fr>
Reviewed-on: https://review.coreboot.org/26201
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Felix Held <felix-coreboot@felixheld.de>
| -rw-r--r-- | src/northbridge/intel/i945/raminit.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/northbridge/intel/i945/raminit.c b/src/northbridge/intel/i945/raminit.c index b2348f5c07..483d5d3d70 100644 --- a/src/northbridge/intel/i945/raminit.c +++ b/src/northbridge/intel/i945/raminit.c @@ -504,14 +504,12 @@ static void gather_common_timing(struct sys_info *sysinfo, } dimm_mask |= (1 << i); } - if (!dimm_mask) { + if (!dimm_mask) die("No memory installed.\n"); - } - if (!(dimm_mask & ((1 << DIMM_SOCKETS) - 1))) { + if (!(dimm_mask & ((1 << DIMM_SOCKETS) - 1))) /* Possibly does not boot in this case */ printk(BIOS_INFO, "Channel 0 has no memory populated.\n"); - } } static void choose_tclk(struct sys_info *sysinfo, |