diff options
author | Shawn Nematbakhsh <shawnn@chromium.org> | 2017-02-04 11:58:16 -0800 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2017-02-13 18:25:51 -0800 |
commit | 1a736ed9542eb4c94ab0a49dba7696cda7faf864 (patch) | |
tree | 57d7a29b781f9afb33cd3da59983571096d74165 /board/kevin | |
parent | f51fdf223dd846341a489b00c8f43db92a37ce37 (diff) | |
download | chrome-ec-1a736ed9542eb4c94ab0a49dba7696cda7faf864.tar.gz |
kevin / gru: Reduce SRAM footprint
Remove console commands and add CONFIG options to reduce RAM usage.
BUG=chrome-os-partner:54099
BRANCH=gru
TEST=Verify charge_ramp CONFIG + task builds for gru.
Signed-off-by: Shawn Nematbakhsh <shawnn@chromium.org>
Change-Id: I2d7bc77d1fc032c6cb75eb1ec8d13dacb676658d
Reviewed-on: https://chromium-review.googlesource.com/437662
Commit-Ready: Shawn N <shawnn@chromium.org>
Tested-by: Shawn N <shawnn@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Diffstat (limited to 'board/kevin')
-rw-r--r-- | board/kevin/board.h | 15 | ||||
-rw-r--r-- | board/kevin/led_kevin.c | 28 |
2 files changed, 5 insertions, 38 deletions
diff --git a/board/kevin/board.h b/board/kevin/board.h index 6304d351eb..a12844249f 100644 --- a/board/kevin/board.h +++ b/board/kevin/board.h @@ -12,6 +12,7 @@ #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 @@ -77,7 +78,7 @@ /* 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 4096 +#define CONFIG_UART_TX_BUF_SIZE 2048 /* Sensors */ #define CONFIG_ACCEL_BMA255 @@ -156,17 +157,10 @@ #define PD_POWER_SUPPLY_TURN_OFF_DELAY 50000 /* us */ #define PD_VCONN_SWAP_DELAY 5000 /* us */ -/* Optional features for test commands */ -#define CONFIG_CMD_CHARGER_PSYS - #ifdef BOARD_GRU #define CONFIG_CMD_ALS #endif -/* Set PSYS gain for 50W max measurement */ -#define BD9995X_PSYS_GAIN_SELECT \ - BD9995X_CMD_PMON_IOUT_CTRL_SET_PMON_GAIN_SET_08UAW - #define CONFIG_UART_HOST 0 /* Optional feature - used by nuvoton */ @@ -182,18 +176,19 @@ #undef CONFIG_PSTORE /* Reduce code size */ -#ifdef BOARD_GRU #define CONFIG_COMMON_GPIO_SHORTNAMES #define GPIO_NAME_BY_PIN(port, index) #port#index -#endif +#undef CONFIG_CONSOLE_VERBOSE #define CONFIG_HOSTCMD_ALIGNED /* Modules we want to exclude */ +#undef CONFIG_CMD_BATTFAKE #undef CONFIG_CMD_FLASH #undef CONFIG_CMD_HASH #undef CONFIG_CMD_I2C_SCAN #undef CONFIG_CMD_MD +#undef CONFIG_CMD_MMAPINFO #undef CONFIG_CMD_POWERINDEBUG #undef CONFIG_CMD_TIMERINFO #undef CONFIG_CONSOLE_CMDHELP diff --git a/board/kevin/led_kevin.c b/board/kevin/led_kevin.c index a06c1553ef..2dda9499e8 100644 --- a/board/kevin/led_kevin.c +++ b/board/kevin/led_kevin.c @@ -142,31 +142,3 @@ static void led_tick(void) } } DECLARE_HOOK(HOOK_TICK, led_tick, HOOK_PRIO_DEFAULT); - -/******************************************************************/ -/* Console commands */ -static int command_led_color(int argc, char **argv) -{ - if (argc > 1) { - if (!strcasecmp(argv[1], "debug")) { - led_debug ^= 1; - CPRINTF("led_debug = %d\n", led_debug); - } else if (!strcasecmp(argv[1], "off")) { - set_color(LED_OFF); - } else if (!strcasecmp(argv[1], "red")) { - set_color(LED_RED); - } else if (!strcasecmp(argv[1], "green")) { - set_color(LED_GREEN); - } else if (!strcasecmp(argv[1], "blue")) { - set_color(LED_BLUE); - } else { - /* maybe handle charger_discharge_on_ac() too? */ - return EC_ERROR_PARAM1; - } - } - - return EC_SUCCESS; -} -DECLARE_CONSOLE_COMMAND(ledcolor, command_led_color, - "[debug|red|green|blue|off]", - "Change LED color"); |