From a7ad347585dd305ea55e6a0d7f8fd97f58ebfb33 Mon Sep 17 00:00:00 2001 From: Alexandru M Stan Date: Fri, 13 Jul 2018 17:41:43 -0700 Subject: scarlet: Change FIFO settings Similar to CL:1134494 Reduce the amount of datum to trigger a FIFO interruption. To improve timestamping in the kernel, we need to send a continuous stream of data to the host. In case of CTS batch test, we may not send anything for x100ms; the filter will not have any data to estimate the drift and may output invalid timestamps. Change the FIFO threshold to try to have more than ~10 samples in a singe batch. We still need a long FIFO when the EC is busy and can not process events from the sensor. BUG=b:73551961 BRANCH=scarlet TEST=in CtsHardwareTestCases, tests SensorBatchingTests pass more reliably. Change-Id: I254230498fcf270dfa303cf5eacec5d8abdd1225 Signed-off-by: Gwendal Grignou Signed-off-by: Alexandru M Stan Reviewed-on: https://chromium-review.googlesource.com/1137454 Reviewed-by: Philip Chen --- board/scarlet/board.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/scarlet/board.h b/board/scarlet/board.h index 2d562a9787..a6120be824 100644 --- a/board/scarlet/board.h +++ b/board/scarlet/board.h @@ -99,7 +99,7 @@ /* FIFO size is in power of 2. */ #define CONFIG_ACCEL_FIFO 256 -#define CONFIG_ACCEL_FIFO_THRES (CONFIG_ACCEL_FIFO / 3) +#define CONFIG_ACCEL_FIFO_THRES 10 /* USB PD config */ #define CONFIG_CHARGE_MANAGER -- cgit v1.2.1