Fix draw() function to actually draw whole bitmap.

4x4_matrix_part_1/main.c

@@ -38,45 +38,42 @@ static void init(void) {
 
 }
 
-void draw(uint8_t buffer[4][4]) {
+void draw(uint8_t buffer[4][4], uint8_t delay) {
-    static uint8_t row = 0;
-
-    /* Turn off whole row. */
-    digital_write(row_pins[row], HIGH);
-    
-    /* Advance one row and make sure it is 0..4 */    
-    row++;
-    row = row % 4;
     
+    for (uint8_t row=0; row<4; ++row) {
         /* Connect or disconnect columns as needed. */
-    for (uint8_t column=0; column<4; column++) {
+        for (uint8_t column=0; column<4; ++column) {
             digital_write(column_pins[column], buffer[row][column]);                    
         }
         
         /* Turn on whole row. */
         digital_write(row_pins[row], LOW);
+
+        _delay_ms(delay);
+
+        /* Turn off whole row. */
+        digital_write(row_pins[row], HIGH);
+    }
 }
 
-int main(void) {    
+uint8_t main(void) {    
     
     init();
     
     /* With 100ms delay eye can see updating row by row. */
-    for (uint8_t i=0; i<40; i++) { 
+    for (uint8_t i=0; i<10; i++) { 
-        draw(pattern);
+        draw(pattern, 100);
-        _delay_ms(100);
     }
 
     /* With 10ms delay pattern appears but flickers. */
-    for (uint16_t i=0; i<400; i++) { 
+    for (uint16_t i=0; i<100; i++) { 
-        draw(pattern);
+        draw(pattern, 10);
-        _delay_ms(10);
     }
     
     /* Withoud delay solid pattern appears. */
     while (1) {
-        draw(pattern);        
+        draw(pattern, 1);        
     }
-    return 0;
     
+    return 0;
 }

4x4_matrix_part_1/main.h

@@ -1,3 +1,3 @@
 static void init(void);
-void draw(uint8_t buffer[4][4]);
+void draw(uint8_t buffer[4][4], uint8_t delay);
-int main(void);
+uint8_t main(void);