diff options
| author | Joel Challis <git@zvecr.com> | 2021-09-08 23:44:18 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-09-08 15:44:18 -0700 |
| commit | 2e0659f7fddd4c872b230c4a7d2e2a5517042414 (patch) | |
| tree | 3ff74ddb4e604102fe843fcc3f8384f4e4673449 | |
| parent | 9c175f180138558748b0b240e76cb04fa11f9927 (diff) | |
| download | qmk_firmware-2e0659f7fddd4c872b230c4a7d2e2a5517042414.tar.gz qmk_firmware-2e0659f7fddd4c872b230c4a7d2e2a5517042414.zip | |
Fixes for recent i2c migrations (#14352)
| -rw-r--r-- | keyboards/3w6/rev1/matrix.c | 2 | ||||
| -rw-r--r-- | keyboards/3w6/rev2/matrix.c | 5 | ||||
| -rw-r--r-- | keyboards/handwired/dactyl/dactyl.h | 2 | ||||
| -rwxr-xr-x | keyboards/sx60/sx60.h | 2 |
4 files changed, 6 insertions, 5 deletions
diff --git a/keyboards/3w6/rev1/matrix.c b/keyboards/3w6/rev1/matrix.c index af3d210670..ae2f96bfa1 100644 --- a/keyboards/3w6/rev1/matrix.c +++ b/keyboards/3w6/rev1/matrix.c @@ -34,7 +34,7 @@ extern i2c_status_t tca9555_status; // All address pins of the tca9555 are connected to the ground // | 0 | 1 | 0 | 0 | A2 | A1 | A0 | // | 0 | 1 | 0 | 0 | 0 | 0 | 0 | -#define I2C_ADDR 0b0100000 +#define I2C_ADDR (0b0100000 << 1) // Register addresses #define IODIRA 0x06 // i/o direction register diff --git a/keyboards/3w6/rev2/matrix.c b/keyboards/3w6/rev2/matrix.c index 0fc0322b6e..c47c24e1dc 100644 --- a/keyboards/3w6/rev2/matrix.c +++ b/keyboards/3w6/rev2/matrix.c @@ -34,7 +34,7 @@ extern i2c_status_t tca9555_status; // All address pins of the tca9555 are connected to the ground // | 0 | 1 | 0 | 0 | A2 | A1 | A0 | // | 0 | 1 | 0 | 0 | 0 | 0 | 0 | -#define I2C_ADDR 0b0100000 +#define I2C_ADDR (0b0100000 << 1) // Register addresses #define IODIRA 0x06 // i/o direction register @@ -192,6 +192,7 @@ static matrix_row_t read_cols(uint8_t row) { // do nothing return 0; } else { + port0 = ~port0; // We read all the pins on GPIOA. // The initial state was all ones and any depressed key at a given column for the currently selected row will have its bit flipped to zero. // The return value is a row as represented in the generic matrix code were the rightmost bits represent the lower columns and zeroes represent non-depressed keys while ones represent depressed keys. @@ -247,7 +248,7 @@ static void select_row(uint8_t row) { default: break; } - tca9555_status = i2c_writeReg(I2C_ADDR, OREGP1, &port1, 2, I2C_TIMEOUT); + tca9555_status = i2c_writeReg(I2C_ADDR, OREGP1, &port1, 1, I2C_TIMEOUT); // Select the desired row by writing a byte for the entire GPIOB bus where only the bit representing the row we want to select is a zero (write instruction) and every other bit is a one. // Note that the row - MATRIX_ROWS_PER_SIDE reflects the fact that being on the right hand, the columns are numbered from MATRIX_ROWS_PER_SIDE to MATRIX_ROWS, but the pins we want to write to are indexed from zero up on the GPIOB bus. } diff --git a/keyboards/handwired/dactyl/dactyl.h b/keyboards/handwired/dactyl/dactyl.h index f7be886ac6..a6a043496c 100644 --- a/keyboards/handwired/dactyl/dactyl.h +++ b/keyboards/handwired/dactyl/dactyl.h @@ -7,7 +7,7 @@ #include "i2c_master.h" #include <util/delay.h> -#define I2C_ADDR 0b0100000 +#define I2C_ADDR (0b0100000 << 1) #define I2C_TIMEOUT 100 #define IODIRA 0x00 // i/o direction register #define IODIRB 0x01 diff --git a/keyboards/sx60/sx60.h b/keyboards/sx60/sx60.h index 507bddf7b0..f533d8e899 100755 --- a/keyboards/sx60/sx60.h +++ b/keyboards/sx60/sx60.h @@ -8,7 +8,7 @@ #include <util/delay.h> /* I2C aliases and register addresses (see "mcp23018.md") */ -#define I2C_ADDR 0b0100000 +#define I2C_ADDR (0b0100000 << 1) #define I2C_TIMEOUT 100 #define IODIRA 0x00 /* i/o direction register */ #define IODIRB 0x01 |