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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20250617165044.21146-1-trannamatk@gmail.com>
Date: Tue, 17 Jun 2025 23:50:44 +0700
From: Nam Tran <trannamatk@...il.com>
To: rdunlap@...radead.org
Cc: lee@...nel.org,
	pavel@...nel.org,
	krzk+dt@...nel.org,
	robh@...nel.org,
	conor+dt@...nel.org,
	corbet@....net,
	linux-leds@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	devicetree@...r.kernel.org,
	linux-doc@...r.kernel.org
Subject: Re: [PATCH v9 2/4] leds: add TI/National Semiconductor LP5812 LED Driver

On Tue, 10 Jun 2025, Randy Dunlap wrote:

> > +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.

Thanks for pointing that out.
I'll change it to 'x' for better compatibility with config tools.

Appreciate your time and feedback.

Best regards,
Nam Tran