2023-10-11 10:48:02

by Luca Ceresoli

[permalink] [raw]
Subject: [PATCH] dt-bindings: display: remove backlight node from panel examples

The examples for these panel drivers have a backlight node in addition to
the actual panel node. However the exact backlight is outside the scope of
this binding and should be dropped from the example.

Link: https://lore.kernel.org/linux-devicetree/[email protected]/
Suggested-by: Rob Herring <[email protected]>
Signed-off-by: Luca Ceresoli <[email protected]>
---
Documentation/devicetree/bindings/display/ilitek,ili9486.yaml | 4 ----
Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml | 4 ----
Documentation/devicetree/bindings/display/sitronix,st7735r.yaml | 5 -----
3 files changed, 13 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml b/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml
index 1f8f2182e2f1..9cc1fd0751cd 100644
--- a/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml
+++ b/Documentation/devicetree/bindings/display/ilitek,ili9486.yaml
@@ -50,10 +50,6 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>

- backlight: backlight {
- compatible = "gpio-backlight";
- gpios = <&gpio 22 GPIO_ACTIVE_HIGH>;
- };
spi {
#address-cells = <1>;
#size-cells = <0>;
diff --git a/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml b/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml
index 90e323e19edb..3cabbba86581 100644
--- a/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml
+++ b/Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml
@@ -48,10 +48,6 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>

- backlight: backlight {
- compatible = "gpio-backlight";
- gpios = <&gpio 22 GPIO_ACTIVE_HIGH>;
- };
spi {
#address-cells = <1>;
#size-cells = <0>;
diff --git a/Documentation/devicetree/bindings/display/sitronix,st7735r.yaml b/Documentation/devicetree/bindings/display/sitronix,st7735r.yaml
index 621f27148419..3b0ebc0db8e0 100644
--- a/Documentation/devicetree/bindings/display/sitronix,st7735r.yaml
+++ b/Documentation/devicetree/bindings/display/sitronix,st7735r.yaml
@@ -54,11 +54,6 @@ examples:
- |
#include <dt-bindings/gpio/gpio.h>

- backlight: backlight {
- compatible = "gpio-backlight";
- gpios = <&gpio 44 GPIO_ACTIVE_HIGH>;
- };
-
spi {
#address-cells = <1>;
#size-cells = <0>;

---
base-commit: 2430fa6470d5e76be39a3e0d6d01474234582f94
change-id: 20231008-dt-panel-example-no-backlight-cfdaeb861b97

Best regards,
--
Luca Ceresoli <[email protected]>


2023-10-11 14:07:07

by David Lechner

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: display: remove backlight node from panel examples

On 10/11/23 5:47 AM, Luca Ceresoli wrote:
> The examples for these panel drivers have a backlight node in addition to
> the actual panel node. However the exact backlight is outside the scope of
> this binding and should be dropped from the example.
>
> Link: https://lore.kernel.org/linux-devicetree/[email protected]/
> Suggested-by: Rob Herring <[email protected]>
> Signed-off-by: Luca Ceresoli <[email protected]>
> ---

Acked-by: David Lechner <[email protected]>


2023-10-17 18:44:57

by Rob Herring

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: display: remove backlight node from panel examples


On Wed, 11 Oct 2023 12:47:38 +0200, Luca Ceresoli wrote:
> The examples for these panel drivers have a backlight node in addition to
> the actual panel node. However the exact backlight is outside the scope of
> this binding and should be dropped from the example.
>
> Link: https://lore.kernel.org/linux-devicetree/[email protected]/
> Suggested-by: Rob Herring <[email protected]>
> Signed-off-by: Luca Ceresoli <[email protected]>
> ---
> Documentation/devicetree/bindings/display/ilitek,ili9486.yaml | 4 ----
> Documentation/devicetree/bindings/display/panel/ilitek,ili9163.yaml | 4 ----
> Documentation/devicetree/bindings/display/sitronix,st7735r.yaml | 5 -----
> 3 files changed, 13 deletions(-)
>

Applied, thanks!