You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

116 lines
3.5 KiB

Make solo half of split keyboards (more) usable. (#13523) * Make solo half of split keyboards (more) usable. Using only one half of a split keyboard (that's using the split_common framework to communicate) is not a great experience, since several read timeouts per scan cycle cause an unusably slow scan rate. This change blocks all split communication attempts for 500 ms (configurable) after an error occurs, causing the scan rate to become at least _more_ usable, but might need some tweaking to work fully on most keyboards. One read timeout still needs to occur after the 500 ms has passed, and if that timeout isn't low enough, some scan cycles may still be too slow. * Fix lint complaint. * Require 25 consecutive comm errors to see comms as disconnected. The number of max errors can be overridden by defining `SPLIT_MAX_CONNECTION_ERRORS`. * Add comments to new defines, and ability to disable disconnection check. Also increase `SPLIT_MAX_CONNECTION_ERRORS` to 40, since it's divisible by most relevant numbers for the description. * Make lint happy ...again * Only update `connection_check_timer` when needed. * Add new defines to split keyboard documentation. * Move connection timeout logic to transport.c, add `is_transport_connected`. * Use split_common disconnection logic in matrix.c. Instead of doing more or less the same thing twice. * Move disconnection logic to `transport_master`. Is a cleaner implementation, and causes the scan rate while disconnected to increase instead of decrease. * Lint fixes. * Lower default `SERIAL_USART_TIMEOUT` to 20 ms. The read timeout must be low enough to not cause exessively long scan cycles when using a solo split half. 10 ms was determined from testing to work fine even with the slowest defined baudrate of 19200 (5 ms was too low for that case), so 20 ms should be fine for most cases. * Remove `SERIAL_USART_TIMEOUT` from ergodox_infinity/config.h Was somewhat mistakenly included in an earlier PR. * Fix building with `USE_I2C`. * Reduce built firmware size. Not really sure why this works, the idea was taken from tzarc's work on split disconnection. * Tweak and improve opt-out for split disconnection logic. There are now two ways to opt out from this feature: * Set `SPLIT_MAX_CONNECTION_ERRORS` to 0. This will completely disable the connection status checks (also affects the slave matrix reset logic in matrix.c, though). * Set `SPLIT_CONNECTION_CHECK_TIMEOUT` to 0. This will only disable the communication throttling while disconnected. Will make the firmware smaller. * Make split disconnection logic work with custom transports. Includes a fallback implementation for keyboards using a custom split_util.c but not a custom matrix.c (currently no such keyboard seems to be merged, though). * Remove unnecessary include of timer.h Co-authored-by: Joel Challis <git@zvecr.com> Co-authored-by: Joel Challis <git@zvecr.com>
2 years ago
  1. /* Copyright 2021 QMK
  2. *
  3. * This program is free software: you can redistribute it and/or modify
  4. * it under the terms of the GNU General Public License as published by
  5. * the Free Software Foundation, either version 3 of the License, or
  6. * (at your option) any later version.
  7. *
  8. * This program is distributed in the hope that it will be useful,
  9. * but WITHOUT ANY WARRANTY; without even the implied warranty of
  10. * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
  11. * GNU General Public License for more details.
  12. *
  13. * You should have received a copy of the GNU General Public License
  14. * along with this program. If not, see <http://www.gnu.org/licenses/>.
  15. */
  16. #pragma once
  17. #include "quantum.h"
  18. #include "serial.h"
  19. #include "printf.h"
  20. #include <ch.h>
  21. #include <hal.h>
  22. #if !defined(SERIAL_USART_DRIVER)
  23. # define SERIAL_USART_DRIVER SD1
  24. #endif
  25. #if !defined(USE_GPIOV1)
  26. /* The default PAL alternate modes are used to signal that the pins are used for USART. */
  27. # if !defined(SERIAL_USART_TX_PAL_MODE)
  28. # define SERIAL_USART_TX_PAL_MODE 7
  29. # endif
  30. # if !defined(SERIAL_USART_RX_PAL_MODE)
  31. # define SERIAL_USART_RX_PAL_MODE 7
  32. # endif
  33. #endif
  34. #if defined(SOFT_SERIAL_PIN)
  35. # define SERIAL_USART_TX_PIN SOFT_SERIAL_PIN
  36. #endif
  37. #if !defined(SERIAL_USART_TX_PIN)
  38. # define SERIAL_USART_TX_PIN A9
  39. #endif
  40. #if !defined(SERIAL_USART_RX_PIN)
  41. # define SERIAL_USART_RX_PIN A10
  42. #endif
  43. #if !defined(USART_CR1_M0)
  44. # define USART_CR1_M0 USART_CR1_M // some platforms (f1xx) dont have this so
  45. #endif
  46. #if !defined(SERIAL_USART_CR1)
  47. # define SERIAL_USART_CR1 (USART_CR1_PCE | USART_CR1_PS | USART_CR1_M0) // parity enable, odd parity, 9 bit length
  48. #endif
  49. #if !defined(SERIAL_USART_CR2)
  50. # define SERIAL_USART_CR2 (USART_CR2_STOP_1) // 2 stop bits
  51. #endif
  52. #if !defined(SERIAL_USART_CR3)
  53. # define SERIAL_USART_CR3 0
  54. #endif
  55. #if defined(USART1_REMAP)
  56. # define USART_REMAP \
  57. do { \
  58. (AFIO->MAPR |= AFIO_MAPR_USART1_REMAP); \
  59. } while (0)
  60. #elif defined(USART2_REMAP)
  61. # define USART_REMAP \
  62. do { \
  63. (AFIO->MAPR |= AFIO_MAPR_USART2_REMAP); \
  64. } while (0)
  65. #elif defined(USART3_PARTIALREMAP)
  66. # define USART_REMAP \
  67. do { \
  68. (AFIO->MAPR |= AFIO_MAPR_USART3_REMAP_PARTIALREMAP); \
  69. } while (0)
  70. #elif defined(USART3_FULLREMAP)
  71. # define USART_REMAP \
  72. do { \
  73. (AFIO->MAPR |= AFIO_MAPR_USART3_REMAP_FULLREMAP); \
  74. } while (0)
  75. #endif
  76. #if !defined(SELECT_SOFT_SERIAL_SPEED)
  77. # define SELECT_SOFT_SERIAL_SPEED 1
  78. #endif
  79. #if defined(SERIAL_USART_SPEED)
  80. // Allow advanced users to directly set SERIAL_USART_SPEED
  81. #elif SELECT_SOFT_SERIAL_SPEED == 0
  82. # define SERIAL_USART_SPEED 460800
  83. #elif SELECT_SOFT_SERIAL_SPEED == 1
  84. # define SERIAL_USART_SPEED 230400
  85. #elif SELECT_SOFT_SERIAL_SPEED == 2
  86. # define SERIAL_USART_SPEED 115200
  87. #elif SELECT_SOFT_SERIAL_SPEED == 3
  88. # define SERIAL_USART_SPEED 57600
  89. #elif SELECT_SOFT_SERIAL_SPEED == 4
  90. # define SERIAL_USART_SPEED 38400
  91. #elif SELECT_SOFT_SERIAL_SPEED == 5
  92. # define SERIAL_USART_SPEED 19200
  93. #else
  94. # error invalid SELECT_SOFT_SERIAL_SPEED value
  95. #endif
  96. #if !defined(SERIAL_USART_TIMEOUT)
  97. # define SERIAL_USART_TIMEOUT 20
  98. #endif
  99. #define HANDSHAKE_MAGIC 7