Re: [PATCH v3 1/3] dt-bindings: usb: usb251xb: support usage case without I2C control

[Krzysztof Kozlowski <krzk@kernel.org>]

On Thu, Aug 21, 2025 at 12:17:41AM +0800, Jisheng Zhang wrote:
> Currently, the usb251xb assumes i2c control, and the corresponding
> dt node looks like the following:
> 
> i2c {
> 	usb-hub@2c {
> 		compatible = "microchip,usb2512b";
> 		reg = <0x2c>;
> 		reset-gpios = <&porta 8 GPIO_ACTIVE_LOW>;
> 	};
> };

Above is redundant, obvious from stagtement that USB can be a device on
I2C bus. Just keep it simple.

> 
> But from HW point of view, the hub supports usage case without any i2c
> control, I.E we only want the gpio controls, for example the following
> dt node:
> 
> usb-hub {
> 	compatible = "microchip,usb2512b";
> 	reset-gpios = <&porta 8 GPIO_ACTIVE_LOW>;
> };
> 
> Modify the dt-binding of usb2512b to support this usage case, and add
> usage example to the examples section.
> 
> Signed-off-by: Jisheng Zhang <jszhang@kernel.org>
> ---
>  Documentation/devicetree/bindings/usb/usb251xb.yaml | 9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Best regards,
Krzysztof