Cleanup

README.md

@@ -104,4 +104,9 @@ If you are under OSX - you can install MASM via HomeBrew and use it to compile a
 - https://coronax.wordpress.com/projects/project65/
 - http://archive.computerhistory.org/resources/text/Apple/Apple.AppleI.1976.102646518.pdf (apple 1 official manual)
 - https://en.wikipedia.org/wiki/WDC_65C02
-- http://www.westerndesigncenter.com/wdc/w65c21-chip.cfm
+- http://www.westerndesigncenter.com/wdc/w65c21-chip.cfm
+
+
+## TODO
+    - Hook to real keyboard - frame rate of 60.05 Hz, could support 40 characters per line at 24 lines, with automatic scrolling. 
+    - Hook to real display - 

lib/readme.txt

@@ -32,7 +32,5 @@ Then in `src/main.c` you should use:
 PlatformIO will find your libraries automatically, configure preprocessor's
 include paths and build them.
 
-See additional options for PlatformIO Library Dependency Finder `lib_*`:
+More information about PlatformIO Library Dependency Finder
-
+- http://docs.platformio.org/page/librarymanager/ldf.html
-http://docs.platformio.org/en/latest/projectconf.html#lib-install
-

platformio.ini

@@ -1,23 +1,14 @@
-#
+; PlatformIO Project Configuration File
-# Project Configuration File
+;
-#
+;   Build options: build flags, source filter
-# A detailed documentation with the EXAMPLES is located here:
+;   Upload options: custom upload port, speed and extra flags
-# http://docs.platformio.org/en/latest/projectconf.html
+;   Library options: dependencies, extra library storages
-#
+;   Advanced options: extra scripting
-
+;
-# A sign `#` at the beginning of the line indicates a comment
+; Please visit documentation for the other options and examples
-# Comment lines are ignored.
+; http://docs.platformio.org/page/projectconf.html
-
-# Simple and base environment
-# [env:mybaseenv]
-# platform = %INSTALLED_PLATFORM_NAME_HERE%
-# framework =
-# board =
-#
-# Automatic targets - enable auto-uploading
-# targets = upload
 
 [env:due]
 platform = atmelsam
-framework = arduino
 board = due
+framework = arduino

src/main.cpp

@@ -1,4 +1,4 @@
-#include "Arduino.h"
+#include <Arduino.h>
 #include "rom.h"
 #include "programs.h"
 
@@ -147,6 +147,7 @@ void byteToDataBus(unsigned char data) {
 void PIAWrite() {
   switch (address) {
 
+    // Keyboard
     case KBD_ADDR:
       KBD=bus_data;
       break;
@@ -155,8 +156,9 @@ void PIAWrite() {
       KBDCR=bus_data;
       break;
 
+    // Display
     case DSP_ADDR:
-      DSP = bus_data;
+      DSP=bus_data;
 
       switch(DSP) {
         case CR:
@@ -219,6 +221,7 @@ unsigned char PIARead() {
     case DSPCR_ADDR:
       val=DSPCR;
       break;
+
     default:
       val=0;
       break;
@@ -231,22 +234,33 @@ void writeToDataBus() {
   unsigned char val=0;
 
   switch (address >> 12) {
+
+    // $0000-$0FFF 4KB Standard RAM
     case 0x0:
       val=RAM_BANK_1[address-RAM_BANK1_ADDR];
       break;
+
+    // $E000-$EFFF 4KB Extended RAM
     case 0xE:
       val=RAM_BANK_2[address-RAM_BANK2_ADDR];
       break;
+
+    // $FF00-$FFFF 256 Bytes ROM
     case 0xF:
       val=ROM[address-ROM_ADDR];
       break;
+
+    // $D010-$D013 PIA (6821) [KBD & DSP]
     case 0xD:
       val=PIARead();
       break;
+
+    // Segmentation Fault. Just return 0
     default:
       val=0;
       break;
   }
+
   byteToDataBus(val);
 }
 
@@ -345,8 +359,7 @@ void handleClock() {
 }
 
 void handleBusRW() {
-
+  // If nothing changed from the last cycle, we don't need to update anything
-  // If nothing changed from the last cycle, we don't need to upadte anything
   if (pre_address != address || pre_rw_state != rw_state) {
     // READ OR WRITE TO BUS?
     rw_state ? writeToDataBus() : readFromDataBus();
@@ -356,7 +369,6 @@ void handleBusRW() {
 }
 
 void step() {
-
   CLOCK_DELAY=analogRead(CLOCK_DELAY_PIN); // Can be removed, see setup()
   handleClock();
   readAddress();