summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authoryiancar <yiangosyiangou@cytanet.com.cy>2020-01-11 17:25:27 +0000
committerJames Young <18669334+noroadsleft@users.noreply.github.com>2020-01-11 09:25:27 -0800
commit70309bef3d47916f1b988aacc2fac598bd1f9d22 (patch)
tree5496316f69ce9e3e8e90cc7fd0bcdfbdd82d616e
parent2d051d8de3787de889ff0f0301153dbb4e2e1272 (diff)
downloadqmk_firmware-70309bef3d47916f1b988aacc2fac598bd1f9d22.tar.gz
qmk_firmware-70309bef3d47916f1b988aacc2fac598bd1f9d22.zip
[Keyboard] NK65 Refactor (#7862)
* Nk65 Refractor - Changed Via keymap to via instead of default_via - Moved Caps/Layer indicator code outside of wilba_tech rgb. This is so they can more easily be modified * Update keyboards/nk65/nk65.c
-rwxr-xr-xkeyboards/nk65/keymaps/via/keymap.c (renamed from keyboards/nk65/keymaps/default_via/keymap.c)0
-rwxr-xr-xkeyboards/nk65/keymaps/via/readme.md (renamed from keyboards/nk65/keymaps/default_via/readme.md)0
-rwxr-xr-xkeyboards/nk65/keymaps/via/rules.mk (renamed from keyboards/nk65/keymaps/default_via/rules.mk)0
-rwxr-xr-xkeyboards/nk65/nk65.c42
-rw-r--r--keyboards/wilba_tech/wt_rgb_backlight.c16
5 files changed, 39 insertions, 19 deletions
diff --git a/keyboards/nk65/keymaps/default_via/keymap.c b/keyboards/nk65/keymaps/via/keymap.c
index a793a8bf41..a793a8bf41 100755
--- a/keyboards/nk65/keymaps/default_via/keymap.c
+++ b/keyboards/nk65/keymaps/via/keymap.c
diff --git a/keyboards/nk65/keymaps/default_via/readme.md b/keyboards/nk65/keymaps/via/readme.md
index 6689191e4c..6689191e4c 100755
--- a/keyboards/nk65/keymaps/default_via/readme.md
+++ b/keyboards/nk65/keymaps/via/readme.md
diff --git a/keyboards/nk65/keymaps/default_via/rules.mk b/keyboards/nk65/keymaps/via/rules.mk
index 1e5b99807c..1e5b99807c 100755
--- a/keyboards/nk65/keymaps/default_via/rules.mk
+++ b/keyboards/nk65/keymaps/via/rules.mk
diff --git a/keyboards/nk65/nk65.c b/keyboards/nk65/nk65.c
index 4952462183..972373e1b2 100755
--- a/keyboards/nk65/nk65.c
+++ b/keyboards/nk65/nk65.c
@@ -13,6 +13,42 @@
* You should have received a copy of the GNU General Public License
* along with this program. If not, see <http://www.gnu.org/licenses/>.
*/
- #ifndef RGB_BACKLIGHT_NK65
- #error RGB_BACKLIGHT_NK65 not defined, recheck config.h
- #endif
+#ifndef RGB_BACKLIGHT_NK65
+#error RGB_BACKLIGHT_NK65 not defined, recheck config.h
+#endif
+
+#include "nk65.h"
+#include "drivers/issi/is31fl3733.h"
+
+/* Indicator LEDS are part of the LED driver
+ * Top LED is blue only. LED driver 2 RGB 7 Green channel
+ * Middle LED is blue and red. LED driver 2 RGB 6 Red and Blue channel
+ * Bottom LED is red only LED driver 2 RGB 6 Green channel.
+ */
+bool led_update_kb(led_t led_state) {
+ bool res = led_update_user(led_state);
+ if(res) {
+ if (led_state.caps_lock) {
+ IS31FL3733_set_color( 7+64-1, 0, 255, 0 );
+ } else {
+ IS31FL3733_set_color( 7+64-1, 0, 0, 0 );
+ }
+ }
+ return res;
+}
+
+__attribute__((weak)) layer_state_t layer_state_set_user(layer_state_t state) {
+ uint8_t R = 0;
+ uint8_t G = 0;
+ uint8_t B = 0;
+ if (state & (1UL << 1)) {
+ R = 255;
+ B = 255;
+ }
+ if (state & (1UL << 2)) {
+ G = 255;
+ }
+
+ IS31FL3733_set_color( 6+64-1, R, G, B );
+ return state;
+}
diff --git a/keyboards/wilba_tech/wt_rgb_backlight.c b/keyboards/wilba_tech/wt_rgb_backlight.c
index 31cfabddd7..1ca3117fb4 100644
--- a/keyboards/wilba_tech/wt_rgb_backlight.c
+++ b/keyboards/wilba_tech/wt_rgb_backlight.c
@@ -1551,22 +1551,6 @@ void backlight_effect_indicators(void)
{
backlight_effect_indicators_set_colors( g_config.caps_lock_indicator.index, g_config.caps_lock_indicator.color );
}
-
-#if defined(RGB_BACKLIGHT_NK65)
- if ( IS_LED_ON(g_indicator_state, USB_LED_CAPS_LOCK) ) {
- IS31FL3733_set_color( 7+64-1, 0, 255, 0 );
- } else {
- IS31FL3733_set_color( 7+64-1, 0, 0, 0 );
- }
- if ( IS_LAYER_ON(1) ) {
- IS31FL3733_set_color( 6+64-1, 255, 0, 255 );
- } else if ( IS_LAYER_ON(2) ) {
- IS31FL3733_set_color( 6+64-1, 0, 255, 0 );
- } else {
- IS31FL3733_set_color( 6+64-1, 0, 0, 0 );
- }
-#endif
-
// This if/else if structure allows higher layers to
// override lower ones. If we set layer 3's indicator
// to none, then it will NOT show layer 2 or layer 1