From 8136cf4bfb2664a0a492f7346132afc78110cce0 Mon Sep 17 00:00:00 2001 From: Ryan Date: Fri, 10 Nov 2023 08:39:15 +1100 Subject: [PATCH] LED drivers: clean up `SWx`/`CSy` pullup/down resistor config (#22381) --- drivers/led/issi/is31fl3733-simple.c | 16 +++---- drivers/led/issi/is31fl3733-simple.h | 42 ++++++++++------- drivers/led/issi/is31fl3733.c | 16 +++---- drivers/led/issi/is31fl3733.h | 42 ++++++++++------- drivers/led/issi/is31fl3736-simple.c | 16 +++---- drivers/led/issi/is31fl3736-simple.h | 45 +++++++++++-------- drivers/led/issi/is31fl3736.c | 16 +++---- drivers/led/issi/is31fl3736.h | 45 +++++++++++-------- drivers/led/issi/is31fl3737-simple.c | 16 +++---- drivers/led/issi/is31fl3737-simple.h | 45 +++++++++++-------- drivers/led/issi/is31fl3737.c | 16 +++---- drivers/led/issi/is31fl3737.h | 45 +++++++++++-------- drivers/led/issi/is31fl3741-simple.c | 10 ++--- drivers/led/issi/is31fl3741-simple.h | 45 +++++++++++-------- drivers/led/issi/is31fl3741.c | 10 ++--- drivers/led/issi/is31fl3741.h | 45 +++++++++++-------- keyboards/input_club/k_type/is31fl3733-dual.c | 16 +++---- keyboards/input_club/k_type/is31fl3733-dual.h | 34 +++++++++----- 18 files changed, 302 insertions(+), 218 deletions(-) diff --git a/drivers/led/issi/is31fl3733-simple.c b/drivers/led/issi/is31fl3733-simple.c index 966a8d7cf2985ea73eed6875e4466a479159e0c3..63e9f533e2e7f8f2e874a30c8401b74694eb0fe4 100644 --- a/drivers/led/issi/is31fl3733-simple.c +++ b/drivers/led/issi/is31fl3733-simple.c @@ -36,8 +36,8 @@ #define IS31FL3733_REG_CONFIGURATION 0x00 // PG3 #define IS31FL3733_REG_GLOBALCURRENT 0x01 // PG3 #define IS31FL3733_REG_RESET 0x11 // PG3 -#define IS31FL3733_REG_SWPULLUP 0x0F // PG3 -#define IS31FL3733_REG_CSPULLUP 0x10 // PG3 +#define IS31FL3733_REG_SW_PULLUP 0x0F // PG3 +#define IS31FL3733_REG_CS_PULLDOWN 0x10 // PG3 #define IS31FL3733_PWM_REGISTER_COUNT 192 #define IS31FL3733_LED_CONTROL_REGISTER_COUNT 24 @@ -54,12 +54,12 @@ # define IS31FL3733_PWM_FREQUENCY IS31FL3733_PWM_FREQUENCY_8K4_HZ // PFS - IS31FL3733B only #endif -#ifndef IS31FL3733_SWPULLUP -# define IS31FL3733_SWPULLUP IS31FL3733_PUR_0R +#ifndef IS31FL3733_SW_PULLUP +# define IS31FL3733_SW_PULLUP IS31FL3733_PUR_0_OHM #endif -#ifndef IS31FL3733_CSPULLUP -# define IS31FL3733_CSPULLUP IS31FL3733_PUR_0R +#ifndef IS31FL3733_CS_PULLDOWN +# define IS31FL3733_CSPULLDOWN IS31FL3733_PDR_0_OHM #endif #ifndef IS31FL3733_GLOBALCURRENT @@ -206,9 +206,9 @@ void is31fl3733_init(uint8_t addr, uint8_t sync) { // Select PG3 is31fl3733_write_register(addr, IS31FL3733_COMMANDREGISTER, IS31FL3733_PAGE_FUNCTION); // Set de-ghost pull-up resistors (SWx) - is31fl3733_write_register(addr, IS31FL3733_REG_SWPULLUP, IS31FL3733_SWPULLUP); + is31fl3733_write_register(addr, IS31FL3733_REG_SW_PULLUP, IS31FL3733_SW_PULLUP); // Set de-ghost pull-down resistors (CSx) - is31fl3733_write_register(addr, IS31FL3733_REG_CSPULLUP, IS31FL3733_CSPULLUP); + is31fl3733_write_register(addr, IS31FL3733_REG_CS_PULLDOWN, IS31FL3733_CS_PULLDOWN); // Set global current to maximum. is31fl3733_write_register(addr, IS31FL3733_REG_GLOBALCURRENT, IS31FL3733_GLOBALCURRENT); // Disable software shutdown. diff --git a/drivers/led/issi/is31fl3733-simple.h b/drivers/led/issi/is31fl3733-simple.h index 65d3298a50e416c2b8497fdba8d39139b7ff6a9e..2c6014455f07f1ba8980b66a691d159238a2f067 100644 --- a/drivers/led/issi/is31fl3733-simple.h +++ b/drivers/led/issi/is31fl3733-simple.h @@ -39,10 +39,10 @@ # define IS31FL3733_PWM_FREQUENCY ISSI_PWM_FREQUENCY #endif #ifdef ISSI_SWPULLUP -# define IS31FL3733_SWPULLUP ISSI_SWPULLUP +# define IS31FL3733_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3733_CSPULLUP ISSI_CSPULLUP +# define IS31FL3733_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3733_GLOBALCURRENT ISSI_GLOBALCURRENT @@ -51,13 +51,13 @@ #define is31_led is31fl3733_led_t #define g_is31_leds g_is31fl3733_leds -#define PUR_0R IS31FL3733_PUR_0R -#define PUR_05KR IS31FL3733_PUR_05KR -#define PUR_3KR IS31FL3733_PUR_3KR -#define PUR_4KR IS31FL3733_PUR_4KR -#define PUR_8KR IS31FL3733_PUR_8KR -#define PUR_16KR IS31FL3733_PUR_16KR -#define PUR_32KR IS31FL3733_PUR_32KR +#define PUR_0R IS31FL3733_PUR_0_OHM +#define PUR_05KR IS31FL3733_PUR_1K_OHM +#define PUR_3KR IS31FL3733_PUR_2K_OHM +#define PUR_4KR IS31FL3733_PUR_4K_OHM +#define PUR_8KR IS31FL3733_PUR_8K_OHM +#define PUR_16KR IS31FL3733_PUR_16K_OHM +#define PUR_32KR IS31FL3733_PUR_32K_OHM // ======== #define IS31FL3733_I2C_ADDRESS_GND_GND 0x50 @@ -107,13 +107,23 @@ void is31fl3733_update_led_control_registers(uint8_t addr, uint8_t index); void is31fl3733_flush(void); -#define IS31FL3733_PUR_0R 0x00 // No PUR resistor -#define IS31FL3733_PUR_05KR 0x02 // 0.5k Ohm resistor in t_NOL -#define IS31FL3733_PUR_3KR 0x03 // 3.0k Ohm resistor on all the time -#define IS31FL3733_PUR_4KR 0x04 // 4.0k Ohm resistor on all the time -#define IS31FL3733_PUR_8KR 0x05 // 8.0k Ohm resistor on all the time -#define IS31FL3733_PUR_16KR 0x06 // 16k Ohm resistor on all the time -#define IS31FL3733_PUR_32KR 0x07 // 32k Ohm resistor in t_NOL +#define IS31FL3733_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3733_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3733_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3733_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3733_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3733_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3733_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3733_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3733_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3733_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3733_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3733_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3733_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3733_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3733_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3733_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3733_PWM_FREQUENCY_8K4_HZ 0b000 #define IS31FL3733_PWM_FREQUENCY_4K2_HZ 0b001 diff --git a/drivers/led/issi/is31fl3733.c b/drivers/led/issi/is31fl3733.c index 729d2bc57ced0bf5a14c6d3e1d627ce51d61d32e..5a5f352a6ab52ee87ebed61d3299aadcc1a7d2bc 100644 --- a/drivers/led/issi/is31fl3733.c +++ b/drivers/led/issi/is31fl3733.c @@ -35,8 +35,8 @@ #define IS31FL3733_REG_CONFIGURATION 0x00 // PG3 #define IS31FL3733_REG_GLOBALCURRENT 0x01 // PG3 #define IS31FL3733_REG_RESET 0x11 // PG3 -#define IS31FL3733_REG_SWPULLUP 0x0F // PG3 -#define IS31FL3733_REG_CSPULLUP 0x10 // PG3 +#define IS31FL3733_REG_SW_PULLUP 0x0F // PG3 +#define IS31FL3733_REG_CS_PULLDOWN 0x10 // PG3 #define IS31FL3733_PWM_REGISTER_COUNT 192 #define IS31FL3733_LED_CONTROL_REGISTER_COUNT 24 @@ -53,12 +53,12 @@ # define IS31FL3733_PWM_FREQUENCY IS31FL3733_PWM_FREQUENCY_8K4_HZ // PFS - IS31FL3733B only #endif -#ifndef IS31FL3733_SWPULLUP -# define IS31FL3733_SWPULLUP IS31FL3733_PUR_0R +#ifndef IS31FL3733_SW_PULLUP +# define IS31FL3733_SW_PULLUP IS31FL3733_PUR_0_OHM #endif -#ifndef IS31FL3733_CSPULLUP -# define IS31FL3733_CSPULLUP IS31FL3733_PUR_0R +#ifndef IS31FL3733_CS_PULLDOWN +# define IS31FL3733_CS_PULLDOWN IS31FL3733_PDR_0_OHM #endif #ifndef IS31FL3733_GLOBALCURRENT @@ -205,9 +205,9 @@ void is31fl3733_init(uint8_t addr, uint8_t sync) { // Select PG3 is31fl3733_write_register(addr, IS31FL3733_COMMANDREGISTER, IS31FL3733_PAGE_FUNCTION); // Set de-ghost pull-up resistors (SWx) - is31fl3733_write_register(addr, IS31FL3733_REG_SWPULLUP, IS31FL3733_SWPULLUP); + is31fl3733_write_register(addr, IS31FL3733_REG_SW_PULLUP, IS31FL3733_SW_PULLUP); // Set de-ghost pull-down resistors (CSx) - is31fl3733_write_register(addr, IS31FL3733_REG_CSPULLUP, IS31FL3733_CSPULLUP); + is31fl3733_write_register(addr, IS31FL3733_REG_CS_PULLDOWN, IS31FL3733_CS_PULLDOWN); // Set global current to maximum. is31fl3733_write_register(addr, IS31FL3733_REG_GLOBALCURRENT, IS31FL3733_GLOBALCURRENT); // Disable software shutdown. diff --git a/drivers/led/issi/is31fl3733.h b/drivers/led/issi/is31fl3733.h index a8407f21548f4fc666dde3c031325bc9d57a65d4..a87d790c44622798976953800a1ff8f2df10e3c5 100644 --- a/drivers/led/issi/is31fl3733.h +++ b/drivers/led/issi/is31fl3733.h @@ -62,10 +62,10 @@ # define IS31FL3733_PWM_FREQUENCY ISSI_PWM_FREQUENCY #endif #ifdef ISSI_SWPULLUP -# define IS31FL3733_SWPULLUP ISSI_SWPULLUP +# define IS31FL3733_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3733_CSPULLUP ISSI_CSPULLUP +# define IS31FL3733_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3733_GLOBALCURRENT ISSI_GLOBALCURRENT @@ -74,13 +74,13 @@ #define is31_led is31fl3733_led_t #define g_is31_leds g_is31fl3733_leds -#define PUR_0R IS31FL3733_PUR_0R -#define PUR_05KR IS31FL3733_PUR_05KR -#define PUR_3KR IS31FL3733_PUR_3KR -#define PUR_4KR IS31FL3733_PUR_4KR -#define PUR_8KR IS31FL3733_PUR_8KR -#define PUR_16KR IS31FL3733_PUR_16KR -#define PUR_32KR IS31FL3733_PUR_32KR +#define PUR_0R IS31FL3733_PUR_0_OHM +#define PUR_05KR IS31FL3733_PUR_1K_OHM +#define PUR_3KR IS31FL3733_PUR_2K_OHM +#define PUR_4KR IS31FL3733_PUR_4K_OHM +#define PUR_8KR IS31FL3733_PUR_8K_OHM +#define PUR_16KR IS31FL3733_PUR_16K_OHM +#define PUR_32KR IS31FL3733_PUR_32K_OHM // ======== #define IS31FL3733_I2C_ADDRESS_GND_GND 0x50 @@ -132,13 +132,23 @@ void is31fl3733_update_led_control_registers(uint8_t addr, uint8_t index); void is31fl3733_flush(void); -#define IS31FL3733_PUR_0R 0x00 // No PUR resistor -#define IS31FL3733_PUR_05KR 0x02 // 0.5k Ohm resistor in t_NOL -#define IS31FL3733_PUR_3KR 0x03 // 3.0k Ohm resistor on all the time -#define IS31FL3733_PUR_4KR 0x04 // 4.0k Ohm resistor on all the time -#define IS31FL3733_PUR_8KR 0x05 // 8.0k Ohm resistor on all the time -#define IS31FL3733_PUR_16KR 0x06 // 16k Ohm resistor on all the time -#define IS31FL3733_PUR_32KR 0x07 // 32k Ohm resistor in t_NOL +#define IS31FL3733_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3733_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3733_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3733_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3733_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3733_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3733_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3733_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3733_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3733_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3733_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3733_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3733_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3733_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3733_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3733_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3733_PWM_FREQUENCY_8K4_HZ 0b000 #define IS31FL3733_PWM_FREQUENCY_4K2_HZ 0b001 diff --git a/drivers/led/issi/is31fl3736-simple.c b/drivers/led/issi/is31fl3736-simple.c index 45daaf31875773a73ad8456f789eb8f0497a81a8..e0d76d07c6dcdd583e37952ebaccd91dd646951c 100644 --- a/drivers/led/issi/is31fl3736-simple.c +++ b/drivers/led/issi/is31fl3736-simple.c @@ -33,8 +33,8 @@ #define IS31FL3736_REG_CONFIGURATION 0x00 // PG3 #define IS31FL3736_REG_GLOBALCURRENT 0x01 // PG3 #define IS31FL3736_REG_RESET 0x11 // PG3 -#define IS31FL3736_REG_SWPULLUP 0x0F // PG3 -#define IS31FL3736_REG_CSPULLUP 0x10 // PG3 +#define IS31FL3736_REG_SW_PULLUP 0x0F // PG3 +#define IS31FL3736_REG_CS_PULLDOWN 0x10 // PG3 #define IS31FL3736_PWM_REGISTER_COUNT 192 // actually 96 #define IS31FL3736_LED_CONTROL_REGISTER_COUNT 24 @@ -51,12 +51,12 @@ # define IS31FL3736_PWM_FREQUENCY IS31FL3736_PWM_FREQUENCY_8K4_HZ // PFS - IS31FL3736B only #endif -#ifndef IS31FL3736_SWPULLUP -# define IS31FL3736_SWPULLUP IS31FL3736_PUR_0R +#ifndef IS31FL3736_SW_PULLUP +# define IS31FL3736_SW_PULLUP IS31FL3736_PUR_0_OHM #endif -#ifndef IS31FL3736_CSPULLUP -# define IS31FL3736_CSPULLUP IS31FL3736_PUR_0R +#ifndef IS31FL3736_CS_PULLDOWN +# define IS31FL3736_CS_PULLDOWN IS31FL3736_PDR_0_OHM #endif #ifndef IS31FL3736_GLOBALCURRENT @@ -178,9 +178,9 @@ void is31fl3736_init(uint8_t addr) { // Select PG3 is31fl3736_write_register(addr, IS31FL3736_COMMANDREGISTER, IS31FL3736_PAGE_FUNCTION); // Set de-ghost pull-up resistors (SWx) - is31fl3736_write_register(addr, IS31FL3736_REG_SWPULLUP, IS31FL3736_SWPULLUP); + is31fl3736_write_register(addr, IS31FL3736_REG_SW_PULLUP, IS31FL3736_SW_PULLUP); // Set de-ghost pull-down resistors (CSx) - is31fl3736_write_register(addr, IS31FL3736_REG_CSPULLUP, IS31FL3736_CSPULLUP); + is31fl3736_write_register(addr, IS31FL3736_REG_CS_PULLDOWN, IS31FL3736_CS_PULLDOWN); // Set global current to maximum. is31fl3736_write_register(addr, IS31FL3736_REG_GLOBALCURRENT, IS31FL3736_GLOBALCURRENT); // Disable software shutdown. diff --git a/drivers/led/issi/is31fl3736-simple.h b/drivers/led/issi/is31fl3736-simple.h index cb751a917ef0f563c27167612d53b0ac1d8f6ec5..7e3e728641fff06c6d6e6ddfc95a608b1804c449 100644 --- a/drivers/led/issi/is31fl3736-simple.h +++ b/drivers/led/issi/is31fl3736-simple.h @@ -33,10 +33,10 @@ # define IS31FL3736_I2C_PERSISTENCE ISSI_PERSISTENCE #endif #ifdef ISSI_SWPULLUP -# define IS31FL3736_SWPULLUP ISSI_SWPULLUP +# define IS31FL3736_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3736_CSPULLUP ISSI_CSPULLUP +# define IS31FL3736_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3736_GLOBALCURRENT ISSI_GLOBALCURRENT @@ -45,14 +45,14 @@ #define is31_led is31fl3736_led_t #define g_is31_leds g_is31fl3736_leds -#define PUR_0R IS31FL3736_PUR_0R -#define PUR_05KR IS31FL3736_PUR_05KR -#define PUR_1KR IS31FL3736_PUR_1KR -#define PUR_2KR IS31FL3736_PUR_2KR -#define PUR_4KR IS31FL3736_PUR_4KR -#define PUR_8KR IS31FL3736_PUR_8KR -#define PUR_16KR IS31FL3736_PUR_16KR -#define PUR_32KR IS31FL3736_PUR_32KR +#define PUR_0R IS31FL3736_PUR_0_OHM +#define PUR_05KR IS31FL3736_PUR_05K_OHM +#define PUR_1KR IS31FL3736_PUR_1K_OHM +#define PUR_2KR IS31FL3736_PUR_2K_OHM +#define PUR_4KR IS31FL3736_PUR_4K_OHM +#define PUR_8KR IS31FL3736_PUR_8K_OHM +#define PUR_16KR IS31FL3736_PUR_16K_OHM +#define PUR_32KR IS31FL3736_PUR_32K_OHM // ======== #define IS31FL3736_I2C_ADDRESS_GND_GND 0x50 @@ -102,14 +102,23 @@ void is31fl3736_update_led_control_registers(uint8_t addr, uint8_t index); void is31fl3736_flush(void); -#define IS31FL3736_PUR_0R 0x00 // No PUR resistor -#define IS31FL3736_PUR_05KR 0x01 // 0.5k Ohm resistor -#define IS31FL3736_PUR_1KR 0x02 // 1.0k Ohm resistor -#define IS31FL3736_PUR_2KR 0x03 // 2.0k Ohm resistor -#define IS31FL3736_PUR_4KR 0x04 // 4.0k Ohm resistor -#define IS31FL3736_PUR_8KR 0x05 // 8.0k Ohm resistor -#define IS31FL3736_PUR_16KR 0x06 // 16k Ohm resistor -#define IS31FL3736_PUR_32KR 0x07 // 32k Ohm resistor +#define IS31FL3736_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3736_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3736_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3736_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3736_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3736_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3736_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3736_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3736_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3736_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3736_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3736_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3736_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3736_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3736_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3736_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3736_PWM_FREQUENCY_8K4_HZ 0b000 #define IS31FL3736_PWM_FREQUENCY_4K2_HZ 0b001 diff --git a/drivers/led/issi/is31fl3736.c b/drivers/led/issi/is31fl3736.c index 36cd132d468e4174e4caf6e4d669124e5524f005..0e3205f19b1615ef6847cbe0dd07db59b5c86115 100644 --- a/drivers/led/issi/is31fl3736.c +++ b/drivers/led/issi/is31fl3736.c @@ -33,8 +33,8 @@ #define IS31FL3736_REG_CONFIGURATION 0x00 // PG3 #define IS31FL3736_REG_GLOBALCURRENT 0x01 // PG3 #define IS31FL3736_REG_RESET 0x11 // PG3 -#define IS31FL3736_REG_SWPULLUP 0x0F // PG3 -#define IS31FL3736_REG_CSPULLUP 0x10 // PG3 +#define IS31FL3736_REG_SW_PULLUP 0x0F // PG3 +#define IS31FL3736_REG_CS_PULLDOWN 0x10 // PG3 #define IS31FL3736_PWM_REGISTER_COUNT 192 // actually 96 #define IS31FL3736_LED_CONTROL_REGISTER_COUNT 24 @@ -51,12 +51,12 @@ # define IS31FL3736_PWM_FREQUENCY IS31FL3736_PWM_FREQUENCY_8K4_HZ // PFS - IS31FL3736B only #endif -#ifndef IS31FL3736_SWPULLUP -# define IS31FL3736_SWPULLUP IS31FL3736_PUR_0R +#ifndef IS31FL3736_SW_PULLUP +# define IS31FL3736_SW_PULLUP IS31FL3736_PUR_0_OHM #endif -#ifndef IS31FL3736_CSPULLUP -# define IS31FL3736_CSPULLUP IS31FL3736_PUR_0R +#ifndef IS31FL3736_CS_PULLDOWN +# define IS31FL3736_CS_PULLDOWN IS31FL3736_PDR_0_OHM #endif #ifndef IS31FL3736_GLOBALCURRENT @@ -178,9 +178,9 @@ void is31fl3736_init(uint8_t addr) { // Select PG3 is31fl3736_write_register(addr, IS31FL3736_COMMANDREGISTER, IS31FL3736_PAGE_FUNCTION); // Set de-ghost pull-up resistors (SWx) - is31fl3736_write_register(addr, IS31FL3736_REG_SWPULLUP, IS31FL3736_SWPULLUP); + is31fl3736_write_register(addr, IS31FL3736_REG_SW_PULLUP, IS31FL3736_SW_PULLUP); // Set de-ghost pull-down resistors (CSx) - is31fl3736_write_register(addr, IS31FL3736_REG_CSPULLUP, IS31FL3736_CSPULLUP); + is31fl3736_write_register(addr, IS31FL3736_REG_CS_PULLDOWN, IS31FL3736_CS_PULLDOWN); // Set global current to maximum. is31fl3736_write_register(addr, IS31FL3736_REG_GLOBALCURRENT, IS31FL3736_GLOBALCURRENT); // Disable software shutdown. diff --git a/drivers/led/issi/is31fl3736.h b/drivers/led/issi/is31fl3736.h index 79d8c3276dfc2197f5313fd0581b9f0774c8aee0..bdb787494df160d741196f2f2a187588c2e14f21 100644 --- a/drivers/led/issi/is31fl3736.h +++ b/drivers/led/issi/is31fl3736.h @@ -45,10 +45,10 @@ # define IS31FL3736_I2C_PERSISTENCE ISSI_PERSISTENCE #endif #ifdef ISSI_SWPULLUP -# define IS31FL3736_SWPULLUP ISSI_SWPULLUP +# define IS31FL3736_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3736_CSPULLUP ISSI_CSPULLUP +# define IS31FL3736_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3736_GLOBALCURRENT ISSI_GLOBALCURRENT @@ -57,14 +57,14 @@ #define is31_led is31fl3736_led_t #define g_is31_leds g_is31fl3736_leds -#define PUR_0R IS31FL3736_PUR_0R -#define PUR_05KR IS31FL3736_PUR_05KR -#define PUR_1KR IS31FL3736_PUR_1KR -#define PUR_2KR IS31FL3736_PUR_2KR -#define PUR_4KR IS31FL3736_PUR_4KR -#define PUR_8KR IS31FL3736_PUR_8KR -#define PUR_16KR IS31FL3736_PUR_16KR -#define PUR_32KR IS31FL3736_PUR_32KR +#define PUR_0R IS31FL3736_PUR_0_OHM +#define PUR_05KR IS31FL3736_PUR_0K5_OHM +#define PUR_1KR IS31FL3736_PUR_1K_OHM +#define PUR_2KR IS31FL3736_PUR_2K_OHM +#define PUR_4KR IS31FL3736_PUR_4K_OHM +#define PUR_8KR IS31FL3736_PUR_8K_OHM +#define PUR_16KR IS31FL3736_PUR_16K_OHM +#define PUR_32KR IS31FL3736_PUR_32K_OHM // ======== #define IS31FL3736_I2C_ADDRESS_GND_GND 0x50 @@ -116,14 +116,23 @@ void is31fl3736_update_led_control_registers(uint8_t addr, uint8_t index); void is31fl3736_flush(void); -#define IS31FL3736_PUR_0R 0x00 // No PUR resistor -#define IS31FL3736_PUR_05KR 0x01 // 0.5k Ohm resistor -#define IS31FL3736_PUR_1KR 0x02 // 1.0k Ohm resistor -#define IS31FL3736_PUR_2KR 0x03 // 2.0k Ohm resistor -#define IS31FL3736_PUR_4KR 0x04 // 4.0k Ohm resistor -#define IS31FL3736_PUR_8KR 0x05 // 8.0k Ohm resistor -#define IS31FL3736_PUR_16KR 0x06 // 16k Ohm resistor -#define IS31FL3736_PUR_32KR 0x07 // 32k Ohm resistor +#define IS31FL3736_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3736_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3736_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3736_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3736_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3736_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3736_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3736_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3736_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3736_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3736_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3736_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3736_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3736_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3736_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3736_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3736_PWM_FREQUENCY_8K4_HZ 0b000 #define IS31FL3736_PWM_FREQUENCY_4K2_HZ 0b001 diff --git a/drivers/led/issi/is31fl3737-simple.c b/drivers/led/issi/is31fl3737-simple.c index 0549af16f2e0008b79300873ce4bc9929eb1a17d..904b7c65bd810fd85f9ab24bbacbd95d4c299233 100644 --- a/drivers/led/issi/is31fl3737-simple.c +++ b/drivers/led/issi/is31fl3737-simple.c @@ -35,8 +35,8 @@ #define IS31FL3737_REG_CONFIGURATION 0x00 // PG3 #define IS31FL3737_REG_GLOBALCURRENT 0x01 // PG3 #define IS31FL3737_REG_RESET 0x11 // PG3 -#define IS31FL3737_REG_SWPULLUP 0x0F // PG3 -#define IS31FL3737_REG_CSPULLUP 0x10 // PG3 +#define IS31FL3737_REG_SW_PULLUP 0x0F // PG3 +#define IS31FL3737_REG_CS_PULLDOWN 0x10 // PG3 #define IS31FL3737_PWM_REGISTER_COUNT 192 // actually 144 #define IS31FL3737_LED_CONTROL_REGISTER_COUNT 24 @@ -53,12 +53,12 @@ # define IS31FL3737_PWM_FREQUENCY IS31FL3737_PWM_FREQUENCY_8K4_HZ // PFS - IS31FL3737B only #endif -#ifndef IS31FL3737_SWPULLUP -# define IS31FL3737_SWPULLUP IS31FL3737_PUR_0R +#ifndef IS31FL3737_SW_PULLUP +# define IS31FL3737_SW_PULLUP IS31FL3737_PUR_0_OHM #endif -#ifndef IS31FL3737_CSPULLUP -# define IS31FL3737_CSPULLUP IS31FL3737_PUR_0R +#ifndef IS31FL3737_CS_PULLDOWN +# define IS31FL3737_CS_PULLDOWN IS31FL3737_PDR_0_OHM #endif #ifndef IS31FL3737_GLOBALCURRENT @@ -181,9 +181,9 @@ void is31fl3737_init(uint8_t addr) { // Select PG3 is31fl3737_write_register(addr, IS31FL3737_COMMANDREGISTER, IS31FL3737_PAGE_FUNCTION); // Set de-ghost pull-up resistors (SWx) - is31fl3737_write_register(addr, IS31FL3737_REG_SWPULLUP, IS31FL3737_SWPULLUP); + is31fl3737_write_register(addr, IS31FL3737_REG_SW_PULLUP, IS31FL3737_SW_PULLUP); // Set de-ghost pull-down resistors (CSx) - is31fl3737_write_register(addr, IS31FL3737_REG_CSPULLUP, IS31FL3737_CSPULLUP); + is31fl3737_write_register(addr, IS31FL3737_REG_CS_PULLDOWN, IS31FL3737_CS_PULLDOWN); // Set global current to maximum. is31fl3737_write_register(addr, IS31FL3737_REG_GLOBALCURRENT, IS31FL3737_GLOBALCURRENT); // Disable software shutdown. diff --git a/drivers/led/issi/is31fl3737-simple.h b/drivers/led/issi/is31fl3737-simple.h index dfc2189decbe743a2e6f965e706a38b5570776ff..3262987697b318613d7510b6d9d901ed488edcd7 100644 --- a/drivers/led/issi/is31fl3737-simple.h +++ b/drivers/led/issi/is31fl3737-simple.h @@ -38,23 +38,23 @@ # define IS31FL3737_PWM_FREQUENCY ISSI_PWM_FREQUENCY #endif #ifdef ISSI_SWPULLUP -# define IS31FL3737_SWPULLUP ISSI_SWPULLUP +# define IS31FL3737_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3737_CSPULLUP ISSI_CSPULLUP +# define IS31FL3737_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3737_GLOBALCURRENT ISSI_GLOBALCURRENT #endif -#define PUR_0R IS31FL3737_PUR_0R -#define PUR_05KR IS31FL3737_PUR_05KR -#define PUR_1KR IS31FL3737_PUR_1KR -#define PUR_2KR IS31FL3737_PUR_2KR -#define PUR_4KR IS31FL3737_PUR_4KR -#define PUR_8KR IS31FL3737_PUR_8KR -#define PUR_16KR IS31FL3737_PUR_16KR -#define PUR_32KR IS31FL3737_PUR_32KR +#define PUR_0R IS31FL3737_PUR_0_OHM +#define PUR_05KR IS31FL3737_PUR_0K5_OHM +#define PUR_1KR IS31FL3737_PUR_1K_OHM +#define PUR_2KR IS31FL3737_PUR_2K_OHM +#define PUR_4KR IS31FL3737_PUR_4K_OHM +#define PUR_8KR IS31FL3737_PUR_8K_OHM +#define PUR_16KR IS31FL3737_PUR_16K_OHM +#define PUR_32KR IS31FL3737_PUR_32K_OHM // ======== #define IS31FL3737_I2C_ADDRESS_GND 0x50 @@ -92,14 +92,23 @@ void is31fl3737_update_led_control_registers(uint8_t addr, uint8_t index); void is31fl3737_flush(void); -#define IS31FL3737_PUR_0R 0x00 // No PUR resistor -#define IS31FL3737_PUR_05KR 0x01 // 0.5k Ohm resistor in t_NOL -#define IS31FL3737_PUR_1KR 0x02 // 1.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_2KR 0x03 // 2.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_4KR 0x04 // 4.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_8KR 0x05 // 8.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_16KR 0x06 // 16k Ohm resistor in t_NOL -#define IS31FL3737_PUR_32KR 0x07 // 32k Ohm resistor in t_NOL +#define IS31FL3737_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3737_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3737_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3737_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3737_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3737_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3737_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3737_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3737_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3737_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3737_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3737_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3737_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3737_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3737_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3737_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3737_PWM_FREQUENCY_8K4_HZ 0b000 #define IS31FL3737_PWM_FREQUENCY_4K2_HZ 0b001 diff --git a/drivers/led/issi/is31fl3737.c b/drivers/led/issi/is31fl3737.c index 1599e59058ffee2f40102af255f364778c87deec..27860558a57e9d1d123b519ddd07fdaabcd6c06c 100644 --- a/drivers/led/issi/is31fl3737.c +++ b/drivers/led/issi/is31fl3737.c @@ -35,8 +35,8 @@ #define IS31FL3737_REG_CONFIGURATION 0x00 // PG3 #define IS31FL3737_REG_GLOBALCURRENT 0x01 // PG3 #define IS31FL3737_REG_RESET 0x11 // PG3 -#define IS31FL3737_REG_SWPULLUP 0x0F // PG3 -#define IS31FL3737_REG_CSPULLUP 0x10 // PG3 +#define IS31FL3737_REG_SW_PULLUP 0x0F // PG3 +#define IS31FL3737_REG_CS_PULLDOWN 0x10 // PG3 #define IS31FL3737_PWM_REGISTER_COUNT 192 // actually 144 #define IS31FL3737_LED_CONTROL_REGISTER_COUNT 24 @@ -53,12 +53,12 @@ # define IS31FL3737_PWM_FREQUENCY IS31FL3737_PWM_FREQUENCY_8K4_HZ // PFS - IS31FL3737B only #endif -#ifndef IS31FL3737_SWPULLUP -# define IS31FL3737_SWPULLUP IS31FL3737_PUR_0R +#ifndef IS31FL3737_SW_PULLUP +# define IS31FL3737_SW_PULLUP IS31FL3737_PUR_0_OHM #endif -#ifndef IS31FL3737_CSPULLUP -# define IS31FL3737_CSPULLUP IS31FL3737_PUR_0R +#ifndef IS31FL3737_CS_PULLDONW +# define IS31FL3737_CS_PULLDOWN IS31FL3737_PDR_0_OHM #endif #ifndef IS31FL3737_GLOBALCURRENT @@ -181,9 +181,9 @@ void is31fl3737_init(uint8_t addr) { // Select PG3 is31fl3737_write_register(addr, IS31FL3737_COMMANDREGISTER, IS31FL3737_PAGE_FUNCTION); // Set de-ghost pull-up resistors (SWx) - is31fl3737_write_register(addr, IS31FL3737_REG_SWPULLUP, IS31FL3737_SWPULLUP); + is31fl3737_write_register(addr, IS31FL3737_REG_SW_PULLUP, IS31FL3737_SW_PULLUP); // Set de-ghost pull-down resistors (CSx) - is31fl3737_write_register(addr, IS31FL3737_REG_CSPULLUP, IS31FL3737_CSPULLUP); + is31fl3737_write_register(addr, IS31FL3737_REG_CS_PULLDOWN, IS31FL3737_CS_PULLDOWN); // Set global current to maximum. is31fl3737_write_register(addr, IS31FL3737_REG_GLOBALCURRENT, IS31FL3737_GLOBALCURRENT); // Disable software shutdown. diff --git a/drivers/led/issi/is31fl3737.h b/drivers/led/issi/is31fl3737.h index 38482584e034fbe3e15c0725d6040925435544a6..17d2635dd8649bce4933d38d09afd9515a41d85c 100644 --- a/drivers/led/issi/is31fl3737.h +++ b/drivers/led/issi/is31fl3737.h @@ -50,10 +50,10 @@ # define IS31FL3737_PWM_FREQUENCY ISSI_PWM_FREQUENCY #endif #ifdef ISSI_SWPULLUP -# define IS31FL3737_SWPULLUP ISSI_SWPULLUP +# define IS31FL3737_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3737_CSPULLUP ISSI_CSPULLUP +# define IS31FL3737_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3737_GLOBALCURRENT ISSI_GLOBALCURRENT @@ -62,14 +62,14 @@ #define is31_led is31fl3737_led_t #define g_is31_leds g_is31fl3737_leds -#define PUR_0R IS31FL3737_PUR_0R -#define PUR_05KR IS31FL3737_PUR_05KR -#define PUR_1KR IS31FL3737_PUR_1KR -#define PUR_2KR IS31FL3737_PUR_2KR -#define PUR_4KR IS31FL3737_PUR_4KR -#define PUR_8KR IS31FL3737_PUR_8KR -#define PUR_16KR IS31FL3737_PUR_16KR -#define PUR_32KR IS31FL3737_PUR_32KR +#define PUR_0R IS31FL3737_PUR_0_OHM +#define PUR_05KR IS31FL3737_PUR_0K5_OHM +#define PUR_1KR IS31FL3737_PUR_1K_OHM +#define PUR_2KR IS31FL3737_PUR_2K_OHM +#define PUR_4KR IS31FL3737_PUR_4K_OHM +#define PUR_8KR IS31FL3737_PUR_8K_OHM +#define PUR_16KR IS31FL3737_PUR_16K_OHM +#define PUR_32KR IS31FL3737_PUR_32K_OHM // ======== #define IS31FL3737_I2C_ADDRESS_GND 0x50 @@ -109,14 +109,23 @@ void is31fl3737_update_led_control_registers(uint8_t addr, uint8_t index); void is31fl3737_flush(void); -#define IS31FL3737_PUR_0R 0x00 // No PUR resistor -#define IS31FL3737_PUR_05KR 0x01 // 0.5k Ohm resistor in t_NOL -#define IS31FL3737_PUR_1KR 0x02 // 1.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_2KR 0x03 // 2.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_4KR 0x04 // 4.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_8KR 0x05 // 8.0k Ohm resistor in t_NOL -#define IS31FL3737_PUR_16KR 0x06 // 16k Ohm resistor in t_NOL -#define IS31FL3737_PUR_32KR 0x07 // 32k Ohm resistor in t_NOL +#define IS31FL3737_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3737_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3737_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3737_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3737_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3737_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3737_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3737_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3737_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3737_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3737_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3737_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3737_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3737_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3737_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3737_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3737_PWM_FREQUENCY_8K4_HZ 0b000 #define IS31FL3737_PWM_FREQUENCY_4K2_HZ 0b001 diff --git a/drivers/led/issi/is31fl3741-simple.c b/drivers/led/issi/is31fl3741-simple.c index 83a1f12f6fd943f3fa15d266f5ee9feb2ee8b8cb..582551c05e18b037c36713cecd6b7cf2956e0438 100644 --- a/drivers/led/issi/is31fl3741-simple.c +++ b/drivers/led/issi/is31fl3741-simple.c @@ -58,12 +58,12 @@ # define IS31FL3741_PWM_FREQUENCY IS31FL3741_PWM_FREQUENCY_29K_HZ #endif -#ifndef IS31FL3741_SWPULLUP -# define IS31FL3741_SWPULLUP IS31FL3741_PUR_32KR +#ifndef IS31FL3741_SW_PULLUP +# define IS31FL3741_SW_PULLUP IS31FL3741_PUR_32K_OHM #endif -#ifndef IS31FL3741_CSPULLUP -# define IS31FL3741_CSPULLUP IS31FL3741_PUR_32KR +#ifndef IS31FL3741_CS_PULLDOWN +# define IS31FL3741_CS_PULLDOWN IS31FL3741_PDR_32K_OHM #endif #ifndef IS31FL3741_GLOBALCURRENT @@ -192,7 +192,7 @@ void is31fl3741_init(uint8_t addr) { // Set Golbal Current Control Register is31fl3741_write_register(addr, IS31FL3741_REG_GLOBALCURRENT, IS31FL3741_GLOBALCURRENT); // Set Pull up & Down for SWx CSy - is31fl3741_write_register(addr, IS31FL3741_REG_PULLDOWNUP, ((IS31FL3741_CSPULLUP << 4) | IS31FL3741_SWPULLUP)); + is31fl3741_write_register(addr, IS31FL3741_REG_PULLDOWNUP, ((IS31FL3741_CS_PULLDOWN << 4) | IS31FL3741_SW_PULLUP)); // Set PWM frequency is31fl3741_write_register(addr, IS31FL3741_REG_PWM_FREQUENCY, (IS31FL3741_PWM_FREQUENCY & 0b1111)); diff --git a/drivers/led/issi/is31fl3741-simple.h b/drivers/led/issi/is31fl3741-simple.h index d022f3a865c3af4031f2f874587cf6f40b191408..b8cda3da34049b4f75f6b881c81e413f5f2ef451 100644 --- a/drivers/led/issi/is31fl3741-simple.h +++ b/drivers/led/issi/is31fl3741-simple.h @@ -38,23 +38,23 @@ # define IS31FL3741_CONFIGURATION ISSI_CONFIGURATION #endif #ifdef ISSI_SWPULLUP -# define IS31FL3741_SWPULLUP ISSI_SWPULLUP +# define IS31FL3741_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3741_CSPULLUP ISSI_CSPULLUP +# define IS31FL3741_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3741_GLOBALCURRENT ISSI_GLOBALCURRENT #endif -#define PUR_0R IS31FL3741_PUR_0R -#define PUR_05KR IS31FL3741_PUR_05KR -#define PUR_1KR IS31FL3741_PUR_1KR -#define PUR_2KR IS31FL3741_PUR_2KR -#define PUR_4KR IS31FL3741_PUR_4KR -#define PUR_8KR IS31FL3741_PUR_8KR -#define PUR_16KR IS31FL3741_PUR_16KR -#define PUR_32KR IS31FL3741_PUR_32KR +#define PUR_0R IS31FL3741_PUR_0_OHM +#define PUR_05KR IS31FL3741_PUR_0K5_OHM +#define PUR_1KR IS31FL3741_PUR_1K_OHM +#define PUR_2KR IS31FL3741_PUR_2K_OHM +#define PUR_4KR IS31FL3741_PUR_4K_OHM +#define PUR_8KR IS31FL3741_PUR_8K_OHM +#define PUR_16KR IS31FL3741_PUR_16K_OHM +#define PUR_32KR IS31FL3741_PUR_32K_OHM // ======== #define IS31FL3741_I2C_ADDRESS_GND 0x30 @@ -95,14 +95,23 @@ void is31fl3741_set_pwm_buffer(const is31fl3741_led *pled, uint8_t value); void is31fl3741_flush(void); -#define IS31FL3741_PUR_0R 0x00 // No PUR resistor -#define IS31FL3741_PUR_05KR 0x01 // 0.5k Ohm resistor -#define IS31FL3741_PUR_1KR 0x02 // 1.0k Ohm resistor -#define IS31FL3741_PUR_2KR 0x03 // 2.0k Ohm resistor -#define IS31FL3741_PUR_4KR 0x04 // 4.0k Ohm resistor -#define IS31FL3741_PUR_8KR 0x05 // 8.0k Ohm resistor -#define IS31FL3741_PUR_16KR 0x06 // 16k Ohm resistor -#define IS31FL3741_PUR_32KR 0x07 // 32k Ohm resistor +#define IS31FL3741_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3741_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3741_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3741_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3741_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3741_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3741_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3741_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3741_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3741_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3741_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3741_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3741_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3741_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3741_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3741_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3741_PWM_FREQUENCY_29K_HZ 0b0000 #define IS31FL3741_PWM_FREQUENCY_3K6_HZ 0b0011 diff --git a/drivers/led/issi/is31fl3741.c b/drivers/led/issi/is31fl3741.c index 214cccf2ca48d588d5a10cb2451065449ed4f965..9925a169c316211ec33a178d350c6a88e791bbf0 100644 --- a/drivers/led/issi/is31fl3741.c +++ b/drivers/led/issi/is31fl3741.c @@ -58,12 +58,12 @@ # define IS31FL3741_PWM_FREQUENCY IS31FL3741_PWM_FREQUENCY_29K_HZ #endif -#ifndef IS31FL3741_SWPULLUP -# define IS31FL3741_SWPULLUP IS31FL3741_PUR_32KR +#ifndef IS31FL3741_SW_PULLUP +# define IS31FL3741_SW_PULLUP IS31FL3741_PUR_32K_OHM #endif -#ifndef IS31FL3741_CSPULLUP -# define IS31FL3741_CSPULLUP IS31FL3741_PUR_32KR +#ifndef IS31FL3741_CS_PULLDOWN +# define IS31FL3741_CS_PULLDOWN IS31FL3741_PDR_32K_OHM #endif #ifndef IS31FL3741_GLOBALCURRENT @@ -192,7 +192,7 @@ void is31fl3741_init(uint8_t addr) { // Set Golbal Current Control Register is31fl3741_write_register(addr, IS31FL3741_REG_GLOBALCURRENT, IS31FL3741_GLOBALCURRENT); // Set Pull up & Down for SWx CSy - is31fl3741_write_register(addr, IS31FL3741_REG_PULLDOWNUP, ((IS31FL3741_CSPULLUP << 4) | IS31FL3741_SWPULLUP)); + is31fl3741_write_register(addr, IS31FL3741_REG_PULLDOWNUP, ((IS31FL3741_CS_PULLDOWN << 4) | IS31FL3741_SW_PULLUP)); // Set PWM frequency is31fl3741_write_register(addr, IS31FL3741_REG_PWM_FREQUENCY, (IS31FL3741_PWM_FREQUENCY & 0b1111)); diff --git a/drivers/led/issi/is31fl3741.h b/drivers/led/issi/is31fl3741.h index 2b0e4495547250c8015ccf2646ccb45385deecc6..7f3c31a8abd7ce1fdcc958f141a8fde9321024b7 100644 --- a/drivers/led/issi/is31fl3741.h +++ b/drivers/led/issi/is31fl3741.h @@ -50,10 +50,10 @@ # define IS31FL3741_CONFIGURATION ISSI_CONFIGURATION #endif #ifdef ISSI_SWPULLUP -# define IS31FL3741_SWPULLUP ISSI_SWPULLUP +# define IS31FL3741_SW_PULLUP ISSI_SWPULLUP #endif #ifdef ISSI_CSPULLUP -# define IS31FL3741_CSPULLUP ISSI_CSPULLUP +# define IS31FL3741_CS_PULLDOWN ISSI_CSPULLUP #endif #ifdef ISSI_GLOBALCURRENT # define IS31FL3741_GLOBALCURRENT ISSI_GLOBALCURRENT @@ -62,14 +62,14 @@ #define is31_led is31fl3741_led_t #define g_is31_leds g_is31fl3741_leds -#define PUR_0R IS31FL3741_PUR_0R -#define PUR_05KR IS31FL3741_PUR_05KR -#define PUR_1KR IS31FL3741_PUR_1KR -#define PUR_2KR IS31FL3741_PUR_2KR -#define PUR_4KR IS31FL3741_PUR_4KR -#define PUR_8KR IS31FL3741_PUR_8KR -#define PUR_16KR IS31FL3741_PUR_16KR -#define PUR_32KR IS31FL3741_PUR_32KR +#define PUR_0R IS31FL3741_PUR_0_OHM +#define PUR_05KR IS31FL3741_PUR_0K5_OHM +#define PUR_1KR IS31FL3741_PUR_1K_OHM +#define PUR_2KR IS31FL3741_PUR_2K_OHM +#define PUR_4KR IS31FL3741_PUR_4K_OHM +#define PUR_8KR IS31FL3741_PUR_8K_OHM +#define PUR_16KR IS31FL3741_PUR_16K_OHM +#define PUR_32KR IS31FL3741_PUR_32K_OHM // ======== #define IS31FL3741_I2C_ADDRESS_GND 0x30 @@ -112,14 +112,23 @@ void is31fl3741_set_pwm_buffer(const is31fl3741_led_t *pled, uint8_t red, uint8_ void is31fl3741_flush(void); -#define IS31FL3741_PUR_0R 0x00 // No PUR resistor -#define IS31FL3741_PUR_05KR 0x01 // 0.5k Ohm resistor -#define IS31FL3741_PUR_1KR 0x02 // 1.0k Ohm resistor -#define IS31FL3741_PUR_2KR 0x03 // 2.0k Ohm resistor -#define IS31FL3741_PUR_4KR 0x04 // 4.0k Ohm resistor -#define IS31FL3741_PUR_8KR 0x05 // 8.0k Ohm resistor -#define IS31FL3741_PUR_16KR 0x06 // 16k Ohm resistor -#define IS31FL3741_PUR_32KR 0x07 // 32k Ohm resistor +#define IS31FL3741_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3741_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3741_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3741_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3741_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3741_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3741_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3741_PDR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3741_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3741_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3741_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3741_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3741_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3741_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3741_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3741_PUR_32K_OHM 0b111 // 32 kOhm resistor #define IS31FL3741_PWM_FREQUENCY_29K_HZ 0b0000 #define IS31FL3741_PWM_FREQUENCY_3K6_HZ 0b0011 diff --git a/keyboards/input_club/k_type/is31fl3733-dual.c b/keyboards/input_club/k_type/is31fl3733-dual.c index d9cf3b5170452d0bf366448607463b0530f4e81b..8aa9344c8508c075407311882d3ac17c68317576 100644 --- a/keyboards/input_club/k_type/is31fl3733-dual.c +++ b/keyboards/input_club/k_type/is31fl3733-dual.c @@ -33,8 +33,8 @@ #define IS31FL3733_REG_CONFIGURATION 0x00 // PG3 #define IS31FL3733_REG_GLOBALCURRENT 0x01 // PG3 #define IS31FL3733_REG_RESET 0x11 // PG3 -#define IS31FL3733_REG_SWPULLUP 0x0F // PG3 -#define IS31FL3733_REG_CSPULLUP 0x10 // PG3 +#define IS31FL3733_REG_SW_PULLUP 0x0F // PG3 +#define IS31FL3733_REG_CS_PULLDOWN 0x10 // PG3 #define IS31FL3733_PWM_REGISTER_COUNT 192 #define IS31FL3733_LED_CONTROL_REGISTER_COUNT 24 @@ -51,12 +51,12 @@ # define IS31FL3733_PWM_FREQUENCY IS31FL3733_PWM_FREQUENCY_8K4_HZ // PFS - IS31FL3733B only #endif -#ifndef IS31FL3733_SWPULLUP -# define IS31FL3733_SWPULLUP IS31FL3733_PUR_0R +#ifndef IS31FL3733_SW_PULLUP +# define IS31FL3733_SW_PULLUP IS31FL3733_PUR_0_OHM #endif -#ifndef IS31FL3733_CSPULLUP -# define IS31FL3733_CSPULLUP IS31FL3733_PUR_0R +#ifndef IS31FL3733_CS_PULLDOWN +# define IS31FL3733_CS_PULLDOWN IS31FL3733_PDR_0_OHM #endif #ifndef IS31FL3733_GLOBALCURRENT @@ -193,9 +193,9 @@ void is31fl3733_init(uint8_t bus, uint8_t addr, uint8_t sync) { // Select PG3 is31fl3733_write_register(bus, addr, IS31FL3733_COMMANDREGISTER, IS31FL3733_PAGE_FUNCTION); // Set de-ghost pull-up resistors (SWx) - is31fl3733_write_register(bus, addr, IS31FL3733_REG_SWPULLUP, IS31FL3733_SWPULLUP); + is31fl3733_write_register(bus, addr, IS31FL3733_REG_SW_PULLUP, IS31FL3733_SW_PULLUP); // Set de-ghost pull-down resistors (CSx) - is31fl3733_write_register(bus, addr, IS31FL3733_REG_CSPULLUP, IS31FL3733_CSPULLUP); + is31fl3733_write_register(bus, addr, IS31FL3733_REG_CS_PULLDOWN, IS31FL3733_CS_PULLDOWN); // Set global current to maximum. is31fl3733_write_register(bus, addr, IS31FL3733_REG_GLOBALCURRENT, IS31FL3733_GLOBALCURRENT); // Disable software shutdown. diff --git a/keyboards/input_club/k_type/is31fl3733-dual.h b/keyboards/input_club/k_type/is31fl3733-dual.h index 696c234b678309f4c948f65de4bb69d370d586e9..51c4492ff83682ccb29ada7c112b9befd1eae1ce 100644 --- a/keyboards/input_club/k_type/is31fl3733-dual.h +++ b/keyboards/input_club/k_type/is31fl3733-dual.h @@ -67,19 +67,29 @@ void is31fl3733_update_led_control_registers(uint8_t addr, uint8_t index); void is31fl3733_flush(void); -#define IS31FL3733_PUR_0R 0x00 // No PUR resistor -#define IS31FL3733_PUR_05KR 0x02 // 0.5k Ohm resistor in t_NOL -#define IS31FL3733_PUR_3KR 0x03 // 3.0k Ohm resistor on all the time -#define IS31FL3733_PUR_4KR 0x04 // 4.0k Ohm resistor on all the time -#define IS31FL3733_PUR_8KR 0x05 // 8.0k Ohm resistor on all the time -#define IS31FL3733_PUR_16KR 0x06 // 16k Ohm resistor on all the time -#define IS31FL3733_PUR_32KR 0x07 // 32k Ohm resistor in t_NOL +#define IS31FL3733_PDR_0_OHM 0b000 // No pull-down resistor +#define IS31FL3733_PDR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3733_PDR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3733_PDR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3733_PDR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3733_PDR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3733_PDR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3733_PDR_32K_OHM 0b111 // 32 kOhm resistor -#define IS31FL3733_PWM_FREQUENCY_8K4_HZ 0x00 -#define IS31FL3733_PWM_FREQUENCY_4K2_HZ 0x01 -#define IS31FL3733_PWM_FREQUENCY_26K7_HZ 0x02 -#define IS31FL3733_PWM_FREQUENCY_2K1_HZ 0x03 -#define IS31FL3733_PWM_FREQUENCY_1K05_HZ 0x04 +#define IS31FL3733_PUR_0_OHM 0b000 // No pull-up resistor +#define IS31FL3733_PUR_0K5_OHM 0b001 // 0.5 kOhm resistor +#define IS31FL3733_PUR_1K_OHM 0b010 // 1 kOhm resistor +#define IS31FL3733_PUR_2K_OHM 0b011 // 2 kOhm resistor +#define IS31FL3733_PUR_4K_OHM 0b100 // 4 kOhm resistor +#define IS31FL3733_PUR_8K_OHM 0b101 // 8 kOhm resistor +#define IS31FL3733_PUR_16K_OHM 0b110 // 16 kOhm resistor +#define IS31FL3733_PUR_32K_OHM 0b111 // 32 kOhm resistor + +#define IS31FL3733_PWM_FREQUENCY_8K4_HZ 0b000 +#define IS31FL3733_PWM_FREQUENCY_4K2_HZ 0b001 +#define IS31FL3733_PWM_FREQUENCY_26K7_HZ 0b010 +#define IS31FL3733_PWM_FREQUENCY_2K1_HZ 0b011 +#define IS31FL3733_PWM_FREQUENCY_1K05_HZ 0b100 #define IS31FL3733_SYNC_NONE 0b00 #define IS31FL3733_SYNC_MASTER 0b01