From 8fda04449f5ff89af6fd2a7b1ca071806bc7def7 Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Sat, 12 Mar 2016 12:12:14 -0800 Subject: [PATCH] libpayload: Drop CONFIG_LP_CHROMEOS This is adding complexity to the code more than it saves space, plus some of the tables could potentially be interesting outside of the ChromeOS context. Change-Id: I4bf24608f3e26d3b7871a5031ae8f03bc2c8c21f Signed-off-by: Stefan Reinauer Reviewed-on: https://review.coreboot.org/14070 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- payloads/libpayload/Kconfig | 6 ------ payloads/libpayload/configs/config.arm64-generic | 1 - payloads/libpayload/configs/config.purin | 1 - payloads/libpayload/configs/config.veyron | 1 - payloads/libpayload/configs/defconfig | 1 - payloads/libpayload/configs/defconfig-arm | 1 - payloads/libpayload/configs/defconfig-mips | 1 - payloads/libpayload/configs/defconfig-tinycurses | 1 - payloads/libpayload/include/sysinfo.h | 6 ------ payloads/libpayload/libc/coreboot.c | 5 ----- 10 files changed, 24 deletions(-) diff --git a/payloads/libpayload/Kconfig b/payloads/libpayload/Kconfig index 95bd045a38..f2cb90a729 100644 --- a/payloads/libpayload/Kconfig +++ b/payloads/libpayload/Kconfig @@ -80,12 +80,6 @@ config REMOTEGDB help Enable Remote GDB debugging support. -config CHROMEOS - bool "ChromeOS specific features" - default n - help - Enable ChromeOS specific features. - endmenu menu "Architecture Options" diff --git a/payloads/libpayload/configs/config.arm64-generic b/payloads/libpayload/configs/config.arm64-generic index cca00b3e3f..a2d596c562 100644 --- a/payloads/libpayload/configs/config.arm64-generic +++ b/payloads/libpayload/configs/config.arm64-generic @@ -11,7 +11,6 @@ # CONFIG_LP_DEVELOPER is not set CONFIG_LP_COMPILER_GCC=y # CONFIG_LP_COMPILER_LLVM_CLANG is not set -CONFIG_LP_CHROMEOS=y # # Architecture Options diff --git a/payloads/libpayload/configs/config.purin b/payloads/libpayload/configs/config.purin index adb93b161b..c4f225b4d8 100644 --- a/payloads/libpayload/configs/config.purin +++ b/payloads/libpayload/configs/config.purin @@ -12,7 +12,6 @@ CONFIG_LP_GPL=y CONFIG_LP_COMPILER_GCC=y # CONFIG_LP_COMPILER_LLVM_CLANG is not set # CONFIG_LP_REMOTEGDB is not set -CONFIG_LP_CHROMEOS=y # # Architecture Options diff --git a/payloads/libpayload/configs/config.veyron b/payloads/libpayload/configs/config.veyron index aca5817dc8..47d54ead8b 100644 --- a/payloads/libpayload/configs/config.veyron +++ b/payloads/libpayload/configs/config.veyron @@ -12,7 +12,6 @@ CONFIG_LP_GPL=y CONFIG_LP_COMPILER_GCC=y # CONFIG_LP_COMPILER_LLVM_CLANG is not set # CONFIG_LP_REMOTEGDB is not set -CONFIG_LP_CHROMEOS=y # # Architecture Options diff --git a/payloads/libpayload/configs/defconfig b/payloads/libpayload/configs/defconfig index 1d516f7709..53320e7705 100644 --- a/payloads/libpayload/configs/defconfig +++ b/payloads/libpayload/configs/defconfig @@ -11,7 +11,6 @@ # CONFIG_LP_DEVELOPER is not set CONFIG_LP_COMPILER_GCC=y # CONFIG_LP_COMPILER_LLVM_CLANG is not set -# CONFIG_LP_CHROMEOS is not set # # Architecture Options diff --git a/payloads/libpayload/configs/defconfig-arm b/payloads/libpayload/configs/defconfig-arm index fa7e4f69c5..5b0ebe6a06 100644 --- a/payloads/libpayload/configs/defconfig-arm +++ b/payloads/libpayload/configs/defconfig-arm @@ -11,7 +11,6 @@ # CONFIG_LP_DEVELOPER is not set CONFIG_LP_COMPILER_GCC=y # CONFIG_LP_COMPILER_LLVM_CLANG is not set -# CONFIG_LP_CHROMEOS is not set # # Architecture Options diff --git a/payloads/libpayload/configs/defconfig-mips b/payloads/libpayload/configs/defconfig-mips index 0b05f872f0..5b7877787e 100644 --- a/payloads/libpayload/configs/defconfig-mips +++ b/payloads/libpayload/configs/defconfig-mips @@ -11,7 +11,6 @@ # CONFIG_LP_DEVELOPER is not set CONFIG_LP_COMPILER_GCC=y # CONFIG_LP_COMPILER_LLVM_CLANG is not set -# CONFIG_LP_CHROMEOS is not set # # Architecture Options diff --git a/payloads/libpayload/configs/defconfig-tinycurses b/payloads/libpayload/configs/defconfig-tinycurses index 7e74fb29ac..ad0a8cfb08 100644 --- a/payloads/libpayload/configs/defconfig-tinycurses +++ b/payloads/libpayload/configs/defconfig-tinycurses @@ -11,7 +11,6 @@ # CONFIG_LP_DEVELOPER is not set CONFIG_LP_COMPILER_GCC=y # CONFIG_LP_COMPILER_LLVM_CLANG is not set -# CONFIG_LP_CHROMEOS is not set # # Architecture Options diff --git a/payloads/libpayload/include/sysinfo.h b/payloads/libpayload/include/sysinfo.h index b97ae14771..2110d8803f 100644 --- a/payloads/libpayload/include/sysinfo.h +++ b/payloads/libpayload/include/sysinfo.h @@ -66,10 +66,8 @@ struct sysinfo_t { u32 cmos_range_start; u32 cmos_range_end; u32 cmos_checksum_location; -#if IS_ENABLED(CONFIG_LP_CHROMEOS) u32 vbnv_start; u32 vbnv_size; -#endif char *version; char *extra_version; @@ -86,25 +84,21 @@ struct sysinfo_t { struct cb_framebuffer *framebuffer; -#if IS_ENABLED(CONFIG_LP_CHROMEOS) int num_gpios; struct cb_gpio gpios[SYSINFO_MAX_GPIOS]; int num_macs; struct mac_address macs[SYSINFO_MAX_MACS]; char *serialno; -#endif unsigned long *mbtable; /** Pointer to the multiboot table */ struct cb_header *header; struct cb_mainboard *mainboard; -#if IS_ENABLED(CONFIG_LP_CHROMEOS) void *vboot_handoff; u32 vboot_handoff_size; void *vdat_addr; u32 vdat_size; -#endif #if IS_ENABLED(CONFIG_LP_ARCH_X86) int x86_rom_var_mtrr_index; diff --git a/payloads/libpayload/libc/coreboot.c b/payloads/libpayload/libc/coreboot.c index 3abd610be2..7ad8e066d4 100644 --- a/payloads/libpayload/libc/coreboot.c +++ b/payloads/libpayload/libc/coreboot.c @@ -78,7 +78,6 @@ static void cb_parse_serial(void *ptr, struct sysinfo_t *info) info->serial = ((struct cb_serial *)ptr); } -#if IS_ENABLED(CONFIG_LP_CHROMEOS) static void cb_parse_vboot_handoff(unsigned char *ptr, struct sysinfo_t *info) { struct lb_range *vbho = (struct lb_range *)ptr; @@ -128,8 +127,6 @@ static void cb_parse_mac_addresses(unsigned char *ptr, info->macs[i] = macs->mac_addrs[i]; } -#endif - static void cb_parse_tstamp(unsigned char *ptr, struct sysinfo_t *info) { struct cb_cbmem_tab *const cbmem = (struct cb_cbmem_tab *)ptr; @@ -349,7 +346,6 @@ int cb_parse_header(void *addr, int len, struct sysinfo_t *info) case CB_TAG_MAINBOARD: info->mainboard = (struct cb_mainboard *)ptr; break; -#if IS_ENABLED(CONFIG_LP_CHROMEOS) case CB_TAG_GPIO: cb_parse_gpios(ptr, info); break; @@ -368,7 +364,6 @@ int cb_parse_header(void *addr, int len, struct sysinfo_t *info) case CB_TAG_SERIALNO: cb_parse_string(ptr, &info->serialno); break; -#endif case CB_TAG_TIMESTAMPS: cb_parse_tstamp(ptr, info); break;