linux-kernel - [PATCH RFC v2 2/2] dt-bindings: leds: lp50xx: Document child reg, fix example

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: <20250522-led-fix-v2-2-652046323ec3@axis.com>
Date: Thu, 22 May 2025 16:14:49 +0200
From: Johan Adolfsson <johan.adolfsson@...s.com>
To: Lee Jones <lee@...nel.org>, Pavel Machek <pavel@...nel.org>, "Jacek
 Anaszewski" <jacek.anaszewski@...il.com>
CC: <linux-leds@...r.kernel.org>, <linux-kernel@...r.kernel.org>, "Johan
 Adolfsson" <johan.adolfsson@...s.com>, <kernel@...s.com>
Subject: [PATCH RFC v2 2/2] dt-bindings: leds: lp50xx: Document child reg,
 fix example

The led child reg node is the index within the bank, document that
and update the example accordingly.

Signed-off-by: Johan Adolfsson <johan.adolfsson@...s.com>
---
 Documentation/devicetree/bindings/leds/leds-lp50xx.yaml | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
index 402c25424525..8ce327d00c2d 100644
--- a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
+++ b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
@@ -82,6 +82,11 @@ patternProperties:
         properties:
           reg:
             maxItems: 1
+            description:
+              This property denotes the index within the LED bank, 0, 1 or 2.
+              The value is actually optional for backward compatibility reasons
+              but is highly recommended to handle reversed devicetree
+              processing properly.
 
         required:
           - reg
@@ -139,17 +144,17 @@ examples:
                 function = LED_FUNCTION_STANDBY;
 
                 led@3 {
-                    reg = <0x3>;
+                    reg = <0x0>;
                     color = <LED_COLOR_ID_RED>;
                 };
 
                 led@4 {
-                    reg = <0x4>;
+                    reg = <0x1>;
                     color = <LED_COLOR_ID_GREEN>;
                 };
 
                 led@5 {
-                    reg = <0x5>;
+                    reg = <0x2>;
                     color = <LED_COLOR_ID_BLUE>;
                 };
             };

-- 
2.30.2