[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Wed, 6 May 2020 00:38:13 +0300
From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
To: Douglas Anderson <dianders@...omium.org>
Cc: linus.walleij@...aro.org, bgolaszewski@...libre.com,
airlied@...ux.ie, daniel@...ll.ch, robh+dt@...nel.org,
narmstrong@...libre.com, a.hajda@...sung.com,
spanda@...eaurora.org, linux-gpio@...r.kernel.org, jonas@...boo.se,
robdclark@...omium.org, bjorn.andersson@...aro.org,
jeffrey.l.hugo@...il.com, jernej.skrabec@...l.net,
dri-devel@...ts.freedesktop.org, linux-arm-msm@...r.kernel.org,
devicetree@...r.kernel.org, swboyd@...omium.org,
Sam Ravnborg <sam@...nborg.org>,
Thierry Reding <thierry.reding@...il.com>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 2/6] dt-bindings: display: Add hpd-gpios to
panel-common bindings
Hi Doug,
Thank you for the patch.
On Thu, Apr 30, 2020 at 12:46:13PM -0700, Douglas Anderson 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/20200417180819.GE5861@pendragon.ideasonboard.com
>
> Signed-off-by: Douglas Anderson <dianders@...omium.org>
> Reviewed-by: Stephen Boyd <swboyd@...omium.org>
> Reviewed-by: Linus Walleij <linus.walleij@...aro.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> ---
>
> Changes in v4: None
> 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
--
Regards,
Laurent Pinchart
Powered by blists - more mailing lists