diff options
author | Jett Rink <jettrink@chromium.org> | 2018-06-13 15:59:17 -0600 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-06-21 19:56:34 -0700 |
commit | 64aa8638f134f8aca4f48f3efc5fa7427caeb772 (patch) | |
tree | 083874370dc45ca50bfd2ff5468bb3eaaf073c37 | |
parent | ab212b5a417e57fb452f3962a9a00d3bbda7345e (diff) | |
download | chrome-ec-64aa8638f134f8aca4f48f3efc5fa7427caeb772.tar.gz |
kevin/gru: remove board almost completely
Kevin/Gru are running out of space and we already have another ARM board on
ToT for development. Remove maintenance burden of keeping them around.
BRANCH=none
BUG=chromium:851512
TEST=build_packages --board=kevin works with dependent CLs.
CQ-DEPEND=CL:1102565,CL:*640973,CL:*640974
Change-Id: Ie7c65c7799acc9f4d266e40b29b37240fec345c6
Signed-off-by: Jett Rink <jettrink@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1100011
Reviewed-by: Ilja H. Friedel <ihf@chromium.org>
Reviewed-by: Randall Spangler <rspangler@chromium.org>
-rw-r--r-- | COMMIT-QUEUE.ini | 5 | ||||
-rw-r--r-- | Makefile.rules | 2 | ||||
-rw-r--r-- | board/kevin/battery.c | 152 | ||||
-rw-r--r-- | board/kevin/board.c | 638 | ||||
-rw-r--r-- | board/kevin/board.h | 315 | ||||
-rw-r--r-- | board/kevin/build.mk | 6 | ||||
-rw-r--r-- | board/kevin/charge_ramp.c | 22 | ||||
-rw-r--r-- | board/kevin/ec.tasklist | 45 | ||||
-rw-r--r-- | board/kevin/gpio.inc | 177 | ||||
-rw-r--r-- | board/kevin/led_gru.c | 141 | ||||
-rw-r--r-- | board/kevin/led_kevin.c | 144 | ||||
-rw-r--r-- | board/kevin/usb_pd_policy.c | 417 | ||||
-rw-r--r-- | extra/usb_power/board.README | 10 | ||||
-rwxr-xr-x | util/flash_ec | 8 |
14 files changed, 18 insertions, 2064 deletions
diff --git a/COMMIT-QUEUE.ini b/COMMIT-QUEUE.ini index 3e4e88a588..0dc5447736 100644 --- a/COMMIT-QUEUE.ini +++ b/COMMIT-QUEUE.ini @@ -10,8 +10,9 @@ # Board-specific no-vmtest-pre-cq, include boards that actually build # chromeos-ec. We use the no-vmtest-pre-cq configs since the tests won't # actually test against our EC changes. (That's what FAFT is for) -pre-cq-configs: gru-no-vmtest-pre-cq reef-no-vmtest-pre-cq chell-no-vmtest-pre-cq - celes-no-vmtest-pre-cq fizz-no-vmtest-pre-cq +pre-cq-configs: kevin-no-vmtest-pre-cq reef-no-vmtest-pre-cq + chell-no-vmtest-pre-cq celes-no-vmtest-pre-cq + fizz-no-vmtest-pre-cq # Stages to ignore in the commit queue. If these steps break, your CL will be # submitted anyway. Use with caution. diff --git a/Makefile.rules b/Makefile.rules index 242031bf53..50fe0a017d 100644 --- a/Makefile.rules +++ b/Makefile.rules @@ -17,7 +17,7 @@ build-srcs := $(foreach u,$(build-util-bin),$(sort $($(u)-objs:%.o=util/%.c) $(w host-srcs := $(foreach u,$(host-util-bin),$(sort $($(u)-objs:%.o=util/%.c) $(wildcard util/$(u).c))) # Don't do a build test on the following boards: -skip_boards = OWNERS host +skip_boards = OWNERS host kevin gru boards := $(filter-out $(skip_boards),$(notdir $(wildcard board/* private*/board/*))) # Create output directories if necessary diff --git a/board/kevin/battery.c b/board/kevin/battery.c deleted file mode 100644 index 019dad2f3c..0000000000 --- a/board/kevin/battery.c +++ /dev/null @@ -1,152 +0,0 @@ -/* Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - * - * Battery pack vendor provided charging profile - */ - -#include "battery.h" -#include "battery_smart.h" -#include "charge_state.h" -#include "console.h" -#include "ec_commands.h" -#include "extpower.h" -#include "util.h" - -/* Shutdown mode parameter to write to manufacturer access register */ -#define SB_SHUTDOWN_DATA 0x0010 - -#ifdef BOARD_KEVIN -static const struct battery_info info = { - .voltage_max = 8688, /* 8700mA, round down for chg reg */ - .voltage_normal = 7600, - .voltage_min = 6000, - .precharge_current = 200, - .start_charging_min_c = 0, - .start_charging_max_c = 45, - .charging_min_c = 0, - .charging_max_c = 60, - .discharging_min_c = -20, - .discharging_max_c = 70, -}; -#elif defined(BOARD_GRU) -static const struct battery_info info = { - .voltage_max = 8688, /* 8700mA, round down for chg reg */ - .voltage_normal = 7600, - .voltage_min = 5800, - .precharge_current = 256, - .start_charging_min_c = 0, - .start_charging_max_c = 50, - .charging_min_c = 0, - .charging_max_c = 50, - .discharging_min_c = -20, - .discharging_max_c = 60, -}; -#endif - -const struct battery_info *battery_get_info(void) -{ - return &info; -} - -int board_cut_off_battery(void) -{ - int rv; - - /* Ship mode command must be sent twice to take effect */ - rv = sb_write(SB_MANUFACTURER_ACCESS, SB_SHUTDOWN_DATA); - if (rv != EC_SUCCESS) - return EC_RES_ERROR; - - rv = sb_write(SB_MANUFACTURER_ACCESS, SB_SHUTDOWN_DATA); - return rv ? EC_RES_ERROR : EC_RES_SUCCESS; -} - -enum battery_disconnect_state battery_get_disconnect_state(void) -{ - uint8_t data[6]; - int rv; - - /* - * Take note if we find that the battery isn't in disconnect state, - * and always return NOT_DISCONNECTED without probing the battery. - * This assumes the battery will not go to disconnect state during - * runtime. - */ - static int not_disconnected; - - if (not_disconnected) - return BATTERY_NOT_DISCONNECTED; - - if (extpower_is_present()) { - /* Check if battery charging + discharging is disabled. */ - rv = sb_read_mfgacc(PARAM_OPERATION_STATUS, - SB_ALT_MANUFACTURER_ACCESS, data, sizeof(data)); - if (rv) - return BATTERY_DISCONNECT_ERROR; - if (~data[3] & (BATTERY_DISCHARGING_DISABLED | - BATTERY_CHARGING_DISABLED)) { - not_disconnected = 1; - return BATTERY_NOT_DISCONNECTED; - } - - /* - * Battery is neither charging nor discharging. Verify that - * we didn't enter this state due to a safety fault. - */ - rv = sb_read_mfgacc(PARAM_SAFETY_STATUS, - SB_ALT_MANUFACTURER_ACCESS, data, sizeof(data)); - if (rv || data[2] || data[3] || data[4] || data[5]) - return BATTERY_DISCONNECT_ERROR; - - /* No safety fault, battery is disconnected */ - return BATTERY_DISCONNECTED; - } - not_disconnected = 1; - return BATTERY_NOT_DISCONNECTED; -} - -int charger_profile_override(struct charge_state_data *curr) -{ - const struct battery_info *batt_info = battery_get_info(); - int now_discharging; - - /* battery temp in 0.1 deg C */ - int bat_temp_c = curr->batt.temperature - 2731; - - if (curr->state == ST_CHARGE) { - /* Don't charge if outside of allowable temperature range */ - if (bat_temp_c >= batt_info->charging_max_c * 10 || - bat_temp_c < batt_info->charging_min_c * 10) { - curr->requested_current = curr->requested_voltage = 0; - curr->batt.flags &= ~BATT_FLAG_WANT_CHARGE; - curr->state = ST_IDLE; - now_discharging = 0; - /* Don't start charging if battery is nearly full */ - } else if (curr->batt.status & STATUS_FULLY_CHARGED) { - curr->requested_current = curr->requested_voltage = 0; - curr->batt.flags &= ~BATT_FLAG_WANT_CHARGE; - curr->state = ST_DISCHARGE; - now_discharging = 1; - } else - now_discharging = 0; - charger_discharge_on_ac(now_discharging); - } - - return 0; -} - -/* Customs options controllable by host command. */ -#define PARAM_FASTCHARGE (CS_PARAM_CUSTOM_PROFILE_MIN + 0) - -enum ec_status charger_profile_override_get_param(uint32_t param, - uint32_t *value) -{ - return EC_RES_INVALID_PARAM; -} - -enum ec_status charger_profile_override_set_param(uint32_t param, - uint32_t value) -{ - return EC_RES_INVALID_PARAM; -} diff --git a/board/kevin/board.c b/board/kevin/board.c index 71bc55243b..7cafbebcef 100644 --- a/board/kevin/board.c +++ b/board/kevin/board.c @@ -3,639 +3,7 @@ * found in the LICENSE file. */ -#include "adc.h" -#include "adc_chip.h" -#include "als.h" -#include "backlight.h" -#include "button.h" -#include "charge_manager.h" -#include "charge_state.h" -#include "charger.h" -#include "chipset.h" -#include "common.h" -#include "console.h" -#include "ec_commands.h" -#include "driver/accel_bma2x2.h" -#include "driver/accel_kionix.h" -#include "driver/accel_kx022.h" -#include "driver/accelgyro_bmi160.h" -#include "driver/als_opt3001.h" -#include "driver/baro_bmp280.h" -#include "driver/charger/bd9995x.h" -#include "driver/tcpm/fusb302.h" -#include "extpower.h" -#include "gpio.h" -#include "hooks.h" -#include "host_command.h" -#include "i2c.h" -#include "keyboard_scan.h" -#include "lid_switch.h" -#include "power.h" -#include "power_button.h" -#include "pwm.h" -#include "pwm_chip.h" -#include "registers.h" -#include "shi_chip.h" -#include "spi.h" -#include "switch.h" -#include "system.h" -#include "task.h" -#include "tcpm.h" -#include "timer.h" -#include "thermal.h" -#include "usb_charge.h" -#include "usb_mux.h" -#include "usb_pd_tcpm.h" -#include "util.h" - -#define CPRINTS(format, args...) cprints(CC_USBCHARGE, format, ## args) -#define CPRINTF(format, args...) cprintf(CC_USBCHARGE, format, ## args) - -static void tcpc_alert_event(enum gpio_signal signal) -{ -#ifdef HAS_TASK_PDCMD - /* Exchange status with TCPCs */ - host_command_pd_send_status(PD_CHARGE_NO_CHANGE); -#endif -} - -static void overtemp_interrupt(enum gpio_signal signal) -{ - CPRINTS("AP wants shutdown"); - chipset_force_shutdown(); -} - -static void warm_reset_request_interrupt(enum gpio_signal signal) -{ - CPRINTS("AP wants warm reset"); - chipset_reset(); -} - -#include "gpio_list.h" - -/******************************************************************************/ -/* ADC channels. Must be in the exactly same order as in enum adc_channel. */ -const struct adc_t adc_channels[] = { - [ADC_BOARD_ID] = { - "BOARD_ID", NPCX_ADC_CH0, ADC_MAX_VOLT, ADC_READ_MAX+1, 0 }, - [ADC_PP900_AP] = { - "PP900_AP", NPCX_ADC_CH1, ADC_MAX_VOLT, ADC_READ_MAX+1, 0 }, - [ADC_PP1200_LPDDR] = { - "PP1200_LPDDR", NPCX_ADC_CH2, ADC_MAX_VOLT, ADC_READ_MAX+1, 0 }, - [ADC_PPVAR_CLOGIC] = { - "PPVAR_CLOGIC", - NPCX_ADC_CH3, ADC_MAX_VOLT, ADC_READ_MAX+1, 0 }, - [ADC_PPVAR_LOGIC] = { - "PPVAR_LOGIC", NPCX_ADC_CH4, ADC_MAX_VOLT, ADC_READ_MAX+1, 0 }, -}; -BUILD_ASSERT(ARRAY_SIZE(adc_channels) == ADC_CH_COUNT); - -/******************************************************************************/ -/* PWM channels. Must be in the exactly same order as in enum pwm_channel. */ -const struct pwm_t pwm_channels[] = { -#ifdef BOARD_KEVIN - [PWM_CH_LED_GREEN] = { 0, PWM_CONFIG_DSLEEP, 100 }, -#endif -#ifdef BOARD_KEVIN - [PWM_CH_DISPLIGHT] = { 2, 0, 210 }, -#else - /* ArcticSand part on Gru requires >= 2.6KHz */ - [PWM_CH_DISPLIGHT] = { 2, 0, 2600 }, -#endif - [PWM_CH_LED_RED] = { 3, PWM_CONFIG_DSLEEP, 100 }, -#ifdef BOARD_KEVIN - [PWM_CH_LED_BLUE] = { 4, PWM_CONFIG_DSLEEP, 100 }, +/* Board is only valid for host tools */ +#ifndef HOST_TOOLS_BUILD +#error "Can only build for host tools" #endif -}; -BUILD_ASSERT(ARRAY_SIZE(pwm_channels) == PWM_CH_COUNT); - -/******************************************************************************/ -/* I2C ports */ -const struct i2c_port_t i2c_ports[] = { - {"tcpc0", NPCX_I2C_PORT0_0, 1000, GPIO_I2C0_SCL0, GPIO_I2C0_SDA0}, - {"tcpc1", NPCX_I2C_PORT0_1, 1000, GPIO_I2C0_SCL1, GPIO_I2C0_SDA1}, - {"sensors", NPCX_I2C_PORT1, 400, GPIO_I2C1_SCL, GPIO_I2C1_SDA}, - {"charger", NPCX_I2C_PORT2, 400, GPIO_I2C2_SCL, GPIO_I2C2_SDA}, - {"battery", NPCX_I2C_PORT3, 100, GPIO_I2C3_SCL, GPIO_I2C3_SDA}, -}; -const unsigned int i2c_ports_used = ARRAY_SIZE(i2c_ports); - -/* power signal list. Must match order of enum power_signal. */ -const struct power_signal_info power_signal_list[] = { - {GPIO_PP5000_PG, POWER_SIGNAL_ACTIVE_HIGH, "PP5000_PWR_GOOD"}, - {GPIO_TPS65261_PG, POWER_SIGNAL_ACTIVE_HIGH, "SYS_PWR_GOOD"}, - {GPIO_AP_CORE_PG, POWER_SIGNAL_ACTIVE_HIGH, "AP_PWR_GOOD"}, - {GPIO_AP_EC_S3_S0_L, POWER_SIGNAL_ACTIVE_LOW, "SUSPEND_DEASSERTED"}, -}; -BUILD_ASSERT(ARRAY_SIZE(power_signal_list) == POWER_SIGNAL_COUNT); - -/******************************************************************************/ -/* SPI devices */ -const struct spi_device_t spi_devices[] = { - { CONFIG_SPI_ACCEL_PORT, 1, GPIO_SPI_SENSOR_CS_L } -}; -const unsigned int spi_devices_used = ARRAY_SIZE(spi_devices); - -/******************************************************************************/ -/* Wake-up pins for hibernate */ -const enum gpio_signal hibernate_wake_pins[] = { - GPIO_POWER_BUTTON_L, GPIO_CHARGER_INT_L, GPIO_LID_OPEN -}; -const int hibernate_wake_pins_used = ARRAY_SIZE(hibernate_wake_pins); - -/******************************************************************************/ -/* Keyboard scan setting */ -struct keyboard_scan_config keyscan_config = { -#ifdef BOARD_KEVIN - .output_settle_us = 40, -#else - /* Extra delay when KSO2 is tied to cr50 */ - .output_settle_us = 60, -#endif - .debounce_down_us = 6 * MSEC, - .debounce_up_us = 30 * MSEC, - .scan_period_us = 1500, - .min_post_scan_delay_us = 1000, - .poll_timeout_us = SECOND, - .actual_key_mask = { - 0x14, 0xff, 0xff, 0xff, 0xff, 0xf5, 0xff, - 0xa4, 0xff, 0xfe, 0x55, 0xfa, 0xc8 /* full set with lock key */ - }, -}; - -const struct tcpc_config_t tcpc_config[CONFIG_USB_PD_PORT_COUNT] = { - {I2C_PORT_TCPC0, FUSB302_I2C_SLAVE_ADDR, &fusb302_tcpm_drv}, - {I2C_PORT_TCPC1, FUSB302_I2C_SLAVE_ADDR, &fusb302_tcpm_drv}, -}; - -struct usb_mux usb_muxes[CONFIG_USB_PD_PORT_COUNT] = { - { - .port_addr = 0, - .driver = &virtual_usb_mux_driver, - .hpd_update = &virtual_hpd_update, - }, - { - .port_addr = 1, - .driver = &virtual_usb_mux_driver, - .hpd_update = &virtual_hpd_update, - }, -}; - -void board_reset_pd_mcu(void) -{ -} - -uint16_t tcpc_get_alert_status(void) -{ - uint16_t status = 0; - - if (!gpio_get_level(GPIO_USB_C0_PD_INT_L)) - status |= PD_STATUS_TCPC_ALERT_0; - if (!gpio_get_level(GPIO_USB_C1_PD_INT_L)) - status |= PD_STATUS_TCPC_ALERT_1; - - return status; -} - -int board_set_active_charge_port(int charge_port) -{ - enum bd9995x_charge_port bd9995x_port; - int bd9995x_port_select = 1; - - switch (charge_port) { - case 0: case 1: - /* Don't charge from a source port */ - if (board_vbus_source_enabled(charge_port)) - return -1; - - bd9995x_port = charge_port; - break; - case CHARGE_PORT_NONE: - bd9995x_port_select = 0; - bd9995x_port = BD9995X_CHARGE_PORT_BOTH; - break; - default: - panic("Invalid charge port\n"); - break; - } - - CPRINTS("New chg p%d", charge_port); - - return bd9995x_select_input_port(bd9995x_port, bd9995x_port_select); -} - -void board_set_charge_limit(int port, int supplier, int charge_ma, - int max_ma, int charge_mv) -{ - charge_set_input_current_limit(MAX(charge_ma, - CONFIG_CHARGER_INPUT_CURRENT), charge_mv); -} - -int extpower_is_present(void) -{ - int port; - int p0_src = board_vbus_source_enabled(0); - int p1_src = board_vbus_source_enabled(1); - - /* - * The charger will indicate VBUS presence if we're sourcing 5V, - * so exclude such ports. - */ - if (p0_src && p1_src) - return 0; - else if (!p0_src && !p1_src) - port = BD9995X_CHARGE_PORT_BOTH; - else - port = p0_src; - - return bd9995x_is_vbus_provided(port); -} - -int pd_snk_is_vbus_provided(int port) -{ - if (port != 0 && port != 1) - panic("Invalid charge port\n"); - - return bd9995x_is_vbus_provided(port); -} - -static void board_spi_enable(void) -{ - spi_enable(CONFIG_SPI_ACCEL_PORT, 1); -} -DECLARE_HOOK(HOOK_CHIPSET_STARTUP, - board_spi_enable, - MOTION_SENSE_HOOK_PRIO - 1); - -static void board_spi_disable(void) -{ - spi_enable(CONFIG_SPI_ACCEL_PORT, 0); -} -DECLARE_HOOK(HOOK_CHIPSET_SHUTDOWN, - board_spi_disable, - MOTION_SENSE_HOOK_PRIO + 1); - -/* - * Reset our charger IC on power-on. This will briefly cut extpower to the - * system, so skip the reset if our battery can't provide sufficient charge - * to briefly power the system. - * TODO(shawnn): Move to common code. - */ -static void board_reset_charger(void) -{ - int bat_pct = 0; - - if (!system_jumped_to_this_image() && - battery_is_present() == BP_YES && - battery_get_disconnect_state() != BATTERY_DISCONNECTED) { - if (battery_state_of_charge_abs(&bat_pct) || - bat_pct < CONFIG_CHARGER_MIN_BAT_PCT_FOR_POWER_ON) - return; - charger_set_mode(CHARGE_FLAG_POR_RESET); - } -} -DECLARE_HOOK(HOOK_INIT, board_reset_charger, HOOK_PRIO_INIT_EXTPOWER - 1); - -static void board_init(void) -{ - /* Enable TCPC alert interrupts */ - gpio_enable_interrupt(GPIO_USB_C0_PD_INT_L); - gpio_enable_interrupt(GPIO_USB_C1_PD_INT_L); - - /* Enable charger interrupt for BC1.2 detection on attach / detach */ - gpio_enable_interrupt(GPIO_CHARGER_INT_L); - - /* Enable reboot / shutdown control inputs from AP */ - gpio_enable_interrupt(GPIO_WARM_RESET_REQ); - gpio_enable_interrupt(GPIO_AP_OVERTEMP); - - /* Enable interrupts from BMI160 sensor. */ - gpio_enable_interrupt(GPIO_BASE_SIXAXIS_INT_L); - - /* Sensor Init */ - if (system_jumped_to_this_image() && chipset_in_state(CHIPSET_STATE_ON)) - board_spi_enable(); -} -DECLARE_HOOK(HOOK_INIT, board_init, HOOK_PRIO_DEFAULT); - -void board_hibernate(void) -{ - int i; - int rv; - - /* - * Disable the power enables for the TCPCs since we're going into - * hibernate. The charger VBUS interrupt will wake us up and reset the - * EC. Upon init, we'll reinitialize the TCPCs to be at full power. - */ - CPRINTS("Set TCPCs to low power"); - for (i = 0; i < CONFIG_USB_PD_PORT_COUNT; i++) { - rv = tcpc_write(i, TCPC_REG_POWER, TCPC_REG_POWER_PWR_LOW); - if (rv) - CPRINTS("Error setting TCPC %d", i); - } - - cflush(); -} - -enum kevin_board_version { - BOARD_VERSION_UNKNOWN = -1, - BOARD_VERSION_REV0 = 0, - BOARD_VERSION_REV1 = 1, - BOARD_VERSION_REV2 = 2, - BOARD_VERSION_REV3 = 3, - BOARD_VERSION_REV4 = 4, - BOARD_VERSION_REV5 = 5, - BOARD_VERSION_REV6 = 6, - BOARD_VERSION_REV7 = 7, - BOARD_VERSION_REV8 = 8, - BOARD_VERSION_REV9 = 9, - BOARD_VERSION_REV10 = 10, - BOARD_VERSION_REV11 = 11, - BOARD_VERSION_REV12 = 12, - BOARD_VERSION_REV13 = 13, - BOARD_VERSION_REV14 = 14, - BOARD_VERSION_REV15 = 15, - BOARD_VERSION_COUNT, -}; - -struct { - enum kevin_board_version version; - int expect_mv; -} const kevin_boards[] = { - { BOARD_VERSION_REV0, 109 }, /* 51.1K , 2.2K(gru 3.3K) ohm */ - { BOARD_VERSION_REV1, 211 }, /* 51.1k , 6.8K ohm */ - { BOARD_VERSION_REV2, 319 }, /* 51.1K , 11K ohm */ - { BOARD_VERSION_REV3, 427 }, /* 56K , 17.4K ohm */ - { BOARD_VERSION_REV4, 542 }, /* 51.1K , 22K ohm */ - { BOARD_VERSION_REV5, 666 }, /* 51.1K , 30K ohm */ - { BOARD_VERSION_REV6, 781 }, /* 51.1K , 39.2K ohm */ - { BOARD_VERSION_REV7, 900 }, /* 56K , 56K ohm */ - { BOARD_VERSION_REV8, 1023 }, /* 47K , 61.9K ohm */ - { BOARD_VERSION_REV9, 1137 }, /* 47K , 80.6K ohm */ - { BOARD_VERSION_REV10, 1240 }, /* 56K , 124K ohm */ - { BOARD_VERSION_REV11, 1343 }, /* 51.1K , 150K ohm */ - { BOARD_VERSION_REV12, 1457 }, /* 47K , 200K ohm */ - { BOARD_VERSION_REV13, 1576 }, /* 47K , 330K ohm */ - { BOARD_VERSION_REV14, 1684 }, /* 47K , 680K ohm */ - { BOARD_VERSION_REV15, 1800 }, /* 56K , NC */ -}; -BUILD_ASSERT(ARRAY_SIZE(kevin_boards) == BOARD_VERSION_COUNT); - -#define THRESHOLD_MV 56 /* Simply assume 1800/16/2 */ - -int board_get_version(void) -{ - static int version = BOARD_VERSION_UNKNOWN; - int mv; - int i; - - if (version != BOARD_VERSION_UNKNOWN) - return version; - - gpio_set_level(GPIO_EC_BOARD_ID_EN_L, 0); - /* Wait to allow cap charge */ - msleep(10); - mv = adc_read_channel(ADC_BOARD_ID); - - /* TODO(crosbug.com/p/54971): Fix failure on first ADC conversion. */ - if (mv == ADC_READ_ERROR) - mv = adc_read_channel(ADC_BOARD_ID); - - gpio_set_level(GPIO_EC_BOARD_ID_EN_L, 1); - - for (i = 0; i < BOARD_VERSION_COUNT; ++i) { - if (mv < kevin_boards[i].expect_mv + THRESHOLD_MV) { - version = kevin_boards[i].version; - break; - } - } - - return version; -} - -/* Mutexes */ -static struct mutex g_base_mutex; -static struct mutex g_lid_mutex; - -static struct bmi160_drv_data_t g_bmi160_data; - -#ifdef BOARD_KEVIN -/* BMA255 private data */ -static struct accelgyro_saved_data_t g_bma255_data; - -/* Matrix to rotate accelrator into standard reference frame */ -const matrix_3x3_t base_standard_ref = { - { 0, FLOAT_TO_FP(1), 0}, - { FLOAT_TO_FP(1), 0, 0}, - { 0, 0, FLOAT_TO_FP(-1)} -}; - -const matrix_3x3_t lid_standard_ref = { - { 0, FLOAT_TO_FP(1), 0}, - { FLOAT_TO_FP(-1), 0, 0}, - { 0, 0, FLOAT_TO_FP(1)} -}; -#else -/* Matrix to rotate accelerometer into standard reference frame */ -const matrix_3x3_t base_standard_ref = { - { FLOAT_TO_FP(-1), 0, 0}, - { 0, FLOAT_TO_FP(1), 0}, - { 0, 0, FLOAT_TO_FP(-1)} -}; - -const matrix_3x3_t lid_standard_ref = { - { 0, FLOAT_TO_FP(1), 0}, - { FLOAT_TO_FP(-1), 0, 0}, - { 0, 0, FLOAT_TO_FP(1)} -}; - -static struct kionix_accel_data g_kx022_data; -static struct bmp280_drv_data_t bmp280_drv_data; - -/* ALS instances. Must be in same order as enum als_id. */ -struct als_t als[] = { - /* FIXME(dhendrix): verify attenuation_factor */ - {"TI", opt3001_init, opt3001_read_lux, 5}, -}; -BUILD_ASSERT(ARRAY_SIZE(als) == ALS_COUNT); -#endif /* BOARD_KEVIN */ - -struct motion_sensor_t motion_sensors[] = { - /* - * Note: bmi160: supports accelerometer and gyro sensor - * Requirement: accelerometer sensor must init before gyro sensor - * DO NOT change the order of the following table. - */ - [BASE_ACCEL] = { - .name = "Base Accel", - .active_mask = SENSOR_ACTIVE_S0_S3, - .chip = MOTIONSENSE_CHIP_BMI160, - .type = MOTIONSENSE_TYPE_ACCEL, - .location = MOTIONSENSE_LOC_BASE, - .drv = &bmi160_drv, - .mutex = &g_base_mutex, - .drv_data = &g_bmi160_data, - .port = CONFIG_SPI_ACCEL_PORT, - .addr = BMI160_SET_SPI_ADDRESS(CONFIG_SPI_ACCEL_PORT), - .rot_standard_ref = &base_standard_ref, - .default_range = 2, /* g, enough for laptop. */ - .min_frequency = BMI160_ACCEL_MIN_FREQ, - .max_frequency = BMI160_ACCEL_MAX_FREQ, - .config = { - /* EC use accel for angle detection */ - [SENSOR_CONFIG_EC_S0] = { - .odr = 10000 | ROUND_UP_FLAG, - .ec_rate = 100 * MSEC, - }, - /* EC use accel for angle detection */ - [SENSOR_CONFIG_EC_S3] = { - .odr = 10000 | ROUND_UP_FLAG, - }, - }, - }, - [BASE_GYRO] = { - .name = "Base Gyro", - .active_mask = SENSOR_ACTIVE_S0_S3, - .chip = MOTIONSENSE_CHIP_BMI160, - .type = MOTIONSENSE_TYPE_GYRO, - .location = MOTIONSENSE_LOC_BASE, - .drv = &bmi160_drv, - .mutex = &g_base_mutex, - .drv_data = &g_bmi160_data, - .port = CONFIG_SPI_ACCEL_PORT, - .addr = BMI160_SET_SPI_ADDRESS(CONFIG_SPI_ACCEL_PORT), - .default_range = 1000, /* dps */ -#ifdef BOARD_KEVIN - .rot_standard_ref = &base_standard_ref, -#else - .rot_standard_ref = NULL, /* Identity matrix. */ -#endif - .min_frequency = BMI160_GYRO_MIN_FREQ, - .max_frequency = BMI160_GYRO_MAX_FREQ, - }, -#ifdef BOARD_KEVIN - [LID_ACCEL] = { - .name = "Lid Accel", - .active_mask = SENSOR_ACTIVE_S0_S3, - .chip = MOTIONSENSE_CHIP_BMA255, - .type = MOTIONSENSE_TYPE_ACCEL, - .location = MOTIONSENSE_LOC_LID, - .drv = &bma2x2_accel_drv, - .mutex = &g_lid_mutex, - .drv_data = &g_bma255_data, - .port = I2C_PORT_ACCEL, - .addr = BMA2x2_I2C_ADDR1, - .rot_standard_ref = &lid_standard_ref, - .default_range = 2, /* g, enough for laptop. */ - .min_frequency = BMA255_ACCEL_MIN_FREQ, - .max_frequency = BMA255_ACCEL_MAX_FREQ, - .config = { - /* EC use accel for angle detection */ - [SENSOR_CONFIG_EC_S0] = { - .odr = 10000 | ROUND_UP_FLAG, - }, - /* EC use accel for angle detection */ - [SENSOR_CONFIG_EC_S3] = { - .odr = 10000 | ROUND_UP_FLAG, - }, - }, - }, -#else - [LID_ACCEL] = { - .name = "Lid Accel", - .active_mask = SENSOR_ACTIVE_S0_S3, - .chip = MOTIONSENSE_CHIP_KX022, - .type = MOTIONSENSE_TYPE_ACCEL, - .location = MOTIONSENSE_LOC_LID, - .drv = &kionix_accel_drv, - .mutex = &g_lid_mutex, - .drv_data = &g_kx022_data, - .port = I2C_PORT_ACCEL, - .addr = KX022_ADDR0, - .rot_standard_ref = &lid_standard_ref, - .default_range = 2, /* g, enough for laptop. */ - .min_frequency = KX022_ACCEL_MIN_FREQ, - .max_frequency = KX022_ACCEL_MAX_FREQ, - .config = { - /* EC use accel for angle detection */ - [SENSOR_CONFIG_EC_S0] = { - .odr = 10000 | ROUND_UP_FLAG, - }, - /* EC use accel for angle detection */ - [SENSOR_CONFIG_EC_S3] = { - .odr = 10000 | ROUND_UP_FLAG, - }, - }, - }, - [BASE_BARO] = { - .name = "Base Baro", - .active_mask = SENSOR_ACTIVE_S0_S3, - .chip = MOTIONSENSE_CHIP_BMP280, - .type = MOTIONSENSE_TYPE_BARO, - .location = MOTIONSENSE_LOC_BASE, - .drv = &bmp280_drv, - .drv_data = &bmp280_drv_data, - .port = I2C_PORT_BARO, - .addr = BMP280_I2C_ADDRESS1, - .default_range = 1 << 18, /* 1bit = 4 Pa, 16bit ~= 2600 hPa */ - .min_frequency = BMP280_BARO_MIN_FREQ, - .max_frequency = BMP280_BARO_MAX_FREQ, - }, -#endif /* BOARD_KEVIN */ -}; -const unsigned int motion_sensor_count = ARRAY_SIZE(motion_sensors); - -#ifndef TEST_BUILD -void lid_angle_peripheral_enable(int enable) -{ - keyboard_scan_enable(enable, KB_SCAN_DISABLE_LID_ANGLE); -} -#endif - -#ifdef BOARD_GRU -static void usb_charge_resume(void) -{ - /* Turn on USB-A ports on as we go into S0 from S3. */ - gpio_set_level(GPIO_USB_A_EN, 1); - gpio_set_level(GPIO_USB_A_CHARGE_EN, 1); -} -DECLARE_HOOK(HOOK_CHIPSET_RESUME, usb_charge_resume, HOOK_PRIO_DEFAULT); - -static void usb_charge_shutdown(void) -{ - /* Turn off USB-A ports as we go back to S5. */ - gpio_set_level(GPIO_USB_A_CHARGE_EN, 0); - gpio_set_level(GPIO_USB_A_EN, 0); -} -DECLARE_HOOK(HOOK_CHIPSET_SHUTDOWN, usb_charge_shutdown, HOOK_PRIO_DEFAULT); -#endif - -#define PWM_DISPLIGHT_SYSJUMP_TAG 0x5044 /* "PD" */ -#define PWM_HOOK_VERSION 1 - -static void pwm_displight_restore_state(void) -{ - const int *prev; - int version, size; - - prev = (const int *)system_get_jump_tag(PWM_DISPLIGHT_SYSJUMP_TAG, - &version, &size); - if (prev && version == PWM_HOOK_VERSION && size == sizeof(*prev)) - pwm_set_raw_duty(PWM_CH_DISPLIGHT, *prev); -} -DECLARE_HOOK(HOOK_INIT, pwm_displight_restore_state, HOOK_PRIO_INIT_PWM + 1); - -static void pwm_displight_preserve_state(void) -{ - int pwm_displight_duty = pwm_get_raw_duty(PWM_CH_DISPLIGHT); - - system_add_jump_tag(PWM_DISPLIGHT_SYSJUMP_TAG, PWM_HOOK_VERSION, - sizeof(pwm_displight_duty), &pwm_displight_duty); -} -DECLARE_HOOK(HOOK_SYSJUMP, pwm_displight_preserve_state, HOOK_PRIO_DEFAULT); - -int board_allow_i2c_passthru(int port) -{ - return (port == I2C_PORT_VIRTUAL_BATTERY); -} diff --git a/board/kevin/board.h b/board/kevin/board.h index 152c7890ba..19953f2538 100644 --- a/board/kevin/board.h +++ b/board/kevin/board.h @@ -3,317 +3,4 @@ * found in the LICENSE file. */ -/* Configuration for Nuvoton M4 EB */ - -#ifndef __CROS_EC_BOARD_H -#define __CROS_EC_BOARD_H - -#undef CONFIG_HOST_EVENT64 - -/* Optional modules */ -#define CONFIG_ADC -#define CONFIG_CHIPSET_RK3399 -#define CONFIG_CMD_RTC -#define CONFIG_FPU -#define CONFIG_HOSTCMD_RTC -#define CONFIG_HOSTCMD_SPS -#define CONFIG_I2C -#define CONFIG_I2C_MASTER -#define CONFIG_I2C_VIRTUAL_BATTERY -#define CONFIG_I2C_PASSTHRU_RESTRICTED -#define CONFIG_LED_COMMON -#define CONFIG_LOW_POWER_IDLE -#define CONFIG_POWER_COMMON -#define CONFIG_PWM -#define CONFIG_PWM_DISPLIGHT -#define CONFIG_SPI -#define CONFIG_SPI_MASTER -#define CONFIG_SPI_FLASH_GD25LQ40 -#define CONFIG_SPI_FLASH_REGS - -#define CONFIG_SYSTEM_UNLOCKED /* Allow dangerous commands for testing */ - -/* - * We are code space-constrained on kevin, so take 10K that is normally used - * as data RAM (was 30K, now 22K) and use it for code RAM (was 96K, now 104K) - */ -#define RAM_SHIFT_SIZE (8 * 1024) -#undef CONFIG_RO_SIZE -#define CONFIG_RO_SIZE (NPCX_PROGRAM_MEMORY_SIZE + RAM_SHIFT_SIZE) -#undef CONFIG_RAM_BASE -#define CONFIG_RAM_BASE (0x200C0000 + RAM_SHIFT_SIZE) -#undef CONFIG_RAM_SIZE -#undef CONFIG_DATA_RAM_SIZE -#define CONFIG_DATA_RAM_SIZE (0x00008000 - RAM_SHIFT_SIZE) -#define CONFIG_RAM_SIZE (CONFIG_DATA_RAM_SIZE - 0x800) - -/* Optional features */ -#define CONFIG_BOARD_VERSION_CUSTOM -#define CONFIG_FLASH_SIZE 0x00080000 /* 512KB spi flash */ -#define CONFIG_HOST_COMMAND_STATUS -#define CONFIG_HOSTCMD_SECTION_SORTED /* Host commands are sorted. */ -/* By default, set hcdebug to off */ -#undef CONFIG_HOSTCMD_DEBUG_MODE -#define CONFIG_HOSTCMD_DEBUG_MODE HCDEBUG_OFF -#define CONFIG_KEYBOARD_BOARD_CONFIG -#define CONFIG_KEYBOARD_COL2_INVERTED -#define CONFIG_KEYBOARD_PROTOCOL_MKBP /* Instead of 8042 protocol of keyboard */ -#define CONFIG_KEYBOARD_PWRBTN_ASSERTS_KSI2 -#define CONFIG_LTO -#define CONFIG_POWER_BUTTON -#define CONFIG_POWER_TRACK_HOST_SLEEP_STATE -#define CONFIG_VBOOT_HASH -#define CONFIG_VOLUME_BUTTONS - -#define CONFIG_CHARGER -#define CONFIG_CHARGER_BD9995X -#define CONFIG_CHARGER_INPUT_CURRENT 512 -#define CONFIG_CHARGER_MAINTAIN_VBAT -#define CONFIG_CHARGER_DISCHARGE_ON_AC -#define CONFIG_CHARGER_V2 -#define CONFIG_CHARGER_MIN_BAT_PCT_FOR_POWER_ON 2 -#define CONFIG_CHARGER_LIMIT_POWER_THRESH_BAT_PCT 2 -#define CONFIG_CHARGER_LIMIT_POWER_THRESH_CHG_MW 15000 -#define CONFIG_CHARGER_PROFILE_OVERRIDE -#define CONFIG_USB_CHARGER -#define CONFIG_USB_MUX_VIRTUAL - -/* Increase tx buffer size, as we'd like to stream EC log to AP. */ -#undef CONFIG_UART_TX_BUF_SIZE -#define CONFIG_UART_TX_BUF_SIZE 2048 - -/* Sensors */ -#define CONFIG_ACCEL_BMA255 -#define CONFIG_ACCEL_KX022 -#define CONFIG_ACCELGYRO_BMI160 -#define CONFIG_ACCEL_INTERRUPTS -#define CONFIG_ACCELGYRO_BMI160_INT_EVENT TASK_EVENT_CUSTOM(4) -#define CONFIG_LID_ANGLE -#define CONFIG_LID_ANGLE_INVALID_CHECK -#define CONFIG_LID_ANGLE_TABLET_MODE -#define CONFIG_LID_ANGLE_UPDATE -#define CONFIG_LID_ANGLE_SENSOR_BASE BASE_ACCEL -#define CONFIG_LID_ANGLE_SENSOR_LID LID_ACCEL - -#ifdef BOARD_GRU -#define CONFIG_ALS_OPT3001 -#define OPT3001_I2C_ADDR OPT3001_I2C_ADDR1 -#define CONFIG_BARO_BMP280 -#endif -/* FIFO size is in power of 2. */ -#define CONFIG_ACCEL_FIFO 128 -#define CONFIG_ACCEL_FIFO_THRES (CONFIG_ACCEL_FIFO / 3) - -/* Sensors without hardware FIFO are in forced mode */ -#ifdef BOARD_KEVIN -#define CONFIG_ACCEL_FORCE_MODE_MASK (1 << LID_ACCEL) -#else -#define CONFIG_ACCEL_FORCE_MODE_MASK \ - ((1 << LID_ACCEL) | (1 << BASE_BARO)) -#endif - -#define CONFIG_TABLET_MODE -#define CONFIG_TABLET_MODE_SWITCH - -/* USB PD config */ -#define CONFIG_CHARGE_MANAGER -#define CONFIG_CHARGE_RAMP_SW -#define CONFIG_USB_POWER_DELIVERY -#define CONFIG_USB_PD_ALT_MODE -#define CONFIG_USB_PD_ALT_MODE_DFP -#define CONFIG_USB_PD_DISCHARGE_GPIO -#define CONFIG_USB_PD_DUAL_ROLE -#define CONFIG_USB_PD_LOGGING -#define CONFIG_USB_PD_PORT_COUNT 2 -#define CONFIG_USB_PD_TCPM_FUSB302 -#define CONFIG_USB_PD_MAX_SINGLE_SOURCE_CURRENT TYPEC_RP_3A0 -#define CONFIG_USB_PD_VBUS_DETECT_CHARGER -#define CONFIG_USB_PD_TRY_SRC -#define CONFIG_USBC_SS_MUX -#define CONFIG_USBC_VCONN -#define CONFIG_USBC_VCONN_SWAP -#define CONFIG_USB_PD_COMM_LOCKED - -#define CONFIG_BATTERY_CUT_OFF -#define CONFIG_BATTERY_PRESENT_GPIO GPIO_EC_BATT_PRES_L -#define CONFIG_BATTERY_REVIVE_DISCONNECT -#define CONFIG_BATTERY_SMART - -#ifdef BOARD_KEVIN -#define CONFIG_BATTERY_REQUESTS_NIL_WHEN_DEAD -#define CONFIG_USB_PD_GIVE_BACK -#endif - -#define PD_OPERATING_POWER_MW 15000 -/* Kevin board accommodate 40W input charge current */ -#ifdef BOARD_KEVIN -#define PD_MAX_POWER_MW 40000 -#else -/* 60W for Gru */ -#define PD_MAX_POWER_MW 60000 -#endif -#define PD_MAX_CURRENT_MA 3000 -#define PD_MAX_VOLTAGE_MV 20000 - -#define PD_MIN_CURRENT_MA 500 -#define PD_MIN_POWER_MW 2500 - -#define PD_POWER_SUPPLY_TURN_ON_DELAY 30000 /* us */ -#define PD_POWER_SUPPLY_TURN_OFF_DELAY 50000 /* us */ -#define PD_VCONN_SWAP_DELAY 5000 /* us */ - -#define CONFIG_UART_HOST 0 - -/* Optional feature - used by nuvoton */ -#define NPCX_UART_MODULE2 1 /* 0:GPIO10/11 1:GPIO64/65 as UART */ -#define NPCX_JTAG_MODULE2 0 /* 0:GPIO21/17/16/20 1:GPIOD5/E2/D4/E5 as JTAG*/ -#define NPCX_TACH_SEL2 0 /* 0:GPIO40/73 1:GPIO93/A6 as TACH */ -/* Enable SHI PU on transition to S0. Disable the PU otherwise for leakage. */ -#define NPCX_SHI_CS_PU -#define NPCX_SHI_BYPASS_OVER_256B - -/* Optional for testing */ -#undef CONFIG_PSTORE - -/* Reduce code size */ -#define CONFIG_COMMON_GPIO_SHORTNAMES -#define GPIO_NAME_BY_PIN(port, index) #port#index -#undef CONFIG_CONSOLE_VERBOSE - -#define CONFIG_HOSTCMD_ALIGNED - -/* Modules we want to exclude */ -#undef CONFIG_CMD_BATTFAKE -#undef CONFIG_CMD_CRASH -#undef CONFIG_CMD_FLASH -#undef CONFIG_CMD_HASH -#undef CONFIG_CMD_HCDEBUG -#undef CONFIG_CMD_I2C_SCAN -#undef CONFIG_CMD_MD -#undef CONFIG_CMD_MMAPINFO -#undef CONFIG_CMD_POWERINDEBUG -#undef CONFIG_CMD_PWR_AVG -#undef CONFIG_CMD_TIMERINFO -#undef CONFIG_CONSOLE_CMDHELP -#undef CONFIG_CONSOLE_HISTORY -#undef CONFIG_EC_CMD_PD_CHIP_INFO - -#undef CONFIG_CMD_ACCELSPOOF -#undef CONFIG_CMD_FLASHINFO -#undef CONFIG_CMD_I2C_XFER - -/* Gru is especially limited on code space */ -#ifdef BOARD_GRU -#undef CONFIG_CMD_IDLE_STATS -#undef CONFIG_USB_PD_LOGGING -#undef CONFIG_CMD_SHMEM -#undef CONFIG_CMD_USBMUX -#undef CONFIG_CMD_TYPEC -#endif - -/* - * Remove task profiling to improve SHI interrupt latency. - * TODO(crosbug.com/p/55710): Re-define once interrupt latency is within - * tolerance. - */ -#undef CONFIG_TASK_PROFILING - -#define I2C_PORT_TCPC0 NPCX_I2C_PORT0_0 -#define I2C_PORT_TCPC1 NPCX_I2C_PORT0_1 -#define I2C_PORT_ACCEL NPCX_I2C_PORT1 -#define I2C_PORT_ALS NPCX_I2C_PORT1 -#define I2C_PORT_BARO NPCX_I2C_PORT1 -#define I2C_PORT_CHARGER NPCX_I2C_PORT2 -#define I2C_PORT_BATTERY NPCX_I2C_PORT3 -#define I2C_PORT_VIRTUAL_BATTERY I2C_PORT_BATTERY - -/* Enable Accel over SPI */ -#define CONFIG_SPI_ACCEL_PORT 0 /* SPI master port (SPIP) form BMI160 */ - -#define CONFIG_MKBP_EVENT -/* Define the MKBP events which are allowed to wakeup AP in S3. */ -#define CONFIG_MKBP_WAKEUP_MASK \ - (EC_HOST_EVENT_MASK(EC_HOST_EVENT_LID_OPEN) |\ - EC_HOST_EVENT_MASK(EC_HOST_EVENT_POWER_BUTTON) |\ - EC_HOST_EVENT_MASK(EC_HOST_EVENT_KEY_PRESSED) |\ - EC_HOST_EVENT_MASK(EC_HOST_EVENT_RTC)) - -/* - * Define the host events which are to be reported to the kernel. - * - * Linux 4.4 kernel uses EC_HOST_EVENT_PD_MCU, EC_HOST_EVENT_USB_MUX, - * and EC_HOST_EVENT_RTC and all enabled WAKE events. - * - * Linux 3.18 kernel uses EC_HOST_EVENT_PD_MCU and all enabled WAKE events. - */ -#undef CONFIG_HOST_EVENT_REPORT_MASK -#define CONFIG_HOST_EVENT_REPORT_MASK \ - (CONFIG_MKBP_WAKEUP_MASK |\ - EC_HOST_EVENT_MASK(EC_HOST_EVENT_PD_MCU) |\ - EC_HOST_EVENT_MASK(EC_HOST_EVENT_RTC) |\ - EC_HOST_EVENT_MASK(EC_HOST_EVENT_USB_MUX)) - -#ifndef __ASSEMBLER__ - -enum adc_channel { - /* Real ADC channels begin here */ - ADC_BOARD_ID = 0, - ADC_PP900_AP, - ADC_PP1200_LPDDR, - ADC_PPVAR_CLOGIC, - ADC_PPVAR_LOGIC, - ADC_CH_COUNT -}; - -enum pwm_channel { -/* don't change the order or add anything between, this is ABI to kernel dts! */ -#ifdef BOARD_KEVIN - PWM_CH_LED_GREEN, -#endif - PWM_CH_DISPLIGHT, - PWM_CH_LED_RED, -#ifdef BOARD_KEVIN - PWM_CH_LED_BLUE, -#endif - /* Number of PWM channels */ - PWM_CH_COUNT -}; - -/* power signal definitions */ -enum power_signal { - PP5000_PWR_GOOD = 0, - SYS_PWR_GOOD, - AP_PWR_GOOD, - SUSPEND_DEASSERTED, - - /* Number of signals */ - POWER_SIGNAL_COUNT, -}; - -/* Light sensors */ -#ifdef BOARD_GRU -enum als_id { - ALS_OPT3001 = 0, - ALS_COUNT -}; -#endif - -/* Motion sensors */ -enum sensor_id { - BASE_ACCEL = 0, - BASE_GYRO, - LID_ACCEL, -#ifdef BOARD_GRU - BASE_BARO, -#endif -}; - -#include "gpio_signal.h" -#include "registers.h" - -void board_reset_pd_mcu(void); -int board_get_version(void); - -#endif /* !__ASSEMBLER__ */ - -#endif /* __CROS_EC_BOARD_H */ +/* Board is only valid for host tools */ diff --git a/board/kevin/build.mk b/board/kevin/build.mk index e0efe5638f..ef2ea54f3f 100644 --- a/board/kevin/build.mk +++ b/board/kevin/build.mk @@ -6,10 +6,8 @@ # Board specific files build # -# the IC is Nuvoton M-Series EC (npcx5m5g, npcx5m6g) +# Board is only valid for host tools CHIP:=npcx CHIP_VARIANT:=npcx5m5g -board-y=battery.o board.o charge_ramp.o usb_pd_policy.o -board-$(BOARD_GRU)+=led_gru.o -board-$(BOARD_KEVIN)+=led_kevin.o +board-y=board.o diff --git a/board/kevin/charge_ramp.c b/board/kevin/charge_ramp.c deleted file mode 100644 index 134f2e62de..0000000000 --- a/board/kevin/charge_ramp.c +++ /dev/null @@ -1,22 +0,0 @@ -/* Copyright 2017 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - -/* Board-specific charge ramp callbacks. */ - -#include "common.h" - -#include "bd9995x.h" -#include "charge_manager.h" -#include "charge_ramp.h" -#include "charge_state.h" -#include "system.h" - -/** - * Return true if VBUS is sagging too low - */ -int board_is_vbus_too_low(int port, enum chg_ramp_vbus_state ramp_state) -{ - return charger_get_vbus_voltage(port) < BD9995X_BC12_MIN_VOLTAGE; -} diff --git a/board/kevin/ec.tasklist b/board/kevin/ec.tasklist index 8fa08478e8..7ab3eccbce 100644 --- a/board/kevin/ec.tasklist +++ b/board/kevin/ec.tasklist @@ -3,46 +3,5 @@ * found in the LICENSE file. */ -/** - * List of enabled tasks in the priority order - * - * The first one has the lowest priority. - * - * For each task, use the macro TASK_ALWAYS(n, r, d, s) for base tasks and - * TASK_NOTEST(n, r, d, s) for tasks that can be excluded in test binaries, - * where : - * 'n' is the name of the task - * 'r' is the main routine of the task - * 'd' is an opaque parameter passed to the routine at startup - * 's' is the stack size in bytes; must be a multiple of 8 - */ -#ifdef BOARD_KEVIN -#define CONFIG_TASK_LIST \ - TASK_ALWAYS(HOOKS, hook_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_ALWAYS(CHG_RAMP, chg_ramp_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(USB_CHG, usb_charger_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(CHARGER, charger_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_NOTEST(CHIPSET, chipset_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_ALWAYS(MOTIONSENSE, motion_sense_task, NULL, VENTI_TASK_STACK_SIZE) \ - TASK_NOTEST(PDCMD, pd_command_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_ALWAYS(HOSTCMD, host_command_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(CONSOLE, console_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_NOTEST(KEYSCAN, keyboard_scan_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(PD_C0, pd_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_ALWAYS(PD_C1, pd_task, NULL, LARGER_TASK_STACK_SIZE) -#else -#define CONFIG_TASK_LIST \ - TASK_ALWAYS(HOOKS, hook_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_ALWAYS(ALS, als_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(CHG_RAMP, chg_ramp_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(USB_CHG, usb_charger_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(CHARGER, charger_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_NOTEST(CHIPSET, chipset_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_ALWAYS(MOTIONSENSE, motion_sense_task, NULL, VENTI_TASK_STACK_SIZE) \ - TASK_NOTEST(PDCMD, pd_command_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(HOSTCMD, host_command_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(CONSOLE, console_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_NOTEST(KEYSCAN, keyboard_scan_task, NULL, TASK_STACK_SIZE) \ - TASK_ALWAYS(PD_C0, pd_task, NULL, LARGER_TASK_STACK_SIZE) \ - TASK_ALWAYS(PD_C1, pd_task, NULL, LARGER_TASK_STACK_SIZE) -#endif +/* Board is only valid for host tools */ +#define CONFIG_TASK_LIST diff --git a/board/kevin/gpio.inc b/board/kevin/gpio.inc deleted file mode 100644 index d8b8d6ab91..0000000000 --- a/board/kevin/gpio.inc +++ /dev/null @@ -1,177 +0,0 @@ -/* -*- mode:c -*- - * - * Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - -/* - * Declare symbolic names for all the GPIOs that we care about. - * Note: Those with interrupt handlers must be declared first. - */ - - -GPIO_INT(WP_L, PIN(9, 3), GPIO_INT_BOTH, switch_interrupt) -GPIO_INT(SHI_CS_L, PIN(5, 3), GPIO_INT_FALLING | GPIO_PULL_DOWN, - shi_cs_event) -GPIO_INT(USB_C0_PD_INT_L, PIN(6, 0), GPIO_INT_FALLING | GPIO_PULL_UP, - tcpc_alert_event) -GPIO_INT(USB_C1_PD_INT_L, PIN(6, 2), GPIO_INT_FALLING | GPIO_PULL_UP, - tcpc_alert_event) -GPIO_INT(VOLUME_UP_L, PIN(8, 2), GPIO_INT_BOTH | GPIO_PULL_UP, - button_interrupt) -GPIO_INT(VOLUME_DOWN_L, PIN(8, 3), GPIO_INT_BOTH | GPIO_PULL_UP, - button_interrupt) -GPIO_INT(POWER_BUTTON_L, PIN(0, 4), GPIO_INT_BOTH | GPIO_PULL_UP, - power_button_interrupt) -GPIO_INT(LID_OPEN, PIN(9, 7), GPIO_INT_BOTH | GPIO_SEL_1P8V, - lid_interrupt) - -GPIO_INT(PP5000_PG, PIN(7, 1), GPIO_INT_BOTH | GPIO_PULL_UP, - power_signal_interrupt) -GPIO_INT(TPS65261_PG, PIN(7, 5), GPIO_INT_BOTH | GPIO_PULL_UP, - power_signal_interrupt) -/* TODO: Remove PD in S3 for power savings */ -GPIO_INT(AP_EC_S3_S0_L, PIN(C, 1), GPIO_INT_BOTH | GPIO_PULL_DOWN, - power_signal_interrupt) -GPIO_INT(AP_CORE_PG, PIN(6, 7), GPIO_INT_BOTH | GPIO_PULL_UP, - power_signal_interrupt) -GPIO_INT(WARM_RESET_REQ, PIN(7, 3), GPIO_INT_RISING | GPIO_PULL_DOWN, - warm_reset_request_interrupt) -GPIO_INT(AP_OVERTEMP, PIN(7, 4), GPIO_INT_RISING | GPIO_PULL_DOWN, - overtemp_interrupt) -GPIO_INT(CHARGER_INT_L, PIN(3, 3), GPIO_INT_FALLING | GPIO_PULL_UP, - bd9995x_vbus_interrupt) -GPIO_INT(BASE_SIXAXIS_INT_L,PIN(4, 0), GPIO_INT_FALLING | GPIO_SEL_1P8V, - bmi160_interrupt) - -/* VR EN */ -GPIO(AP_CORE_EN, PIN(7, 2), GPIO_OUT_LOW) -GPIO(LPDDR_PWR_EN, PIN(8, 6), GPIO_OUT_LOW) -GPIO(PPVAR_CLOGIC_EN, PIN(C, 5), GPIO_OUT_LOW) -GPIO(PPVAR_LOGIC_EN, PIN(8, 5), GPIO_OUT_LOW) - -GPIO(PP900_AP_EN, PIN(B, 7), GPIO_OUT_LOW) -GPIO(PP900_DDRPLL_EN, PIN(C, 0), GPIO_OUT_LOW) -GPIO(PP900_PLL_EN, PIN(5, 4), GPIO_OUT_LOW) -GPIO(PP900_PMU_EN, PIN(C, 2), GPIO_OUT_LOW) -GPIO(PP900_USB_EN, PIN(A, 5), GPIO_OUT_LOW) -GPIO(PP900_PCIE_EN, PIN(0, 0), GPIO_OUT_LOW) - -/* NC */ -GPIO(GPIO81_NC, PIN(8, 1), GPIO_INPUT | GPIO_PULL_UP) - -GPIO(PP1800_SENSOR_EN_L, PIN(A, 7), GPIO_OUT_HIGH) -GPIO(PP1800_USB_EN_L, PIN(A, 6), GPIO_OUT_HIGH) -GPIO(PP1800_LID_EN_L, PIN(B, 0), GPIO_OUT_HIGH) -GPIO(PP1800_PMU_EN_L, PIN(5, 1), GPIO_OUT_HIGH) -GPIO(PP1800_AP_AVDD_EN_L, PIN(5, 2), GPIO_OUT_HIGH) -GPIO(PP1800_S0_EN_L, PIN(5, 0), GPIO_OUT_HIGH) -GPIO(PP1800_SIXAXIS_EN_L, PIN(5, 6), GPIO_OUT_HIGH) - -GPIO(PP3300_TRACKPAD_EN_L, PIN(3, 2), GPIO_OUT_HIGH) -GPIO(PP3300_USB_EN_L, PIN(3, 7), GPIO_OUT_HIGH) - -GPIO(PP5000_EN, PIN(C, 6), GPIO_OUT_LOW) - -/* - * I2C pins should be configured as inputs until I2C module is - * initialized. This will avoid driving the lines unintentionally. - */ -GPIO(I2C0_SCL0, PIN(B, 5), GPIO_ODR_HIGH | GPIO_SEL_1P8V) -GPIO(I2C0_SDA0, PIN(B, 4), GPIO_ODR_HIGH | GPIO_SEL_1P8V) -GPIO(I2C0_SCL1, PIN(B, 3), GPIO_ODR_HIGH | GPIO_SEL_1P8V) -GPIO(I2C0_SDA1, PIN(B, 2), GPIO_ODR_HIGH | GPIO_SEL_1P8V) -GPIO(I2C1_SCL, PIN(9, 0), GPIO_ODR_HIGH | GPIO_SEL_1P8V) -GPIO(I2C1_SDA, PIN(8, 7), GPIO_ODR_HIGH | GPIO_SEL_1P8V) -GPIO(I2C2_SCL, PIN(9, 2), GPIO_ODR_HIGH) -GPIO(I2C2_SDA, PIN(9, 1), GPIO_ODR_HIGH) -GPIO(I2C3_SCL, PIN(D, 1), GPIO_ODR_HIGH) -GPIO(I2C3_SDA, PIN(D, 0), GPIO_ODR_HIGH) - -/* Attached to push-pull interrupt pin of accel, but unused */ -GPIO(LID_ACCEL_INT_L, PIN(C, 7), GPIO_INPUT) - -/* KSO2 is inverted */ -GPIO(KBD_KSO2, PIN(1, 7), GPIO_OUT_LOW) - -GPIO(USB_C0_5V_EN, PIN(D, 3), GPIO_OUT_LOW | GPIO_PULL_UP) -GPIO(USB_C1_5V_EN, PIN(D, 2), GPIO_OUT_LOW | GPIO_PULL_UP) - -GPIO(ENTERING_RW, PIN(7, 6), GPIO_OUT_LOW) -GPIO(SYS_RST_L, PIN(6, 1), GPIO_OUT_HIGH | GPIO_OPEN_DRAIN) -GPIO(EC_INT_L, PIN(5, 7), GPIO_OUT_HIGH | GPIO_OPEN_DRAIN) -GPIO(EC_BOARD_ID_EN_L, PIN(3, 5), GPIO_OUT_HIGH) - -GPIO(USB_DP_HPD, PIN(6, 6), GPIO_OUT_LOW) -GPIO(CHARGER_RESET_L, PIN(0, 1), GPIO_OUT_HIGH | GPIO_OPEN_DRAIN) -GPIO(TPM_ALLOW_RST, PIN(0, 2), GPIO_OUT_HIGH | GPIO_OPEN_DRAIN) -GPIO(EC_BATT_PRES_L, PIN(3, 4), GPIO_INPUT) -GPIO(LID_360_L, PIN(3, 6), GPIO_INPUT | GPIO_SEL_1P8V) -GPIO(CCD_MODE_ODL, PIN(6, 3), GPIO_INPUT | GPIO_PULL_UP) -GPIO(PP3300_S0_EN_L, PIN(7, 0), GPIO_OUT_HIGH) -GPIO(SPI_SENSOR_CS_L, PIN(9, 4), GPIO_OUT_HIGH | GPIO_OPEN_DRAIN) - -GPIO(USB_C0_DISCHARGE, PIN(0, 3), GPIO_OUT_LOW) -GPIO(USB_C1_DISCHARGE, PIN(B, 1), GPIO_OUT_LOW) - -#if defined(BOARD_KEVIN) -GPIO(GPIO84_NC, PIN(8, 4), GPIO_INPUT | GPIO_PULL_UP) -#elif defined(BOARD_GRU) -GPIO(USB_A_EN, PIN(C, 3), GPIO_OUT_LOW) -GPIO(USB_A_CHARGE_EN, PIN(8, 4), GPIO_OUT_LOW) -GPIO(GPIOB6_NC, PIN(B, 6), GPIO_INPUT | GPIO_PULL_UP) -#endif - -/* - * SPI host interface - enable PDs by default. These will be made functional - * by the SHI driver when the AP powers up, and restored back to GPIO when - * the AP powers down. - */ -GPIO(SHI_SDI, PIN(4, 6), GPIO_INPUT | GPIO_PULL_DOWN) -GPIO(SHI_SDO, PIN(4, 7), GPIO_INPUT | GPIO_PULL_DOWN) -GPIO(SHI_SCLK, PIN(5, 5), GPIO_INPUT | GPIO_PULL_DOWN) - -/* SPIP_MOSI/SPIP_SCLK GPIOA3/A1 */ -ALTERNATE(PIN_MASK(A, 0x0A), 1, MODULE_SPI, 0) -/* SPIP_MISO GPIO95 */ -ALTERNATE(PIN_MASK(9, 0x20), 1, MODULE_SPI, 0) -/* I2C0SDA1/I2C0SCL1 GPIOB2/B3 */ -ALTERNATE(PIN_MASK(B, 0x0C), 1, MODULE_I2C, 0) -/* I2C0SDA0/I2C0SCL0 GPIOB4/B5 */ -ALTERNATE(PIN_MASK(B, 0x30), 1, MODULE_I2C, 0) -/* I2C1SDA GPIO87 */ -ALTERNATE(PIN_MASK(8, 0x80), 1, MODULE_I2C, 0) -/* I2C1SCL/I2C2SDA/I2C2SCL GPIO90/91/92 */ -ALTERNATE(PIN_MASK(9, 0x07), 1, MODULE_I2C, 0) -/* I2C3SDA/I2C3SCL GPIOD0/D1 */ -ALTERNATE(PIN_MASK(D, 0x03), 1, MODULE_I2C, 0) -/* PWM2 / BLPWM */ -ALTERNATE(PIN_MASK(C, 0x10), 1, MODULE_PWM, 0) -/* PWM3 / LED_RED(net LED_CHARGE) */ -ALTERNATE(PIN_MASK(8, 0x01), 1, MODULE_PWM, 0) - -/* Kevin-only LEDs */ -#ifdef BOARD_KEVIN -/* PWM0 / LED_GREEN(net LED_ACIN) */ -ALTERNATE(PIN_MASK(C, 0x08), 1, MODULE_PWM, 0) -/* PWM4 / LED_BLUE(net LED_POWER) */ -ALTERNATE(PIN_MASK(B, 0x40), 1, MODULE_PWM, 0) -#endif - -/* CR_SIN/SOUT GPIO64/65 */ -ALTERNATE(PIN_MASK(6, 0x30), 1, MODULE_UART, GPIO_PULL_UP) -/* ADC0-4 */ -ALTERNATE(PIN_MASK(4, 0x3e), 1, MODULE_ADC, 0) - -/* Keyboard Columns */ -ALTERNATE(PIN_MASK(0, 0xE0), 0, MODULE_KEYBOARD_SCAN, 0) -ALTERNATE(PIN_MASK(1, 0x7F), 0, MODULE_KEYBOARD_SCAN, 0) -ALTERNATE(PIN_MASK(2, 0x03), 0, MODULE_KEYBOARD_SCAN, 0) - -/* Keyboard Rows */ -ALTERNATE(PIN_MASK(2, 0xFC), 0, MODULE_KEYBOARD_SCAN, 0) -ALTERNATE(PIN_MASK(3, 0x03), 0, MODULE_KEYBOARD_SCAN, 0) - -/* External 32KHz input clock - GPIOE7 */ -ALTERNATE(PIN_MASK(E, 0x80), 1, MODULE_CLOCK, 0) diff --git a/board/kevin/led_gru.c b/board/kevin/led_gru.c deleted file mode 100644 index f02ecb868e..0000000000 --- a/board/kevin/led_gru.c +++ /dev/null @@ -1,141 +0,0 @@ -/* Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - * - * gru battery LED control - derived from standard policy with PWM - * color control rather than GPIO. - */ - -#include "battery.h" -#include "charge_state.h" -#include "chipset.h" -#include "hooks.h" -#include "led_common.h" -#include "lid_switch.h" -#include "pwm.h" -#include "pwm_chip.h" -#include "util.h" - -#define GRU_BAT_LED_PWM PWM_CH_LED_RED - -const enum ec_led_id supported_led_ids[] = { EC_LED_ID_BATTERY_LED }; - -const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids); - -enum led_color { - LED_OFF = 0, - LED_RED, - LED_AMBER, - LED_GREEN, - LED_COLOR_COUNT /* Number of colors, not a color itself */ -}; - -/* One LED active at a time. edge frequency determines which LED is active. */ -static const int led_color_to_pwm_duty[LED_COLOR_COUNT] = { - [LED_OFF] = 100, - [LED_RED] = 0, - [LED_AMBER] = 80, - [LED_GREEN] = 70, -}; -static const int led_color_to_pwm_frequency[LED_COLOR_COUNT] = { - [LED_OFF] = 1, - [LED_RED] = 1, - [LED_AMBER] = 1100, - [LED_GREEN] = 200, -}; - -static int bat_led_set_color(enum led_color color) -{ - pwm_set_freq(GRU_BAT_LED_PWM, led_color_to_pwm_frequency[color]); - pwm_set_duty(GRU_BAT_LED_PWM, led_color_to_pwm_duty[color]); - return EC_SUCCESS; -} - -void led_get_brightness_range(enum ec_led_id led_id, uint8_t *brightness_range) -{ - switch (led_id) { - case EC_LED_ID_BATTERY_LED: - brightness_range[EC_LED_COLOR_RED] = 1; - brightness_range[EC_LED_COLOR_AMBER] = 1; - brightness_range[EC_LED_COLOR_GREEN] = 1; - break; - default: - /* ignore */ - break; - } -} - -int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness) -{ - switch (led_id) { - case EC_LED_ID_BATTERY_LED: - if (brightness[EC_LED_COLOR_RED] != 0) - bat_led_set_color(LED_RED); - else if (brightness[EC_LED_COLOR_AMBER] != 0) - bat_led_set_color(LED_AMBER); - else if (brightness[EC_LED_COLOR_GREEN] != 0) - bat_led_set_color(LED_GREEN); - else - bat_led_set_color(LED_OFF); - break; - default: - return EC_ERROR_UNKNOWN; - } - return EC_SUCCESS; -} - -static void gru_led_set_battery(void) -{ - static int battery_second; - uint32_t chflags = charge_get_flags(); - - battery_second++; - - /* - * BAT LED behavior: Follow chromeos specification. - * Green/Amber for CHARGE_FLAG_FORCE_IDLE - */ - switch (charge_get_state()) { - case PWR_STATE_CHARGE: - bat_led_set_color(LED_AMBER); - break; - case PWR_STATE_DISCHARGE: - if (charge_get_percent() < 3) - bat_led_set_color((battery_second & 1) - ? LED_OFF : LED_AMBER); - else if (charge_get_percent() < 10) - bat_led_set_color((battery_second & 3) - ? LED_OFF : LED_AMBER); - else if (charge_get_percent() >= BATTERY_LEVEL_NEAR_FULL && - (chflags & CHARGE_FLAG_EXTERNAL_POWER)) - bat_led_set_color(LED_GREEN); - else - bat_led_set_color(LED_OFF); - break; - case PWR_STATE_ERROR: - bat_led_set_color((battery_second & 1) ? LED_OFF : LED_RED); - break; - case PWR_STATE_CHARGE_NEAR_FULL: - bat_led_set_color(LED_GREEN); - break; - case PWR_STATE_IDLE: /* External power connected in IDLE. */ - if (chflags & CHARGE_FLAG_FORCE_IDLE) - bat_led_set_color( - (battery_second & 0x2) ? LED_GREEN : LED_AMBER); - else - bat_led_set_color(LED_GREEN); - break; - default: - /* Other states don't alter LED behavior */ - break; - } -} - -/* Called by hook task every 1 sec */ -static void led_second(void) -{ - if (led_auto_control_is_enabled(EC_LED_ID_BATTERY_LED)) - gru_led_set_battery(); -} -DECLARE_HOOK(HOOK_SECOND, led_second, HOOK_PRIO_DEFAULT); - diff --git a/board/kevin/led_kevin.c b/board/kevin/led_kevin.c deleted file mode 100644 index 2dda9499e8..0000000000 --- a/board/kevin/led_kevin.c +++ /dev/null @@ -1,144 +0,0 @@ -/* Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - * - * Power/Battery LED control for Kevin - */ - -#include "charge_state.h" -#include "chipset.h" -#include "console.h" -#include "extpower.h" -#include "gpio.h" -#include "hooks.h" -#include "led_common.h" -#include "pwm.h" -#include "registers.h" -#include "util.h" - -#define CPRINTF(format, args...) cprintf(CC_PWM, format, ## args) -#define CPRINTS(format, args...) cprints(CC_PWM, format, ## args) - -#define LED_TOTAL_TICKS 16 -#define LED_ON_TICKS 8 - -static int led_debug; - -const enum ec_led_id supported_led_ids[] = { - EC_LED_ID_POWER_LED, EC_LED_ID_BATTERY_LED}; -const int supported_led_ids_count = ARRAY_SIZE(supported_led_ids); - -enum led_color { - LED_OFF = 0, - LED_RED, - LED_GREEN, - LED_BLUE, - - /* Number of colors, not a color itself */ - LED_COLOR_COUNT -}; - -/* Brightness vs. color, in the order of off, red, green and blue */ -static const uint8_t color_brightness[LED_COLOR_COUNT][3] = { - /* {Red, Blue, Green}, */ - [LED_OFF] = {100, 100, 100}, - [LED_RED] = {90, 100, 100}, - [LED_GREEN] = {100, 100, 80}, - [LED_BLUE] = {100, 80, 100}, -}; - -/** - * Set LED color - * - * @param color Enumerated color value - */ -static void set_color(enum led_color color) -{ - pwm_set_duty(PWM_CH_LED_RED, color_brightness[color][0]); - pwm_set_duty(PWM_CH_LED_BLUE, color_brightness[color][1]); - pwm_set_duty(PWM_CH_LED_GREEN, color_brightness[color][2]); -} - -void led_get_brightness_range(enum ec_led_id led_id, uint8_t *brightness_range) -{ - brightness_range[EC_LED_COLOR_RED] = 100; - brightness_range[EC_LED_COLOR_BLUE] = 100; - brightness_range[EC_LED_COLOR_GREEN] = 100; -} - -int led_set_brightness(enum ec_led_id led_id, const uint8_t *brightness) -{ - pwm_set_duty(PWM_CH_LED_RED, brightness[EC_LED_COLOR_RED]); - pwm_set_duty(PWM_CH_LED_BLUE, brightness[EC_LED_COLOR_BLUE]); - pwm_set_duty(PWM_CH_LED_GREEN, brightness[EC_LED_COLOR_GREEN]); - return EC_SUCCESS; -} - -static void kevin_led_set_power_battery(void) -{ - static int power_ticks; - - if (chipset_in_state(CHIPSET_STATE_ON)) { - set_color(LED_BLUE); - return; - } - - /* CHIPSET_STATE_OFF */ - switch (charge_get_state()) { - case PWR_STATE_DISCHARGE: - if ((charge_get_flags() & CHARGE_FLAG_EXTERNAL_POWER) && - charge_get_percent() >= BATTERY_LEVEL_NEAR_FULL) - set_color(LED_GREEN); - else - set_color(LED_OFF); - break; - case PWR_STATE_CHARGE: - set_color(LED_RED); - break; - case PWR_STATE_ERROR: - power_ticks++; - set_color(((power_ticks % LED_TOTAL_TICKS) - < LED_ON_TICKS) ? LED_RED : LED_GREEN); - break; - case PWR_STATE_CHARGE_NEAR_FULL: - case PWR_STATE_IDLE: /* External power connected in IDLE. */ - set_color(LED_GREEN); - break; - default: - set_color(LED_RED); - break; - } - if ((charge_get_state()) != PWR_STATE_ERROR) - power_ticks = 0; -} - -static void led_init(void) -{ - /* - * Enable PWMs and set to 0% duty cycle. If they're disabled, - * seems to ground the pins instead of letting them float. - */ - pwm_enable(PWM_CH_LED_RED, 1); - pwm_enable(PWM_CH_LED_GREEN, 1); - pwm_enable(PWM_CH_LED_BLUE, 1); - - set_color(LED_OFF); -} -/* After pwm_pin_init() */ -DECLARE_HOOK(HOOK_INIT, led_init, HOOK_PRIO_DEFAULT); - -/** - * Called by hook task every 250 ms - */ -static void led_tick(void) -{ - if (led_debug) - return; - - if (led_auto_control_is_enabled(EC_LED_ID_POWER_LED) && - led_auto_control_is_enabled(EC_LED_ID_BATTERY_LED)) { - kevin_led_set_power_battery(); - return; - } -} -DECLARE_HOOK(HOOK_TICK, led_tick, HOOK_PRIO_DEFAULT); diff --git a/board/kevin/usb_pd_policy.c b/board/kevin/usb_pd_policy.c deleted file mode 100644 index b8a4766984..0000000000 --- a/board/kevin/usb_pd_policy.c +++ /dev/null @@ -1,417 +0,0 @@ -/* Copyright 2016 The Chromium OS Authors. All rights reserved. - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - -#include "atomic.h" -#include "charge_manager.h" -#include "common.h" -#include "console.h" -#include "driver/charger/bd9995x.h" -#include "gpio.h" -#include "hooks.h" -#include "host_command.h" -#include "registers.h" -#include "system.h" -#include "task.h" -#include "timer.h" -#include "util.h" -#include "usb_mux.h" -#include "usb_pd.h" -#include "usb_pd_tcpm.h" - -#define CPRINTF(format, args...) cprintf(CC_USBPD, format, ## args) -#define CPRINTS(format, args...) cprints(CC_USBPD, format, ## args) - -#define PDO_FIXED_FLAGS (PDO_FIXED_DUAL_ROLE | PDO_FIXED_DATA_SWAP |\ - PDO_FIXED_COMM_CAP) - -const uint32_t pd_src_pdo[] = { - PDO_FIXED(5000, 1500, PDO_FIXED_FLAGS), -}; -const int pd_src_pdo_cnt = ARRAY_SIZE(pd_src_pdo); -const uint32_t pd_src_pdo_max[] = { - PDO_FIXED(5000, 3000, PDO_FIXED_FLAGS), -}; -const int pd_src_pdo_max_cnt = ARRAY_SIZE(pd_src_pdo_max); - -const uint32_t pd_snk_pdo[] = { - PDO_FIXED(5000, 500, PDO_FIXED_FLAGS), - PDO_BATT(4750, 21000, 15000), - PDO_VAR(4750, 21000, 3000), -}; -const int pd_snk_pdo_cnt = ARRAY_SIZE(pd_snk_pdo); - -int pd_is_valid_input_voltage(int mv) -{ - return 1; -} - -void pd_transition_voltage(int idx) -{ - /* No-operation: we are always 5V */ -} - -static uint8_t vbus_en[CONFIG_USB_PD_PORT_COUNT]; -static uint8_t vbus_rp[CONFIG_USB_PD_PORT_COUNT] = {TYPEC_RP_1A5, TYPEC_RP_1A5}; - -int board_vbus_source_enabled(int port) -{ - return vbus_en[port]; -} - -void pd_snk_give_back(int port, uint32_t * const ma, uint32_t * const mv) -{ - /* Just reduce the current */ - *ma = PD_MIN_CURRENT_MA; - pd_set_input_current_limit(port, *ma, *mv); -#ifdef CONFIG_CHARGE_MANAGER - charge_manager_set_ceil(port, CEIL_REQUESTOR_PD, *ma); -#endif -} - -static void board_vbus_update_source_current(int port) -{ - enum gpio_signal gpio = port ? GPIO_USB_C1_5V_EN : GPIO_USB_C0_5V_EN; - int flags = (vbus_rp[port] == TYPEC_RP_1A5 && vbus_en[port]) ? - (GPIO_INPUT | GPIO_PULL_UP) : (GPIO_OUTPUT | GPIO_PULL_UP); - - /* - * Driving USB_Cx_5V_EN high, actually put a 16.5k resistance - * (2x 33k in parallel) on the NX5P3290 load switch ILIM pin, - * setting a minimum OCP current of 3186 mA. - * Putting an internal pull-up on USB_Cx_5V_EN, effectively put a 33k - * resistor on ILIM, setting a minimum OCP current of 1505 mA. - */ - gpio_set_level(gpio, vbus_en[port]); - gpio_set_flags(gpio, flags); -} - -int pd_set_power_supply_ready(int port) -{ - /* Ensure we're not charging from this port */ - bd9995x_select_input_port(port, 0); - - /* Ensure we advertise the proper available current quota */ - charge_manager_source_port(port, 1); - - pd_set_vbus_discharge(port, 0); - /* Provide VBUS */ - vbus_en[port] = 1; - board_vbus_update_source_current(port); - - /* notify host of power info change */ - pd_send_host_event(PD_EVENT_POWER_CHANGE); - - return EC_SUCCESS; /* we are ready */ -} - -void pd_power_supply_reset(int port) -{ - int prev_en; - - prev_en = vbus_en[port]; - /* Disable VBUS */ - vbus_en[port] = 0; - board_vbus_update_source_current(port); - /* Enable discharge if we were previously sourcing 5V */ - if (prev_en) - pd_set_vbus_discharge(port, 1); - - /* Give back the current quota we are no longer using */ - charge_manager_source_port(port, 0); - - /* notify host of power info change */ - pd_send_host_event(PD_EVENT_POWER_CHANGE); -} - -void typec_set_source_current_limit(int port, int rp) -{ - vbus_rp[port] = rp; - - /* change the GPIO driving the load switch if needed */ - board_vbus_update_source_current(port); -} - -int pd_board_checks(void) -{ - return EC_SUCCESS; -} - -int pd_check_power_swap(int port) -{ - /* - * Allow power swap as long as we are acting as a dual role device, - * otherwise assume our role is fixed (not in S0 or console command - * to fix our role). - */ - return pd_get_dual_role() == PD_DRP_TOGGLE_ON ? 1 : 0; -} - -int pd_check_data_swap(int port, int data_role) -{ - /* - * Allow data swap if we are a UFP, otherwise don't allow. - * - * When we are still in the Read-Only firmware, avoid swapping roles - * so we don't jump in RW as a SNK/DFP and potentially confuse the - * power supply by sending a soft-reset with wrong data role. - */ - return (data_role == PD_ROLE_UFP) && - (system_get_image_copy() != SYSTEM_IMAGE_RO) ? 1 : 0; -} - -int pd_check_vconn_swap(int port) -{ - return gpio_get_level(GPIO_PP5000_EN); -} - -void pd_execute_data_swap(int port, int data_role) -{ - /* Do nothing */ -} - -void pd_check_pr_role(int port, int pr_role, int flags) -{ - /* - * If partner is dual-role power and dualrole toggling is on, consider - * if a power swap is necessary. - */ - if ((flags & PD_FLAGS_PARTNER_DR_POWER) && - pd_get_dual_role() == PD_DRP_TOGGLE_ON) { - /* - * If we are a sink and partner is not externally powered, then - * swap to become a source. If we are source and partner is - * externally powered, swap to become a sink. - */ - int partner_extpower = flags & PD_FLAGS_PARTNER_EXTPOWER; - - if ((!partner_extpower && pr_role == PD_ROLE_SINK) || - (partner_extpower && pr_role == PD_ROLE_SOURCE)) - pd_request_power_swap(port); - } -} - -void pd_check_dr_role(int port, int dr_role, int flags) -{ - /* If UFP, try to switch to DFP */ - if ((flags & PD_FLAGS_PARTNER_DR_DATA) && - dr_role == PD_ROLE_UFP && - system_get_image_copy() != SYSTEM_IMAGE_RO) - pd_request_data_swap(port); -} -/* ----------------- Vendor Defined Messages ------------------ */ -const struct svdm_response svdm_rsp = { - .identity = NULL, - .svids = NULL, - .modes = NULL, -}; - -int pd_custom_vdm(int port, int cnt, uint32_t *payload, - uint32_t **rpayload) -{ - int cmd = PD_VDO_CMD(payload[0]); - uint16_t dev_id = 0; - int is_rw, is_latest; - - /* make sure we have some payload */ - if (cnt == 0) - return 0; - - switch (cmd) { - case VDO_CMD_VERSION: - /* guarantee last byte of payload is null character */ - *(payload + cnt - 1) = 0; - CPRINTF("version: %s\n", (char *)(payload+1)); - break; - case VDO_CMD_READ_INFO: - case VDO_CMD_SEND_INFO: - /* copy hash */ - if (cnt == 7) { - dev_id = VDO_INFO_HW_DEV_ID(payload[6]); - is_rw = VDO_INFO_IS_RW(payload[6]); - - is_latest = pd_dev_store_rw_hash(port, - dev_id, - payload + 1, - is_rw ? - SYSTEM_IMAGE_RW : - SYSTEM_IMAGE_RO); - - /* - * Send update host event unless our RW hash is - * already known to be the latest update RW. - */ - if (!is_rw || !is_latest) - pd_send_host_event(PD_EVENT_UPDATE_DEVICE); - - CPRINTF("DevId:%d.%d SW:%d RW:%d\n", - HW_DEV_ID_MAJ(dev_id), - HW_DEV_ID_MIN(dev_id), - VDO_INFO_SW_DBG_VER(payload[6]), - is_rw); - } else if (cnt == 6) { - /* really old devices don't have last byte */ - pd_dev_store_rw_hash(port, dev_id, payload + 1, - SYSTEM_IMAGE_UNKNOWN); - } - break; - case VDO_CMD_CURRENT: - CPRINTF("Current: %dmA\n", payload[1]); - break; - case VDO_CMD_FLIP: - usb_mux_flip(port); - break; -#ifdef CONFIG_USB_PD_LOGGING - case VDO_CMD_GET_LOG: - pd_log_recv_vdm(port, cnt, payload); - break; -#endif /* CONFIG_USB_PD_LOGGING */ - } - - return 0; -} - -#ifdef CONFIG_USB_PD_ALT_MODE_DFP -static int dp_flags[CONFIG_USB_PD_PORT_COUNT]; -/* DP Status VDM as returned by UFP */ -static uint32_t dp_status[CONFIG_USB_PD_PORT_COUNT]; - -static void svdm_safe_dp_mode(int port) -{ - /* make DP interface safe until configure */ - dp_flags[port] = 0; - dp_status[port] = 0; - usb_mux_set(port, TYPEC_MUX_NONE, - USB_SWITCH_CONNECT, pd_get_polarity(port)); -} - -static int svdm_enter_dp_mode(int port, uint32_t mode_caps) -{ - /* Only enter mode if device is DFP_D capable */ - if (mode_caps & MODE_DP_SNK) { - svdm_safe_dp_mode(port); - return 0; - } - - return -1; -} - -static int svdm_dp_status(int port, uint32_t *payload) -{ - int opos = pd_alt_mode(port, USB_SID_DISPLAYPORT); - - payload[0] = VDO(USB_SID_DISPLAYPORT, 1, - CMD_DP_STATUS | VDO_OPOS(opos)); - payload[1] = VDO_DP_STATUS(0, /* HPD IRQ ... not applicable */ - 0, /* HPD level ... not applicable */ - 0, /* exit DP? ... no */ - 0, /* usb mode? ... no */ - 0, /* multi-function ... no */ - (!!(dp_flags[port] & DP_FLAGS_DP_ON)), - 0, /* power low? ... no */ - (!!(dp_flags[port] & DP_FLAGS_DP_ON))); - return 2; -}; - -static int svdm_dp_config(int port, uint32_t *payload) -{ - int opos = pd_alt_mode(port, USB_SID_DISPLAYPORT); - int pin_mode = pd_dfp_dp_get_pin_mode(port, dp_status[port]); - - if (!pin_mode) - return 0; - - payload[0] = VDO(USB_SID_DISPLAYPORT, 1, - CMD_DP_CONFIG | VDO_OPOS(opos)); - payload[1] = VDO_DP_CFG(pin_mode, /* pin mode */ - 1, /* DPv1.3 signaling */ - 2); /* UFP connected */ - return 2; -}; - -static void svdm_dp_post_config(int port) -{ - dp_flags[port] |= DP_FLAGS_DP_ON; -} - -static int svdm_dp_attention(int port, uint32_t *payload) -{ - const struct usb_mux *mux = &usb_muxes[port]; - int lvl = PD_VDO_DPSTS_HPD_LVL(payload[1]); - int irq = PD_VDO_DPSTS_HPD_IRQ(payload[1]); - int mf_pref = PD_VDO_DPSTS_MF_PREF(payload[1]); - - dp_status[port] = payload[1]; - - mux->hpd_update(port, lvl, irq); - - if (lvl) - usb_mux_set(port, mf_pref ? TYPEC_MUX_DOCK : TYPEC_MUX_DP, - USB_SWITCH_CONNECT, pd_get_polarity(port)); - else - usb_mux_set(port, mf_pref ? TYPEC_MUX_USB : TYPEC_MUX_NONE, - USB_SWITCH_CONNECT, pd_get_polarity(port)); - - return 1; -} - -static void svdm_exit_dp_mode(int port) -{ - const struct usb_mux *mux = &usb_muxes[port]; - svdm_safe_dp_mode(port); - mux->hpd_update(port, 0, 0); -} - -static int svdm_enter_gfu_mode(int port, uint32_t mode_caps) -{ - /* Always enter GFU mode */ - return 0; -} - -static void svdm_exit_gfu_mode(int port) -{ -} - -static int svdm_gfu_status(int port, uint32_t *payload) -{ - /* - * This is called after enter mode is successful, send unstructured - * VDM to read info. - */ - pd_send_vdm(port, USB_VID_GOOGLE, VDO_CMD_READ_INFO, NULL, 0); - return 0; -} - -static int svdm_gfu_config(int port, uint32_t *payload) -{ - return 0; -} - -static int svdm_gfu_attention(int port, uint32_t *payload) -{ - return 0; -} - -const struct svdm_amode_fx supported_modes[] = { - { - .svid = USB_SID_DISPLAYPORT, - .enter = &svdm_enter_dp_mode, - .status = &svdm_dp_status, - .config = &svdm_dp_config, - .post_config = &svdm_dp_post_config, - .attention = &svdm_dp_attention, - .exit = &svdm_exit_dp_mode, - }, - { - .svid = USB_VID_GOOGLE, - .enter = &svdm_enter_gfu_mode, - .status = &svdm_gfu_status, - .config = &svdm_gfu_config, - .attention = &svdm_gfu_attention, - .exit = &svdm_exit_gfu_mode, - } -}; -const int supported_modes_cnt = ARRAY_SIZE(supported_modes); -#endif /* CONFIG_USB_PD_ALT_MODE_DFP */ - diff --git a/extra/usb_power/board.README b/extra/usb_power/board.README index fc9b055489..5ebfb382ca 100644 --- a/extra/usb_power/board.README +++ b/extra/usb_power/board.README @@ -16,24 +16,24 @@ Many configs can be found for the servo_ina_board in hdctools/servo/data/. Sweetberry is plug compatible with servo_ina headers, and config files can be converted with the following tool: -./convert_servo_ina.py kevin_r0_loc.py +./convert_servo_ina.py <board>_r0_loc.py -This will produce kevin_r0_loc.board and kevin_r0_loc.scenario which +This will produce <board>_r0_loc.board and <board>_r0_loc.scenario which can be used with powerlog.py. Method 2 (preferred) - -If you are using powerlog.py within the chroot, copy kevin_r0_loc.py to +If you are using powerlog.py within the chroot, copy <board>_r0_loc.py to src/third_party/hdctools/servo/data, then add line to file: config_type = 'sweetberry' and run command in terminal: sudo emerge hdctools The command will install the corresponding .board and .scenario file in the chroot. To use powerlog.py use the command: -./powerlog.py -b kevin_r0_loc.board -c kevin_r0_loc.scenario +./powerlog.py -b <board>_r0_loc.board -c <board>_r0_loc.scenario There is no need to specify the absolute path to the .board and .scenario file, once they are installed into the chroot. If there is any changes to -kevin_r0_loc.py, you need to emerge hdctools again. +<board>_r0_loc.py, you need to emerge hdctools again. Board files: diff --git a/util/flash_ec b/util/flash_ec index 59632cb769..eb23861044 100755 --- a/util/flash_ec +++ b/util/flash_ec @@ -119,9 +119,7 @@ BOARDS_NPCX_SPI=( eve fizz glkrvp - gru - kevin - lux + lux nami nautilus poppy @@ -158,8 +156,6 @@ BOARDS_MEC1322=( BOARDS_SPI_1800MV=( coral - gru - kevin reef ) @@ -167,8 +163,6 @@ BOARDS_RAIDEN=( coral eve fizz - gru - kevin nami nautilus poppy |