[PATCH 3/6] dt-bindings: usb: maxim,max33359: Add supply property for VBUS in OTG mode
Heikki Krogerus
heikki.krogerus at linux.intel.com
Wed Nov 26 02:01:08 PST 2025
Sun, Nov 23, 2025 at 08:35:50AM +0000, Amit Sunil Dhamne via B4 Relay kirjoitti:
> From: Amit Sunil Dhamne <amitsd at google.com>
>
> Add a regulator supply property for VBUS when usb is in OTG mode.
What is "OTG mode"?
OTG is usually used to refer to the USB in device role, even though the
specification actually defines OTG device as a device capable of both
host and device roles. So the term was confusing already before.
Nevertheless, the emphasis is always on data-role, _not_ power-role.
Here it seems MAX33359 uses the term OTG as a synonym for "source", so
power-role?
Please don't use the term OTG unless you really have to - it's too
confusing. I know the MAX33359 datasheet uses it, but what you really
do here is regulate VBUS. So please:
s/otg-vbus/vbus/
thanks,
> Signed-off-by: Amit Sunil Dhamne <amitsd at google.com>
> Reviewed-by: Badhri Jagan Sridharan <badhri at google.com>
> ---
> Documentation/devicetree/bindings/usb/maxim,max33359.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/usb/maxim,max33359.yaml b/Documentation/devicetree/bindings/usb/maxim,max33359.yaml
> index 3de4dc40b791..a529f18c4918 100644
> --- a/Documentation/devicetree/bindings/usb/maxim,max33359.yaml
> +++ b/Documentation/devicetree/bindings/usb/maxim,max33359.yaml
> @@ -32,6 +32,9 @@ properties:
> description:
> Properties for usb c connector.
>
> + otg-vbus-supply:
> + description: Regulator to control OTG VBUS supply.
> +
> required:
> - compatible
> - reg
> @@ -53,6 +56,7 @@ examples:
> reg = <0x25>;
> interrupt-parent = <&gpa8>;
> interrupts = <2 IRQ_TYPE_LEVEL_LOW>;
> + otg-vbus-supply = <&otg_vbus_reg>;
>
> connector {
> compatible = "usb-c-connector";
>
> --
> 2.52.0.rc2.455.g230fcf2819-goog
>
--
heikki
More information about the linux-arm-kernel
mailing list