Re: [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Gergo Koteles <soyer@xxxxxx>, Ike Panhc <ike.pan@xxxxxxxxxxxxx>, Hans de Goede <hdegoede@xxxxxxxxxx>, Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>, Pavel Machek <pavel@xxxxxx>, Lee Jones <lee@xxxxxxxxxx>, Rob Herring <robh@xxxxxxxxxx>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>, Conor Dooley <conor+dt@xxxxxxxxxx>
Subject: Re: [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK
From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
Date: Tue, 2 Apr 2024 15:55:55 +0200
Cc: platform-driver-x86@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-leds@xxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx
In-reply-to: <8ac95e85a53dc0b8cce1e27fc1cab6d19221543b.1712063200.git.soyer@irl.hu>
References: <cover.1712063200.git.soyer@irl.hu> <8ac95e85a53dc0b8cce1e27fc1cab6d19221543b.1712063200.git.soyer@irl.hu>
User-agent: Mozilla Thunderbird

On 02/04/2024 15:21, Gergo Koteles wrote:
> Newer laptops have FnLock LED.
> 
> Add a define for this very common function.
> 
> Signed-off-by: Gergo Koteles <soyer@xxxxxx>
> ---
>  include/dt-bindings/leds/common.h | 1 +

Do we really need to define all these possible LED functions? Please
link to DTS user for this.

Best regards,
Krzysztof

Follow-Ups:
- Re: [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK
  - From: Hans de Goede
- Re: [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK
  - From: Gergo Koteles

References:
- [PATCH 0/3] add FnLock LED class device to ideapad laptops
  - From: Gergo Koteles
- [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK
  - From: Gergo Koteles

Prev by Date: [PATCH 3/3] platform/x86: ideapad-laptop: add FnLock LED class device
Next by Date: Re: [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK
Previous by thread: [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK
Next by thread: Re: [PATCH 1/3] dt-bindings: leds: add LED_FUNCTION_FNLOCK
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Security] [Bugtraq] [Linux OMAP] [Linux MIPS] [ECOS] [Asterisk Internet PBX] [Linux API]