diff mbox series

dt: bindings: lp55xx: Updte yaml examples with new color ID

Message ID 20200812193248.11325-1-dmurphy@ti.com
State Not Applicable, archived
Headers show
Series dt: bindings: lp55xx: Updte yaml examples with new color ID | expand

Checks

Context Check Description
robh/checkpatch success

Commit Message

Dan Murphy Aug. 12, 2020, 7:32 p.m. UTC
Update the binding examples for the color ID to LED_COLOR_ID_RGB

Signed-off-by: Dan Murphy <dmurphy@ti.com>
---
 Documentation/devicetree/bindings/leds/leds-lp55xx.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Dan Murphy Aug. 18, 2020, 7:06 p.m. UTC | #1
Pavel

On 8/12/20 2:32 PM, Dan Murphy wrote:
> Update the binding examples for the color ID to LED_COLOR_ID_RGB
>
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
> ---
>   Documentation/devicetree/bindings/leds/leds-lp55xx.yaml | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml b/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
> index b1bb3feb0f4d..89f69d62493e 100644
> --- a/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
> +++ b/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
> @@ -189,7 +189,7 @@ examples:
>                  #address-cells = <1>;
>                  #size-cells = <0>;
>                  reg = <0x2>;
> -               color = <LED_COLOR_ID_MULTI>;
> +               color = <LED_COLOR_ID_RGB>;
>                  function = LED_FUNCTION_STANDBY;
>                  linux,default-trigger = "heartbeat";
>   

Don't forget this simple update to the LP55xx yaml file.

Dan
Rob Herring Aug. 24, 2020, 11:25 p.m. UTC | #2
On Wed, 12 Aug 2020 14:32:48 -0500, Dan Murphy wrote:
> Update the binding examples for the color ID to LED_COLOR_ID_RGB
> 
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
> ---
>  Documentation/devicetree/bindings/leds/leds-lp55xx.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Rob Herring <robh@kernel.org>
Pavel Machek Sept. 9, 2020, 9:13 a.m. UTC | #3
On Wed 2020-08-12 14:32:48, Dan Murphy wrote:
> Update the binding examples for the color ID to LED_COLOR_ID_RGB
> 
> Signed-off-by: Dan Murphy <dmurphy@ti.com>

Thanks, applied.

BR,
									Pavel
Rob Herring Sept. 28, 2020, 4:18 p.m. UTC | #4
On Wed, Aug 12, 2020 at 2:32 PM Dan Murphy <dmurphy@ti.com> wrote:
>
> Update the binding examples for the color ID to LED_COLOR_ID_RGB
>
> Signed-off-by: Dan Murphy <dmurphy@ti.com>
> ---
>  Documentation/devicetree/bindings/leds/leds-lp55xx.yaml | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml b/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
> index b1bb3feb0f4d..89f69d62493e 100644
> --- a/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
> +++ b/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
> @@ -189,7 +189,7 @@ examples:
>                 #address-cells = <1>;
>                 #size-cells = <0>;
>                 reg = <0x2>;
> -               color = <LED_COLOR_ID_MULTI>;
> +               color = <LED_COLOR_ID_RGB>;

Looks like common.yaml needs updating too:

Documentation/devicetree/bindings/leds/leds-lp55xx.example.dt.yaml:
led-controller@33: multi-led@2:color:0:0: 9 is greater than the
maximum of 8
From schema: /builds/robherring/linux-dt-bindings/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml

Rob
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml b/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
index b1bb3feb0f4d..89f69d62493e 100644
--- a/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
+++ b/Documentation/devicetree/bindings/leds/leds-lp55xx.yaml
@@ -189,7 +189,7 @@  examples:
                #address-cells = <1>;
                #size-cells = <0>;
                reg = <0x2>;
-               color = <LED_COLOR_ID_MULTI>;
+               color = <LED_COLOR_ID_RGB>;
                function = LED_FUNCTION_STANDBY;
                linux,default-trigger = "heartbeat";