[Keyboard] Align xd84 and xd96 with moon (#6465)

* Refactor xd84 to use LINK_TIME_OPTIMIZATION_ENABLE

* Refactor xd96 to use LINK_TIME_OPTIMIZATION_ENABLE

* Align xd84 and xd96 with moon

* Update keyboards/xd96/rules.mk

Co-Authored-By: Drashna Jaelre <drashna@live.com>
* Update keyboards/xd84/rules.mk

Co-Authored-By: Drashna Jaelre <drashna@live.com>

diff --git a/keyboards/xd84/config.h b/keyboards/xd84/config.h
index 138c2c4c853a5a05388d90d429a409c4c8bb78b2..1ff5e1cfd7cdddc45c2a69dc584eef692b4648ab 100644 (file)
--- a/keyboards/xd84/config.h
+++ b/keyboards/xd84/config.h
@@ -237,7 +237,3 @@
 /* Bootmagic Lite key configuration */
 // #define BOOTMAGIC_LITE_ROW 0
 // #define BOOTMAGIC_LITE_COLUMN 0
-
-// LTO options
-#define NO_ACTION_MACRO
-#define NO_ACTION_FUNCTION

diff --git a/keyboards/xd84/custom_matrix_helper.c b/keyboards/xd84/custom_matrix_helper.c
index a6a0a041d9da16fcb437a599771f2e3ba5053c24..a4c5b6afa54713438fad22d824befd9476f89d60 100644 (file)
--- a/keyboards/xd84/custom_matrix_helper.c
+++ b/keyboards/xd84/custom_matrix_helper.c
@@ -1,3 +1,18 @@
+/* Copyright 2019
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * 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 <stdint.h>
 #include <stdbool.h>
 #include "wait.h"

diff --git a/keyboards/xd84/matrix.c b/keyboards/xd84/matrix.c
index a3590504777aa64894acb661b4ee1eb0fb1c6142..4cb5544ca48d688bc5487b8a48c78b4a2aa18b51 100644 (file)
--- a/keyboards/xd84/matrix.c
+++ b/keyboards/xd84/matrix.c
@@ -65,7 +65,7 @@ static bool read_cols_on_row(matrix_row_t current_matrix[], uint8_t current_row)
   // Clear data in matrix row
   current_matrix[current_row] = 0;
 
-  // Select row and wait for row selecton to stabilize
+  // Select row and wait for row selection to stabilize
   select_row(current_row);
   wait_us(30);
 

diff --git a/keyboards/xd84/pca9555.c b/keyboards/xd84/pca9555.c
index df4631e9d74d4b47757960e7a11886e04636721e..b0e542d8deffcc8afd9b644d828e3d7464b3b9d4 100644 (file)
--- a/keyboards/xd84/pca9555.c
+++ b/keyboards/xd84/pca9555.c
@@ -1,9 +1,24 @@
+/* Copyright 2019
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * 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 "i2c_master.h"
 #include "pca9555.h"
 
 #include "debug.h"
 
-#define SLAVE_TO_ADDR(n)   (n<<1)
+#define SLAVE_TO_ADDR(n) (n << 1)
 #define TIMEOUT 100
 
 enum {
@@ -14,7 +29,7 @@ enum {
   CMD_INVERSION_0,
   CMD_INVERSION_1,
   CMD_CONFIG_0,
-  CMD_CONFIG_1
+  CMD_CONFIG_1,
 };
 
 void pca9555_init(uint8_t slave_addr) {
@@ -22,17 +37,17 @@ void pca9555_init(uint8_t slave_addr) {
   if (!s_init) {
     i2c_init();
 
-    s_init=1;
+    s_init = 1;
   }
 
   // TODO: could check device connected
-  //i2c_start(SLAVE_TO_ADDR(slave) | I2C_WRITE);
-  //i2c_stop();
+  // i2c_start(SLAVE_TO_ADDR(slave) | I2C_WRITE);
+  // i2c_stop();
 }
 
 void pca9555_set_config(uint8_t slave_addr, uint8_t port, uint8_t conf) {
   uint8_t addr = SLAVE_TO_ADDR(slave_addr);
-  uint8_t cmd = port ? CMD_OUTPUT_1 : CMD_OUTPUT_0;
+  uint8_t cmd  = port ? CMD_CONFIG_1 : CMD_CONFIG_0;
 
   i2c_status_t ret = i2c_writeReg(addr, cmd, &conf, sizeof(conf), TIMEOUT);
   if (ret != I2C_STATUS_SUCCESS) {
@@ -42,7 +57,7 @@ void pca9555_set_config(uint8_t slave_addr, uint8_t port, uint8_t conf) {
 
 void pca9555_set_output(uint8_t slave_addr, uint8_t port, uint8_t conf) {
   uint8_t addr = SLAVE_TO_ADDR(slave_addr);
-  uint8_t cmd = port ? CMD_CONFIG_1 : CMD_CONFIG_0;
+  uint8_t cmd  = port ? CMD_OUTPUT_1 : CMD_OUTPUT_0;
 
   i2c_status_t ret = i2c_writeReg(addr, cmd, &conf, sizeof(conf), TIMEOUT);
   if (ret != I2C_STATUS_SUCCESS) {
@@ -52,10 +67,10 @@ void pca9555_set_output(uint8_t slave_addr, uint8_t port, uint8_t conf) {
 
 uint8_t pca9555_readPins(uint8_t slave_addr, uint8_t port) {
   uint8_t addr = SLAVE_TO_ADDR(slave_addr);
-  uint8_t cmd = port ? CMD_INPUT_1 : CMD_INPUT_0;
+  uint8_t cmd  = port ? CMD_INPUT_1 : CMD_INPUT_0;
 
-  uint8_t data = 0;
-  i2c_status_t ret = i2c_readReg(addr, cmd, &data, sizeof(data), TIMEOUT);
+  uint8_t      data = 0;
+  i2c_status_t ret  = i2c_readReg(addr, cmd, &data, sizeof(data), TIMEOUT);
   if (ret != I2C_STATUS_SUCCESS) {
     print("pca9555_readPins::FAILED\n");
   }

diff --git a/keyboards/xd84/pca9555.h b/keyboards/xd84/pca9555.h
index 6aaee8a6f88aaf4f9f2b1afaaa7eb5585817cdcd..ebb97e2f301ac9f8a66cb7ba8cac9905570bb280 100644 (file)
--- a/keyboards/xd84/pca9555.h
+++ b/keyboards/xd84/pca9555.h
@@ -18,15 +18,15 @@
 /*
             PCA9555
          ,----------.
-   SDA --| SDA  P00 |-- P1
-   SCL --| SCL  P01 |-- P2
-   INT --| INT  P02 |-- P3
-         |      P03 |-- P4
-   A0  --| A0   P04 |-- P5
-   A1  --| A1   P05 |-- P6
-   A2  --| A2   P06 |-- P7
-         |      P07 |-- P8
-         |          |      
+   SDA --| SDA  P00 |-- P00
+   SCL --| SCL  P01 |-- P01
+   INT --| INT  P02 |-- P02
+         |      P03 |-- P03
+   A0  --| A0   P04 |-- P04
+   A1  --| A1   P05 |-- P05
+   A2  --| A2   P06 |-- P06
+         |      P07 |-- P07
+         |          |
          |      P10 |-- P10
          |      P11 |-- P11
          |      P12 |-- P12
@@ -35,7 +35,7 @@
          |      P15 |-- P15
          |      P16 |-- P16
          |      P17 |-- P17
-         `----------'      
+         `----------'
 */
 
 #define PCA9555_PORT0 0
@@ -52,4 +52,4 @@ void pca9555_set_config(uint8_t slave_addr, uint8_t port, uint8_t conf);
 
 void pca9555_set_output(uint8_t slave_addr, uint8_t port, uint8_t conf);
 
-uint8_t pca9555_readPins(uint8_t slave_addr, uint8_t port);
\ No newline at end of file
+uint8_t pca9555_readPins(uint8_t slave_addr, uint8_t port);

diff --git a/keyboards/xd84/readme.md b/keyboards/xd84/readme.md
index 38b4efcba29f0dd233cdc54b3dba166c3c44cc2b..b8d9c5dbb80722f77e8a1d5d41c0edb4a853fc1c 100644 (file)
--- a/keyboards/xd84/readme.md
+++ b/keyboards/xd84/readme.md
@@ -2,7 +2,7 @@
 
 ![XD84](https://cdn.shopify.com/s/files/1/2711/4238/products/HTB17eeJSXXXXXbIXFXXq6xXFXXXp_1024x1024.jpg?v=1515505994)
 
-Compact With 84 Keys & RGB LED Underglow
+Keyboard with 84 Keys & RGB LED Underglow
 - Designed by Xiudi
 - Up to 87 keys
   - iso and ansi support

diff --git a/keyboards/xd84/rules.mk b/keyboards/xd84/rules.mk
index 283d4babc7a76c2532e400a5598aa2bbba941d7b..e0982e0401c10663d7cd47c851d2c1219a19734d 100644 (file)
--- a/keyboards/xd84/rules.mk
+++ b/keyboards/xd84/rules.mk
@@ -78,11 +78,11 @@ BLUETOOTH_ENABLE = no       # Enable Bluetooth with the Adafruit EZ-Key HID
 AUDIO_ENABLE = no           # Audio output on port C6
 FAUXCLICKY_ENABLE = no      # Use buzzer to emulate clicky switches
 HD44780_ENABLE = no            # Enable support for HD44780 based LCDs (+400)
+LINK_TIME_OPTIMIZATION_ENABLE = yes
 
 # custom matrix setup
 CUSTOM_MATRIX = yes
-SRC = i2c_master.c custom_matrix_helper.c pca9555.c matrix.c
+SRC += custom_matrix_helper.c pca9555.c matrix.c
+QUANTUM_LIB_SRC += i2c_master.c
 
-EXTRAFLAGS += -flto
-
-LAYOUTS = 75_ansi 75_iso
\ No newline at end of file
+LAYOUTS = 75_ansi 75_iso

diff --git a/keyboards/xd96/config.h b/keyboards/xd96/config.h
index ea6c878e9e34de52cdb2930df4e8cbaf615c6676..b0fa47795231e862b42be8fd88ef15153d1de37a 100644 (file)
--- a/keyboards/xd96/config.h
+++ b/keyboards/xd96/config.h
@@ -238,7 +238,3 @@
 /* Bootmagic Lite key configuration */
 // #define BOOTMAGIC_LITE_ROW 0
 // #define BOOTMAGIC_LITE_COLUMN 0
-
-// LTO options
-#define NO_ACTION_MACRO
-#define NO_ACTION_FUNCTION

diff --git a/keyboards/xd96/matrix.c b/keyboards/xd96/matrix.c
index 550dae4d02e80cc73b09284d396cc1d37bdcc692..e8fd850edbfbf81f34150174e82cdfa3db96e3ca 100644 (file)
--- a/keyboards/xd96/matrix.c
+++ b/keyboards/xd96/matrix.c
@@ -67,7 +67,7 @@ static bool read_cols_on_row(matrix_row_t current_matrix[], uint8_t current_row)
   // Clear data in matrix row
   current_matrix[current_row] = 0;
 
-  // Select row and wait for row selecton to stabilize
+  // Select row and wait for row selection to stabilize
   select_row(current_row);
   wait_us(30);
 

diff --git a/keyboards/xd96/pca9555.c b/keyboards/xd96/pca9555.c
index 960df4f4e523a81d685281bd04769350d1e92b0e..b0e542d8deffcc8afd9b644d828e3d7464b3b9d4 100644 (file)
--- a/keyboards/xd96/pca9555.c
+++ b/keyboards/xd96/pca9555.c
@@ -18,7 +18,7 @@
 
 #include "debug.h"
 
-#define SLAVE_TO_ADDR(n)   (n<<1)
+#define SLAVE_TO_ADDR(n) (n << 1)
 #define TIMEOUT 100
 
 enum {
@@ -29,7 +29,7 @@ enum {
   CMD_INVERSION_0,
   CMD_INVERSION_1,
   CMD_CONFIG_0,
-  CMD_CONFIG_1
+  CMD_CONFIG_1,
 };
 
 void pca9555_init(uint8_t slave_addr) {
@@ -37,17 +37,17 @@ void pca9555_init(uint8_t slave_addr) {
   if (!s_init) {
     i2c_init();
 
-    s_init=1;
+    s_init = 1;
   }
 
   // TODO: could check device connected
-  //i2c_start(SLAVE_TO_ADDR(slave) | I2C_WRITE);
-  //i2c_stop();
+  // i2c_start(SLAVE_TO_ADDR(slave) | I2C_WRITE);
+  // i2c_stop();
 }
 
 void pca9555_set_config(uint8_t slave_addr, uint8_t port, uint8_t conf) {
   uint8_t addr = SLAVE_TO_ADDR(slave_addr);
-  uint8_t cmd = port ? CMD_OUTPUT_1 : CMD_OUTPUT_0;
+  uint8_t cmd  = port ? CMD_CONFIG_1 : CMD_CONFIG_0;
 
   i2c_status_t ret = i2c_writeReg(addr, cmd, &conf, sizeof(conf), TIMEOUT);
   if (ret != I2C_STATUS_SUCCESS) {
@@ -57,7 +57,7 @@ void pca9555_set_config(uint8_t slave_addr, uint8_t port, uint8_t conf) {
 
 void pca9555_set_output(uint8_t slave_addr, uint8_t port, uint8_t conf) {
   uint8_t addr = SLAVE_TO_ADDR(slave_addr);
-  uint8_t cmd = port ? CMD_CONFIG_1 : CMD_CONFIG_0;
+  uint8_t cmd  = port ? CMD_OUTPUT_1 : CMD_OUTPUT_0;
 
   i2c_status_t ret = i2c_writeReg(addr, cmd, &conf, sizeof(conf), TIMEOUT);
   if (ret != I2C_STATUS_SUCCESS) {
@@ -67,10 +67,10 @@ void pca9555_set_output(uint8_t slave_addr, uint8_t port, uint8_t conf) {
 
 uint8_t pca9555_readPins(uint8_t slave_addr, uint8_t port) {
   uint8_t addr = SLAVE_TO_ADDR(slave_addr);
-  uint8_t cmd = port ? CMD_INPUT_1 : CMD_INPUT_0;
+  uint8_t cmd  = port ? CMD_INPUT_1 : CMD_INPUT_0;
 
-  uint8_t data = 0;
-  i2c_status_t ret = i2c_readReg(addr, cmd, &data, sizeof(data), TIMEOUT);
+  uint8_t      data = 0;
+  i2c_status_t ret  = i2c_readReg(addr, cmd, &data, sizeof(data), TIMEOUT);
   if (ret != I2C_STATUS_SUCCESS) {
     print("pca9555_readPins::FAILED\n");
   }

diff --git a/keyboards/xd96/pca9555.h b/keyboards/xd96/pca9555.h
index 6aaee8a6f88aaf4f9f2b1afaaa7eb5585817cdcd..ebb97e2f301ac9f8a66cb7ba8cac9905570bb280 100644 (file)
--- a/keyboards/xd96/pca9555.h
+++ b/keyboards/xd96/pca9555.h
@@ -18,15 +18,15 @@
 /*
             PCA9555
          ,----------.
-   SDA --| SDA  P00 |-- P1
-   SCL --| SCL  P01 |-- P2
-   INT --| INT  P02 |-- P3
-         |      P03 |-- P4
-   A0  --| A0   P04 |-- P5
-   A1  --| A1   P05 |-- P6
-   A2  --| A2   P06 |-- P7
-         |      P07 |-- P8
-         |          |      
+   SDA --| SDA  P00 |-- P00
+   SCL --| SCL  P01 |-- P01
+   INT --| INT  P02 |-- P02
+         |      P03 |-- P03
+   A0  --| A0   P04 |-- P04
+   A1  --| A1   P05 |-- P05
+   A2  --| A2   P06 |-- P06
+         |      P07 |-- P07
+         |          |
          |      P10 |-- P10
          |      P11 |-- P11
          |      P12 |-- P12
@@ -35,7 +35,7 @@
          |      P15 |-- P15
          |      P16 |-- P16
          |      P17 |-- P17
-         `----------'      
+         `----------'
 */
 
 #define PCA9555_PORT0 0
@@ -52,4 +52,4 @@ void pca9555_set_config(uint8_t slave_addr, uint8_t port, uint8_t conf);
 
 void pca9555_set_output(uint8_t slave_addr, uint8_t port, uint8_t conf);
 
-uint8_t pca9555_readPins(uint8_t slave_addr, uint8_t port);
\ No newline at end of file
+uint8_t pca9555_readPins(uint8_t slave_addr, uint8_t port);

diff --git a/keyboards/xd96/readme.md b/keyboards/xd96/readme.md
index 237d7a05165fc94c46944e87d82d39ea209e2742..bf01db8bb7e4e2600f0477c94ed72f05e08b0e8a 100644 (file)
--- a/keyboards/xd96/readme.md
+++ b/keyboards/xd96/readme.md
@@ -2,7 +2,7 @@
 
 ![XD96](https://cdn.shopify.com/s/files/1/2711/4238/products/xd96_pcb_1024x1024.jpg?v=1515425370)
 
-Compact With 96 Keys & RGB LED Underglow
+Keyboard with 96 Keys & RGB LED Underglow
 - Designed by Xiudi
 - ISO and ANSI support
 - Uses Kimera core

diff --git a/keyboards/xd96/rules.mk b/keyboards/xd96/rules.mk
index c9f8044712914cd5b298dbf705418f7e016a4b50..91736bd8785968626ad6ddad4a506956e1187e70 100644 (file)
--- a/keyboards/xd96/rules.mk
+++ b/keyboards/xd96/rules.mk
@@ -78,9 +78,9 @@ BLUETOOTH_ENABLE = no       # Enable Bluetooth with the Adafruit EZ-Key HID
 AUDIO_ENABLE = no           # Audio output on port C6
 FAUXCLICKY_ENABLE = no      # Use buzzer to emulate clicky switches
 HD44780_ENABLE = no            # Enable support for HD44780 based LCDs (+400)
+LINK_TIME_OPTIMIZATION_ENABLE = yes
 
 # custom matrix setup
 CUSTOM_MATRIX = yes
-SRC = i2c_master.c custom_matrix_helper.c pca9555.c matrix.c
-
-EXTRAFLAGS += -flto
+SRC += custom_matrix_helper.c pca9555.c matrix.c
+QUANTUM_LIB_SRC += i2c_master.c