public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH v2 0/4] Properly support FriendlyElec HD702E
@ 2026-01-23 19:22 Robin Murphy
  2026-01-23 19:22 ` [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP Robin Murphy
                   ` (4 more replies)
  0 siblings, 5 replies; 11+ messages in thread
From: Robin Murphy @ 2026-01-23 19:22 UTC (permalink / raw)
  To: heiko, neil.armstrong, dianders, thierry.reding, sam
  Cc: jesszhan0024, dri-devel, linux-rockchip, linux-arm-kernel

v1: https://lore.kernel.org/linux-rockchip/cover.1767111804.git.robin.murphy@arm.com/

Changelogs in the patches - fingers crossed this is good to go now,
thanks all for the acks and reviews.

Cheers,
Robin.


Robin Murphy (4):
  dt-bindings: display: panel: Move FriendlyElec HD702E to eDP
  drm/panel-edp: Move FriendlyELEC HD702E
  arm64: dts: rockchip: Move RK3399 eDP pinctrl to boards
  arm64: dts: rockchip: Add overlay for FriendlyElec HD702E

 .../display/panel/panel-edp-legacy.yaml       |  2 +
 .../bindings/display/panel/panel-simple.yaml  |  2 -
 arch/arm64/boot/dts/rockchip/Makefile         |  5 +
 arch/arm64/boot/dts/rockchip/rk3399-base.dtsi |  3 +-
 .../dts/rockchip/rk3399-gru-chromebook.dtsi   |  2 +
 .../dts/rockchip/rk3399-nanopc-t4-hd702e.dtso | 96 +++++++++++++++++++
 .../boot/dts/rockchip/rk3399-pinebook-pro.dts |  2 -
 .../rockchip/rk3399-sapphire-excavator.dts    |  2 +
 drivers/gpu/drm/panel/panel-edp.c             | 26 +++++
 drivers/gpu/drm/panel/panel-simple.c          | 25 -----
 10 files changed, 134 insertions(+), 31 deletions(-)
 create mode 100644 arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso

-- 
2.34.1



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

* [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP
  2026-01-23 19:22 [PATCH v2 0/4] Properly support FriendlyElec HD702E Robin Murphy
@ 2026-01-23 19:22 ` Robin Murphy
  2026-01-24 23:51   ` Doug Anderson
  2026-02-02 16:52   ` Doug Anderson
  2026-01-23 19:22 ` [PATCH v2 2/4] drm/panel-edp: Move FriendlyELEC HD702E Robin Murphy
                   ` (3 subsequent siblings)
  4 siblings, 2 replies; 11+ messages in thread
From: Robin Murphy @ 2026-01-23 19:22 UTC (permalink / raw)
  To: heiko, neil.armstrong, dianders, thierry.reding, sam
  Cc: jesszhan0024, dri-devel, linux-rockchip, linux-arm-kernel,
	devicetree, Rob Herring

The "E" alludes to the fact that FriendlyElec's HD702E is actually an
eDP panel - move its compatible to the appropriate binding doc.

Cc: <devicetree@vger.kernel.org>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
Signed-off-by: Robin Murphy <robin.murphy@arm.com>
---
 .../devicetree/bindings/display/panel/panel-edp-legacy.yaml     | 2 ++
 .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 --
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/display/panel/panel-edp-legacy.yaml b/Documentation/devicetree/bindings/display/panel/panel-edp-legacy.yaml
index b308047c1edf..afe7dc54ebf4 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-edp-legacy.yaml
+++ b/Documentation/devicetree/bindings/display/panel/panel-edp-legacy.yaml
@@ -44,6 +44,8 @@ properties:
       - boe,nv133fhm-n62
         # BOE NV140FHM-N49 14.0" FHD a-Si FT panel
       - boe,nv140fhmn49
+        # FriendlyELEC HD702E 800x1280 LCD panel
+      - friendlyarm,hd702e
         # Innolux Corporation 11.6" WXGA (1366x768) TFT LCD panel
       - innolux,n116bca-ea1
         # Innolux Corporation 11.6" WXGA (1366x768) TFT LCD panel
diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
index 24e277b19094..a01cf025aad1 100644
--- a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
+++ b/Documentation/devicetree/bindings/display/panel/panel-simple.yaml
@@ -144,8 +144,6 @@ properties:
       - foxlink,fl500wvr00-a0t
         # Frida FRD350H54004 3.5" QVGA TFT LCD panel
       - frida,frd350h54004
-        # FriendlyELEC HD702E 800x1280 LCD panel
-      - friendlyarm,hd702e
         # GiantPlus GPG48273QS5 4.3" (480x272) WQVGA TFT LCD panel
       - giantplus,gpg48273qs5
         # GiantPlus GPM940B0 3.0" QVGA TFT LCD panel
-- 
2.34.1



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

* [PATCH v2 2/4] drm/panel-edp: Move FriendlyELEC HD702E
  2026-01-23 19:22 [PATCH v2 0/4] Properly support FriendlyElec HD702E Robin Murphy
  2026-01-23 19:22 ` [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP Robin Murphy
@ 2026-01-23 19:22 ` Robin Murphy
  2026-01-24 23:52   ` Doug Anderson
  2026-01-23 19:22 ` [PATCH v2 3/4] arm64: dts: rockchip: Move RK3399 eDP pinctrl to boards Robin Murphy
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 11+ messages in thread
From: Robin Murphy @ 2026-01-23 19:22 UTC (permalink / raw)
  To: heiko, neil.armstrong, dianders, thierry.reding, sam
  Cc: jesszhan0024, dri-devel, linux-rockchip, linux-arm-kernel

FriendlyELEC's HD702E module is an eDP panel (in as much as it's some
LVDS LCD behind a Chrontel CH7511B eDP bridge), so move its data over
to the eDP driver, also resolving the warning about the missing bpc
value in the process.

The unfortunate combination of HPD not being wired up and the RK3399 eDP
controller's behaviour seems to result in the EDID not being readable
over DP-AUX without probing the panel first, thus the hard-coded mode is
still needed to get things going.

Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
Signed-off-by: Robin Murphy <robin.murphy@arm.com>
---
v2: Clarify issue with the generic binding
---
 drivers/gpu/drm/panel/panel-edp.c    | 26 ++++++++++++++++++++++++++
 drivers/gpu/drm/panel/panel-simple.c | 25 -------------------------
 2 files changed, 26 insertions(+), 25 deletions(-)

diff --git a/drivers/gpu/drm/panel/panel-edp.c b/drivers/gpu/drm/panel/panel-edp.c
index 415b894890ad..dd53ccc209ce 100644
--- a/drivers/gpu/drm/panel/panel-edp.c
+++ b/drivers/gpu/drm/panel/panel-edp.c
@@ -1256,6 +1256,29 @@ static const struct panel_desc boe_nv140fhmn49 = {
 	},
 };
 
+static const struct drm_display_mode friendlyarm_hd702e_mode = {
+	.clock		= 67185,
+	.hdisplay	= 800,
+	.hsync_start	= 800 + 20,
+	.hsync_end	= 800 + 20 + 24,
+	.htotal		= 800 + 20 + 24 + 20,
+	.vdisplay	= 1280,
+	.vsync_start	= 1280 + 4,
+	.vsync_end	= 1280 + 4 + 8,
+	.vtotal		= 1280 + 4 + 8 + 4,
+	.flags		= DRM_MODE_FLAG_NVSYNC | DRM_MODE_FLAG_NHSYNC,
+};
+
+static const struct panel_desc friendlyarm_hd702e = {
+	.modes = &friendlyarm_hd702e_mode,
+	.num_modes = 1,
+	.bpc = 8,
+	.size = {
+		.width	= 94,
+		.height	= 151,
+	},
+};
+
 static const struct drm_display_mode innolux_n116bca_ea1_mode = {
 	.clock = 76420,
 	.hdisplay = 1366,
@@ -1663,6 +1686,9 @@ static const struct of_device_id platform_of_match[] = {
 	}, {
 		.compatible = "boe,nv140fhmn49",
 		.data = &boe_nv140fhmn49,
+	}, {
+		.compatible = "friendlyarm,hd702e",
+		.data = &friendlyarm_hd702e,
 	}, {
 		.compatible = "innolux,n116bca-ea1",
 		.data = &innolux_n116bca_ea1,
diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-simple.c
index b26b682826bc..3ea52667b858 100644
--- a/drivers/gpu/drm/panel/panel-simple.c
+++ b/drivers/gpu/drm/panel/panel-simple.c
@@ -2359,28 +2359,6 @@ static const struct panel_desc frida_frd350h54004 = {
 	.connector_type = DRM_MODE_CONNECTOR_DPI,
 };
 
-static const struct drm_display_mode friendlyarm_hd702e_mode = {
-	.clock		= 67185,
-	.hdisplay	= 800,
-	.hsync_start	= 800 + 20,
-	.hsync_end	= 800 + 20 + 24,
-	.htotal		= 800 + 20 + 24 + 20,
-	.vdisplay	= 1280,
-	.vsync_start	= 1280 + 4,
-	.vsync_end	= 1280 + 4 + 8,
-	.vtotal		= 1280 + 4 + 8 + 4,
-	.flags		= DRM_MODE_FLAG_NVSYNC | DRM_MODE_FLAG_NHSYNC,
-};
-
-static const struct panel_desc friendlyarm_hd702e = {
-	.modes = &friendlyarm_hd702e_mode,
-	.num_modes = 1,
-	.size = {
-		.width	= 94,
-		.height	= 151,
-	},
-};
-
 static const struct drm_display_mode giantplus_gpg482739qs5_mode = {
 	.clock = 9000,
 	.hdisplay = 480,
@@ -5235,9 +5213,6 @@ static const struct of_device_id platform_of_match[] = {
 	}, {
 		.compatible = "frida,frd350h54004",
 		.data = &frida_frd350h54004,
-	}, {
-		.compatible = "friendlyarm,hd702e",
-		.data = &friendlyarm_hd702e,
 	}, {
 		.compatible = "giantplus,gpg482739qs5",
 		.data = &giantplus_gpg482739qs5
-- 
2.34.1



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

* [PATCH v2 3/4] arm64: dts: rockchip: Move RK3399 eDP pinctrl to boards
  2026-01-23 19:22 [PATCH v2 0/4] Properly support FriendlyElec HD702E Robin Murphy
  2026-01-23 19:22 ` [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP Robin Murphy
  2026-01-23 19:22 ` [PATCH v2 2/4] drm/panel-edp: Move FriendlyELEC HD702E Robin Murphy
@ 2026-01-23 19:22 ` Robin Murphy
  2026-01-23 19:22 ` [PATCH v2 4/4] arm64: dts: rockchip: Add overlay for FriendlyElec HD702E Robin Murphy
  2026-02-22 22:39 ` (subset) [PATCH v2 0/4] Properly support " Heiko Stuebner
  4 siblings, 0 replies; 11+ messages in thread
From: Robin Murphy @ 2026-01-23 19:22 UTC (permalink / raw)
  To: heiko, neil.armstrong, dianders, thierry.reding, sam
  Cc: jesszhan0024, dri-devel, linux-rockchip, linux-arm-kernel,
	Dragan Simic

The EDP_HOTPLUG pin is optional, and muxed with other functions (notably
HDMI CEC), so move its selection from the SoC DTSI to the boards which
apparently want it, namely those which enable eDP without "force-hpd".
By the same token we drop it from Pinebook Pro, which already uses
"force-hpd", and according to the schematics does not have the pin wired
at all.

Reviewed-by: Dragan Simic <dsimic@manjaro.org>
Signed-off-by: Robin Murphy <robin.murphy@arm.com>
---
v2: Add /omit-if-no-ref/ attribute [Dragan]
---
 arch/arm64/boot/dts/rockchip/rk3399-base.dtsi              | 3 +--
 arch/arm64/boot/dts/rockchip/rk3399-gru-chromebook.dtsi    | 2 ++
 arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts       | 2 --
 arch/arm64/boot/dts/rockchip/rk3399-sapphire-excavator.dts | 2 ++
 4 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-base.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-base.dtsi
index 4dcceb9136b7..49c68fe65de6 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-base.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-base.dtsi
@@ -2145,8 +2145,6 @@ edp: dp@ff970000 {
 		interrupts = <GIC_SPI 10 IRQ_TYPE_LEVEL_HIGH 0>;
 		clocks = <&cru PCLK_EDP>, <&cru PCLK_EDP_CTRL>, <&cru PCLK_VIO_GRF>;
 		clock-names = "dp", "pclk", "grf";
-		pinctrl-names = "default";
-		pinctrl-0 = <&edp_hpd>;
 		power-domains = <&power RK3399_PD_EDP>;
 		resets = <&cru SRST_P_EDP_CTRL>;
 		reset-names = "dp";
@@ -2384,6 +2382,7 @@ cif_clkouta: cif-clkouta {
 		};
 
 		edp {
+			/omit-if-no-ref/
 			edp_hpd: edp-hpd {
 				rockchip,pins =
 					<4 RK_PC7 2 &pcfg_pull_none>;
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-gru-chromebook.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-gru-chromebook.dtsi
index 9d07353df52c..3f3cb0eb5809 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-gru-chromebook.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-gru-chromebook.dtsi
@@ -241,6 +241,8 @@ &dmc {
 };
 
 &edp {
+	pinctrl-names = "default";
+	pinctrl-0 = <&edp_hpd>;
 	status = "okay";
 
 	/*
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts b/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts
index eaaca08a7601..dcab04863d28 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dts
@@ -401,8 +401,6 @@ &cpu_l3 {
 
 &edp {
 	force-hpd;
-	pinctrl-names = "default";
-	pinctrl-0 = <&edp_hpd>;
 	status = "okay";
 };
 
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-sapphire-excavator.dts b/arch/arm64/boot/dts/rockchip/rk3399-sapphire-excavator.dts
index a4ceafe6dd7a..80d6ea0eda84 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-sapphire-excavator.dts
+++ b/arch/arm64/boot/dts/rockchip/rk3399-sapphire-excavator.dts
@@ -141,6 +141,8 @@ sdio_pwrseq: sdio-pwrseq {
 };
 
 &edp {
+	pinctrl-names = "default";
+	pinctrl-0 = <&edp_hpd>;
 	status = "okay";
 };
 
-- 
2.34.1



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

* [PATCH v2 4/4] arm64: dts: rockchip: Add overlay for FriendlyElec HD702E
  2026-01-23 19:22 [PATCH v2 0/4] Properly support FriendlyElec HD702E Robin Murphy
                   ` (2 preceding siblings ...)
  2026-01-23 19:22 ` [PATCH v2 3/4] arm64: dts: rockchip: Move RK3399 eDP pinctrl to boards Robin Murphy
@ 2026-01-23 19:22 ` Robin Murphy
  2026-02-22 13:59   ` Heiko Stuebner
  2026-02-22 22:39 ` (subset) [PATCH v2 0/4] Properly support " Heiko Stuebner
  4 siblings, 1 reply; 11+ messages in thread
From: Robin Murphy @ 2026-01-23 19:22 UTC (permalink / raw)
  To: heiko, neil.armstrong, dianders, thierry.reding, sam
  Cc: jesszhan0024, dri-devel, linux-rockchip, linux-arm-kernel

Add an overlay to support FriendlyElec's HD702E 7" eDP LCD touchscreen
module for the NanoPC-T4 board:

https://www.friendlyelec.com/index.php?route=product/product&path=81&product_id=230

Signed-off-by: Robin Murphy <robin.murphy@arm.com>
---
v2:
 - Add pinctrl, tweak some naming [Doug]
 - Add main 3.3v regulator to tidy up power supplies
---
 arch/arm64/boot/dts/rockchip/Makefile         |  5 +
 .../dts/rockchip/rk3399-nanopc-t4-hd702e.dtso | 96 +++++++++++++++++++
 2 files changed, 101 insertions(+)
 create mode 100644 arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso

diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile
index dbdda9783e93..4ab58a5961aa 100644
--- a/arch/arm64/boot/dts/rockchip/Makefile
+++ b/arch/arm64/boot/dts/rockchip/Makefile
@@ -62,6 +62,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-khadas-edge-v.dtb
 dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-kobol-helios64.dtb
 dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-leez-p710.dtb
 dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-nanopc-t4.dtb
+dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-nanopc-t4-hd702e.dtbo
 dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-nanopi-m4.dtb
 dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-nanopi-m4b.dtb
 dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-nanopi-neo4.dtb
@@ -238,6 +239,10 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += px30-ringneck-haikou-haikou-video-demo.dtb
 px30-ringneck-haikou-haikou-video-demo-dtbs := px30-ringneck-haikou.dtb \
 	px30-ringneck-haikou-video-demo.dtbo
 
+dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-nanopc-t4-hd702e.dtb
+rk3399-nanopc-t4-hd702e-dtbs := rk3399-nanopc-t4.dtb \
+	rk3399-nanopc-t4-hd702e.dtbo
+
 dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-puma-haikou-haikou-video-demo.dtb
 rk3399-puma-haikou-haikou-video-demo-dtbs := rk3399-puma-haikou.dtb \
 	rk3399-puma-haikou-video-demo.dtbo
diff --git a/arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso b/arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso
new file mode 100644
index 000000000000..66ac530392b7
--- /dev/null
+++ b/arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso
@@ -0,0 +1,96 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+/*
+ * FriendlyElec HD702E LCD on NanoPC-T4 board
+ */
+
+/dts-v1/;
+/plugin/;
+
+#include <dt-bindings/gpio/gpio.h>
+#include <dt-bindings/interrupt-controller/irq.h>
+#include <dt-bindings/pinctrl/rockchip.h>
+
+&{/} {
+	vdd_3_3v: regulator-vdd_3_3v {
+		compatible = "regulator-fixed";
+		regulator-always-on;
+		regulator-boot-on;
+		regulator-min-microvolt = <3300000>;
+		regulator-max-microvolt = <3300000>;
+		regulator-name = "vdd_3.3v";
+		vin-supply = <&vcc12v0_sys>;
+	};
+
+	backlight: backlight {
+		compatible = "pwm-backlight";
+		pwms = <&pwm0 0 25000 0>;
+		enable-gpios = <&gpio4 RK_PD5 GPIO_ACTIVE_HIGH>;
+		brightness-levels = <0 255>;
+		default-brightness-level = <200>;
+		num-interpolated-steps = <255>;
+		pinctrl-0 = <&bl_en>;
+		pinctrl-names = "default";
+	};
+};
+
+&edp {
+	force-hpd;
+	status = "okay";
+
+	aux-bus {
+		edp-panel {
+			compatible = "friendlyarm,hd702e";
+			backlight = <&backlight>;
+			no-hpd;
+			power-supply = <&vdd_3_3v>;
+
+			port {
+				panel_in_edp: endpoint {
+					remote-endpoint = <&edp_out_panel>;
+				};
+			};
+		};
+	};
+};
+
+&edp_out {
+	edp_out_panel: endpoint {
+		remote-endpoint = <&panel_in_edp>;
+	};
+};
+
+&i2c4 {
+	#address-cells = <1>;
+	#size-cells = <0>;
+
+	touchscreen@5d {
+		compatible = "goodix,gt9271";
+		reg = <0x5d>;
+		interrupt-parent = <&gpio1>;
+		interrupts = <RK_PC4 IRQ_TYPE_EDGE_FALLING>;
+		irq-gpios = <&gpio1 RK_PC4 GPIO_ACTIVE_HIGH>;
+		reset-gpios = <&gpio1 RK_PB5 GPIO_ACTIVE_HIGH>;
+		pinctrl-0 = <&touch_int &touch_rst>;
+		pinctrl-names = "default";
+		AVDD28-supply = <&vdd_3_3v>;
+		VDDIO-supply = <&vdd_3_3v>;
+	};
+};
+
+&pinctrl {
+	backlight {
+		bl_en: bl-en {
+			rockchip,pins = <4 RK_PD5 RK_FUNC_GPIO &pcfg_pull_none>;
+		};
+	};
+
+	touchscreen {
+		touch_int: touch-int {
+			rockchip,pins = <1 RK_PC4 RK_FUNC_GPIO &pcfg_pull_none>;
+		};
+
+		touch_rst: touch-rst {
+			rockchip,pins = <1 RK_PB5 RK_FUNC_GPIO &pcfg_pull_none>;
+		};
+	};
+};
-- 
2.34.1



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

* Re: [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP
  2026-01-23 19:22 ` [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP Robin Murphy
@ 2026-01-24 23:51   ` Doug Anderson
  2026-02-02 16:52   ` Doug Anderson
  1 sibling, 0 replies; 11+ messages in thread
From: Doug Anderson @ 2026-01-24 23:51 UTC (permalink / raw)
  To: Robin Murphy
  Cc: heiko, neil.armstrong, thierry.reding, sam, jesszhan0024,
	dri-devel, linux-rockchip, linux-arm-kernel, devicetree,
	Rob Herring

Hi,

On Fri, Jan 23, 2026 at 11:22 AM Robin Murphy <robin.murphy@arm.com> wrote:
>
> The "E" alludes to the fact that FriendlyElec's HD702E is actually an
> eDP panel - move its compatible to the appropriate binding doc.
>
> Cc: <devicetree@vger.kernel.org>
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> ---
>  .../devicetree/bindings/display/panel/panel-edp-legacy.yaml     | 2 ++
>  .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 --
>  2 files changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Douglas Anderson <dianders@chromium.org>


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

* Re: [PATCH v2 2/4] drm/panel-edp: Move FriendlyELEC HD702E
  2026-01-23 19:22 ` [PATCH v2 2/4] drm/panel-edp: Move FriendlyELEC HD702E Robin Murphy
@ 2026-01-24 23:52   ` Doug Anderson
  2026-02-02 16:52     ` Doug Anderson
  0 siblings, 1 reply; 11+ messages in thread
From: Doug Anderson @ 2026-01-24 23:52 UTC (permalink / raw)
  To: Robin Murphy
  Cc: heiko, neil.armstrong, thierry.reding, sam, jesszhan0024,
	dri-devel, linux-rockchip, linux-arm-kernel

Hi,

On Fri, Jan 23, 2026 at 11:22 AM Robin Murphy <robin.murphy@arm.com> wrote:
>
> FriendlyELEC's HD702E module is an eDP panel (in as much as it's some
> LVDS LCD behind a Chrontel CH7511B eDP bridge), so move its data over
> to the eDP driver, also resolving the warning about the missing bpc
> value in the process.
>
> The unfortunate combination of HPD not being wired up and the RK3399 eDP
> controller's behaviour seems to result in the EDID not being readable
> over DP-AUX without probing the panel first, thus the hard-coded mode is
> still needed to get things going.
>
> Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> ---
> v2: Clarify issue with the generic binding
> ---
>  drivers/gpu/drm/panel/panel-edp.c    | 26 ++++++++++++++++++++++++++
>  drivers/gpu/drm/panel/panel-simple.c | 25 -------------------------
>  2 files changed, 26 insertions(+), 25 deletions(-)

Reviewed-by: Douglas Anderson <dianders@chromium.org>

I'll plan to apply patches #1 and #2 in ~1 week to drm-misc-next if
there are no other comments. If someone else wants to beat me to it,
though, I'm also OK w/ that.

-Doug


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

* Re: [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP
  2026-01-23 19:22 ` [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP Robin Murphy
  2026-01-24 23:51   ` Doug Anderson
@ 2026-02-02 16:52   ` Doug Anderson
  1 sibling, 0 replies; 11+ messages in thread
From: Doug Anderson @ 2026-02-02 16:52 UTC (permalink / raw)
  To: Robin Murphy
  Cc: heiko, neil.armstrong, thierry.reding, sam, jesszhan0024,
	dri-devel, linux-rockchip, linux-arm-kernel, devicetree,
	Rob Herring

Hi,

On Fri, Jan 23, 2026 at 11:22 AM Robin Murphy <robin.murphy@arm.com> wrote:
>
> The "E" alludes to the fact that FriendlyElec's HD702E is actually an
> eDP panel - move its compatible to the appropriate binding doc.
>
> Cc: <devicetree@vger.kernel.org>
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
> Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> ---
>  .../devicetree/bindings/display/panel/panel-edp-legacy.yaml     | 2 ++
>  .../devicetree/bindings/display/panel/panel-simple.yaml         | 2 --
>  2 files changed, 2 insertions(+), 2 deletions(-)

Pushed patches to drm-misc-next:

[1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP
      commit: e2242223b714ac9fd8233546ca74ce972024111f


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

* Re: [PATCH v2 2/4] drm/panel-edp: Move FriendlyELEC HD702E
  2026-01-24 23:52   ` Doug Anderson
@ 2026-02-02 16:52     ` Doug Anderson
  0 siblings, 0 replies; 11+ messages in thread
From: Doug Anderson @ 2026-02-02 16:52 UTC (permalink / raw)
  To: Robin Murphy
  Cc: heiko, neil.armstrong, thierry.reding, sam, jesszhan0024,
	dri-devel, linux-rockchip, linux-arm-kernel

Hi,

On Sat, Jan 24, 2026 at 3:52 PM Doug Anderson <dianders@chromium.org> wrote:
>
> Hi,
>
> On Fri, Jan 23, 2026 at 11:22 AM Robin Murphy <robin.murphy@arm.com> wrote:
> >
> > FriendlyELEC's HD702E module is an eDP panel (in as much as it's some
> > LVDS LCD behind a Chrontel CH7511B eDP bridge), so move its data over
> > to the eDP driver, also resolving the warning about the missing bpc
> > value in the process.
> >
> > The unfortunate combination of HPD not being wired up and the RK3399 eDP
> > controller's behaviour seems to result in the EDID not being readable
> > over DP-AUX without probing the panel first, thus the hard-coded mode is
> > still needed to get things going.
> >
> > Reviewed-by: Neil Armstrong <neil.armstrong@linaro.org>
> > Signed-off-by: Robin Murphy <robin.murphy@arm.com>
> > ---
> > v2: Clarify issue with the generic binding
> > ---
> >  drivers/gpu/drm/panel/panel-edp.c    | 26 ++++++++++++++++++++++++++
> >  drivers/gpu/drm/panel/panel-simple.c | 25 -------------------------
> >  2 files changed, 26 insertions(+), 25 deletions(-)
>
> Reviewed-by: Douglas Anderson <dianders@chromium.org>
>
> I'll plan to apply patches #1 and #2 in ~1 week to drm-misc-next if
> there are no other comments. If someone else wants to beat me to it,
> though, I'm also OK w/ that.

Pushed patches to drm-misc-next:

[2/4] drm/panel-edp: Move FriendlyELEC HD702E
      commit: 5c323ea948ff0f9e50418119cbbb3d94f2bd6a96


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

* Re: [PATCH v2 4/4] arm64: dts: rockchip: Add overlay for FriendlyElec HD702E
  2026-01-23 19:22 ` [PATCH v2 4/4] arm64: dts: rockchip: Add overlay for FriendlyElec HD702E Robin Murphy
@ 2026-02-22 13:59   ` Heiko Stuebner
  0 siblings, 0 replies; 11+ messages in thread
From: Heiko Stuebner @ 2026-02-22 13:59 UTC (permalink / raw)
  To: neil.armstrong, dianders, thierry.reding, sam, Robin Murphy
  Cc: jesszhan0024, dri-devel, linux-rockchip, linux-arm-kernel

Am Freitag, 23. Januar 2026, 20:22:23 Mitteleuropäische Normalzeit schrieb Robin Murphy:
> Add an overlay to support FriendlyElec's HD702E 7" eDP LCD touchscreen
> module for the NanoPC-T4 board:
> 
> https://www.friendlyelec.com/index.php?route=product/product&path=81&product_id=230
> 
> Signed-off-by: Robin Murphy <robin.murphy@arm.com>

[...]

> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso b/arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso
> new file mode 100644
> index 000000000000..66ac530392b7
> --- /dev/null
> +++ b/arch/arm64/boot/dts/rockchip/rk3399-nanopc-t4-hd702e.dtso
> @@ -0,0 +1,96 @@
> +&edp {
> +	force-hpd;
> +	status = "okay";
> +
> +	aux-bus {
> +		edp-panel {

The dtbs-check was unhappy, because the auxbus binding wishes for this
node to be named "panel". I'll just change this when applying.


Heiko

> +			compatible = "friendlyarm,hd702e";
> +			backlight = <&backlight>;
> +			no-hpd;
> +			power-supply = <&vdd_3_3v>;
> +
> +			port {
> +				panel_in_edp: endpoint {
> +					remote-endpoint = <&edp_out_panel>;
> +				};
> +			};
> +		};
> +	};
> +};





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

* Re: (subset) [PATCH v2 0/4] Properly support FriendlyElec HD702E
  2026-01-23 19:22 [PATCH v2 0/4] Properly support FriendlyElec HD702E Robin Murphy
                   ` (3 preceding siblings ...)
  2026-01-23 19:22 ` [PATCH v2 4/4] arm64: dts: rockchip: Add overlay for FriendlyElec HD702E Robin Murphy
@ 2026-02-22 22:39 ` Heiko Stuebner
  4 siblings, 0 replies; 11+ messages in thread
From: Heiko Stuebner @ 2026-02-22 22:39 UTC (permalink / raw)
  To: neil.armstrong, dianders, thierry.reding, sam, Robin Murphy
  Cc: Heiko Stuebner, jesszhan0024, dri-devel, linux-rockchip,
	linux-arm-kernel


On Fri, 23 Jan 2026 19:22:19 +0000, Robin Murphy wrote:
> v1: https://lore.kernel.org/linux-rockchip/cover.1767111804.git.robin.murphy@arm.com/
> 
> Changelogs in the patches - fingers crossed this is good to go now,
> thanks all for the acks and reviews.
> 
> Cheers,
> Robin.
> 
> [...]

Applied, thanks!

[3/4] arm64: dts: rockchip: Move RK3399 eDP pinctrl to boards
      commit: 638fa970ac669e5d0fb2c5cc0bea3d4443299ac6
[4/4] arm64: dts: rockchip: Add overlay for FriendlyElec HD702E
      commit: 25afddad23b8e91fcff625756e51a805e288ab38

Best regards,
-- 
Heiko Stuebner <heiko@sntech.de>


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

end of thread, other threads:[~2026-02-22 22:40 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-01-23 19:22 [PATCH v2 0/4] Properly support FriendlyElec HD702E Robin Murphy
2026-01-23 19:22 ` [PATCH v2 1/4] dt-bindings: display: panel: Move FriendlyElec HD702E to eDP Robin Murphy
2026-01-24 23:51   ` Doug Anderson
2026-02-02 16:52   ` Doug Anderson
2026-01-23 19:22 ` [PATCH v2 2/4] drm/panel-edp: Move FriendlyELEC HD702E Robin Murphy
2026-01-24 23:52   ` Doug Anderson
2026-02-02 16:52     ` Doug Anderson
2026-01-23 19:22 ` [PATCH v2 3/4] arm64: dts: rockchip: Move RK3399 eDP pinctrl to boards Robin Murphy
2026-01-23 19:22 ` [PATCH v2 4/4] arm64: dts: rockchip: Add overlay for FriendlyElec HD702E Robin Murphy
2026-02-22 13:59   ` Heiko Stuebner
2026-02-22 22:39 ` (subset) [PATCH v2 0/4] Properly support " Heiko Stuebner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox