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(-)
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.
The KTD259 works just like KTD253 so add this compatible to the driver.
Signed-off-by: Linus Walleij linus.walleij@linaro.org --- drivers/video/backlight/ktd253-backlight.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/video/backlight/ktd253-backlight.c b/drivers/video/backlight/ktd253-backlight.c index d7b287cffd5c..a7df5bcca9da 100644 --- a/drivers/video/backlight/ktd253-backlight.c +++ b/drivers/video/backlight/ktd253-backlight.c @@ -173,6 +173,7 @@ static int ktd253_backlight_probe(struct platform_device *pdev)
static const struct of_device_id ktd253_backlight_of_match[] = { { .compatible = "kinetic,ktd253" }, + { .compatible = "kinetic,ktd259" }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(of, ktd253_backlight_of_match);
On Sun, Mar 21, 2021 at 12:43:41PM +0100, Linus Walleij wrote:
The KTD259 works just like KTD253 so add this compatible to the driver.
Signed-off-by: Linus Walleij linus.walleij@linaro.org
Oh no... there's more of them ;-) .
Reviewed-by: Daniel Thompson daniel.thompson@linaro.org
Daniel.
drivers/video/backlight/ktd253-backlight.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/video/backlight/ktd253-backlight.c b/drivers/video/backlight/ktd253-backlight.c index d7b287cffd5c..a7df5bcca9da 100644 --- a/drivers/video/backlight/ktd253-backlight.c +++ b/drivers/video/backlight/ktd253-backlight.c @@ -173,6 +173,7 @@ static int ktd253_backlight_probe(struct platform_device *pdev)
static const struct of_device_id ktd253_backlight_of_match[] = { { .compatible = "kinetic,ktd253" },
- { .compatible = "kinetic,ktd259" }, { /* sentinel */ }
}; MODULE_DEVICE_TABLE(of, ktd253_backlight_of_match); -- 2.29.2
On Sun, 21 Mar 2021, Linus Walleij wrote:
The KTD259 works just like KTD253 so add this compatible to the driver.
Signed-off-by: Linus Walleij linus.walleij@linaro.org
drivers/video/backlight/ktd253-backlight.c | 1 + 1 file changed, 1 insertion(+)
Applied, thanks.
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
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.
dri-devel@lists.freedesktop.org