Re: [PATCH v4 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support
On Thu, Jun 16, 2022 at 12:57 PM Prashant Malani wrote:
>
> On Thu, Jun 16, 2022 at 12:34 PM Rob Herring wrote:
> >
> > On Thu, Jun 16, 2022 at 01:54:36AM -0700, Prashant Malani wrote:
> > > On Thu, Jun 16, 2022 at 12:42 AM Stephen Boyd wrote:
> > > >
> > > > Quoting Prashant Malani (2022-06-15 10:20:20)
> > > > >
> > > > > .../display/bridge/analogix,anx7625.yaml | 64
> > > > > +++
> > > > > 1 file changed, 64 insertions(+)
> > > >
> > > > Can this file get a link to the product brief[1]? It helps to quickly
> > > > find the block diagram.
> > >
> > > Sure, but I don't really think that should be included in this patch
> > > (or series).
> > > I'd be happy to submit a separate patch once this series is resolved.
> > >
> > > >
> > > > >
> > > > > diff --git
> > > > > a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > > >
> > > > > b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > > > index 35a48515836e..bc6f7644db31 100644
> > > > > ---
> > > > > a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > > > +++
> > > > > b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > > > @@ -105,6 +105,34 @@ properties:
> > > > >- port@0
> > > > >- port@1
> > > > >
> > > > > + switches:
> > > > > +type: object
> > > > > +description: Set of switches controlling DisplayPort traffic on
> > > > > + outgoing RX/TX lanes to Type C ports.
> > > > > +additionalProperties: false
> > > > > +
> > > > > +properties:
> > > > > + '#address-cells':
> > > > > +const: 1
> > > > > +
> > > > > + '#size-cells':
> > > > > +const: 0
> > > > > +
> > > > > +patternProperties:
> > > > > + '^switch@[01]$':
> > > > > +$ref: /schemas/usb/typec-switch.yaml#
> > > > > +unevaluatedProperties: false
> > > > > +
> > > > > +properties:
> > > > > + reg:
> > > > > +maxItems: 1
> > > > > +
> > > > > +required:
> > > > > + - reg
> > > > > +
> > > > > +required:
> > > > > + - switch@0
> > > > > +
> > > > > required:
> > > > >- compatible
> > > > >- reg
> > > > > @@ -167,5 +195,41 @@ examples:
> > > > > };
> > > > > };
> > > > > };
> > > > > +switches {
> > > >
> > > > Is "switches" a bus?
> > >
> > > No.
> > >
> > > >
> > > > > +#address-cells = <1>;
> > > > > +#size-cells = <0>;
> > > > > +switch@0 {
> > > > > +compatible = "typec-switch";
> > > >
> > > > Is this compatible matched against a driver that's populated on this
> > > > "switches" bus?
> > >
> > > No. Patch 6/7 has the implementation details on how the anx driver
> > > performs the enumeration of switches.
> > >
> > > >
> > > > > +reg = <0>;
> > > > > +mode-switch;
> > > > > +
> > > > > +ports {
> > > > > +#address-cells = <1>;
> > > > > +#size-cells = <0>;
> > > > > +port@0 {
> > > > > +reg = <0>;
> > > > > +anx_typec0: endpoint {
> > > > > +remote-endpoint = <_port0>;
> > > > > +};
> > > > > +};
> > > > > +};
> > > >
> > > > I was expecting to see these simply be more ports in the existing graph
> > > > binding of this device, and then have the 'mode-switch' or
> > > > 'orientation-switch' properties be at the same level as the compatible
> > > > string "analogix,anx7625". Here's the reasoning, based on looking at the
> > > > product brief and the existing binding/implementation.
> > > >
> > > > Looking at the only existing implementation of this binding upstream in
> > > > mt8183-kukui-jacuzzi.dtsi it looks like one of these typec ports is
> > > > actually the same physically as the 'anx7625_out' endpoint (reg address
> > > > of 1) that is already defined in the binding. It seems that MIPI DSI/DPI
> > > > comes in and is output through 2 lanes, SSRX2 and SSTX2 according to the
> > > > product brief[1], and that is connected to some eDP panel
> > > > ("auo,b116xw03"). Presumably that is the same as anx_typec1 in this
> > > > patch? I suspect the USB3.1 input is not connected on this board, and
> > > > thus the crosspoint switch is never used, nor the SSRX1/SSTX1 pins.
> > > >
> > > > The existing binding defines the MIPI DSI/DPI input as port0 and two of
> > > > the four lanes of output that is probably by default connected to the
> > > > "DisplayPort Transmitter" as port1 because that's how the crosspoint
> > > > switch comes out of reset. That leaves the USB3.1 input possibly needing
> > > > a port in the ports binding, and the other two lanes of output needing a
> > > > port in the ports
Re: [PATCH v4 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support
On Thu, Jun 16, 2022 at 12:34 PM Rob Herring wrote:
>
> On Thu, Jun 16, 2022 at 01:54:36AM -0700, Prashant Malani wrote:
> > On Thu, Jun 16, 2022 at 12:42 AM Stephen Boyd wrote:
> > >
> > > Quoting Prashant Malani (2022-06-15 10:20:20)
> > > >
> > > > .../display/bridge/analogix,anx7625.yaml | 64 +++
> > > > 1 file changed, 64 insertions(+)
> > >
> > > Can this file get a link to the product brief[1]? It helps to quickly
> > > find the block diagram.
> >
> > Sure, but I don't really think that should be included in this patch
> > (or series).
> > I'd be happy to submit a separate patch once this series is resolved.
> >
> > >
> > > >
> > > > diff --git
> > > > a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > >
> > > > b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > > index 35a48515836e..bc6f7644db31 100644
> > > > ---
> > > > a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > > +++
> > > > b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > > @@ -105,6 +105,34 @@ properties:
> > > >- port@0
> > > >- port@1
> > > >
> > > > + switches:
> > > > +type: object
> > > > +description: Set of switches controlling DisplayPort traffic on
> > > > + outgoing RX/TX lanes to Type C ports.
> > > > +additionalProperties: false
> > > > +
> > > > +properties:
> > > > + '#address-cells':
> > > > +const: 1
> > > > +
> > > > + '#size-cells':
> > > > +const: 0
> > > > +
> > > > +patternProperties:
> > > > + '^switch@[01]$':
> > > > +$ref: /schemas/usb/typec-switch.yaml#
> > > > +unevaluatedProperties: false
> > > > +
> > > > +properties:
> > > > + reg:
> > > > +maxItems: 1
> > > > +
> > > > +required:
> > > > + - reg
> > > > +
> > > > +required:
> > > > + - switch@0
> > > > +
> > > > required:
> > > >- compatible
> > > >- reg
> > > > @@ -167,5 +195,41 @@ examples:
> > > > };
> > > > };
> > > > };
> > > > +switches {
> > >
> > > Is "switches" a bus?
> >
> > No.
> >
> > >
> > > > +#address-cells = <1>;
> > > > +#size-cells = <0>;
> > > > +switch@0 {
> > > > +compatible = "typec-switch";
> > >
> > > Is this compatible matched against a driver that's populated on this
> > > "switches" bus?
> >
> > No. Patch 6/7 has the implementation details on how the anx driver
> > performs the enumeration of switches.
> >
> > >
> > > > +reg = <0>;
> > > > +mode-switch;
> > > > +
> > > > +ports {
> > > > +#address-cells = <1>;
> > > > +#size-cells = <0>;
> > > > +port@0 {
> > > > +reg = <0>;
> > > > +anx_typec0: endpoint {
> > > > +remote-endpoint = <_port0>;
> > > > +};
> > > > +};
> > > > +};
> > >
> > > I was expecting to see these simply be more ports in the existing graph
> > > binding of this device, and then have the 'mode-switch' or
> > > 'orientation-switch' properties be at the same level as the compatible
> > > string "analogix,anx7625". Here's the reasoning, based on looking at the
> > > product brief and the existing binding/implementation.
> > >
> > > Looking at the only existing implementation of this binding upstream in
> > > mt8183-kukui-jacuzzi.dtsi it looks like one of these typec ports is
> > > actually the same physically as the 'anx7625_out' endpoint (reg address
> > > of 1) that is already defined in the binding. It seems that MIPI DSI/DPI
> > > comes in and is output through 2 lanes, SSRX2 and SSTX2 according to the
> > > product brief[1], and that is connected to some eDP panel
> > > ("auo,b116xw03"). Presumably that is the same as anx_typec1 in this
> > > patch? I suspect the USB3.1 input is not connected on this board, and
> > > thus the crosspoint switch is never used, nor the SSRX1/SSTX1 pins.
> > >
> > > The existing binding defines the MIPI DSI/DPI input as port0 and two of
> > > the four lanes of output that is probably by default connected to the
> > > "DisplayPort Transmitter" as port1 because that's how the crosspoint
> > > switch comes out of reset. That leaves the USB3.1 input possibly needing
> > > a port in the ports binding, and the other two lanes of output needing a
> > > port in the ports binding to describe their connection to the downstream
> > > device. And finally information about if the crosspoint switch needs to
> > > be registered with the typec framework to do typec things, which can be
> > > achieved by the presence of the 'mode-switch' property.
> > >
> > > On a board like kukui-jacuzzi
Re: [PATCH v4 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support
On Thu, Jun 16, 2022 at 01:54:36AM -0700, Prashant Malani wrote:
> On Thu, Jun 16, 2022 at 12:42 AM Stephen Boyd wrote:
> >
> > Quoting Prashant Malani (2022-06-15 10:20:20)
> > >
> > > .../display/bridge/analogix,anx7625.yaml | 64 +++
> > > 1 file changed, 64 insertions(+)
> >
> > Can this file get a link to the product brief[1]? It helps to quickly
> > find the block diagram.
>
> Sure, but I don't really think that should be included in this patch
> (or series).
> I'd be happy to submit a separate patch once this series is resolved.
>
> >
> > >
> > > diff --git
> > > a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > index 35a48515836e..bc6f7644db31 100644
> > > ---
> > > a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > +++
> > > b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > > @@ -105,6 +105,34 @@ properties:
> > >- port@0
> > >- port@1
> > >
> > > + switches:
> > > +type: object
> > > +description: Set of switches controlling DisplayPort traffic on
> > > + outgoing RX/TX lanes to Type C ports.
> > > +additionalProperties: false
> > > +
> > > +properties:
> > > + '#address-cells':
> > > +const: 1
> > > +
> > > + '#size-cells':
> > > +const: 0
> > > +
> > > +patternProperties:
> > > + '^switch@[01]$':
> > > +$ref: /schemas/usb/typec-switch.yaml#
> > > +unevaluatedProperties: false
> > > +
> > > +properties:
> > > + reg:
> > > +maxItems: 1
> > > +
> > > +required:
> > > + - reg
> > > +
> > > +required:
> > > + - switch@0
> > > +
> > > required:
> > >- compatible
> > >- reg
> > > @@ -167,5 +195,41 @@ examples:
> > > };
> > > };
> > > };
> > > +switches {
> >
> > Is "switches" a bus?
>
> No.
>
> >
> > > +#address-cells = <1>;
> > > +#size-cells = <0>;
> > > +switch@0 {
> > > +compatible = "typec-switch";
> >
> > Is this compatible matched against a driver that's populated on this
> > "switches" bus?
>
> No. Patch 6/7 has the implementation details on how the anx driver
> performs the enumeration of switches.
>
> >
> > > +reg = <0>;
> > > +mode-switch;
> > > +
> > > +ports {
> > > +#address-cells = <1>;
> > > +#size-cells = <0>;
> > > +port@0 {
> > > +reg = <0>;
> > > +anx_typec0: endpoint {
> > > +remote-endpoint = <_port0>;
> > > +};
> > > +};
> > > +};
> >
> > I was expecting to see these simply be more ports in the existing graph
> > binding of this device, and then have the 'mode-switch' or
> > 'orientation-switch' properties be at the same level as the compatible
> > string "analogix,anx7625". Here's the reasoning, based on looking at the
> > product brief and the existing binding/implementation.
> >
> > Looking at the only existing implementation of this binding upstream in
> > mt8183-kukui-jacuzzi.dtsi it looks like one of these typec ports is
> > actually the same physically as the 'anx7625_out' endpoint (reg address
> > of 1) that is already defined in the binding. It seems that MIPI DSI/DPI
> > comes in and is output through 2 lanes, SSRX2 and SSTX2 according to the
> > product brief[1], and that is connected to some eDP panel
> > ("auo,b116xw03"). Presumably that is the same as anx_typec1 in this
> > patch? I suspect the USB3.1 input is not connected on this board, and
> > thus the crosspoint switch is never used, nor the SSRX1/SSTX1 pins.
> >
> > The existing binding defines the MIPI DSI/DPI input as port0 and two of
> > the four lanes of output that is probably by default connected to the
> > "DisplayPort Transmitter" as port1 because that's how the crosspoint
> > switch comes out of reset. That leaves the USB3.1 input possibly needing
> > a port in the ports binding, and the other two lanes of output needing a
> > port in the ports binding to describe their connection to the downstream
> > device. And finally information about if the crosspoint switch needs to
> > be registered with the typec framework to do typec things, which can be
> > achieved by the presence of the 'mode-switch' property.
> >
> > On a board like kukui-jacuzzi these new properties and ports wouldn't be
> > specified, because what is there is already sufficient. If this chip is
> > connected to a usb-c-connector then I'd expect to see a connection from
> > the output ports in the graph binding to the connector node's ports.
> > There aren't any ports in the
Re: [PATCH v4 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support
On Thu, Jun 16, 2022 at 12:42 AM Stephen Boyd wrote:
>
> Quoting Prashant Malani (2022-06-15 10:20:20)
> >
> > .../display/bridge/analogix,anx7625.yaml | 64 +++
> > 1 file changed, 64 insertions(+)
>
> Can this file get a link to the product brief[1]? It helps to quickly
> find the block diagram.
Sure, but I don't really think that should be included in this patch
(or series).
I'd be happy to submit a separate patch once this series is resolved.
>
> >
> > diff --git
> > a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > index 35a48515836e..bc6f7644db31 100644
> > --- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > +++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> > @@ -105,6 +105,34 @@ properties:
> >- port@0
> >- port@1
> >
> > + switches:
> > +type: object
> > +description: Set of switches controlling DisplayPort traffic on
> > + outgoing RX/TX lanes to Type C ports.
> > +additionalProperties: false
> > +
> > +properties:
> > + '#address-cells':
> > +const: 1
> > +
> > + '#size-cells':
> > +const: 0
> > +
> > +patternProperties:
> > + '^switch@[01]$':
> > +$ref: /schemas/usb/typec-switch.yaml#
> > +unevaluatedProperties: false
> > +
> > +properties:
> > + reg:
> > +maxItems: 1
> > +
> > +required:
> > + - reg
> > +
> > +required:
> > + - switch@0
> > +
> > required:
> >- compatible
> >- reg
> > @@ -167,5 +195,41 @@ examples:
> > };
> > };
> > };
> > +switches {
>
> Is "switches" a bus?
No.
>
> > +#address-cells = <1>;
> > +#size-cells = <0>;
> > +switch@0 {
> > +compatible = "typec-switch";
>
> Is this compatible matched against a driver that's populated on this
> "switches" bus?
No. Patch 6/7 has the implementation details on how the anx driver
performs the enumeration of switches.
>
> > +reg = <0>;
> > +mode-switch;
> > +
> > +ports {
> > +#address-cells = <1>;
> > +#size-cells = <0>;
> > +port@0 {
> > +reg = <0>;
> > +anx_typec0: endpoint {
> > +remote-endpoint = <_port0>;
> > +};
> > +};
> > +};
>
> I was expecting to see these simply be more ports in the existing graph
> binding of this device, and then have the 'mode-switch' or
> 'orientation-switch' properties be at the same level as the compatible
> string "analogix,anx7625". Here's the reasoning, based on looking at the
> product brief and the existing binding/implementation.
>
> Looking at the only existing implementation of this binding upstream in
> mt8183-kukui-jacuzzi.dtsi it looks like one of these typec ports is
> actually the same physically as the 'anx7625_out' endpoint (reg address
> of 1) that is already defined in the binding. It seems that MIPI DSI/DPI
> comes in and is output through 2 lanes, SSRX2 and SSTX2 according to the
> product brief[1], and that is connected to some eDP panel
> ("auo,b116xw03"). Presumably that is the same as anx_typec1 in this
> patch? I suspect the USB3.1 input is not connected on this board, and
> thus the crosspoint switch is never used, nor the SSRX1/SSTX1 pins.
>
> The existing binding defines the MIPI DSI/DPI input as port0 and two of
> the four lanes of output that is probably by default connected to the
> "DisplayPort Transmitter" as port1 because that's how the crosspoint
> switch comes out of reset. That leaves the USB3.1 input possibly needing
> a port in the ports binding, and the other two lanes of output needing a
> port in the ports binding to describe their connection to the downstream
> device. And finally information about if the crosspoint switch needs to
> be registered with the typec framework to do typec things, which can be
> achieved by the presence of the 'mode-switch' property.
>
> On a board like kukui-jacuzzi these new properties and ports wouldn't be
> specified, because what is there is already sufficient. If this chip is
> connected to a usb-c-connector then I'd expect to see a connection from
> the output ports in the graph binding to the connector node's ports.
> There aren't any ports in the usb-c-connector binding though from what I
> see.
>
> I believe there's also one more use case here where USB3.1 or MIPI
> DSI/DPI is connected on the input side and this device is used to steer
> USB3.1 or DP through the crosspoint switch to either of the two output
> pairs. This last scenario means that we have to describe both
Re: [PATCH v4 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support
Quoting Prashant Malani (2022-06-15 10:20:20)
>
> .../display/bridge/analogix,anx7625.yaml | 64 +++
> 1 file changed, 64 insertions(+)
Can this file get a link to the product brief[1]? It helps to quickly
find the block diagram.
>
> diff --git
> a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> index 35a48515836e..bc6f7644db31 100644
> --- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> +++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
> @@ -105,6 +105,34 @@ properties:
>- port@0
>- port@1
>
> + switches:
> +type: object
> +description: Set of switches controlling DisplayPort traffic on
> + outgoing RX/TX lanes to Type C ports.
> +additionalProperties: false
> +
> +properties:
> + '#address-cells':
> +const: 1
> +
> + '#size-cells':
> +const: 0
> +
> +patternProperties:
> + '^switch@[01]$':
> +$ref: /schemas/usb/typec-switch.yaml#
> +unevaluatedProperties: false
> +
> +properties:
> + reg:
> +maxItems: 1
> +
> +required:
> + - reg
> +
> +required:
> + - switch@0
> +
> required:
>- compatible
>- reg
> @@ -167,5 +195,41 @@ examples:
> };
> };
> };
> +switches {
Is "switches" a bus?
> +#address-cells = <1>;
> +#size-cells = <0>;
> +switch@0 {
> +compatible = "typec-switch";
Is this compatible matched against a driver that's populated on this
"switches" bus?
> +reg = <0>;
> +mode-switch;
> +
> +ports {
> +#address-cells = <1>;
> +#size-cells = <0>;
> +port@0 {
> +reg = <0>;
> +anx_typec0: endpoint {
> +remote-endpoint = <_port0>;
> +};
> +};
> +};
I was expecting to see these simply be more ports in the existing graph
binding of this device, and then have the 'mode-switch' or
'orientation-switch' properties be at the same level as the compatible
string "analogix,anx7625". Here's the reasoning, based on looking at the
product brief and the existing binding/implementation.
Looking at the only existing implementation of this binding upstream in
mt8183-kukui-jacuzzi.dtsi it looks like one of these typec ports is
actually the same physically as the 'anx7625_out' endpoint (reg address
of 1) that is already defined in the binding. It seems that MIPI DSI/DPI
comes in and is output through 2 lanes, SSRX2 and SSTX2 according to the
product brief[1], and that is connected to some eDP panel
("auo,b116xw03"). Presumably that is the same as anx_typec1 in this
patch? I suspect the USB3.1 input is not connected on this board, and
thus the crosspoint switch is never used, nor the SSRX1/SSTX1 pins.
The existing binding defines the MIPI DSI/DPI input as port0 and two of
the four lanes of output that is probably by default connected to the
"DisplayPort Transmitter" as port1 because that's how the crosspoint
switch comes out of reset. That leaves the USB3.1 input possibly needing
a port in the ports binding, and the other two lanes of output needing a
port in the ports binding to describe their connection to the downstream
device. And finally information about if the crosspoint switch needs to
be registered with the typec framework to do typec things, which can be
achieved by the presence of the 'mode-switch' property.
On a board like kukui-jacuzzi these new properties and ports wouldn't be
specified, because what is there is already sufficient. If this chip is
connected to a usb-c-connector then I'd expect to see a connection from
the output ports in the graph binding to the connector node's ports.
There aren't any ports in the usb-c-connector binding though from what I
see.
I believe there's also one more use case here where USB3.1 or MIPI
DSI/DPI is connected on the input side and this device is used to steer
USB3.1 or DP through the crosspoint switch to either of the two output
pairs. This last scenario means that we have to describe both output
pairs, SSRX1/SSTX1 and SSRX2/SSTX2, as different ports in the binding so
they can be connected to different usb-c-connectors if the hardware
engineer wired the output pins that way.
TL;DR: Can we add 'mode-switch' as an optional property and two more
ports at address 2 and 3 for the USB3.1 input and the SSRX1/SSTX1 pair
respectively to the existing graph part of this binding?
> +};
> +switch@1 {
> +compatible = "typec-switch";
> +reg = <1>;
> +
Re: [PATCH v4 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support
On 15/06/2022 10:20, Prashant Malani wrote: > Analogix 7625 can be used in systems to switch USB Type-C DisplayPort > alternate mode lane traffic between 2 Type-C ports. > > Update the binding to accommodate this usage by introducing a switch > property. > > Reviewed-by: Nícolas F. R. A. Prado > Tested-by: Nícolas F. R. A. Prado > Signed-off-by: Prashant Malani > --- > > Changes since v3: > - Fix unevaluatedProperties usage. > - Add additionalProperties to top level "switches" nodes. > - Make quotes consistent. > - Add '^..$' to regex. > (All suggested by Krzysztof Kozlowski) Looks good to me. Reviewed-by: Krzysztof Kozlowski Best regards, Krzysztof
[PATCH v4 4/7] dt-bindings: drm/bridge: anx7625: Add mode-switch support
Analogix 7625 can be used in systems to switch USB Type-C DisplayPort
alternate mode lane traffic between 2 Type-C ports.
Update the binding to accommodate this usage by introducing a switch
property.
Reviewed-by: Nícolas F. R. A. Prado
Tested-by: Nícolas F. R. A. Prado
Signed-off-by: Prashant Malani
---
Changes since v3:
- Fix unevaluatedProperties usage.
- Add additionalProperties to top level "switches" nodes.
- Make quotes consistent.
- Add '^..$' to regex.
(All suggested by Krzysztof Kozlowski)
Changes since v2:
- Added Reviewed-by and Tested-by tags.
Changes since v1:
- Introduced patternProperties for "switch" children (suggested by
Krzysztof Kozlowski).
- Added unevaluatedProperties descriptor (suggested by Krzysztof
Kozlowski).
- Added "address-cells" and "size-cells" properties to "switches".
.../display/bridge/analogix,anx7625.yaml | 64 +++
1 file changed, 64 insertions(+)
diff --git
a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
index 35a48515836e..bc6f7644db31 100644
--- a/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/analogix,anx7625.yaml
@@ -105,6 +105,34 @@ properties:
- port@0
- port@1
+ switches:
+type: object
+description: Set of switches controlling DisplayPort traffic on
+ outgoing RX/TX lanes to Type C ports.
+additionalProperties: false
+
+properties:
+ '#address-cells':
+const: 1
+
+ '#size-cells':
+const: 0
+
+patternProperties:
+ '^switch@[01]$':
+$ref: /schemas/usb/typec-switch.yaml#
+unevaluatedProperties: false
+
+properties:
+ reg:
+maxItems: 1
+
+required:
+ - reg
+
+required:
+ - switch@0
+
required:
- compatible
- reg
@@ -167,5 +195,41 @@ examples:
};
};
};
+switches {
+#address-cells = <1>;
+#size-cells = <0>;
+switch@0 {
+compatible = "typec-switch";
+reg = <0>;
+mode-switch;
+
+ports {
+#address-cells = <1>;
+#size-cells = <0>;
+port@0 {
+reg = <0>;
+anx_typec0: endpoint {
+remote-endpoint = <_port0>;
+};
+};
+};
+};
+switch@1 {
+compatible = "typec-switch";
+reg = <1>;
+mode-switch;
+
+ports {
+#address-cells = <1>;
+#size-cells = <0>;
+port@0 {
+reg = <0>;
+anx_typec1: endpoint {
+remote-endpoint = <_port1>;
+};
+};
+};
+};
+};
};
};
--
2.36.1.476.g0c4daa206d-goog
