megamanmalte/qmk_firmware
0
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Unbreak switchplate/southpaw_65 after #16277 (#20015)

Browse source 
Apparently the changes done in #16277 swapped the values read from the
PCA9555 expanders by mistake, which resulted in mixed up matrix columns.
Fix the chip and register addresses to match the original code.
This commit is contained in:
Sergey Vlasov 2023-03-05 16:29:10 +03:00 committed by GitHub
parent 77f7351991
commit 07b7fed122
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
keyboards/switchplate/southpaw_65/matrix.c
View file

@ -54,9 +54,9 @@ static uint32_t read_cols(void) {
uint8_t state_1 = 0; uint8_t state_1 = 0;
uint8_t state_2 = 0; uint8_t state_2 = 0;
uint8_t state_3 = 0; uint8_t state_3 = 0;
pca9555_readPins(IC2, PCA9555_PORT0, &state_1); pca9555_readPins(IC1, PCA9555_PORT1, &state_1);
pca9555_readPins(IC2, PCA9555_PORT1, &state_2); pca9555_readPins(IC2, PCA9555_PORT0, &state_2);
pca9555_readPins(IC1, PCA9555_PORT1, &state_3); pca9555_readPins(IC2, PCA9555_PORT1, &state_3);
uint32_t state = ((((uint32_t)state_3 & 0b01111111) << 12) | ((uint32_t)state_2 << 4) | (((uint32_t)state_1 & 0b11110000) >> 4)); uint32_t state = ((((uint32_t)state_3 & 0b01111111) << 12) | ((uint32_t)state_2 << 4) | (((uint32_t)state_1 & 0b11110000) >> 4));
return ~state; return ~state;