diff mbox series

[1/2] ARM: dts: imx6ul-tx6ul: Use preferred i2c-gpios properties

Message ID 20230912213645.1803914-1-festevam@gmail.com (mailing list archive)
State New, archived
Headers show
Series [1/2] ARM: dts: imx6ul-tx6ul: Use preferred i2c-gpios properties | expand

Commit Message

Fabio Estevam Sept. 12, 2023, 9:36 p.m. UTC
The 'gpios' property to describe the SDA and SCL GPIOs is considered
deprecated according to i2c-gpio.yaml.

Switch to the preferred 'sda-gpios' and 'scl-gpios' properties.

Thsis fixes the following schema warnings:

imx6ul-tx6ul-0011.dtb: i2c-gpio: 'sda-gpios' is a required property
	from schema $id: http://devicetree.org/schemas/i2c/i2c-gpio.yaml#
imx6ul-tx6ul-0011.dtb: i2c-gpio: 'scl-gpios' is a required property
	from schema $id: http://devicetree.org/schemas/i2c/i2c-gpio.yaml#

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
 arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

Comments

Shawn Guo Sept. 25, 2023, 2:33 a.m. UTC | #1
On Tue, Sep 12, 2023 at 06:36:44PM -0300, Fabio Estevam wrote:
> The 'gpios' property to describe the SDA and SCL GPIOs is considered
> deprecated according to i2c-gpio.yaml.
> 
> Switch to the preferred 'sda-gpios' and 'scl-gpios' properties.
> 
> Thsis fixes the following schema warnings:
> 
> imx6ul-tx6ul-0011.dtb: i2c-gpio: 'sda-gpios' is a required property
> 	from schema $id: http://devicetree.org/schemas/i2c/i2c-gpio.yaml#
> imx6ul-tx6ul-0011.dtb: i2c-gpio: 'scl-gpios' is a required property
> 	from schema $id: http://devicetree.org/schemas/i2c/i2c-gpio.yaml#
> 
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

Applied both, thanks!
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi b/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
index 6bd90473050b..1db146ac1c17 100644
--- a/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
+++ b/arch/arm/boot/dts/nxp/imx/imx6ul-tx6ul.dtsi
@@ -114,10 +114,8 @@  i2c_gpio: i2c-gpio {
 		#size-cells = <0>;
 		pinctrl-names = "default";
 		pinctrl-0 = <&pinctrl_i2c_gpio>;
-		gpios = <
-			&gpio5 1 GPIO_ACTIVE_HIGH /* SDA */
-			&gpio5 0 GPIO_ACTIVE_HIGH /* SCL */
-		>;
+		sda-gpios = <&gpio5 1 GPIO_ACTIVE_HIGH>;
+		scl-gpios = <&gpio5 0 GPIO_ACTIVE_HIGH>;
 		clock-frequency = <400000>;
 		status = "okay";