erebion/coreboot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "gru: Show the current time on start-up"

Browse source 
This reverts commit d0361193e0.

google_chromeec_init() is a weird function that can lead to confusing
behavior. I'm not sure how it's meant to work on the boards that use it,
but it causes problems on Kevin and other non-x86 boards have never used
it either. It doesn't really do anything anyway (the EC works fine
without an initial HELLO), so at best it's just a waste of time... let's
take it back out.

There's also no need to display the current time on every boot... other
boards don't do that and the eventlog already fills the same purpose.
Cut it out to avoid one extra host command overhead.

BRANCH=None
BUG=chrome-os-partner:55995
TEST=Recovery reasons now get correctly propagated across the EC reboot.

Change-Id: I58fd5e6094e1c8cb6368e7a4569ab9231375fbc9
Signed-off-by: Julius Werner <jwerner@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/367351
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Shelley Chen <shchen@chromium.org>
This commit is contained in:
Julius Werner 2016-08-09 13:23:02 -07:00 • committed by chrome-bot
commit 103d86e68c
1 changed files with 0 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

15
src/mainboard/google/gru/mainboard.c
View file

@ -18,10 +18,7 @@
#include <delay.h>
#include <device/device.h>
#include <device/i2c.h>
#include <ec/google/chromeec/ec.h>
#include <elog.h>
#include <gpio.h>
#include <rtc.h>
#include <soc/bl31_plat_params.h>
#include <soc/clock.h>
#include <soc/display.h>
@ -165,17 +162,6 @@ static void setup_usb(void)
setup_usb_otg1();
}
static void setup_rtc(void)
{
struct rtc_time time;
int ret;
/* Show the current time to see that the EC RTC is working */
google_chromeec_init();
ret = rtc_get(&time);
rtc_display(&time);
}
static void mainboard_init(device_t dev)
{
configure_sdmmc();
@ -185,7 +171,6 @@ static void mainboard_init(device_t dev)
setup_usb();
register_reset_to_bl31();
register_poweroff_to_bl31();
setup_rtc();
}
static void enable_backlight_booster(void)