* [PATCH v5 1/2] dt-bindings: extcon-usb-gpio: convert to DT schema format
2024-02-15 9:32 [PATCH v5 0/2] extcon-usb-gpio YAML conversion Alexander Stein
@ 2024-02-15 9:32 ` Alexander Stein
2024-06-28 9:14 ` Alexander Stein
2024-02-15 9:32 ` [PATCH v5 2/2] arm64: dts: mediatek: Replace deprecated extcon-usb-gpio id-gpio/vbus-gpio properties Alexander Stein
2024-02-15 11:09 ` (subset) [PATCH v5 0/2] extcon-usb-gpio YAML conversion AngeloGioacchino Del Regno
2 siblings, 1 reply; 5+ messages in thread
From: Alexander Stein @ 2024-02-15 9:32 UTC (permalink / raw)
To: MyungJoo Ham, Chanwoo Choi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Matthias Brugger, AngeloGioacchino Del Regno
Cc: Alexander Stein, devicetree, linux-arm-kernel, linux-mediatek
Convert the binding to DT schema format. Change the GPIO properties to new
naming convention using -gpios as well.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
.../bindings/extcon/extcon-usb-gpio.txt | 21 --------
.../bindings/extcon/extcon-usb-gpio.yaml | 51 +++++++++++++++++++
2 files changed, 51 insertions(+), 21 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt
create mode 100644 Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml
diff --git a/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt b/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt
deleted file mode 100644
index dfc14f71e81fb..0000000000000
--- a/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt
+++ /dev/null
@@ -1,21 +0,0 @@
-USB GPIO Extcon device
-
-This is a virtual device used to generate USB cable states from the USB ID pin
-connected to a GPIO pin.
-
-Required properties:
-- compatible: Should be "linux,extcon-usb-gpio"
-
-Either one of id-gpio or vbus-gpio must be present. Both can be present as well.
-- id-gpio: gpio for USB ID pin. See gpio binding.
-- vbus-gpio: gpio for USB VBUS pin.
-
-Example: Examples of extcon-usb-gpio node in dra7-evm.dts as listed below:
- extcon_usb1 {
- compatible = "linux,extcon-usb-gpio";
- id-gpio = <&gpio6 1 GPIO_ACTIVE_HIGH>;
- }
-
- &omap_dwc3_1 {
- extcon = <&extcon_usb1>;
- };
diff --git a/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml b/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml
new file mode 100644
index 0000000000000..136f865b87816
--- /dev/null
+++ b/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml
@@ -0,0 +1,51 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/extcon/extcon-usb-gpio.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: USB GPIO Extcon device
+
+maintainers:
+ - Alexander Stein <alexander.stein@ew.tq-group.com>
+
+description:
+ This is a virtual device used to generate USB cable states from the
+ USB ID pin connected to a GPIO pin.
+ Deprecated, use USB connector node instead.
+
+deprecated: true
+
+properties:
+ compatible:
+ const: linux,extcon-usb-gpio
+
+ id-gpios:
+ description: An input gpio for USB ID pin.
+ maxItems: 1
+
+ vbus-gpios:
+ description: An input gpio for USB VBus pin, used to detect presence of
+ VBUS 5V.
+ maxItems: 1
+
+required:
+ - compatible
+
+anyOf:
+ - required:
+ - id-gpios
+ - required:
+ - vbus-gpios
+
+additionalProperties: false
+
+examples:
+ - |
+ #include <dt-bindings/gpio/gpio.h>
+
+ extcon-usb1 {
+ compatible = "linux,extcon-usb-gpio";
+ id-gpios = <&gpio6 1 GPIO_ACTIVE_HIGH>;
+ vbus-gpios = <&gpio6 2 GPIO_ACTIVE_HIGH>;
+ };
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH v5 1/2] dt-bindings: extcon-usb-gpio: convert to DT schema format
2024-02-15 9:32 ` [PATCH v5 1/2] dt-bindings: extcon-usb-gpio: convert to DT schema format Alexander Stein
@ 2024-06-28 9:14 ` Alexander Stein
0 siblings, 0 replies; 5+ messages in thread
From: Alexander Stein @ 2024-06-28 9:14 UTC (permalink / raw)
To: MyungJoo Ham, Chanwoo Choi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Matthias Brugger, AngeloGioacchino Del Regno
Cc: devicetree, linux-arm-kernel, linux-mediatek
Am Donnerstag, 15. Februar 2024, 10:32:13 CEST schrieb Alexander Stein:
> Convert the binding to DT schema format. Change the GPIO properties to new
> naming convention using -gpios as well.
>
> Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
Patch 2 has been applied as 5b6df373ec95cf051264f655be0dbe1d6caa173c
Who is going to pick this?
Thanks
Alexander
> ---
> .../bindings/extcon/extcon-usb-gpio.txt | 21 --------
> .../bindings/extcon/extcon-usb-gpio.yaml | 51 +++++++++++++++++++
> 2 files changed, 51 insertions(+), 21 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt
> create mode 100644 Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml
>
> diff --git a/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt b/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt
> deleted file mode 100644
> index dfc14f71e81fb..0000000000000
> --- a/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.txt
> +++ /dev/null
> @@ -1,21 +0,0 @@
> -USB GPIO Extcon device
> -
> -This is a virtual device used to generate USB cable states from the USB ID pin
> -connected to a GPIO pin.
> -
> -Required properties:
> -- compatible: Should be "linux,extcon-usb-gpio"
> -
> -Either one of id-gpio or vbus-gpio must be present. Both can be present as well.
> -- id-gpio: gpio for USB ID pin. See gpio binding.
> -- vbus-gpio: gpio for USB VBUS pin.
> -
> -Example: Examples of extcon-usb-gpio node in dra7-evm.dts as listed below:
> - extcon_usb1 {
> - compatible = "linux,extcon-usb-gpio";
> - id-gpio = <&gpio6 1 GPIO_ACTIVE_HIGH>;
> - }
> -
> - &omap_dwc3_1 {
> - extcon = <&extcon_usb1>;
> - };
> diff --git a/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml b/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml
> new file mode 100644
> index 0000000000000..136f865b87816
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/extcon/extcon-usb-gpio.yaml
> @@ -0,0 +1,51 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/extcon/extcon-usb-gpio.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: USB GPIO Extcon device
> +
> +maintainers:
> + - Alexander Stein <alexander.stein@ew.tq-group.com>
> +
> +description:
> + This is a virtual device used to generate USB cable states from the
> + USB ID pin connected to a GPIO pin.
> + Deprecated, use USB connector node instead.
> +
> +deprecated: true
> +
> +properties:
> + compatible:
> + const: linux,extcon-usb-gpio
> +
> + id-gpios:
> + description: An input gpio for USB ID pin.
> + maxItems: 1
> +
> + vbus-gpios:
> + description: An input gpio for USB VBus pin, used to detect presence of
> + VBUS 5V.
> + maxItems: 1
> +
> +required:
> + - compatible
> +
> +anyOf:
> + - required:
> + - id-gpios
> + - required:
> + - vbus-gpios
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/gpio/gpio.h>
> +
> + extcon-usb1 {
> + compatible = "linux,extcon-usb-gpio";
> + id-gpios = <&gpio6 1 GPIO_ACTIVE_HIGH>;
> + vbus-gpios = <&gpio6 2 GPIO_ACTIVE_HIGH>;
> + };
>
--
TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht München, HRB 105018
Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
http://www.tq-group.com/
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v5 2/2] arm64: dts: mediatek: Replace deprecated extcon-usb-gpio id-gpio/vbus-gpio properties
2024-02-15 9:32 [PATCH v5 0/2] extcon-usb-gpio YAML conversion Alexander Stein
2024-02-15 9:32 ` [PATCH v5 1/2] dt-bindings: extcon-usb-gpio: convert to DT schema format Alexander Stein
@ 2024-02-15 9:32 ` Alexander Stein
2024-02-15 11:09 ` (subset) [PATCH v5 0/2] extcon-usb-gpio YAML conversion AngeloGioacchino Del Regno
2 siblings, 0 replies; 5+ messages in thread
From: Alexander Stein @ 2024-02-15 9:32 UTC (permalink / raw)
To: MyungJoo Ham, Chanwoo Choi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Matthias Brugger, AngeloGioacchino Del Regno
Cc: Alexander Stein, devicetree, linux-arm-kernel, linux-mediatek,
Shawn Guo, Krzysztof Kozlowski
Use id-gpios and vbus-gpios instead.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Acked-by: Shawn Guo <shawnguo@kernel.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
arch/arm64/boot/dts/mediatek/mt2712-evb.dts | 4 ++--
arch/arm64/boot/dts/mediatek/mt8173-evb.dts | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/mediatek/mt2712-evb.dts b/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
index fffdb7bbf889e..0c38f7b517637 100644
--- a/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
+++ b/arch/arm64/boot/dts/mediatek/mt2712-evb.dts
@@ -43,12 +43,12 @@ cpus_fixed_vproc1: regulator-vproc-buck1 {
extcon_usb: extcon_iddig {
compatible = "linux,extcon-usb-gpio";
- id-gpio = <&pio 12 GPIO_ACTIVE_HIGH>;
+ id-gpios = <&pio 12 GPIO_ACTIVE_HIGH>;
};
extcon_usb1: extcon_iddig1 {
compatible = "linux,extcon-usb-gpio";
- id-gpio = <&pio 14 GPIO_ACTIVE_HIGH>;
+ id-gpios = <&pio 14 GPIO_ACTIVE_HIGH>;
};
usb_p0_vbus: regulator-usb-p0-vbus {
diff --git a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
index 0e5c628d1ec3e..3fab21f59d183 100644
--- a/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
+++ b/arch/arm64/boot/dts/mediatek/mt8173-evb.dts
@@ -41,7 +41,7 @@ hdmi_connector_in: endpoint {
extcon_usb: extcon_iddig {
compatible = "linux,extcon-usb-gpio";
- id-gpio = <&pio 16 GPIO_ACTIVE_HIGH>;
+ id-gpios = <&pio 16 GPIO_ACTIVE_HIGH>;
};
usb_p1_vbus: regulator-usb-p1 {
--
2.34.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: (subset) [PATCH v5 0/2] extcon-usb-gpio YAML conversion
2024-02-15 9:32 [PATCH v5 0/2] extcon-usb-gpio YAML conversion Alexander Stein
2024-02-15 9:32 ` [PATCH v5 1/2] dt-bindings: extcon-usb-gpio: convert to DT schema format Alexander Stein
2024-02-15 9:32 ` [PATCH v5 2/2] arm64: dts: mediatek: Replace deprecated extcon-usb-gpio id-gpio/vbus-gpio properties Alexander Stein
@ 2024-02-15 11:09 ` AngeloGioacchino Del Regno
2 siblings, 0 replies; 5+ messages in thread
From: AngeloGioacchino Del Regno @ 2024-02-15 11:09 UTC (permalink / raw)
To: MyungJoo Ham, Chanwoo Choi, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, Matthias Brugger, Alexander Stein
Cc: AngeloGioacchino Del Regno, devicetree, linux-arm-kernel,
linux-mediatek
On Thu, 15 Feb 2024 10:32:12 +0100, Alexander Stein wrote:
> it seems these two patches got missed from v4[1]. So v5 is a new version only
> containing the patches not applied yet. The rebase also includes a slight
> change to the mediatek patch.
>
> Changes in v5:
> * rebased to next-20240215
>
> [...]
Applied to v6.8-next/dts64, thanks!
[2/2] arm64: dts: mediatek: Replace deprecated extcon-usb-gpio id-gpio/vbus-gpio properties
https://git.kernel.org/mediatek/c/5b6df373
Cheers,
Angelo
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 5+ messages in thread