Commits

iorodeo committed 22b959f

Last check of old version.

Comments (0)

Files changed (14)

comm_firmware/comm_firmware.ino

 #include <Wire.h>
 #include <Streaming.h>
+#include "i2cmaster.h"
 #include "panels_g4.h"
 
-const uint8_t SLAVE_ADDRESS = 1; 
 
+void speedTest();
 void displayColTest();
 void displayRowTest();
 void displayPixelTest();
 
 void setup()
 {
-    Wire.begin();
+    pinMode(A2,OUTPUT);
+    //Wire.begin();
+    i2c_init();   
+    //TWBR = 50;
+    //TWBR = 4;
+    //TWBR = 2;  // 1MHz
+    TWBR = 17;   // 400kHz
 }
 
 void loop()
 {
-    displayColTest();
+    speedTest();
+    //displayColTest();
     //displayRowTest();
     //displayPixelTest();
     //displayOnAndOffTest();
 // Test functions
 // ----------------------------------------------------------------------------
 
+
+void speedTest()
+{
+    while(1)
+    {
+        const uint8_t numGrayScaleValues = 2;
+        static bool isFirst = true;
+        static uint8_t row = 0;
+        static I2CMessageBuffer msgArray[DISPLAY_MATRIX_SIZE];
+
+        if (isFirst)
+        {
+            for (uint8_t i=0; i<DISPLAY_MATRIX_SIZE; i++)
+            {
+                DisplayMatrix  matrix;
+                matrix.setNumGrayScaleValues(numGrayScaleValues);
+                matrix.setAllZero();
+                matrix.setCol(i,numGrayScaleValues-1);
+                matrix.getMsgFromData(msgArray[i]);
+            }
+            isFirst = false;
+        }
+
+        for (uint8_t slaveInd=0; slaveInd < 1; slaveInd++) //I2C_NUM_SLAVES; slaveInd++)
+        {
+            uint8_t slaveAddress = I2C_SLAVE_ADDRESS[slaveInd]; 
+            //Wire.beginTransmission(slaveAddress);
+            //Wire.write(msgArray[row].getDataArray(), msgArray[row].getLength());
+            //Wire.write(msgArray[row].getDataArray(), 20);
+            //Wire.endTransmission();
+
+            uint8_t (&data)[I2C_BUFFER_SIZE] = msgArray[row].getDataArray();
+            uint8_t dataLength = msgArray[row].getLength();
+            i2c_start((slaveAddress << 1) + I2C_WRITE);     
+            //for (uint8_t i=0; i< dataLength; i++)
+            for (uint8_t i=0; i<50; i++) //dataLength; i++)
+            {
+                //i2c_write(data[i]);
+                i2c_write(i);
+            }
+            i2c_stop();                                   
+        }
+
+        delay(500);
+        //delayMicroseconds(6500);
+        digitalWrite(A2,HIGH);
+        delayMicroseconds(200);
+        digitalWrite(A2,LOW);
+        delayMicroseconds(1000);
+
+        row = (row+1)%DISPLAY_MATRIX_SIZE;
+    }
+}
+
+
 void displayColTest()
 {
+    const uint8_t numGrayScaleValues = 7;
     static uint8_t col=0;
     DisplayMatrix matrix;
     I2CMessageBuffer msg;
-    matrix.setNumGrayScaleValues(MAX_NUM_GRAYSCALE_VALUES);
+    matrix.setNumGrayScaleValues(numGrayScaleValues);
 
     for (uint8_t slaveInd=0; slaveInd<I2C_NUM_SLAVES; slaveInd++)
     {
         uint8_t slaveAddress = I2C_SLAVE_ADDRESS[slaveInd];
+        //uint8_t slaveAddress = I2C_SLAVE_ADDRESS[I2C_NUM_SLAVES - slaveInd -1];
         matrix.setAllZero();
         if ((col < DISPLAY_MATRIX_SIZE) && ((slaveInd==0)||(slaveInd==2)))
         {
-            matrix.setCol(col,MAX_NUM_GRAYSCALE_VALUES-1);
+            matrix.setCol(col,numGrayScaleValues-1);
         }
         if ((col >= DISPLAY_MATRIX_SIZE) && ((slaveInd==1)||(slaveInd==3)))
         {
-            matrix.setCol(col%DISPLAY_MATRIX_SIZE,MAX_NUM_GRAYSCALE_VALUES-1);
-        }
-        msg = matrix.getMsgFromData();
-        Wire.beginTransmission(slaveAddress);
-        Wire.write(msg.getDataArray(), msg.getLength());
-        Wire.endTransmission();
-
-    }
-
-    delay(5);
-    msg.setByte(0,CMD_SWAP_DISPLAY_BUFFER);
-    msg.setByte(1,0);
-    msg.setLength(2);
-
-    for (uint8_t slaveInd=0; slaveInd<I2C_NUM_SLAVES; slaveInd++)
-    {
-        uint8_t slaveAddress = I2C_SLAVE_ADDRESS[slaveInd];
-        Wire.beginTransmission(slaveAddress);
-        Wire.write(msg.getDataArray(), msg.getLength());
-        Wire.endTransmission();
-    }
-
-    col++;
-    if (col >= 2*DISPLAY_MATRIX_SIZE)
-    {
-        col=0;
-    }
-    delay(10);
-}
-
-void displayRowTest()
-{
-    static uint8_t row=0;
-    DisplayMatrix matrix;
-    I2CMessageBuffer msg;
-    matrix.setNumGrayScaleValues(MAX_NUM_GRAYSCALE_VALUES);
-
-    for (uint8_t slaveInd=0; slaveInd<I2C_NUM_SLAVES; slaveInd++)
-    {
-        uint8_t slaveAddress = I2C_SLAVE_ADDRESS[slaveInd];
-        matrix.setAllZero();
-        if ((row < DISPLAY_MATRIX_SIZE) && (slaveInd<2))
-        {
-            matrix.setRow(row,MAX_NUM_GRAYSCALE_VALUES-1);
-        }
-        if ((row >= DISPLAY_MATRIX_SIZE) && (slaveInd>=2))
-        {
-            matrix.setRow(row%DISPLAY_MATRIX_SIZE,MAX_NUM_GRAYSCALE_VALUES-1);
+            matrix.setCol(col%DISPLAY_MATRIX_SIZE,numGrayScaleValues-1);
         }
         msg = matrix.getMsgFromData();
         Wire.beginTransmission(slaveAddress);
         Wire.endTransmission();
     }
 
-    msg.setByte(0,CMD_SWAP_DISPLAY_BUFFER);
-    msg.setByte(1,0);
-    msg.setLength(2);
+    delayMicroseconds(500);
+    digitalWrite(A2,HIGH);
+    delayMicroseconds(50);
+    digitalWrite(A2,LOW);
+    delayMicroseconds(50);
+
+    col++;
+    if (col >= 2*DISPLAY_MATRIX_SIZE)
+    {
+        col=0;
+    }
+    
+}
+
+void displayRowTest()
+{
+    const uint8_t numGrayScaleValues = 6;
+    static uint8_t row=0;
+    I2CMessageBuffer msg;;
+    DisplayMatrix matrix;
+    matrix.setNumGrayScaleValues(numGrayScaleValues);
 
     for (uint8_t slaveInd=0; slaveInd<I2C_NUM_SLAVES; slaveInd++)
     {
+        matrix.setAllZero();
+        if ((row < DISPLAY_MATRIX_SIZE) && (slaveInd<2))
+        {
+            matrix.setRow(row,numGrayScaleValues-1);
+        }
+        if ((row >= DISPLAY_MATRIX_SIZE) && (slaveInd>=2))
+        {
+            matrix.setRow(row%DISPLAY_MATRIX_SIZE,numGrayScaleValues-1);
+        }
+        msg = matrix.getMsgFromData();
+
         uint8_t slaveAddress = I2C_SLAVE_ADDRESS[slaveInd];
         Wire.beginTransmission(slaveAddress);
         Wire.write(msg.getDataArray(), msg.getLength());
         Wire.endTransmission();
     }
 
+    delay(100);
+    //delayMicroseconds(10000);
+    digitalWrite(A2,HIGH);
+    delayMicroseconds(500);
+    digitalWrite(A2,LOW);
+    delayMicroseconds(500);
+
     row++;
     if (row >= 2*DISPLAY_MATRIX_SIZE)
     {
         row=0;
     }
-    delay(5);
 
 }
 
 { 
     static uint8_t row = 0;
     static uint8_t col = 0;
+    const uint8_t numGrayScaleValues = 8;
 
     DisplayMatrix matrix;
     I2CMessageBuffer msg;
-    matrix.setNumGrayScaleValues(MAX_NUM_GRAYSCALE_VALUES);
+    matrix.setNumGrayScaleValues(numGrayScaleValues);
     matrix.setAllZero();
-    matrix.setPixel(row,col,MAX_NUM_GRAYSCALE_VALUES-1);
+    matrix.setPixel(row,col,numGrayScaleValues-1);
     //matrix.setPixel(row,col,1);
 
     for (uint8_t slaveInd=0; slaveInd<I2C_NUM_SLAVES; slaveInd++)
         Wire.endTransmission();
     }
 
-    msg.setByte(0,CMD_SWAP_DISPLAY_BUFFER);
-    msg.setByte(1,0);
-    msg.setLength(2);
-
-    for (uint8_t slaveInd=0; slaveInd<I2C_NUM_SLAVES; slaveInd++)
-    {
-        uint8_t slaveAddress = I2C_SLAVE_ADDRESS[slaveInd];
-        Wire.beginTransmission(slaveAddress);
-        Wire.write(msg.getDataArray(), msg.getLength());
-        Wire.endTransmission();
-    }
+    delay(5);
+    digitalWrite(A2,HIGH);
+    delayMicroseconds(100);
+    digitalWrite(A2,LOW);
+    delay(1);
 
     row++;
     if (row >= DISPLAY_MATRIX_SIZE)
         }
     }   
 
-    delay(50);
     //delayMicroseconds(3333);
 }
 
-void displayOnAndOffTest()
-{
-    static int cnt = 0;
-    uint8_t numItems;
-    I2CMessageBuffer msg;
-    uint8_t cmd;
-
-    if (cnt%2 == 0)
-    {
-        cmd = CMD_SET_DISPLAY_ALL_ON;
-    }
-    else
-    {
-        cmd = CMD_SET_DISPLAY_ALL_OFF;
-    }
-    msg.setByte(0,cmd);
-    msg.setByte(1,0);
-    msg.setLength(2);
-
-    Wire.beginTransmission(SLAVE_ADDRESS);
-    Wire.write(msg.getDataArray(), msg.getLength());
-    Wire.endTransmission();
-
-    delay(10);
-
-    msg.setByte(0,CMD_SWAP_DISPLAY_BUFFER);
-    msg.setByte(1,0);
-    msg.setLength(2);
-
-    Wire.beginTransmission(SLAVE_ADDRESS);
-    Wire.write(msg.getDataArray(), msg.getLength());
-    Wire.endTransmission();
-
-    cnt++;
-    delay(1000);
-}
+//void displayOnAndOffTest()
+//{
+//    static int cnt = 0;
+//    uint8_t numItems;
+//    I2CMessageBuffer msg;
+//    uint8_t cmd;
+//
+//    if (cnt%2 == 0)
+//    {
+//        cmd = CMD_SET_DISPLAY_ALL_ON;
+//    }
+//    else
+//    {
+//        cmd = CMD_SET_DISPLAY_ALL_OFF;
+//    }
+//    msg.setByte(0,cmd);
+//    msg.setByte(1,0);
+//    msg.setLength(2);
+//
+//    Wire.beginTransmission(SLAVE_ADDRESS);
+//    Wire.write(msg.getDataArray(), msg.getLength());
+//    Wire.endTransmission();
+//
+//    delay(10);
+//    digitalWrite(A2,HIGH);
+//    delayMicroseconds(100);
+//    digitalWrite(A2,LOW);
+//    delay(20);
+//
+//    cnt++;
+//    delay(100);
+//}

comm_firmware/test_i2cmaster.c.tmp

+/****************************************************************************
+Title:    Access serial EEPROM 24C02 using I2C interace
+Author:   Peter Fleury <pfleury@gmx.ch> http://jump.to/fleury
+File:     $Id: test_i2cmaster.c,v 1.2 2003/12/06 17:07:18 peter Exp $
+Software: AVR-GCC 3.3
+Hardware: AT90S8515 at 4 Mhz, any AVR device can be used
+
+Description:
+    This example shows how the I2C library i2cmaster.S can be used to 
+    access a serial eeprom.
+    Based on Atmel Application Note AVR300, adapted to AVR-GCC C interface
+
+*****************************************************************************/
+#include <avr/io.h>
+#include "i2cmaster.h"
+
+
+#define Dev24C02  0xA2      // device address of EEPROM 24C02, see datasheet
+
+
+int main(void)
+{
+    unsigned char ret;
+    
+
+    DDRB  = 0xff;                              // use all pins on port B for output 
+    PORTB = 0xff;                              // (active low LED's )
+
+    i2c_init();                                // init I2C interface
+
+    /* write 0x75 to eeprom address 0x05 (Byte Write) */
+    ret = i2c_start(Dev24C02+I2C_WRITE);       // set device address and write mode
+    if ( ret ) {
+        /* failed to issue start condition, possibly no device found */
+        i2c_stop();
+        PORTB=0x00;                            // activate all 8 LED to show error */
+    }else {
+        /* issuing start condition ok, device accessible */
+        i2c_write(0x05);                       // write address = 5
+        i2c_write(0x75);                       // ret=0 -> Ok, ret=1 -> no ACK 
+        i2c_stop();                            // set stop conditon = release bus
+
+        /* write ok, read value back from eeprom address 0x05, wait until 
+           the device is no longer busy from the previous write operation */
+        i2c_start_wait(Dev24C02+I2C_WRITE);     // set device address and write mode
+        i2c_write(0x05);                        // write address = 5
+        i2c_rep_start(Dev24C02+I2C_READ);       // set device address and read mode
+        ret = i2c_readNak();                    // read one byte
+        i2c_stop();
+        
+        PORTB = ~ret;                           // output byte on the LED's
+
+        /* write 0x70,0x71,072,073 to eeprom address 0x00..0x03 (Page Write),
+           wait until the device is no longer busy from the previous write operation */
+        i2c_start_wait(Dev24C02+I2C_WRITE);     // set device address and write mode
+        i2c_write(0x00);                        // write start address = 0
+        i2c_write(0x70);                        // write data to address 0
+        i2c_write(0x71);                        //    "    "   "    "    1
+        i2c_write(0x72);                        //    "    "   "    "    2
+        i2c_write(0x74);                        //    "    "   "    "    3
+        i2c_stop();                             // set stop conditon = release bus
+    
+        /* write ok, read value back from eeprom address 0..3 (Sequencial Read),
+           wait until the device is no longer busy from the previous write operation */
+        i2c_start_wait(Dev24C02+I2C_WRITE);      // set device address and write mode
+        i2c_write(0x00);                         // write address = 0
+        i2c_rep_start(Dev24C02+I2C_READ);        // set device address and read mode
+        ret = i2c_readAck();                       // read one byte form address 0
+        ret = i2c_readAck();                       //  "    "    "    "     "    1
+        ret = i2c_readAck();                       //  "    "    "    "     "    2
+        ret = i2c_readNak();                       //  "    "    "    "     "    3
+        i2c_stop();                              // set stop condition = release bus
+    
+        PORTB = ~ret;                            // output byte on the LED's        
+    }
+    
+    for(;;);	
+}

driver_firmware/constants.cpp

 
 // Display constants
 // ----------------------------------------------------------------------------
-const uint32_t DEFAULT_DISPLAY_TIMER_PERIOD = 25;
+const uint32_t DEFAULT_DISPLAY_TIMER_PERIOD = 31;
+const bool DEFAULT_USE_LOOKUP_TABLE = true;
 
 // Pin masks for fast direct port IO
 // ----------------------------------------------------------------------------

driver_firmware/constants.h

 // ----------------------------------------------------------------------------
 enum {DISPLAY_NUM_BUFFER = 2};
 extern const uint32_t DEFAULT_DISPLAY_TIMER_PERIOD;
+extern const bool DEFAULT_USE_LOOKUP_TABLE;
 
 // Pin masks for fast direct port IO
 // ----------------------------------------------------------------------------

driver_firmware/display_buffer.cpp

     createLookupTable();
 }
 
-void DisplayBuffer::createLookupTable()
-{
-    for (uint8_t row=0; row<DISPLAY_MATRIX_SIZE; row++)
-    {
-        for (uint8_t pwmCount=0; pwmCount < numGrayScaleValues_; pwmCount++)
-        {
-            PinMask colValues = {0x00, 0x00, 0x00};
-            for (uint8_t col=0; col<DISPLAY_MATRIX_SIZE; col++)
-            {
-                uint8_t val = getPixel(row,col,false);
-                if (pwmCount < val)
-                {
-                    colValues.B |= DISPLAY_COL_PIN_MASK[col].B;
-                    colValues.C |= DISPLAY_COL_PIN_MASK[col].C;
-                    colValues.D |= DISPLAY_COL_PIN_MASK[col].D;
-                }
-            }
+//void DisplayBuffer::createLookupTable()
+//{
+//    for (uint8_t row=0; row<DISPLAY_MATRIX_SIZE; row++)
+//    {
+//        for (uint8_t pwmCount=0; pwmCount < numGrayScaleValues_; pwmCount++)
+//        {
+//            PinMask colValues = {0x00, 0x00, 0x00};
+//            for (uint8_t col=0; col<DISPLAY_MATRIX_SIZE; col++)
+//            {
+//                uint8_t val = getPixel(row,col,false);
+//                if (pwmCount < val)
+//                {
+//                    colValues.B |= DISPLAY_COL_PIN_MASK[col].B;
+//                    colValues.C |= DISPLAY_COL_PIN_MASK[col].C;
+//                    colValues.D |= DISPLAY_COL_PIN_MASK[col].D;
+//                }
+//            }
+//
+//            lookupTable_[row][pwmCount].B = 
+//                ( DISPLAY_ALL_COL_PIN_MASK.B & ~colValues.B) + 
+//                ( DISPLAY_ALL_ROW_PIN_MASK.B & DISPLAY_ROW_PIN_MASK[row].B);
+//
+//            lookupTable_[row][pwmCount].C = 
+//                ( DISPLAY_ALL_COL_PIN_MASK.C & ~colValues.C) + 
+//                ( DISPLAY_ALL_ROW_PIN_MASK.C & DISPLAY_ROW_PIN_MASK[row].C);
+//
+//            lookupTable_[row][pwmCount].D = 
+//                ( DISPLAY_ALL_COL_PIN_MASK.D & ~colValues.D) + 
+//                ( DISPLAY_ALL_ROW_PIN_MASK.D & DISPLAY_ROW_PIN_MASK[row].D);
+//        }
+//    }
+//}
 
-            lookupTable_[row][pwmCount].B = 
-                ( DISPLAY_ALL_COL_PIN_MASK.B & ~colValues.B) + 
-                ( DISPLAY_ALL_ROW_PIN_MASK.B & DISPLAY_ROW_PIN_MASK[row].B);
-
-            lookupTable_[row][pwmCount].C = 
-                ( DISPLAY_ALL_COL_PIN_MASK.C & ~colValues.C) + 
-                ( DISPLAY_ALL_ROW_PIN_MASK.C & DISPLAY_ROW_PIN_MASK[row].C);
-
-            lookupTable_[row][pwmCount].D = 
-                ( DISPLAY_ALL_COL_PIN_MASK.D & ~colValues.D) + 
-                ( DISPLAY_ALL_ROW_PIN_MASK.D & DISPLAY_ROW_PIN_MASK[row].D);
-        }
-    }
-}
-

driver_firmware/display_buffer.h

 
     public:
         DisplayBuffer();
-        void createLookupTable(); 
+        inline void createLookupTable(); 
 
     private:
         PinMask lookupTable_[DISPLAY_MATRIX_SIZE][MAX_NUM_GRAYSCALE_VALUES];
 };
 
+
+inline void DisplayBuffer::createLookupTable()
+{
+    for (uint8_t row=0; row<DISPLAY_MATRIX_SIZE; row++)
+    {
+        for (uint8_t pwmCount=0; pwmCount < numGrayScaleValues_; pwmCount++)
+        {
+            PinMask colValues = {0x00, 0x00, 0x00};
+            for (uint8_t col=0; col<DISPLAY_MATRIX_SIZE; col++)
+            {
+                uint8_t val = getPixel(row,col,false);
+                if (pwmCount < val)
+                {
+                    colValues.B |= DISPLAY_COL_PIN_MASK[col].B;
+                    colValues.C |= DISPLAY_COL_PIN_MASK[col].C;
+                    colValues.D |= DISPLAY_COL_PIN_MASK[col].D;
+                }
+            }
+
+            lookupTable_[row][pwmCount].B = 
+                ( DISPLAY_ALL_COL_PIN_MASK.B & ~colValues.B) + 
+                ( DISPLAY_ALL_ROW_PIN_MASK.B & DISPLAY_ROW_PIN_MASK[row].B);
+
+            lookupTable_[row][pwmCount].C = 
+                ( DISPLAY_ALL_COL_PIN_MASK.C & ~colValues.C) + 
+                ( DISPLAY_ALL_ROW_PIN_MASK.C & DISPLAY_ROW_PIN_MASK[row].C);
+
+            lookupTable_[row][pwmCount].D = 
+                ( DISPLAY_ALL_COL_PIN_MASK.D & ~colValues.D) + 
+                ( DISPLAY_ALL_ROW_PIN_MASK.D & DISPLAY_ROW_PIN_MASK[row].D);
+        }
+    }
+}
+
 #endif

driver_firmware/display_driver.cpp

     activeBuffer_ = 0;
     writeBuffer_ = 1; 
     swapBufferFlag_ = false;
+    swapIntFlag_ = false;
+    newDataReady_ = false;
+    useLookupTable_ = DEFAULT_USE_LOOKUP_TABLE;
 }
 
-
 void DisplayDriver::initialize()
 {
     setIOPortDDR();
     buffer_[writeBuffer_].createLookupTable();
 }
 
+void DisplayDriver::setSwapIntFlag()
+{
+    ATOMIC_BLOCK(ATOMIC_RESTORESTATE)
+    {
+        swapIntFlag_ = true;
+    }
+}
+
+
+void DisplayDriver::setNewDataReady()
+{
+    ATOMIC_BLOCK(ATOMIC_RESTORESTATE)
+    {
+        newDataReady_ = true;
+    }
+}
+
+void DisplayDriver::checkForSwap()
+{
+    ATOMIC_BLOCK(ATOMIC_RESTORESTATE)
+    {
+        if ( swapIntFlag_ && newDataReady_)
+        {
+            swapBufferFlag_ = true;;
+            swapIntFlag_ = false;
+            newDataReady_ = false;
+        }
+    }
+}
+
+
+bool DisplayDriver::useLookupTable()
+{
+    return useLookupTable_;
+}
+
 
 #ifdef WITH_SERIAL_DEBUG
 void DisplayDriver::bufferPrintMatrixData()

driver_firmware/display_driver.h

         void bufferSetNumGrayScaleValues(uint8_t val);
         void bufferCreateLookupTable();
         bool bufferSetDataFromMsg(I2CMessageBuffer &msg);
+        bool useLookupTable();
         inline void updateDisplay();
 
+        void setSwapIntFlag();
+        void setNewDataReady();
+        void checkForSwap();
+
 #ifdef WITH_SERIAL_DEBUG
         void bufferPrintMatrixData();
 #endif
         volatile uint8_t activeBuffer_;
         volatile uint8_t writeBuffer_;
         volatile bool swapBufferFlag_;
+        volatile bool swapIntFlag_;
+        volatile bool newDataReady_;
+        bool useLookupTable_;
         void setIOPortDDR();
 };
 
 {
     // To be called in display update timer callback function
     static uint8_t pwm = 0;
-    static uint8_t ind = 0;
+    static uint8_t row = 0;
 
     pwm += 1;
     if (pwm >= buffer_[activeBuffer_].numGrayScaleValues_)
     {
         pwm = 0;
-        ind += 1; 
-        if (ind >= DISPLAY_MATRIX_SIZE)
+        row += 1; 
+        if (row >= DISPLAY_MATRIX_SIZE)
         {
-            ind = 0;
+            row = 0;
             if (swapBufferFlag_)
             {
                 activeBuffer_ = (activeBuffer_+1)%DISPLAY_NUM_BUFFER;
                 writeBuffer_  = (activeBuffer_+1)%DISPLAY_NUM_BUFFER; 
+
                 swapBufferFlag_ = false;
             }
         }
         PORTC = (DISPLAY_NOT_PIN_MASK.C & PORTC) + DISPLAY_ALL_COL_PIN_MASK.C;  
         PORTD = (DISPLAY_NOT_PIN_MASK.D & PORTD) + DISPLAY_ALL_COL_PIN_MASK.D;  
     }
-    PORTB = (DISPLAY_NOT_PIN_MASK.B & PORTB) + buffer_[activeBuffer_].lookupTable_[ind][pwm].B;
-    PORTC = (DISPLAY_NOT_PIN_MASK.C & PORTC) + buffer_[activeBuffer_].lookupTable_[ind][pwm].C;
-    PORTD = (DISPLAY_NOT_PIN_MASK.D & PORTD) + buffer_[activeBuffer_].lookupTable_[ind][pwm].D;
+
+    if (useLookupTable_)
+    {
+        PORTB = (DISPLAY_NOT_PIN_MASK.B & PORTB) + buffer_[activeBuffer_].lookupTable_[row][pwm].B;
+        PORTC = (DISPLAY_NOT_PIN_MASK.C & PORTC) + buffer_[activeBuffer_].lookupTable_[row][pwm].C;
+        PORTD = (DISPLAY_NOT_PIN_MASK.D & PORTD) + buffer_[activeBuffer_].lookupTable_[row][pwm].D;
+    }
+    else
+    {
+        PinMask colValues = {0x00, 0x00, 0x00};
+        for (uint8_t col=0; col<DISPLAY_MATRIX_SIZE; col++)
+        {
+            uint8_t val = buffer_[activeBuffer_].getPixel(row,col,false);
+            if (pwm < val)
+            {
+                colValues.B |= DISPLAY_COL_PIN_MASK[col].B;
+                colValues.C |= DISPLAY_COL_PIN_MASK[col].C;
+                colValues.D |= DISPLAY_COL_PIN_MASK[col].D;
+            }
+        }
+
+        PinMask outputVals;
+
+        outputVals.B = (DISPLAY_ALL_COL_PIN_MASK.B & ~colValues.B) + 
+            (DISPLAY_ALL_ROW_PIN_MASK.B & DISPLAY_ROW_PIN_MASK[row].B);
+
+        outputVals.C = (DISPLAY_ALL_COL_PIN_MASK.C & ~colValues.C) + 
+            (DISPLAY_ALL_ROW_PIN_MASK.C & DISPLAY_ROW_PIN_MASK[row].C);
+
+        outputVals.D = (DISPLAY_ALL_COL_PIN_MASK.D & ~colValues.D) + 
+            ( DISPLAY_ALL_ROW_PIN_MASK.D & DISPLAY_ROW_PIN_MASK[row].D);
+
+        PORTB = (DISPLAY_NOT_PIN_MASK.B & PORTB) + outputVals.B;
+        PORTC = (DISPLAY_NOT_PIN_MASK.C & PORTC) + outputVals.C;
+        PORTD = (DISPLAY_NOT_PIN_MASK.D & PORTD) + outputVals.D;
+    }
 }
 
 extern DisplayDriver displayDriver; 

driver_firmware/driver_firmware.ino

 #include <util/atomic.h>
 #include <Wire.h>
 #include <TimerOne.h>
-#include "panels_g4.h"
+#include <panels_g4.h>
+#include <PinChangeInt.h>
 #include "constants.h" 
 #include "display_driver.h"
 #include "i2c_handler.h"
     // Setup i2c message handler
     i2cHandler.initialize();
     i2cHandler.setReceiveEventCallback(i2cEventCallback);
+
+    pinMode(A2,INPUT);
+    PCintPort::attachInterrupt(A2,swapFrameCallback,RISING);
 }
 
 
 void loop()
 {
     i2cHandler.processMsg();
+    displayDriver.checkForSwap();
     //displayPixelTest();
 }
 
 }
 
 
+inline void swapFrameCallback()
+{
+    displayDriver.setSwapIntFlag();
+}
+
 
 // DEVEL
 // ----------------------------------------------------------------------------

driver_firmware/event_dispatcher.cpp

     if (rtnStatus == true)
     {
         displayDriver.bufferCreateLookupTable();
+        displayDriver.setNewDataReady();
     }
 } 
 

driver_firmware/i2c_handler.cpp

     {
         if (haveNewMsg_)
         {
+            // Don't need to copy the entire buffer
             for (uint8_t i=0; i<I2C_BUFFER_SIZE; i++)
             {
                 msgCopy.data_[i] = msg_.data_[i];

library/ic2master/gpl-2.0.txt

+                    GNU GENERAL PUBLIC LICENSE
+                       Version 2, June 1991
+
+ Copyright (C) 1989, 1991 Free Software Foundation, Inc.,
+ 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ Everyone is permitted to copy and distribute verbatim copies
+ of this license document, but changing it is not allowed.
+
+                            Preamble
+
+  The licenses for most software are designed to take away your
+freedom to share and change it.  By contrast, the GNU General Public
+License is intended to guarantee your freedom to share and change free
+software--to make sure the software is free for all its users.  This
+General Public License applies to most of the Free Software
+Foundation's software and to any other program whose authors commit to
+using it.  (Some other Free Software Foundation software is covered by
+the GNU Lesser General Public License instead.)  You can apply it to
+your programs, too.
+
+  When we speak of free software, we are referring to freedom, not
+price.  Our General Public Licenses are designed to make sure that you
+have the freedom to distribute copies of free software (and charge for
+this service if you wish), that you receive source code or can get it
+if you want it, that you can change the software or use pieces of it
+in new free programs; and that you know you can do these things.
+
+  To protect your rights, we need to make restrictions that forbid
+anyone to deny you these rights or to ask you to surrender the rights.
+These restrictions translate to certain responsibilities for you if you
+distribute copies of the software, or if you modify it.
+
+  For example, if you distribute copies of such a program, whether
+gratis or for a fee, you must give the recipients all the rights that
+you have.  You must make sure that they, too, receive or can get the
+source code.  And you must show them these terms so they know their
+rights.
+
+  We protect your rights with two steps: (1) copyright the software, and
+(2) offer you this license which gives you legal permission to copy,
+distribute and/or modify the software.
+
+  Also, for each author's protection and ours, we want to make certain
+that everyone understands that there is no warranty for this free
+software.  If the software is modified by someone else and passed on, we
+want its recipients to know that what they have is not the original, so
+that any problems introduced by others will not reflect on the original
+authors' reputations.
+
+  Finally, any free program is threatened constantly by software
+patents.  We wish to avoid the danger that redistributors of a free
+program will individually obtain patent licenses, in effect making the
+program proprietary.  To prevent this, we have made it clear that any
+patent must be licensed for everyone's free use or not licensed at all.
+
+  The precise terms and conditions for copying, distribution and
+modification follow.
+
+                    GNU GENERAL PUBLIC LICENSE
+   TERMS AND CONDITIONS FOR COPYING, DISTRIBUTION AND MODIFICATION
+
+  0. This License applies to any program or other work which contains
+a notice placed by the copyright holder saying it may be distributed
+under the terms of this General Public License.  The "Program", below,
+refers to any such program or work, and a "work based on the Program"
+means either the Program or any derivative work under copyright law:
+that is to say, a work containing the Program or a portion of it,
+either verbatim or with modifications and/or translated into another
+language.  (Hereinafter, translation is included without limitation in
+the term "modification".)  Each licensee is addressed as "you".
+
+Activities other than copying, distribution and modification are not
+covered by this License; they are outside its scope.  The act of
+running the Program is not restricted, and the output from the Program
+is covered only if its contents constitute a work based on the
+Program (independent of having been made by running the Program).
+Whether that is true depends on what the Program does.
+
+  1. You may copy and distribute verbatim copies of the Program's
+source code as you receive it, in any medium, provided that you
+conspicuously and appropriately publish on each copy an appropriate
+copyright notice and disclaimer of warranty; keep intact all the
+notices that refer to this License and to the absence of any warranty;
+and give any other recipients of the Program a copy of this License
+along with the Program.
+
+You may charge a fee for the physical act of transferring a copy, and
+you may at your option offer warranty protection in exchange for a fee.
+
+  2. You may modify your copy or copies of the Program or any portion
+of it, thus forming a work based on the Program, and copy and
+distribute such modifications or work under the terms of Section 1
+above, provided that you also meet all of these conditions:
+
+    a) You must cause the modified files to carry prominent notices
+    stating that you changed the files and the date of any change.
+
+    b) You must cause any work that you distribute or publish, that in
+    whole or in part contains or is derived from the Program or any
+    part thereof, to be licensed as a whole at no charge to all third
+    parties under the terms of this License.
+
+    c) If the modified program normally reads commands interactively
+    when run, you must cause it, when started running for such
+    interactive use in the most ordinary way, to print or display an
+    announcement including an appropriate copyright notice and a
+    notice that there is no warranty (or else, saying that you provide
+    a warranty) and that users may redistribute the program under
+    these conditions, and telling the user how to view a copy of this
+    License.  (Exception: if the Program itself is interactive but
+    does not normally print such an announcement, your work based on
+    the Program is not required to print an announcement.)
+
+These requirements apply to the modified work as a whole.  If
+identifiable sections of that work are not derived from the Program,
+and can be reasonably considered independent and separate works in
+themselves, then this License, and its terms, do not apply to those
+sections when you distribute them as separate works.  But when you
+distribute the same sections as part of a whole which is a work based
+on the Program, the distribution of the whole must be on the terms of
+this License, whose permissions for other licensees extend to the
+entire whole, and thus to each and every part regardless of who wrote it.
+
+Thus, it is not the intent of this section to claim rights or contest
+your rights to work written entirely by you; rather, the intent is to
+exercise the right to control the distribution of derivative or
+collective works based on the Program.
+
+In addition, mere aggregation of another work not based on the Program
+with the Program (or with a work based on the Program) on a volume of
+a storage or distribution medium does not bring the other work under
+the scope of this License.
+
+  3. You may copy and distribute the Program (or a work based on it,
+under Section 2) in object code or executable form under the terms of
+Sections 1 and 2 above provided that you also do one of the following:
+
+    a) Accompany it with the complete corresponding machine-readable
+    source code, which must be distributed under the terms of Sections
+    1 and 2 above on a medium customarily used for software interchange; or,
+
+    b) Accompany it with a written offer, valid for at least three
+    years, to give any third party, for a charge no more than your
+    cost of physically performing source distribution, a complete
+    machine-readable copy of the corresponding source code, to be
+    distributed under the terms of Sections 1 and 2 above on a medium
+    customarily used for software interchange; or,
+
+    c) Accompany it with the information you received as to the offer
+    to distribute corresponding source code.  (This alternative is
+    allowed only for noncommercial distribution and only if you
+    received the program in object code or executable form with such
+    an offer, in accord with Subsection b above.)
+
+The source code for a work means the preferred form of the work for
+making modifications to it.  For an executable work, complete source
+code means all the source code for all modules it contains, plus any
+associated interface definition files, plus the scripts used to
+control compilation and installation of the executable.  However, as a
+special exception, the source code distributed need not include
+anything that is normally distributed (in either source or binary
+form) with the major components (compiler, kernel, and so on) of the
+operating system on which the executable runs, unless that component
+itself accompanies the executable.
+
+If distribution of executable or object code is made by offering
+access to copy from a designated place, then offering equivalent
+access to copy the source code from the same place counts as
+distribution of the source code, even though third parties are not
+compelled to copy the source along with the object code.
+
+  4. You may not copy, modify, sublicense, or distribute the Program
+except as expressly provided under this License.  Any attempt
+otherwise to copy, modify, sublicense or distribute the Program is
+void, and will automatically terminate your rights under this License.
+However, parties who have received copies, or rights, from you under
+this License will not have their licenses terminated so long as such
+parties remain in full compliance.
+
+  5. You are not required to accept this License, since you have not
+signed it.  However, nothing else grants you permission to modify or
+distribute the Program or its derivative works.  These actions are
+prohibited by law if you do not accept this License.  Therefore, by
+modifying or distributing the Program (or any work based on the
+Program), you indicate your acceptance of this License to do so, and
+all its terms and conditions for copying, distributing or modifying
+the Program or works based on it.
+
+  6. Each time you redistribute the Program (or any work based on the
+Program), the recipient automatically receives a license from the
+original licensor to copy, distribute or modify the Program subject to
+these terms and conditions.  You may not impose any further
+restrictions on the recipients' exercise of the rights granted herein.
+You are not responsible for enforcing compliance by third parties to
+this License.
+
+  7. If, as a consequence of a court judgment or allegation of patent
+infringement or for any other reason (not limited to patent issues),
+conditions are imposed on you (whether by court order, agreement or
+otherwise) that contradict the conditions of this License, they do not
+excuse you from the conditions of this License.  If you cannot
+distribute so as to satisfy simultaneously your obligations under this
+License and any other pertinent obligations, then as a consequence you
+may not distribute the Program at all.  For example, if a patent
+license would not permit royalty-free redistribution of the Program by
+all those who receive copies directly or indirectly through you, then
+the only way you could satisfy both it and this License would be to
+refrain entirely from distribution of the Program.
+
+If any portion of this section is held invalid or unenforceable under
+any particular circumstance, the balance of the section is intended to
+apply and the section as a whole is intended to apply in other
+circumstances.
+
+It is not the purpose of this section to induce you to infringe any
+patents or other property right claims or to contest validity of any
+such claims; this section has the sole purpose of protecting the
+integrity of the free software distribution system, which is
+implemented by public license practices.  Many people have made
+generous contributions to the wide range of software distributed
+through that system in reliance on consistent application of that
+system; it is up to the author/donor to decide if he or she is willing
+to distribute software through any other system and a licensee cannot
+impose that choice.
+
+This section is intended to make thoroughly clear what is believed to
+be a consequence of the rest of this License.
+
+  8. If the distribution and/or use of the Program is restricted in
+certain countries either by patents or by copyrighted interfaces, the
+original copyright holder who places the Program under this License
+may add an explicit geographical distribution limitation excluding
+those countries, so that distribution is permitted only in or among
+countries not thus excluded.  In such case, this License incorporates
+the limitation as if written in the body of this License.
+
+  9. The Free Software Foundation may publish revised and/or new versions
+of the General Public License from time to time.  Such new versions will
+be similar in spirit to the present version, but may differ in detail to
+address new problems or concerns.
+
+Each version is given a distinguishing version number.  If the Program
+specifies a version number of this License which applies to it and "any
+later version", you have the option of following the terms and conditions
+either of that version or of any later version published by the Free
+Software Foundation.  If the Program does not specify a version number of
+this License, you may choose any version ever published by the Free Software
+Foundation.
+
+  10. If you wish to incorporate parts of the Program into other free
+programs whose distribution conditions are different, write to the author
+to ask for permission.  For software which is copyrighted by the Free
+Software Foundation, write to the Free Software Foundation; we sometimes
+make exceptions for this.  Our decision will be guided by the two goals
+of preserving the free status of all derivatives of our free software and
+of promoting the sharing and reuse of software generally.
+
+                            NO WARRANTY
+
+  11. BECAUSE THE PROGRAM IS LICENSED FREE OF CHARGE, THERE IS NO WARRANTY
+FOR THE PROGRAM, TO THE EXTENT PERMITTED BY APPLICABLE LAW.  EXCEPT WHEN
+OTHERWISE STATED IN WRITING THE COPYRIGHT HOLDERS AND/OR OTHER PARTIES
+PROVIDE THE PROGRAM "AS IS" WITHOUT WARRANTY OF ANY KIND, EITHER EXPRESSED
+OR IMPLIED, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
+MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.  THE ENTIRE RISK AS
+TO THE QUALITY AND PERFORMANCE OF THE PROGRAM IS WITH YOU.  SHOULD THE
+PROGRAM PROVE DEFECTIVE, YOU ASSUME THE COST OF ALL NECESSARY SERVICING,
+REPAIR OR CORRECTION.
+
+  12. IN NO EVENT UNLESS REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING
+WILL ANY COPYRIGHT HOLDER, OR ANY OTHER PARTY WHO MAY MODIFY AND/OR
+REDISTRIBUTE THE PROGRAM AS PERMITTED ABOVE, BE LIABLE TO YOU FOR DAMAGES,
+INCLUDING ANY GENERAL, SPECIAL, INCIDENTAL OR CONSEQUENTIAL DAMAGES ARISING
+OUT OF THE USE OR INABILITY TO USE THE PROGRAM (INCLUDING BUT NOT LIMITED
+TO LOSS OF DATA OR DATA BEING RENDERED INACCURATE OR LOSSES SUSTAINED BY
+YOU OR THIRD PARTIES OR A FAILURE OF THE PROGRAM TO OPERATE WITH ANY OTHER
+PROGRAMS), EVEN IF SUCH HOLDER OR OTHER PARTY HAS BEEN ADVISED OF THE
+POSSIBILITY OF SUCH DAMAGES.
+
+                     END OF TERMS AND CONDITIONS
+
+            How to Apply These Terms to Your New Programs
+
+  If you develop a new program, and you want it to be of the greatest
+possible use to the public, the best way to achieve this is to make it
+free software which everyone can redistribute and change under these terms.
+
+  To do so, attach the following notices to the program.  It is safest
+to attach them to the start of each source file to most effectively
+convey the exclusion of warranty; and each file should have at least
+the "copyright" line and a pointer to where the full notice is found.
+
+    <one line to give the program's name and a brief idea of what it does.>
+    Copyright (C) <year>  <name of author>
+
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    You should have received a copy of the GNU General Public License along
+    with this program; if not, write to the Free Software Foundation, Inc.,
+    51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+Also add information on how to contact you by electronic and paper mail.
+
+If the program is interactive, make it output a short notice like this
+when it starts in an interactive mode:
+
+    Gnomovision version 69, Copyright (C) year name of author
+    Gnomovision comes with ABSOLUTELY NO WARRANTY; for details type `show w'.
+    This is free software, and you are welcome to redistribute it
+    under certain conditions; type `show c' for details.
+
+The hypothetical commands `show w' and `show c' should show the appropriate
+parts of the General Public License.  Of course, the commands you use may
+be called something other than `show w' and `show c'; they could even be
+mouse-clicks or menu items--whatever suits your program.
+
+You should also get your employer (if you work as a programmer) or your
+school, if any, to sign a "copyright disclaimer" for the program, if
+necessary.  Here is a sample; alter the names:
+
+  Yoyodyne, Inc., hereby disclaims all copyright interest in the program
+  `Gnomovision' (which makes passes at compilers) written by James Hacker.
+
+  <signature of Ty Coon>, 1 April 1989
+  Ty Coon, President of Vice
+
+This General Public License does not permit incorporating your program into
+proprietary programs.  If your program is a subroutine library, you may
+consider it more useful to permit linking proprietary applications with the
+library.  If this is what you want to do, use the GNU Lesser General
+Public License instead of this License.

library/panels_g4/display_matrix.cpp

     flip_ = DEFAULT_DISPLAY_FLIP;
 }
 
-
-uint8_t DisplayMatrix::getPixel(uint8_t row, uint8_t col, bool flipAndRotate)
-{
-    if ((row < DISPLAY_MATRIX_SIZE) && (col < DISPLAY_MATRIX_SIZE))
-    {
-        if (flipAndRotate)
-        {
-            getFlippedRowAndCol(row, col); 
-            getRotatedRowAndCol(row, col);
-        }
-        return matrixData_[row][col];
-    }
-    else
-    {
-        return 0;
-    }
-}
-
-
-void DisplayMatrix::setPixel(uint8_t row, uint8_t col, uint8_t val, bool flipAndRotate)
-{
-    if ((row < DISPLAY_MATRIX_SIZE) && (col < DISPLAY_MATRIX_SIZE))
-    {
-        if (flipAndRotate)
-        {
-            getFlippedRowAndCol(row, col); 
-            getRotatedRowAndCol(row, col);
-        }
-        matrixData_[row][col] = val;
-    }
-}
-
-
 void DisplayMatrix::setRow(uint8_t row, uint8_t val)
 {
     for (uint8_t col=0; col<DISPLAY_MATRIX_SIZE; col++)
 I2CMessageBuffer DisplayMatrix::getMsgFromData()
 {
     I2CMessageBuffer msg;
+    getMsgFromData(msg);
+    return msg;
+}
+
+
+void DisplayMatrix::getMsgFromData(I2CMessageBuffer &msg)
+{
     uint8_t bitsPerElem = getNumBitsPerElem(numGrayScaleValues_);
     uint8_t numDataBytesInMsg = getNumBytesForGrayScale(numGrayScaleValues_)+1;
     uint8_t dataStartByte = 3;
             }
         }
     }
-    msg.setLength(numDataBytesInMsg+2); 
-    return msg;
+    msg.setLength(numDataBytesInMsg+3); 
 }
 
 

library/panels_g4/display_matrix.h

 {
     public:
         DisplayMatrix();
-        void setPixel(uint8_t row, uint8_t col, uint8_t val, bool flipAndRotate=true);
-        uint8_t getPixel(uint8_t row, uint8_t col, bool flipAndRotate=true);
+        inline void setPixel(uint8_t row, uint8_t col, uint8_t val, bool flipAndRotate=true);
+        inline uint8_t getPixel(uint8_t row, uint8_t col, bool flipAndRotate=true);
         void setRow(uint8_t row, uint8_t val);
         void setCol(uint8_t col, uint8_t val);
         void setAll(uint8_t val);
         void setDisplayFlip(DisplayFlip value);
         bool setDataFromMsg(I2CMessageBuffer &msg);
         I2CMessageBuffer getMsgFromData();
+        void getMsgFromData(I2CMessageBuffer &msg);
         uint8_t getNumGrayScaleValues();
 
 #ifdef WITH_SERIAL_DEBUG
 };
 
 
+inline uint8_t DisplayMatrix::getPixel(uint8_t row, uint8_t col, bool flipAndRotate)
+{
+    if ((row < DISPLAY_MATRIX_SIZE) && (col < DISPLAY_MATRIX_SIZE))
+    {
+        if (flipAndRotate)
+        {
+            getFlippedRowAndCol(row, col); 
+            getRotatedRowAndCol(row, col);
+        }
+        return matrixData_[row][col];
+    }
+    else
+    {
+        return 0;
+    }
+}
+
+
+inline void DisplayMatrix::setPixel(uint8_t row, uint8_t col, uint8_t val, bool flipAndRotate)
+{
+    if ((row < DISPLAY_MATRIX_SIZE) && (col < DISPLAY_MATRIX_SIZE))
+    {
+        if (flipAndRotate)
+        {
+            getFlippedRowAndCol(row, col); 
+            getRotatedRowAndCol(row, col);
+        }
+        matrixData_[row][col] = val;
+    }
+}
+
 // Utility functions
 uint8_t getNumBitsPerElem(uint8_t numGrayScaleValues);
 uint8_t getNumBitsPerRow(uint8_t numGrayScaleValues);