linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs
@ 2025-07-14 14:17 Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 01/12] arm64: dts: imx8mm-kontron: Add overlay for LTE extension board Frieder Schrempf
                   ` (11 more replies)
  0 siblings, 12 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, Frieder Schrempf, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Alexander Stein, Andrej Picej, Annette Kobou, Eberhard Stoll,
	Fabio Estevam, Frank Li, Liu Ying, Oualid Derouiche,
	Pengutronix Kernel Team, Teresa Remmet, Tim Harvey

From: Frieder Schrempf <frieder.schrempf@kontron.de>

This is a collection of changes for several Kontron ARM64
board devicetrees. Some are related to hardware changes,
some are bugfixes and some are cleanups, some are improvements.

Annette Kobou (3):
  arm64: dts: imx8mm-kontron: Add overlay for LTE extension board
  arm64: dts: imx8mp-kontron: Fix CAN_ADDR0 and CAN_ADDR1 GPIOs
  arm64: dts: imx93-kontron: Fix GPIO for panel regulator

Eberhard Stoll (1):
  arm64: dts: imx8mm-kontron: Use GPIO for RS485 transceiver control

Frieder Schrempf (7):
  arm64: dts: imx8mm-kontron: Remove unused regulator
  arm64: dts: imx8mm-kontron: Sort reg nodes alphabetically
  arm64: dts: imx8mm-kontron: Name USB regulators according to OSM
    scheme
  arm64: dts: imx8mp-kontron: Fix GPIO labels for latest BL board
  arm64: dts: imx8mp-kontron: Fix USB hub reset
  arm64: dts: imx93-kontron: Add RTC interrupt signal
  arm64: dts: imx93-kontron: Fix USB port assignment

Oualid Derouiche (1):
  arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL
    devicetree

 arch/arm64/boot/dts/freescale/Makefile        |   2 +
 .../dts/freescale/imx8mm-kontron-bl-lte.dtso  | 186 ++++++++++++++++++
 .../dts/freescale/imx8mm-kontron-bl-osm-s.dts |   8 -
 .../boot/dts/freescale/imx8mm-kontron-bl.dts  |  12 +-
 .../boot/dts/freescale/imx8mm-kontron-dl.dtso |  11 ++
 .../dts/freescale/imx8mm-kontron-osm-s.dtsi   |  50 ++---
 .../dts/freescale/imx8mp-kontron-bl-osm-s.dts |  59 +++---
 .../dts/freescale/imx93-kontron-bl-osm-s.dts  |  32 +--
 .../dts/freescale/imx93-kontron-osm-s.dtsi    |   9 +
 9 files changed, 300 insertions(+), 69 deletions(-)
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-lte.dtso

-- 
2.50.1

^ permalink raw reply	[flat|nested] 17+ messages in thread

* [PATCH 01/12] arm64: dts: imx8mm-kontron: Add overlay for LTE extension board
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 02/12] arm64: dts: imx8mm-kontron: Remove unused regulator Frieder Schrempf
                   ` (10 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Annette Kobou, Frieder Schrempf, Alexander Stein, Andrej Picej,
	Fabio Estevam, Frank Li, Liu Ying, Pengutronix Kernel Team,
	Teresa Remmet, Tim Harvey

From: Annette Kobou <annette.kobou@kontron.de>

This is an addon for the BL i.MX8MM that features an LTE
modem, a TPM module, some LEDs and a pushbutton.

Signed-off-by: Annette Kobou <annette.kobou@kontron.de>
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm64/boot/dts/freescale/Makefile        |   2 +
 .../dts/freescale/imx8mm-kontron-bl-lte.dtso  | 186 ++++++++++++++++++
 2 files changed, 188 insertions(+)
 create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-lte.dtso

diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
index 0b473a23d1200..05d58ce3f6550 100644
--- a/arch/arm64/boot/dts/freescale/Makefile
+++ b/arch/arm64/boot/dts/freescale/Makefile
@@ -341,8 +341,10 @@ imx95-19x19-evk-pcie1-ep-dtbs += imx95-19x19-evk.dtb imx-pcie1-ep.dtbo
 dtb-$(CONFIG_ARCH_MXC) += imx95-19x19-evk-pcie0-ep.dtb imx95-19x19-evk-pcie1-ep.dtb
 
 imx8mm-kontron-dl-dtbs			:= imx8mm-kontron-bl.dtb imx8mm-kontron-dl.dtbo
+imx8mm-kontron-bl-lte-dtbs		:= imx8mm-kontron-bl.dtb imx8mm-kontron-bl-lte.dtbo
 
 dtb-$(CONFIG_ARCH_MXC) += imx8mm-kontron-dl.dtb
+dtb-$(CONFIG_ARCH_MXC) += imx8mm-kontron-bl-lte.dtb
 
 imx8mm-venice-gw72xx-0x-imx219-dtbs	:= imx8mm-venice-gw72xx-0x.dtb imx8mm-venice-gw72xx-0x-imx219.dtbo
 imx8mm-venice-gw72xx-0x-rpidsi-dtbs	:= imx8mm-venice-gw72xx-0x.dtb imx8mm-venice-gw72xx-0x-rpidsi.dtbo
diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-lte.dtso b/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-lte.dtso
new file mode 100644
index 0000000000000..324004b0eca3e
--- /dev/null
+++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-lte.dtso
@@ -0,0 +1,186 @@
+// SPDX-License-Identifier: GPL-2.0+ OR MIT
+/*
+ * Copyright (C) 2025 Kontron Electronics GmbH
+ */
+
+/dts-v1/;
+/plugin/;
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/input/input.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+#include <dt-bindings/leds/common.h>
+#include "imx8mm-pinfunc.h"
+
+&{/} {
+	compatible = "kontron,imx8mm-bl", "kontron,imx8mm-sl", "fsl,imx8mm";
+
+	gpio-keys {
+		compatible = "gpio-keys";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_gpio_keys>;
+
+		key-user {
+			label = "user";
+			linux,code = <BTN_0>;
+			gpios = <&gpio4 12 GPIO_ACTIVE_LOW>;
+		};
+	};
+
+	leds {
+		compatible = "gpio-leds";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_gpio_led_lte>;
+
+		lte-led1-b {
+			label = "lte-led1-blue";
+			color = <LED_COLOR_ID_BLUE>;
+			gpios = <&gpio3 17 GPIO_ACTIVE_HIGH>;
+		};
+
+		lte-led1-g {
+			label = "lte-led1-green";
+			color = <LED_COLOR_ID_GREEN>;
+			gpios = <&gpio3 18 GPIO_ACTIVE_HIGH>;
+		};
+
+		lte-led1-r {
+			label = "lte-led1-red";
+			color = <LED_COLOR_ID_RED>;
+			gpios = <&gpio5 23 GPIO_ACTIVE_HIGH>;
+		};
+
+		lte-led2-b {
+			label = "lte-led2-blue";
+			color = <LED_COLOR_ID_BLUE>;
+			gpios = <&gpio5 25 GPIO_ACTIVE_HIGH>;
+		};
+
+		lte-led2-g {
+			label = "lte-led2-green";
+			color = <LED_COLOR_ID_GREEN>;
+			gpios = <&gpio5 22 GPIO_ACTIVE_HIGH>;
+		};
+
+		lte-led2-r {
+			label = "lte-led2-red";
+			color = <LED_COLOR_ID_RED>;
+			gpios = <&gpio5 24 GPIO_ACTIVE_HIGH>;
+		};
+	};
+};
+
+&ecspi3 {
+	status = "disabled";
+};
+
+&i2c2 {
+	#address-cells = <1>;
+	#size-cells = <0>;
+	clock-frequency = <400000>;
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_i2c2>;
+	status = "okay";
+
+	tpm@2e {
+		compatible = "infineon,slb9673", "tcg,tpm-tis-i2c";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_tpm>;
+		reg = <0x2e>;
+		interrupt-parent = <&gpio3>;
+		interrupts = <11 IRQ_TYPE_LEVEL_LOW>;
+	};
+};
+
+&gpio3 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_gpio3>;
+	gpio-line-names = "", "", "", "",
+			  "", "", "", "",
+			  "", "", "VDD_IO_REF", "TPM_PIRQ#",
+			  "TPM_RESET# ", "", "", "",
+			  "", "LTE_LED1_B", "LTE_LED1_G", "",
+			  "";
+
+	vdd-io-ref-hog {
+		gpio-hog;
+		gpios = <10 GPIO_ACTIVE_HIGH>;
+		line-name = "VDD_IO_REF";
+		output-high;
+	};
+
+	tpm-reset-hog {
+		gpio-hog;
+		gpios = <12 GPIO_ACTIVE_LOW>;
+		line-name = "TPM_RESET#";
+		output-low;
+	};
+};
+
+&gpio4 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_gpio4>;
+	gpio-line-names = "", "", "LTE_RESET", "",
+			  "", "", "", "",
+			  "", "", "", "LTE_PWRKEY",
+			  "", "", "", "",
+			  "", "", "", "",
+			  "LTE_PWR_EN";
+};
+
+&gpio5 {
+	gpio-line-names = "", "", "", "",
+			  "", "", "", "",
+			  "", "", "", "",
+			  "", "", "", "",
+			  "", "", "", "",
+			  "", "", "LTE_LED2_G", "LTE_LED1_R",
+			  "LTE_LED2_R", "LTE_LED2_B";
+};
+
+&iomuxc {
+	pinctrl_gpio3: gpio3grp {
+		fsl,pins = <
+			MX8MM_IOMUXC_NAND_DATA04_GPIO3_IO10		0x19 /* VDD_IO_REF */
+		>;
+	};
+
+	pinctrl_gpio4: gpio4grp {
+		fsl,pins = <
+			MX8MM_IOMUXC_SAI1_RXD0_GPIO4_IO2		0x19 /* LTE_RESET */
+			MX8MM_IOMUXC_SAI1_TXC_GPIO4_IO11 		0x19 /* LTE_PWRKEY */
+			MX8MM_IOMUXC_SAI1_MCLK_GPIO4_IO20		0x19 /* LTE_PWR_EN */
+		>;
+	};
+
+	pinctrl_gpio_keys: gpiokeysgrp {
+		fsl,pins = <
+			MX8MM_IOMUXC_SAI1_TXD0_GPIO4_IO12		0x19 /* Pushbutton */
+		>;
+	};
+
+	pinctrl_gpio_led_lte: gpioledltegrp {
+		fsl,pins = <
+			MX8MM_IOMUXC_NAND_WE_B_GPIO3_IO17		0x19 /* LTE_LED1_B */
+			MX8MM_IOMUXC_NAND_WP_B_GPIO3_IO18		0x19 /* LTE_LED1_G */
+			MX8MM_IOMUXC_UART1_TXD_GPIO5_IO23		0x19 /* LTE_LED1_R */
+			MX8MM_IOMUXC_UART2_TXD_GPIO5_IO25		0x19 /* LTE_LED2_B */
+			MX8MM_IOMUXC_UART1_RXD_GPIO5_IO22		0x19 /* LTE_LED2_G */
+			MX8MM_IOMUXC_UART2_RXD_GPIO5_IO24		0x19 /* LTE_LED2_R */
+		>;
+	};
+
+	pinctrl_i2c2: i2c2grp {
+		fsl,pins = <
+			MX8MM_IOMUXC_I2C2_SCL_I2C2_SCL			0x40000083 /* I2C_A_SCL */
+			MX8MM_IOMUXC_I2C2_SDA_I2C2_SDA			0x40000083 /* I2C_A_SDA */
+		>;
+	};
+
+	pinctrl_tpm: tpmgrp {
+		fsl,pins = <
+			MX8MM_IOMUXC_NAND_DATA05_GPIO3_IO11		0x19 /* TPM_PIRQ# */
+			MX8MM_IOMUXC_NAND_DATA06_GPIO3_IO12		0x39 /* TPM_RESET# */
+		>;
+	};
+};
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 02/12] arm64: dts: imx8mm-kontron: Remove unused regulator
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 01/12] arm64: dts: imx8mm-kontron: Add overlay for LTE extension board Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 03/12] arm64: dts: imx8mm-kontron: Use GPIO for RS485 transceiver control Frieder Schrempf
                   ` (9 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Frieder Schrempf, Fabio Estevam, Pengutronix Kernel Team

From: Frieder Schrempf <frieder.schrempf@kontron.de>

This regulator is not used anywhere and is not available in hardware.
Remove it.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-osm-s.dts | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-osm-s.dts b/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-osm-s.dts
index 33f8d7d1970e0..3a166cf0afcb7 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-osm-s.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl-osm-s.dts
@@ -48,14 +48,6 @@ pwm-beeper {
 		pwms = <&pwm2 0 5000 0>;
 	};
 
-	reg_rst_eth2: regulator-rst-eth2 {
-		compatible = "regulator-fixed";
-		gpio = <&gpio1 1 GPIO_ACTIVE_HIGH>;
-		enable-active-high;
-		regulator-always-on;
-		regulator-name = "rst-usb-eth2";
-	};
-
 	reg_vdd_5v: regulator-5v {
 		compatible = "regulator-fixed";
 		regulator-always-on;
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 03/12] arm64: dts: imx8mm-kontron: Use GPIO for RS485 transceiver control
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 01/12] arm64: dts: imx8mm-kontron: Add overlay for LTE extension board Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 02/12] arm64: dts: imx8mm-kontron: Remove unused regulator Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 04/12] arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL devicetree Frieder Schrempf
                   ` (8 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Eberhard Stoll, Frieder Schrempf, Fabio Estevam,
	Pengutronix Kernel Team

From: Eberhard Stoll <eberhard.stoll@kontron.de>

For this IP the correct control of the CTS signal for transceiver direction
switching is difficult and - maybe also buggy - in the driver. Especially
the bootup requires special handling for most hardware implementations.

Therefore we simply use a GPIO now, which is fully under software control
and which is not problematic on bootup.

Signed-off-by: Eberhard Stoll <eberhard.stoll@kontron.de>
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm64/boot/dts/freescale/imx8mm-kontron-bl.dts | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl.dts b/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl.dts
index d16490d876874..d96c07757e1d2 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-bl.dts
@@ -268,8 +268,16 @@ &uart1 {
 &uart2 {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_uart2>;
+	/*
+	 * During bootup the CTS needs to stay LOW, which is only possible if this
+	 * pin is controlled by a GPIO. The UART IP always sets CTS to HIGH if not
+	 * running. So using 'uart-has-rtscts' is not a good choice here! There are
+	 * workarounds for this, but they introduce unneccessary complexity and are
+	 * therefore avoided here. For more information about this see:
+	 * https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit?id=79d0224f6bf296d04cd843cfc49921b19c97bb09
+	 */
+	rts-gpios = <&gpio4 29 GPIO_ACTIVE_HIGH>;
 	linux,rs485-enabled-at-boot-time;
-	uart-has-rtscts;
 	status = "okay";
 };
 
@@ -439,7 +447,7 @@ pinctrl_uart2: uart2grp {
 			MX8MM_IOMUXC_SAI3_TXFS_UART2_DCE_RX		0x0
 			MX8MM_IOMUXC_SAI3_TXC_UART2_DCE_TX		0x0
 			MX8MM_IOMUXC_SAI3_RXD_UART2_DCE_RTS_B		0x0
-			MX8MM_IOMUXC_SAI3_RXC_UART2_DCE_CTS_B		0x0
+			MX8MM_IOMUXC_SAI3_RXC_GPIO4_IO29		0x19
 		>;
 	};
 
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 04/12] arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL devicetree
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (2 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 03/12] arm64: dts: imx8mm-kontron: Use GPIO for RS485 transceiver control Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 16:21   ` Fabio Estevam
  2025-07-14 14:17 ` [PATCH 05/12] arm64: dts: imx8mm-kontron: Sort reg nodes alphabetically Frieder Schrempf
                   ` (7 subsequent siblings)
  11 siblings, 1 reply; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Oualid Derouiche, Frieder Schrempf, Fabio Estevam,
	Pengutronix Kernel Team

From: Oualid Derouiche <oualid.derouiche@kontron.de>

Some new panels have the Sitronix touch instead of Goodix. Support them
by adding a node for the new controller. The bootloader needs to detect
the correct controller and enable/disable the nodes accordingly.

Signed-off-by: Oualid Derouiche <oualid.derouiche@kontron.de>
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtso | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtso b/arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtso
index 1db27731b581c..5682ea9c5c7f7 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtso
+++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-dl.dtso
@@ -117,6 +117,17 @@ touchscreen@5d {
 		reset-gpios = <&gpio3 23 0>;
 		irq-gpios = <&gpio3 22 0>;
 	};
+
+	st1633@55 {
+		compatible = "sitronix,st1633";
+		reg = <0x55>;
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_touch>;
+		interrupts = <22 8>;
+		interrupt-parent = <&gpio3>;
+		gpios = <&gpio3 22 0>;
+		status = "disabled";
+	};
 };
 
 &lvds {
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 05/12] arm64: dts: imx8mm-kontron: Sort reg nodes alphabetically
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (3 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 04/12] arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL devicetree Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 06/12] arm64: dts: imx8mm-kontron: Name USB regulators according to OSM scheme Frieder Schrempf
                   ` (6 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Frieder Schrempf, Fabio Estevam, Pengutronix Kernel Team

From: Frieder Schrempf <frieder.schrempf@kontron.de>

Sort the regulator nodes alphabetically.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 .../dts/freescale/imx8mm-kontron-osm-s.dtsi   | 46 +++++++++----------
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi
index d455429652305..264553248d5cc 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi
@@ -30,29 +30,6 @@ chosen {
 		stdout-path = &uart3;
 	};
 
-	reg_vdd_carrier: regulator-vdd-carrier {
-		compatible = "regulator-fixed";
-		pinctrl-names = "default";
-		pinctrl-0 = <&pinctrl_reg_vdd_carrier>;
-		gpio = <&gpio3 16 GPIO_ACTIVE_HIGH>;
-		enable-active-high;
-		regulator-always-on;
-		regulator-boot-on;
-		regulator-name = "VDD_CARRIER";
-
-		regulator-state-standby {
-			regulator-on-in-suspend;
-		};
-
-		regulator-state-mem {
-			regulator-off-in-suspend;
-		};
-
-		regulator-state-disk {
-			regulator-off-in-suspend;
-		};
-	};
-
 	reg_usb1_vbus: regulator-usb1-vbus {
 		compatible = "regulator-fixed";
 		pinctrl-names = "default";
@@ -96,6 +73,29 @@ reg_usdhc3_vcc: regulator-usdhc3-vcc {
 		regulator-max-microvolt = <3300000>;
 		regulator-name = "VCC_SDIO_B";
 	};
+
+	reg_vdd_carrier: regulator-vdd-carrier {
+		compatible = "regulator-fixed";
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_reg_vdd_carrier>;
+		gpio = <&gpio3 16 GPIO_ACTIVE_HIGH>;
+		enable-active-high;
+		regulator-always-on;
+		regulator-boot-on;
+		regulator-name = "VDD_CARRIER";
+
+		regulator-state-standby {
+			regulator-on-in-suspend;
+		};
+
+		regulator-state-mem {
+			regulator-off-in-suspend;
+		};
+
+		regulator-state-disk {
+			regulator-off-in-suspend;
+		};
+	};
 };
 
 &A53_0 {
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 06/12] arm64: dts: imx8mm-kontron: Name USB regulators according to OSM scheme
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (4 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 05/12] arm64: dts: imx8mm-kontron: Sort reg nodes alphabetically Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 07/12] arm64: dts: imx8mp-kontron: Fix CAN_ADDR0 and CAN_ADDR1 GPIOs Frieder Schrempf
                   ` (5 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Frieder Schrempf, Fabio Estevam, Pengutronix Kernel Team

From: Frieder Schrempf <frieder.schrempf@kontron.de>

Use the names from the OSM specification.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi b/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi
index 264553248d5cc..96987910609f1 100644
--- a/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8mm-kontron-osm-s.dtsi
@@ -38,7 +38,7 @@ reg_usb1_vbus: regulator-usb1-vbus {
 		gpio = <&gpio3 25 GPIO_ACTIVE_HIGH>;
 		regulator-min-microvolt = <5000000>;
 		regulator-max-microvolt = <5000000>;
-		regulator-name = "VBUS_USB1";
+		regulator-name = "VBUS_USB_A";
 	};
 
 	reg_usb2_vbus: regulator-usb2-vbus {
@@ -49,7 +49,7 @@ reg_usb2_vbus: regulator-usb2-vbus {
 		gpio = <&gpio3 20 GPIO_ACTIVE_HIGH>;
 		regulator-min-microvolt = <5000000>;
 		regulator-max-microvolt = <5000000>;
-		regulator-name = "VBUS_USB2";
+		regulator-name = "VBUS_USB_B";
 	};
 
 	reg_usdhc2_vcc: regulator-usdhc2-vcc {
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 07/12] arm64: dts: imx8mp-kontron: Fix CAN_ADDR0 and CAN_ADDR1 GPIOs
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (5 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 06/12] arm64: dts: imx8mm-kontron: Name USB regulators according to OSM scheme Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 08/12] arm64: dts: imx8mp-kontron: Fix GPIO labels for latest BL board Frieder Schrempf
                   ` (4 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Annette Kobou, Frieder Schrempf, Fabio Estevam,
	Pengutronix Kernel Team

From: Annette Kobou <annette.kobou@kontron.de>

Some signal assignments were modified between hardware revisions
1 and 2:

Revision 1:

  - SPI_A_WP   -> CAN_ADDR0
  - SPI_A_HOLD -> CAN_ADDR1

Revision 2 and later:

  - SPI_A_SDI -> CAN_ADDR0
  - SPI_A_SDO -> CAN_ADDR1

Fix the labels and add the missing pinctrls.

Signed-off-by: Annette Kobou <annette.kobou@kontron.de>
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 .../dts/freescale/imx8mp-kontron-bl-osm-s.dts | 31 ++++++++++++++++---
 1 file changed, 27 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts b/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
index 0eb9e726a9b81..4aa5c261b865d 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
@@ -123,14 +123,12 @@ &gpio2 {
 
 /*
  * Rename SoM signals according to board usage:
- *   SPI_A_WP      -> CAN_ADDR0
- *   SPI_A_HOLD    -> CAN_ADDR1
  *   GPIO_B_0      -> DIO1_OUT
  *   GPIO_B_1      -> DIO2_OUT
  */
 &gpio3 {
 	gpio-line-names = "PCIE_WAKE", "PCIE_CLKREQ", "PCIE_A_PERST", "SDIO_B_D5",
-			  "SDIO_B_D6", "SDIO_B_D7", "CAN_ADDR0", "CAN_ADDR1",
+			  "SDIO_B_D6", "SDIO_B_D7", "SPI_A_WP", "SPI_A_HOLD",
 			  "UART_B_RTS", "UART_B_CTS", "SDIO_B_D0", "SDIO_B_D1",
 			  "SDIO_B_D2", "SDIO_B_D3", "SDIO_B_WP", "SDIO_B_D4",
 			  "PCIE_SM_ALERT", "SDIO_B_CLK", "SDIO_B_CMD", "DIO1_OUT",
@@ -159,6 +157,24 @@ &gpio4 {
 			  "GPIO_A_7", "CARRIER_PWR_EN", "I2S_A_DATA_IN", "I2S_LRCLK";
 };
 
+/*
+ * Rename SoM signals according to board usage:
+ *   SPI_A_SDI	-> CAN_ADDR0
+ *   SPI_A_SDO	-> CAN_ADDR1
+ */
+&gpio5 {
+	pinctrl-names = "default";
+	pinctrl-0 = <&pinctrl_gpio5>;
+	gpio-line-names = "I2S_BITCLK", "I2S_A_DATA_OUT", "I2S_MCLK", "PWM_2",
+			  "PWM_1", "PWM_0", "SPI_A_SCK", "CAN_ADDR1",
+			  "CAN_ADDR0", "SPI_A_CS0", "SPI_B_SCK", "SPI_B_SDO",
+			  "SPI_B_SDI", "SPI_B_CS0", "I2C_A_SCL", "I2C_A_SDA",
+			  "I2C_B_SCL", "I2C_B_SDA", "PCIE_SMCLK", "PCIE_SMDAT",
+			  "I2C_CAM_SCL", "I2C_CAM_SDA", "UART_A_RX", "UART_A_TX",
+			  "UART_C_RX", "UART_C_TX", "UART_CON_RX", "UART_CON_TX",
+			  "UART_B_RX", "UART_B_TX";
+};
+
 &hdmi_pvi {
 	status = "okay";
 };
@@ -297,9 +313,16 @@ MX8MP_IOMUXC_SD2_WP__GPIO2_IO20			0x46
 		>;
 	};
 
+	pinctrl_gpio5: gpio5grp {
+		fsl,pins = <
+			MX8MP_IOMUXC_ECSPI1_MOSI__GPIO5_IO07		0x46 /* CAN_ADR0 */
+			MX8MP_IOMUXC_ECSPI1_MISO__GPIO5_IO08		0x46 /* CAN_ADR1 */
+		>;
+	};
+
 	pinctrl_usb_hub: usbhubgrp {
 		fsl,pins = <
 			MX8MP_IOMUXC_NAND_DQS__GPIO3_IO14		0x46
 		>;
 	};
-};
+};
\ No newline at end of file
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 08/12] arm64: dts: imx8mp-kontron: Fix GPIO labels for latest BL board
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (6 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 07/12] arm64: dts: imx8mp-kontron: Fix CAN_ADDR0 and CAN_ADDR1 GPIOs Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 09/12] arm64: dts: imx8mp-kontron: Fix USB hub reset Frieder Schrempf
                   ` (3 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Frieder Schrempf, Annette Kobou, Fabio Estevam,
	Pengutronix Kernel Team

From: Frieder Schrempf <frieder.schrempf@kontron.de>

Hardware rev 3 changed a few signals. Reflect these changes in
the GPIO labels.

* digital IOs were moved to GPIO expander
* remove labels for unused signals
* add labels for TFT, CSI and USB hub IOs

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 .../dts/freescale/imx8mp-kontron-bl-osm-s.dts | 30 +++++++++----------
 1 file changed, 14 insertions(+), 16 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts b/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
index 4aa5c261b865d..bbcd76e9e9913 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
@@ -123,38 +123,36 @@ &gpio2 {
 
 /*
  * Rename SoM signals according to board usage:
- *   GPIO_B_0      -> DIO1_OUT
- *   GPIO_B_1      -> DIO2_OUT
+ *   GPIO_B_0      -> IO_EXP_INT
+ *   GPIO_B_1      -> IO_EXP_RST
  */
 &gpio3 {
 	gpio-line-names = "PCIE_WAKE", "PCIE_CLKREQ", "PCIE_A_PERST", "SDIO_B_D5",
 			  "SDIO_B_D6", "SDIO_B_D7", "SPI_A_WP", "SPI_A_HOLD",
 			  "UART_B_RTS", "UART_B_CTS", "SDIO_B_D0", "SDIO_B_D1",
 			  "SDIO_B_D2", "SDIO_B_D3", "SDIO_B_WP", "SDIO_B_D4",
-			  "PCIE_SM_ALERT", "SDIO_B_CLK", "SDIO_B_CMD", "DIO1_OUT",
-			  "DIO2_OUT", "", "BOOT_SEL0", "BOOT_SEL1",
+			  "PCIE_SM_ALERT", "SDIO_B_CLK", "SDIO_B_CMD", "IO_EXP_INT",
+			  "IO_EXP_RST", "", "BOOT_SEL0", "BOOT_SEL1",
 			  "", "", "SDIO_B_CD", "SDIO_B_PWR_EN",
 			  "HDMI_CEC", "HDMI_HPD";
 };
 
 /*
- * Rename SoM signals according to board usage:
- *   GPIO_B_5      -> DIO2_IN
- *   GPIO_B_6      -> DIO3_IN
- *   GPIO_B_7      -> DIO4_IN
- *   GPIO_B_3      -> DIO4_OUT
- *   GPIO_B_4      -> DIO1_IN
- *   GPIO_B_2      -> DIO3_OUT
+ * Rename SoM signals according to board usage and remove labels for unsed pins:
+ *   GPIO_A_6      -> TFT_RESET
+ *   GPIO_A_7      -> TFT_STBY
+ *   GPIO_B_3      -> CSI_ENABLE
+ *   GPIO_B_2      -> USB_HUB_RST
  */
 &gpio4 {
-	gpio-line-names = "DIO2_IN", "DIO3_IN", "DIO4_IN", "GPIO_C_0",
+	gpio-line-names = "", "", "", "",
 			  "ETH_A_MDC", "ETH_A_MDIO", "ETH_A_RXD0", "ETH_A_RXD1",
 			  "ETH_A_RXD2", "ETH_A_RXD3", "ETH_A_RX_DV", "ETH_A_RX_CLK",
 			  "ETH_A_TXD0", "ETH_A_TXD1", "ETH_A_TXD2", "ETH_A_TXD3",
-			  "ETH_A_TX_EN", "ETH_A_TX_CLK", "DIO4_OUT", "DIO1_IN",
-			  "DIO3_OUT", "GPIO_A_6", "CAN_A_TX", "UART_A_CTS",
+			  "ETH_A_TX_EN", "ETH_A_TX_CLK", "CSI_ENABLE", "",
+			  "USB_HUB_RST", "TFT_RESET", "CAN_A_TX", "UART_A_CTS",
 			  "UART_A_RTS", "CAN_A_RX", "CAN_B_TX", "CAN_B_RX",
-			  "GPIO_A_7", "CARRIER_PWR_EN", "I2S_A_DATA_IN", "I2S_LRCLK";
+			  "TFT_STBY", "CARRIER_PWR_EN", "I2S_A_DATA_IN", "I2S_LRCLK";
 };
 
 /*
@@ -325,4 +323,4 @@ pinctrl_usb_hub: usbhubgrp {
 			MX8MP_IOMUXC_NAND_DQS__GPIO3_IO14		0x46
 		>;
 	};
-};
\ No newline at end of file
+};
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 09/12] arm64: dts: imx8mp-kontron: Fix USB hub reset
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (7 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 08/12] arm64: dts: imx8mp-kontron: Fix GPIO labels for latest BL board Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 10/12] arm64: dts: imx93-kontron: Add RTC interrupt signal Frieder Schrempf
                   ` (2 subsequent siblings)
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Frieder Schrempf, Annette Kobou, Fabio Estevam,
	Pengutronix Kernel Team

From: Frieder Schrempf <frieder.schrempf@kontron.de>

The latest hardware revision uses GPIO_B_2 as reset for
the USB hub. Fix this and remove the pinctrl as we already
have this in the OSM-S devicetree.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 .../boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts     | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts b/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
index bbcd76e9e9913..614b4ce330b1c 100644
--- a/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
+++ b/arch/arm64/boot/dts/freescale/imx8mp-kontron-bl-osm-s.dts
@@ -250,8 +250,6 @@ &usb_dwc3_0 {
 };
 
 &usb_dwc3_1 {
-	pinctrl-names = "default";
-	pinctrl-0 = <&pinctrl_usb_hub>;
 	#address-cells = <1>;
 	#size-cells = <0>;
 	dr_mode = "host";
@@ -260,7 +258,7 @@ &usb_dwc3_1 {
 	usb-hub@1 {
 		compatible = "usb424,2514";
 		reg = <1>;
-		reset-gpios = <&gpio3 14 GPIO_ACTIVE_LOW>;
+		reset-gpios = <&gpio4 20 GPIO_ACTIVE_LOW>;
 	};
 };
 
@@ -317,10 +315,4 @@ MX8MP_IOMUXC_ECSPI1_MOSI__GPIO5_IO07		0x46 /* CAN_ADR0 */
 			MX8MP_IOMUXC_ECSPI1_MISO__GPIO5_IO08		0x46 /* CAN_ADR1 */
 		>;
 	};
-
-	pinctrl_usb_hub: usbhubgrp {
-		fsl,pins = <
-			MX8MP_IOMUXC_NAND_DQS__GPIO3_IO14		0x46
-		>;
-	};
 };
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 10/12] arm64: dts: imx93-kontron: Add RTC interrupt signal
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (8 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 09/12] arm64: dts: imx8mp-kontron: Fix USB hub reset Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-16  6:21   ` Primoz Fiser
  2025-07-14 14:17 ` [PATCH 11/12] arm64: dts: imx93-kontron: Fix GPIO for panel regulator Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 12/12] arm64: dts: imx93-kontron: Fix USB port assignment Frieder Schrempf
  11 siblings, 1 reply; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Frieder Schrempf, Fabio Estevam, Pengutronix Kernel Team

From: Frieder Schrempf <frieder.schrempf@kontron.de>

The RTC EVI ouptut is connected to a GPIO. Add the interrupt
so it can be used by the RTC driver.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi b/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
index 119a162070596..c79b1df339db1 100644
--- a/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
@@ -205,6 +205,9 @@ eeprom@50 {
 	rv3028: rtc@52 {
 		compatible = "microcrystal,rv3028";
 		reg = <0x52>;
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_rtc>;
+		interrupts-extended = <&gpio3 19 IRQ_TYPE_LEVEL_LOW>;
 	};
 };
 
@@ -468,6 +471,12 @@ MX93_PAD_CCM_CLKO4__GPIO4_IO29			0x31e /* CARRIER_PWR_EN */
 		>;
 	};
 
+	pinctrl_rtc: rtcgrp {
+		fsl,pins = <
+			MX93_PAD_SD2_VSELECT__GPIO3_IO19		0x31e
+		>;
+	};
+
 	pinctrl_sai3: sai3grp {
 		fsl,pins = <
 			MX93_PAD_GPIO_IO20__SAI3_RX_DATA00		0x31e /* I2S_A_DATA_IN */
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 11/12] arm64: dts: imx93-kontron: Fix GPIO for panel regulator
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (9 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 10/12] arm64: dts: imx93-kontron: Add RTC interrupt signal Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  2025-07-14 14:17 ` [PATCH 12/12] arm64: dts: imx93-kontron: Fix USB port assignment Frieder Schrempf
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, Frieder Schrempf, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Annette Kobou, Fabio Estevam, Pengutronix Kernel Team

From: Annette Kobou <annette.kobou@kontron.de>

The regulator uses the wrong GPIO. Fix this.

Signed-off-by: Annette Kobou <annette.kobou@kontron.de>
Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Fixes: 2b52fd6035b7 ("arm64: dts: Add support for Kontron i.MX93 OSM-S SoM and BL carrier board")
---
 .../boot/dts/freescale/imx93-kontron-bl-osm-s.dts    | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts b/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts
index 89e97c604bd3e..9a9e5d0daf3ba 100644
--- a/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts
+++ b/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts
@@ -33,7 +33,9 @@ pwm-beeper {
 
 	reg_vcc_panel: regulator-vcc-panel {
 		compatible = "regulator-fixed";
-		gpio = <&gpio4 3 GPIO_ACTIVE_HIGH>;
+		pinctrl-names = "default";
+		pinctrl-0 = <&pinctrl_reg_vcc_panel>;
+		gpio = <&gpio2 21 GPIO_ACTIVE_HIGH>;
 		enable-active-high;
 		regulator-max-microvolt = <3300000>;
 		regulator-min-microvolt = <3300000>;
@@ -161,3 +163,11 @@ &usdhc2 {
 	vmmc-supply = <&reg_vdd_3v3>;
 	status = "okay";
 };
+
+&iomuxc {
+	pinctrl_reg_vcc_panel: regvccpanelgrp {
+		fsl,pins = <
+			MX93_PAD_GPIO_IO21__GPIO2_IO21		0x31e /* PWM_2 */
+		>;
+	};
+};
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* [PATCH 12/12] arm64: dts: imx93-kontron: Fix USB port assignment
  2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
                   ` (10 preceding siblings ...)
  2025-07-14 14:17 ` [PATCH 11/12] arm64: dts: imx93-kontron: Fix GPIO for panel regulator Frieder Schrempf
@ 2025-07-14 14:17 ` Frieder Schrempf
  11 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-14 14:17 UTC (permalink / raw)
  To: linux-arm-kernel, Conor Dooley, devicetree, Frieder Schrempf, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Annette Kobou, Fabio Estevam, Pengutronix Kernel Team

From: Frieder Schrempf <frieder.schrempf@kontron.de>

The assignment of the USB ports is wrong and needs to be swapped.
The OTG (USB-C) port is on the first port and the host port with
the onboard hub is on the second port.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Fixes: 2b52fd6035b7 ("arm64: dts: Add support for Kontron i.MX93 OSM-S SoM and BL carrier board")
---
 .../dts/freescale/imx93-kontron-bl-osm-s.dts  | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts b/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts
index 9a9e5d0daf3ba..c3d2ddd887fdf 100644
--- a/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts
+++ b/arch/arm64/boot/dts/freescale/imx93-kontron-bl-osm-s.dts
@@ -137,6 +137,16 @@ &tpm6 {
 };
 
 &usbotg1 {
+	adp-disable;
+	hnp-disable;
+	srp-disable;
+	disable-over-current;
+	dr_mode = "otg";
+	usb-role-switch;
+	status = "okay";
+};
+
+&usbotg2 {
 	#address-cells = <1>;
 	#size-cells = <0>;
 	disable-over-current;
@@ -149,16 +159,6 @@ usb1@1 {
 	};
 };
 
-&usbotg2 {
-	adp-disable;
-	hnp-disable;
-	srp-disable;
-	disable-over-current;
-	dr_mode = "otg";
-	usb-role-switch;
-	status = "okay";
-};
-
 &usdhc2 {
 	vmmc-supply = <&reg_vdd_3v3>;
 	status = "okay";
-- 
2.50.1


^ permalink raw reply related	[flat|nested] 17+ messages in thread

* Re: [PATCH 04/12] arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL devicetree
  2025-07-14 14:17 ` [PATCH 04/12] arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL devicetree Frieder Schrempf
@ 2025-07-14 16:21   ` Fabio Estevam
  2025-07-15  6:39     ` Frieder Schrempf
  0 siblings, 1 reply; 17+ messages in thread
From: Fabio Estevam @ 2025-07-14 16:21 UTC (permalink / raw)
  To: Frieder Schrempf
  Cc: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo, Oualid Derouiche, Frieder Schrempf,
	Pengutronix Kernel Team

On Mon, Jul 14, 2025 at 11:20 AM Frieder Schrempf <frieder@fris.de> wrote:

> +
> +       st1633@55 {

Node name should be generic: touchscreen@55

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 04/12] arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL devicetree
  2025-07-14 16:21   ` Fabio Estevam
@ 2025-07-15  6:39     ` Frieder Schrempf
  0 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-15  6:39 UTC (permalink / raw)
  To: Fabio Estevam, Frieder Schrempf
  Cc: linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo, Oualid Derouiche, Pengutronix Kernel Team

Am 14.07.25 um 18:21 schrieb Fabio Estevam:
> On Mon, Jul 14, 2025 at 11:20 AM Frieder Schrempf <frieder@fris.de> wrote:
> 
>> +
>> +       st1633@55 {
> 
> Node name should be generic: touchscreen@55

Right, will be fixed in v2.

^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 10/12] arm64: dts: imx93-kontron: Add RTC interrupt signal
  2025-07-14 14:17 ` [PATCH 10/12] arm64: dts: imx93-kontron: Add RTC interrupt signal Frieder Schrempf
@ 2025-07-16  6:21   ` Primoz Fiser
  2025-07-16  6:28     ` Frieder Schrempf
  0 siblings, 1 reply; 17+ messages in thread
From: Primoz Fiser @ 2025-07-16  6:21 UTC (permalink / raw)
  To: Frieder Schrempf, linux-arm-kernel, Conor Dooley, devicetree, imx,
	Krzysztof Kozlowski, linux-kernel, Rob Herring, Sascha Hauer,
	Shawn Guo
  Cc: Frieder Schrempf, Fabio Estevam, Pengutronix Kernel Team

Hi Frieder,

On 14. 07. 25 16:17, Frieder Schrempf wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>
> The RTC EVI ouptut is connected to a GPIO. Add the interrupt
> so it can be used by the RTC driver.

AFAIK, RV3028's EVI is an input pin (EVent Input).

Please check.

BR,

Primoz

>
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> ---
>  arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi | 9 +++++++++
>  1 file changed, 9 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi b/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
> index 119a162070596..c79b1df339db1 100644
> --- a/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
> @@ -205,6 +205,9 @@ eeprom@50 {
>  	rv3028: rtc@52 {
>  		compatible = "microcrystal,rv3028";
>  		reg = <0x52>;
> +		pinctrl-names = "default";
> +		pinctrl-0 = <&pinctrl_rtc>;
> +		interrupts-extended = <&gpio3 19 IRQ_TYPE_LEVEL_LOW>;
>  	};
>  };
>  
> @@ -468,6 +471,12 @@ MX93_PAD_CCM_CLKO4__GPIO4_IO29			0x31e /* CARRIER_PWR_EN */
>  		>;
>  	};
>  
> +	pinctrl_rtc: rtcgrp {
> +		fsl,pins = <
> +			MX93_PAD_SD2_VSELECT__GPIO3_IO19		0x31e
> +		>;
> +	};
> +
>  	pinctrl_sai3: sai3grp {
>  		fsl,pins = <
>  			MX93_PAD_GPIO_IO20__SAI3_RX_DATA00		0x31e /* I2S_A_DATA_IN */


-- 
Primoz Fiser
phone: +386-41-390-545
email: primoz.fiser@norik.com
--
Norik systems d.o.o.
Your embedded software partner
Slovenia, EU
phone: +386-41-540-545
email: info@norik.com


^ permalink raw reply	[flat|nested] 17+ messages in thread

* Re: [PATCH 10/12] arm64: dts: imx93-kontron: Add RTC interrupt signal
  2025-07-16  6:21   ` Primoz Fiser
@ 2025-07-16  6:28     ` Frieder Schrempf
  0 siblings, 0 replies; 17+ messages in thread
From: Frieder Schrempf @ 2025-07-16  6:28 UTC (permalink / raw)
  To: Primoz Fiser, Frieder Schrempf, linux-arm-kernel, Conor Dooley,
	devicetree, imx, Krzysztof Kozlowski, linux-kernel, Rob Herring,
	Sascha Hauer, Shawn Guo
  Cc: Fabio Estevam, Pengutronix Kernel Team

Hi Primoz,

Am 16.07.25 um 08:21 schrieb Primoz Fiser:
> Hi Frieder,
> 
> On 14. 07. 25 16:17, Frieder Schrempf wrote:
>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>
>> The RTC EVI ouptut is connected to a GPIO. Add the interrupt
>> so it can be used by the RTC driver.
> 
> AFAIK, RV3028's EVI is an input pin (EVent Input).
> 
> Please check.

Thanks for noticing! It seems like the commit message is wrong here. The
patch is about the INT signal which is an output on the RTC. I will fix
the description in v2.

Thanks
Frieder

>>
>> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
>> ---
>>  arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi | 9 +++++++++
>>  1 file changed, 9 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi b/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
>> index 119a162070596..c79b1df339db1 100644
>> --- a/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
>> +++ b/arch/arm64/boot/dts/freescale/imx93-kontron-osm-s.dtsi
>> @@ -205,6 +205,9 @@ eeprom@50 {
>>  	rv3028: rtc@52 {
>>  		compatible = "microcrystal,rv3028";
>>  		reg = <0x52>;
>> +		pinctrl-names = "default";
>> +		pinctrl-0 = <&pinctrl_rtc>;
>> +		interrupts-extended = <&gpio3 19 IRQ_TYPE_LEVEL_LOW>;
>>  	};
>>  };
>>  
>> @@ -468,6 +471,12 @@ MX93_PAD_CCM_CLKO4__GPIO4_IO29			0x31e /* CARRIER_PWR_EN */
>>  		>;
>>  	};
>>  
>> +	pinctrl_rtc: rtcgrp {
>> +		fsl,pins = <
>> +			MX93_PAD_SD2_VSELECT__GPIO3_IO19		0x31e
>> +		>;
>> +	};
>> +
>>  	pinctrl_sai3: sai3grp {
>>  		fsl,pins = <
>>  			MX93_PAD_GPIO_IO20__SAI3_RX_DATA00		0x31e /* I2S_A_DATA_IN */
> 
> 


^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2025-07-16  6:28 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-14 14:17 [PATCH 00/12] Misc fixups and changes for Kontron i.MX8M and i.MX93 DTs Frieder Schrempf
2025-07-14 14:17 ` [PATCH 01/12] arm64: dts: imx8mm-kontron: Add overlay for LTE extension board Frieder Schrempf
2025-07-14 14:17 ` [PATCH 02/12] arm64: dts: imx8mm-kontron: Remove unused regulator Frieder Schrempf
2025-07-14 14:17 ` [PATCH 03/12] arm64: dts: imx8mm-kontron: Use GPIO for RS485 transceiver control Frieder Schrempf
2025-07-14 14:17 ` [PATCH 04/12] arm64: dts: imx8mm-kontron: Add Sitronix touch controller in DL devicetree Frieder Schrempf
2025-07-14 16:21   ` Fabio Estevam
2025-07-15  6:39     ` Frieder Schrempf
2025-07-14 14:17 ` [PATCH 05/12] arm64: dts: imx8mm-kontron: Sort reg nodes alphabetically Frieder Schrempf
2025-07-14 14:17 ` [PATCH 06/12] arm64: dts: imx8mm-kontron: Name USB regulators according to OSM scheme Frieder Schrempf
2025-07-14 14:17 ` [PATCH 07/12] arm64: dts: imx8mp-kontron: Fix CAN_ADDR0 and CAN_ADDR1 GPIOs Frieder Schrempf
2025-07-14 14:17 ` [PATCH 08/12] arm64: dts: imx8mp-kontron: Fix GPIO labels for latest BL board Frieder Schrempf
2025-07-14 14:17 ` [PATCH 09/12] arm64: dts: imx8mp-kontron: Fix USB hub reset Frieder Schrempf
2025-07-14 14:17 ` [PATCH 10/12] arm64: dts: imx93-kontron: Add RTC interrupt signal Frieder Schrempf
2025-07-16  6:21   ` Primoz Fiser
2025-07-16  6:28     ` Frieder Schrempf
2025-07-14 14:17 ` [PATCH 11/12] arm64: dts: imx93-kontron: Fix GPIO for panel regulator Frieder Schrempf
2025-07-14 14:17 ` [PATCH 12/12] arm64: dts: imx93-kontron: Fix USB port assignment Frieder Schrempf

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