summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJack Humbert <jack.humb@gmail.com>2016-06-21 12:53:21 -0400
committerGitHub <noreply@github.com>2016-06-21 12:53:21 -0400
commit80c87054193b9243670aeb85adefbe1aa6c0fda0 (patch)
tree9ee304b9ca913328f1ea45500b34410f0d53ccfe
parenta8375fa15a6ca9285eb15ae89bcda898349e06f8 (diff)
downloadqmk_firmware-80c87054193b9243670aeb85adefbe1aa6c0fda0.tar.gz
qmk_firmware-80c87054193b9243670aeb85adefbe1aa6c0fda0.zip
reduces rgblight warnings, integrates completely (#428)
-rw-r--r--keyboard/satan/led.c2
-rw-r--r--keyboard/satan/satan.h2
-rw-r--r--quantum/rgblight.c8
-rw-r--r--quantum/rgblight.h1
-rw-r--r--tmk_core/common/avr/eeconfig.c5
-rw-r--r--tmk_core/common/eeconfig.h1
6 files changed, 11 insertions, 8 deletions
diff --git a/keyboard/satan/led.c b/keyboard/satan/led.c
index 5a9f2af672..94606e8959 100644
--- a/keyboard/satan/led.c
+++ b/keyboard/satan/led.c
@@ -20,7 +20,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
#include "led.h"
-void led_init_ports() {
+void led_init_ports(void) {
// * Set our LED pins as output
DDRB |= (1<<2);
}
diff --git a/keyboard/satan/satan.h b/keyboard/satan/satan.h
index 12dec26a1c..464289fdbd 100644
--- a/keyboard/satan/satan.h
+++ b/keyboard/satan/satan.h
@@ -4,7 +4,7 @@
#include "matrix.h"
#include "keymap.h"
#include <stddef.h>
-
+#include "action_util.h"
/* Clueboard matrix layout
* ,-----------------------------------------------------------.
diff --git a/quantum/rgblight.c b/quantum/rgblight.c
index 8c9ad77364..c29ffedc38 100644
--- a/quantum/rgblight.c
+++ b/quantum/rgblight.c
@@ -42,7 +42,7 @@ void sethsv(uint16_t hue, uint8_t sat, uint8_t val, struct cRGB *led1) {
The DIM_CURVE is used only on brightness/value and on saturation (inverted).
This looks the most natural.
*/
- uint8_t r, g, b;
+ uint8_t r = 0, g = 0, b = 0;
val = pgm_read_byte(&DIM_CURVE[val]);
sat = 255 - pgm_read_byte(&DIM_CURVE[255 - sat]);
@@ -154,7 +154,7 @@ void rgblight_init(void) {
}
void rgblight_increase(void) {
- uint8_t mode;
+ uint8_t mode = 0;
if (rgblight_config.mode < RGBLIGHT_MODES) {
mode = rgblight_config.mode + 1;
}
@@ -162,7 +162,7 @@ void rgblight_increase(void) {
}
void rgblight_decrease(void) {
- uint8_t mode;
+ uint8_t mode = 0;
if (rgblight_config.mode > 1) { //mode will never < 1, if mode is less than 1, eeprom need to be initialized.
mode = rgblight_config.mode-1;
}
@@ -170,7 +170,7 @@ void rgblight_decrease(void) {
}
void rgblight_step(void) {
- uint8_t mode;
+ uint8_t mode = 0;
mode = rgblight_config.mode + 1;
if (mode > RGBLIGHT_MODES) {
mode = 1;
diff --git a/quantum/rgblight.h b/quantum/rgblight.h
index 37e207578c..64f92523e0 100644
--- a/quantum/rgblight.h
+++ b/quantum/rgblight.h
@@ -64,7 +64,6 @@ void rgblight_decrease_val(void);
void rgblight_sethsv(uint16_t hue, uint8_t sat, uint8_t val);
void rgblight_setrgb(uint8_t r, uint8_t g, uint8_t b);
-#define EECONFIG_RGBLIGHT (uint8_t *)7
uint32_t eeconfig_read_rgblight(void);
void eeconfig_update_rgblight(uint32_t val);
void eeconfig_update_rgblight_default(void);
diff --git a/tmk_core/common/avr/eeconfig.c b/tmk_core/common/avr/eeconfig.c
index c5391f5cf5..656938fb33 100644
--- a/tmk_core/common/avr/eeconfig.c
+++ b/tmk_core/common/avr/eeconfig.c
@@ -14,7 +14,10 @@ void eeconfig_init(void)
eeprom_update_byte(EECONFIG_BACKLIGHT, 0);
#endif
#ifdef AUDIO_ENABLE
- eeprom_update_byte(EECONFIG_AUDIO, 0xFF); // On by default
+ eeprom_update_byte(EECONFIG_AUDIO, 0xFF); // On by default
+#endif
+#ifdef RGBLIGHT_ENABLE
+ eeprom_update_dword(EECONFIG_RGBLIGHT, 0);
#endif
}
diff --git a/tmk_core/common/eeconfig.h b/tmk_core/common/eeconfig.h
index ca47e0d2fd..d8caa346f9 100644
--- a/tmk_core/common/eeconfig.h
+++ b/tmk_core/common/eeconfig.h
@@ -32,6 +32,7 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
#define EECONFIG_MOUSEKEY_ACCEL (uint8_t *)5
#define EECONFIG_BACKLIGHT (uint8_t *)6
#define EECONFIG_AUDIO (uint8_t *)7
+#define EECONFIG_RGBLIGHT (uint32_t *)8
/* debug bit */