2021-04-14 18:08:32

by Rajeev Nandan

[permalink] [raw]
Subject: [v1 2/3] dt-bindings: drm/bridge: ti-sn65dsi86: Document use-aux-backlight

If the panel connected to the bridge supports backlight control
using DPCD registers on the DisplayPort aux channel, setting
"use-aux-backlight" property in the bridge node will enable the
registration of a DP aux backlight device from the bridge driver.

Signed-off-by: Rajeev Nandan <[email protected]>
---
.../devicetree/bindings/display/bridge/ti,sn65dsi86.yaml | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml
index 26932d2..c8d8c00 100644
--- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml
@@ -58,6 +58,12 @@ properties:
clock-names:
const: refclk

+ use-aux-backlight:
+ type: boolean
+ description:
+ The panel backlight to be controlled using DPCD registers on
+ the DP aux channel.
+
gpio-controller: true
'#gpio-cells':
const: 2
@@ -188,6 +194,8 @@ examples:

no-hpd;

+ use-aux-backlight;
+
ports {
#address-cells = <1>;
#size-cells = <0>;
--
2.7.4


2021-04-15 23:20:32

by Rob Herring (Arm)

[permalink] [raw]
Subject: Re: [v1 2/3] dt-bindings: drm/bridge: ti-sn65dsi86: Document use-aux-backlight

On Wed, Apr 14, 2021 at 10:09:49PM +0530, Rajeev Nandan wrote:
> If the panel connected to the bridge supports backlight control
> using DPCD registers on the DisplayPort aux channel, setting
> "use-aux-backlight" property in the bridge node will enable the
> registration of a DP aux backlight device from the bridge driver.
>
> Signed-off-by: Rajeev Nandan <[email protected]>
> ---
> .../devicetree/bindings/display/bridge/ti,sn65dsi86.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml
> index 26932d2..c8d8c00 100644
> --- a/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/ti,sn65dsi86.yaml
> @@ -58,6 +58,12 @@ properties:
> clock-names:
> const: refclk
>
> + use-aux-backlight:

use-dp-aux-backlight perhaps.

> + type: boolean
> + description:
> + The panel backlight to be controlled using DPCD registers on
> + the DP aux channel.

Sounds like a property of the panel, not the bridge. So it should be in
the panel node.

Rob