platform/chrome: cros_ec_sensorhub: Fix EC timestamp overflow
authorGwendal Grignou <gwendal@chromium.org>
Tue, 30 Jun 2020 07:52:03 +0000 (00:52 -0700)
committerEnric Balletbo i Serra <enric.balletbo@collabora.com>
Tue, 30 Jun 2020 14:35:22 +0000 (16:35 +0200)
EC is using 32 bit timestamps (us), and before converting it to 64bit
they were not casted, so it would overflow every 4s.
Regular overflow every ~70 minutes was not taken into account either.

Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
drivers/platform/chrome/cros_ec_sensorhub_ring.c

index 24e48d9..b1c641c 100644 (file)
@@ -419,9 +419,7 @@ cros_ec_sensor_ring_process_event(struct cros_ec_sensorhub *sensorhub,
                         * Disable filtering since we might add more jitter
                         * if b is in a random point in time.
                         */
-                       new_timestamp = fifo_timestamp -
-                                       fifo_info->timestamp  * 1000 +
-                                       in->timestamp * 1000;
+                       new_timestamp = c - b * 1000 + a * 1000;
                        /*
                         * The timestamp can be stale if we had to use the fifo
                         * info timestamp.