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

drivers: sensors: Add SENSOR_CHAN_FREQUENCY channel #83511

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

nandojve
Copy link
Member

@nandojve nandojve commented Jan 2, 2025

Add new SENSOR_CHAN_FREQUENCY constant to allow implement sensors that output frequency values in Hertz.

@henrikbrixandersen
Copy link
Member

You need to add the new sensor channel to sensor_shell.c as well.

@nandojve nandojve force-pushed the sensors/add-frequency branch 2 times, most recently from 34370cc to ca790a7 Compare January 3, 2025 15:04
@zephyrbot zephyrbot added the area: Samples Samples label Jan 3, 2025
@zephyrbot zephyrbot requested review from kartben and nashif January 3, 2025 15:04
teburd
teburd previously approved these changes Jan 3, 2025
jeppenodgaard
jeppenodgaard previously approved these changes Jan 5, 2025
ubieda
ubieda previously approved these changes Jan 7, 2025
asemjonovs
asemjonovs previously approved these changes Jan 8, 2025
MaureenHelm
MaureenHelm previously approved these changes Jan 9, 2025
@kartben
Copy link
Collaborator

kartben commented Jan 9, 2025

@nandojve needs rebase please

Add new SENSOR_CHAN_FREQUENCY constant to allow implement sensors that
output frequency values in Hertz.

Signed-off-by: Gerson Fernando Budke <[email protected]>
@nandojve nandojve dismissed stale reviews from jeppenodgaard and teburd via 99f0fc4 January 9, 2025 16:41
@nandojve nandojve force-pushed the sensors/add-frequency branch from ca790a7 to 99f0fc4 Compare January 9, 2025 16:41
@kartben kartben merged commit ac579a8 into zephyrproject-rtos:main Jan 14, 2025
24 checks passed
@nandojve nandojve deleted the sensors/add-frequency branch January 14, 2025 20:27
@otavio
Copy link
Contributor

otavio commented Jan 16, 2025

Would be possible to backport this change to the 4.0 branch?

@henrikbrixandersen
Copy link
Member

Would be possible to backport this change to the 4.0 branch?

Only bugfixes are backported. You can add a custom channel definition out-of-tree, though.

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.

10 participants