google/jecht: Fix LED for guado/rikku variants

When guado/rikku/tidus were rolled into jecht, an error was
made in set_power_led() as guado/rikku set the polarity
differently than tidus.  Fix the power LED for guado/rikku
by setting the polarity correctly.

Test: boot guado/rikku and observe proper function of power LED
under S0, S3, and S5 power states.

Change-Id: I23072ac60bc9683776f748ca1326d98257c3c54f
Signed-off-by: Matt DeVillier <matt.devillier@gmail.com>
Reviewed-on: https://review.coreboot.org/18249
Tested-by: build bot (Jenkins)
Reviewed-by: Martin Roth <martinroth@google.com>
This commit is contained in:
Matt DeVillier 2017-01-26 10:58:15 -06:00 committed by Martin Roth
parent 3054ca164e
commit aaa4ae766d
1 changed files with 9 additions and 1 deletions

View File

@ -19,10 +19,18 @@
void set_power_led(int state) void set_power_led(int state)
{ {
int polarity;
if (IS_ENABLED(CONFIG_BOARD_GOOGLE_TIDUS)) {
polarity = state == LED_OFF ? 0x00 : 0x01;
} else {
polarity = state == LED_BLINK ? 0x01 : 0x00;
}
it8772f_gpio_led(IT8772F_GPIO_DEV, it8772f_gpio_led(IT8772F_GPIO_DEV,
1, /* set */ 1, /* set */
0x01, /* select */ 0x01, /* select */
state == LED_OFF ? 0x00 : 0x01, /* polarity */ polarity, /* polarity */
state == LED_BLINK ? 0x01 : 0x00, /* pullup/pulldown */ state == LED_BLINK ? 0x01 : 0x00, /* pullup/pulldown */
0x01, /* output */ 0x01, /* output */
state == LED_BLINK ? 0x00 : 0x01, /* I/O function */ state == LED_BLINK ? 0x00 : 0x01, /* I/O function */