Closed tomcombriat closed 1 month ago
Memory usage change @ a5c26391de71d36589af9ac5ac6c19da490a55f9
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
:small_red_triangle: +24 - +32 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Fairly straightforward!
EXTERNAL_TIMED/PWM was initially tested to see that the small patch corrected the detuning.
PWM output has been switched to DMA buffered transfer using PWMAudio. Both mono and stereo have been tested. Crude EXTERNAL_TIMED (with pwm in user sketch) has been tested (not clean at all, the cycle is probably too low, but seems at the correct frequency).
Memory usage change @ 7cd06acced578eb569a3f19ad89c8bb759fb9df1
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
:small_red_triangle: +2168 - +2496 | +0.1 - +0.12 | :green_heart: -624 - -592 | -0.24 - -0.23 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Memory usage change @ b89e39f56d60d2f21c2dc7f7d15270dce054a342
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
:small_red_triangle: +2168 - +2496 | +0.1 - +0.12 | :green_heart: -624 - -592 | -0.24 - -0.23 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
PS: this incurs a quite penalty on flash, but is more beneficial for RAM in terms of percentage so I think this is worth it. There is no lack of flash on the RP…
Memory usage change @ fc3e0f90696a24f61127cd32ed2a514bb7aeb49e
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
:small_red_triangle: +2168 - +2496 | +0.1 - +0.12 | :green_heart: -624 - -592 | -0.24 - -0.23 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Memory usage change @ 2b0d752e63ab38f2adbc173b17d7f412c7a691ec
Board | flash | % | RAM for global variables | % |
---|---|---|---|---|
STMicroelectronics:stm32:GenF1:pnum=BLUEPILL_F103C8 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:mega |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:avr:uno |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:mbed_giga:giga |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:renesas_uno:minima |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
arduino:samd:adafruit_circuitplayground_m0 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
esp8266:esp8266:huzzah |
N/A | N/A | N/A | N/A |
rp2040:rp2040:rpipico |
:small_red_triangle: +2168 - +2496 | +0.1 - +0.12 | :green_heart: -624 - -592 | -0.24 - -0.23 |
teensy:avr:teensy36 |
0 - 0 | 0.0 - 0.0 | 0 - 0 | 0.0 - 0.0 |
teensy:avr:teensy41 |
N/A | N/A | N/A | N/A |
Ok for merging this?
Fix #251
Draft for now, until DMA driven PWM is implemented.