[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240614091220.GA3029315@google.com>
Date: Fri, 14 Jun 2024 10:12:20 +0100
From: Lee Jones <lee@...nel.org>
To: Thomas Weißschuh <linux@...ssschuh.net>
Cc: Pavel Machek <pavel@....cz>,
Thomas Weißschuh <thomas@...ssschuh.net>,
Benson Leung <bleung@...omium.org>,
Guenter Roeck <groeck@...omium.org>,
Tzung-Bi Shih <tzungbi@...nel.org>, linux-leds@...r.kernel.org,
linux-kernel@...r.kernel.org, chrome-platform@...ts.linux.dev,
Dustin Howett <dustin@...ett.net>,
Mario Limonciello <mario.limonciello@....com>
Subject: Re: [PATCH v3 0/5] ChromeOS Embedded Controller LED driver
On Thu, 13 Jun 2024, Thomas Weißschuh wrote:
> Add a LED driver that supports the LED devices exposed by the
> ChromeOS Embedded Controller.
>
> Patch 1-3 add a utility function to the led subsystem.
> Patch 4 introduces the actual driver.
> Patch 5 registers the driver through the cros_ec mfd devices.
>
> Currently the driver introduces some non-standard LED functions.
> (See "cros_ec_led_functions")
>
> Tested on a Framework 13 AMD, Firmware 3.05.
>
> Signed-off-by: Thomas Weißschuh <linux@...ssschuh.net>
> ---
> Changes in v3:
> - Set default_trigger explicitly as the LED core doesn't do this anymore
> - Only set intensity for first subled by default
> - Link to v2: https://lore.kernel.org/r/20240531-cros_ec-led-v2-0-6cc34408b40d@weissschuh.net
>
> Changes in v2:
> - Cosmetic cleanups (Tzung-Bi)
> - Add trailing comma to MFD cell array
> - Rename LEDs and trigger to "chromeos" prefix, to align with kbd
> backlight driver
> - Don't use type "rgb" anymore, they are only "multicolor"
> - Align commit messages and subject to subsystem standards (Lee)
> - Rename led_color_name() to led_get_color_name()
> - The same for cros_ec_led_color_name()
> - Link to v1: https://lore.kernel.org/r/20240520-cros_ec-led-v1-0-4068fc5c051a@weissschuh.net
>
> ---
> Thomas Weißschuh (5):
> leds: core: Introduce led_get_color_name() function
> leds: multicolor: Use led_get_color_name() function
> leds: core: Unexport led_colors[] array
> leds: Add ChromeOS EC driver
> mfd: cros_ec: Register LED subdevice
>
> MAINTAINERS | 5 +
> drivers/leds/Kconfig | 15 ++
> drivers/leds/Makefile | 1 +
> drivers/leds/led-class-multicolor.c | 2 +-
> drivers/leds/led-core.c | 12 +-
> drivers/leds/leds-cros_ec.c | 299 ++++++++++++++++++++++++++++++++++++
> drivers/leds/leds.h | 1 -
> drivers/mfd/cros_ec_dev.c | 9 ++
> include/linux/leds.h | 10 ++
> 9 files changed, 350 insertions(+), 4 deletions(-)
> ---
> base-commit: 2ccbdf43d5e758f8493a95252073cf9078a5fea5
> change-id: 20240519-cros_ec-led-3efa24e3991e
Applied and submitted for testing.
All being well, I'll follow-up with a cross-subsystem pull-request shortly
--
Lee Jones [李琼斯]
Powered by blists - more mailing lists