]> git.donarmstrong.com Git - qmk_firmware.git/commitdiff
Whitespace cleanup.
authoralex-ong <the.onga@gmail.com>
Sat, 26 Jan 2019 06:38:52 +0000 (17:38 +1100)
committeralex-ong <the.onga@gmail.com>
Sat, 26 Jan 2019 06:38:52 +0000 (17:38 +1100)
common_features.mk
docs/feature_debounce_algo.md
keyboards/handwired/xealous/rules.mk
tmk_core/common.mk
tmk_core/common/keyboard.c

index 9b9d017a7bc8510e12f56378126cc8e9c6915201..deb30706addd4612bfeab9727d635b7dee982c7a 100644 (file)
@@ -249,15 +249,11 @@ endif
 
 include $(DRIVER_PATH)/qwiic/qwiic.mk
 
-
 QUANTUM_SRC:= \
     $(QUANTUM_DIR)/quantum.c \
     $(QUANTUM_DIR)/keymap_common.c \
     $(QUANTUM_DIR)/keycode_config.c
 
-
-
-
 # Include the standard or split matrix code if needed
 ifneq ($(strip $(CUSTOM_MATRIX)), yes)
     ifeq ($(strip $(SPLIT_KEYBOARD)), yes)
index 2c694cdfbde415e10ea7d33ab6b0ddf9c54dfb87..e4489662e3e837ac16c40d34aded7c1751cc2fcd 100644 (file)
@@ -7,18 +7,16 @@ The underlying debounce algorithm is determined by which matrix.c file you are u
 The logic for which debounce method called is below. It checks various defines that you have set in rules.mk
 
 ```
-ifeq ($(strip $(SPLIT_KEYBOARD)), yes)
-    # Do nothing, debouncing is inside matrix.c inside split_common
-else ifeq ($(strip $(DEBOUNCE_ALGO)), manual)
+ifeq ($(strip $(DEBOUNCE_ALGO)), manual)
     # Do nothing. do your debouncing in matrix.c
 else ifeq ($(strip $(DEBOUNCE_ALGO)), sym_g)
-    TMK_COMMON_SRC += $(DEBOUNCE)/debounce_sym_g.c
+    QUANTUM_SRC += $(DEBOUNCE)/debounce_sym_g.c
 else ifeq ($(strip $(DEBOUNCE_ALGO)), eager_pk)
-    TMK_COMMON_SRC += $(DEBOUNCE)/debounce_eager_pk.c
+    QUANTUM_SRC += $(DEBOUNCE)/debounce_eager_pk.c
 else ifeq ($(strip $(CUSTOM_MATRIX)), yes)
     # Do nothing. Custom matrix code.
 else # default algorithm
-    TMK_COMMON_SRC += $(DEBOUNCE)/debounce_sym_g.c
+    QUANTUM_SRC += $(DEBOUNCE)/debounce_sym_g.c
 endif
 ```
 
@@ -32,11 +30,10 @@ The following is for keyboards where ```SPLIT_KEYBOARD``` is **not** defined as
 | sym_g / eager_pk | Not defined   | You are using the included matrix.c and debounce.c          | Nothing. Chosen debounce method used. |
 | Not defined      | YES           | You have your own matrix.c, and your own debounce           | Write the fully debounced matrix into matrix.c's matrix |
 | manual           | YES           | Same as above                                               | same as above                                           |
-| sym_g/ eager_pk  | YES           | You are using your own matrix.c, but included debounce      | Write the raw matrix values into matrix.c's matrix      |
+| sym_g / eager_pk | YES           | You are using your own matrix.c, but included debounce      | Write the raw matrix values into matrix.c's matrix      |
 
-**Note**: 
-If ```SPLIT_KEYBOARD = YES``` is defined, the algorithm inside split_common will be used.
-A future pull request will fix this to use the debounce.c code.
+**Regarding split keyboards**: 
+The debounce code is compatible with split keyboards.
 
 # Use your own debouncing code
 * Set ```DEBOUNCE_ALGO = manual```.
index 07e1c875e603b19e4f94361862909dedb61d41a8..37afdaff93faeabe44c36e9c9406930011d8c2e9 100644 (file)
@@ -68,7 +68,7 @@ SUBPROJECT_rev1 = yes
 SLEEP_LED_ENABLE = no    # Breathing sleep LED during USB suspend
 
 CUSTOM_MATRIX = no
-CUSTOM_DEBOUNCE = yes
+DEBOUNCE_ALGO = manual
 
 LAYOUTS = split60
 
index 335a36cfe7627c3b456432a2b565628b4bd3a2e0..063115acb1ab803bcb4deabc228eeb0124a13839 100644 (file)
@@ -1,5 +1,4 @@
 COMMON_DIR = common
-
 ifeq ($(PLATFORM),AVR)
        PLATFORM_COMMON_DIR = $(COMMON_DIR)/avr
 else ifeq ($(PLATFORM),CHIBIOS)
@@ -62,6 +61,7 @@ ifeq ($(PLATFORM),TEST)
 endif
 
 
+
 # Option modules
 BOOTMAGIC_ENABLE ?= no
 VALID_MAGIC_TYPES := yes full lite
index d223001160a7b1a425e08d8dcc76a8c91a3f6697..25be28d02a91e2ba8d2b617cb260d664c0ae6863 100644 (file)
@@ -163,7 +163,7 @@ bool is_keyboard_master(void) {
  */
 void keyboard_init(void) {
     timer_init();
-    matrix_init();    
+    matrix_init();
 #ifdef QWIIC_ENABLE
     qwiic_init();
 #endif
@@ -223,7 +223,7 @@ void keyboard_task(void)
     uint8_t keys_processed = 0;
 #endif
 
-    matrix_scan();    
+    matrix_scan();
 
     if (is_keyboard_master()) {
         for (uint8_t r = 0; r < MATRIX_ROWS; r++) {
@@ -231,7 +231,7 @@ void keyboard_task(void)
             matrix_change = matrix_row ^ matrix_prev[r];
             if (matrix_change) {
 #ifdef MATRIX_HAS_GHOST
-                if (has_ghost_in_row(r, matrix_row)) continue;
+                if (has_ghost_in_row(r, matrix_row)) { continue; }
 #endif
                 if (debug_matrix) matrix_print();
                 for (uint8_t c = 0; c < MATRIX_COLS; c++) {