| Message ID | 20181229070649.21308-4-peda@axentia.se |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | drm/bridge: various small lvds-encoder things | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success |
On Sat, 29 Dec 2018 07:07:31 +0000, Peter Rosin wrote: > The name powerdown-gpios is the standard property name for the > functionality covered by the previous pwdn-gpios name. This rename > should be safe to do since the linux driver supporting the binding > (lvds-encoder.c) never implemented the property, and no dts file > names it. At least not upstream. > > Signed-off-by: Peter Rosin <peda@axentia.se> > --- > Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Rob Herring <robh@kernel.org>
Hi Peter, Thank you for the patch. On Sat, Dec 29, 2018 at 07:07:31AM +0000, Peter Rosin wrote: > The name powerdown-gpios is the standard property name for the > functionality covered by the previous pwdn-gpios name. This rename > should be safe to do since the linux driver supporting the binding > (lvds-encoder.c) never implemented the property, and no dts file > names it. At least not upstream. > > Signed-off-by: Peter Rosin <peda@axentia.se> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > --- > Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt > index 527e236e9a2a..fee3c88e1a17 100644 > --- a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt > +++ b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt > @@ -10,7 +10,7 @@ Required properties: > > Optional properties: > > -- pwdn-gpios: Power down control GPIO > +- powerdown-gpios: Power down control GPIO (the /PWDN pin, active low). > > Required nodes: >
diff --git a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt index 527e236e9a2a..fee3c88e1a17 100644 --- a/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt +++ b/Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt @@ -10,7 +10,7 @@ Required properties: Optional properties: -- pwdn-gpios: Power down control GPIO +- powerdown-gpios: Power down control GPIO (the /PWDN pin, active low). Required nodes:
The name powerdown-gpios is the standard property name for the functionality covered by the previous pwdn-gpios name. This rename should be safe to do since the linux driver supporting the binding (lvds-encoder.c) never implemented the property, and no dts file names it. At least not upstream. Signed-off-by: Peter Rosin <peda@axentia.se> --- Documentation/devicetree/bindings/display/bridge/thine,thc63lvdm83d.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)