diff options
author | Enrico Granata <egranata@chromium.org> | 2019-03-14 17:02:47 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-03-26 04:43:11 -0700 |
commit | 184410c9ee7530404813f5bd0a80a0a9d21f2a5a (patch) | |
tree | 44f76ee1cf0213cc673d0449de0eb49b6ae3e1c8 /include | |
parent | 5ea11347a3600d898ba473884735ff11f758fb9d (diff) | |
download | chrome-ec-184410c9ee7530404813f5bd0a80a0a9d21f2a5a.tar.gz |
motion: Split configuration for sensor FIFO and tight timestamps
CONFIG_ACCEL_FIFO was being used both to control the size of the
sensor FIFO, and the notion of tight timestamps. The latter is
related to the format the EC uses to send sensor event timestamps
and not to the size of the FIFO.
Split this latter portion into its own configuration flag,
CONFIG_SENSOR_TIGHT_TIMESTAMPS.
This defaults to enabled, and should be turned on for all new
boards. It will be selectively disabled on a few boards where
the AP-side filtering this enables does not perform optimally
due to jitter issues.
BUG=b:123700100
BRANCH=rammus
TEST=observe tight_timestamps on the sensor_ring device in kernel
be enabled/disabled depending on whether CONFIG_SENSOR_TIGHT_TIMESTAMPS
is #undef'ed or not
Change-Id: I806ba6bb45a0007512afec9151c57c60d30fd604
Signed-off-by: Enrico Granata <egranata@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1524666
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Reviewed-by: Alexandru M Stan <amstan@chromium.org>
Diffstat (limited to 'include')
-rw-r--r-- | include/config.h | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/include/config.h b/include/config.h index 5607fecde2..bd31e034fd 100644 --- a/include/config.h +++ b/include/config.h @@ -151,6 +151,30 @@ /* Specify type of Gyrometers attached. */ #undef CONFIG_GYRO_L3GD20H +/* + * If this is defined, motion_sense sends sensor events to the AP in the format + * +-----------+ + * | Timestamp | + * | Payload | + * | Timestamp | + * | Payload | + * | ... | + * +-----------+ + * + * If this is not defined, the events will be sent in the format + * +-----------+ + * | Payload | + * | Payload | + * | Payload | + * | ... | + * | Timestamp | + * +-----------+ + * + * The former format enables improved filtering of sensor event timestamps on + * the AP, but comes with stricter jitter requirements. + */ +#define CONFIG_SENSOR_TIGHT_TIMESTAMPS + /* Sync event driver */ #undef CONFIG_SYNC |