Skip to content

AC Power Board: Improve LED handling#27

Open
lategoodbye wants to merge 3 commits into
v6.12.34-2.1.0-phy-cbfrom
v6.12.34-2.1.0-phy-cb-rgb-led
Open

AC Power Board: Improve LED handling#27
lategoodbye wants to merge 3 commits into
v6.12.34-2.1.0-phy-cbfrom
v6.12.34-2.1.0-phy-cb-rgb-led

Conversation

@lategoodbye
Copy link
Copy Markdown
Member

This pull request improve the LED support on AC Power Board. The main goal is to make the LEDs blink white during boot, but it contains some further improvements.

Current config wasn't prepared for AC Power Board, which has 4 individual
LEDs (red, green, blue, white) driven by NXP PCA9632.

- Enable LEDS_GROUP_MULTICOLOR to group all 4 individual LEDs to a RGBW
  one
- Make LEDS_CLASS_MULTICOLOR and LEDS_PCA963X built-in for sooner LED
  handling during boot
- Enable LEDS_TRIGGER_PATTERN for RGB fading effects

Signed-off-by: Stefan Wahren <stefan.wahren@chargebyte.com>
Before this change the driver initialized all LEDs with maximum
intensity. This isn't useful for LEDs, which needs to be configured
via DT. So introduce a new DT property to define the default
intensity of all indiviual LEDs. In case something went wrong
the driver fallsback to maximum brightness.

Additionally this also works with triggers. So it should be possible
to blink with a specific RGB color.

This has been tested with Tarragon board.

Signed-off-by: Stefan Wahren <stefan.wahren@chargebyte.com>
This groups all 4 individual LEDs to an additional RGBW LED (rgb:status).
Be aware that the sysfs of the individual LEDs become read-only.

Additionally the LEDs will now blink white during boot. Unfortunately
this can only be tested with real hardware.

Signed-off-by: Stefan Wahren <stefan.wahren@chargebyte.com>
@lategoodbye lategoodbye requested review from barsnick and mhei May 13, 2026 12:36
Copy link
Copy Markdown
Member

@mhei mhei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not tested, but LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants