Re: [PATCH v2 1/1] dt-bindings: connector: Add role‑switch provider phandle

[Dmitry Baryshkov]

On Tue, 24 Mar 2026 at 19:29, Elson Serrao
<elson.serrao@xxxxxxxxxxxxxxxx> wrote:
>
> Add an optional consumer→provider phandle on USB connectors to reference
> the USB role-switch provider when no direct graph link exists. The DRD
> controller remains the provider via its 'usb-role-switch' property.
>
> Signed-off-by: Elson Serrao <elson.serrao@xxxxxxxxxxxxxxxx>
> ---
>  .../devicetree/bindings/connector/usb-connector.yaml      | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/connector/usb-connector.yaml b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> index 11e40d225b9f..ef8d3d26461b 100644
> --- a/Documentation/devicetree/bindings/connector/usb-connector.yaml
> +++ b/Documentation/devicetree/bindings/connector/usb-connector.yaml
> @@ -95,6 +95,14 @@ properties:
>        - device
>        - dual
>
> +  usb-role-switch:
> +    $ref: /schemas/types.yaml#/definitions/phandle
> +    description:
> +      A phandle to the USB role-switch provider. The provider is typically
> +      a dual-role (DRD) USB controller node that declares the boolean
> +      'usb-role-switch' property. Use this when the connector is not
> +      directly linked to the provider in the OF graph.

This kind of properties has already been discussed at least twice:
- First for referencing the DisplayPort controller from the AltMode
node (USB-C connector)
- Then for referencing the USB-C muxes / orientation switches when
there is an intermediate chip

In both cases the agreement was to not add such out of band handle
references. The solution was on the software side, letting drivers for
the intermediate devices in the OF graph translate and resend
necessary events.

Previously we didn't have such an issue for the usb-role-switch,
because there always have been a direct link between the USB connector
(be it gpio-usb-b-connector or usb-c-connector) and the USB controller
(implementing usb-role-switch). As with the EUD this is no longer a
case, my suggestion would be to follow prior art and let EUD receive,
interpret and resend usb-role-switch events.

> +
>    typec-power-opmode:
>      description: Determines the power operation mode that the Type C connector
>        will support and will advertise through CC pins when it has no power
> --
> 2.34.1
>


-- 
With best wishes
Dmitry