[coreboot-gerrit] Patch set updated for coreboot: mainboard/google/reef: declare mainboard_ec_init() in each C file

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Sat Sep 3 02:33:15 CEST 2016


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16420

-gerrit

commit d950498f0c4cca892890e4d7f7d427efd50224f1
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Fri Sep 2 17:25:35 2016 -0500

    mainboard/google/reef: declare mainboard_ec_init() in each C file
    
    There's no common EC header file in the code base, and I didn't
    want to use a header file for single declaration. Therefore,
    just move the declaration to each file that uses that symbol.
    
    BUG=chrome-os-partner:56677
    
    Change-Id: Ibaebb0ea6a07029aec02d5185cf05ffb8593b117
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/mainboard/google/reef/bootblock.c                               | 2 ++
 src/mainboard/google/reef/ec.c                                      | 2 ++
 src/mainboard/google/reef/mainboard.c                               | 2 ++
 src/mainboard/google/reef/variants/baseboard/include/baseboard/ec.h | 4 ----
 4 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/mainboard/google/reef/bootblock.c b/src/mainboard/google/reef/bootblock.c
index ab010e2..da66009 100644
--- a/src/mainboard/google/reef/bootblock.c
+++ b/src/mainboard/google/reef/bootblock.c
@@ -19,6 +19,8 @@
 #include <soc/gpio.h>
 #include <variant/ec.h>
 
+void mainboard_ec_init(void);
+
 void bootblock_mainboard_init(void)
 {
 	const struct pad_config *pads;
diff --git a/src/mainboard/google/reef/ec.c b/src/mainboard/google/reef/ec.c
index 612708e..e1832fd 100644
--- a/src/mainboard/google/reef/ec.c
+++ b/src/mainboard/google/reef/ec.c
@@ -20,6 +20,8 @@
 #include <soc/lpc.h>
 #include <variant/ec.h>
 
+void mainboard_ec_init(void);
+
 static void ramstage_ec_init(void)
 {
 	printk(BIOS_ERR, "mainboard: EC init\n");
diff --git a/src/mainboard/google/reef/mainboard.c b/src/mainboard/google/reef/mainboard.c
index 7d5a7e1..a2421df 100644
--- a/src/mainboard/google/reef/mainboard.c
+++ b/src/mainboard/google/reef/mainboard.c
@@ -25,6 +25,8 @@
 #include <variant/ec.h>
 #include <variant/gpio.h>
 
+void mainboard_ec_init(void);
+
 static void mainboard_init(void *chip_info)
 {
 	int boardid;
diff --git a/src/mainboard/google/reef/variants/baseboard/include/baseboard/ec.h b/src/mainboard/google/reef/variants/baseboard/include/baseboard/ec.h
index 7763630..5c8e306 100644
--- a/src/mainboard/google/reef/variants/baseboard/include/baseboard/ec.h
+++ b/src/mainboard/google/reef/variants/baseboard/include/baseboard/ec.h
@@ -70,8 +70,4 @@
 	 EC_HOST_EVENT_MASK(EC_HOST_EVENT_BATTERY_SHUTDOWN)|\
 	 EC_HOST_EVENT_MASK(EC_HOST_EVENT_PANIC))
 
-#ifndef __ACPI__
-extern void mainboard_ec_init(void);
-#endif
-
 #endif



More information about the coreboot-gerrit mailing list