Re: [PATCH] dt-bindings: display/bridge: document enable-panel for, sn65dsi83

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 25, 2021 at 12:21:44PM +0200, Daniels Umanovskis wrote:
> The driver supports an optional GPIO specifier for a pin that
> enables the connected panel, but it's not documented in dt-bindings

The subject looks mangled with the ',' and 'ti' is missing.

(There's a common issue with a space getting inserted after a comma on 
some maillists like dridevel, but the DT list doesn't have that issue.)

> 
> Signed-off-by: Daniels Umanovskis <du@xxxxxxxxxx>
> ---
>  .../devicetree/bindings/display/bridge/ti,sn65dsi83.yaml      | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git
> a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> index d101233ae17..1aca6c1a8e0 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi83.yaml
> @@ -32,6 +32,10 @@ properties:
>      maxItems: 1
>      description: GPIO specifier for bridge_en pin (active high).
> 
> +  enable-panel-gpios:
> +    maxItems: 1
> +    description: GPIO specifier for the panel's enable pin (active high).
> +

There's already the common 'enable-gpios' property. Use that.

>    ports:
>      $ref: /schemas/graph.yaml#/properties/ports
> 
> -- 
> 2.30.2
> 
> 



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux