[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-Id: <20220607075247.58048-1-krzysztof.kozlowski@linaro.org>
Date: Tue, 7 Jun 2022 09:52:46 +0200
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: Pavel Machek <pavel@....cz>, Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Dan Murphy <dmurphy@...com>,
Jacek Anaszewski <jacek.anaszewski@...il.com>,
Sven Schwermer <sven.schwermer@...ruptive-technologies.com>,
linux-leds@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
Rob Herring <robh@...nel.org>,
Kyle Swenson <kyle.swenson@....tech>
Subject: [PATCH v3 1/2] dt-bindings: leds: lp50xx: correct reg/unit addresses in example
The multi-led node defined address/size cells, so it is intended to have
children with unit addresses.
The second multi-led's reg property defined three LED indexes within one
reg item, which is not correct - these are three separate items.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Reviewed-by: Rob Herring <robh@...nel.org>
---
Changes since v2:
1. Minor grammar change in commit msg.
2. Add Rb tag.
Changes since v1:
1. Correct multi-led unit address as well (Rob).
Cc: Kyle Swenson <kyle.swenson@....tech>
---
.../devicetree/bindings/leds/leds-lp50xx.yaml | 24 ++++++++++++-------
1 file changed, 15 insertions(+), 9 deletions(-)
diff --git a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
index f12fe5b53f30..29ce0cb7d449 100644
--- a/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
+++ b/Documentation/devicetree/bindings/leds/leds-lp50xx.yaml
@@ -99,35 +99,41 @@ examples:
color = <LED_COLOR_ID_RGB>;
function = LED_FUNCTION_CHARGING;
- led-0 {
+ led@0 {
+ reg = <0x0>;
color = <LED_COLOR_ID_RED>;
};
- led-1 {
+ led@1 {
+ reg = <0x1>;
color = <LED_COLOR_ID_GREEN>;
};
- led-2 {
+ led@2 {
+ reg = <0x2>;
color = <LED_COLOR_ID_BLUE>;
};
};
- multi-led@2 {
+ multi-led@3 {
#address-cells = <1>;
- #size-cells = <2>;
- reg = <0x2 0x3 0x5>;
+ #size-cells = <0>;
+ reg = <0x3>, <0x4>, <0x5>;
color = <LED_COLOR_ID_RGB>;
function = LED_FUNCTION_STANDBY;
- led-6 {
+ led@3 {
+ reg = <0x3>;
color = <LED_COLOR_ID_RED>;
};
- led-7 {
+ led@4 {
+ reg = <0x4>;
color = <LED_COLOR_ID_GREEN>;
};
- led-8 {
+ led@5 {
+ reg = <0x5>;
color = <LED_COLOR_ID_BLUE>;
};
};
--
2.34.1
Powered by blists - more mailing lists