Am 22.09.2021 um 22:11 schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
Hi,
Few comments...
* Code calls Statusbar_SetDspLed(), but that's missing
Sorry, that is an oversight. It is specific to Previous and indicates that the DSP is active.
* Add defines for magic (mode / mode_wait) constants
I do not understand. Mode is a number between 0 and 3 (you find it in the datasheet of the 56001) and mode_wait is for the timed bootstrap ROM enable/disable.
Why BOOTSTRAP_ROM ifdef is needed i.e. what downside it has?
At least with Previous the extension causes some loss of efficiency because the waiting loop (waiting for program to be supplied by the host) is permanently running.