diff --git a/emulator-patches/kinetoscope.c b/emulator-patches/kinetoscope.c index 41e9af8..849686e 100644 --- a/emulator-patches/kinetoscope.c +++ b/emulator-patches/kinetoscope.c @@ -195,9 +195,9 @@ static void write_sram(const uint8_t* data, uint32_t size) { for (size_t i = 0; i < size; i++) { // XOR with 1 swaps every 2 bytes in the output. - // TODO: WHY? This shouldn't be necessary! Is this a bug in the emulator? - // An assumption about what kind of data is in a direct-access buffer? - // Should this nonsense only happen for little-endian builds? + // You wouldn't assume this to be necessary, but the emulated memory is + // accessed as big-endian 16-bit ints or something. + // TODO: Should this nonsense only happen for little-endian builds? kinetoscope.sram_buffer[kinetoscope.sram_offset ^ 1] = data[i]; kinetoscope.sram_offset++; } diff --git a/firmware/firmware.ino b/firmware/firmware.ino index c4c97cb..f91a6b5 100644 --- a/firmware/firmware.ino +++ b/firmware/firmware.ino @@ -358,12 +358,6 @@ static void process_command(uint8_t command, uint8_t arg) { } } -#ifdef DEBUG - // Don't clear the flag too quickly. - // TODO: Determine if this is necessary once hardware prototypes are working. - delay(10 /* ms */); -#endif - clear_cmd(); Serial.println("Command complete."); }