Patch set updated for coreboot: df5ab13 mainboard/*/*/ec.c: Do not include `chromeos/chromeos.h`
Paul Menzel (paulepanter@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@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@gmail.com Signed-off-by: Paul Menzel paulepanter@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>
-
Paul Menzel