linux-kernel - Re: [PATCH v5 0/5] leds: add new LED driver for TI LP5812

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] [day] [month] [year] [list]

Message-Id: <20250418044345.2491-1-trannamatk@gmail.com>
Date: Fri, 18 Apr 2025 11:43:45 +0700
From: Nam Tran <trannamatk@...il.com>
To: krzk+dt@...nel.org
Cc: pavel@...nel.org,
	lee@...nel.org,
	robh@...nel.org,
	conor+dt@...nel.org,
	corbet@....net,
	devicetree@...r.kernel.org,
	linux-leds@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v5 0/5] leds: add new LED driver for TI LP5812

On Mon, 14 Apr 2025, Krzysztof Kozlowski wrote:

>On 14/04/2025 16:57, Nam Tran wrote:
>> This patch series adds support for the TI/National Semiconductor LP5812
>> 4x3 matrix RGB LED driver. The driver supports features such as autonomous
>> animation and time-cross-multiplexing (TCM) for dynamic LED effects.
>> 
>> Signed-off-by: Nam Tran <trannamatk@...il.com>
>> ---
>> Changes in v5:
>> - Rebase on v6.15-rc2
>> - Removed unused functions (lp5812_dump_regs, lp5812_update_bit).
>> - Address Krzysztof's review comments
>Which ones? What changed?

Apologies for the delayed reply.

I’ve updated the ABI document based on your previous comments.
Regarding the binding, I’ve removed redundant descriptions and fixed the indentation as you pointed out.

I’ll make sure to describe the changes more clearly in the next patch submission.

Best regards,
Nam Tran