diff --git a/drivers/gpio/mcp23018.c b/drivers/gpio/mcp23018.c index 0791db08ddab..ee10633631a8 100644 --- a/drivers/gpio/mcp23018.c +++ b/drivers/gpio/mcp23018.c @@ -7,7 +7,9 @@ #include "debug.h" #define SLAVE_TO_ADDR(n) (n << 1) -#define TIMEOUT 100 +#ifndef MCP23018_TIMEOUT +# define MCP23018_TIMEOUT 100 +#endif // MCP23018_TIMEOUT enum { CMD_IODIRA = 0x00, // i/o direction register @@ -33,13 +35,13 @@ bool mcp23018_set_config(uint8_t slave_addr, mcp23018_port_t port, uint8_t conf) uint8_t cmdDirection = port ? CMD_IODIRB : CMD_IODIRA; uint8_t cmdPullup = port ? CMD_GPPUB : CMD_GPPUA; - i2c_status_t ret = i2c_write_register(addr, cmdDirection, &conf, sizeof(conf), TIMEOUT); + i2c_status_t ret = i2c_write_register(addr, cmdDirection, &conf, sizeof(conf), MCP23018_TIMEOUT); if (ret != I2C_STATUS_SUCCESS) { dprintf("mcp23018_set_config::directionFAILED::%u\n", ret); return false; } - ret = i2c_write_register(addr, cmdPullup, &conf, sizeof(conf), TIMEOUT); + ret = i2c_write_register(addr, cmdPullup, &conf, sizeof(conf), MCP23018_TIMEOUT); if (ret != I2C_STATUS_SUCCESS) { dprintf("mcp23018_set_config::pullupFAILED::%u\n", ret); return false; @@ -52,7 +54,7 @@ bool mcp23018_set_output(uint8_t slave_addr, mcp23018_port_t port, uint8_t conf) uint8_t addr = SLAVE_TO_ADDR(slave_addr); uint8_t cmd = port ? CMD_GPIOB : CMD_GPIOA; - i2c_status_t ret = i2c_write_register(addr, cmd, &conf, sizeof(conf), TIMEOUT); + i2c_status_t ret = i2c_write_register(addr, cmd, &conf, sizeof(conf), MCP23018_TIMEOUT); if (ret != I2C_STATUS_SUCCESS) { dprintf("mcp23018_set_output::FAILED::%u\n", ret); return false; @@ -65,7 +67,7 @@ bool mcp23018_set_output_all(uint8_t slave_addr, uint8_t confA, uint8_t confB) { uint8_t addr = SLAVE_TO_ADDR(slave_addr); uint8_t conf[2] = {confA, confB}; - i2c_status_t ret = i2c_write_register(addr, CMD_GPIOA, &conf[0], sizeof(conf), TIMEOUT); + i2c_status_t ret = i2c_write_register(addr, CMD_GPIOA, &conf[0], sizeof(conf), MCP23018_TIMEOUT); if (ret != I2C_STATUS_SUCCESS) { dprintf("mcp23018_set_output::FAILED::%u\n", ret); return false; @@ -78,7 +80,7 @@ bool mcp23018_read_pins(uint8_t slave_addr, mcp23018_port_t port, uint8_t* out) uint8_t addr = SLAVE_TO_ADDR(slave_addr); uint8_t cmd = port ? CMD_GPIOB : CMD_GPIOA; - i2c_status_t ret = i2c_read_register(addr, cmd, out, sizeof(uint8_t), TIMEOUT); + i2c_status_t ret = i2c_read_register(addr, cmd, out, sizeof(uint8_t), MCP23018_TIMEOUT); if (ret != I2C_STATUS_SUCCESS) { dprintf("mcp23018_read_pins::FAILED::%u\n", ret); return false; @@ -97,7 +99,7 @@ bool mcp23018_read_pins_all(uint8_t slave_addr, uint16_t* out) { data16 data = {.u16 = 0}; - i2c_status_t ret = i2c_read_register(addr, CMD_GPIOA, &data.u8[0], sizeof(data), TIMEOUT); + i2c_status_t ret = i2c_read_register(addr, CMD_GPIOA, &data.u8[0], sizeof(data), MCP23018_TIMEOUT); if (ret != I2C_STATUS_SUCCESS) { dprintf("mcp23018_read_pins_all::FAILED::%u\n", ret); return false; diff --git a/keyboards/zsa/ergodox_ez/ergodox_ez.h b/keyboards/zsa/ergodox_ez/ergodox_ez.h index 633f6a148721..338c97c5f73f 100644 --- a/keyboards/zsa/ergodox_ez/ergodox_ez.h +++ b/keyboards/zsa/ergodox_ez/ergodox_ez.h @@ -31,7 +31,7 @@ along with this program. If not, see . extern i2c_status_t mcp23018_status; #ifndef ERGODOX_EZ_I2C_TIMEOUT -# define ERGODOX_EZ_I2C_TIMEOUT 80 +# define ERGODOX_EZ_I2C_TIMEOUT 10 #endif #ifndef MCP23018_EXPANDER_I2C_ADDR # define MCP23018_EXPANDER_I2C_ADDR (0x20 << 1) diff --git a/keyboards/zsa/ergodox_ez/post_config.h b/keyboards/zsa/ergodox_ez/post_config.h index b8f06da2f408..b7889cc6e73f 100644 --- a/keyboards/zsa/ergodox_ez/post_config.h +++ b/keyboards/zsa/ergodox_ez/post_config.h @@ -40,6 +40,6 @@ along with this program. If not, see . #endif #define WS2812_LED_COUNT RGBLIGHT_LED_COUNT -#ifndef ISSI_TIMEOUT -# define ISSI_TIMEOUT 3 +#ifndef IS31FL3731_I2C_TIMEOUT +# define IS31FL3731_I2C_TIMEOUT 3 #endif diff --git a/keyboards/zsa/moonlander/config.h b/keyboards/zsa/moonlander/config.h index 4e82cec8bb0d..a76474d869a5 100644 --- a/keyboards/zsa/moonlander/config.h +++ b/keyboards/zsa/moonlander/config.h @@ -22,6 +22,9 @@ #define MATRIX_ROWS 12 #define MATRIX_COLS 7 +#define MCP23018_TIMEOUT 10 + + #define EEPROM_I2C_24LC128 #define IS31FL3731_I2C_ADDRESS_1 IS31FL3731_I2C_ADDRESS_GND diff --git a/keyboards/zsa/planck_ez/config.h b/keyboards/zsa/planck_ez/config.h index 1145df9ef97e..ea28be219af0 100644 --- a/keyboards/zsa/planck_ez/config.h +++ b/keyboards/zsa/planck_ez/config.h @@ -24,3 +24,4 @@ #define AUDIO_PIN_ALT_AS_NEGATIVE #define IS31FL3737_I2C_ADDRESS_1 IS31FL3737_I2C_ADDRESS_GND +#define IS31FL3737_I2C_TIMEOUT 5 diff --git a/keyboards/zsa/voyager/config.h b/keyboards/zsa/voyager/config.h index 27460c591044..2310d0888d00 100644 --- a/keyboards/zsa/voyager/config.h +++ b/keyboards/zsa/voyager/config.h @@ -7,6 +7,8 @@ #define MATRIX_COLS 7 #define MATRIX_ROWS 12 +#define MCP23018_TIMEOUT 10 + #define IS31FL3731_I2C_ADDRESS_1 IS31FL3731_I2C_ADDRESS_GND #define IS31FL3731_I2C_ADDRESS_2 IS31FL3731_I2C_ADDRESS_VCC