[1/2] dt-bindings: backlight: Add Kinetic KTD259 bindings

Message ID 20210321114341.944696-1-linus.walleij@linaro.org
State Accepted
Commit 4b79ec9a553de3a438631dfe6cb18fdfcaee2098
Headers show
Series
  • [1/2] dt-bindings: backlight: Add Kinetic KTD259 bindings
Related show

Commit Message

Linus Walleij March 21, 2021, 11:43 a.m.
The KTD259 is just a newer version of the KTD253 and works
all the same.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>

---
 .../bindings/leds/backlight/kinetic,ktd253.yaml          | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

-- 
2.29.2

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Comments

Daniel Thompson March 22, 2021, 2:08 p.m. | #1
On Sun, Mar 21, 2021 at 12:43:40PM +0100, Linus Walleij wrote:
> The KTD259 is just a newer version of the KTD253 and works

> all the same.

> 

> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>


(For Lee) Acked-by: Daniel Thompson <daniel.thompson@linaro.org>


Daniel.

> ---

>  .../bindings/leds/backlight/kinetic,ktd253.yaml          | 9 ++++++---

>  1 file changed, 6 insertions(+), 3 deletions(-)

> 

> diff --git a/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml b/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml

> index 7a6ec1f8c0f3..73fa59e62181 100644

> --- a/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml

> +++ b/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml

> @@ -4,13 +4,13 @@

>  $id: http://devicetree.org/schemas/leds/backlight/kinetic,ktd253.yaml#

>  $schema: http://devicetree.org/meta-schemas/core.yaml#

>  

> -title: Kinetic Technologies KTD253 one-wire backlight

> +title: Kinetic Technologies KTD253 and KTD259 one-wire backlight

>  

>  maintainers:

>    - Linus Walleij <linus.walleij@linaro.org>

>  

>  description: |

> -  The Kinetic Technologies KTD253 is a white LED backlight that is

> +  The Kinetic Technologies KTD253 and KTD259 are white LED backlights

>    controlled by a single GPIO line. If you just turn on the backlight

>    it goes to maximum backlight then you can set the level of backlight

>    using pulses on the enable wire. This is sometimes referred to as

> @@ -21,7 +21,10 @@ allOf:

>  

>  properties:

>    compatible:

> -    const: kinetic,ktd253

> +    items:

> +      - enum:

> +          - kinetic,ktd253

> +          - kinetic,ktd259

>  

>    enable-gpios:

>      description: GPIO to use to enable/disable and dim the backlight.

> -- 

> 2.29.2

> 

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Lee Jones March 22, 2021, 2:32 p.m. | #2
On Sun, 21 Mar 2021, Linus Walleij wrote:

> The KTD259 is just a newer version of the KTD253 and works
> all the same.
> 
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
>  .../bindings/leds/backlight/kinetic,ktd253.yaml          | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)

Applied, thanks.

Patch

diff --git a/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml b/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml
index 7a6ec1f8c0f3..73fa59e62181 100644
--- a/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml
+++ b/Documentation/devicetree/bindings/leds/backlight/kinetic,ktd253.yaml
@@ -4,13 +4,13 @@ 
 $id: http://devicetree.org/schemas/leds/backlight/kinetic,ktd253.yaml#
 $schema: http://devicetree.org/meta-schemas/core.yaml#
 
-title: Kinetic Technologies KTD253 one-wire backlight
+title: Kinetic Technologies KTD253 and KTD259 one-wire backlight
 
 maintainers:
   - Linus Walleij <linus.walleij@linaro.org>
 
 description: |
-  The Kinetic Technologies KTD253 is a white LED backlight that is
+  The Kinetic Technologies KTD253 and KTD259 are white LED backlights
   controlled by a single GPIO line. If you just turn on the backlight
   it goes to maximum backlight then you can set the level of backlight
   using pulses on the enable wire. This is sometimes referred to as
@@ -21,7 +21,10 @@  allOf:
 
 properties:
   compatible:
-    const: kinetic,ktd253
+    items:
+      - enum:
+          - kinetic,ktd253
+          - kinetic,ktd259
 
   enable-gpios:
     description: GPIO to use to enable/disable and dim the backlight.