Merge remote-tracking branch 'origin/develop' into xap

data/mappings/info_config.hjson

@@ -10,6 +10,9 @@
     // deprecated: Default `false`. Set to `true` to turn on warning when a value exists
     // invalid: Default `false`. Set to `true` to generate errors when a value exists
     // replace_with: use with a key marked deprecated or invalid to designate a replacement
+    "APA102_DI_PIN": {"info_key": "apa102.data_pin"},
+    "APA102_CI_PIN": {"info_key": "apa102.clock_pin"},
+    "APA102_DEFAULT_BRIGHTNESS": {"info_key": "apa102.default_brightness", "value_type": "int"},
     "AUDIO_VOICES": {"info_key": "audio.voices", "value_type": "bool"},
     "BACKLIGHT_BREATHING": {"info_key": "backlight.breathing", "value_type": "bool"},
     "BREATHING_PERIOD": {"info_key": "backlight.breathing_period", "value_type": "int"},

data/schemas/keyboard.jsonschema

@@ -96,6 +96,19 @@
                 "unknown"
             ]
         },
+        "apa102": {
+            "type": "object",
+            "additionalProperties": false,
+            "properties": {
+                "data_pin": {"$ref": "qmk.definitions.v1#/mcu_pin"},
+                "clock_pin": {"$ref": "qmk.definitions.v1#/mcu_pin"},
+                "default_brightness": {
+                    "type": "integer",
+                    "minimum": 0,
+                    "maximum": 31
+                }
+            }
+        },
         "audio": {
             "type": "object",
             "additionalProperties": false,

docs/feature_rgb_matrix.md

@@ -383,9 +383,9 @@ Configure the hardware via your `config.h`:
 
 ```c
 // The pin connected to the data pin of the LEDs
-#define RGB_DI_PIN D7
+#define APA102_DI_PIN D7
 // The pin connected to the clock pin of the LEDs
-#define RGB_CI_PIN D6
+#define APA102_CI_PIN D6
 // The number of LEDs connected
 #define RGB_MATRIX_LED_COUNT 70
 ```

docs/feature_rgblight.md

@@ -35,8 +35,9 @@ At minimum you must define the data pin your LED strip is connected to, and the
 
 |Define         |Description                                                                                              |
 |---------------|---------------------------------------------------------------------------------------------------------|
-|`RGB_DI_PIN`   |The pin connected to the data pin of the LEDs                                                            |
+|`RGB_DI_PIN`   |The pin connected to the data pin of the LEDs (WS2812)                                                   |
-|`RGB_CI_PIN`   |The pin connected to the clock pin of the LEDs (APA102 only)                                             |
+|`APA102_DI_PIN`|The pin connected to the data pin of the LEDs (APA102)                                                   |
+|`APA102_CI_PIN`|The pin connected to the clock pin of the LEDs (APA102)                                                  |
 |`RGBLED_NUM`   |The number of LEDs connected                                                                             |
 |`RGBLED_SPLIT` |(Optional) For split keyboards, the number of LEDs connected on each half directly wired to `RGB_DI_PIN` |
 

drivers/led/apa102.c

@@ -27,7 +27,7 @@
 #        if defined(STM32F0XX) || defined(STM32F1XX) || defined(STM32F3XX) || defined(STM32F4XX) || defined(STM32L0XX) || defined(GD32VF103)
 #            define APA102_NOPS (100 / (1000000000L / (CPU_CLOCK / 4))) // This calculates how many loops of 4 nops to run to delay 100 ns
 #        else
-#            error("APA102_NOPS configuration required")
+#            error APA102_NOPS configuration required
 #            define APA102_NOPS 0 // this just pleases the compile so the above error is easier to spot
 #        endif
 #    endif
@@ -43,14 +43,14 @@
         }                                       \
     } while (0)
 
-#define APA102_SEND_BIT(byte, bit)               \
+#define APA102_SEND_BIT(byte, bit)                  \
-    do {                                         \
+    do {                                            \
-        writePin(RGB_DI_PIN, (byte >> bit) & 1); \
+        writePin(APA102_DI_PIN, (byte >> bit) & 1); \
-        io_wait;                                 \
+        io_wait;                                    \
-        writePinHigh(RGB_CI_PIN);                \
+        writePinHigh(APA102_CI_PIN);                \
-        io_wait;                                 \
+        io_wait;                                    \
-        writePinLow(RGB_CI_PIN);                 \
+        writePinLow(APA102_CI_PIN);                 \
-        io_wait;                                 \
+        io_wait;                                    \
     } while (0)
 
 uint8_t apa102_led_brightness = APA102_DEFAULT_BRIGHTNESS;
@@ -77,11 +77,11 @@ void rgblight_call_driver(LED_TYPE *start_led, uint8_t num_leds) {
 }
 
 void static apa102_init(void) {
-    setPinOutput(RGB_DI_PIN);
+    setPinOutput(APA102_DI_PIN);
-    setPinOutput(RGB_CI_PIN);
+    setPinOutput(APA102_CI_PIN);
 
-    writePinLow(RGB_DI_PIN);
+    writePinLow(APA102_DI_PIN);
-    writePinLow(RGB_CI_PIN);
+    writePinLow(APA102_CI_PIN);
 }
 
 void apa102_set_brightness(uint8_t brightness) {

keyboards/25keys/cassette42/common/oled_helper.c

@@ -1,7 +1,9 @@
+#include "oled_helper.h"
+#include "quantum.h"
+#include <stdio.h>
+#include <string.h>
+
 #ifdef OLED_ENABLE
-#    include QMK_KEYBOARD_H
-#    include <stdio.h>
-#    include <string.h>
 
 void render_logo(void) {
     static const char PROGMEM logo[] = {0x80, 0x81, 0x82, 0x83, 0x84, 0x85, 0x86, 0x87, 0x88, 0x89, 0x8a, 0x8b, 0x8c, 0x8d, 0x8e, 0x8f, 0x90, 0x91, 0x92, 0x93, 0x94, 0xa0, 0xa1, 0xa2, 0xa3, 0xa4, 0xa5, 0xa6, 0xa7, 0xa8, 0xa9, 0xaa, 0xab, 0xac, 0xad, 0xae, 0xaf, 0xb0, 0xb1, 0xb2, 0xb3, 0xb4, 0};

keyboards/3w6/rev1/matrix.c

@@ -24,7 +24,9 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
  * to repeating that information all over the place.
  */
 
-#include QMK_KEYBOARD_H
+#include "matrix.h"
+#include "debug.h"
+#include "wait.h"
 #include "i2c_master.h"
 
 extern i2c_status_t tca9555_status;

keyboards/3w6/rev2/matrix.c

@@ -24,7 +24,9 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
  * to repeating that information all over the place.
  */
 
-#include QMK_KEYBOARD_H
+#include "matrix.h"
+#include "debug.h"
+#include "wait.h"
 #include "i2c_master.h"
 
 extern i2c_status_t tca9555_status;

keyboards/adm42/rev4/rev4.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 led_config_t g_led_config = {
     {

keyboards/argo_works/ishi/80/mk0_avr_extra/mk0_avr_extra.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 #ifdef OLED_ENABLE
 static void render_logo(void) {

keyboards/converter/palm_usb/matrix.c

@@ -17,7 +17,9 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-#include QMK_KEYBOARD_H
+#include "matrix.h"
+#include "debug.h"
+#include "wait.h"
 #include "uart.h"
 #include "timer.h"
 
@@ -134,14 +136,14 @@ uint8_t rts_reset(void) {
     if (firstread) {
         writePinLow(RTS_PIN);
     } 
-     _delay_ms(10);
+     wait_ms(10);
     writePinHigh(RTS_PIN);
     
 
 /* the future is Arm 
     if (!palReadPad(RTS_PIN_IOPRT))
   {
-    _delay_ms(10);
+    wait_ms(10);
     palSetPadMode(RTS_PINn_IOPORT, PinDirectionOutput_PUSHPULL);
     palSetPad(RTS_PORT, RTS_PIN);
   }
@@ -150,13 +152,13 @@ uint8_t rts_reset(void) {
     palSetPadMode(RTS_PIN_RTS_PORT, PinDirectionOutput_PUSHPULL);
     palSetPad(RTS_PORT, RTS_PIN);
     palClearPad(RTS_PORT, RTS_PIN);
-    _delay_ms(10);
+    wait_ms(10);
     palSetPad(RTS_PORT, RTS_PIN);
   }
 */
 
 
- _delay_ms(5);  
+ wait_ms(5);  
  //print("rts\n");
  return 1;
 }
@@ -222,7 +224,7 @@ uint8_t handspring_handshake(void) {
 
 uint8_t handspring_reset(void) {
     writePinLow(VCC_PIN);
-    _delay_ms(5);
+    wait_ms(5);
     writePinHigh(VCC_PIN);
 
     if ( handspring_handshake() ) {
@@ -257,7 +259,7 @@ void matrix_init(void)
         last_activity = timer_read();
     } else { 
         print("failed handshake");
-        _delay_ms(1000);
+        wait_ms(1000);
         //BUG /should/ power cycle or toggle RTS & reset, but this usually works. 
     }
 
@@ -271,7 +273,7 @@ void matrix_init(void)
         last_activity = timer_read();
     } else { 
         print("failed handshake");
-        _delay_ms(1000);
+        wait_ms(1000);
         //BUG /should/ power cycle or toggle RTS & reset, but this usually works. 
     }
 

keyboards/converter/sun_usb/command_extra.c

@@ -1,64 +0,0 @@
-#include QMK_KEYBOARD_H
-#include "uart.h"
-
-bool sun_bell = false;
-bool sun_click = false;
-
-
-bool command_extra(uint8_t code)
-{
-    switch (code) {
-        case KC_H:
-        case KC_SLASH: /* ? */
-            print("\n\n----- Sun converter Help -----\n");
-            print("Home:        Toggle Bell\n");
-            print("End:         Toggle Click\n");
-            print("PgUp:        LED all On\n");
-            print("PgDown:      LED all Off\n");
-            print("Insert:      Layout\n");
-            print("Delete:      Reset\n");
-            return false;
-        case KC_DEL:
-            print("Reset\n");
-            uart_write(0x01);
-            break;
-        case KC_HOME:
-	    sun_bell = !sun_bell;
-	    if (sun_bell) {
-                print("Bell On\n");
-	        uart_write(0x02);
-	    } else {
-	        print("Bell Off\n");
-	        uart_write(0x03);
-	    }
-            break;
-        case KC_END:
-	    sun_click = !sun_click;
-	    if (sun_click) {
-	        print("Click On\n");
-		uart_write(0x0A);
-	    } else {
-	        print("Click Off\n");
-                uart_write(0x0B);
-	    }
-	    break;
-        case KC_PGUP:
-            print("LED all on\n");
-            uart_write(0x0E);
-            uart_write(0xFF);
-            break;
-        case KC_PGDN:
-            print("LED all off\n");
-            uart_write(0x0E);
-            uart_write(0x00);
-            break;
-        case KC_INSERT:
-            print("layout\n");
-            uart_write(0x0F);
-            break;
-        default:
-            xprintf("Unknown extra command: %02X\n", code);
-            return false;
-    }
-    return true;
-}

keyboards/converter/sun_usb/led.c

@@ -15,7 +15,8 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-#include QMK_KEYBOARD_H
+#include "led.h"
+#include "print.h"
 #include "uart.h"
 
 void led_set(uint8_t usb_led)

keyboards/converter/sun_usb/matrix.c

@@ -15,7 +15,11 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-#include QMK_KEYBOARD_H
+#include "matrix.h"
+#include "host.h"
+#include "led.h"
+#include "debug.h"
+#include "wait.h"
 #include "uart.h"
 
 /*
@@ -86,9 +90,9 @@ void matrix_init(void)
     /*     print("."); */
     /*     while (uart_read()); */
     /*     uart_write(0x01); */
-    /*     _delay_ms(500); */
+    /*     wait_ms(500); */
     /*     if (uart_read() == 0xFF) { */
-    /*         _delay_ms(500); */
+    /*         wait_ms(500); */
     /*         if (uart_read() == 0x04) */
     /*             break; */
     /*     } */
@@ -112,7 +116,7 @@ uint8_t matrix_scan(void)
     switch (code) {
         case 0xFF:  // reset success: FF 04
             print("reset: ");
-            _delay_ms(500);
+            wait_ms(500);
             code = uart_read();
             xprintf("%02X\n", code);
             if (code == 0x04) {
@@ -122,12 +126,12 @@ uint8_t matrix_scan(void)
             return 0;
         case 0xFE:  // layout: FE <layout>
             print("layout: ");
-            _delay_ms(500);
+            wait_ms(500);
             xprintf("%02X\n", uart_read());
             return 0;
         case 0x7E:  // reset fail: 7E 01
             print("reset fail: ");
-            _delay_ms(500);
+            wait_ms(500);
             xprintf("%02X\n", uart_read());
             return 0;
         case 0x7F:

keyboards/converter/xmk/xmk.c

@@ -3,7 +3,7 @@
 
 // SPDX-License-Identifier: GPL-2.0-or-later
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 #if defined (XMK_DEBUG)
 void keyboard_post_init_kb(void) {

keyboards/converter/xmk/xmk_matrix.c

@@ -4,6 +4,8 @@
 // SPDX-License-Identifier: GPL-2.0-or-later
 
 #include "xmk_matrix.h"
+#include "matrix.h"
+#include "debug.h"
 
 bool xmk_changed = false;
 matrix_row_t xmk_rows[MATRIX_ROWS];

keyboards/converter/xmk/xmk_matrix.h

@@ -5,6 +5,7 @@
 
 #pragma once
 
-#include QMK_KEYBOARD_H
+#include <stdbool.h>
+#include <stdint.h>
 
 void xmk_matrix_key(bool press, uint8_t key);

keyboards/crkbd/lib/layer_state_reader.c

@@ -1,7 +1,5 @@
-
+#include "quantum.h"
-#include QMK_KEYBOARD_H
 #include <stdio.h>
-#include "crkbd.h"
 
 // in the future, should use (1U<<_LAYER_NAME) instead, but needs to be moved to keymap,c
 #define L_BASE 0

keyboards/crkbd/lib/rgb_state_reader.c

@@ -1,6 +1,5 @@
 #ifdef RGBLIGHT_ENABLE
 
-#include QMK_KEYBOARD_H
 #include <stdio.h>
 
 extern rgblight_config_t rgblight_config;

keyboards/cxt_studio/cxt_studio.c

@@ -1,7 +1,7 @@
 // Copyright 2023 Colin Kinloch (@ColinKinloch)
 // SPDX-License-Identifier: GPL-2.0-or-later
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 static uint8_t anim = 0;
 

keyboards/ergodox_stm32/ergodox_stm32.c

@@ -1,5 +1,5 @@
+#include "ergodox_stm32.h"
 #include "i2c_master.h"
-#include QMK_KEYBOARD_H
 
 extern inline void ergodox_board_led_1_on(void);
 extern inline void ergodox_board_led_2_on(void);

keyboards/ergodox_stm32/matrix.c

@@ -1,13 +1,10 @@
-#include <stdint.h>
+#include "matrix.h"
-#include <stdbool.h>
 #include <string.h>
-#include <hal.h>
 #include "timer.h"
 #include "wait.h"
-#include "print.h"
+#include "debug.h"
-#include "matrix.h"
 #include "i2c_master.h"
-#include QMK_KEYBOARD_H
+#include "ergodox_stm32.h"
 
 #ifndef DEBOUNCE
 #define DEBOUNCE 10

keyboards/fc980c/matrix.c

@@ -29,8 +29,6 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 #include "matrix.h"
 #include "led.h"
 #include "avr/timer_avr.h"
-// #include QMK_KEYBOARD_H
-
 
 // Timer resolution check
 #if (1000000/TIMER_RAW_FREQ > 20)

keyboards/ferris/0_1/matrix.c

@@ -23,7 +23,9 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
  * to repeating that information all over the place.
  */
 
-#include QMK_KEYBOARD_H
+#include "matrix.h"
+#include "debug.h"
+#include "wait.h"
 #include "i2c_master.h"
 
 extern i2c_status_t mcp23017_status;

keyboards/ferris/0_2/matrix.c

@@ -23,9 +23,10 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
  * to repeating that information all over the place.
  */
 
-#include QMK_KEYBOARD_H
+#include "matrix.h"
+#include "debug.h"
+#include "wait.h"
 #include "i2c_master.h"
-#include <print.h>
 
 extern i2c_status_t mcp23017_status;
 #define MCP23017_I2C_TIMEOUT 1000

keyboards/gboards/butterstick/sten.h

@@ -7,7 +7,7 @@
 
 #pragma once
 
-#include QMK_KEYBOARD_H
+#include "butterstick.h"
 #include "mousekey.h"
 #include "keymap_steno.h"
 #include "wait.h"

keyboards/gboards/engine/engine.h

@@ -11,13 +11,11 @@
 */
 
 #pragma once
-#include QMK_KEYBOARD_H
+
+#include "quantum.h"
 #include <string.h>
-#include <stdint.h>
 #include <stdio.h>
 #include "config_engine.h"
-#include <avr/pgmspace.h>
-#include "wait.h"
 
 // Maximum values for combos
 #define COMBO_END 0x00

keyboards/gboards/ergotaco/ergotaco.c

@@ -1,4 +1,4 @@
-#include QMK_KEYBOARD_H
+#include "ergotaco.h"
 
 bool i2c_initialized = 0;
 i2c_status_t mcp23018_status = 0x20;

keyboards/gboards/ergotaco/matrix.c

@@ -17,15 +17,10 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "matrix.h"
-#include <stdint.h>
-#include <stdbool.h>
-#include <avr/io.h>
 #include "wait.h"
-#include "action_layer.h"
-#include "print.h"
 #include "debug.h"
 #include "util.h"
-#include QMK_KEYBOARD_H
+#include "ergotaco.h"
 
 #ifndef DEBOUNCE
 #   define DEBOUNCE	5

keyboards/gboards/g/engine.h

@@ -12,16 +12,10 @@
 
 #pragma once
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 #include <string.h>
-#include <stdint.h>
 #include <stdio.h>
 #include "config_engine.h"
-#include <avr/pgmspace.h>
-#include "wait.h"
-#ifdef MOUSEKEY_ENABLE
-#    include "mousekey.h"
-#endif
 
 // Set defaults
 #ifndef IN_CHORD_MASK

keyboards/gboards/georgi/georgi.c

@@ -1,4 +1,4 @@
-#include QMK_KEYBOARD_H
+#include "georgi.h"
 
 bool i2c_initialized = 0;
 i2c_status_t mcp23018_status = 0x20;

keyboards/gboards/georgi/matrix.c

@@ -17,16 +17,10 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "matrix.h"
-#include <stdint.h>
-#include <stdbool.h>
-#include <avr/io.h>
 #include "wait.h"
-#include "action_layer.h"
-#include "print.h"
 #include "debug.h"
 #include "util.h"
-#include "keymap_steno.h"
+#include "georgi.h"
-#include QMK_KEYBOARD_H
 
 #ifndef DEBOUNCE
 #   define DEBOUNCE	5

keyboards/gboards/georgi/sten.h

@@ -6,7 +6,7 @@
 // Amen.
 #pragma once
 
-#include QMK_KEYBOARD_H
+#include "georgi.h"
 #include "mousekey.h"
 #include "keymap_steno.h"
 #include "wait.h"

keyboards/gboards/gergo/gergo.c

@@ -1,4 +1,4 @@
-#include QMK_KEYBOARD_H
+#include "gergo.h"
 
 bool i2c_initialized = 0;
 i2c_status_t mcp23018_status = 0x20;

keyboards/gboards/gergo/matrix.c

@@ -17,16 +17,11 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 #include "matrix.h"
-#include <stdint.h>
-#include <stdbool.h>
-#include <avr/io.h>
 #include "wait.h"
-#include "action_layer.h"
-#include "print.h"
 #include "debug.h"
 #include "util.h"
 #include "debounce.h"
-#include QMK_KEYBOARD_H
+#include "gergo.h"
 
 #ifdef BALLER
 #include <avr/interrupt.h>

keyboards/gl516/xr63gl/xr63gl.c

@@ -1,7 +1,7 @@
 // Copyright 2022 gachiham (@gachiham)
 // SPDX-License-Identifier: GPL-2.0-or-later
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 const matrix_row_t matrix_mask[MATRIX_ROWS] = {
     0b1111111111111110,

keyboards/handwired/onekey/blackpill_f401/config.h

@@ -23,8 +23,6 @@
 
 #define ADC_PIN A0
 
-#define RGB_CI_PIN A2
-
 #define SOLENOID_PIN B12
 #define SOLENOID_PINS { B12, B13, B14, B15 }
 #define SOLENOID_PINS_ACTIVE_STATE { high, high, low }

keyboards/handwired/onekey/blackpill_f401/info.json

@@ -10,5 +10,9 @@
     },
     "rgblight": {
         "pin": "A1"
+    },
+    "apa102": {
+        "data_pin": "A1",
+        "clock_pin": "A2"
     }
 }

keyboards/handwired/onekey/blackpill_f401_tinyuf2/config.h

@@ -23,8 +23,6 @@
 
 #define ADC_PIN A0
 
-#define RGB_CI_PIN A2
-
 #define SOLENOID_PIN B12
 #define SOLENOID_PINS { B12, B13, B14, B15 }
 #define SOLENOID_PINS_ACTIVE_STATE { high, high, low }

keyboards/handwired/onekey/blackpill_f401_tinyuf2/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A1"
+    },
+    "apa102": {
+        "data_pin": "A1",
+        "clock_pin": "A2"
     }
 }

keyboards/handwired/onekey/blackpill_f411/config.h

@@ -23,8 +23,6 @@
 
 #define ADC_PIN A0
 
-#define RGB_CI_PIN A2
-
 #define SOLENOID_PIN B12
 #define SOLENOID_PINS { B12, B13, B14, B15 }
 #define SOLENOID_PINS_ACTIVE_STATE { high, high, low }

keyboards/handwired/onekey/blackpill_f411/info.json

@@ -10,5 +10,9 @@
     },
     "rgblight": {
         "pin": "A1"
+    },
+    "apa102": {
+        "data_pin": "A1",
+        "clock_pin": "A2"
     }
 }

keyboards/handwired/onekey/blackpill_f411_tinyuf2/config.h

@@ -23,8 +23,6 @@
 
 #define ADC_PIN A0
 
-#define RGB_CI_PIN A2
-
 #define SOLENOID_PIN B12
 #define SOLENOID_PINS { B12, B13, B14, B15 }
 #define SOLENOID_PINS_ACTIVE_STATE { high, high, low }

keyboards/handwired/onekey/blackpill_f411_tinyuf2/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A1"
+    },
+    "apa102": {
+        "data_pin": "A1",
+        "clock_pin": "A2"
     }
 }

keyboards/handwired/onekey/bluepill/config.h

@@ -21,5 +21,3 @@
 #define BACKLIGHT_PWM_CHANNEL 1
 
 #define ADC_PIN A0
-
-#define RGB_CI_PIN A2

keyboards/handwired/onekey/bluepill/info.json

@@ -10,5 +10,9 @@
     },
     "rgblight": {
         "pin": "A1"
+    },
+    "apa102": {
+        "data_pin": "A1",
+        "clock_pin": "A2"
     }
 }

keyboards/handwired/onekey/bluepill_f103c6/config.h

@@ -22,8 +22,6 @@
 
 #define ADC_PIN A0
 
-#define RGB_CI_PIN A2
-
 // This code does not fit into the really small flash of STM32F103x6 together
 // with CONSOLE_ENABLE=yes, and the debugging console is probably more
 // important for the "onekey" testing firmware.  In a real firmware you may be

keyboards/handwired/onekey/bluepill_f103c6/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A1"
+    },
+    "apa102": {
+        "data_pin": "A1",
+        "clock_pin": "A2"
     }
 }

keyboards/handwired/onekey/bluepill_uf2boot/config.h

@@ -21,5 +21,3 @@
 #define BACKLIGHT_PWM_CHANNEL 1
 
 #define ADC_PIN A0
-
-#define RGB_CI_PIN A2

keyboards/handwired/onekey/bluepill_uf2boot/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A1"
+    },
+    "apa102": {
+        "data_pin": "A1",
+        "clock_pin": "A2"
     }
 }

keyboards/handwired/onekey/elite_c/config.h

@@ -16,9 +16,6 @@
 
 #pragma once
 
-
-#define RGB_CI_PIN B1
-
 #define ADC_PIN F6
 
 #define QMK_WAITING_TEST_BUSY_PIN F6

keyboards/handwired/onekey/elite_c/info.json

@@ -10,5 +10,9 @@
     },
     "rgblight": {
         "pin": "F6"
+    },
+    "apa102": {
+        "data_pin": "F6",
+        "clock_pin": "B1"
     }
 }

keyboards/handwired/onekey/evb_wb32f3g71/config.h

@@ -11,7 +11,6 @@
 #define BACKLIGHT_PAL_MODE    2
 
 #define APA102_NOPS (100 / (1000000000L / (CPU_CLOCK / 4)))
-#define RGB_CI_PIN B8
 
 #define SOLENOID_PIN B12
 #define SOLENOID_PINS { B12, B13, B14, B15 }

keyboards/handwired/onekey/evb_wb32f3g71/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A0"
+    },
+    "apa102": {
+        "data_pin": "A0",
+        "clock_pin": "B8"
     }
 }

keyboards/handwired/onekey/evb_wb32fq95/config.h

@@ -11,7 +11,6 @@
 #define BACKLIGHT_PAL_MODE    2
 
 #define APA102_NOPS (100 / (1000000000L / (CPU_CLOCK / 4)))
-#define RGB_CI_PIN B8
 
 #define SOLENOID_PIN B12
 #define SOLENOID_PINS { B12, B13, B14, B15 }

keyboards/handwired/onekey/evb_wb32fq95/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A0"
+    },
+    "apa102": {
+        "data_pin": "A0",
+        "clock_pin": "B8"
     }
 }

keyboards/handwired/onekey/nucleo_f446re/config.h

@@ -7,8 +7,6 @@
 #define BACKLIGHT_PWM_CHANNEL 3
 #define BACKLIGHT_PAL_MODE    2
 
-#define RGB_CI_PIN B13
-
 #define ADC_PIN A0
 
 #define SOLENOID_PINS { B12, B13, B14, B15 }

keyboards/handwired/onekey/nucleo_f446re/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A0"
+    },
+    "apa102": {
+        "data_pin": "A0",
+        "clock_pin": "B13"
     }
 }

keyboards/handwired/onekey/nucleo_l432kc/config.h

@@ -7,6 +7,4 @@
 #define BACKLIGHT_PWM_CHANNEL 3
 #define BACKLIGHT_PAL_MODE    2
 
-#define RGB_CI_PIN B13
-
 #define ADC_PIN A0

keyboards/handwired/onekey/nucleo_l432kc/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "A0"
+    },
+    "apa102": {
+        "data_pin": "A0",
+        "clock_pin": "B13"
     }
 }

keyboards/handwired/onekey/promicro/config.h

@@ -16,9 +16,6 @@
 
 #pragma once
 
-
-#define RGB_CI_PIN B1
-
 #define ADC_PIN F6
 
 #define QMK_WAITING_TEST_BUSY_PIN F6

keyboards/handwired/onekey/promicro/info.json

@@ -10,5 +10,9 @@
     },
     "rgblight": {
         "pin": "F6"
+    },
+    "apa102": {
+        "data_pin": "F6",
+        "clock_pin": "B1"
     }
 }

keyboards/handwired/onekey/proton_c/config.h

@@ -21,6 +21,4 @@
 #define BACKLIGHT_PWM_CHANNEL 3
 #define BACKLIGHT_PAL_MODE    2
 
-#define RGB_CI_PIN B13
-
 #define ADC_PIN A0

keyboards/handwired/onekey/proton_c/info.json

@@ -10,5 +10,9 @@
     },
     "rgblight": {
         "pin": "A0"
+    },
+    "apa102": {
+        "data_pin": "A0",
+        "clock_pin": "B13"
     }
 }

keyboards/handwired/onekey/sipeed_longan_nano/config.h

@@ -20,8 +20,6 @@
 #define BACKLIGHT_PWM_DRIVER PWMD5 /* GD32 numbering scheme starts from 0, TIMER4 on GD32 boards is TIMER5 on STM32 boards. */
 #define BACKLIGHT_PWM_CHANNEL 2    /* GD32 numbering scheme starts from 0, Channel 1 on GD32 boards is Channel 2 on STM32 boards. */
 
-#define RGB_CI_PIN B13
-
 #define ADC_PIN A0
 
 #define I2C1_CLOCK_SPEED 1000000 /* GD32VF103 supports fast mode plus. */

keyboards/handwired/onekey/sipeed_longan_nano/info.json

@@ -12,5 +12,9 @@
     },
     "rgblight": {
         "pin": "A2"
+    },
+    "apa102": {
+        "data_pin": "A2",
+        "clock_pin": "B13"
     }
 }

keyboards/handwired/onekey/stm32f0_disco/config.h

@@ -22,5 +22,3 @@
 #define BACKLIGHT_PAL_MODE    0
 
 #define ADC_PIN A0
-
-#define RGB_CI_PIN B13

keyboards/handwired/onekey/stm32f0_disco/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "B15"
+    },
+    "apa102": {
+        "data_pin": "B15",
+        "clock_pin": "B13"
     }
 }

keyboards/handwired/onekey/teensy_2/config.h

@@ -19,7 +19,5 @@
 
 #define ADC_PIN F6
 
-#define RGB_CI_PIN F7
-
 #define QMK_WAITING_TEST_BUSY_PIN F6
 #define QMK_WAITING_TEST_YIELD_PIN F7

keyboards/handwired/onekey/teensy_2/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "F6"
+    },
+    "apa102": {
+        "data_pin": "F6",
+        "clock_pin": "F7"
     }
 }

keyboards/handwired/onekey/teensy_2pp/config.h

@@ -19,7 +19,5 @@
 
 #define ADC_PIN F6
 
-#define RGB_CI_PIN F7
-
 #define QMK_WAITING_TEST_BUSY_PIN F6
 #define QMK_WAITING_TEST_YIELD_PIN F7

keyboards/handwired/onekey/teensy_2pp/info.json

@@ -11,5 +11,9 @@
     },
     "rgblight": {
         "pin": "F6"
+    },
+    "apa102": {
+        "data_pin": "F6",
+        "clock_pin": "F7"
     }
 }

keyboards/hardwareabstraction/handwire/handwire.c

@@ -1,7 +1,8 @@
 // Copyright 2022 Andy Tsai (@atsai)
 // SPDX-License-Identifier: GPL-2.0-or-later
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
+
 static uint16_t buzzer_timer = 0;
 static uint8_t buzzer_dwell = 15;
 static uint8_t buzzer_dwell_change = 1;

keyboards/hotdox76v2/hotdox76v2.c

@@ -2,7 +2,7 @@
 // Copyright 2022 Drashna Jael're (@Drashna Jael're)
 // SPDX-License-Identifier: GPL-2.0-or-later
 
-#include QMK_KEYBOARD_H
+#include "hotdox76v2.h"
 #include <string.h>
 #include <transactions.h>
 #include "oled_font_lib/logo2.h"

keyboards/ingrained/matrix.c

@@ -23,7 +23,9 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
  * to repeating that information all over the place.
  */
 
-#include QMK_KEYBOARD_H
+#include "matrix.h"
+#include "debug.h"
+#include "wait.h"
 #include "i2c_master.h"
 
 extern i2c_status_t mcp23017_status;

keyboards/input_club/ergodox_infinity/ergodox_infinity.c

@@ -1,4 +1,4 @@
-#include QMK_KEYBOARD_H
+#include "ergodox_infinity.h"
 #include <ch.h>
 #include <hal.h>
 #include <string.h>

keyboards/kc60se/kc60se.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "kc60se.h"
 
 void matrix_init_kb(void){
     setPinOutput(B2);

keyboards/keychron/common/keychron_common.c

@@ -14,8 +14,8 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
 #include "keychron_common.h"
+#include "sync_timer.h"
 
 bool is_siri_active = false;
 uint32_t siri_timer = 0;

keyboards/keychron/common/keychron_common.h

@@ -16,10 +16,14 @@
 
 #pragma once
 
-#include "stdint.h"
+#include <stdint.h>
+#include <stdbool.h>
+#include "action.h"
+
 #ifdef VIA_ENABLE
 #    include "via.h"
 #endif
+
 #include "quantum_keycodes.h"
 
 enum custom_keycodes {

keyboards/lily58/lib/layer_state_reader.c

@@ -1,5 +1,4 @@
-
+#include "quantum.h"
-#include QMK_KEYBOARD_H
 #include <stdio.h>
 
 #define L_BASE 0

keyboards/lily58/lib/rgb_state_reader.c

@@ -1,6 +1,5 @@
 #ifdef RGBLIGHT_ENABLE
 
-#include QMK_KEYBOARD_H
 #include <stdio.h>
 
 extern rgblight_config_t rgblight_config;

keyboards/marksard/rhymestone/common/oled_helper.c

@@ -1,8 +1,10 @@
-#ifdef OLED_ENABLE
+#include "oled_helper.h"
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 #include <stdio.h>
 #include <string.h>
 
+#ifdef OLED_ENABLE
+
 void render_logo(void) {
 
   static const char PROGMEM logo_buf[]={

keyboards/marksard/rhymestone/common/oled_helper.h

@@ -1,3 +1,6 @@
+#include <stdint.h>
+#include "action.h"
+
 #ifdef OLED_ENABLE
 
 void render_logo(void);

keyboards/marksard/treadstone48/common/oled_helper.c

@@ -1,8 +1,10 @@
-#ifdef OLED_ENABLE
+#include "oled_helper.h"
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 #include <stdio.h>
 #include <string.h>
 
+#ifdef OLED_ENABLE
+
 void render_logo(void) {
 
   const char logo_buf[]={

keyboards/marksard/treadstone48/common/oled_helper.h

@@ -1,5 +1,8 @@
 #ifdef OLED_ENABLE
 
+#include <stdint.h>
+#include "action.h"
+
 void render_logo(void);
 void update_key_status(uint16_t keycode, keyrecord_t *record);
 void render_key_status(void);

keyboards/mk65/mk65.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 #ifdef ENCODER_ENABLE
 bool encoder_update_kb(uint8_t index, bool clockwise) {

keyboards/mxss/mxss.c

@@ -16,7 +16,7 @@
  * EEPROM management code from ../cannonkeys/stm32f072/keyboard.c
  */
 
-#include QMK_KEYBOARD_H
+#include "mxss.h"
 #include "eeprom.h"
 #include "action_layer.h"
 #include "rgblight.h"

keyboards/nack/nack.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "nack.h"
 
 #ifdef RGB_MATRIX_ENABLE
     led_config_t g_led_config = { {

keyboards/nullbitsco/nibble/nibble.c

@@ -13,7 +13,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
-#include QMK_KEYBOARD_H
+#include "nibble.h"
 
 // Use Bit-C LED to show CAPS LOCK status
 void led_update_ports(led_t led_state) {

keyboards/nullbitsco/tidbit/tidbit.c

@@ -14,7 +14,7 @@
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-#include QMK_KEYBOARD_H
+#include "tidbit.h"
 
 typedef struct PACKED {
     uint8_t r;

keyboards/omnikeyish/dynamic_macro.c

@@ -1,4 +1,4 @@
-#include QMK_KEYBOARD_H
+#include "omnikeyish.h"
 #include <string.h>
 
 dynamic_macro_t dynamic_macros[DYNAMIC_MACRO_COUNT];

keyboards/palette1202/lib/oled_helper.c

@@ -1,8 +1,10 @@
-#ifdef OLED_ENABLE
+#include "oled_helper.h"
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 #include <stdio.h>
 #include <string.h>
 
+#ifdef OLED_ENABLE
+
 // returns character cord of the logo by line number
 char *read_logo(int row) {
   static char logoLines[][18] = {

keyboards/ploopyco/mouse/mouse.c

@@ -16,7 +16,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "mouse.h"
 
 #ifndef OPT_DEBOUNCE
 #    define OPT_DEBOUNCE 5  // (ms) 			Time between scroll events

keyboards/rubi/lib/oled.c

@@ -15,8 +15,8 @@ You should have received a copy of the GNU General Public License
 along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-#include QMK_KEYBOARD_H
+#include "oled.h"
-#include "./lib/oled.h"
+#include "rubi.h"
 
 bool process_record_user_oled(uint16_t keycode, keyrecord_t *record) {
     return process_record_user(keycode, record);

keyboards/rubi/lib/oled.h

@@ -17,6 +17,10 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
 #pragma once
 
+#include <stdbool.h>
+#include <stdint.h>
+#include "progmem.h"
+
 #define OLED_FRAME_TIMEOUT (1000 / 30)  // 30 fps
 #define OLED_LOGO_TIMEOUT 3000          // 3 sec
 

keyboards/rura66/rev1/oled_display.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "rev1.h"
 
 // Defines names for use in layer keycodes and the keymap
 enum layer_names {

keyboards/splitkb/aurora/corne/rev1/rev1.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 #ifdef SWAP_HANDS_ENABLE
 // clang-format off

keyboards/splitkb/aurora/lily58/rev1/rev1.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 #ifdef SWAP_HANDS_ENABLE
 // clang-format off

keyboards/splitkb/aurora/sweep/rev1/rev1.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 #ifdef SWAP_HANDS_ENABLE
 // clang-format off

keyboards/teleport/native/ansi/ansi.c

@@ -14,7 +14,7 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#include QMK_KEYBOARD_H
+#include "quantum.h"
 
 #ifdef RGB_MATRIX_ENABLE
 led_config_t g_led_config = { {