search

espressif / esp-dsp

DSP library for ESP-IDF
Apache License 2.0
442 stars 87 forks source link

ESP-IDF 5.1 Support? (DSP-110) #72

Closed timsterc closed 11 months ago

timsterc commented 1 year ago

I'm getting a fair number of errors when trying to build w/ the esp-idf-5.1rc2. Curious if there is a schedule for when esp-dsp will be updated to build w/ 5.1?

dmitry1945 commented 1 year ago

Hi @timsterc

Yes, it supports.

Are you able to build test_app or any esp-dsp example?

Regards, Dmitry

PS. I've checked and able to build.

timsterc commented 1 year ago

Ah, my bad - I forgot that dsps_biquad_f32_aes3.S was missing and had added it in manually. Is there a reason that was left out of the CMakeLists.txt ?

                "modules/iir/biquad/dsps_biquad_f32_aes3.S"
dmitry1945 commented 1 year ago

@timsterc

yes, the reason that this is a bug. Just forgot to include. We will fix it.

Thank you for the feedback.

Regards, Dmitry

timsterc commented 1 year ago

@dmitry1945 - great, please let me know when it's updated.

dmitry1945 commented 1 year ago

@timsterc the fix is in master. Please pull.

Regards, Dmitry

timsterc commented 1 year ago

@dmitry1945 - I'm using the managed_component feature - idf.py add-dependency "espressif/esp-dsp" - in my project - but it seems like it's still pulling the older CMakeLists.txt file?

timsterc commented 1 year ago

Changed my idf_component.yml file to specifically point to the git repo instead of using the component registry - that did the trick.

espressif/esp-dsp: git: https://github.com/espressif/esp-dsp.git

dmitry1945 commented 1 year ago

@timsterc yes.

Now the component manager files are up to date with lates changes. It should work.

Thanks & best Regards, Dmitry