electro-smith / DaisyDuino

Arduino Support for the Daisy Audio Platform.
https://www.electro-smith.com/daisy
MIT License
92 stars 20 forks source link

fixed sai redefinition issues #29

Closed CorvusPrudens closed 3 years ago

CorvusPrudens commented 3 years ago

Also changed all imports from hal with double quotes to angle brackets. This should indicate the source of these imports a little better (i.e. not local files).

This addresses the problem raised in #27.

The example sketches build fine on my system (Linux / Ubuntu), and I doubt any other users will encounter platform-specific issues.

beserge commented 3 years ago

There's a little bandaid in audio.cpp we need to remove. Line 448.

//temporary bandaid until the cache fix is released via stm32duino 
for(int i = 0; i < kAudioMaxChannels / 2; i++){
    dsy_dma_invalidate_cache_for_buffer(dsy_audio_rx_buffer[i], kAudioMaxBufferSize);
    dsy_dma_clear_cache_for_buffer(dsy_audio_tx_buffer[i], kAudioMaxBufferSize);
}

Still works fine with it in, but it's not needed any more.