Re: [PATCH v9 2/4] leds: add TI/National Semiconductor LP5812 LED Driver

From: Randy Dunlap
Date: Tue Jun 10 2025 - 14:13:44 EST




On 6/10/25 10:43 AM, Nam Tran wrote:
> diff --git a/drivers/leds/rgb/Kconfig b/drivers/leds/rgb/Kconfig
> index 222d943d826a..becee5c1d21c 100644
> --- a/drivers/leds/rgb/Kconfig
> +++ b/drivers/leds/rgb/Kconfig
> @@ -26,6 +26,19 @@ config LEDS_KTD202X
> To compile this driver as a module, choose M here: the module
> will be called leds-ktd202x.
>
> +config LEDS_LP5812
> + tristate "LED support for Texas Instruments LP5812"
> + depends on I2C
> + help
> + If you say Y here you get support for TI LP5812 LED driver.
> + The LP5812 is a 4 × 3 matrix RGB LED driver with autonomous

The '×' character does not display well (not at all) in menuconfig
or nconfig. The graphical configs (gconfig, xconfig) can display it.
I would change it to 4x3 (letter 'x') but it's up to you.


> + animation engine control.
> +
> + To compile this driver as a module, choose M here: the
> + module will be called leds-lp5812.
> +
> + If unsure, say N.

--
~Randy