diff options
| author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-04-15 14:32:53 +1000 |
|---|---|---|
| committer | Alexandru Gagniuc <mr.nuke.me@gmail.com> | 2014-04-20 17:43:34 +0200 |
| commit | 5c97142419a7b7204786fea5b785c23a3bb41836 (patch) | |
| tree | 0c77638f91b5a7b75b0860e47bef1ecc5143fe62 | |
| parent | 3bc6eb544ccfcd2f338652ca109442a8b8a01161 (diff) | |
| download | coreboot-5c97142419a7b7204786fea5b785c23a3bb41836.tar.xz | |
drivers/elog: Fix implicit function declaration issue
Fix compilation. Relying on the pre-processor to condition an if
statement will lead to warnings of implicitly defined functions. To
solve this dilemma add symbols to resolve to at compile time.
Change-Id: Id0117528c5579cc1dec750a8a17a76fab4314b3f
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/5504
Tested-by: build bot (Jenkins)
Reviewed-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
| -rw-r--r-- | src/include/elog.h | 4 | ||||
| -rw-r--r-- | src/include/pc80/mc146818rtc.h | 2 |
2 files changed, 5 insertions, 1 deletions
diff --git a/src/include/elog.h b/src/include/elog.h index f9b5d53941..eb8352dd3c 100644 --- a/src/include/elog.h +++ b/src/include/elog.h @@ -162,7 +162,9 @@ extern u32 gsmi_exec(u8 command, u32 *param); #if CONFIG_ELOG_BOOT_COUNT u32 boot_count_read(void); u32 boot_count_increment(void); -#endif +#else +static inline u32 boot_count_read(void) { return 0; } +#endif /* CONFIG_ELOG_BOOT_COUNT */ #endif /* !CONFIG_ELOG */ diff --git a/src/include/pc80/mc146818rtc.h b/src/include/pc80/mc146818rtc.h index fd403087a6..cf4a9514c9 100644 --- a/src/include/pc80/mc146818rtc.h +++ b/src/include/pc80/mc146818rtc.h @@ -220,6 +220,8 @@ static inline enum cb_err get_option(void *dest __attribute__((unused)), #define CMOS_POST_EXTRA_DEV_PATH 0x01 void cmos_post_log(void); +#else +static inline void cmos_post_log(void) {} #endif /* CONFIG_CMOS_POST */ #endif /* PC80_MC146818RTC_H */ |