Skip to content

Commit 36e3be8

Browse files
committed
Merge branch 'fix/led_indicator_scale_rgb_to_pwm_duty_range' into 'master'
fix(led_indicator): Scale RGB values (0-255) to PWM duty range (0-8191) See merge request ae_group/esp-iot-solution!1240
2 parents faac5a7 + ebc8c9c commit 36e3be8

File tree

3 files changed

+14
-3
lines changed

3 files changed

+14
-3
lines changed

components/led/led_indicator/CHANGELOG.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,16 @@
11
# ChangeLog
22

3+
## v1.1.1 - 2025-03-11
4+
5+
### Bugfix
6+
7+
* Fix: Scale RGB values (0-255) to PWM duty range (0-8191)
8+
39
## v1.1.0 - 2025-02-24
410

511
### Improve:
612

7-
- Update the version of dependent cmake_utilities to *
13+
* Update the version of dependent cmake_utilities to *
814

915
## v1.0.0 - 2024-8-14
1016

components/led/led_indicator/idf_component.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
version: "1.1.0"
1+
version: "1.1.1"
22
description: LED indicator driver
33
url: https://github.com/espressif/esp-iot-solution/tree/master/components/led/led_indicator
44
dependencies:

components/led/led_indicator/src/led_rgb.c

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* SPDX-FileCopyrightText: 2023 Espressif Systems (Shanghai) CO LTD
2+
* SPDX-FileCopyrightText: 2023-2025 Espressif Systems (Shanghai) CO LTD
33
*
44
* SPDX-License-Identifier: Apache-2.0
55
*/
@@ -95,6 +95,11 @@ esp_err_t led_indicator_rgb_set_on_off(void *rgb_handle, bool on_off)
9595
if (on_off) {
9696
uint32_t rgb[3] = {0};
9797
led_indicator_hsv2rgb(p_rgb->hsv.value, &rgb[0], &rgb[1], &rgb[2]);
98+
99+
for (int i = 0; i < 3; i++) {
100+
rgb[i] = rgb[i] * p_rgb->max_duty / UINT8_MAX;
101+
}
102+
98103
ret = led_indicator_rgb_set_duty(p_rgb, rgb);
99104
LED_RGB_CHECK(ESP_OK == ret, "LEDC set duty error", return ret);
100105
} else {

0 commit comments

Comments
 (0)