[PATCH 3/4] ARM: dts: microchip: use recent scl/sda gpio bindings
Claudiu Beznea
claudiu.beznea at tuxon.dev
Sat Jun 7 04:17:29 PDT 2025
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 at sang-engineering.com>
Reviewed-by: Claudiu Beznea <claudiu.beznea at 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 at 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 at 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 */
More information about the linux-arm-kernel
mailing list