2020-04-23 16:30:05

by Doug Anderson

[permalink] [raw]
Subject: [PATCH v3 2/6] dt-bindings: display: Add hpd-gpios to panel-common bindings

In the cases where there is no connector in a system there's no great
place to put "hpd-gpios". As per discussion [1] the best place to put
it is in the panel. Add this to the device tree bindings.

[1] https://lore.kernel.org/r/[email protected]

Signed-off-by: Douglas Anderson <[email protected]>
Reviewed-by: Stephen Boyd <[email protected]>
---

Changes in v3: None
Changes in v2:
- ("dt-bindings: display: Add hpd-gpios to panel-common...") new for v2

.../devicetree/bindings/display/panel/panel-common.yaml | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/panel/panel-common.yaml b/Documentation/devicetree/bindings/display/panel/panel-common.yaml
index ed051ba12084..e9a04a3a4f5f 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-common.yaml
+++ b/Documentation/devicetree/bindings/display/panel/panel-common.yaml
@@ -96,6 +96,12 @@ properties:
(hot plug detect) signal, but the signal isn't hooked up so we should
hardcode the max delay from the panel spec when powering up the panel.

+ hpd-gpios:
+ maxItems: 1
+ description:
+ If Hot Plug Detect (HPD) is connected to a GPIO in the system rather
+ than a dedicated HPD pin the pin can be specified here.
+
# Control I/Os

# Many display panels can be controlled through pins driven by GPIOs. The nature
--
2.26.1.301.g55bc3eb7cb9-goog


2020-04-28 12:19:41

by Linus Walleij

[permalink] [raw]
Subject: Re: [PATCH v3 2/6] dt-bindings: display: Add hpd-gpios to panel-common bindings

On Thu, Apr 23, 2020 at 6:26 PM Douglas Anderson <[email protected]> wrote:

> In the cases where there is no connector in a system there's no great
> place to put "hpd-gpios". As per discussion [1] the best place to put
> it is in the panel. Add this to the device tree bindings.
>
> [1] https://lore.kernel.org/r/[email protected]
>
> Signed-off-by: Douglas Anderson <[email protected]>
> Reviewed-by: Stephen Boyd <[email protected]>

Reviewed-by: Linus Walleij <[email protected]>

Yours,
Linus Walleij