* [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
* 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
* [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
* 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
* [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
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).