From 44215ade188e75cd7ac39da68911a740e27a7516 Mon Sep 17 00:00:00 2001 From: Jacob Alexander Date: Fri, 13 May 2016 14:38:05 -0700 Subject: [PATCH] =?UTF-8?q?Revert=20"Fixed=20ghosting=20matrix=20check=20(?= =?UTF-8?q?did=20send=20bad=20keys=20on=20release),=20now=20using=E2=80=A6?= =?UTF-8?q?"?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- Scan/MatrixARM/matrix_scan.c | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/Scan/MatrixARM/matrix_scan.c b/Scan/MatrixARM/matrix_scan.c index 76f7583..7d8897b 100644 --- a/Scan/MatrixARM/matrix_scan.c +++ b/Scan/MatrixARM/matrix_scan.c @@ -80,7 +80,6 @@ KeyGhost Matrix_ghostArray[ Matrix_colsNum * Matrix_rowsNum ]; uint8_t col_use[Matrix_colsNum], row_use[Matrix_rowsNum]; // used count uint8_t col_ghost[Matrix_colsNum], row_ghost[Matrix_rowsNum]; // marked as having ghost if 1 -uint8_t col_ghost_old[Matrix_colsNum], row_ghost_old[Matrix_rowsNum]; // old ghost state #endif @@ -212,18 +211,12 @@ void Matrix_setup() for ( uint8_t pin = 0; pin < Matrix_colsNum; pin++ ) { Matrix_pin( Matrix_cols[ pin ], Type_StrobeSetup ); - col_use[pin] = 0; - col_ghost[pin] = 0; - col_ghost_old[pin] = 0; } // Setup Sense Pins for ( uint8_t pin = 0; pin < Matrix_rowsNum; pin++ ) { Matrix_pin( Matrix_rows[ pin ], Type_SenseSetup ); - row_use[pin] = 0; - row_ghost[pin] = 0; - row_ghost_old[pin] = 0; } // Clear out Debounce Array @@ -455,6 +448,7 @@ void Matrix_scan( uint16_t scanNum ) // strobe = column, sense = row // Count (rows) use for columns + //print("C "); for ( uint8_t col = 0; col < Matrix_colsNum; col++ ) { uint8_t used = 0; @@ -465,12 +459,13 @@ void Matrix_scan( uint16_t scanNum ) if ( keyOn(state->curState) ) used++; } + //printInt8(used); col_use[col] = used; - col_ghost_old[col] = col_ghost[col]; col_ghost[col] = 0; // clear } // Count (columns) use for rows + //print(" R "); for ( uint8_t row = 0; row < Matrix_rowsNum; row++ ) { uint8_t used = 0; @@ -481,13 +476,14 @@ void Matrix_scan( uint16_t scanNum ) if ( keyOn(state->curState) ) used++; } + //printInt8(used); row_use[row] = used; - row_ghost_old[row] = row_ghost[row]; row_ghost[row] = 0; // clear } // Check if matrix has ghost // Happens when key is pressed and some other key is pressed in same row and another in same column + //print(" G "); for ( uint8_t col = 0; col < Matrix_colsNum; col++ ) { for ( uint8_t row = 0; row < Matrix_rowsNum; row++ ) @@ -499,9 +495,11 @@ void Matrix_scan( uint16_t scanNum ) // mark col and row as having ghost col_ghost[col] = 1; row_ghost[row] = 1; + //print(" "); printInt8(col); print(","); printInt8(row); } } } + //print( NL ); // Send keys for ( uint8_t col = 0; col < Matrix_colsNum; col++ ) @@ -514,8 +512,6 @@ void Matrix_scan( uint16_t scanNum ) // col or row is ghosting (crossed) uint8_t ghost = (col_ghost[col] > 0 || row_ghost[row] > 0) ? 1 : 0; - uint8_t ghost_old = (col_ghost_old[col] > 0 || row_ghost_old[row] > 0) ? 1 : 0; - ghost = ghost || ghost_old ? 1 : 0; st->prev = st->cur; // previous // save state if no ghost or outside ghosted area