Skip to content
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

memcpy_tointerleave allow AUDIO_BLOCK_SAMPLES == 4 #423

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ericfont
Copy link
Contributor

@ericfont ericfont commented Dec 7, 2021

Previous code won't work for AUDIO_BLOCK_SAMPLES < 8. So I would like to add a case for AUDIO_BLOCK_SAMPLES == 4, using the same pattern of interleaving code as for AUDIO_BLOCK_SAMPLES == 8, but just half as much.

I've tested on my Teensy 4.1 with Audio Board Rev D with L/R audio for memcpy_tointerleaveLR, as well as with L-only and R-only for memcpy_tointerleaveL and memcpy_tointerleaveR.

@PaulStoffregen
Copy link
Owner

looks fine, but now conflicting due to #424

@ericfont ericfont force-pushed the memcpy_interleave_4_AUDIO_BLOCK_SAMPLES branch from eba80d6 to 46117a7 Compare December 7, 2021 19:17
@ericfont
Copy link
Contributor Author

ericfont commented Dec 7, 2021

ok, I fixed that conflict and repushed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants