[coreboot-gerrit] Patch set updated for coreboot: 34f5a4b drivers/elog: Fix implicit function declaration issue
Edward O'Callaghan (eocallaghan@alterapraxis.com)
gerrit at coreboot.org
Sun Apr 20 13:16:37 CEST 2014
Edward O'Callaghan (eocallaghan at alterapraxis.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5504
-gerrit
commit 34f5a4b220605716824645d1cffa210dc0dab614
Author: Edward O'Callaghan <eocallaghan at alterapraxis.com>
Date: Tue Apr 15 14:32:53 2014 +1000
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 at alterapraxis.com>
---
src/include/elog.h | 4 +++-
src/include/pc80/mc146818rtc.h | 2 ++
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/include/elog.h b/src/include/elog.h
index f9b5d53..eb8352d 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 fd40308..cf4a951 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 */
More information about the coreboot-gerrit
mailing list