From d10e14b37909545f161b40f8a1d9cfb041356d21 Mon Sep 17 00:00:00 2001 From: Nicolas Boichat Date: Wed, 28 Mar 2018 11:07:23 +0800 Subject: poppy: Increase console buffer size to 4096 bytes Increase the size from 1024 to 4096 to reduce the likelihood of overflows. To make space for the larger buffer, we reduce CONFIG_ACCEL_FIFO to 512 entries (from 1024 entries: saves 4096 bytes of RAM). grep shared_mem_buf build/lux/RW/ec.RW.smap Before this patch: 200c74e0 D __shared_mem_buf After this patch: 200c70e0 D __shared_mem_buf (we saved 1024 bytes of RAM) BRANCH=poppy BUG=b:77159941, b:120097505 TEST=Flash lux, see that we do not lose EC logs in /var/log/cros_ec.log. Change-Id: I320c370369364b280e59f490a86f057fbb502da3 Reviewed-on: https://chromium-review.googlesource.com/c/1418254 Reviewed-by: Nicolas Boichat Commit-Queue: Enrico Granata Tested-by: Enrico Granata --- board/poppy/board.h | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/board/poppy/board.h b/board/poppy/board.h index 562cf77223..5c39e7ecfb 100644 --- a/board/poppy/board.h +++ b/board/poppy/board.h @@ -127,7 +127,7 @@ #define CONFIG_EC_MAX_SENSOR_FREQ_MILLIHZ 100000 /* FIFO size is in power of 2. */ -#define CONFIG_ACCEL_FIFO 1024 +#define CONFIG_ACCEL_FIFO 512 /* Depends on how fast the AP boots and typical ODRs */ #define CONFIG_ACCEL_FIFO_THRES (CONFIG_ACCEL_FIFO / 3) @@ -135,6 +135,9 @@ #define CONFIG_TABLET_MODE #define CONFIG_TABLET_MODE_SWITCH +#undef CONFIG_UART_TX_BUF_SIZE +#define CONFIG_UART_TX_BUF_SIZE 4096 + /* USB */ #define CONFIG_USB_CHARGER #define CONFIG_USB_PD_ALT_MODE -- cgit v1.2.1