ARM-software / CMSIS_5

CMSIS Version 5 Development Repository
http://arm-software.github.io/CMSIS_5/index.html
Apache License 2.0
1.33k stars 1.08k forks source link

Support large FFT lengths for arm_rfft_fast_f32 #364

Closed WBABD closed 4 years ago

WBABD commented 6 years ago

I think support for FFTs up to 16,384 would be very useful for general use.
I understand there would be a huge code size hit due to the twiddleCoef array needed but perhaps there could be a makefile define to enable this functionality.

Thank you for this fantastic library anyway.

christophe0606 commented 5 years ago

We are currently working on some changes which will allow to only include the needed tables in CMSIS-DSP. So, it should be possible to support bigger FFT lengths without impact on the code size since if the table is not needed it won't be included.

WBABD3 commented 5 years ago

Thank you christophe0606

alexheslop commented 4 years ago

Hi @christophe0606, what is the current maximum FFT size available?

christophe0606 commented 4 years ago

Improvements to the FFT will require a full rework since we need to address many different requests and needs.

So, this won't be fixed in current implementation but we keep track of the need and request for future.