On 15/09/2023 08:50, Javier Carrasco wrote: > The TPS6598x driver supports an optional high-level reset GPIO. What driver supports or not, it does not matter. The question is (and commit msg should say) whether hardware has such reset pin or not. > Document the new property and add it to the example. > > Signed-off-by: Javier Carrasco <javier.carrasco@xxxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/usb/ti,tps6598x.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/ti,tps6598x.yaml b/Documentation/devicetree/bindings/usb/ti,tps6598x.yaml > index 5497a60cddbc..b1a621e06127 100644 > --- a/Documentation/devicetree/bindings/usb/ti,tps6598x.yaml > +++ b/Documentation/devicetree/bindings/usb/ti,tps6598x.yaml > @@ -23,6 +23,10 @@ properties: > reg: > maxItems: 1 > > + reset-gpios: > + description: GPIO used for the HRESET pin. This probably answers that yes - it is a pin in hardware... Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof
