diff options
| author | Tzung-Bi Shih <tzungbi@kernel.org> | 2026-07-15 02:44:53 +0000 |
|---|---|---|
| committer | Tzung-Bi Shih <tzungbi@kernel.org> | 2026-07-16 02:40:57 +0000 |
| commit | 9a3f43b30373c61477d0d3ab52946c05f9492bf9 (patch) | |
| tree | 749884efcb224e8357a590da664e1d4ca2a52dc2 /tools/perf/scripts/python/bin/stackcollapse-record | |
| parent | d1ceb2b2324717fa30b44d56ef0c52813e239569 (diff) | |
platform/chrome: sensorhub: Fix dropped timestamp events and log spam
Commit 833740a2333c ("platform/chrome: sensorhub: Bound the EC-reported
sensor number") evaluated the `sensor_num` against the bounds limit even
for timestamp events. A timestamp event typically has a `sensor_num` of
0xff [1], causing the driver to flag it as invalid and skip to the next
event.
As a result, we'd see a flooding of "Invalid sensor number 255 from EC"
warning logs and these timestamp events were being dropped.
Move the bounds-check into cros_ec_sensor_ring_process_event() and
evaluate it only after standalone timestamp events have already been
processed and returned early.
[1] https://crrev.com/219ca6ef82ba266da788b673ee4ad50bd3ea1285/common/motion_sense_fifo.c#427
Fixes: 833740a2333c ("platform/chrome: sensorhub: Bound the EC-reported sensor number")
Reviewed-by: Tomasz Figa <tfiga@chromium.org>
Link: https://lore.kernel.org/r/20260715024454.4127571-1-tzungbi@kernel.org
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/bin/stackcollapse-record')
0 files changed, 0 insertions, 0 deletions
