Message ID | 1438033652-30435-7-git-send-email-sjg@chromium.org |
---|---|
State | Accepted |
Delegated to: | Simon Glass |
Headers | show |
On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass <sjg@chromium.org> wrote: > Drop these from the header file and use Kconfig instead. > > Signed-off-by: Simon Glass <sjg@chromium.org> > --- > > configs/chromebook_link_defconfig | 3 +++ > configs/chromebox_panther_defconfig | 3 +++ > include/configs/x86-chromebook.h | 3 --- > 3 files changed, 6 insertions(+), 3 deletions(-) > > diff --git a/configs/chromebook_link_defconfig b/configs/chromebook_link_defconfig > index e394dab..b987f3f 100644 > --- a/configs/chromebook_link_defconfig > +++ b/configs/chromebook_link_defconfig > @@ -14,6 +14,9 @@ CONFIG_CMD_BOOTSTAGE=y > CONFIG_OF_CONTROL=y > CONFIG_DM_PCI=y > CONFIG_SPI_FLASH=y > +CONFIG_CMD_CROS_EC=y > +CONFIG_CROS_EC=y > +CONFIG_CROS_EC_LPC=y > CONFIG_VIDEO_VESA=y > CONFIG_FRAMEBUFFER_SET_VESA_MODE=y > CONFIG_FRAMEBUFFER_VESA_MODE_11A=y > diff --git a/configs/chromebox_panther_defconfig b/configs/chromebox_panther_defconfig > index 340510f..e82c8ec 100644 > --- a/configs/chromebox_panther_defconfig > +++ b/configs/chromebox_panther_defconfig > @@ -14,6 +14,9 @@ CONFIG_CMD_BOOTSTAGE=y > CONFIG_OF_CONTROL=y > CONFIG_DM_PCI=y > CONFIG_SPI_FLASH=y > +CONFIG_CMD_CROS_EC=y > +CONFIG_CROS_EC=y > +CONFIG_CROS_EC_LPC=y > CONFIG_VIDEO_VESA=y > CONFIG_FRAMEBUFFER_SET_VESA_MODE=y > CONFIG_FRAMEBUFFER_VESA_MODE_11A=y > diff --git a/include/configs/x86-chromebook.h b/include/configs/x86-chromebook.h > index 408cbb1..e8f0bac 100644 > --- a/include/configs/x86-chromebook.h > +++ b/include/configs/x86-chromebook.h > @@ -45,9 +45,6 @@ > #define VIDEO_IO_OFFSET 0 > #define CONFIG_X86EMU_RAW_IO > > -#define CONFIG_CROS_EC > -#define CONFIG_CROS_EC_LPC > -#define CONFIG_CMD_CROS_EC > #define CONFIG_ARCH_EARLY_INIT_R > > #undef CONFIG_ENV_IS_NOWHERE > -- Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
On 28 July 2015 at 01:48, Bin Meng <bmeng.cn@gmail.com> wrote: > On Tue, Jul 28, 2015 at 5:47 AM, Simon Glass <sjg@chromium.org> wrote: >> Drop these from the header file and use Kconfig instead. >> >> Signed-off-by: Simon Glass <sjg@chromium.org> >> --- >> >> configs/chromebook_link_defconfig | 3 +++ >> configs/chromebox_panther_defconfig | 3 +++ >> include/configs/x86-chromebook.h | 3 --- >> 3 files changed, 6 insertions(+), 3 deletions(-) >> >> diff --git a/configs/chromebook_link_defconfig b/configs/chromebook_link_defconfig >> index e394dab..b987f3f 100644 >> --- a/configs/chromebook_link_defconfig >> +++ b/configs/chromebook_link_defconfig >> @@ -14,6 +14,9 @@ CONFIG_CMD_BOOTSTAGE=y >> CONFIG_OF_CONTROL=y >> CONFIG_DM_PCI=y >> CONFIG_SPI_FLASH=y >> +CONFIG_CMD_CROS_EC=y >> +CONFIG_CROS_EC=y >> +CONFIG_CROS_EC_LPC=y >> CONFIG_VIDEO_VESA=y >> CONFIG_FRAMEBUFFER_SET_VESA_MODE=y >> CONFIG_FRAMEBUFFER_VESA_MODE_11A=y >> diff --git a/configs/chromebox_panther_defconfig b/configs/chromebox_panther_defconfig >> index 340510f..e82c8ec 100644 >> --- a/configs/chromebox_panther_defconfig >> +++ b/configs/chromebox_panther_defconfig >> @@ -14,6 +14,9 @@ CONFIG_CMD_BOOTSTAGE=y >> CONFIG_OF_CONTROL=y >> CONFIG_DM_PCI=y >> CONFIG_SPI_FLASH=y >> +CONFIG_CMD_CROS_EC=y >> +CONFIG_CROS_EC=y >> +CONFIG_CROS_EC_LPC=y >> CONFIG_VIDEO_VESA=y >> CONFIG_FRAMEBUFFER_SET_VESA_MODE=y >> CONFIG_FRAMEBUFFER_VESA_MODE_11A=y >> diff --git a/include/configs/x86-chromebook.h b/include/configs/x86-chromebook.h >> index 408cbb1..e8f0bac 100644 >> --- a/include/configs/x86-chromebook.h >> +++ b/include/configs/x86-chromebook.h >> @@ -45,9 +45,6 @@ >> #define VIDEO_IO_OFFSET 0 >> #define CONFIG_X86EMU_RAW_IO >> >> -#define CONFIG_CROS_EC >> -#define CONFIG_CROS_EC_LPC >> -#define CONFIG_CMD_CROS_EC >> #define CONFIG_ARCH_EARLY_INIT_R >> >> #undef CONFIG_ENV_IS_NOWHERE >> -- > > Reviewed-by: Bin Meng <bmeng.cn@gmail.com> Applied to u-boot-x86.
diff --git a/configs/chromebook_link_defconfig b/configs/chromebook_link_defconfig index e394dab..b987f3f 100644 --- a/configs/chromebook_link_defconfig +++ b/configs/chromebook_link_defconfig @@ -14,6 +14,9 @@ CONFIG_CMD_BOOTSTAGE=y CONFIG_OF_CONTROL=y CONFIG_DM_PCI=y CONFIG_SPI_FLASH=y +CONFIG_CMD_CROS_EC=y +CONFIG_CROS_EC=y +CONFIG_CROS_EC_LPC=y CONFIG_VIDEO_VESA=y CONFIG_FRAMEBUFFER_SET_VESA_MODE=y CONFIG_FRAMEBUFFER_VESA_MODE_11A=y diff --git a/configs/chromebox_panther_defconfig b/configs/chromebox_panther_defconfig index 340510f..e82c8ec 100644 --- a/configs/chromebox_panther_defconfig +++ b/configs/chromebox_panther_defconfig @@ -14,6 +14,9 @@ CONFIG_CMD_BOOTSTAGE=y CONFIG_OF_CONTROL=y CONFIG_DM_PCI=y CONFIG_SPI_FLASH=y +CONFIG_CMD_CROS_EC=y +CONFIG_CROS_EC=y +CONFIG_CROS_EC_LPC=y CONFIG_VIDEO_VESA=y CONFIG_FRAMEBUFFER_SET_VESA_MODE=y CONFIG_FRAMEBUFFER_VESA_MODE_11A=y diff --git a/include/configs/x86-chromebook.h b/include/configs/x86-chromebook.h index 408cbb1..e8f0bac 100644 --- a/include/configs/x86-chromebook.h +++ b/include/configs/x86-chromebook.h @@ -45,9 +45,6 @@ #define VIDEO_IO_OFFSET 0 #define CONFIG_X86EMU_RAW_IO -#define CONFIG_CROS_EC -#define CONFIG_CROS_EC_LPC -#define CONFIG_CMD_CROS_EC #define CONFIG_ARCH_EARLY_INIT_R #undef CONFIG_ENV_IS_NOWHERE
Drop these from the header file and use Kconfig instead. Signed-off-by: Simon Glass <sjg@chromium.org> --- configs/chromebook_link_defconfig | 3 +++ configs/chromebox_panther_defconfig | 3 +++ include/configs/x86-chromebook.h | 3 --- 3 files changed, 6 insertions(+), 3 deletions(-)