GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

console/post: be explicit about conditional cmos_post_log() compiling 

The current code was using !__PRE_RAM__ as a proxy for ramstage
conditional compilation. In the face of postcar stage not defining
__PRE_RAM__ (because it's after RAM is up) these code paths
can fail to compile with a __SIMPLE_DEVICE__ defined for the entire
stage. Remedy the current situation by just compiling explicity for
ramstage because that was the original intent. In the future,
the __SIMPLE_DEVICE__ selection for postcar can also be re-evaluated.

Change-Id: I0f887f1e45f0cf5c235ae5144eaa227921e7119b
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://review.coreboot.org/14958
Tested-by: build bot (Jenkins)
Reviewed-by: Furquan Shaikh <furquan@google.com>
This commit is contained in:
Aaron Durbin 2016-05-24 14:47:10 -05:00
parent 277279343f
commit 9acd39d95e
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/console/post.c
View File

@ -20,6 +20,7 @@
#include <device/device.h> #include <device/device.h>
#include <pc80/mc146818rtc.h> #include <pc80/mc146818rtc.h>
#include <smp/spinlock.h> #include <smp/spinlock.h>
#include <rules.h>
/* Write POST information */ /* Write POST information */
@ -41,7 +42,7 @@ void __attribute__((weak)) mainboard_post(uint8_t value)
DECLARE_SPIN_LOCK(cmos_post_lock) DECLARE_SPIN_LOCK(cmos_post_lock)
#if !defined(__PRE_RAM__) #if ENV_RAMSTAGE
void cmos_post_log(void) void cmos_post_log(void)
{ {
u8 code = 0; u8 code = 0;
@ -122,7 +123,7 @@ void post_log_clear(void)
post_log_extra(0); post_log_extra(0);
} }
#endif /* CONFIG_CMOS_POST_EXTRA */ #endif /* CONFIG_CMOS_POST_EXTRA */
#endif /* !__PRE_RAM__ */ #endif /* ENV_RAMSTAGE */
static void cmos_post_code(u8 value) static void cmos_post_code(u8 value)
{ {