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

sensors: Create sensor trig subcommand #58257

Merged

Conversation

tristan-google
Copy link
Collaborator

This PR relocates the sensor trigger sample application from the sensor_shell sample to a subcommand in the actual sensor shell. The subcommand has a UI for enabling a given trigger on a given sensor. A built-in handler for the data_ready trigger is included that prints the latest data to the log. Currently, only SENSOR_TRIG_DATA_READY is supported but the groundwork is there to add others. Tested on a tdk_robokit1 board.

drivers/sensor/sensor_shell.c Outdated Show resolved Hide resolved
drivers/sensor/sensor_shell.c Outdated Show resolved Hide resolved
drivers/sensor/sensor_shell.c Outdated Show resolved Hide resolved
drivers/sensor/sensor_shell.c Outdated Show resolved Hide resolved
yperess
yperess previously approved these changes May 25, 2023
drivers/sensor/sensor_shell.c Outdated Show resolved Hide resolved
yperess
yperess previously approved these changes May 25, 2023
teburd
teburd previously approved these changes May 25, 2023
@nashif
Copy link
Member

nashif commented May 26, 2023

needs rebase

This PR relocates the sensor trigger sample application from the
`sensor_shell` sample to a subcommand in the actual sensor shell. The
subcommand has a UI for enabling a given trigger on a given sensor.
A built-in handler for the data_ready trigger is included that prints
the latest data to the log. Currently, only `SENSOR_TRIG_DATA_READY` is
supported but the groundwork is there to add others. Tested on a
`tdk_robokit1` board.

Signed-off-by: Tristan Honscheid <honscheid@google.com>
@tristan-google tristan-google dismissed stale reviews from teburd and yperess via 1d7029d May 26, 2023 22:21
@nashif nashif merged commit dc9bd3a into zephyrproject-rtos:main May 27, 2023
24 checks passed
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.

None yet

6 participants