<html><head><meta name="color-scheme" content="light dark"></head><body><pre style="word-wrap: break-word; white-space: pre-wrap;">From 2a77bbf7f6494d4d618fc6ff70b1a21f04cb8a22 Mon Sep 17 00:00:00 2001
From: Sandor Yu &lt;Sandor.yu@nxp.com&gt;
Date: Wed, 10 Jul 2019 10:50:25 +0800
Subject: [PATCH] drm/rockchip: prepare common code for cdns and rk dpi/dp
 driver

- Extracted common fields from cdn_dp_device to a new cdns_mhdp_device
  structure which will be used by two separate drivers later on.
- Moved some datatypes (audio_format, audio_info,
  vic_pxl_encoding_format, video_info) from cdn-dp-core.c to cdn-dp-reg.h.
- Changed prefixes from cdn_dp to cdns_mhdp cdn -&gt; cdns to match the other Cadence's drivers
  dp -&gt; mhdp to distinguish it from a "just a DP" as the IP underneath this registers map can be a HDMI (which is
  internally different, but the interface for commands, events is pretty much the same).
- Modified cdn-dp-core.c to use the new driver structure and new function names.

Signed-off-by: Damian Kos&lt;dkos@cadence.com&gt;
Reviewed-by: Andrzej Hajda&lt;a.hajda@samsung.com&gt;
Signed-off-by: Sandor Yu &lt;Sandor.yu@nxp.com&gt;
---
 drivers/gpu/drm/rockchip/cdn-dp-core.c | 220 +++++++++--------
 drivers/gpu/drm/rockchip/cdn-dp-core.h |  40 +---
 drivers/gpu/drm/rockchip/cdn-dp-reg.c  | 415 +++++++++++++++++----------------
 drivers/gpu/drm/rockchip/cdn-dp-reg.h  | 114 +++++++--
 4 files changed, 423 insertions(+), 366 deletions(-)

--- a/drivers/gpu/drm/rockchip/cdn-dp-core.c
+++ b/drivers/gpu/drm/rockchip/cdn-dp-core.c
@@ -22,11 +22,10 @@
 #include &lt;drm/drm_probe_helper.h&gt;
 
 #include "cdn-dp-core.h"
-#include "cdn-dp-reg.h"
 #include "rockchip_drm_vop.h"
 
 #define connector_to_dp(c) \
-		container_of(c, struct cdn_dp_device, connector)
+		container_of(c, struct cdn_dp_device, mhdp.connector)
 
 #define encoder_to_dp(c) \
 		container_of(c, struct cdn_dp_device, encoder)
@@ -61,17 +60,18 @@ MODULE_DEVICE_TABLE(of, cdn_dp_dt_ids);
 static int cdn_dp_grf_write(struct cdn_dp_device *dp,
 			    unsigned int reg, unsigned int val)
 {
+	struct device *dev = dp-&gt;mhdp.dev;
 	int ret;
 
 	ret = clk_prepare_enable(dp-&gt;grf_clk);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to prepare_enable grf clock\n");
+		DRM_DEV_ERROR(dev, "Failed to prepare_enable grf clock\n");
 		return ret;
 	}
 
 	ret = regmap_write(dp-&gt;grf, reg, val);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Could not write to GRF: %d\n", ret);
+		DRM_DEV_ERROR(dev, "Could not write to GRF: %d\n", ret);
 		clk_disable_unprepare(dp-&gt;grf_clk);
 		return ret;
 	}
@@ -83,24 +83,25 @@ static int cdn_dp_grf_write(struct cdn_d
 
 static int cdn_dp_clk_enable(struct cdn_dp_device *dp)
 {
+	struct device *dev = dp-&gt;mhdp.dev;
 	int ret;
 	unsigned long rate;
 
 	ret = clk_prepare_enable(dp-&gt;pclk);
 	if (ret &lt; 0) {
-		DRM_DEV_ERROR(dp-&gt;dev, "cannot enable dp pclk %d\n", ret);
+		DRM_DEV_ERROR(dev, "cannot enable dp pclk %d\n", ret);
 		goto err_pclk;
 	}
 
 	ret = clk_prepare_enable(dp-&gt;core_clk);
 	if (ret &lt; 0) {
-		DRM_DEV_ERROR(dp-&gt;dev, "cannot enable core_clk %d\n", ret);
+		DRM_DEV_ERROR(dev, "cannot enable core_clk %d\n", ret);
 		goto err_core_clk;
 	}
 
-	ret = pm_runtime_get_sync(dp-&gt;dev);
+	ret = pm_runtime_get_sync(dev);
 	if (ret &lt; 0) {
-		DRM_DEV_ERROR(dp-&gt;dev, "cannot get pm runtime %d\n", ret);
+		DRM_DEV_ERROR(dev, "cannot get pm runtime %d\n", ret);
 		goto err_pm_runtime_get;
 	}
 
@@ -113,18 +114,18 @@ static int cdn_dp_clk_enable(struct cdn_
 
 	rate = clk_get_rate(dp-&gt;core_clk);
 	if (!rate) {
-		DRM_DEV_ERROR(dp-&gt;dev, "get clk rate failed\n");
+		DRM_DEV_ERROR(dev, "get clk rate failed\n");
 		ret = -EINVAL;
 		goto err_set_rate;
 	}
 
-	cdn_dp_set_fw_clk(dp, rate);
-	cdn_dp_clock_reset(dp);
+	cdns_mhdp_set_fw_clk(&amp;dp-&gt;mhdp, rate);
+	cdns_mhdp_clock_reset(&amp;dp-&gt;mhdp);
 
 	return 0;
 
 err_set_rate:
-	pm_runtime_put(dp-&gt;dev);
+	pm_runtime_put(dev);
 err_pm_runtime_get:
 	clk_disable_unprepare(dp-&gt;core_clk);
 err_core_clk:
@@ -135,7 +136,7 @@ err_pclk:
 
 static void cdn_dp_clk_disable(struct cdn_dp_device *dp)
 {
-	pm_runtime_put_sync(dp-&gt;dev);
+	pm_runtime_put_sync(dp-&gt;mhdp.dev);
 	clk_disable_unprepare(dp-&gt;pclk);
 	clk_disable_unprepare(dp-&gt;core_clk);
 }
@@ -168,7 +169,7 @@ static int cdn_dp_get_sink_count(struct
 	u8 value;
 
 	*sink_count = 0;
-	ret = cdn_dp_dpcd_read(dp, DP_SINK_COUNT, &amp;value, 1);
+	ret = cdns_mhdp_dpcd_read(&amp;dp-&gt;mhdp, DP_SINK_COUNT, &amp;value, 1);
 	if (ret)
 		return ret;
 
@@ -192,12 +193,13 @@ static struct cdn_dp_port *cdn_dp_connec
 
 static bool cdn_dp_check_sink_connection(struct cdn_dp_device *dp)
 {
+	struct device *dev = dp-&gt;mhdp.dev;
 	unsigned long timeout = jiffies + msecs_to_jiffies(CDN_DPCD_TIMEOUT_MS);
 	struct cdn_dp_port *port;
 	u8 sink_count = 0;
 
 	if (dp-&gt;active_port &lt; 0 || dp-&gt;active_port &gt;= dp-&gt;ports) {
-		DRM_DEV_ERROR(dp-&gt;dev, "active_port is wrong!\n");
+		DRM_DEV_ERROR(dev, "active_port is wrong!\n");
 		return false;
 	}
 
@@ -219,7 +221,7 @@ static bool cdn_dp_check_sink_connection
 		usleep_range(5000, 10000);
 	}
 
-	DRM_DEV_ERROR(dp-&gt;dev, "Get sink capability timed out\n");
+	DRM_DEV_ERROR(dev, "Get sink capability timed out\n");
 	return false;
 }
 
@@ -261,7 +263,8 @@ static int cdn_dp_connector_get_modes(st
 	mutex_lock(&amp;dp-&gt;lock);
 	edid = dp-&gt;edid;
 	if (edid) {
-		DRM_DEV_DEBUG_KMS(dp-&gt;dev, "got edid: width[%d] x height[%d]\n",
+		DRM_DEV_DEBUG_KMS(dp-&gt;mhdp.dev,
+				  "got edid: width[%d] x height[%d]\n",
 				  edid-&gt;width_cm, edid-&gt;height_cm);
 
 		dp-&gt;sink_has_audio = drm_detect_monitor_audio(edid);
@@ -280,7 +283,8 @@ cdn_dp_connector_mode_valid(struct drm_c
 			    struct drm_display_mode *mode)
 {
 	struct cdn_dp_device *dp = connector_to_dp(connector);
-	struct drm_display_info *display_info = &amp;dp-&gt;connector.display_info;
+	struct drm_display_info *display_info =
+		&amp;dp-&gt;mhdp.connector.display_info;
 	u32 requested, actual, rate, sink_max, source_max = 0;
 	u8 lanes, bpc;
 
@@ -306,7 +310,7 @@ cdn_dp_connector_mode_valid(struct drm_c
 	sink_max = drm_dp_max_lane_count(dp-&gt;dpcd);
 	lanes = min(source_max, sink_max);
 
-	source_max = drm_dp_bw_code_to_link_rate(CDN_DP_MAX_LINK_RATE);
+	source_max = drm_dp_bw_code_to_link_rate(CDNS_DP_MAX_LINK_RATE);
 	sink_max = drm_dp_max_link_rate(dp-&gt;dpcd);
 	rate = min(source_max, sink_max);
 
@@ -316,7 +320,7 @@ cdn_dp_connector_mode_valid(struct drm_c
 	actual = actual * 8 / 10;
 
 	if (requested &gt; actual) {
-		DRM_DEV_DEBUG_KMS(dp-&gt;dev,
+		DRM_DEV_DEBUG_KMS(dp-&gt;mhdp.dev,
 				  "requested=%d, actual=%d, clock=%d\n",
 				  requested, actual, mode-&gt;clock);
 		return MODE_CLOCK_HIGH;
@@ -336,28 +340,29 @@ static int cdn_dp_firmware_init(struct c
 	const u32 *iram_data, *dram_data;
 	const struct firmware *fw = dp-&gt;fw;
 	const struct cdn_firmware_header *hdr;
+	struct device *dev = dp-&gt;mhdp.dev;
 
 	hdr = (struct cdn_firmware_header *)fw-&gt;data;
 	if (fw-&gt;size != le32_to_cpu(hdr-&gt;size_bytes)) {
-		DRM_DEV_ERROR(dp-&gt;dev, "firmware is invalid\n");
+		DRM_DEV_ERROR(dev, "firmware is invalid\n");
 		return -EINVAL;
 	}
 
 	iram_data = (const u32 *)(fw-&gt;data + hdr-&gt;header_size);
 	dram_data = (const u32 *)(fw-&gt;data + hdr-&gt;header_size + hdr-&gt;iram_size);
 
-	ret = cdn_dp_load_firmware(dp, iram_data, hdr-&gt;iram_size,
-				   dram_data, hdr-&gt;dram_size);
+	ret = cdns_mhdp_load_firmware(&amp;dp-&gt;mhdp, iram_data, hdr-&gt;iram_size,
+				      dram_data, hdr-&gt;dram_size);
 	if (ret)
 		return ret;
 
-	ret = cdn_dp_set_firmware_active(dp, true);
+	ret = cdns_mhdp_set_firmware_active(&amp;dp-&gt;mhdp, true);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "active ucpu failed: %d\n", ret);
+		DRM_DEV_ERROR(dev, "active ucpu failed: %d\n", ret);
 		return ret;
 	}
 
-	return cdn_dp_event_config(dp);
+	return cdns_mhdp_event_config(&amp;dp-&gt;mhdp);
 }
 
 static int cdn_dp_get_sink_capability(struct cdn_dp_device *dp)
@@ -367,28 +372,29 @@ static int cdn_dp_get_sink_capability(st
 	if (!cdn_dp_check_sink_connection(dp))
 		return -ENODEV;
 
-	ret = cdn_dp_dpcd_read(dp, DP_DPCD_REV, dp-&gt;dpcd,
-			       DP_RECEIVER_CAP_SIZE);
+	ret = cdns_mhdp_dpcd_read(&amp;dp-&gt;mhdp, DP_DPCD_REV, dp-&gt;dpcd,
+				  DP_RECEIVER_CAP_SIZE);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to get caps %d\n", ret);
+		DRM_DEV_ERROR(dp-&gt;mhdp.dev, "Failed to get caps %d\n", ret);
 		return ret;
 	}
 
 	kfree(dp-&gt;edid);
-	dp-&gt;edid = drm_do_get_edid(&amp;dp-&gt;connector,
-				   cdn_dp_get_edid_block, dp);
+	dp-&gt;edid = drm_do_get_edid(&amp;dp-&gt;mhdp.connector,
+				   cdns_mhdp_get_edid_block, &amp;dp-&gt;mhdp);
 	return 0;
 }
 
 static int cdn_dp_enable_phy(struct cdn_dp_device *dp, struct cdn_dp_port *port)
 {
+	struct device *dev = dp-&gt;mhdp.dev;
 	union extcon_property_value property;
 	int ret;
 
 	if (!port-&gt;phy_enabled) {
 		ret = phy_power_on(port-&gt;phy);
 		if (ret) {
-			DRM_DEV_ERROR(dp-&gt;dev, "phy power on failed: %d\n",
+			DRM_DEV_ERROR(dev, "phy power on failed: %d\n",
 				      ret);
 			goto err_phy;
 		}
@@ -398,28 +404,28 @@ static int cdn_dp_enable_phy(struct cdn_
 	ret = cdn_dp_grf_write(dp, GRF_SOC_CON26,
 			       DPTX_HPD_SEL_MASK | DPTX_HPD_SEL);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to write HPD_SEL %d\n", ret);
+		DRM_DEV_ERROR(dev, "Failed to write HPD_SEL %d\n", ret);
 		goto err_power_on;
 	}
 
-	ret = cdn_dp_get_hpd_status(dp);
+	ret = cdns_mhdp_get_hpd_status(&amp;dp-&gt;mhdp);
 	if (ret &lt;= 0) {
 		if (!ret)
-			DRM_DEV_ERROR(dp-&gt;dev, "hpd does not exist\n");
+			DRM_DEV_ERROR(dev, "hpd does not exist\n");
 		goto err_power_on;
 	}
 
 	ret = extcon_get_property(port-&gt;extcon, EXTCON_DISP_DP,
 				  EXTCON_PROP_USB_TYPEC_POLARITY, &amp;property);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "get property failed\n");
+		DRM_DEV_ERROR(dev, "get property failed\n");
 		goto err_power_on;
 	}
 
 	port-&gt;lanes = cdn_dp_get_port_lanes(port);
-	ret = cdn_dp_set_host_cap(dp, port-&gt;lanes, property.intval);
+	ret = cdns_mhdp_set_host_cap(&amp;dp-&gt;mhdp, port-&gt;lanes, property.intval);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "set host capabilities failed: %d\n",
+		DRM_DEV_ERROR(dev, "set host capabilities failed: %d\n",
 			      ret);
 		goto err_power_on;
 	}
@@ -429,7 +435,7 @@ static int cdn_dp_enable_phy(struct cdn_
 
 err_power_on:
 	if (phy_power_off(port-&gt;phy))
-		DRM_DEV_ERROR(dp-&gt;dev, "phy power off failed: %d", ret);
+		DRM_DEV_ERROR(dev, "phy power off failed: %d", ret);
 	else
 		port-&gt;phy_enabled = false;
 
@@ -447,7 +453,8 @@ static int cdn_dp_disable_phy(struct cdn
 	if (port-&gt;phy_enabled) {
 		ret = phy_power_off(port-&gt;phy);
 		if (ret) {
-			DRM_DEV_ERROR(dp-&gt;dev, "phy power off failed: %d", ret);
+			DRM_DEV_ERROR(dp-&gt;mhdp.dev,
+				      "phy power off failed: %d", ret);
 			return ret;
 		}
 	}
@@ -471,16 +478,16 @@ static int cdn_dp_disable(struct cdn_dp_
 	ret = cdn_dp_grf_write(dp, GRF_SOC_CON26,
 			       DPTX_HPD_SEL_MASK | DPTX_HPD_DEL);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to clear hpd sel %d\n",
+		DRM_DEV_ERROR(dp-&gt;mhdp.dev, "Failed to clear hpd sel %d\n",
 			      ret);
 		return ret;
 	}
 
-	cdn_dp_set_firmware_active(dp, false);
+	cdns_mhdp_set_firmware_active(&amp;dp-&gt;mhdp, false);
 	cdn_dp_clk_disable(dp);
 	dp-&gt;active = false;
-	dp-&gt;link.rate = 0;
-	dp-&gt;link.num_lanes = 0;
+	dp-&gt;mhdp.link.rate = 0;
+	dp-&gt;mhdp.link.num_lanes = 0;
 	if (!dp-&gt;connected) {
 		kfree(dp-&gt;edid);
 		dp-&gt;edid = NULL;
@@ -493,11 +500,11 @@ static int cdn_dp_enable(struct cdn_dp_d
 {
 	int ret, i, lanes;
 	struct cdn_dp_port *port;
+	struct device *dev = dp-&gt;mhdp.dev;
 
 	port = cdn_dp_connected_port(dp);
 	if (!port) {
-		DRM_DEV_ERROR(dp-&gt;dev,
-			      "Can't enable without connection\n");
+		DRM_DEV_ERROR(dev, "Can't enable without connection\n");
 		return -ENODEV;
 	}
 
@@ -510,7 +517,7 @@ static int cdn_dp_enable(struct cdn_dp_d
 
 	ret = cdn_dp_firmware_init(dp);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "firmware init failed: %d", ret);
+		DRM_DEV_ERROR(dp-&gt;mhdp.dev, "firmware init failed: %d", ret);
 		goto err_clk_disable;
 	}
 
@@ -544,8 +551,9 @@ static void cdn_dp_encoder_mode_set(stru
 				    struct drm_display_mode *adjusted)
 {
 	struct cdn_dp_device *dp = encoder_to_dp(encoder);
-	struct drm_display_info *display_info = &amp;dp-&gt;connector.display_info;
-	struct video_info *video = &amp;dp-&gt;video_info;
+	struct drm_display_info *display_info =
+		&amp;dp-&gt;mhdp.connector.display_info;
+	struct video_info *video = &amp;dp-&gt;mhdp.video_info;
 
 	switch (display_info-&gt;bpc) {
 	case 10:
@@ -563,7 +571,7 @@ static void cdn_dp_encoder_mode_set(stru
 	video-&gt;v_sync_polarity = !!(mode-&gt;flags &amp; DRM_MODE_FLAG_NVSYNC);
 	video-&gt;h_sync_polarity = !!(mode-&gt;flags &amp; DRM_MODE_FLAG_NHSYNC);
 
-	drm_mode_copy(&amp;dp-&gt;mode, adjusted);
+	drm_mode_copy(&amp;dp-&gt;mhdp.mode, adjusted);
 }
 
 static bool cdn_dp_check_link_status(struct cdn_dp_device *dp)
@@ -572,11 +580,11 @@ static bool cdn_dp_check_link_status(str
 	struct cdn_dp_port *port = cdn_dp_connected_port(dp);
 	u8 sink_lanes = drm_dp_max_lane_count(dp-&gt;dpcd);
 
-	if (!port || !dp-&gt;link.rate || !dp-&gt;link.num_lanes)
+	if (!port || !dp-&gt;mhdp.link.rate || !dp-&gt;mhdp.link.num_lanes)
 		return false;
 
-	if (cdn_dp_dpcd_read(dp, DP_LANE0_1_STATUS, link_status,
-			     DP_LINK_STATUS_SIZE)) {
+	if (cdns_mhdp_dpcd_read(&amp;dp-&gt;mhdp, DP_LANE0_1_STATUS, link_status,
+				DP_LINK_STATUS_SIZE)) {
 		DRM_ERROR("Failed to get link status\n");
 		return false;
 	}
@@ -588,15 +596,16 @@ static bool cdn_dp_check_link_status(str
 static void cdn_dp_encoder_enable(struct drm_encoder *encoder)
 {
 	struct cdn_dp_device *dp = encoder_to_dp(encoder);
+	struct device *dev = dp-&gt;mhdp.dev;
 	int ret, val;
 
-	ret = drm_of_encoder_active_endpoint_id(dp-&gt;dev-&gt;of_node, encoder);
+	ret = drm_of_encoder_active_endpoint_id(dev-&gt;of_node, encoder);
 	if (ret &lt; 0) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Could not get vop id, %d", ret);
+		DRM_DEV_ERROR(dev, "Could not get vop id, %d", ret);
 		return;
 	}
 
-	DRM_DEV_DEBUG_KMS(dp-&gt;dev, "vop %s output to cdn-dp\n",
+	DRM_DEV_DEBUG_KMS(dev, "vop %s output to cdn-dp\n",
 			  (ret) ? "LIT" : "BIG");
 	if (ret)
 		val = DP_SEL_VOP_LIT | (DP_SEL_VOP_LIT &lt;&lt; 16);
@@ -611,33 +620,33 @@ static void cdn_dp_encoder_enable(struct
 
 	ret = cdn_dp_enable(dp);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to enable encoder %d\n",
+		DRM_DEV_ERROR(dev, "Failed to enable encoder %d\n",
 			      ret);
 		goto out;
 	}
 	if (!cdn_dp_check_link_status(dp)) {
-		ret = cdn_dp_train_link(dp);
+		ret = cdns_mhdp_train_link(&amp;dp-&gt;mhdp);
 		if (ret) {
-			DRM_DEV_ERROR(dp-&gt;dev, "Failed link train %d\n", ret);
+			DRM_DEV_ERROR(dev, "Failed link train %d\n", ret);
 			goto out;
 		}
 	}
 
-	ret = cdn_dp_set_video_status(dp, CONTROL_VIDEO_IDLE);
+	ret = cdns_mhdp_set_video_status(&amp;dp-&gt;mhdp, CONTROL_VIDEO_IDLE);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to idle video %d\n", ret);
+		DRM_DEV_ERROR(dev, "Failed to idle video %d\n", ret);
 		goto out;
 	}
 
-	ret = cdn_dp_config_video(dp);
+	ret = cdns_mhdp_config_video(&amp;dp-&gt;mhdp);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to config video %d\n", ret);
+		DRM_DEV_ERROR(dev, "Failed to config video %d\n", ret);
 		goto out;
 	}
 
-	ret = cdn_dp_set_video_status(dp, CONTROL_VIDEO_VALID);
+	ret = cdns_mhdp_set_video_status(&amp;dp-&gt;mhdp, CONTROL_VIDEO_VALID);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to valid video %d\n", ret);
+		DRM_DEV_ERROR(dev, "Failed to valid video %d\n", ret);
 		goto out;
 	}
 out:
@@ -653,7 +662,8 @@ static void cdn_dp_encoder_disable(struc
 	if (dp-&gt;active) {
 		ret = cdn_dp_disable(dp);
 		if (ret) {
-			DRM_DEV_ERROR(dp-&gt;dev, "Failed to disable encoder %d\n",
+			DRM_DEV_ERROR(dp-&gt;mhdp.dev,
+				      "Failed to disable encoder %d\n",
 				      ret);
 		}
 	}
@@ -697,7 +707,7 @@ static const struct drm_encoder_funcs cd
 
 static int cdn_dp_parse_dt(struct cdn_dp_device *dp)
 {
-	struct device *dev = dp-&gt;dev;
+	struct device *dev = dp-&gt;mhdp.dev;
 	struct device_node *np = dev-&gt;of_node;
 	struct platform_device *pdev = to_platform_device(dev);
 	struct resource *res;
@@ -709,10 +719,10 @@ static int cdn_dp_parse_dt(struct cdn_dp
 	}
 
 	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	dp-&gt;regs = devm_ioremap_resource(dev, res);
-	if (IS_ERR(dp-&gt;regs)) {
+	dp-&gt;mhdp.regs = devm_ioremap_resource(dev, res);
+	if (IS_ERR(dp-&gt;mhdp.regs)) {
 		DRM_DEV_ERROR(dev, "ioremap reg failed\n");
-		return PTR_ERR(dp-&gt;regs);
+		return PTR_ERR(dp-&gt;mhdp.regs);
 	}
 
 	dp-&gt;core_clk = devm_clk_get(dev, "core-clk");
@@ -727,10 +737,10 @@ static int cdn_dp_parse_dt(struct cdn_dp
 		return PTR_ERR(dp-&gt;pclk);
 	}
 
-	dp-&gt;spdif_clk = devm_clk_get(dev, "spdif");
-	if (IS_ERR(dp-&gt;spdif_clk)) {
+	dp-&gt;mhdp.spdif_clk = devm_clk_get(dev, "spdif");
+	if (IS_ERR(dp-&gt;mhdp.spdif_clk)) {
 		DRM_DEV_ERROR(dev, "cannot get spdif_clk\n");
-		return PTR_ERR(dp-&gt;spdif_clk);
+		return PTR_ERR(dp-&gt;mhdp.spdif_clk);
 	}
 
 	dp-&gt;grf_clk = devm_clk_get(dev, "grf");
@@ -739,10 +749,10 @@ static int cdn_dp_parse_dt(struct cdn_dp
 		return PTR_ERR(dp-&gt;grf_clk);
 	}
 
-	dp-&gt;spdif_rst = devm_reset_control_get(dev, "spdif");
-	if (IS_ERR(dp-&gt;spdif_rst)) {
+	dp-&gt;mhdp.spdif_rst = devm_reset_control_get(dev, "spdif");
+	if (IS_ERR(dp-&gt;mhdp.spdif_rst)) {
 		DRM_DEV_ERROR(dev, "no spdif reset control found\n");
-		return PTR_ERR(dp-&gt;spdif_rst);
+		return PTR_ERR(dp-&gt;mhdp.spdif_rst);
 	}
 
 	dp-&gt;dptx_rst = devm_reset_control_get(dev, "dptx");
@@ -797,7 +807,7 @@ static int cdn_dp_audio_hw_params(struct
 		goto out;
 	}
 
-	ret = cdn_dp_audio_config(dp, &amp;audio);
+	ret = cdns_mhdp_audio_config(&amp;dp-&gt;mhdp, &amp;audio);
 	if (!ret)
 		dp-&gt;audio_info = audio;
 
@@ -815,7 +825,7 @@ static void cdn_dp_audio_shutdown(struct
 	if (!dp-&gt;active)
 		goto out;
 
-	ret = cdn_dp_audio_stop(dp, &amp;dp-&gt;audio_info);
+	ret = cdns_mhdp_audio_stop(&amp;dp-&gt;mhdp, &amp;dp-&gt;audio_info);
 	if (!ret)
 		dp-&gt;audio_info.format = AFMT_UNUSED;
 out:
@@ -834,7 +844,7 @@ static int cdn_dp_audio_digital_mute(str
 		goto out;
 	}
 
-	ret = cdn_dp_audio_mute(dp, enable);
+	ret = cdns_mhdp_audio_mute(&amp;dp-&gt;mhdp, enable);
 
 out:
 	mutex_unlock(&amp;dp-&gt;lock);
@@ -846,7 +856,8 @@ static int cdn_dp_audio_get_eld(struct d
 {
 	struct cdn_dp_device *dp = dev_get_drvdata(dev);
 
-	memcpy(buf, dp-&gt;connector.eld, min(sizeof(dp-&gt;connector.eld), len));
+	memcpy(buf, dp-&gt;mhdp.connector.eld,
+	       min(sizeof(dp-&gt;mhdp.connector.eld), len));
 
 	return 0;
 }
@@ -880,6 +891,7 @@ static int cdn_dp_request_firmware(struc
 	int ret;
 	unsigned long timeout = jiffies + msecs_to_jiffies(CDN_FW_TIMEOUT_MS);
 	unsigned long sleep = 1000;
+	struct device *dev = dp-&gt;mhdp.dev;
 
 	WARN_ON(!mutex_is_locked(&amp;dp-&gt;lock));
 
@@ -890,13 +902,13 @@ static int cdn_dp_request_firmware(struc
 	mutex_unlock(&amp;dp-&gt;lock);
 
 	while (time_before(jiffies, timeout)) {
-		ret = request_firmware(&amp;dp-&gt;fw, CDN_DP_FIRMWARE, dp-&gt;dev);
+		ret = request_firmware(&amp;dp-&gt;fw, CDN_DP_FIRMWARE, dev);
 		if (ret == -ENOENT) {
 			msleep(sleep);
 			sleep *= 2;
 			continue;
 		} else if (ret) {
-			DRM_DEV_ERROR(dp-&gt;dev,
+			DRM_DEV_ERROR(dev,
 				      "failed to request firmware: %d\n", ret);
 			goto out;
 		}
@@ -906,7 +918,7 @@ static int cdn_dp_request_firmware(struc
 		goto out;
 	}
 
-	DRM_DEV_ERROR(dp-&gt;dev, "Timed out trying to load firmware\n");
+	DRM_DEV_ERROR(dev, "Timed out trying to load firmware\n");
 	ret = -ETIMEDOUT;
 out:
 	mutex_lock(&amp;dp-&gt;lock);
@@ -917,8 +929,9 @@ static void cdn_dp_pd_event_work(struct
 {
 	struct cdn_dp_device *dp = container_of(work, struct cdn_dp_device,
 						event_work);
-	struct drm_connector *connector = &amp;dp-&gt;connector;
+	struct drm_connector *connector = &amp;dp-&gt;mhdp.connector;
 	enum drm_connector_status old_status;
+	struct device *dev = dp-&gt;mhdp.dev;
 
 	int ret;
 
@@ -935,44 +948,45 @@ static void cdn_dp_pd_event_work(struct
 
 	/* Not connected, notify userspace to disable the block */
 	if (!cdn_dp_connected_port(dp)) {
-		DRM_DEV_INFO(dp-&gt;dev, "Not connected. Disabling cdn\n");
+		DRM_DEV_INFO(dev, "Not connected. Disabling cdn\n");
 		dp-&gt;connected = false;
 
 	/* Connected but not enabled, enable the block */
 	} else if (!dp-&gt;active) {
-		DRM_DEV_INFO(dp-&gt;dev, "Connected, not enabled. Enabling cdn\n");
+		DRM_DEV_INFO(dev, "Connected, not enabled. Enabling cdn\n");
 		ret = cdn_dp_enable(dp);
 		if (ret) {
-			DRM_DEV_ERROR(dp-&gt;dev, "Enable dp failed %d\n", ret);
+			DRM_DEV_ERROR(dev, "Enable dp failed %d\n", ret);
 			dp-&gt;connected = false;
 		}
 
 	/* Enabled and connected to a dongle without a sink, notify userspace */
 	} else if (!cdn_dp_check_sink_connection(dp)) {
-		DRM_DEV_INFO(dp-&gt;dev, "Connected without sink. Assert hpd\n");
+		DRM_DEV_INFO(dev, "Connected without sink. Assert hpd\n");
 		dp-&gt;connected = false;
 
 	/* Enabled and connected with a sink, re-train if requested */
 	} else if (!cdn_dp_check_link_status(dp)) {
-		unsigned int rate = dp-&gt;link.rate;
-		unsigned int lanes = dp-&gt;link.num_lanes;
-		struct drm_display_mode *mode = &amp;dp-&gt;mode;
+		unsigned int rate = dp-&gt;mhdp.link.rate;
+		unsigned int lanes = dp-&gt;mhdp.link.num_lanes;
+		struct drm_display_mode *mode = &amp;dp-&gt;mhdp.mode;
 
-		DRM_DEV_INFO(dp-&gt;dev, "Connected with sink. Re-train link\n");
-		ret = cdn_dp_train_link(dp);
+		DRM_DEV_INFO(dev, "Connected with sink. Re-train link\n");
+		ret = cdns_mhdp_train_link(&amp;dp-&gt;mhdp);
 		if (ret) {
 			dp-&gt;connected = false;
-			DRM_DEV_ERROR(dp-&gt;dev, "Train link failed %d\n", ret);
+			DRM_DEV_ERROR(dev, "Train link failed %d\n", ret);
 			goto out;
 		}
 
 		/* If training result is changed, update the video config */
 		if (mode-&gt;clock &amp;&amp;
-		    (rate != dp-&gt;link.rate || lanes != dp-&gt;link.num_lanes)) {
-			ret = cdn_dp_config_video(dp);
+		    (rate != dp-&gt;mhdp.link.rate ||
+		     lanes != dp-&gt;mhdp.link.num_lanes)) {
+			ret = cdns_mhdp_config_video(&amp;dp-&gt;mhdp);
 			if (ret) {
 				dp-&gt;connected = false;
-				DRM_DEV_ERROR(dp-&gt;dev,
+				DRM_DEV_ERROR(dev,
 					      "Failed to config video %d\n",
 					      ret);
 			}
@@ -1041,7 +1055,7 @@ static int cdn_dp_bind(struct device *de
 
 	drm_encoder_helper_add(encoder, &amp;cdn_dp_encoder_helper_funcs);
 
-	connector = &amp;dp-&gt;connector;
+	connector = &amp;dp-&gt;mhdp.connector;
 	connector-&gt;polled = DRM_CONNECTOR_POLL_HPD;
 	connector-&gt;dpms = DRM_MODE_DPMS_OFF;
 
@@ -1065,7 +1079,7 @@ static int cdn_dp_bind(struct device *de
 		port = dp-&gt;port[i];
 
 		port-&gt;event_nb.notifier_call = cdn_dp_pd_event;
-		ret = devm_extcon_register_notifier(dp-&gt;dev, port-&gt;extcon,
+		ret = devm_extcon_register_notifier(dp-&gt;mhdp.dev, port-&gt;extcon,
 						    EXTCON_DISP_DP,
 						    &amp;port-&gt;event_nb);
 		if (ret) {
@@ -1092,7 +1106,7 @@ static void cdn_dp_unbind(struct device
 {
 	struct cdn_dp_device *dp = dev_get_drvdata(dev);
 	struct drm_encoder *encoder = &amp;dp-&gt;encoder;
-	struct drm_connector *connector = &amp;dp-&gt;connector;
+	struct drm_connector *connector = &amp;dp-&gt;mhdp.connector;
 
 	cancel_work_sync(&amp;dp-&gt;event_work);
 	cdn_dp_encoder_disable(encoder);
@@ -1152,7 +1166,7 @@ static int cdn_dp_probe(struct platform_
 	dp = devm_kzalloc(dev, sizeof(*dp), GFP_KERNEL);
 	if (!dp)
 		return -ENOMEM;
-	dp-&gt;dev = dev;
+	dp-&gt;mhdp.dev = dev;
 
 	match = of_match_node(cdn_dp_dt_ids, pdev-&gt;dev.of_node);
 	dp_data = (struct cdn_dp_data *)match-&gt;data;
@@ -1197,7 +1211,7 @@ static int cdn_dp_remove(struct platform
 	struct cdn_dp_device *dp = platform_get_drvdata(pdev);
 
 	platform_device_unregister(dp-&gt;audio_pdev);
-	cdn_dp_suspend(dp-&gt;dev);
+	cdn_dp_suspend(dp-&gt;mhdp.dev);
 	component_del(&amp;pdev-&gt;dev, &amp;cdn_dp_component_ops);
 
 	return 0;
@@ -1207,7 +1221,7 @@ static void cdn_dp_shutdown(struct platf
 {
 	struct cdn_dp_device *dp = platform_get_drvdata(pdev);
 
-	cdn_dp_suspend(dp-&gt;dev);
+	cdn_dp_suspend(dp-&gt;mhdp.dev);
 }
 
 static const struct dev_pm_ops cdn_dp_pm_ops = {
--- a/drivers/gpu/drm/rockchip/cdn-dp-core.h
+++ b/drivers/gpu/drm/rockchip/cdn-dp-core.h
@@ -12,38 +12,10 @@
 #include &lt;drm/drm_probe_helper.h&gt;
 
 #include "rockchip_drm_drv.h"
+#include "cdn-dp-reg.h"
 
 #define MAX_PHY		2
 
-enum audio_format {
-	AFMT_I2S = 0,
-	AFMT_SPDIF = 1,
-	AFMT_UNUSED,
-};
-
-struct audio_info {
-	enum audio_format format;
-	int sample_rate;
-	int channels;
-	int sample_width;
-};
-
-enum vic_pxl_encoding_format {
-	PXL_RGB = 0x1,
-	YCBCR_4_4_4 = 0x2,
-	YCBCR_4_2_2 = 0x4,
-	YCBCR_4_2_0 = 0x8,
-	Y_ONLY = 0x10,
-};
-
-struct video_info {
-	bool h_sync_polarity;
-	bool v_sync_polarity;
-	bool interlaced;
-	int color_depth;
-	enum vic_pxl_encoding_format color_fmt;
-};
-
 struct cdn_firmware_header {
 	u32 size_bytes; /* size of the entire header+image(s) in bytes */
 	u32 header_size; /* size of just the header in bytes */
@@ -62,11 +34,9 @@ struct cdn_dp_port {
 };
 
 struct cdn_dp_device {
-	struct device *dev;
+	struct cdns_mhdp_device mhdp;
 	struct drm_device *drm_dev;
-	struct drm_connector connector;
 	struct drm_encoder encoder;
-	struct drm_display_mode mode;
 	struct platform_device *audio_pdev;
 	struct work_struct event_work;
 	struct edid *edid;
@@ -77,22 +47,16 @@ struct cdn_dp_device {
 	bool suspended;
 
 	const struct firmware *fw;	/* cdn dp firmware */
-	unsigned int fw_version;	/* cdn fw version */
 	bool fw_loaded;
 
-	void __iomem *regs;
 	struct regmap *grf;
 	struct clk *core_clk;
 	struct clk *pclk;
-	struct clk *spdif_clk;
 	struct clk *grf_clk;
-	struct reset_control *spdif_rst;
 	struct reset_control *dptx_rst;
 	struct reset_control *apb_rst;
 	struct reset_control *core_rst;
 	struct audio_info audio_info;
-	struct video_info video_info;
-	struct drm_dp_link link;
 	struct cdn_dp_port *port[MAX_PHY];
 	u8 ports;
 	u8 lanes;
--- a/drivers/gpu/drm/rockchip/cdn-dp-reg.c
+++ b/drivers/gpu/drm/rockchip/cdn-dp-reg.c
@@ -14,19 +14,19 @@
 #include "cdn-dp-core.h"
 #include "cdn-dp-reg.h"
 
-#define CDN_DP_SPDIF_CLK		200000000
+#define CDNS_DP_SPDIF_CLK		200000000
 #define FW_ALIVE_TIMEOUT_US		1000000
 #define MAILBOX_RETRY_US		1000
 #define MAILBOX_TIMEOUT_US		5000000
 #define LINK_TRAINING_RETRY_MS		20
 #define LINK_TRAINING_TIMEOUT_MS	500
 
-void cdn_dp_set_fw_clk(struct cdn_dp_device *dp, unsigned long clk)
+void cdns_mhdp_set_fw_clk(struct cdns_mhdp_device *mhdp, unsigned long clk)
 {
-	writel(clk / 1000000, dp-&gt;regs + SW_CLK_H);
+	writel(clk / 1000000, mhdp-&gt;regs + SW_CLK_H);
 }
 
-void cdn_dp_clock_reset(struct cdn_dp_device *dp)
+void cdns_mhdp_clock_reset(struct cdns_mhdp_device *mhdp)
 {
 	u32 val;
 
@@ -42,16 +42,16 @@ void cdn_dp_clock_reset(struct cdn_dp_de
 	      DPTX_SYS_CLK_EN |
 	      CFG_DPTX_VIF_CLK_RSTN_EN |
 	      CFG_DPTX_VIF_CLK_EN;
-	writel(val, dp-&gt;regs + SOURCE_DPTX_CAR);
+	writel(val, mhdp-&gt;regs + SOURCE_DPTX_CAR);
 
 	val = SOURCE_PHY_RSTN_EN | SOURCE_PHY_CLK_EN;
-	writel(val, dp-&gt;regs + SOURCE_PHY_CAR);
+	writel(val, mhdp-&gt;regs + SOURCE_PHY_CAR);
 
 	val = SOURCE_PKT_SYS_RSTN_EN |
 	      SOURCE_PKT_SYS_CLK_EN |
 	      SOURCE_PKT_DATA_RSTN_EN |
 	      SOURCE_PKT_DATA_CLK_EN;
-	writel(val, dp-&gt;regs + SOURCE_PKT_CAR);
+	writel(val, mhdp-&gt;regs + SOURCE_PKT_CAR);
 
 	val = SPDIF_CDR_CLK_RSTN_EN |
 	      SPDIF_CDR_CLK_EN |
@@ -59,53 +59,53 @@ void cdn_dp_clock_reset(struct cdn_dp_de
 	      SOURCE_AIF_SYS_CLK_EN |
 	      SOURCE_AIF_CLK_RSTN_EN |
 	      SOURCE_AIF_CLK_EN;
-	writel(val, dp-&gt;regs + SOURCE_AIF_CAR);
+	writel(val, mhdp-&gt;regs + SOURCE_AIF_CAR);
 
 	val = SOURCE_CIPHER_SYSTEM_CLK_RSTN_EN |
 	      SOURCE_CIPHER_SYS_CLK_EN |
 	      SOURCE_CIPHER_CHAR_CLK_RSTN_EN |
 	      SOURCE_CIPHER_CHAR_CLK_EN;
-	writel(val, dp-&gt;regs + SOURCE_CIPHER_CAR);
+	writel(val, mhdp-&gt;regs + SOURCE_CIPHER_CAR);
 
 	val = SOURCE_CRYPTO_SYS_CLK_RSTN_EN |
 	      SOURCE_CRYPTO_SYS_CLK_EN;
-	writel(val, dp-&gt;regs + SOURCE_CRYPTO_CAR);
+	writel(val, mhdp-&gt;regs + SOURCE_CRYPTO_CAR);
 
 	/* enable Mailbox and PIF interrupt */
-	writel(0, dp-&gt;regs + APB_INT_MASK);
+	writel(0, mhdp-&gt;regs + APB_INT_MASK);
 }
 
-static int cdn_dp_mailbox_read(struct cdn_dp_device *dp)
+static int cdns_mhdp_mailbox_read(struct cdns_mhdp_device *mhdp)
 {
 	int val, ret;
 
-	ret = readx_poll_timeout(readl, dp-&gt;regs + MAILBOX_EMPTY_ADDR,
+	ret = readx_poll_timeout(readl, mhdp-&gt;regs + MAILBOX_EMPTY_ADDR,
 				 val, !val, MAILBOX_RETRY_US,
 				 MAILBOX_TIMEOUT_US);
 	if (ret &lt; 0)
 		return ret;
 
-	return readl(dp-&gt;regs + MAILBOX0_RD_DATA) &amp; 0xff;
+	return readl(mhdp-&gt;regs + MAILBOX0_RD_DATA) &amp; 0xff;
 }
 
-static int cdp_dp_mailbox_write(struct cdn_dp_device *dp, u8 val)
+static int cdp_dp_mailbox_write(struct cdns_mhdp_device *mhdp, u8 val)
 {
 	int ret, full;
 
-	ret = readx_poll_timeout(readl, dp-&gt;regs + MAILBOX_FULL_ADDR,
+	ret = readx_poll_timeout(readl, mhdp-&gt;regs + MAILBOX_FULL_ADDR,
 				 full, !full, MAILBOX_RETRY_US,
 				 MAILBOX_TIMEOUT_US);
 	if (ret &lt; 0)
 		return ret;
 
-	writel(val, dp-&gt;regs + MAILBOX0_WR_DATA);
+	writel(val, mhdp-&gt;regs + MAILBOX0_WR_DATA);
 
 	return 0;
 }
 
-static int cdn_dp_mailbox_validate_receive(struct cdn_dp_device *dp,
-					   u8 module_id, u8 opcode,
-					   u16 req_size)
+static int cdns_mhdp_mailbox_validate_receive(struct cdns_mhdp_device *mhdp,
+					      u8 module_id, u8 opcode,
+					      u16 req_size)
 {
 	u32 mbox_size, i;
 	u8 header[4];
@@ -113,7 +113,7 @@ static int cdn_dp_mailbox_validate_recei
 
 	/* read the header of the message */
 	for (i = 0; i &lt; 4; i++) {
-		ret = cdn_dp_mailbox_read(dp);
+		ret = cdns_mhdp_mailbox_read(mhdp);
 		if (ret &lt; 0)
 			return ret;
 
@@ -129,7 +129,7 @@ static int cdn_dp_mailbox_validate_recei
 		 * clear the mailbox by reading its contents.
 		 */
 		for (i = 0; i &lt; mbox_size; i++)
-			if (cdn_dp_mailbox_read(dp) &lt; 0)
+			if (cdns_mhdp_mailbox_read(mhdp) &lt; 0)
 				break;
 
 		return -EINVAL;
@@ -138,14 +138,14 @@ static int cdn_dp_mailbox_validate_recei
 	return 0;
 }
 
-static int cdn_dp_mailbox_read_receive(struct cdn_dp_device *dp,
-				       u8 *buff, u16 buff_size)
+static int cdns_mhdp_mailbox_read_receive(struct cdns_mhdp_device *mhdp,
+					  u8 *buff, u16 buff_size)
 {
 	u32 i;
 	int ret;
 
 	for (i = 0; i &lt; buff_size; i++) {
-		ret = cdn_dp_mailbox_read(dp);
+		ret = cdns_mhdp_mailbox_read(mhdp);
 		if (ret &lt; 0)
 			return ret;
 
@@ -155,8 +155,8 @@ static int cdn_dp_mailbox_read_receive(s
 	return 0;
 }
 
-static int cdn_dp_mailbox_send(struct cdn_dp_device *dp, u8 module_id,
-			       u8 opcode, u16 size, u8 *message)
+static int cdns_mhdp_mailbox_send(struct cdns_mhdp_device *mhdp, u8 module_id,
+				  u8 opcode, u16 size, u8 *message)
 {
 	u8 header[4];
 	int ret, i;
@@ -167,13 +167,13 @@ static int cdn_dp_mailbox_send(struct cd
 	header[3] = size &amp; 0xff;
 
 	for (i = 0; i &lt; 4; i++) {
-		ret = cdp_dp_mailbox_write(dp, header[i]);
+		ret = cdp_dp_mailbox_write(mhdp, header[i]);
 		if (ret)
 			return ret;
 	}
 
 	for (i = 0; i &lt; size; i++) {
-		ret = cdp_dp_mailbox_write(dp, message[i]);
+		ret = cdp_dp_mailbox_write(mhdp, message[i]);
 		if (ret)
 			return ret;
 	}
@@ -181,7 +181,7 @@ static int cdn_dp_mailbox_send(struct cd
 	return 0;
 }
 
-static int cdn_dp_reg_write(struct cdn_dp_device *dp, u16 addr, u32 val)
+static int cdns_mhdp_reg_write(struct cdns_mhdp_device *mhdp, u16 addr, u32 val)
 {
 	u8 msg[6];
 
@@ -191,12 +191,12 @@ static int cdn_dp_reg_write(struct cdn_d
 	msg[3] = (val &gt;&gt; 16) &amp; 0xff;
 	msg[4] = (val &gt;&gt; 8) &amp; 0xff;
 	msg[5] = val &amp; 0xff;
-	return cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_WRITE_REGISTER,
-				   sizeof(msg), msg);
+	return cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				      DPTX_WRITE_REGISTER, sizeof(msg), msg);
 }
 
-static int cdn_dp_reg_write_bit(struct cdn_dp_device *dp, u16 addr,
-				u8 start_bit, u8 bits_no, u32 val)
+static int cdns_mhdp_reg_write_bit(struct cdns_mhdp_device *mhdp, u16 addr,
+				   u8 start_bit, u8 bits_no, u32 val)
 {
 	u8 field[8];
 
@@ -209,11 +209,12 @@ static int cdn_dp_reg_write_bit(struct c
 	field[6] = (val &gt;&gt; 8) &amp; 0xff;
 	field[7] = val &amp; 0xff;
 
-	return cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_WRITE_FIELD,
-				   sizeof(field), field);
+	return cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				      DPTX_WRITE_FIELD, sizeof(field), field);
 }
 
-int cdn_dp_dpcd_read(struct cdn_dp_device *dp, u32 addr, u8 *data, u16 len)
+int cdns_mhdp_dpcd_read(struct cdns_mhdp_device *mhdp,
+			u32 addr, u8 *data, u16 len)
 {
 	u8 msg[5], reg[5];
 	int ret;
@@ -223,28 +224,28 @@ int cdn_dp_dpcd_read(struct cdn_dp_devic
 	msg[2] = (addr &gt;&gt; 16) &amp; 0xff;
 	msg[3] = (addr &gt;&gt; 8) &amp; 0xff;
 	msg[4] = addr &amp; 0xff;
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_READ_DPCD,
-				  sizeof(msg), msg);
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				     DPTX_READ_DPCD, sizeof(msg), msg);
 	if (ret)
 		goto err_dpcd_read;
 
-	ret = cdn_dp_mailbox_validate_receive(dp, MB_MODULE_ID_DP_TX,
-					      DPTX_READ_DPCD,
-					      sizeof(reg) + len);
+	ret = cdns_mhdp_mailbox_validate_receive(mhdp, MB_MODULE_ID_DP_TX,
+						 DPTX_READ_DPCD,
+						 sizeof(reg) + len);
 	if (ret)
 		goto err_dpcd_read;
 
-	ret = cdn_dp_mailbox_read_receive(dp, reg, sizeof(reg));
+	ret = cdns_mhdp_mailbox_read_receive(mhdp, reg, sizeof(reg));
 	if (ret)
 		goto err_dpcd_read;
 
-	ret = cdn_dp_mailbox_read_receive(dp, data, len);
+	ret = cdns_mhdp_mailbox_read_receive(mhdp, data, len);
 
 err_dpcd_read:
 	return ret;
 }
 
-int cdn_dp_dpcd_write(struct cdn_dp_device *dp, u32 addr, u8 value)
+int cdns_mhdp_dpcd_write(struct cdns_mhdp_device *mhdp, u32 addr, u8 value)
 {
 	u8 msg[6], reg[5];
 	int ret;
@@ -255,17 +256,17 @@ int cdn_dp_dpcd_write(struct cdn_dp_devi
 	msg[3] = (addr &gt;&gt; 8) &amp; 0xff;
 	msg[4] = addr &amp; 0xff;
 	msg[5] = value;
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_WRITE_DPCD,
-				  sizeof(msg), msg);
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				     DPTX_WRITE_DPCD, sizeof(msg), msg);
 	if (ret)
 		goto err_dpcd_write;
 
-	ret = cdn_dp_mailbox_validate_receive(dp, MB_MODULE_ID_DP_TX,
-					      DPTX_WRITE_DPCD, sizeof(reg));
+	ret = cdns_mhdp_mailbox_validate_receive(mhdp, MB_MODULE_ID_DP_TX,
+						 DPTX_WRITE_DPCD, sizeof(reg));
 	if (ret)
 		goto err_dpcd_write;
 
-	ret = cdn_dp_mailbox_read_receive(dp, reg, sizeof(reg));
+	ret = cdns_mhdp_mailbox_read_receive(mhdp, reg, sizeof(reg));
 	if (ret)
 		goto err_dpcd_write;
 
@@ -274,53 +275,53 @@ int cdn_dp_dpcd_write(struct cdn_dp_devi
 
 err_dpcd_write:
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "dpcd write failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "dpcd write failed: %d\n", ret);
 	return ret;
 }
 
-int cdn_dp_load_firmware(struct cdn_dp_device *dp, const u32 *i_mem,
-			 u32 i_size, const u32 *d_mem, u32 d_size)
+int cdns_mhdp_load_firmware(struct cdns_mhdp_device *mhdp, const u32 *i_mem,
+			    u32 i_size, const u32 *d_mem, u32 d_size)
 {
 	u32 reg;
 	int i, ret;
 
 	/* reset ucpu before load firmware*/
 	writel(APB_IRAM_PATH | APB_DRAM_PATH | APB_XT_RESET,
-	       dp-&gt;regs + APB_CTRL);
+	       mhdp-&gt;regs + APB_CTRL);
 
 	for (i = 0; i &lt; i_size; i += 4)
-		writel(*i_mem++, dp-&gt;regs + ADDR_IMEM + i);
+		writel(*i_mem++, mhdp-&gt;regs + ADDR_IMEM + i);
 
 	for (i = 0; i &lt; d_size; i += 4)
-		writel(*d_mem++, dp-&gt;regs + ADDR_DMEM + i);
+		writel(*d_mem++, mhdp-&gt;regs + ADDR_DMEM + i);
 
 	/* un-reset ucpu */
-	writel(0, dp-&gt;regs + APB_CTRL);
+	writel(0, mhdp-&gt;regs + APB_CTRL);
 
 	/* check the keep alive register to make sure fw working */
-	ret = readx_poll_timeout(readl, dp-&gt;regs + KEEP_ALIVE,
+	ret = readx_poll_timeout(readl, mhdp-&gt;regs + KEEP_ALIVE,
 				 reg, reg, 2000, FW_ALIVE_TIMEOUT_US);
 	if (ret &lt; 0) {
-		DRM_DEV_ERROR(dp-&gt;dev, "failed to loaded the FW reg = %x\n",
+		DRM_DEV_ERROR(mhdp-&gt;dev, "failed to loaded the FW reg = %x\n",
 			      reg);
 		return -EINVAL;
 	}
 
-	reg = readl(dp-&gt;regs + VER_L) &amp; 0xff;
-	dp-&gt;fw_version = reg;
-	reg = readl(dp-&gt;regs + VER_H) &amp; 0xff;
-	dp-&gt;fw_version |= reg &lt;&lt; 8;
-	reg = readl(dp-&gt;regs + VER_LIB_L_ADDR) &amp; 0xff;
-	dp-&gt;fw_version |= reg &lt;&lt; 16;
-	reg = readl(dp-&gt;regs + VER_LIB_H_ADDR) &amp; 0xff;
-	dp-&gt;fw_version |= reg &lt;&lt; 24;
+	reg = readl(mhdp-&gt;regs + VER_L) &amp; 0xff;
+	mhdp-&gt;fw_version = reg;
+	reg = readl(mhdp-&gt;regs + VER_H) &amp; 0xff;
+	mhdp-&gt;fw_version |= reg &lt;&lt; 8;
+	reg = readl(mhdp-&gt;regs + VER_LIB_L_ADDR) &amp; 0xff;
+	mhdp-&gt;fw_version |= reg &lt;&lt; 16;
+	reg = readl(mhdp-&gt;regs + VER_LIB_H_ADDR) &amp; 0xff;
+	mhdp-&gt;fw_version |= reg &lt;&lt; 24;
 
-	DRM_DEV_DEBUG(dp-&gt;dev, "firmware version: %x\n", dp-&gt;fw_version);
+	DRM_DEV_DEBUG(mhdp-&gt;dev, "firmware version: %x\n", mhdp-&gt;fw_version);
 
 	return 0;
 }
 
-int cdn_dp_set_firmware_active(struct cdn_dp_device *dp, bool enable)
+int cdns_mhdp_set_firmware_active(struct cdns_mhdp_device *mhdp, bool enable)
 {
 	u8 msg[5];
 	int ret, i;
@@ -332,14 +333,14 @@ int cdn_dp_set_firmware_active(struct cd
 	msg[4] = enable ? FW_ACTIVE : FW_STANDBY;
 
 	for (i = 0; i &lt; sizeof(msg); i++) {
-		ret = cdp_dp_mailbox_write(dp, msg[i]);
+		ret = cdp_dp_mailbox_write(mhdp, msg[i]);
 		if (ret)
 			goto err_set_firmware_active;
 	}
 
 	/* read the firmware state */
 	for (i = 0; i &lt; sizeof(msg); i++)  {
-		ret = cdn_dp_mailbox_read(dp);
+		ret = cdns_mhdp_mailbox_read(mhdp);
 		if (ret &lt; 0)
 			goto err_set_firmware_active;
 
@@ -350,16 +351,16 @@ int cdn_dp_set_firmware_active(struct cd
 
 err_set_firmware_active:
 	if (ret &lt; 0)
-		DRM_DEV_ERROR(dp-&gt;dev, "set firmware active failed\n");
+		DRM_DEV_ERROR(mhdp-&gt;dev, "set firmware active failed\n");
 	return ret;
 }
 
-int cdn_dp_set_host_cap(struct cdn_dp_device *dp, u8 lanes, bool flip)
+int cdns_mhdp_set_host_cap(struct cdns_mhdp_device *mhdp, u8 lanes, bool flip)
 {
 	u8 msg[8];
 	int ret;
 
-	msg[0] = CDN_DP_MAX_LINK_RATE;
+	msg[0] = CDNS_DP_MAX_LINK_RATE;
 	msg[1] = lanes | SCRAMBLER_EN;
 	msg[2] = VOLTAGE_LEVEL_2;
 	msg[3] = PRE_EMPHASIS_LEVEL_3;
@@ -368,22 +369,22 @@ int cdn_dp_set_host_cap(struct cdn_dp_de
 	msg[6] = flip ? LANE_MAPPING_FLIPPED : LANE_MAPPING_NORMAL;
 	msg[7] = ENHANCED;
 
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX,
-				  DPTX_SET_HOST_CAPABILITIES,
-				  sizeof(msg), msg);
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				     DPTX_SET_HOST_CAPABILITIES,
+				     sizeof(msg), msg);
 	if (ret)
 		goto err_set_host_cap;
 
-	ret = cdn_dp_reg_write(dp, DP_AUX_SWAP_INVERSION_CONTROL,
-			       AUX_HOST_INVERT);
+	ret = cdns_mhdp_reg_write(mhdp, DP_AUX_SWAP_INVERSION_CONTROL,
+				  AUX_HOST_INVERT);
 
 err_set_host_cap:
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "set host cap failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "set host cap failed: %d\n", ret);
 	return ret;
 }
 
-int cdn_dp_event_config(struct cdn_dp_device *dp)
+int cdns_mhdp_event_config(struct cdns_mhdp_device *mhdp)
 {
 	u8 msg[5];
 	int ret;
@@ -392,49 +393,50 @@ int cdn_dp_event_config(struct cdn_dp_de
 
 	msg[0] = DPTX_EVENT_ENABLE_HPD | DPTX_EVENT_ENABLE_TRAINING;
 
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_ENABLE_EVENT,
-				  sizeof(msg), msg);
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				     DPTX_ENABLE_EVENT, sizeof(msg), msg);
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "set event config failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "set event config failed: %d\n", ret);
 
 	return ret;
 }
 
-u32 cdn_dp_get_event(struct cdn_dp_device *dp)
+u32 cdns_mhdp_get_event(struct cdns_mhdp_device *mhdp)
 {
-	return readl(dp-&gt;regs + SW_EVENTS0);
+	return readl(mhdp-&gt;regs + SW_EVENTS0);
 }
 
-int cdn_dp_get_hpd_status(struct cdn_dp_device *dp)
+int cdns_mhdp_get_hpd_status(struct cdns_mhdp_device *mhdp)
 {
 	u8 status;
 	int ret;
 
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_HPD_STATE,
-				  0, NULL);
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				     DPTX_HPD_STATE, 0, NULL);
 	if (ret)
 		goto err_get_hpd;
 
-	ret = cdn_dp_mailbox_validate_receive(dp, MB_MODULE_ID_DP_TX,
-					      DPTX_HPD_STATE, sizeof(status));
+	ret = cdns_mhdp_mailbox_validate_receive(mhdp, MB_MODULE_ID_DP_TX,
+						 DPTX_HPD_STATE,
+						 sizeof(status));
 	if (ret)
 		goto err_get_hpd;
 
-	ret = cdn_dp_mailbox_read_receive(dp, &amp;status, sizeof(status));
+	ret = cdns_mhdp_mailbox_read_receive(mhdp, &amp;status, sizeof(status));
 	if (ret)
 		goto err_get_hpd;
 
 	return status;
 
 err_get_hpd:
-	DRM_DEV_ERROR(dp-&gt;dev, "get hpd status failed: %d\n", ret);
+	DRM_DEV_ERROR(mhdp-&gt;dev, "get hpd status failed: %d\n", ret);
 	return ret;
 }
 
-int cdn_dp_get_edid_block(void *data, u8 *edid,
+int cdns_mhdp_get_edid_block(void *data, u8 *edid,
 			  unsigned int block, size_t length)
 {
-	struct cdn_dp_device *dp = data;
+	struct cdns_mhdp_device *mhdp = data;
 	u8 msg[2], reg[2], i;
 	int ret;
 
@@ -442,22 +444,23 @@ int cdn_dp_get_edid_block(void *data, u8
 		msg[0] = block / 2;
 		msg[1] = block % 2;
 
-		ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_GET_EDID,
-					  sizeof(msg), msg);
+		ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+					     DPTX_GET_EDID, sizeof(msg), msg);
 		if (ret)
 			continue;
 
-		ret = cdn_dp_mailbox_validate_receive(dp, MB_MODULE_ID_DP_TX,
-						      DPTX_GET_EDID,
-						      sizeof(reg) + length);
+		ret = cdns_mhdp_mailbox_validate_receive(mhdp,
+							 MB_MODULE_ID_DP_TX,
+							 DPTX_GET_EDID,
+							 sizeof(reg) + length);
 		if (ret)
 			continue;
 
-		ret = cdn_dp_mailbox_read_receive(dp, reg, sizeof(reg));
+		ret = cdns_mhdp_mailbox_read_receive(mhdp, reg, sizeof(reg));
 		if (ret)
 			continue;
 
-		ret = cdn_dp_mailbox_read_receive(dp, edid, length);
+		ret = cdns_mhdp_mailbox_read_receive(mhdp, edid, length);
 		if (ret)
 			continue;
 
@@ -466,13 +469,13 @@ int cdn_dp_get_edid_block(void *data, u8
 	}
 
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "get block[%d] edid failed: %d\n", block,
-			      ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "get block[%d] edid failed: %d\n",
+			      block, ret);
 
 	return ret;
 }
 
-static int cdn_dp_training_start(struct cdn_dp_device *dp)
+static int cdns_mhdp_training_start(struct cdns_mhdp_device *mhdp)
 {
 	unsigned long timeout;
 	u8 msg, event[2];
@@ -481,26 +484,28 @@ static int cdn_dp_training_start(struct
 	msg = LINK_TRAINING_RUN;
 
 	/* start training */
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_TRAINING_CONTROL,
-				  sizeof(msg), &amp;msg);
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+				     DPTX_TRAINING_CONTROL, sizeof(msg), &amp;msg);
 	if (ret)
 		goto err_training_start;
 
 	timeout = jiffies + msecs_to_jiffies(LINK_TRAINING_TIMEOUT_MS);
 	while (time_before(jiffies, timeout)) {
 		msleep(LINK_TRAINING_RETRY_MS);
-		ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX,
-					  DPTX_READ_EVENT, 0, NULL);
+		ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX,
+					     DPTX_READ_EVENT, 0, NULL);
 		if (ret)
 			goto err_training_start;
 
-		ret = cdn_dp_mailbox_validate_receive(dp, MB_MODULE_ID_DP_TX,
-						      DPTX_READ_EVENT,
-						      sizeof(event));
+		ret = cdns_mhdp_mailbox_validate_receive(mhdp,
+							 MB_MODULE_ID_DP_TX,
+							 DPTX_READ_EVENT,
+							 sizeof(event));
 		if (ret)
 			goto err_training_start;
 
-		ret = cdn_dp_mailbox_read_receive(dp, event, sizeof(event));
+		ret = cdns_mhdp_mailbox_read_receive(mhdp, event,
+						     sizeof(event));
 		if (ret)
 			goto err_training_start;
 
@@ -511,76 +516,76 @@ static int cdn_dp_training_start(struct
 	ret = -ETIMEDOUT;
 
 err_training_start:
-	DRM_DEV_ERROR(dp-&gt;dev, "training failed: %d\n", ret);
+	DRM_DEV_ERROR(mhdp-&gt;dev, "training failed: %d\n", ret);
 	return ret;
 }
 
-static int cdn_dp_get_training_status(struct cdn_dp_device *dp)
+static int cdns_mhdp_get_training_status(struct cdns_mhdp_device *mhdp)
 {
 	u8 status[10];
 	int ret;
 
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_READ_LINK_STAT,
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX, DPTX_READ_LINK_STAT,
 				  0, NULL);
 	if (ret)
 		goto err_get_training_status;
 
-	ret = cdn_dp_mailbox_validate_receive(dp, MB_MODULE_ID_DP_TX,
+	ret = cdns_mhdp_mailbox_validate_receive(mhdp, MB_MODULE_ID_DP_TX,
 					      DPTX_READ_LINK_STAT,
 					      sizeof(status));
 	if (ret)
 		goto err_get_training_status;
 
-	ret = cdn_dp_mailbox_read_receive(dp, status, sizeof(status));
+	ret = cdns_mhdp_mailbox_read_receive(mhdp, status, sizeof(status));
 	if (ret)
 		goto err_get_training_status;
 
-	dp-&gt;link.rate = drm_dp_bw_code_to_link_rate(status[0]);
-	dp-&gt;link.num_lanes = status[1];
+	mhdp-&gt;link.rate = drm_dp_bw_code_to_link_rate(status[0]);
+	mhdp-&gt;link.num_lanes = status[1];
 
 err_get_training_status:
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "get training status failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "get training status failed: %d\n", ret);
 	return ret;
 }
 
-int cdn_dp_train_link(struct cdn_dp_device *dp)
+int cdns_mhdp_train_link(struct cdns_mhdp_device *mhdp)
 {
 	int ret;
 
-	ret = cdn_dp_training_start(dp);
+	ret = cdns_mhdp_training_start(mhdp);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to start training %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "Failed to start training %d\n", ret);
 		return ret;
 	}
 
-	ret = cdn_dp_get_training_status(dp);
+	ret = cdns_mhdp_get_training_status(mhdp);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "Failed to get training stat %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "Failed to get training stat %d\n", ret);
 		return ret;
 	}
 
-	DRM_DEV_DEBUG_KMS(dp-&gt;dev, "rate:0x%x, lanes:%d\n", dp-&gt;link.rate,
-			  dp-&gt;link.num_lanes);
+	DRM_DEV_DEBUG_KMS(mhdp-&gt;dev, "rate:0x%x, lanes:%d\n", mhdp-&gt;link.rate,
+			  mhdp-&gt;link.num_lanes);
 	return ret;
 }
 
-int cdn_dp_set_video_status(struct cdn_dp_device *dp, int active)
+int cdns_mhdp_set_video_status(struct cdns_mhdp_device *mhdp, int active)
 {
 	u8 msg;
 	int ret;
 
 	msg = !!active;
 
-	ret = cdn_dp_mailbox_send(dp, MB_MODULE_ID_DP_TX, DPTX_SET_VIDEO,
+	ret = cdns_mhdp_mailbox_send(mhdp, MB_MODULE_ID_DP_TX, DPTX_SET_VIDEO,
 				  sizeof(msg), &amp;msg);
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "set video status failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "set video status failed: %d\n", ret);
 
 	return ret;
 }
 
-static int cdn_dp_get_msa_misc(struct video_info *video,
+static int cdns_mhdp_get_msa_misc(struct video_info *video,
 			       struct drm_display_mode *mode)
 {
 	u32 msa_misc;
@@ -627,10 +632,10 @@ static int cdn_dp_get_msa_misc(struct vi
 	return msa_misc;
 }
 
-int cdn_dp_config_video(struct cdn_dp_device *dp)
+int cdns_mhdp_config_video(struct cdns_mhdp_device *mhdp)
 {
-	struct video_info *video = &amp;dp-&gt;video_info;
-	struct drm_display_mode *mode = &amp;dp-&gt;mode;
+	struct video_info *video = &amp;mhdp-&gt;video_info;
+	struct drm_display_mode *mode = &amp;mhdp-&gt;mode;
 	u64 symbol;
 	u32 val, link_rate, rem;
 	u8 bit_per_pix, tu_size_reg = TU_SIZE;
@@ -639,13 +644,13 @@ int cdn_dp_config_video(struct cdn_dp_de
 	bit_per_pix = (video-&gt;color_fmt == YCBCR_4_2_2) ?
 		      (video-&gt;color_depth * 2) : (video-&gt;color_depth * 3);
 
-	link_rate = dp-&gt;link.rate / 1000;
+	link_rate = mhdp-&gt;link.rate / 1000;
 
-	ret = cdn_dp_reg_write(dp, BND_HSYNC2VSYNC, VIF_BYPASS_INTERLACE);
+	ret = cdns_mhdp_reg_write(mhdp, BND_HSYNC2VSYNC, VIF_BYPASS_INTERLACE);
 	if (ret)
 		goto err_config_video;
 
-	ret = cdn_dp_reg_write(dp, HSYNC2VSYNC_POL_CTRL, 0);
+	ret = cdns_mhdp_reg_write(mhdp, HSYNC2VSYNC_POL_CTRL, 0);
 	if (ret)
 		goto err_config_video;
 
@@ -659,13 +664,13 @@ int cdn_dp_config_video(struct cdn_dp_de
 	do {
 		tu_size_reg += 2;
 		symbol = tu_size_reg * mode-&gt;clock * bit_per_pix;
-		do_div(symbol, dp-&gt;link.num_lanes * link_rate * 8);
+		do_div(symbol, mhdp-&gt;link.num_lanes * link_rate * 8);
 		rem = do_div(symbol, 1000);
 		if (tu_size_reg &gt; 64) {
 			ret = -EINVAL;
-			DRM_DEV_ERROR(dp-&gt;dev,
+			DRM_DEV_ERROR(mhdp-&gt;dev,
 				      "tu error, clk:%d, lanes:%d, rate:%d\n",
-				      mode-&gt;clock, dp-&gt;link.num_lanes,
+				      mode-&gt;clock, mhdp-&gt;link.num_lanes,
 				      link_rate);
 			goto err_config_video;
 		}
@@ -674,16 +679,16 @@ int cdn_dp_config_video(struct cdn_dp_de
 
 	val = symbol + (tu_size_reg &lt;&lt; 8);
 	val |= TU_CNT_RST_EN;
-	ret = cdn_dp_reg_write(dp, DP_FRAMER_TU, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_FRAMER_TU, val);
 	if (ret)
 		goto err_config_video;
 
 	/* set the FIFO Buffer size */
 	val = div_u64(mode-&gt;clock * (symbol + 1), 1000) + link_rate;
-	val /= (dp-&gt;link.num_lanes * link_rate);
+	val /= (mhdp-&gt;link.num_lanes * link_rate);
 	val = div_u64(8 * (symbol + 1), bit_per_pix) - val;
 	val += 2;
-	ret = cdn_dp_reg_write(dp, DP_VC_TABLE(15), val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_VC_TABLE(15), val);
 
 	switch (video-&gt;color_depth) {
 	case 6:
@@ -704,136 +709,137 @@ int cdn_dp_config_video(struct cdn_dp_de
 	};
 
 	val += video-&gt;color_fmt &lt;&lt; 8;
-	ret = cdn_dp_reg_write(dp, DP_FRAMER_PXL_REPR, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_FRAMER_PXL_REPR, val);
 	if (ret)
 		goto err_config_video;
 
 	val = video-&gt;h_sync_polarity ? DP_FRAMER_SP_HSP : 0;
 	val |= video-&gt;v_sync_polarity ? DP_FRAMER_SP_VSP : 0;
-	ret = cdn_dp_reg_write(dp, DP_FRAMER_SP, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_FRAMER_SP, val);
 	if (ret)
 		goto err_config_video;
 
 	val = (mode-&gt;hsync_start - mode-&gt;hdisplay) &lt;&lt; 16;
 	val |= mode-&gt;htotal - mode-&gt;hsync_end;
-	ret = cdn_dp_reg_write(dp, DP_FRONT_BACK_PORCH, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_FRONT_BACK_PORCH, val);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;hdisplay * bit_per_pix / 8;
-	ret = cdn_dp_reg_write(dp, DP_BYTE_COUNT, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_BYTE_COUNT, val);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;htotal | ((mode-&gt;htotal - mode-&gt;hsync_start) &lt;&lt; 16);
-	ret = cdn_dp_reg_write(dp, MSA_HORIZONTAL_0, val);
+	ret = cdns_mhdp_reg_write(mhdp, MSA_HORIZONTAL_0, val);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;hsync_end - mode-&gt;hsync_start;
 	val |= (mode-&gt;hdisplay &lt;&lt; 16) | (video-&gt;h_sync_polarity &lt;&lt; 15);
-	ret = cdn_dp_reg_write(dp, MSA_HORIZONTAL_1, val);
+	ret = cdns_mhdp_reg_write(mhdp, MSA_HORIZONTAL_1, val);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;vtotal;
 	val |= (mode-&gt;vtotal - mode-&gt;vsync_start) &lt;&lt; 16;
-	ret = cdn_dp_reg_write(dp, MSA_VERTICAL_0, val);
+	ret = cdns_mhdp_reg_write(mhdp, MSA_VERTICAL_0, val);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;vsync_end - mode-&gt;vsync_start;
 	val |= (mode-&gt;vdisplay &lt;&lt; 16) | (video-&gt;v_sync_polarity &lt;&lt; 15);
-	ret = cdn_dp_reg_write(dp, MSA_VERTICAL_1, val);
+	ret = cdns_mhdp_reg_write(mhdp, MSA_VERTICAL_1, val);
 	if (ret)
 		goto err_config_video;
 
-	val = cdn_dp_get_msa_misc(video, mode);
-	ret = cdn_dp_reg_write(dp, MSA_MISC, val);
+	val = cdns_mhdp_get_msa_misc(video, mode);
+	ret = cdns_mhdp_reg_write(mhdp, MSA_MISC, val);
 	if (ret)
 		goto err_config_video;
 
-	ret = cdn_dp_reg_write(dp, STREAM_CONFIG, 1);
+	ret = cdns_mhdp_reg_write(mhdp, STREAM_CONFIG, 1);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;hsync_end - mode-&gt;hsync_start;
 	val |= mode-&gt;hdisplay &lt;&lt; 16;
-	ret = cdn_dp_reg_write(dp, DP_HORIZONTAL, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_HORIZONTAL, val);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;vdisplay;
 	val |= (mode-&gt;vtotal - mode-&gt;vsync_start) &lt;&lt; 16;
-	ret = cdn_dp_reg_write(dp, DP_VERTICAL_0, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_VERTICAL_0, val);
 	if (ret)
 		goto err_config_video;
 
 	val = mode-&gt;vtotal;
-	ret = cdn_dp_reg_write(dp, DP_VERTICAL_1, val);
+	ret = cdns_mhdp_reg_write(mhdp, DP_VERTICAL_1, val);
 	if (ret)
 		goto err_config_video;
 
-	ret = cdn_dp_reg_write_bit(dp, DP_VB_ID, 2, 1, 0);
+	ret = cdns_mhdp_reg_write_bit(mhdp, DP_VB_ID, 2, 1, 0);
 
 err_config_video:
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "config video failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "config video failed: %d\n", ret);
 	return ret;
 }
 
-int cdn_dp_audio_stop(struct cdn_dp_device *dp, struct audio_info *audio)
+int cdns_mhdp_audio_stop(struct cdns_mhdp_device *mhdp,
+			 struct audio_info *audio)
 {
 	int ret;
 
-	ret = cdn_dp_reg_write(dp, AUDIO_PACK_CONTROL, 0);
+	ret = cdns_mhdp_reg_write(mhdp, AUDIO_PACK_CONTROL, 0);
 	if (ret) {
-		DRM_DEV_ERROR(dp-&gt;dev, "audio stop failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "audio stop failed: %d\n", ret);
 		return ret;
 	}
 
-	writel(0, dp-&gt;regs + SPDIF_CTRL_ADDR);
+	writel(0, mhdp-&gt;regs + SPDIF_CTRL_ADDR);
 
 	/* clearn the audio config and reset */
-	writel(0, dp-&gt;regs + AUDIO_SRC_CNTL);
-	writel(0, dp-&gt;regs + AUDIO_SRC_CNFG);
-	writel(AUDIO_SW_RST, dp-&gt;regs + AUDIO_SRC_CNTL);
-	writel(0, dp-&gt;regs + AUDIO_SRC_CNTL);
+	writel(0, mhdp-&gt;regs + AUDIO_SRC_CNTL);
+	writel(0, mhdp-&gt;regs + AUDIO_SRC_CNFG);
+	writel(AUDIO_SW_RST, mhdp-&gt;regs + AUDIO_SRC_CNTL);
+	writel(0, mhdp-&gt;regs + AUDIO_SRC_CNTL);
 
 	/* reset smpl2pckt component  */
-	writel(0, dp-&gt;regs + SMPL2PKT_CNTL);
-	writel(AUDIO_SW_RST, dp-&gt;regs + SMPL2PKT_CNTL);
-	writel(0, dp-&gt;regs + SMPL2PKT_CNTL);
+	writel(0, mhdp-&gt;regs + SMPL2PKT_CNTL);
+	writel(AUDIO_SW_RST, mhdp-&gt;regs + SMPL2PKT_CNTL);
+	writel(0, mhdp-&gt;regs + SMPL2PKT_CNTL);
 
 	/* reset FIFO */
-	writel(AUDIO_SW_RST, dp-&gt;regs + FIFO_CNTL);
-	writel(0, dp-&gt;regs + FIFO_CNTL);
+	writel(AUDIO_SW_RST, mhdp-&gt;regs + FIFO_CNTL);
+	writel(0, mhdp-&gt;regs + FIFO_CNTL);
 
 	if (audio-&gt;format == AFMT_SPDIF)
-		clk_disable_unprepare(dp-&gt;spdif_clk);
+		clk_disable_unprepare(mhdp-&gt;spdif_clk);
 
 	return 0;
 }
 
-int cdn_dp_audio_mute(struct cdn_dp_device *dp, bool enable)
+int cdns_mhdp_audio_mute(struct cdns_mhdp_device *mhdp, bool enable)
 {
 	int ret;
 
-	ret = cdn_dp_reg_write_bit(dp, DP_VB_ID, 4, 1, enable);
+	ret = cdns_mhdp_reg_write_bit(mhdp, DP_VB_ID, 4, 1, enable);
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "audio mute failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "audio mute failed: %d\n", ret);
 
 	return ret;
 }
 
-static void cdn_dp_audio_config_i2s(struct cdn_dp_device *dp,
-				    struct audio_info *audio)
+static void cdns_mhdp_audio_config_i2s(struct cdns_mhdp_device *mhdp,
+				       struct audio_info *audio)
 {
 	int sub_pckt_num = 1, i2s_port_en_val = 0xf, i;
 	u32 val;
 
 	if (audio-&gt;channels == 2) {
-		if (dp-&gt;link.num_lanes == 1)
+		if (mhdp-&gt;link.num_lanes == 1)
 			sub_pckt_num = 2;
 		else
 			sub_pckt_num = 4;
@@ -843,15 +849,15 @@ static void cdn_dp_audio_config_i2s(stru
 		i2s_port_en_val = 3;
 	}
 
-	writel(0x0, dp-&gt;regs + SPDIF_CTRL_ADDR);
+	writel(0x0, mhdp-&gt;regs + SPDIF_CTRL_ADDR);
 
-	writel(SYNC_WR_TO_CH_ZERO, dp-&gt;regs + FIFO_CNTL);
+	writel(SYNC_WR_TO_CH_ZERO, mhdp-&gt;regs + FIFO_CNTL);
 
 	val = MAX_NUM_CH(audio-&gt;channels);
 	val |= NUM_OF_I2S_PORTS(audio-&gt;channels);
 	val |= AUDIO_TYPE_LPCM;
 	val |= CFG_SUB_PCKT_NUM(sub_pckt_num);
-	writel(val, dp-&gt;regs + SMPL2PKT_CNFG);
+	writel(val, mhdp-&gt;regs + SMPL2PKT_CNFG);
 
 	if (audio-&gt;sample_width == 16)
 		val = 0;
@@ -863,7 +869,7 @@ static void cdn_dp_audio_config_i2s(stru
 	val |= AUDIO_CH_NUM(audio-&gt;channels);
 	val |= I2S_DEC_PORT_EN(i2s_port_en_val);
 	val |= TRANS_SMPL_WIDTH_32;
-	writel(val, dp-&gt;regs + AUDIO_SRC_CNFG);
+	writel(val, mhdp-&gt;regs + AUDIO_SRC_CNFG);
 
 	for (i = 0; i &lt; (audio-&gt;channels + 1) / 2; i++) {
 		if (audio-&gt;sample_width == 16)
@@ -872,7 +878,7 @@ static void cdn_dp_audio_config_i2s(stru
 			val = (0x0b &lt;&lt; 8) | (0x0b &lt;&lt; 20);
 
 		val |= ((2 * i) &lt;&lt; 4) | ((2 * i + 1) &lt;&lt; 16);
-		writel(val, dp-&gt;regs + STTS_BIT_CH(i));
+		writel(val, mhdp-&gt;regs + STTS_BIT_CH(i));
 	}
 
 	switch (audio-&gt;sample_rate) {
@@ -906,56 +912,57 @@ static void cdn_dp_audio_config_i2s(stru
 		break;
 	}
 	val |= 4;
-	writel(val, dp-&gt;regs + COM_CH_STTS_BITS);
+	writel(val, mhdp-&gt;regs + COM_CH_STTS_BITS);
 
-	writel(SMPL2PKT_EN, dp-&gt;regs + SMPL2PKT_CNTL);
-	writel(I2S_DEC_START, dp-&gt;regs + AUDIO_SRC_CNTL);
+	writel(SMPL2PKT_EN, mhdp-&gt;regs + SMPL2PKT_CNTL);
+	writel(I2S_DEC_START, mhdp-&gt;regs + AUDIO_SRC_CNTL);
 }
 
-static void cdn_dp_audio_config_spdif(struct cdn_dp_device *dp)
+static void cdns_mhdp_audio_config_spdif(struct cdns_mhdp_device *mhdp)
 {
 	u32 val;
 
-	writel(SYNC_WR_TO_CH_ZERO, dp-&gt;regs + FIFO_CNTL);
+	writel(SYNC_WR_TO_CH_ZERO, mhdp-&gt;regs + FIFO_CNTL);
 
 	val = MAX_NUM_CH(2) | AUDIO_TYPE_LPCM | CFG_SUB_PCKT_NUM(4);
-	writel(val, dp-&gt;regs + SMPL2PKT_CNFG);
-	writel(SMPL2PKT_EN, dp-&gt;regs + SMPL2PKT_CNTL);
+	writel(val, mhdp-&gt;regs + SMPL2PKT_CNFG);
+	writel(SMPL2PKT_EN, mhdp-&gt;regs + SMPL2PKT_CNTL);
 
 	val = SPDIF_ENABLE | SPDIF_AVG_SEL | SPDIF_JITTER_BYPASS;
-	writel(val, dp-&gt;regs + SPDIF_CTRL_ADDR);
+	writel(val, mhdp-&gt;regs + SPDIF_CTRL_ADDR);
 
-	clk_prepare_enable(dp-&gt;spdif_clk);
-	clk_set_rate(dp-&gt;spdif_clk, CDN_DP_SPDIF_CLK);
+	clk_prepare_enable(mhdp-&gt;spdif_clk);
+	clk_set_rate(mhdp-&gt;spdif_clk, CDNS_DP_SPDIF_CLK);
 }
 
-int cdn_dp_audio_config(struct cdn_dp_device *dp, struct audio_info *audio)
+int cdns_mhdp_audio_config(struct cdns_mhdp_device *mhdp,
+			   struct audio_info *audio)
 {
 	int ret;
 
 	/* reset the spdif clk before config */
 	if (audio-&gt;format == AFMT_SPDIF) {
-		reset_control_assert(dp-&gt;spdif_rst);
-		reset_control_deassert(dp-&gt;spdif_rst);
+		reset_control_assert(mhdp-&gt;spdif_rst);
+		reset_control_deassert(mhdp-&gt;spdif_rst);
 	}
 
-	ret = cdn_dp_reg_write(dp, CM_LANE_CTRL, LANE_REF_CYC);
+	ret = cdns_mhdp_reg_write(mhdp, CM_LANE_CTRL, LANE_REF_CYC);
 	if (ret)
 		goto err_audio_config;
 
-	ret = cdn_dp_reg_write(dp, CM_CTRL, 0);
+	ret = cdns_mhdp_reg_write(mhdp, CM_CTRL, 0);
 	if (ret)
 		goto err_audio_config;
 
 	if (audio-&gt;format == AFMT_I2S)
-		cdn_dp_audio_config_i2s(dp, audio);
+		cdns_mhdp_audio_config_i2s(mhdp, audio);
 	else if (audio-&gt;format == AFMT_SPDIF)
-		cdn_dp_audio_config_spdif(dp);
+		cdns_mhdp_audio_config_spdif(mhdp);
 
-	ret = cdn_dp_reg_write(dp, AUDIO_PACK_CONTROL, AUDIO_PACK_EN);
+	ret = cdns_mhdp_reg_write(mhdp, AUDIO_PACK_CONTROL, AUDIO_PACK_EN);
 
 err_audio_config:
 	if (ret)
-		DRM_DEV_ERROR(dp-&gt;dev, "audio config failed: %d\n", ret);
+		DRM_DEV_ERROR(mhdp-&gt;dev, "audio config failed: %d\n", ret);
 	return ret;
 }
--- a/drivers/gpu/drm/rockchip/cdn-dp-reg.h
+++ b/drivers/gpu/drm/rockchip/cdn-dp-reg.h
@@ -387,7 +387,7 @@
 #define HDCP_TX_IS_RECEIVER_ID_VALID_EVENT	BIT(7)
 
 #define TU_SIZE					30
-#define CDN_DP_MAX_LINK_RATE			DP_LINK_BW_5_4
+#define CDNS_DP_MAX_LINK_RATE			DP_LINK_BW_5_4
 
 /* audio */
 #define AUDIO_PACK_EN				BIT(8)
@@ -451,24 +451,96 @@ enum vic_bt_type {
 	BT_709 = 0x1,
 };
 
-void cdn_dp_clock_reset(struct cdn_dp_device *dp);
+enum audio_format {
+	AFMT_I2S = 0,
+	AFMT_SPDIF = 1,
+	AFMT_UNUSED,
+};
+
+struct audio_info {
+	enum audio_format format;
+	int sample_rate;
+	int channels;
+	int sample_width;
+};
+
+enum vic_pxl_encoding_format {
+	PXL_RGB = 0x1,
+	YCBCR_4_4_4 = 0x2,
+	YCBCR_4_2_2 = 0x4,
+	YCBCR_4_2_0 = 0x8,
+	Y_ONLY = 0x10,
+};
+
+struct video_info {
+	bool h_sync_polarity;
+	bool v_sync_polarity;
+	bool interlaced;
+	int color_depth;
+	enum vic_pxl_encoding_format color_fmt;
+};
+
+struct cdns_mhdp_host {
+	unsigned int	link_rate;
+	u8	lanes_cnt;
+	u8	volt_swing;
+	u8	pre_emphasis;
+	u8	pattern_supp;
+	u8	fast_link;
+	u8	lane_mapping;
+	u8	enhanced;
+};
+
+struct cdns_mhdp_sink {
+	unsigned int	link_rate;
+	u8	lanes_cnt;
+	u8	pattern_supp;
+	u8	fast_link;
+	u8	enhanced;
+};
+
+struct cdns_mhdp_device {
+	void __iomem		*regs;
+
+	struct device		*dev;
+
+	struct drm_dp_link	link;
+	struct drm_connector	connector;
+	struct clk		*spdif_clk;
+	struct reset_control	*spdif_rst;
+
+	struct drm_dp_aux	aux;
+	struct cdns_mhdp_host	host;
+	struct cdns_mhdp_sink	sink;
+	struct drm_bridge	bridge;
+	struct phy		*phy;
+	void __iomem		*dbg_regs;
+
+	struct video_info	video_info;
+	struct drm_display_mode	mode;
+	unsigned int		fw_version;
+};
 
-void cdn_dp_set_fw_clk(struct cdn_dp_device *dp, unsigned long clk);
-int cdn_dp_load_firmware(struct cdn_dp_device *dp, const u32 *i_mem,
-			 u32 i_size, const u32 *d_mem, u32 d_size);
-int cdn_dp_set_firmware_active(struct cdn_dp_device *dp, bool enable);
-int cdn_dp_set_host_cap(struct cdn_dp_device *dp, u8 lanes, bool flip);
-int cdn_dp_event_config(struct cdn_dp_device *dp);
-u32 cdn_dp_get_event(struct cdn_dp_device *dp);
-int cdn_dp_get_hpd_status(struct cdn_dp_device *dp);
-int cdn_dp_dpcd_write(struct cdn_dp_device *dp, u32 addr, u8 value);
-int cdn_dp_dpcd_read(struct cdn_dp_device *dp, u32 addr, u8 *data, u16 len);
-int cdn_dp_get_edid_block(void *dp, u8 *edid,
-			  unsigned int block, size_t length);
-int cdn_dp_train_link(struct cdn_dp_device *dp);
-int cdn_dp_set_video_status(struct cdn_dp_device *dp, int active);
-int cdn_dp_config_video(struct cdn_dp_device *dp);
-int cdn_dp_audio_stop(struct cdn_dp_device *dp, struct audio_info *audio);
-int cdn_dp_audio_mute(struct cdn_dp_device *dp, bool enable);
-int cdn_dp_audio_config(struct cdn_dp_device *dp, struct audio_info *audio);
+void cdns_mhdp_clock_reset(struct cdns_mhdp_device *mhdp);
+void cdns_mhdp_set_fw_clk(struct cdns_mhdp_device *mhdp, unsigned long clk);
+int cdns_mhdp_load_firmware(struct cdns_mhdp_device *mhdp, const u32 *i_mem,
+			    u32 i_size, const u32 *d_mem, u32 d_size);
+int cdns_mhdp_set_firmware_active(struct cdns_mhdp_device *mhdp, bool enable);
+int cdns_mhdp_set_host_cap(struct cdns_mhdp_device *mhdp, u8 lanes, bool flip);
+int cdns_mhdp_event_config(struct cdns_mhdp_device *mhdp);
+u32 cdns_mhdp_get_event(struct cdns_mhdp_device *mhdp);
+int cdns_mhdp_get_hpd_status(struct cdns_mhdp_device *mhdp);
+int cdns_mhdp_dpcd_write(struct cdns_mhdp_device *mhdp, u32 addr, u8 value);
+int cdns_mhdp_dpcd_read(struct cdns_mhdp_device *mhdp,
+			u32 addr, u8 *data, u16 len);
+int cdns_mhdp_get_edid_block(void *mhdp, u8 *edid,
+			     unsigned int block, size_t length);
+int cdns_mhdp_train_link(struct cdns_mhdp_device *mhdp);
+int cdns_mhdp_set_video_status(struct cdns_mhdp_device *mhdp, int active);
+int cdns_mhdp_config_video(struct cdns_mhdp_device *mhdp);
+int cdns_mhdp_audio_stop(struct cdns_mhdp_device *mhdp,
+			 struct audio_info *audio);
+int cdns_mhdp_audio_mute(struct cdns_mhdp_device *mhdp, bool enable);
+int cdns_mhdp_audio_config(struct cdns_mhdp_device *mhdp,
+			   struct audio_info *audio);
 #endif /* _CDN_DP_REG_H */
</pre></body></html>