* [PATCH] dt-bindings: extcon: linux,extcon-usb-gpio: GPIO must be provided
@ 2025-03-29 12:09 David Heidelberg via B4 Relay
2025-03-31 15:52 ` Conor Dooley
0 siblings, 1 reply; 2+ messages in thread
From: David Heidelberg via B4 Relay @ 2025-03-29 12:09 UTC (permalink / raw)
To: MyungJoo Ham, Chanwoo Choi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Frank Li
Cc: linux-kernel, devicetree, David Heidelberg
From: David Heidelberg <david@ixit.cz>
Without providing either ID or VBUS GPIO the driver is not able to operate.
Original text binding says:
"Either one of id-gpio or vbus-gpio must be present."
Fixes: 79a31ce03f41 ("dt-bindings: extcon: convert extcon-usb-gpio.txt to yaml format")
Signed-off-by: David Heidelberg <david@ixit.cz>
---
Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml b/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml
index 8856107bdd33b8654812ab9c97e85e23dc2ef75a..8f29d333602b95fe5ccd8464aa64e2d1f0c1c781 100644
--- a/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml
+++ b/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml
@@ -25,6 +25,12 @@ properties:
required:
- compatible
+anyOf:
+ - required:
+ - id-gpios
+ - required:
+ - vbus-gpios
+
additionalProperties: false
examples:
---
base-commit: e21edb1638e82460f126a6e49bcdd958d452929c
change-id: 20250329-yaml-extcon-usb-gpio-251b66522287
Best regards,
--
David Heidelberg <david@ixit.cz>
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] dt-bindings: extcon: linux,extcon-usb-gpio: GPIO must be provided
2025-03-29 12:09 [PATCH] dt-bindings: extcon: linux,extcon-usb-gpio: GPIO must be provided David Heidelberg via B4 Relay
@ 2025-03-31 15:52 ` Conor Dooley
0 siblings, 0 replies; 2+ messages in thread
From: Conor Dooley @ 2025-03-31 15:52 UTC (permalink / raw)
To: david
Cc: MyungJoo Ham, Chanwoo Choi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Frank Li, linux-kernel, devicetree
[-- Attachment #1: Type: text/plain, Size: 1573 bytes --]
On Sat, Mar 29, 2025 at 01:09:47PM +0100, David Heidelberg via B4 Relay wrote:
> From: David Heidelberg <david@ixit.cz>
>
> Without providing either ID or VBUS GPIO the driver is not able to operate.
> Original text binding says:
> "Either one of id-gpio or vbus-gpio must be present."
The next sentence in the old binding reads something like "both can be
present as well", so the anyOf below is correct.
Acked-by: Conor Dooley <conor.dooley@microchip.com>
>
> Fixes: 79a31ce03f41 ("dt-bindings: extcon: convert extcon-usb-gpio.txt to yaml format")
> Signed-off-by: David Heidelberg <david@ixit.cz>
> ---
> Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml b/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml
> index 8856107bdd33b8654812ab9c97e85e23dc2ef75a..8f29d333602b95fe5ccd8464aa64e2d1f0c1c781 100644
> --- a/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml
> +++ b/Documentation/devicetree/bindings/extcon/linux,extcon-usb-gpio.yaml
> @@ -25,6 +25,12 @@ properties:
> required:
> - compatible
>
> +anyOf:
> + - required:
> + - id-gpios
> + - required:
> + - vbus-gpios
> +
> additionalProperties: false
>
> examples:
>
> ---
> base-commit: e21edb1638e82460f126a6e49bcdd958d452929c
> change-id: 20250329-yaml-extcon-usb-gpio-251b66522287
>
> Best regards,
> --
> David Heidelberg <david@ixit.cz>
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-03-31 15:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-03-29 12:09 [PATCH] dt-bindings: extcon: linux,extcon-usb-gpio: GPIO must be provided David Heidelberg via B4 Relay
2025-03-31 15:52 ` Conor Dooley
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).