diff options
author | Joel Challis <git@zvecr.com> | 2020-02-25 14:10:04 +0000 |
---|---|---|
committer | Drashna Jael're <drashna@live.com> | 2020-03-26 00:42:13 -0700 |
commit | 28f45b8ff4073528ad8a665e1471ca5e927759e8 (patch) | |
tree | a2ab1f05330add6f6d441c557337d8ffb42b3f33 /quantum/split_common/split_util.c | |
parent | ebb243aacee704cf5bbb63b65439c06703ba5bc1 (diff) |
Split - Avoid race condition during matrix_init_quantum (#8235)
* Avoid race condition during matrix_init_quantum
* spelling is hard
Diffstat (limited to 'quantum/split_common/split_util.c')
-rw-r--r-- | quantum/split_common/split_util.c | 29 |
1 files changed, 14 insertions, 15 deletions
diff --git a/quantum/split_common/split_util.c b/quantum/split_common/split_util.c index 07d3bb36ad..fb6a3b85af 100644 --- a/quantum/split_common/split_util.c +++ b/quantum/split_common/split_util.c @@ -81,19 +81,8 @@ __attribute__((weak)) bool is_keyboard_master(void) { return (usbstate == MASTER); } -static void keyboard_master_setup(void) { -#if defined(USE_I2C) || defined(EH) -# ifdef SSD1306OLED - matrix_master_OLED_init(); -# endif -#endif - transport_master_init(); -} - -static void keyboard_slave_setup(void) { transport_slave_init(); } - // this code runs before the keyboard is fully initialized -void keyboard_split_setup(void) { +void split_pre_init(void) { isLeftHand = is_keyboard_left(); #if defined(RGBLIGHT_ENABLE) && defined(RGBLED_SPLIT) @@ -106,8 +95,18 @@ void keyboard_split_setup(void) { #endif if (is_keyboard_master()) { - keyboard_master_setup(); - } else { - keyboard_slave_setup(); +#if defined(USE_I2C) && defined(SSD1306OLED) + matrix_master_OLED_init(); +#endif + transport_master_init(); + } +} + +// this code runs after the keyboard is fully initialized +// - avoids race condition during matrix_init_quantum where slave can start +// receiving before the init process has completed +void split_post_init(void) { + if (!is_keyboard_master()) { + transport_slave_init(); } } |