* [PATCH 0/4] ARM: dts: use recent scl/sda gpio bindings
@ 2025-05-19 11:21 Wolfram Sang
2025-05-19 11:21 ` [PATCH 1/4] ARM: dts: cirrus: ep7211: " Wolfram Sang
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: Wolfram Sang @ 2025-05-19 11:21 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Wolfram Sang, Alexandre Belloni, Andrew Lunn, Claudiu Beznea,
Conor Dooley, devicetree, Gregory Clement, Krzysztof Kozlowski,
Linus Walleij, Nicolas Ferre, Rob Herring, Sebastian Hesselbarth
Some ARM boards still use the deprecated 'gpios' property to describe a
GPIO based I2C bus. Let them use the proper '{scl|sda}-gpios' property.
Based on linux-next as of 20250516. No dependencies. I suggest that
patches go via their subsystem trees.
Tested on Calao USB boards using AT91 chipsets.
Wolfram Sang (4):
ARM: dts: cirrus: ep7211: use recent scl/sda gpio bindings
ARM: dts: marvell: kirkwood: use recent scl/sda gpio bindings
ARM: dts: microchip: use recent scl/sda gpio bindings
ARM: dts: stm32: use recent scl/sda gpio bindings
arch/arm/boot/dts/cirrus/ep7211-edb7211.dts | 4 ++--
arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi | 4 ++--
arch/arm/boot/dts/microchip/at91rm9200.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9260.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9261.dtsi | 4 ++--
arch/arm/boot/dts/microchip/at91sam9263.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9g45.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9n12.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9rl.dtsi | 8 ++++----
arch/arm/boot/dts/microchip/at91sam9x5.dtsi | 15 ++++++---------
arch/arm/boot/dts/st/ste-nomadik-s8815.dts | 4 ++--
11 files changed, 28 insertions(+), 36 deletions(-)
--
2.47.2
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/4] ARM: dts: cirrus: ep7211: use recent scl/sda gpio bindings
2025-05-19 11:21 [PATCH 0/4] ARM: dts: use recent scl/sda gpio bindings Wolfram Sang
@ 2025-05-19 11:21 ` Wolfram Sang
2025-05-19 11:21 ` [PATCH 2/4] ARM: dts: marvell: kirkwood: " Wolfram Sang
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: Wolfram Sang @ 2025-05-19 11:21 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Wolfram Sang, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
devicetree
We have dedictaded bindings for scl/sda nowadays. Switch away from the
deprecated plain 'gpios' property.
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
arch/arm/boot/dts/cirrus/ep7211-edb7211.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/cirrus/ep7211-edb7211.dts b/arch/arm/boot/dts/cirrus/ep7211-edb7211.dts
index adc74243ed19..0b15ccaa762e 100644
--- a/arch/arm/boot/dts/cirrus/ep7211-edb7211.dts
+++ b/arch/arm/boot/dts/cirrus/ep7211-edb7211.dts
@@ -46,8 +46,8 @@ timing0: timing-320x240 {
i2c: i2c {
compatible = "i2c-gpio";
- gpios = <&portd 4 GPIO_ACTIVE_HIGH>,
- <&portd 5 GPIO_ACTIVE_HIGH>;
+ sda-gpios = <&portd 4 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&portd 5 GPIO_ACTIVE_HIGH>;
i2c-gpio,delay-us = <2>;
i2c-gpio,scl-output-only;
#address-cells = <1>;
--
2.47.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/4] ARM: dts: marvell: kirkwood: use recent scl/sda gpio bindings
2025-05-19 11:21 [PATCH 0/4] ARM: dts: use recent scl/sda gpio bindings Wolfram Sang
2025-05-19 11:21 ` [PATCH 1/4] ARM: dts: cirrus: ep7211: " Wolfram Sang
@ 2025-05-19 11:21 ` Wolfram Sang
2025-05-19 14:31 ` Gregory CLEMENT
2025-05-19 11:21 ` [PATCH 3/4] ARM: dts: microchip: " Wolfram Sang
2025-05-19 11:21 ` [PATCH 4/4] ARM: dts: stm32: " Wolfram Sang
3 siblings, 1 reply; 8+ messages in thread
From: Wolfram Sang @ 2025-05-19 11:21 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Wolfram Sang, Andrew Lunn, Gregory Clement, Sebastian Hesselbarth,
Rob Herring, Krzysztof Kozlowski, Conor Dooley, devicetree
We have dedictaded bindings for scl/sda nowadays. Switch away from the
deprecated plain 'gpios' property.
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi b/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi
index 52baffe45f12..259cb3d5f16d 100644
--- a/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi
+++ b/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi
@@ -27,8 +27,8 @@ serial@12000 {
i2c {
compatible = "i2c-gpio";
- gpios = < &gpio0 8 GPIO_ACTIVE_HIGH /* sda */
- &gpio0 9 GPIO_ACTIVE_HIGH>; /* scl */
+ sda-gpios = <&gpio0 8 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&gpio0 9 GPIO_ACTIVE_HIGH>;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
};
};
--
2.47.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 3/4] ARM: dts: microchip: use recent scl/sda gpio bindings
2025-05-19 11:21 [PATCH 0/4] ARM: dts: use recent scl/sda gpio bindings Wolfram Sang
2025-05-19 11:21 ` [PATCH 1/4] ARM: dts: cirrus: ep7211: " Wolfram Sang
2025-05-19 11:21 ` [PATCH 2/4] ARM: dts: marvell: kirkwood: " Wolfram Sang
@ 2025-05-19 11:21 ` Wolfram Sang
2025-06-07 11:17 ` Claudiu Beznea
2025-06-24 7:13 ` Claudiu Beznea
2025-05-19 11:21 ` [PATCH 4/4] ARM: dts: stm32: " Wolfram Sang
3 siblings, 2 replies; 8+ messages in thread
From: Wolfram Sang @ 2025-05-19 11:21 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Wolfram Sang, Rob Herring, Krzysztof Kozlowski, Conor Dooley,
Nicolas Ferre, Alexandre Belloni, Claudiu Beznea, devicetree
We have dedictaded bindings for scl/sda nowadays. Switch away from the
deprecated plain 'gpios' property.
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
arch/arm/boot/dts/microchip/at91rm9200.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9260.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9261.dtsi | 4 ++--
arch/arm/boot/dts/microchip/at91sam9263.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9g45.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9n12.dtsi | 5 ++---
arch/arm/boot/dts/microchip/at91sam9rl.dtsi | 8 ++++----
arch/arm/boot/dts/microchip/at91sam9x5.dtsi | 15 ++++++---------
8 files changed, 22 insertions(+), 30 deletions(-)
diff --git a/arch/arm/boot/dts/microchip/at91rm9200.dtsi b/arch/arm/boot/dts/microchip/at91rm9200.dtsi
index 2a4c83d88733..e105ad855ce8 100644
--- a/arch/arm/boot/dts/microchip/at91rm9200.dtsi
+++ b/arch/arm/boot/dts/microchip/at91rm9200.dtsi
@@ -714,9 +714,8 @@ usb0: usb@300000 {
i2c-gpio-0 {
compatible = "i2c-gpio";
- gpios = <&pioA 25 GPIO_ACTIVE_HIGH /* sda */
- &pioA 26 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioA 25 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioA 26 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
diff --git a/arch/arm/boot/dts/microchip/at91sam9260.dtsi b/arch/arm/boot/dts/microchip/at91sam9260.dtsi
index ec973f07a961..fc0b6a73204f 100644
--- a/arch/arm/boot/dts/microchip/at91sam9260.dtsi
+++ b/arch/arm/boot/dts/microchip/at91sam9260.dtsi
@@ -781,9 +781,8 @@ nand_controller: nand-controller {
i2c_gpio0: i2c-gpio-0 {
compatible = "i2c-gpio";
- gpios = <&pioA 23 GPIO_ACTIVE_HIGH /* sda */
- &pioA 24 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioA 23 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioA 24 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
diff --git a/arch/arm/boot/dts/microchip/at91sam9261.dtsi b/arch/arm/boot/dts/microchip/at91sam9261.dtsi
index 0b556c234557..d1d678b77e84 100644
--- a/arch/arm/boot/dts/microchip/at91sam9261.dtsi
+++ b/arch/arm/boot/dts/microchip/at91sam9261.dtsi
@@ -655,8 +655,8 @@ i2c-gpio-0 {
compatible = "i2c-gpio";
pinctrl-names = "default";
pinctrl-0 = <&pinctrl_i2c_bitbang>;
- gpios = <&pioA 7 GPIO_ACTIVE_HIGH>, /* sda */
- <&pioA 8 GPIO_ACTIVE_HIGH>; /* scl */
+ sda-gpios = <&pioA 7 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioA 8 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
diff --git a/arch/arm/boot/dts/microchip/at91sam9263.dtsi b/arch/arm/boot/dts/microchip/at91sam9263.dtsi
index 3e9e5ce7c6c8..a4b5d1f228f9 100644
--- a/arch/arm/boot/dts/microchip/at91sam9263.dtsi
+++ b/arch/arm/boot/dts/microchip/at91sam9263.dtsi
@@ -826,9 +826,8 @@ nand_controller1: nand-controller {
i2c-gpio-0 {
compatible = "i2c-gpio";
- gpios = <&pioB 4 GPIO_ACTIVE_HIGH /* sda */
- &pioB 5 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioB 4 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioB 5 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
diff --git a/arch/arm/boot/dts/microchip/at91sam9g45.dtsi b/arch/arm/boot/dts/microchip/at91sam9g45.dtsi
index 535e26e05e99..4e00ed2d3ecd 100644
--- a/arch/arm/boot/dts/microchip/at91sam9g45.dtsi
+++ b/arch/arm/boot/dts/microchip/at91sam9g45.dtsi
@@ -1010,9 +1010,8 @@ nand_controller: nand-controller {
i2c-gpio-0 {
compatible = "i2c-gpio";
- gpios = <&pioA 20 GPIO_ACTIVE_HIGH /* sda */
- &pioA 21 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioA 20 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioA 21 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <5>; /* ~100 kHz */
diff --git a/arch/arm/boot/dts/microchip/at91sam9n12.dtsi b/arch/arm/boot/dts/microchip/at91sam9n12.dtsi
index 2f930c39ce4d..af41c3dbb4bf 100644
--- a/arch/arm/boot/dts/microchip/at91sam9n12.dtsi
+++ b/arch/arm/boot/dts/microchip/at91sam9n12.dtsi
@@ -786,9 +786,8 @@ nand_controller: nand-controller {
i2c-gpio-0 {
compatible = "i2c-gpio";
- gpios = <&pioA 30 GPIO_ACTIVE_HIGH /* sda */
- &pioA 31 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioA 30 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioA 31 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
diff --git a/arch/arm/boot/dts/microchip/at91sam9rl.dtsi b/arch/arm/boot/dts/microchip/at91sam9rl.dtsi
index 1fec9fcc7cd1..de74cf2980a0 100644
--- a/arch/arm/boot/dts/microchip/at91sam9rl.dtsi
+++ b/arch/arm/boot/dts/microchip/at91sam9rl.dtsi
@@ -833,8 +833,8 @@ rtc@fffffe00 {
i2c-gpio-0 {
compatible = "i2c-gpio";
- gpios = <&pioA 23 GPIO_ACTIVE_HIGH>, /* sda */
- <&pioA 24 GPIO_ACTIVE_HIGH>; /* scl */
+ sda-gpios = <&pioA 23 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioA 24 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
@@ -847,8 +847,8 @@ i2c-gpio-0 {
i2c-gpio-1 {
compatible = "i2c-gpio";
- gpios = <&pioD 10 GPIO_ACTIVE_HIGH>, /* sda */
- <&pioD 11 GPIO_ACTIVE_HIGH>; /* scl */
+ sda-gpios = <&pioD 10 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioD 11 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
diff --git a/arch/arm/boot/dts/microchip/at91sam9x5.dtsi b/arch/arm/boot/dts/microchip/at91sam9x5.dtsi
index 17bdf1e4db01..9070fd06995a 100644
--- a/arch/arm/boot/dts/microchip/at91sam9x5.dtsi
+++ b/arch/arm/boot/dts/microchip/at91sam9x5.dtsi
@@ -933,9 +933,8 @@ nand_controller: nand-controller {
i2c-gpio-0 {
compatible = "i2c-gpio";
- gpios = <&pioA 30 GPIO_ACTIVE_HIGH /* sda */
- &pioA 31 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioA 30 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioA 31 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
@@ -948,9 +947,8 @@ &pioA 31 GPIO_ACTIVE_HIGH /* scl */
i2c-gpio-1 {
compatible = "i2c-gpio";
- gpios = <&pioC 0 GPIO_ACTIVE_HIGH /* sda */
- &pioC 1 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioC 0 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioC 1 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
@@ -963,9 +961,8 @@ &pioC 1 GPIO_ACTIVE_HIGH /* scl */
i2c-gpio-2 {
compatible = "i2c-gpio";
- gpios = <&pioB 4 GPIO_ACTIVE_HIGH /* sda */
- &pioB 5 GPIO_ACTIVE_HIGH /* scl */
- >;
+ sda-gpios = <&pioB 4 GPIO_ACTIVE_HIGH>;
+ scl-gpios = <&pioB 5 GPIO_ACTIVE_HIGH>;
i2c-gpio,sda-open-drain;
i2c-gpio,scl-open-drain;
i2c-gpio,delay-us = <2>; /* ~100 kHz */
--
2.47.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 4/4] ARM: dts: stm32: use recent scl/sda gpio bindings
2025-05-19 11:21 [PATCH 0/4] ARM: dts: use recent scl/sda gpio bindings Wolfram Sang
` (2 preceding siblings ...)
2025-05-19 11:21 ` [PATCH 3/4] ARM: dts: microchip: " Wolfram Sang
@ 2025-05-19 11:21 ` Wolfram Sang
3 siblings, 0 replies; 8+ messages in thread
From: Wolfram Sang @ 2025-05-19 11:21 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Wolfram Sang, Linus Walleij, Rob Herring, Krzysztof Kozlowski,
Conor Dooley, devicetree
We have dedictaded bindings for scl/sda nowadays. Switch away from the
deprecated plain 'gpios' property.
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
arch/arm/boot/dts/st/ste-nomadik-s8815.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm/boot/dts/st/ste-nomadik-s8815.dts b/arch/arm/boot/dts/st/ste-nomadik-s8815.dts
index c905c2643a12..aeb58d2c08a1 100644
--- a/arch/arm/boot/dts/st/ste-nomadik-s8815.dts
+++ b/arch/arm/boot/dts/st/ste-nomadik-s8815.dts
@@ -117,8 +117,8 @@ lis3lv02dl@1d {
/* GPIO I2C connected to the USB portions of the STw4811 only */
gpio-i2c {
compatible = "i2c-gpio";
- gpios = <&gpio2 10 0>, /* sda */
- <&gpio2 9 0>; /* scl */
+ sda-gpios = <&gpio2 10 0>;
+ scl-gpios = <&gpio2 9 0>;
#address-cells = <1>;
#size-cells = <0>;
pinctrl-names = "default";
--
2.47.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/4] ARM: dts: marvell: kirkwood: use recent scl/sda gpio bindings
2025-05-19 11:21 ` [PATCH 2/4] ARM: dts: marvell: kirkwood: " Wolfram Sang
@ 2025-05-19 14:31 ` Gregory CLEMENT
0 siblings, 0 replies; 8+ messages in thread
From: Gregory CLEMENT @ 2025-05-19 14:31 UTC (permalink / raw)
To: Wolfram Sang, linux-arm-kernel
Cc: Wolfram Sang, Andrew Lunn, Sebastian Hesselbarth, Rob Herring,
Krzysztof Kozlowski, Conor Dooley, devicetree
Wolfram Sang <wsa+renesas@sang-engineering.com> writes:
> We have dedictaded bindings for scl/sda nowadays. Switch away from the
> deprecated plain 'gpios' property.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Applied on mvebu/dt
Thanks,
Gregory
> ---
> arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi b/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi
> index 52baffe45f12..259cb3d5f16d 100644
> --- a/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi
> +++ b/arch/arm/boot/dts/marvell/kirkwood-km_common.dtsi
> @@ -27,8 +27,8 @@ serial@12000 {
>
> i2c {
> compatible = "i2c-gpio";
> - gpios = < &gpio0 8 GPIO_ACTIVE_HIGH /* sda */
> - &gpio0 9 GPIO_ACTIVE_HIGH>; /* scl */
> + sda-gpios = <&gpio0 8 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&gpio0 9 GPIO_ACTIVE_HIGH>;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> };
> };
> --
> 2.47.2
>
--
Grégory CLEMENT, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 3/4] ARM: dts: microchip: use recent scl/sda gpio bindings
2025-05-19 11:21 ` [PATCH 3/4] ARM: dts: microchip: " Wolfram Sang
@ 2025-06-07 11:17 ` Claudiu Beznea
2025-06-24 7:13 ` Claudiu Beznea
1 sibling, 0 replies; 8+ messages in thread
From: Claudiu Beznea @ 2025-06-07 11:17 UTC (permalink / raw)
To: Wolfram Sang, linux-arm-kernel
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
Alexandre Belloni, devicetree
On 19.05.2025 14:21, Wolfram Sang wrote:
> We have dedictaded bindings for scl/sda nowadays. Switch away from the
> deprecated plain 'gpios' property.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Claudiu Beznea <claudiu.beznea@tuxon.dev>
> ---
> arch/arm/boot/dts/microchip/at91rm9200.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9260.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9261.dtsi | 4 ++--
> arch/arm/boot/dts/microchip/at91sam9263.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9g45.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9n12.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9rl.dtsi | 8 ++++----
> arch/arm/boot/dts/microchip/at91sam9x5.dtsi | 15 ++++++---------
> 8 files changed, 22 insertions(+), 30 deletions(-)
>
> diff --git a/arch/arm/boot/dts/microchip/at91rm9200.dtsi b/arch/arm/boot/dts/microchip/at91rm9200.dtsi
> index 2a4c83d88733..e105ad855ce8 100644
> --- a/arch/arm/boot/dts/microchip/at91rm9200.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91rm9200.dtsi
> @@ -714,9 +714,8 @@ usb0: usb@300000 {
>
> i2c-gpio-0 {
> compatible = "i2c-gpio";
> - gpios = <&pioA 25 GPIO_ACTIVE_HIGH /* sda */
> - &pioA 26 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioA 25 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioA 26 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> diff --git a/arch/arm/boot/dts/microchip/at91sam9260.dtsi b/arch/arm/boot/dts/microchip/at91sam9260.dtsi
> index ec973f07a961..fc0b6a73204f 100644
> --- a/arch/arm/boot/dts/microchip/at91sam9260.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91sam9260.dtsi
> @@ -781,9 +781,8 @@ nand_controller: nand-controller {
>
> i2c_gpio0: i2c-gpio-0 {
> compatible = "i2c-gpio";
> - gpios = <&pioA 23 GPIO_ACTIVE_HIGH /* sda */
> - &pioA 24 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioA 23 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioA 24 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> diff --git a/arch/arm/boot/dts/microchip/at91sam9261.dtsi b/arch/arm/boot/dts/microchip/at91sam9261.dtsi
> index 0b556c234557..d1d678b77e84 100644
> --- a/arch/arm/boot/dts/microchip/at91sam9261.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91sam9261.dtsi
> @@ -655,8 +655,8 @@ i2c-gpio-0 {
> compatible = "i2c-gpio";
> pinctrl-names = "default";
> pinctrl-0 = <&pinctrl_i2c_bitbang>;
> - gpios = <&pioA 7 GPIO_ACTIVE_HIGH>, /* sda */
> - <&pioA 8 GPIO_ACTIVE_HIGH>; /* scl */
> + sda-gpios = <&pioA 7 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioA 8 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> diff --git a/arch/arm/boot/dts/microchip/at91sam9263.dtsi b/arch/arm/boot/dts/microchip/at91sam9263.dtsi
> index 3e9e5ce7c6c8..a4b5d1f228f9 100644
> --- a/arch/arm/boot/dts/microchip/at91sam9263.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91sam9263.dtsi
> @@ -826,9 +826,8 @@ nand_controller1: nand-controller {
>
> i2c-gpio-0 {
> compatible = "i2c-gpio";
> - gpios = <&pioB 4 GPIO_ACTIVE_HIGH /* sda */
> - &pioB 5 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioB 4 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioB 5 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> diff --git a/arch/arm/boot/dts/microchip/at91sam9g45.dtsi b/arch/arm/boot/dts/microchip/at91sam9g45.dtsi
> index 535e26e05e99..4e00ed2d3ecd 100644
> --- a/arch/arm/boot/dts/microchip/at91sam9g45.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91sam9g45.dtsi
> @@ -1010,9 +1010,8 @@ nand_controller: nand-controller {
>
> i2c-gpio-0 {
> compatible = "i2c-gpio";
> - gpios = <&pioA 20 GPIO_ACTIVE_HIGH /* sda */
> - &pioA 21 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioA 20 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioA 21 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <5>; /* ~100 kHz */
> diff --git a/arch/arm/boot/dts/microchip/at91sam9n12.dtsi b/arch/arm/boot/dts/microchip/at91sam9n12.dtsi
> index 2f930c39ce4d..af41c3dbb4bf 100644
> --- a/arch/arm/boot/dts/microchip/at91sam9n12.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91sam9n12.dtsi
> @@ -786,9 +786,8 @@ nand_controller: nand-controller {
>
> i2c-gpio-0 {
> compatible = "i2c-gpio";
> - gpios = <&pioA 30 GPIO_ACTIVE_HIGH /* sda */
> - &pioA 31 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioA 30 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioA 31 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> diff --git a/arch/arm/boot/dts/microchip/at91sam9rl.dtsi b/arch/arm/boot/dts/microchip/at91sam9rl.dtsi
> index 1fec9fcc7cd1..de74cf2980a0 100644
> --- a/arch/arm/boot/dts/microchip/at91sam9rl.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91sam9rl.dtsi
> @@ -833,8 +833,8 @@ rtc@fffffe00 {
>
> i2c-gpio-0 {
> compatible = "i2c-gpio";
> - gpios = <&pioA 23 GPIO_ACTIVE_HIGH>, /* sda */
> - <&pioA 24 GPIO_ACTIVE_HIGH>; /* scl */
> + sda-gpios = <&pioA 23 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioA 24 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> @@ -847,8 +847,8 @@ i2c-gpio-0 {
>
> i2c-gpio-1 {
> compatible = "i2c-gpio";
> - gpios = <&pioD 10 GPIO_ACTIVE_HIGH>, /* sda */
> - <&pioD 11 GPIO_ACTIVE_HIGH>; /* scl */
> + sda-gpios = <&pioD 10 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioD 11 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> diff --git a/arch/arm/boot/dts/microchip/at91sam9x5.dtsi b/arch/arm/boot/dts/microchip/at91sam9x5.dtsi
> index 17bdf1e4db01..9070fd06995a 100644
> --- a/arch/arm/boot/dts/microchip/at91sam9x5.dtsi
> +++ b/arch/arm/boot/dts/microchip/at91sam9x5.dtsi
> @@ -933,9 +933,8 @@ nand_controller: nand-controller {
>
> i2c-gpio-0 {
> compatible = "i2c-gpio";
> - gpios = <&pioA 30 GPIO_ACTIVE_HIGH /* sda */
> - &pioA 31 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioA 30 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioA 31 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> @@ -948,9 +947,8 @@ &pioA 31 GPIO_ACTIVE_HIGH /* scl */
>
> i2c-gpio-1 {
> compatible = "i2c-gpio";
> - gpios = <&pioC 0 GPIO_ACTIVE_HIGH /* sda */
> - &pioC 1 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioC 0 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioC 1 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
> @@ -963,9 +961,8 @@ &pioC 1 GPIO_ACTIVE_HIGH /* scl */
>
> i2c-gpio-2 {
> compatible = "i2c-gpio";
> - gpios = <&pioB 4 GPIO_ACTIVE_HIGH /* sda */
> - &pioB 5 GPIO_ACTIVE_HIGH /* scl */
> - >;
> + sda-gpios = <&pioB 4 GPIO_ACTIVE_HIGH>;
> + scl-gpios = <&pioB 5 GPIO_ACTIVE_HIGH>;
> i2c-gpio,sda-open-drain;
> i2c-gpio,scl-open-drain;
> i2c-gpio,delay-us = <2>; /* ~100 kHz */
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 3/4] ARM: dts: microchip: use recent scl/sda gpio bindings
2025-05-19 11:21 ` [PATCH 3/4] ARM: dts: microchip: " Wolfram Sang
2025-06-07 11:17 ` Claudiu Beznea
@ 2025-06-24 7:13 ` Claudiu Beznea
1 sibling, 0 replies; 8+ messages in thread
From: Claudiu Beznea @ 2025-06-24 7:13 UTC (permalink / raw)
To: Wolfram Sang, linux-arm-kernel
Cc: Rob Herring, Krzysztof Kozlowski, Conor Dooley, Nicolas Ferre,
Alexandre Belloni, devicetree
On 19.05.2025 14:21, Wolfram Sang wrote:
> We have dedictaded bindings for scl/sda nowadays. Switch away from the
> deprecated plain 'gpios' property.
>
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
> ---
> arch/arm/boot/dts/microchip/at91rm9200.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9260.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9261.dtsi | 4 ++--
> arch/arm/boot/dts/microchip/at91sam9263.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9g45.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9n12.dtsi | 5 ++---
> arch/arm/boot/dts/microchip/at91sam9rl.dtsi | 8 ++++----
> arch/arm/boot/dts/microchip/at91sam9x5.dtsi | 15 ++++++---------
> 8 files changed, 22 insertions(+), 30 deletions(-)
Applied to at91-dt, thanks!
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2025-06-24 7:20 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-19 11:21 [PATCH 0/4] ARM: dts: use recent scl/sda gpio bindings Wolfram Sang
2025-05-19 11:21 ` [PATCH 1/4] ARM: dts: cirrus: ep7211: " Wolfram Sang
2025-05-19 11:21 ` [PATCH 2/4] ARM: dts: marvell: kirkwood: " Wolfram Sang
2025-05-19 14:31 ` Gregory CLEMENT
2025-05-19 11:21 ` [PATCH 3/4] ARM: dts: microchip: " Wolfram Sang
2025-06-07 11:17 ` Claudiu Beznea
2025-06-24 7:13 ` Claudiu Beznea
2025-05-19 11:21 ` [PATCH 4/4] ARM: dts: stm32: " Wolfram Sang
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).