* [PATCH] dt-bindings: display: remove backlight node from panel examples
@ 2023-10-11 10:47 Luca Ceresoli
2023-10-11 14:06 ` David Lechner
2023-10-17 18:44 ` Rob Herring
0 siblings, 2 replies; 3+ messages in thread
From: Luca Ceresoli @ 2023-10-11 10:47 UTC (permalink / raw)
To: Kamlesh Gurudasani, David Airlie, Daniel Vetter, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, Neil Armstrong, Sam Ravnborg,
David Lechner, Daniel Mack
Cc: Thomas Petazzoni, dri-devel, devicetree, linux-kernel,
Rob Herring, Luca Ceresoli
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/20230724143152.GA3430423-robh@kernel.org/
Suggested-by: Rob Herring <robh@kernel.org>
Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
---
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 <luca.ceresoli@bootlin.com>
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: display: remove backlight node from panel examples
2023-10-11 10:47 [PATCH] dt-bindings: display: remove backlight node from panel examples Luca Ceresoli
@ 2023-10-11 14:06 ` David Lechner
2023-10-17 18:44 ` Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: David Lechner @ 2023-10-11 14:06 UTC (permalink / raw)
To: Luca Ceresoli, Kamlesh Gurudasani, David Airlie, Daniel Vetter,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, Neil Armstrong,
Sam Ravnborg, Daniel Mack
Cc: Thomas Petazzoni, dri-devel, devicetree, linux-kernel,
Rob Herring
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/20230724143152.GA3430423-robh@kernel.org/
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> ---
Acked-by: David Lechner <david@lechnology.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: display: remove backlight node from panel examples
2023-10-11 10:47 [PATCH] dt-bindings: display: remove backlight node from panel examples Luca Ceresoli
2023-10-11 14:06 ` David Lechner
@ 2023-10-17 18:44 ` Rob Herring
1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring @ 2023-10-17 18:44 UTC (permalink / raw)
To: Luca Ceresoli
Cc: David Airlie, Neil Armstrong, Sam Ravnborg, devicetree,
Kamlesh Gurudasani, Daniel Vetter, linux-kernel, David Lechner,
Daniel Mack, Rob Herring, dri-devel, Thomas Petazzoni,
Conor Dooley, Krzysztof Kozlowski
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/20230724143152.GA3430423-robh@kernel.org/
> Suggested-by: Rob Herring <robh@kernel.org>
> Signed-off-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> ---
> 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!
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-10-17 18:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-10-11 10:47 [PATCH] dt-bindings: display: remove backlight node from panel examples Luca Ceresoli
2023-10-11 14:06 ` David Lechner
2023-10-17 18:44 ` Rob Herring
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).