[coreboot-gerrit] Change in coreboot[master]: mb/google/kahlee: Move ec.h into variant include directories

Martin Roth (Code Review) gerrit at coreboot.org
Fri Nov 17 07:17:37 CET 2017


Martin Roth has uploaded this change for review. ( https://review.coreboot.org/22497


Change subject: mb/google/kahlee: Move ec.h into variant include directories
......................................................................

mb/google/kahlee: Move ec.h into variant include directories

The mainboard directory is included through the PI makefile - most
mainboard directories aren't in the include path at all.  Move the
ec.h file into the baseboard/variant directory that is already in
the include path.

BUG=b:69220826
TEST=Build

Change-Id: I89d361b700c66ba576de724927574fdab9461fc6
Signed-off-by: Martin Roth <martinroth at google.com>
---
M src/mainboard/google/kahlee/bootblock/bootblock.c
M src/mainboard/google/kahlee/dsdt.asl
M src/mainboard/google/kahlee/ec.c
M src/mainboard/google/kahlee/mainboard.c
M src/mainboard/google/kahlee/smihandler.c
R src/mainboard/google/kahlee/variants/baseboard/include/baseboard/ec.h
A src/mainboard/google/kahlee/variants/grunt/include/variant/ec.h
A src/mainboard/google/kahlee/variants/kahlee/include/variant/ec.h
8 files changed, 7 insertions(+), 5 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/97/22497/1

diff --git a/src/mainboard/google/kahlee/bootblock/bootblock.c b/src/mainboard/google/kahlee/bootblock/bootblock.c
index 3041a8f..90c8acb 100644
--- a/src/mainboard/google/kahlee/bootblock/bootblock.c
+++ b/src/mainboard/google/kahlee/bootblock/bootblock.c
@@ -14,8 +14,8 @@
  */
 
 #include <bootblock_common.h>
-#include <ec.h>
 #include <soc/southbridge.h>
+#include <variant/ec.h>
 
 void bootblock_mainboard_init(void)
 {
diff --git a/src/mainboard/google/kahlee/dsdt.asl b/src/mainboard/google/kahlee/dsdt.asl
index 708219a..885dd8d 100644
--- a/src/mainboard/google/kahlee/dsdt.asl
+++ b/src/mainboard/google/kahlee/dsdt.asl
@@ -13,7 +13,7 @@
  * GNU General Public License for more details.
  */
 
-#include "ec.h"
+#include <variant/ec.h>
 
 /* DefinitionBlock Statement */
 DefinitionBlock (
diff --git a/src/mainboard/google/kahlee/ec.c b/src/mainboard/google/kahlee/ec.c
index 0d1dae2..3f309eb 100644
--- a/src/mainboard/google/kahlee/ec.c
+++ b/src/mainboard/google/kahlee/ec.c
@@ -16,9 +16,9 @@
 #include <arch/acpi.h>
 #include <console/console.h>
 #include <ec/google/chromeec/ec.h>
-#include "ec.h"
 #include <rules.h>
 #include <soc/southbridge.h>
+#include <variant/ec.h>
 
 static void ramstage_ec_init(void)
 {
diff --git a/src/mainboard/google/kahlee/mainboard.c b/src/mainboard/google/kahlee/mainboard.c
index e21aa02..643bda1 100644
--- a/src/mainboard/google/kahlee/mainboard.c
+++ b/src/mainboard/google/kahlee/mainboard.c
@@ -19,10 +19,10 @@
 #include <agesawrapper.h>
 #include <amd_pci_util.h>
 #include <cbmem.h>
-#include <ec.h>
 #include <baseboard/variants.h>
 #include <soc/nvs.h>
 #include <soc/smi.h>
+#include <variant/ec.h>
 #include <variant/thermal.h>
 #include <vendorcode/google/chromeos/chromeos.h>
 
diff --git a/src/mainboard/google/kahlee/smihandler.c b/src/mainboard/google/kahlee/smihandler.c
index f1c66a6..6d43fbf 100644
--- a/src/mainboard/google/kahlee/smihandler.c
+++ b/src/mainboard/google/kahlee/smihandler.c
@@ -18,7 +18,7 @@
 #include <cpu/x86/smm.h>
 #include <ec/google/chromeec/smm.h>
 #include <soc/smi.h>
-#include "ec.h"
+#include <variant/ec.h>
 
 void mainboard_smi_sleep(u8 slp_typ)
 {
diff --git a/src/mainboard/google/kahlee/ec.h b/src/mainboard/google/kahlee/variants/baseboard/include/baseboard/ec.h
similarity index 100%
rename from src/mainboard/google/kahlee/ec.h
rename to src/mainboard/google/kahlee/variants/baseboard/include/baseboard/ec.h
diff --git a/src/mainboard/google/kahlee/variants/grunt/include/variant/ec.h b/src/mainboard/google/kahlee/variants/grunt/include/variant/ec.h
new file mode 100644
index 0000000..294a48d
--- /dev/null
+++ b/src/mainboard/google/kahlee/variants/grunt/include/variant/ec.h
@@ -0,0 +1 @@
+#include <baseboard/ec.h>
diff --git a/src/mainboard/google/kahlee/variants/kahlee/include/variant/ec.h b/src/mainboard/google/kahlee/variants/kahlee/include/variant/ec.h
new file mode 100644
index 0000000..294a48d
--- /dev/null
+++ b/src/mainboard/google/kahlee/variants/kahlee/include/variant/ec.h
@@ -0,0 +1 @@
+#include <baseboard/ec.h>

-- 
To view, visit https://review.coreboot.org/22497
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I89d361b700c66ba576de724927574fdab9461fc6
Gerrit-Change-Number: 22497
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth at google.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20171117/1339f64d/attachment-0001.html>


More information about the coreboot-gerrit mailing list