diff mbox series

dt-bindings: display/bridge: document enable-panel for, sn65dsi83

Message ID 131ea94b-2079-3214-3ab9-f2129659cb76@axentia.se
State New
Headers show
Series dt-bindings: display/bridge: document enable-panel for, sn65dsi83 | expand

Commit Message

Daniels Umanovskis Aug. 25, 2021, 10:21 a.m. UTC
The driver supports an optional GPIO specifier for a pin that
enables the connected panel, but it's not documented in dt-bindings

Signed-off-by: Daniels Umanovskis <du@axentia.se>
---
  .../devicetree/bindings/display/bridge/ti,sn65dsi83.yaml      | 4 ++++
  1 file changed, 4 insertions(+)

Comments

Daniels Umanovskis Aug. 26, 2021, 11:12 a.m. UTC | #1
On 8/25/21 6:26 PM, Rob Herring wrote:
>> Signed-off-by: Daniels Umanovskis <du@axentia.se>

>> ---

>>   .../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.

Could you clarify if you're suggesting a change to the driver instead?
The driver uses enable-gpios to specify one GPIO (the bridge's EN pin),
and optionally enable-panel-gpios for the panel. The patch was intended
to document that behavior.
diff mbox series

Patch

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).
+
    ports:
      $ref: /schemas/graph.yaml#/properties/ports