search

DatanoiseTV / PicoADK-Firmware-Template

🎵 🎹 Firmware boilerplate for the RP2040 / RP2350 powered PicoADK Audio Development Boards. Build your own stand alone synthesizers! Includes all nuts and bolts (FreeRTOS, USB MIDI, Vult DSP, Hardware Plumbing, DMA, ..). https://github.com/DatanoiseTV/PicoADK-Hardware
MIT License
149 stars 17 forks source link

How to activate the vult example dsp.vult instead the example in main.cpp #19

Open wehrend opened 1 year ago

wehrend commented 1 year ago

I've seen the example video from Floyd Steinberg where he claims that the dsp.vult can be replaced with any other example (noise, sine). This might work for software simulation of vult. But when compiling for the picoadk firmware I get the following errors:

[ 41%] Building CXX object CMakeFiles/main.dir/src/main.cpp.obj /home/sven/development/PicoADK-Firmware-Template/src/main.cpp: In function 'void i2s_callback_func()': /home/sven/development/PicoADK-Firmware-Template/src/main.cpp:230:32: error: 'Dsp_process_ret_0' was not declared in this scope; did you mean 'Dsp_process_type'? 230 | fix16_t left_out = Dsp_process_ret_0(ctx); | ^~~~~ | Dsp_process_type /home/sven/development/PicoADK-Firmware-Template/src/main.cpp:231:33: error: 'Dsp_process_ret_1' was not declared in this scope; did you mean 'Dsp_process_type'? 231 | fix16_t right_out = Dsp_process_ret_1(ctx); | ^~~~~ | Dsp_process_type make[2]: [CMakeFiles/main.dir/build.make:97: CMakeFiles/main.dir/src/main.cpp.obj] Error 1 make[1]: [CMakeFiles/Makefile2:1915: CMakeFiles/main.dir/all] Error 2

(I ommited the simple error of too few parameters in Dsp_Process) as I could solve this one for myself.

I've also seen that the header file in question vult.h must be generated from vultin.h and vultin.cpp, But I could not find the original functions for this two functions inside.

To Reproduce Steps to reproduce the behavior: copy the example noise.vult (or sine.vult) over dsp.vult export PICO_SDK_FETCH_FROM_GIT=1 mkdir build && cd build cmake .. make

Expected behavior invoking the noise.vult (or sine.vult) example given in dsp.vult

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

DatanoiseTV commented 1 year ago

Could you retry? I just commited a fix.

wehrend commented 1 year ago

Sure, I will review and test the changes this week. I found a workaround for this, but I am still not sure how to integrate the controls (potis) to it. That's another open question of mine.

Best,

Sven

On Mon, Jun 19, 2023, 9:00 AM Sylwester @.***> wrote:

Could you retry? I just commited a fix.

— Reply to this email directly, view it on GitHub https://github.com/DatanoiseTV/PicoADK-Firmware-Template/issues/19#issuecomment-1596617906, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMERECSIDW5LDJZ4NQQGT3XL72HXANCNFSM6AAAAAAYRM57YA . You are receiving this because you authored the thread.Message ID: @.***>

wehrend commented 1 year ago

This is only a change for the cmake file, I see. I dont think that this would solve the issue with the vult sources.