[coreboot-gerrit] Patch set updated for coreboot: df5ab13 mainboard/*/*/ec.c: Do not include `chromeos/chromeos.h`

[Paul Menzel (paulepanter@users.sourceforge.net)]

Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5398

-gerrit

commit df5ab133b7e692650bb936c2f9d2b8f1f4f82e85
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sun Mar 23 00:09:32 2014 +0100

    mainboard/*/*/ec.c: Do not include `chromeos/chromeos.h`
    
    When not selecting the Kconfig option CHROMEOS, compilation of the board
    SAMSUNG Lumpy fails as the included header file uses `chromeos_acpi_t`,
    which is not known to the compiler when not building with support for
    Chrome OS.
    
    	In file included from src/mainboard/samsung/lumpy/ec.c:21:0:
    	src/vendorcode/google/chromeos/chromeos.h:65:43: error: unknown type name 'chromeos_acpi_t'
    
    Luckily, nothing from the header `chromeos.h` is used in `ec.c`, which
    was probably the case for the board the file was copied from, so do
    not include it at all.
    
    Looking further the header is included since commit
    
        6651da3b Add support for Intel Emerald Lake 2 CRB
    
    and is not needed there either and was copied over from then on. So
    remove it for all boards using the following command.
    
    	$ find src/mainboard/ -name 'ec.c' | xargs sed -i '/chromeos.h/d'
    
    Revert the change for Google Stout as `get_recovery_mode_switch()` is
    declared in `chromeos/chromeos.h`.
    
    Change-Id: I7923717bfc5c84698044008e5f2441206041e0dd
    Reported-by: Idwer Vollering <vidwer at gmail.com>
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/mainboard/google/bolt/ec.c        | 1 -
 src/mainboard/google/falco/ec.c       | 1 -
 src/mainboard/google/link/ec.c        | 1 -
 src/mainboard/google/parrot/ec.c      | 1 -
 src/mainboard/google/peppy/ec.c       | 1 -
 src/mainboard/google/rambi/ec.c       | 1 -
 src/mainboard/google/slippy/ec.c      | 1 -
 src/mainboard/intel/emeraldlake2/ec.c | 1 -
 src/mainboard/samsung/lumpy/ec.c      | 1 -
 9 files changed, 9 deletions(-)

diff --git a/src/mainboard/google/bolt/ec.c b/src/mainboard/google/bolt/ec.c
index 0919f0f..04a9931 100644
--- a/src/mainboard/google/bolt/ec.c
+++ b/src/mainboard/google/bolt/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/google/chromeec/ec.h>
diff --git a/src/mainboard/google/falco/ec.c b/src/mainboard/google/falco/ec.c
index 0919f0f..04a9931 100644
--- a/src/mainboard/google/falco/ec.c
+++ b/src/mainboard/google/falco/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/google/chromeec/ec.h>
diff --git a/src/mainboard/google/link/ec.c b/src/mainboard/google/link/ec.c
index 07da080..7dfadeb 100644
--- a/src/mainboard/google/link/ec.c
+++ b/src/mainboard/google/link/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/google/chromeec/ec.h>
diff --git a/src/mainboard/google/parrot/ec.c b/src/mainboard/google/parrot/ec.c
index e6d2d38..56f7d80 100644
--- a/src/mainboard/google/parrot/ec.c
+++ b/src/mainboard/google/parrot/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <device/device.h>
diff --git a/src/mainboard/google/peppy/ec.c b/src/mainboard/google/peppy/ec.c
index 0919f0f..04a9931 100644
--- a/src/mainboard/google/peppy/ec.c
+++ b/src/mainboard/google/peppy/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/google/chromeec/ec.h>
diff --git a/src/mainboard/google/rambi/ec.c b/src/mainboard/google/rambi/ec.c
index 0919f0f..04a9931 100644
--- a/src/mainboard/google/rambi/ec.c
+++ b/src/mainboard/google/rambi/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/google/chromeec/ec.h>
diff --git a/src/mainboard/google/slippy/ec.c b/src/mainboard/google/slippy/ec.c
index 0919f0f..04a9931 100644
--- a/src/mainboard/google/slippy/ec.c
+++ b/src/mainboard/google/slippy/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/google/chromeec/ec.h>
diff --git a/src/mainboard/intel/emeraldlake2/ec.c b/src/mainboard/intel/emeraldlake2/ec.c
index ac37b0b..b50ab65 100644
--- a/src/mainboard/intel/emeraldlake2/ec.c
+++ b/src/mainboard/intel/emeraldlake2/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/smsc/mec1308/ec.h>
diff --git a/src/mainboard/samsung/lumpy/ec.c b/src/mainboard/samsung/lumpy/ec.c
index 40d8319..2406012 100644
--- a/src/mainboard/samsung/lumpy/ec.c
+++ b/src/mainboard/samsung/lumpy/ec.c
@@ -18,7 +18,6 @@
  */
 
 #include <arch/acpi.h>
-#include <vendorcode/google/chromeos/chromeos.h>
 #include <types.h>
 #include <console/console.h>
 #include <ec/smsc/mec1308/ec.h>