djsundog/circuitpython
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #2996 from jepler/mixer-bugfixes

Browse Source 
audiomixer: Fix garbled playback when voice 0 is stopped, audio pops
This commit is contained in:
Scott Shawcroft 2020-06-02 12:50:05 -07:00 committed by GitHub
commit bb3f54a1cd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 12 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
shared-module/audiomixer/Mixer.c
View File

@ -143,19 +143,17 @@ static inline uint32_t pack8(uint32_t val) {
static void mix_down_one_voice(audiomixer_mixer_obj_t* self, static void mix_down_one_voice(audiomixer_mixer_obj_t* self,
audiomixer_mixervoice_obj_t* voice, bool voices_active, audiomixer_mixervoice_obj_t* voice, bool voices_active,
uint32_t* word_buffer, uint32_t length) { uint32_t* word_buffer, uint32_t length) {
bool voice_done = voice->sample == NULL; while (length != 0) {
while (!voice_done && length != 0) {
if (voice->buffer_length == 0) { if (voice->buffer_length == 0) {
if (!voice->more_data) { if (!voice->more_data) {
if (voice->loop) { if (voice->loop) {
audiosample_reset_buffer(voice->sample, false, 0); audiosample_reset_buffer(voice->sample, false, 0);
} else { } else {
voice->sample = NULL; voice->sample = NULL;
voice_done = true;
break; break;
} }
} }
if (!voice_done) { if (voice->sample) {
// Load another buffer // Load another buffer
audioio_get_buffer_result_t result = audiosample_get_buffer(voice->sample, false, 0, (uint8_t**) &voice->remaining_buffer, &voice->buffer_length); audioio_get_buffer_result_t result = audiosample_get_buffer(voice->sample, false, 0, (uint8_t**) &voice->remaining_buffer, &voice->buffer_length);
// Track length in terms of words. // Track length in terms of words.
@ -230,10 +228,8 @@ static void mix_down_one_voice(audiomixer_mixer_obj_t* self,
} }
if (length && !voices_active) { if (length && !voices_active) {
uint32_t sample_value = self->bits_per_sample == 8
? 0x80808080 : 0x80008000;
for (uint32_t i = 0; i<length; i++) { for (uint32_t i = 0; i<length; i++) {
word_buffer[i] = sample_value; word_buffer[i] = 0;
} }
} }
} }
@ -269,10 +265,17 @@ audioio_get_buffer_result_t audiomixer_mixer_get_buffer(audiomixer_mixer_obj_t*
for (int32_t v = 0; v < self->voice_count; v++) { for (int32_t v = 0; v < self->voice_count; v++) {
audiomixer_mixervoice_obj_t* voice = MP_OBJ_TO_PTR(self->voice[v]); audiomixer_mixervoice_obj_t* voice = MP_OBJ_TO_PTR(self->voice[v]);
if(voice->sample) {
mix_down_one_voice(self, voice, voices_active, word_buffer, length); mix_down_one_voice(self, voice, voices_active, word_buffer, length);
voices_active = true; voices_active = true;
} }
}
if (!voices_active) {
for (uint32_t i = 0; i<length; i++) {
word_buffer[i] = 0;
}
}
if (!self->samples_signed) { if (!self->samples_signed) {
if (self->bits_per_sample == 16) { if (self->bits_per_sample == 16) {