From: Frank Li <Frank.li@oss.nxp.com>
To: Steffen Trumtrar <s.trumtrar@pengutronix.de>
Cc: "Rob Herring" <robh@kernel.org>,
"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
"Conor Dooley" <conor+dt@kernel.org>,
"Frank Li" <Frank.Li@nxp.com>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Fabio Estevam" <festevam@gmail.com>,
"Shawn Guo" <shawnguo@kernel.org>, "Søren Andersen" <san@skov.dk>,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v9 2/3] arm64: dts: imx8mp-skov-revc: convert to overlays
Date: Thu, 8 Oct 2026 15:34:38 -0500 [thread overview]
Message-ID: <asf-XrI8a-M2m-Fl@SMW015318> (raw)
In-Reply-To: <20261008-v7-0-topic-imx8mp-skov-dts-jutouch-7inch-v9-2-11fa90f531fc@pengutronix.de>
On Thu, Oct 08, 2026 at 11:16:00AM +0200, Steffen Trumtrar wrote:
> The i.MX8MP Skov RevC jutouch 10" and Tian 7" variants are pretty
> similar, apart from the Display and touch controller.
>
> Move the common bindings to imx8mp-skov-revc-touch.dts and convert both
> variants to use overlays applied on this base version.
>
> Add a compatible for revC-based Skov boards with a touch panel for the
> common dts.
>
> Signed-off-by: Steffen Trumtrar <s.trumtrar@pengutronix.de>
> ---
> Documentation/devicetree/bindings/arm/fsl.yaml | 1 +
Binding change need seperate patch. This is v9.
Does rebase cause problem?
Frank
> arch/arm64/boot/dts/freescale/Makefile | 5 ++
> .../imx8mp-skov-revc-jutouch-jt101tm023.dtso | 38 ++++++++++
> .../dts/freescale/imx8mp-skov-revc-tian-g07017.dts | 81 ----------------------
> .../freescale/imx8mp-skov-revc-tian-g07017.dtso | 40 +++++++++++
> ...h-jt101tm023.dts => imx8mp-skov-revc-touch.dts} | 26 +++----
> 6 files changed, 92 insertions(+), 99 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> index 656f2596e1d53..5599e45ffdcec 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -1155,6 +1155,7 @@ properties:
> - skov,imx8mp-skov-revc-bd500 # SKOV i.MX8MP climate control with LED frontplate
> - skov,imx8mp-skov-revc-jutouch-jt101tm023 # SKOV i.MX8MP climate control with 10" JuTouch panel
> - skov,imx8mp-skov-revc-tian-g07017 # SKOV i.MX8MP climate control with 7" panel
> + - skov,imx8mp-skov-revc-touch # SKOV i.MX8MP climate control revision C with touch panel
> - ultratronik,imx8mp-ultra-mach-sbc # Ultratronik SBC i.MX8MP based board
> - ysoft,imx8mp-iota2-lumpy # Y Soft i.MX8MP IOTA2 Lumpy Board
> - const: fsl,imx8mp
> diff --git a/arch/arm64/boot/dts/freescale/Makefile b/arch/arm64/boot/dts/freescale/Makefile
> index 49025ae465f22..d706290b8dd02 100644
> --- a/arch/arm64/boot/dts/freescale/Makefile
> +++ b/arch/arm64/boot/dts/freescale/Makefile
> @@ -475,6 +475,11 @@ dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-lt6.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revb-mi1010ait-1cp1.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revc-bd500.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revc-hdmi.dtb
> +
> +imx8mp-skov-revc-jutouch-jt101tm023-dtbs += imx8mp-skov-revc-touch.dtb \
> + imx8mp-skov-revc-jutouch-jt101tm023.dtbo
> +imx8mp-skov-revc-tian-g07017-dtbs += imx8mp-skov-revc-touch.dtb \
> + imx8mp-skov-revc-tian-g07017.dtbo
> dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revc-jutouch-jt101tm023.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx8mp-skov-revc-tian-g07017.dtb
> dtb-$(CONFIG_ARCH_MXC) += imx8mp-toradex-smarc-dev.dtb
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-jutouch-jt101tm023.dtso b/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-jutouch-jt101tm023.dtso
> new file mode 100644
> index 0000000000000..33ddd08a7adb5
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-jutouch-jt101tm023.dtso
> @@ -0,0 +1,38 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +
> +/dts-v1/;
> +/plugin/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/interrupt-controller/irq.h>
> +
> +&{/} {
> + model = "SKOV IMX8MP CPU revC - JuTouch JT101TM023";
> + compatible = "skov,imx8mp-skov-revc-jutouch-jt101tm023", "fsl,imx8mp";
> +};
> +
> +&{/panel} {
> + compatible = "jutouch,jt101tm023";
> +};
> +
> +&i2c2 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + touchscreen@2a {
> + compatible = "eeti,exc81w32", "eeti,exc80h84";
> + reg = <0x2a>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_touchscreen>;
> + interrupts-extended = <&gpio4 28 IRQ_TYPE_LEVEL_LOW>;
> + reset-gpios = <&gpio4 29 GPIO_ACTIVE_LOW>;
> + touchscreen-size-x = <1280>;
> + touchscreen-size-y = <800>;
> + vdd-supply = <®_vdd_3v3>;
> + };
> +};
> +
> +&lvds_bridge {
> + /* IMX8MP_VIDEO_PLL1 = IMX8MP_CLK_MEDIA_DISP2_PIX * 7 */
> + assigned-clock-rates = <0>, <506800000>;
> +};
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-tian-g07017.dts b/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-tian-g07017.dts
> deleted file mode 100644
> index 9a562c011f2c2..0000000000000
> --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-tian-g07017.dts
> +++ /dev/null
> @@ -1,81 +0,0 @@
> -// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> -
> -/dts-v1/;
> -
> -#include "imx8mp-skov-reva.dtsi"
> -
> -/ {
> - model = "SKOV IMX8MP CPU revC - TIAN G07017";
> - compatible = "skov,imx8mp-skov-revc-tian-g07017", "fsl,imx8mp";
> -
> - panel {
> - compatible = "topland,tian-g07017-01";
> - backlight = <&backlight>;
> - power-supply = <®_tft_vcom>;
> -
> - port {
> - in_lvds0: endpoint {
> - remote-endpoint = <&ldb_lvds_ch0>;
> - };
> - };
> - };
> -};
> -
> -&backlight {
> - status = "okay";
> -};
> -
> -&i2c2 {
> - clock-frequency = <100000>;
> - status = "okay";
> -
> - touchscreen@38 {
> - compatible = "edt,edt-ft5506";
> - reg = <0x38>;
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_touchscreen>;
> - interrupts-extended = <&gpio4 28 IRQ_TYPE_EDGE_FALLING>;
> - reset-gpios = <&gpio4 29 GPIO_ACTIVE_LOW>;
> - touchscreen-size-x = <1024>;
> - touchscreen-size-y = <600>;
> - vcc-supply = <®_vdd_3v3>;
> - iovcc-supply = <®_vdd_3v3>;
> - wakeup-source;
> - };
> -};
> -
> -&lcdif2 {
> - status = "okay";
> -};
> -
> -&lvds_bridge {
> - assigned-clocks = <&clk IMX8MP_CLK_MEDIA_LDB>,
> - <&clk IMX8MP_VIDEO_PLL1>;
> - assigned-clock-parents = <&clk IMX8MP_VIDEO_PLL1_OUT>;
> - /* IMX8MP_VIDEO_PLL1 = IMX8MP_CLK_MEDIA_DISP2_PIX * 2 * 7 */
> - assigned-clock-rates = <0>, <358400000>;
> - status = "okay";
> -
> - ports {
> - port@1 {
> - ldb_lvds_ch0: endpoint {
> - remote-endpoint = <&in_lvds0>;
> - };
> - };
> - };
> -};
> -
> -&pwm4 {
> - status = "okay";
> -};
> -
> -&pwm1 {
> - status = "okay";
> -};
> -
> -®_tft_vcom {
> - regulator-min-microvolt = <3160000>;
> - regulator-max-microvolt = <3160000>;
> - voltage-table = <3160000 73>;
> - status = "okay";
> -};
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-tian-g07017.dtso b/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-tian-g07017.dtso
> new file mode 100644
> index 0000000000000..b9781a418e4d5
> --- /dev/null
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-tian-g07017.dtso
> @@ -0,0 +1,40 @@
> +// SPDX-License-Identifier: (GPL-2.0 OR MIT)
> +
> +/dts-v1/;
> +/plugin/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/interrupt-controller/irq.h>
> +
> +&{/} {
> + model = "SKOV IMX8MP CPU revC - TIAN G07017";
> + compatible = "skov,imx8mp-skov-revc-tian-g07017", "fsl,imx8mp";
> +};
> +
> +&{/panel} {
> + compatible = "topland,tian-g07017-01";
> +};
> +
> +&i2c2 {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + touchscreen@38 {
> + compatible = "edt,edt-ft5506";
> + reg = <0x38>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&pinctrl_touchscreen>;
> + interrupts-extended = <&gpio4 28 IRQ_TYPE_EDGE_FALLING>;
> + reset-gpios = <&gpio4 29 GPIO_ACTIVE_LOW>;
> + touchscreen-size-x = <1024>;
> + touchscreen-size-y = <600>;
> + vcc-supply = <®_vdd_3v3>;
> + iovcc-supply = <®_vdd_3v3>;
> + wakeup-source;
> + };
> +};
> +
> +&lvds_bridge {
> + /* IMX8MP_VIDEO_PLL1 = IMX8MP_CLK_MEDIA_DISP2_PIX * 2 * 7 */
> + assigned-clock-rates = <0>, <358400000>;
> +};
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-jutouch-jt101tm023.dts b/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-touch.dts
> similarity index 56%
> rename from arch/arm64/boot/dts/freescale/imx8mp-skov-revc-jutouch-jt101tm023.dts
> rename to arch/arm64/boot/dts/freescale/imx8mp-skov-revc-touch.dts
> index 3e320d6dea3a2..c830f3ca8cb45 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-jutouch-jt101tm023.dts
> +++ b/arch/arm64/boot/dts/freescale/imx8mp-skov-revc-touch.dts
> @@ -5,11 +5,10 @@
> #include "imx8mp-skov-reva.dtsi"
>
> / {
> - model = "SKOV IMX8MP CPU revC - JuTouch JT101TM023";
> - compatible = "skov,imx8mp-skov-revc-jutouch-jt101tm023", "fsl,imx8mp";
> + model = "SKOV IMX8MP CPU revC";
> + compatible = "skov,imx8mp-skov-revc-touch", "fsl,imx8mp";
>
> panel {
> - compatible = "jutouch,jt101tm023";
> backlight = <&backlight>;
> power-supply = <®_tft_vcom>;
>
> @@ -28,18 +27,6 @@ &backlight {
> &i2c2 {
> clock-frequency = <100000>;
> status = "okay";
> -
> - touchscreen@2a {
> - compatible = "eeti,exc81w32", "eeti,exc80h84";
> - reg = <0x2a>;
> - pinctrl-names = "default";
> - pinctrl-0 = <&pinctrl_touchscreen>;
> - interrupts-extended = <&gpio4 28 IRQ_TYPE_LEVEL_LOW>;
> - reset-gpios = <&gpio4 29 GPIO_ACTIVE_LOW>;
> - touchscreen-size-x = <1280>;
> - touchscreen-size-y = <800>;
> - vdd-supply = <®_vdd_3v3>;
> - };
> };
>
> &lcdif2 {
> @@ -48,14 +35,17 @@ &lcdif2 {
>
> &lvds_bridge {
> assigned-clocks = <&clk IMX8MP_CLK_MEDIA_LDB>,
> - <&clk IMX8MP_VIDEO_PLL1>;
> + <&clk IMX8MP_VIDEO_PLL1>;
> assigned-clock-parents = <&clk IMX8MP_VIDEO_PLL1_OUT>;
> - /* IMX8MP_VIDEO_PLL1 = IMX8MP_CLK_MEDIA_DISP2_PIX * 7 */
> - assigned-clock-rates = <0>, <506800000>;
> status = "okay";
>
> ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> port@1 {
> + reg = <1>;
> +
> ldb_lvds_ch0: endpoint {
> remote-endpoint = <&in_lvds0>;
> };
>
> --
> 2.51.0
>
>
next prev parent reply other threads:[~2026-10-08 20:34 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-08 9:15 [PATCH v9 0/3] arm64: dts: imx8mp-skov: add new 7" variant Steffen Trumtrar
2026-10-08 9:15 ` [PATCH v9 1/3] arm64: dts: freescale: imx8mp: skov: fix sort order Steffen Trumtrar
2026-10-09 10:21 ` Krzysztof Kozlowski
2026-10-08 9:16 ` [PATCH v9 2/3] arm64: dts: imx8mp-skov-revc: convert to overlays Steffen Trumtrar
2026-10-08 20:34 ` Frank Li [this message]
2026-10-09 10:22 ` Krzysztof Kozlowski
2026-10-08 9:16 ` [PATCH v9 3/3] arm64: dts: imx8mp-skov: support new 7inch panel board Steffen Trumtrar
2026-10-08 20:36 ` [PATCH v9 0/3] arm64: dts: imx8mp-skov: add new 7" variant Rob Herring
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=asf-XrI8a-M2m-Fl@SMW015318 \
--to=frank.li@oss.nxp.com \
--cc=Frank.Li@nxp.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=imx@lists.linux.dev \
--cc=kernel@pengutronix.de \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=robh@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=s.trumtrar@pengutronix.de \
--cc=san@skov.dk \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox