| Message ID | 20240531114124.45346-1-hdegoede@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | KTD2026 indicator LED for X86 Xiaomi Pad2 | expand |
On Fri, 31 May 2024 13:41:18 +0200, Hans de Goede wrote: > Here is v10 of Kate's series to add support for Xiaomi Pad2 indicator LED, > this is unchanged from v9 except for being rebased on top of 6.10-rc1 and > dropping patch 7/7 since that one has been merged through the pdx86 tree. > > Patch 6/6 has an Acked-by from Sebastien for merging this patch through > the leds tree since it depends on the earlier patches. > > [...] Applied, thanks! [1/6] leds: rgb: leds-ktd202x: Get device properties through fwnode to support ACPI commit: f14aa5ea415b8add245e976bfab96a12986c6843 [2/6] leds: rgb: leds-ktd202x: I2C ID tables for KTD2026 and 2027 commit: 75bd07aef47e1a984229e6ec702e8b9aee0226e4 [3/6] leds: rgb: leds-ktd202x: Initialize mutex earlier commit: e1b08c6f5b92d408a9fcc1030a340caeb9852250 [4/6] leds: core: Add led_mc_set_brightness() function commit: 5607ca92e6274dfb85d0ff7c4e91e6c4ddb6d25c [5/6] leds: trigger: Add led_mc_trigger_event() function commit: 0921a57c91648b08857b47a2f26fa7942f06120f [6/6] power: supply: power-supply-leds: Add charging_orange_full_green trigger for RGB LED commit: 9af12f57f1f9785f231d31a7365ad244c656b7ff -- Lee Jones [李琼斯]
On Fri, 31 May 2024, Hans de Goede wrote: > Hi All, > > Here is v10 of Kate's series to add support for Xiaomi Pad2 indicator LED, > this is unchanged from v9 except for being rebased on top of 6.10-rc1 and > dropping patch 7/7 since that one has been merged through the pdx86 tree. > > Patch 6/6 has an Acked-by from Sebastien for merging this patch through > the leds tree since it depends on the earlier patches. > > I have some further power-supply LED triggers work pending: > https://lore.kernel.org/linux-pm/20240510194012.138192-1-hdegoede@redhat.com/ > which applies on top of this series. So this is going to either require > an IB for Sebastian to merge into his tree before applying those, or > (once ready) that series can be applied through the LEDs tree too. Since this > follow up series deals with LED triggers I think that either way makes sense. > > This work includes: > 1. Migrate the original driver to fwnode to support x86 platforms. > 2. Support for multi-color LED trigger events. > 3. The LED shows orange when charging and the LED shows green when the > battery is full. Applied and sent for testing. All being well, I'll submit a PR soon.
Hi All, Here is v10 of Kate's series to add support for Xiaomi Pad2 indicator LED, this is unchanged from v9 except for being rebased on top of 6.10-rc1 and dropping patch 7/7 since that one has been merged through the pdx86 tree. Patch 6/6 has an Acked-by from Sebastien for merging this patch through the leds tree since it depends on the earlier patches. I have some further power-supply LED triggers work pending: https://lore.kernel.org/linux-pm/20240510194012.138192-1-hdegoede@redhat.com/ which applies on top of this series. So this is going to either require an IB for Sebastian to merge into his tree before applying those, or (once ready) that series can be applied through the LEDs tree too. Since this follow up series deals with LED triggers I think that either way makes sense. This work includes: 1. Migrate the original driver to fwnode to support x86 platforms. 2. Support for multi-color LED trigger events. 3. The LED shows orange when charging and the LED shows green when the battery is full. -- Changes in v10: 1. Rebased on top of 6.10-rc1 2. Drop patch 7/7 since that one has been merged through the pdx86 tree Changes in v9: 1. Switch to devm_mutex_init() 2. Add Andy's Reviewed-by to the series Changes in v8: 1. New bugfix: "leds: rgb: leds-ktd202x: Initialize mutex earlier" 2. Make charging_orange_full_green triggers set the colors in RGB order 3. Modify the Pad2 ktd202x fwnode to have the colors in RGB order Changes in v7: 1. Platform: x86-android-tablets: other: Add swnode for Xiaomi pad2 indicator LED was included in Hans' branch. 2. Included the tags from the previous version in the commit message. 3. Fixed the comma issue for the structure initialiser. Changes in v6: 1. The I2C ID table was moved to a separate patch. 2. The LED shows orange when charging. 3. The trigger name was renamed to charging-orange-full-green. 4. The default trigger of Xiaomi Pad2 is "bq27520-0-charging-orange-full-green". Changes in v5: 1. Fix swnode LED color settings. 2. Improve the driver based on the comments. 3. Introduce a LED new API- led_mc_trigger_event() to make the LED color can be changed according to the trigger. 4. Introduced a new trigger "charging-red-full-green". The LED will be red when charging and the LED will be green when the battery is full. 5. Set the default trigger to "bq27520-0-charging-red-full-green" for Xiaomi Pad2. Changes in v4: 1. Fix double casting. 2. Since force casting a pointer value to int will trigger a compiler warning, the type of num_leds was changed to unsigned long. Changes in v3: 1. Drop the patch "leds-ktd202x: Skip regulator settings for Xiaomi pad2" Changes in v2: 1. Typo and style fixes. 2. The patch 0003 skips all the regulator setup for Xiaomi pad2 since KTD2026 on Xiaomi pad2 is already powered by BP25890RTWR. So, the sleep can be removed when removing the module. Regards, Hans Hans de Goede (3): leds: rgb: leds-ktd202x: Initialize mutex earlier leds: core: Add led_mc_set_brightness() function leds: trigger: Add led_mc_trigger_event() function Kate Hsuan (3): leds: rgb: leds-ktd202x: Get device properties through fwnode to support ACPI leds: rgb: leds-ktd202x: I2C ID tables for KTD2026 and 2027 power: supply: power-supply-leds: Add charging_orange_full_green trigger for RGB LED drivers/leds/led-class-multicolor.c | 1 + drivers/leds/led-core.c | 31 +++++++++ drivers/leds/led-triggers.c | 20 ++++++ drivers/leds/rgb/Kconfig | 1 - drivers/leds/rgb/leds-ktd202x.c | 80 ++++++++++++++---------- drivers/power/supply/power_supply_leds.c | 23 +++++++ include/linux/leds.h | 26 ++++++++ include/linux/power_supply.h | 2 + 8 files changed, 149 insertions(+), 35 deletions(-)