diff options
| author | Bill XIE <persmule@gmail.com> | 2017-09-25 15:54:04 +0800 |
|---|---|---|
| committer | Arthur Heymans <arthur@aheymans.xyz> | 2017-09-26 15:53:55 +0000 |
| commit | c8050f4bfcb781dbaf565684b21a4e04b1dc769e (patch) | |
| tree | bb1b15c1381837121d3d6056ac27733e3483e2a5 /src/device/dram/ddr2.c | |
| parent | a4ab665a556da24929ec1537901b38fbe9f4252b (diff) | |
| download | coreboot-c8050f4bfcb781dbaf565684b21a4e04b1dc769e.tar.xz | |
device/dram/ddr2.c: fix a hidden syntax error introduced earlier
A regression is hidden in commit 7eb0157fca33865783c1cc3c8e5cb2e327e551d7
(device/dram/ddr2.c: Decoding byte[12] bit7 as self refresh flag ), which
breaks the build procedure when CONFIG_DEBUG_RAM_SETUP is set.
Maybe it had better implement "printram" and the like as inline
functions instead of macros.
Change-Id: If956435bd0c39b1f3e722c2cfc48d2d0f35abc9b
Signed-off-by: Bill XIE <persmule@gmail.com>
Reviewed-on: https://review.coreboot.org/21673
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Diffstat (limited to 'src/device/dram/ddr2.c')
| -rw-r--r-- | src/device/dram/ddr2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/device/dram/ddr2.c b/src/device/dram/ddr2.c index d524238b22..e06e3423a8 100644 --- a/src/device/dram/ddr2.c +++ b/src/device/dram/ddr2.c @@ -547,7 +547,7 @@ int spd_decode_ddr2(struct dimm_attr_st *dimm, u8 spd[SPD_SIZE_MAX_DDR2]) ret = SPD_STATUS_INVALID_FIELD; dimm->flags.self_refresh = (spd[12] >> 7) & 1; printram("The assembly supports self refresh: %s\n", - dimm->flags.self_refresh ? "true", "false"); + dimm->flags.self_refresh ? "true" : "false"); /* Number of PLLs on DIMM */ if (dimm->rev >= 0x11) |