Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

samples: sensor: lis2dw12: add sample with double tap detection. #83818

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pcurt
Copy link

@pcurt pcurt commented Jan 10, 2025

The LIS2DW12 is an advanced 3-axis linear accelerometer.
The driver exists in Zephyr but it is not very easy to configure it and use with trigger detection.

So I added a samples that configure the accelerometer to detect a double tap, the example is running on a stm32f3_disco board with a steval-mki190v1 adapter board

Copy link
Collaborator

@avisconti avisconti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just few comments.
Also note that you have a typo in the commit message ("aslo" instead of "also")

samples/sensor/lis2dw12/README.rst Outdated Show resolved Hide resolved
samples/sensor/lis2dw12/src/main.c Show resolved Hide resolved
samples/sensor/lis2dw12/src/main.c Show resolved Hide resolved
This example configures a lis2dw12 accelerometer to detect a double
tap with a basic configuration. It can also read the sensor in
polling mode. The example also reads the internal temperature
sensor.

Signed-off-by: Pierrick Curt <pierrickcurt@gmail.com>
@pcurt pcurt force-pushed the feat_lis2dw12_sample branch from 2e17e21 to 16f1526 Compare January 10, 2025 16:32
@pcurt
Copy link
Author

pcurt commented Jan 10, 2025

Just few comments. Also note that you have a typo in the commit message ("aslo" instead of "also")

Fixed, thanks

@pcurt pcurt requested a review from avisconti January 10, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants