Add BP5758 I2C LED driver and product example

.gitignore

@@ -14,3 +14,4 @@ products/*/configuration/output/*
 .DS_Store
 .vscode/
 .environment/
+_codeql_detected_source_root

components/light/CMakeLists.txt

@@ -1,6 +1,6 @@
-idf_component_register(SRC_DIRS led utils ws2812 .
-                       INCLUDE_DIRS led utils ws2812 .
-                       REQUIRES sw_timer rmt)
+idf_component_register(SRC_DIRS led utils ws2812 bp5758 .
+                       INCLUDE_DIRS led utils ws2812 bp5758 .
+                       REQUIRES sw_timer rmt i2c)
 
 target_include_directories(
     ${COMPONENT_LIB} PRIVATE ${COMPONENT_INCLUDES}

components/light/Kconfig

@@ -8,5 +8,8 @@ menu "Light"
 
         config USE_LIGHT_DEVICE_TYPE_LED
         bool "Select LED as light device"
+
+        config USE_LIGHT_DEVICE_TYPE_BP5758
+        bool "Select BP5758 as light device"
     endchoice
 endmenu

components/light/bp5758/CMakeLists.txt

@@ -0,0 +1,3 @@
+idf_component_register(SRCS "bp5758_driver.c"
+                       INCLUDE_DIRS "."
+                       REQUIRES i2c)

components/light/bp5758/bp5758_driver.c

@@ -0,0 +1,239 @@
+// Copyright 2024 Espressif Systems (Shanghai) PTE LTD
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+#include <stdio.h>
+#include <string.h>
+#include "bp5758_driver.h"
+#include "i2c_master.h"
+#include "hal/i2c_types.h"
+
+/* BP5758 I2C address */
+#define BP5758_I2C_ADDR          0x15
+
+/* BP5758 Register addresses */
+#define BP5758_REG_OUTPUT_CTRL   0x01
+#define BP5758_REG_OUT1_CURRENT  0x02
+#define BP5758_REG_OUT2_CURRENT  0x03
+#define BP5758_REG_OUT3_CURRENT  0x04
+#define BP5758_REG_OUT4_CURRENT  0x05
+#define BP5758_REG_OUT5_CURRENT  0x06
+#define BP5758_REG_CURRENT_CTRL  0x07
+
+/* Default maximum current setting (0x1F = 90mA) */
+#define BP5758_MAX_CURRENT       0x1F
+
+/* Number of channels */
+#define BP5758_CHANNEL_COUNT     5
+
+/* I2C port and pins */
+static int s_i2c_port = I2C_NUM_0;
+static gpio_num_t s_scl_io = GPIO_NUM_NC;
+static gpio_num_t s_sda_io = GPIO_NUM_NC;
+
+/* Channel values buffer */
+static uint8_t s_channel_values[BP5758_CHANNEL_COUNT] = {0};
+
+/* Initialized flag */
+static bool s_initialized = false;
+
+static const char *TAG = "bp5758_driver";
+
+/**
+ * @brief Write data to BP5758 register
+ */
+static int bp5758_write_register(uint8_t reg, uint8_t value)
+{
+    uint8_t data[2] = {reg, value};
+    int ret = i2c_master_write_to_device(s_i2c_port, BP5758_I2C_ADDR, data, 2, -1);
+    if (ret != 0) {
+        printf("%s: Failed to write register 0x%02x\n", TAG, reg);
+        return -1;
+    }
+    return 0;
+}
+
+/**
+ * @brief Initialize BP5758 device
+ */
+static int bp5758_device_init(void)
+{
+    int ret;
+
+    /* Enable all 5 output channels */
+    ret = bp5758_write_register(BP5758_REG_OUTPUT_CTRL, 0x1F);
+    if (ret != 0) {
+        return ret;
+    }
+
+    /* Set maximum current for all channels */
+    for (int i = 0; i < BP5758_CHANNEL_COUNT; i++) {
+        ret = bp5758_write_register(BP5758_REG_OUT1_CURRENT + i, BP5758_MAX_CURRENT);
+        if (ret != 0) {
+            return ret;
+        }
+    }
+
+    /* Set current control register */
+    ret = bp5758_write_register(BP5758_REG_CURRENT_CTRL, 0x00);
+    if (ret != 0) {
+        return ret;
+    }
+
+    return 0;
+}
+
+int bp5758_driver_init(void)
+{
+    if (s_initialized) {
+        printf("%s: Already initialized\n", TAG);
+        return 0;
+    }
+
+    if (s_scl_io == GPIO_NUM_NC || s_sda_io == GPIO_NUM_NC) {
+        printf("%s: I2C pins not configured\n", TAG);
+        return -1;
+    }
+
+    /* Initialize I2C master */
+    int ret = i2c_master_init(s_i2c_port, s_scl_io, s_sda_io);
+    if (ret != 0) {
+        printf("%s: Failed to initialize I2C master\n", TAG);
+        return -1;
+    }
+
+    /* Initialize BP5758 device */
+    ret = bp5758_device_init();
+    if (ret != 0) {
+        printf("%s: Failed to initialize BP5758 device\n", TAG);
+        return -1;
+    }
+
+    /* Clear all channels */
+    memset(s_channel_values, 0, sizeof(s_channel_values));
+
+    s_initialized = true;
+    printf("%s: Initialized successfully\n", TAG);
+    return 0;
+}
+
+void bp5758_driver_deinit(void)
+{
+    /* Turn off all channels */
+    memset(s_channel_values, 0, sizeof(s_channel_values));
+    bp5758_driver_update_channels();
+
+    s_initialized = false;
+    printf("%s: Deinitialized\n", TAG);
+}
+
+int bp5758_driver_set_channel(uint8_t channel, uint8_t val)
+{
+    if (!s_initialized) {
+        printf("%s: Driver not initialized\n", TAG);
+        return -1;
+    }
+
+    if (channel >= BP5758_CHANNEL_COUNT) {
+        printf("%s: Invalid channel %d\n", TAG, channel);
+        return -1;
+    }
+
+    /* Scale from 0-255 to 0-255 (no scaling needed) */
+    s_channel_values[channel] = val;
+    return 0;
+}
+
+int bp5758_driver_get_channel(uint8_t channel, uint8_t *val)
+{
+    if (!s_initialized) {
+        printf("%s: Driver not initialized\n", TAG);
+        return -1;
+    }
+
+    if (channel >= BP5758_CHANNEL_COUNT || val == NULL) {
+        printf("%s: Invalid channel %d or null pointer\n", TAG, channel);
+        return -1;
+    }
+
+    *val = s_channel_values[channel];
+    return 0;
+}
+
+int bp5758_driver_update_channels(void)
+{
+    if (!s_initialized) {
+        printf("%s: Driver not initialized\n", TAG);
+        return -1;
+    }
+
+    /* BP5758 protocol for updating grayscale (PWM) values:
+     * The BP5758 uses a specific command sequence to update output channels.
+     * After configuration, grayscale data is written to control LED brightness.
+     * 
+     * This implementation sends each channel's PWM value by writing to
+     * the device. The actual BP5758 may use a different register map for
+     * grayscale vs. current control. This is a functional implementation
+     * that scales brightness values to the configured maximum current.
+     * 
+     * Note: Individual I2C transactions are used for simplicity. If the
+     * BP5758 protocol supports batch updates, consider combining multiple
+     * channel updates into a single I2C transaction to reduce bus overhead.
+     * 
+     * For production use, refer to the BP5758 datasheet for the exact
+     * grayscale register addresses and update sequence.
+     */
+
+    for (int i = 0; i < BP5758_CHANNEL_COUNT; i++) {
+        /* Scale PWM value (0-255) to current setting (0-0x1F)
+         * This approach sets the current level to control brightness.
+         * Actual BP5758 implementation may use separate grayscale registers.
+         * Using rounding to avoid precision loss: (value * max + 127) / 255
+         */
+        uint8_t cmd[2];
+        cmd[0] = BP5758_REG_OUT1_CURRENT + i;
+        cmd[1] = (s_channel_values[i] * BP5758_MAX_CURRENT + 127) / 255;
+
+        int ret = i2c_master_write_to_device(s_i2c_port, BP5758_I2C_ADDR, cmd, 2, -1);
+        if (ret != 0) {
+            printf("%s: Failed to update channel %d\n", TAG, i);
+            return -1;
+        }
+    }
+
+    return 0;
+}
+
+int bp5758_driver_regist_channel(uint8_t channel, gpio_num_t gpio)
+{
+    /* For BP5758, we use this function to configure I2C pins.
+     * The channel parameter explicitly identifies the pin type:
+     * - channel 0: SDA pin
+     * - channel 1: SCL pin
+     * 
+     * This is necessary because the light driver interface expects
+     * per-channel GPIO registration, but BP5758 uses I2C bus.
+     */
+
+    if (channel == 0) {
+        s_sda_io = gpio;
+        printf("%s: Registered SDA pin (channel 0): GPIO%d\n", TAG, gpio);
+    } else if (channel == 1) {
+        s_scl_io = gpio;
+        printf("%s: Registered SCL pin (channel 1): GPIO%d\n", TAG, gpio);
+    } else {
+        printf("%s: Warning - unexpected channel %d for I2C pin registration\n", TAG, channel);
+    }
+
+    return 0;
+}

components/light/bp5758/bp5758_driver.h

@@ -0,0 +1,88 @@
+// Copyright 2024 Espressif Systems (Shanghai) PTE LTD
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+/**
+ * @file bp5758_driver.h
+ * @brief BP5758 I2C LED driver for controlling up to 5-channel LEDs
+ *
+ * BP5758 is an I2C-based 5-channel constant current LED driver.
+ * It supports up to 5 channels for controlling RGB, CW, or RGBCW LEDs.
+ */
+
+#pragma once
+
+#include <stdint.h>
+#include "soc/gpio_num.h"
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+/* BP5758 channel definitions */
+#define BP5758_CHANNEL_RED     0
+#define BP5758_CHANNEL_GREEN   1
+#define BP5758_CHANNEL_BLUE    2
+#define BP5758_CHANNEL_COLD    3
+#define BP5758_CHANNEL_WARM    4
+#define BP5758_CHANNEL_BRIGHTNESS  5  /* Not used by BP5758 but kept for compatibility */
+
+/**
+ * @brief Initialize BP5758 driver
+ *
+ * @return 0 on success, negative value on error
+ */
+int bp5758_driver_init(void);
+
+/**
+ * @brief Deinitialize BP5758 driver
+ */
+void bp5758_driver_deinit(void);
+
+/**
+ * @brief Set value for a specific channel
+ *
+ * @param channel Channel number (0-4)
+ * @param val Value to set (0-255)
+ * @return 0 on success, negative value on error
+ */
+int bp5758_driver_set_channel(uint8_t channel, uint8_t val);
+
+/**
+ * @brief Get value for a specific channel
+ *
+ * @param channel Channel number (0-4)
+ * @param val Pointer to store the value
+ * @return 0 on success, negative value on error
+ */
+int bp5758_driver_get_channel(uint8_t channel, uint8_t *val);
+
+/**
+ * @brief Update all channels to the device
+ *
+ * @return 0 on success, negative value on error
+ */
+int bp5758_driver_update_channels(void);
+
+/**
+ * @brief Register channel with GPIO (for BP5758, this registers I2C pins)
+ *
+ * @param channel Pin type: 0 for SDA, 1 for SCL
+ * @param gpio GPIO number to use for the specified pin type
+ * @return 0 on success, negative value on error
+ */
+int bp5758_driver_regist_channel(uint8_t channel, gpio_num_t gpio);
+
+#ifdef __cplusplus
+}
+#endif