2023-10-18 15:17:09

by Jisheng Zhang

[permalink] [raw]
Subject: [PATCH v2] dt-bindings: usb: vialab,vl817: remove reset-gpios from required list

The "reset-gpios" is optional in real case, for example reset pin is
is hard wired to "high". And this fact is also reflected by the
devm_gpio_get_optional() calling in driver code.

Signed-off-by: Jisheng Zhang <[email protected]>
---

since v1:
- remove th1520 usb dt-binding part, this isn't related.

Documentation/devicetree/bindings/usb/vialab,vl817.yaml | 1 -
1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml
index 76db9071b352..c815010ba9c2 100644
--- a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml
+++ b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml
@@ -37,7 +37,6 @@ properties:
required:
- compatible
- reg
- - reset-gpios
- vdd-supply
- peer-hub

--
2.40.1


2023-10-18 15:48:03

by Conor Dooley

[permalink] [raw]
Subject: Re: [PATCH v2] dt-bindings: usb: vialab,vl817: remove reset-gpios from required list

On Wed, Oct 18, 2023 at 11:04:48PM +0800, Jisheng Zhang wrote:
> The "reset-gpios" is optional in real case, for example reset pin is
> is hard wired to "high". And this fact is also reflected by the
> devm_gpio_get_optional() calling in driver code.
>
> Signed-off-by: Jisheng Zhang <[email protected]>

Acked-by: Conor Dooley <[email protected]>

Thanks,
Conor.

> ---
>
> since v1:
> - remove th1520 usb dt-binding part, this isn't related.
>
> Documentation/devicetree/bindings/usb/vialab,vl817.yaml | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml
> index 76db9071b352..c815010ba9c2 100644
> --- a/Documentation/devicetree/bindings/usb/vialab,vl817.yaml
> +++ b/Documentation/devicetree/bindings/usb/vialab,vl817.yaml
> @@ -37,7 +37,6 @@ properties:
> required:
> - compatible
> - reg
> - - reset-gpios
> - vdd-supply
> - peer-hub
>
> --
> 2.40.1
>


Attachments:
(No filename) (1.02 kB)
signature.asc (235.00 B)
Download all attachments