-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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: cmsis_dsp: add moving average sample #66726
Conversation
4ae7fe4
to
c2e2f2a
Compare
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
01cb322
to
2542423
Compare
This adds a basic example demonstrating how to enable and use the CMSIS-DSP module in Zephyr. Signed-off-by: Benjamin Cabé <[email protected]>
2542423
to
5608fd5
Compare
tags: | ||
- samples | ||
integration_platforms: | ||
- qemu_cortex_m0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Using mps2_an521_remote
here would be a nice opportunity to show MVE in action :).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like it
/* Apply the FIR filter to the input data and measure how many cycles this takes */ | ||
start = k_cycle_get_32(); | ||
arm_fir_q31(&sFIR, input, output, BLOCK_SIZE); | ||
end = k_cycle_get_32(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does averaging 32 samples take long enough to show up at the cycle resolution?
The standard API for profiling durations is zephyr/timing/timing.h
with CONFIG_TIMING_FUNCTIONS=y
from my understanding.
This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time. |
This adds a basic example demonstrating how to enable and use the CMSIS-DSP module in Zephyr.