Skip to content

DSP functions #682

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

Closed
superpanza opened this issue Oct 4, 2019 · 5 comments · Fixed by #685
Closed

DSP functions #682

superpanza opened this issue Oct 4, 2019 · 5 comments · Fixed by #685
Assignees
Labels
enhancement New feature or request

Comments

@superpanza
Copy link

Hello, how can i use the CMSIS-DSP functions?
I'm trying to perform an FFT over some samples but i'm not able to import arm_math.h from DSP lib. It does not find any math_arm.h lib from filesystem. I'm pretty sure that the header file is there (view img
Screenshot 2019-10-04 11 59 00
)
i'm using a nucleoF401RE board, and i've installed the stm32core via Arduino IDE board manager

@fpistm
Copy link
Member

fpistm commented Oct 4, 2019

Hi @superpanza
Currently, this is not provided.
I've already answer this kind of issue here:
#569 (comment)

I think I will add a way to use it.

@superpanza
Copy link
Author

Great!
Thank you for considering adding the feature.
I've just tried to apply the solution you suggest in #569 and now the lib (arm_math.h) is loaded, the code compile successfully, but the linking phase fail with something like

/Users/userino/Library/Arduino15/packages/STM32/tools/arm-none-eabi-gcc/8.2.1-1.7/bin/../lib/gcc/arm-none-eabi/8.2.1/../../../../arm-none-eabi/bin/ld: FFTTest.ino.cpp:(.text._Z6armFFTPfS_i+0x1a): undefined reference to `arm_cmplx_mag_f32'

The prototype of that function is clearly in arm_math.h
Any idea on how to move on?

@fpistm
Copy link
Member

fpistm commented Oct 4, 2019

I guess, you have to add the c source file defining the function in the build:
CMSIS/5.5.1/CMSIS/DSP/Source/ComplexMathFunctions/arm_cmplx_mag_f32.c

like I've made for the filtering:
fpistm@e891921#diff-766ab07e0a5996a0f67900b49895d65f

@fpistm
Copy link
Member

fpistm commented Oct 4, 2019

Do not hesitate to provide a small test sketch then I will be able to test when I will implement this request.

fpistm added a commit to fpistm/Arduino_Core_STM32 that referenced this issue Oct 5, 2019
Fixes stm32duino#682

Signed-off-by: Frederic Pillon <[email protected]>
fpistm added a commit to fpistm/Arduino_Core_STM32 that referenced this issue Oct 7, 2019
To use it, add:
#include <CMSIS_DSP.h>

Fixes stm32duino#682

Signed-off-by: Frederic Pillon <[email protected]>
@fpistm
Copy link
Member

fpistm commented Oct 7, 2019

Hi @superpanza
I've added the support of CMSIS DSP software library here #685
Could you test it and give me your feedback, please ?
Thanks in advance.

@fpistm fpistm added this to the 1.8.0 milestone Oct 7, 2019
@fpistm fpistm self-assigned this Oct 8, 2019
@fpistm fpistm added the waiting feedback Further information is required label Oct 8, 2019
fpistm added a commit to fpistm/Arduino_Core_STM32 that referenced this issue Oct 14, 2019
To use it, add:
#include <CMSIS_DSP.h>

Fixes stm32duino#682

Signed-off-by: Frederic Pillon <[email protected]>
@fpistm fpistm removed the waiting feedback Further information is required label Oct 15, 2019
@fpistm fpistm added enhancement New feature or request and removed Request labels Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants