ARM64: DTS: Fix Firefly board audio driver
[firefly-linux-kernel-4.4.55.git] / drivers / gpu / drm / rockchip / dw-mipi-dsi.c
index 70d194a208e3527bf3741270e93e9ae1518c2d7e..0e948d6110419990b78146c40f112ce90954514d 100644 (file)
 #define RK3288_DSI0_SEL_VOP_LIT                BIT(6)
 #define RK3288_DSI1_SEL_VOP_LIT                BIT(9)
 
+#define RK3288_GRF_SOC_CON9            0x0268
+
+#define RK3288_GRF_SOC_CON14           0x027c
+#define RK3288_TXRX_BASEDIR            BIT(15)
+#define RK3288_TXRX_MASTERSLAVEZ       BIT(14)
+#define RK3288_TXRX_CLKEN              BIT(12)
+
 #define RK3366_GRF_SOC_CON0            0x0400
 #define RK3366_DSI_SEL_VOP_LIT         BIT(2)
 
 
 /* disable turnrequest, turndisable, forcetxstopmode, forcerxmode */
 #define RK3399_GRF_SOC_CON22           0x6258
-#define RK3399_GRF_DSI_MODE            0xffff0000
+#define RK3399_GRF_DSI0_MODE           0xffff0000
+/* disable turndisable, forcetxstopmode, forcerxmode, enable */
+#define RK3399_GRF_SOC_CON23           0x625c
+#define RK3399_GRF_DSI1_MODE1          0xffff0000
+#define RK3399_GRF_DSI1_ENABLE         0x000f000f
+/* disable basedir and enable clk*/
+#define RK3399_GRF_SOC_CON24           0x6260
+#define RK3399_TXRX_MASTERSLAVEZ       BIT(7)
+#define RK3399_TXRX_ENABLECLK          BIT(6)
+#define RK3399_TXRX_BASEDIR            BIT(5)
+#define RK3399_GRF_DSI1_MODE2          0x00600040
 
 #define DSI_VERSION                    0x00
 #define DSI_PWR_UP                     0x04
 #define ENABLE_CMD_MODE                        BIT(0)
 
 #define DSI_VID_MODE_CFG               0x38
+#define VPG_EN                         BIT(16)
 #define FRAME_BTA_ACK                  BIT(14)
-#define ENABLE_LOW_POWER               (0x3f << 8)
-#define ENABLE_LOW_POWER_MASK          (0x3f << 8)
+#define LP_HFP_EN                      BIT(13)
+#define LP_HBP_EN                      BIT(12)
+#define ENABLE_LOW_POWER               (0xf << 8)
+#define ENABLE_LOW_POWER_MASK          (0xf << 8)
 #define VID_MODE_TYPE_BURST_SYNC_PULSES        0x0
 #define VID_MODE_TYPE_BURST_SYNC_EVENTS        0x1
 #define VID_MODE_TYPE_BURST            0x2
@@ -285,6 +305,13 @@ struct dw_mipi_dsi_plat_data {
        u32 grf_switch_reg;
        u32 grf_dsi0_mode;
        u32 grf_dsi0_mode_reg;
+       u32 grf_dsi1_mode;
+       u32 grf_dsi1_mode_reg1;
+       u32 dsi1_basedir;
+       u32 dsi1_masterslavez;
+       u32 dsi1_enableclk;
+       u32 grf_dsi1_mode_reg2;
+       u32 grf_dsi1_cfg_reg;
        unsigned int max_data_lanes;
        u32 max_bit_rate_per_lane;
        bool has_vop_sel;
@@ -292,35 +319,49 @@ struct dw_mipi_dsi_plat_data {
                                           struct drm_display_mode *mode);
 };
 
+struct mipi_dphy {
+       /* SNPS PHY */
+       struct clk *cfg_clk;
+       struct clk *ref_clk;
+       u16 input_div;
+       u16 feedback_div;
+
+       /* Non-SNPS PHY */
+       struct phy *phy;
+       struct clk *hs_clk;
+};
+
 struct dw_mipi_dsi {
        struct drm_encoder encoder;
        struct drm_connector connector;
        struct mipi_dsi_host dsi_host;
-       struct phy *phy;
+       struct mipi_dphy dphy;
        struct drm_panel *panel;
        struct device *dev;
        struct regmap *grf_regmap;
        struct reset_control *rst;
        void __iomem *base;
-
-       struct clk *pllref_clk;
        struct clk *pclk;
-       struct clk *phy_cfg_clk;
 
+       /* dual-channel */
+       struct dw_mipi_dsi *master;
+       struct dw_mipi_dsi *slave;
+       struct device_node *panel_node;
+       int id;
+
+       unsigned long mode_flags;
        unsigned int lane_mbps; /* per lane */
        u32 channel;
        u32 lanes;
        u32 format;
-       u16 input_div;
-       u16 feedback_div;
        struct drm_display_mode mode;
 
        const struct dw_mipi_dsi_plat_data *pdata;
 };
 
 enum dw_mipi_dsi_mode {
-       DW_MIPI_DSI_CMD_MODE,
-       DW_MIPI_DSI_VID_MODE,
+       DSI_COMMAND_MODE,
+       DSI_VIDEO_MODE,
 };
 
 struct dphy_pll_testdin_map {
@@ -382,7 +423,7 @@ static int rockchip_wait_w_pld_fifo_not_full(struct dw_mipi_dsi *dsi)
        u32 sts;
        int ret;
 
-       ret = readx_poll_timeout(readl, dsi->base + DSI_CMD_PKT_STATUS,
+       ret = readl_poll_timeout(dsi->base + DSI_CMD_PKT_STATUS,
                                 sts, !(sts & GEN_PLD_W_FULL), 10,
                                 CMD_PKT_STATUS_TIMEOUT_US);
        if (ret < 0) {
@@ -398,7 +439,7 @@ static int rockchip_wait_cmd_fifo_not_full(struct dw_mipi_dsi *dsi)
        u32 sts;
        int ret;
 
-       ret = readx_poll_timeout(readl, dsi->base + DSI_CMD_PKT_STATUS,
+       ret = readl_poll_timeout(dsi->base + DSI_CMD_PKT_STATUS,
                                 sts, !(sts & GEN_CMD_FULL), 10,
                                 CMD_PKT_STATUS_TIMEOUT_US);
        if (ret < 0) {
@@ -416,7 +457,7 @@ static int rockchip_wait_write_fifo_empty(struct dw_mipi_dsi *dsi)
        int ret;
 
        mask = GEN_CMD_EMPTY | GEN_PLD_W_EMPTY;
-       ret = readx_poll_timeout(readl, dsi->base + DSI_CMD_PKT_STATUS,
+       ret = readl_poll_timeout(dsi->base + DSI_CMD_PKT_STATUS,
                                 sts, (sts & mask) == mask, 10,
                                 CMD_PKT_STATUS_TIMEOUT_US);
        if (ret < 0) {
@@ -464,8 +505,8 @@ static int dw_mipi_dsi_phy_init(struct dw_mipi_dsi *dsi)
 
        dsi_write(dsi, DSI_PWR_UP, POWERUP);
 
-       if (!IS_ERR(dsi->phy_cfg_clk)) {
-               ret = clk_prepare_enable(dsi->phy_cfg_clk);
+       if (!IS_ERR(dsi->dphy.cfg_clk)) {
+               ret = clk_prepare_enable(dsi->dphy.cfg_clk);
                if (ret) {
                        dev_err(dsi->dev, "Failed to enable phy_cfg_clk\n");
                        return ret;
@@ -483,12 +524,12 @@ static int dw_mipi_dsi_phy_init(struct dw_mipi_dsi *dsi)
 
        dw_mipi_dsi_phy_write(dsi, 0x44, HSFREQRANGE_SEL(testdin));
 
+       dw_mipi_dsi_phy_write(dsi, 0x17, INPUT_DIVIDER(dsi->dphy.input_div));
+       val = LOOP_DIV_LOW_SEL(dsi->dphy.feedback_div) | LOW_PROGRAM_EN;
+       dw_mipi_dsi_phy_write(dsi, 0x18, val);
+       val = LOOP_DIV_HIGH_SEL(dsi->dphy.feedback_div) | HIGH_PROGRAM_EN;
+       dw_mipi_dsi_phy_write(dsi, 0x18, val);
        dw_mipi_dsi_phy_write(dsi, 0x19, PLL_LOOP_DIV_EN | PLL_INPUT_DIV_EN);
-       dw_mipi_dsi_phy_write(dsi, 0x17, INPUT_DIVIDER(dsi->input_div));
-       dw_mipi_dsi_phy_write(dsi, 0x18, LOOP_DIV_LOW_SEL(dsi->feedback_div) |
-                                        LOW_PROGRAM_EN);
-       dw_mipi_dsi_phy_write(dsi, 0x18, LOOP_DIV_HIGH_SEL(dsi->feedback_div) |
-                                        HIGH_PROGRAM_EN);
 
        dw_mipi_dsi_phy_write(dsi, 0x20, POWER_CONTROL | INTERNAL_REG_CURRENT |
                                         BIAS_BLOCK_ON | BANDGAP_ON);
@@ -511,15 +552,14 @@ static int dw_mipi_dsi_phy_init(struct dw_mipi_dsi *dsi)
        dsi_write(dsi, DSI_PHY_RSTZ, PHY_ENFORCEPLL | PHY_ENABLECLK |
                                     PHY_UNRSTZ | PHY_UNSHUTDOWNZ);
 
-
-       ret = readx_poll_timeout(readl, dsi->base + DSI_PHY_STATUS,
+       ret = readl_poll_timeout(dsi->base + DSI_PHY_STATUS,
                                 val, val & LOCK, 1000, PHY_STATUS_TIMEOUT_US);
        if (ret < 0) {
                dev_err(dsi->dev, "failed to wait for phy lock state\n");
                goto phy_init_end;
        }
 
-       ret = readx_poll_timeout(readl, dsi->base + DSI_PHY_STATUS,
+       ret = readl_poll_timeout(dsi->base + DSI_PHY_STATUS,
                                 val, val & STOP_STATE_CLK_LANE, 1000,
                                 PHY_STATUS_TIMEOUT_US);
        if (ret < 0)
@@ -527,38 +567,63 @@ static int dw_mipi_dsi_phy_init(struct dw_mipi_dsi *dsi)
                        "failed to wait for phy clk lane stop state\n");
 
 phy_init_end:
-       if (!IS_ERR(dsi->phy_cfg_clk))
-               clk_disable_unprepare(dsi->phy_cfg_clk);
+       if (!IS_ERR(dsi->dphy.cfg_clk))
+               clk_disable_unprepare(dsi->dphy.cfg_clk);
 
        return ret;
 }
 
-static int dw_mipi_dsi_get_lane_bps(struct dw_mipi_dsi *dsi)
+static unsigned long rockchip_dsi_calc_bandwidth(struct dw_mipi_dsi *dsi)
 {
-       unsigned int i, pre;
-       unsigned long mpclk, pllref, tmp;
-       unsigned int m = 1, n = 1, target_mbps = 1000;
-       unsigned int max_mbps = dptdin_map[ARRAY_SIZE(dptdin_map) - 1].max_mbps;
        int bpp;
+       unsigned long mpclk, tmp;
+       unsigned long target_mbps = 1000;
+       unsigned int value;
+       struct device_node *np = dsi->dev->of_node;
+       unsigned int max_mbps;
+       int lanes;
+
+       /* optional override of the desired bandwidth */
+       if (!of_property_read_u32(np, "rockchip,lane-rate", &value))
+               return value;
+
+       max_mbps = dsi->pdata->max_bit_rate_per_lane / USEC_PER_SEC;
 
        bpp = mipi_dsi_pixel_format_to_bpp(dsi->format);
        if (bpp < 0) {
                dev_err(dsi->dev, "failed to get bpp for pixel format %d\n",
                        dsi->format);
-               return bpp;
+               bpp = 24;
        }
 
+       lanes = dsi->slave ? dsi->lanes * 2 : dsi->lanes;
+
        mpclk = DIV_ROUND_UP(dsi->mode.clock, MSEC_PER_SEC);
        if (mpclk) {
                /* take 1 / 0.9, since mbps must big than bandwidth of RGB */
-               tmp = mpclk * (bpp / dsi->lanes) * 10 / 9;
+               tmp = mpclk * (bpp / lanes) * 10 / 9;
                if (tmp < max_mbps)
                        target_mbps = tmp;
                else
                        dev_err(dsi->dev, "DPHY clock frequency is out of range\n");
        }
 
-       pllref = DIV_ROUND_UP(clk_get_rate(dsi->pllref_clk), USEC_PER_SEC);
+       return target_mbps;
+}
+
+static int dw_mipi_dsi_get_lane_bps(struct dw_mipi_dsi *dsi)
+{
+       unsigned int i, pre;
+       unsigned long pllref, tmp;
+       unsigned int m = 1, n = 1;
+       unsigned long target_mbps;
+
+       if (dsi->master)
+               return 0;
+
+       target_mbps = rockchip_dsi_calc_bandwidth(dsi);
+
+       pllref = DIV_ROUND_UP(clk_get_rate(dsi->dphy.ref_clk), USEC_PER_SEC);
        tmp = pllref;
 
        for (i = 1; i < 6; i++) {
@@ -573,20 +638,51 @@ static int dw_mipi_dsi_get_lane_bps(struct dw_mipi_dsi *dsi)
        }
 
        dsi->lane_mbps = pllref / n * m;
-       dsi->input_div = n;
-       dsi->feedback_div = m;
+       dsi->dphy.input_div = n;
+       dsi->dphy.feedback_div = m;
+       if (dsi->slave) {
+               dsi->slave->lane_mbps = dsi->lane_mbps;
+               dsi->slave->dphy.input_div = n;
+               dsi->slave->dphy.feedback_div = m;
+       }
 
        return 0;
 }
 
+static void rockchip_dsi_set_hs_clk(struct dw_mipi_dsi *dsi)
+{
+       int ret;
+       unsigned long target_mbps;
+       unsigned long bw, rate;
+
+       target_mbps = rockchip_dsi_calc_bandwidth(dsi);
+       bw = target_mbps * USEC_PER_SEC;
+
+       rate = clk_round_rate(dsi->dphy.hs_clk, bw);
+       ret = clk_set_rate(dsi->dphy.hs_clk, rate);
+       if (ret)
+               dev_err(dsi->dev, "failed to set hs clock rate: %lu\n",
+                       rate);
+
+       clk_prepare_enable(dsi->dphy.hs_clk);
+
+       dsi->lane_mbps = rate / USEC_PER_SEC;
+}
+
 static int dw_mipi_dsi_host_attach(struct mipi_dsi_host *host,
                                   struct mipi_dsi_device *device)
 {
        struct dw_mipi_dsi *dsi = host_to_dsi(host);
+       int lanes;
+
+       if (dsi->master)
+               return 0;
 
-       if (device->lanes > dsi->pdata->max_data_lanes) {
+       lanes = dsi->slave ? device->lanes / 2 : device->lanes;
+
+       if (lanes > dsi->pdata->max_data_lanes) {
                dev_err(dsi->dev, "the number of data lanes(%u) is too many\n",
-                               device->lanes);
+                               lanes);
                return -EINVAL;
        }
 
@@ -595,9 +691,18 @@ static int dw_mipi_dsi_host_attach(struct mipi_dsi_host *host,
                return -EINVAL;
        }
 
-       dsi->lanes = device->lanes;
+       dsi->lanes = lanes;
        dsi->channel = device->channel;
        dsi->format = device->format;
+       dsi->mode_flags = device->mode_flags;
+
+       if (dsi->slave) {
+               dsi->slave->lanes = lanes;
+               dsi->slave->channel = device->channel;
+               dsi->slave->format = device->format;
+               dsi->slave->mode_flags = device->mode_flags;
+       }
+
        dsi->panel = of_drm_find_panel(device->dev.of_node);
        if (!dsi->panel) {
                DRM_ERROR("failed to find panel\n");
@@ -621,16 +726,18 @@ static int dw_mipi_dsi_host_detach(struct mipi_dsi_host *host,
 
 static void rockchip_set_transfer_mode(struct dw_mipi_dsi *dsi, int flags)
 {
-       if (flags & MIPI_DSI_MSG_USE_LPM)
+       if (flags & MIPI_DSI_MSG_USE_LPM) {
                dsi_write(dsi, DSI_CMD_MODE_CFG, CMD_MODE_ALL_LP);
-       else
+               dsi_write(dsi, DSI_LPCLK_CTRL, 0);
+       } else {
                dsi_write(dsi, DSI_CMD_MODE_CFG, 0);
+               dsi_write(dsi, DSI_LPCLK_CTRL, PHY_TXREQUESTCLKHS);
+       }
 }
 
-static ssize_t dw_mipi_dsi_host_transfer(struct mipi_dsi_host *host,
-                                        const struct mipi_dsi_msg *msg)
+static ssize_t rockchip_dsi_send_packet(struct dw_mipi_dsi *dsi,
+                                       const struct mipi_dsi_msg *msg)
 {
-       struct dw_mipi_dsi *dsi = host_to_dsi(host);
        struct mipi_dsi_packet packet;
        int ret;
        int val;
@@ -686,9 +793,20 @@ static ssize_t dw_mipi_dsi_host_transfer(struct mipi_dsi_host *host,
        if (ret)
                return ret;
 
+       if (dsi->slave)
+               rockchip_dsi_send_packet(dsi->slave, msg);
+
        return len;
 }
 
+static ssize_t dw_mipi_dsi_host_transfer(struct mipi_dsi_host *host,
+                                        const struct mipi_dsi_msg *msg)
+{
+       struct dw_mipi_dsi *dsi = host_to_dsi(host);
+
+       return rockchip_dsi_send_packet(dsi, msg);
+}
+
 static const struct mipi_dsi_host_ops dw_mipi_dsi_host_ops = {
        .attach = dw_mipi_dsi_host_attach,
        .detach = dw_mipi_dsi_host_detach,
@@ -699,7 +817,14 @@ static void dw_mipi_dsi_video_mode_config(struct dw_mipi_dsi *dsi)
 {
        u32 val;
 
-       val = VID_MODE_TYPE_BURST | ENABLE_LOW_POWER;
+       val = LP_HFP_EN | ENABLE_LOW_POWER;
+
+       if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_BURST)
+               val |= VID_MODE_TYPE_BURST;
+       else if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE)
+               val |= VID_MODE_TYPE_BURST_SYNC_PULSES;
+       else
+               val |= VID_MODE_TYPE_BURST_SYNC_EVENTS;
 
        dsi_write(dsi, DSI_VID_MODE_CFG, val);
 }
@@ -707,32 +832,28 @@ static void dw_mipi_dsi_video_mode_config(struct dw_mipi_dsi *dsi)
 static void dw_mipi_dsi_set_mode(struct dw_mipi_dsi *dsi,
                                 enum dw_mipi_dsi_mode mode)
 {
-       if (mode == DW_MIPI_DSI_CMD_MODE) {
-               dsi_write(dsi, DSI_PWR_UP, RESET);
+       if (mode == DSI_COMMAND_MODE) {
                dsi_write(dsi, DSI_MODE_CFG, ENABLE_CMD_MODE);
-               dsi_write(dsi, DSI_PWR_UP, POWERUP);
        } else {
                dsi_write(dsi, DSI_PWR_UP, RESET);
+               dsi_write(dsi, DSI_LPCLK_CTRL, PHY_TXREQUESTCLKHS);
                dsi_write(dsi, DSI_MODE_CFG, ENABLE_VIDEO_MODE);
-               dw_mipi_dsi_video_mode_config(dsi);
                dsi_write(dsi, DSI_PWR_UP, POWERUP);
        }
 }
 
-static void dw_mipi_dsi_disable(struct dw_mipi_dsi *dsi)
-{
-       dsi_write(dsi, DSI_PWR_UP, RESET);
-       dsi_write(dsi, DSI_PHY_RSTZ, PHY_RSTZ);
-}
-
 static void dw_mipi_dsi_init(struct dw_mipi_dsi *dsi)
 {
+       u32 esc_clk_div;
+
        dsi_write(dsi, DSI_PWR_UP, RESET);
        dsi_write(dsi, DSI_PHY_RSTZ, PHY_DISFORCEPLL | PHY_DISABLECLK
                  | PHY_RSTZ | PHY_SHUTDOWNZ);
+
+       /* The maximum value of the escape clock frequency is 20MHz */
+       esc_clk_div = DIV_ROUND_UP(dsi->lane_mbps >> 3, 20);
        dsi_write(dsi, DSI_CLKMGR_CFG, TO_CLK_DIVIDSION(10) |
-                 TX_ESC_CLK_DIVIDSION(7));
-       dsi_write(dsi, DSI_LPCLK_CTRL, PHY_TXREQUESTCLKHS);
+                 TX_ESC_CLK_DIVIDSION(esc_clk_div));
 }
 
 static void dw_mipi_dsi_dpi_config(struct dw_mipi_dsi *dsi,
@@ -755,9 +876,9 @@ static void dw_mipi_dsi_dpi_config(struct dw_mipi_dsi *dsi,
                break;
        }
 
-       if (!(mode->flags & DRM_MODE_FLAG_PVSYNC))
+       if (mode->flags & DRM_MODE_FLAG_PHSYNC)
                val |= VSYNC_ACTIVE_LOW;
-       if (!(mode->flags & DRM_MODE_FLAG_PHSYNC))
+       if (mode->flags & DRM_MODE_FLAG_PVSYNC)
                val |= HSYNC_ACTIVE_LOW;
 
        dsi_write(dsi, DSI_DPI_VCID, DPI_VID(dsi->channel));
@@ -775,30 +896,36 @@ static void dw_mipi_dsi_packet_handler_config(struct dw_mipi_dsi *dsi)
 static void dw_mipi_dsi_video_packet_config(struct dw_mipi_dsi *dsi,
                                            struct drm_display_mode *mode)
 {
-       dsi_write(dsi, DSI_VID_PKT_SIZE, VID_PKT_SIZE(mode->hdisplay));
+       int pkt_size;
+
+       if (dsi->slave || dsi->master)
+               pkt_size = VID_PKT_SIZE(mode->hdisplay / 2 + 4);
+       else
+               pkt_size = VID_PKT_SIZE(mode->hdisplay);
+
+       dsi_write(dsi, DSI_VID_PKT_SIZE, pkt_size);
 }
 
 static void dw_mipi_dsi_command_mode_config(struct dw_mipi_dsi *dsi)
 {
        dsi_write(dsi, DSI_TO_CNT_CFG, HSTX_TO_CNT(1000) | LPRX_TO_CNT(1000));
        dsi_write(dsi, DSI_BTA_TO_CNT, 0xd00);
-       dsi_write(dsi, DSI_MODE_CFG, ENABLE_CMD_MODE);
 }
 
 /* Get lane byte clock cycles. */
 static u32 dw_mipi_dsi_get_hcomponent_lbcc(struct dw_mipi_dsi *dsi,
                                           u32 hcomponent)
 {
-       u32 frac, lbcc;
+       u32 lbcc;
 
        lbcc = hcomponent * dsi->lane_mbps * MSEC_PER_SEC / 8;
 
-       frac = lbcc % dsi->mode.clock;
-       lbcc = lbcc / dsi->mode.clock;
-       if (frac)
-               lbcc++;
+       if (dsi->mode.clock == 0) {
+               dev_err(dsi->dev, "dsi mode clock is 0!\n");
+               return 0;
+       }
 
-       return lbcc;
+       return DIV_ROUND_CLOSEST_ULL(lbcc, dsi->mode.clock);
 }
 
 static void dw_mipi_dsi_line_timer_config(struct dw_mipi_dsi *dsi)
@@ -866,34 +993,57 @@ static void dw_mipi_dsi_encoder_mode_set(struct drm_encoder *encoder,
        struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
 
        drm_mode_copy(&dsi->mode, adjusted_mode);
+
+       if (dsi->slave)
+               drm_mode_copy(&dsi->slave->mode, adjusted_mode);
 }
 
-static void dw_mipi_dsi_encoder_disable(struct drm_encoder *encoder)
+static void rockchip_dsi_pre_disable(struct dw_mipi_dsi *dsi)
 {
-       struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
-
-       drm_panel_disable(dsi->panel);
-
        if (clk_prepare_enable(dsi->pclk)) {
                dev_err(dsi->dev, "%s: Failed to enable pclk\n", __func__);
                return;
        }
 
-       dw_mipi_dsi_set_mode(dsi, DW_MIPI_DSI_CMD_MODE);
-       drm_panel_unprepare(dsi->panel);
-       dw_mipi_dsi_set_mode(dsi, DW_MIPI_DSI_VID_MODE);
+       dw_mipi_dsi_set_mode(dsi, DSI_COMMAND_MODE);
 
-       /*
-        * This is necessary to make sure the peripheral will be driven
-        * normally when the display is enabled again later.
-        */
-       msleep(120);
+       if (dsi->slave)
+               rockchip_dsi_pre_disable(dsi->slave);
+}
+
+static void rockchip_dsi_disable(struct dw_mipi_dsi *dsi)
+{
+       /* host */
+       dsi_write(dsi, DSI_LPCLK_CTRL, 0);
+       dsi_write(dsi, DSI_PWR_UP, RESET);
+
+       /* phy */
+       dsi_write(dsi, DSI_PHY_RSTZ, PHY_RSTZ);
+       if (dsi->dphy.phy) {
+               clk_disable_unprepare(dsi->dphy.hs_clk);
+               phy_power_off(dsi->dphy.phy);
+       }
 
-       dw_mipi_dsi_set_mode(dsi, DW_MIPI_DSI_CMD_MODE);
-       dw_mipi_dsi_disable(dsi);
-       phy_power_off(dsi->phy);
        pm_runtime_put(dsi->dev);
        clk_disable_unprepare(dsi->pclk);
+
+       if (dsi->slave)
+               rockchip_dsi_disable(dsi->slave);
+}
+
+static void dw_mipi_dsi_encoder_disable(struct drm_encoder *encoder)
+{
+       struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
+
+       if (dsi->panel)
+               drm_panel_disable(dsi->panel);
+
+       rockchip_dsi_pre_disable(dsi);
+
+       if (dsi->panel)
+               drm_panel_unprepare(dsi->panel);
+
+       rockchip_dsi_disable(dsi);
 }
 
 static bool dw_mipi_dsi_encoder_mode_fixup(struct drm_encoder *encoder,
@@ -908,14 +1058,46 @@ static void rockchip_dsi_grf_config(struct dw_mipi_dsi *dsi, int vop_id)
        const struct dw_mipi_dsi_plat_data *pdata = dsi->pdata;
        int val = 0;
 
-       if (vop_id)
-               val = pdata->dsi0_en_bit | (pdata->dsi0_en_bit << 16);
-       else
-               val = pdata->dsi0_en_bit << 16;
+       if (dsi->slave) {
+               if (vop_id)
+                       val = pdata->dsi0_en_bit |
+                             (pdata->dsi0_en_bit << 16) |
+                             pdata->dsi1_en_bit |
+                             (pdata->dsi1_en_bit << 16);
+               else
+                       val = (pdata->dsi0_en_bit << 16) |
+                             (pdata->dsi1_en_bit << 16);
+
+               regmap_write(dsi->grf_regmap, pdata->grf_switch_reg, val);
+               val = pdata->dsi1_masterslavez |
+                     (pdata->dsi1_masterslavez << 16) |
+                     (pdata->dsi1_basedir << 16);
+               regmap_write(dsi->grf_regmap, pdata->grf_dsi1_cfg_reg, val);
+
+               if (pdata->grf_dsi0_mode_reg)
+                       regmap_write(dsi->grf_regmap, pdata->grf_dsi0_mode_reg,
+                                    pdata->grf_dsi0_mode);
+               if (pdata->grf_dsi1_mode_reg1)
+                       regmap_write(dsi->grf_regmap, pdata->grf_dsi1_mode_reg1,
+                                    pdata->grf_dsi1_mode);
+               if (pdata->grf_dsi1_mode_reg2)
+                       regmap_write(dsi->grf_regmap, pdata->grf_dsi1_mode_reg2,
+                                    RK3399_GRF_DSI1_MODE2);
+               if (pdata->grf_dsi1_mode_reg1)
+                       regmap_write(dsi->grf_regmap, pdata->grf_dsi1_mode_reg1,
+                                    RK3399_GRF_DSI1_ENABLE);
+       } else {
+               if (vop_id)
+                       val = pdata->dsi0_en_bit |
+                             (pdata->dsi0_en_bit << 16);
+               else
+                       val = pdata->dsi0_en_bit << 16;
 
-       regmap_write(dsi->grf_regmap, pdata->grf_switch_reg, val);
+               regmap_write(dsi->grf_regmap, pdata->grf_switch_reg, val);
 
-       dev_dbg(dsi->dev, "vop %s output to dsi0\n", (vop_id) ? "LIT" : "BIG");
+       }
+
+       dev_info(dsi->dev, "vop %s output to dsi0\n", (vop_id) ? "LIT" : "BIG");
 }
 
 static void rockchip_dsi_pre_init(struct dw_mipi_dsi *dsi)
@@ -925,6 +1107,18 @@ static void rockchip_dsi_pre_init(struct dw_mipi_dsi *dsi)
                return;
        }
 
+       if (clk_prepare_enable(dsi->dphy.ref_clk)) {
+               dev_err(dsi->dev, "Failed to enable pllref_clk\n");
+               return;
+       }
+
+       if (dsi->dphy.phy) {
+               rockchip_dsi_set_hs_clk(dsi);
+               phy_power_on(dsi->dphy.phy);
+       } else {
+               dw_mipi_dsi_get_lane_bps(dsi);
+       }
+
        pm_runtime_get_sync(dsi->dev);
 
        if (dsi->rst) {
@@ -935,32 +1129,32 @@ static void rockchip_dsi_pre_init(struct dw_mipi_dsi *dsi)
                udelay(10);
        }
 
-       if (dsi->phy) {
-               phy_power_on(dsi->phy);
-
-               /*
-                * If using the third party PHY, we get the lane
-                * rate information from PHY.
-                */
-               dsi->lane_mbps = phy_get_bus_width(dsi->phy);
-       } else {
-               dw_mipi_dsi_get_lane_bps(dsi);
-       }
+       dev_info(dsi->dev, "final DSI-Link bandwidth: %u x %d Mbps\n",
+                dsi->lane_mbps, dsi->lanes);
 }
 
 static void rockchip_dsi_host_init(struct dw_mipi_dsi *dsi)
 {
+       const struct dw_mipi_dsi_plat_data *pdata = dsi->pdata;
+
        dw_mipi_dsi_init(dsi);
        dw_mipi_dsi_dpi_config(dsi, &dsi->mode);
        dw_mipi_dsi_packet_handler_config(dsi);
        dw_mipi_dsi_video_mode_config(dsi);
        dw_mipi_dsi_video_packet_config(dsi, &dsi->mode);
        dw_mipi_dsi_command_mode_config(dsi);
+       dw_mipi_dsi_set_mode(dsi, DSI_COMMAND_MODE);
        dw_mipi_dsi_line_timer_config(dsi);
        dw_mipi_dsi_vertical_timing_config(dsi);
        dw_mipi_dsi_dphy_timing_config(dsi);
        dw_mipi_dsi_dphy_interface_config(dsi);
        dw_mipi_dsi_clear_err(dsi);
+
+               if (pdata->grf_dsi0_mode_reg)
+                       regmap_write(dsi->grf_regmap, pdata->grf_dsi0_mode_reg,
+                                    pdata->grf_dsi0_mode);
+
+
 }
 
 static void rockchip_dsi_init(struct dw_mipi_dsi *dsi)
@@ -968,12 +1162,19 @@ static void rockchip_dsi_init(struct dw_mipi_dsi *dsi)
        rockchip_dsi_pre_init(dsi);
        rockchip_dsi_host_init(dsi);
        dw_mipi_dsi_phy_init(dsi);
+
+       if (dsi->slave)
+               rockchip_dsi_init(dsi->slave);
 }
 
 static void rockchip_dsi_enable(struct dw_mipi_dsi *dsi)
 {
-       dw_mipi_dsi_set_mode(dsi, DW_MIPI_DSI_VID_MODE);
+       dw_mipi_dsi_set_mode(dsi, DSI_VIDEO_MODE);
+       clk_disable_unprepare(dsi->dphy.ref_clk);
        clk_disable_unprepare(dsi->pclk);
+
+       if (dsi->slave)
+               rockchip_dsi_enable(dsi->slave);
 }
 
 static void dw_mipi_dsi_encoder_enable(struct drm_encoder *encoder)
@@ -983,7 +1184,6 @@ static void dw_mipi_dsi_encoder_enable(struct drm_encoder *encoder)
 
        vop_id = drm_of_encoder_active_endpoint_id(dsi->dev->of_node, encoder);
 
-       rockchip_dsi_grf_config(dsi, vop_id);
        rockchip_dsi_init(dsi);
 
        if (dsi->panel)
@@ -993,6 +1193,8 @@ static void dw_mipi_dsi_encoder_enable(struct drm_encoder *encoder)
 
        if (dsi->panel)
                drm_panel_enable(dsi->panel);
+
+       rockchip_dsi_grf_config(dsi, vop_id);
 }
 
 static int
@@ -1024,6 +1226,9 @@ dw_mipi_dsi_encoder_atomic_check(struct drm_encoder *encoder,
        if (info->num_bus_formats)
                s->bus_format = info->bus_formats[0];
 
+       if (dsi->slave)
+               s->output_flags = ROCKCHIP_OUTPUT_DSI_DUAL_CHANNEL;
+
        return 0;
 }
 
@@ -1112,6 +1317,26 @@ static struct drm_connector_funcs dw_mipi_dsi_atomic_connector_funcs = {
        .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
 };
 
+static int rockchip_dsi_dual_channel_probe(struct dw_mipi_dsi *dsi)
+{
+       struct device_node *np;
+       struct platform_device *secondary;
+
+       np = of_parse_phandle(dsi->dev->of_node, "rockchip,dual-channel", 0);
+       if (np) {
+               secondary = of_find_device_by_node(np);
+               dsi->slave = platform_get_drvdata(secondary);
+               of_node_put(np);
+
+               if (!dsi->slave)
+                       return -EPROBE_DEFER;
+
+               dsi->slave->master = dsi;
+       }
+
+       return 0;
+}
+
 static int dw_mipi_dsi_register(struct drm_device *drm,
                                      struct dw_mipi_dsi *dsi)
 {
@@ -1156,23 +1381,13 @@ static int dw_mipi_dsi_register(struct drm_device *drm,
        return 0;
 }
 
-static int rockchip_mipi_parse_dt(struct dw_mipi_dsi *dsi)
-{
-       struct device_node *np = dsi->dev->of_node;
-
-       dsi->grf_regmap = syscon_regmap_lookup_by_phandle(np, "rockchip,grf");
-       if (IS_ERR(dsi->grf_regmap)) {
-               dev_err(dsi->dev, "Unable to get rockchip,grf\n");
-               return PTR_ERR(dsi->grf_regmap);
-       }
-
-       return 0;
-}
-
 static struct dw_mipi_dsi_plat_data rk3288_mipi_dsi_drv_data = {
        .dsi0_en_bit = RK3288_DSI0_SEL_VOP_LIT,
        .dsi1_en_bit = RK3288_DSI1_SEL_VOP_LIT,
        .grf_switch_reg = RK3288_GRF_SOC_CON6,
+       .dsi1_basedir = RK3288_TXRX_BASEDIR,
+       .dsi1_masterslavez = RK3288_TXRX_MASTERSLAVEZ,
+       .grf_dsi1_cfg_reg = RK3288_GRF_SOC_CON14,
        .max_data_lanes = 4,
        .max_bit_rate_per_lane = 1500000000,
        .has_vop_sel = true,
@@ -1195,8 +1410,14 @@ static struct dw_mipi_dsi_plat_data rk3399_mipi_dsi_drv_data = {
        .dsi0_en_bit = RK3399_DSI0_SEL_VOP_LIT,
        .dsi1_en_bit = RK3399_DSI1_SEL_VOP_LIT,
        .grf_switch_reg = RK3399_GRF_SOC_CON19,
-       .grf_dsi0_mode = RK3399_GRF_DSI_MODE,
+       .grf_dsi0_mode = RK3399_GRF_DSI0_MODE,
        .grf_dsi0_mode_reg = RK3399_GRF_SOC_CON22,
+       .grf_dsi1_mode = RK3399_GRF_DSI1_MODE1,
+       .grf_dsi1_mode_reg1 = RK3399_GRF_SOC_CON23,
+       .dsi1_basedir = RK3399_TXRX_BASEDIR,
+       .dsi1_masterslavez = RK3399_TXRX_MASTERSLAVEZ,
+       .dsi1_enableclk = RK3399_TXRX_ENABLECLK,
+       .grf_dsi1_mode_reg2 = RK3399_GRF_SOC_CON24,
        .max_data_lanes = 4,
        .max_bit_rate_per_lane = 1500000000,
        .has_vop_sel = true,
@@ -1223,33 +1444,80 @@ MODULE_DEVICE_TABLE(of, dw_mipi_dsi_dt_ids);
 static int dw_mipi_dsi_bind(struct device *dev, struct device *master,
                             void *data)
 {
-       struct platform_device *pdev = to_platform_device(dev);
        struct drm_device *drm = data;
        struct dw_mipi_dsi *dsi = dev_get_drvdata(dev);
-       struct resource *res;
        int ret;
 
+       ret = rockchip_dsi_dual_channel_probe(dsi);
+       if (ret)
+               return ret;
+
+       if (dsi->master)
+               return 0;
+
        if (!dsi->panel)
                return -EPROBE_DEFER;
 
-       ret = rockchip_mipi_parse_dt(dsi);
-       if (ret)
+       ret = dw_mipi_dsi_register(drm, dsi);
+       if (ret) {
+               dev_err(dev, "Failed to register mipi_dsi: %d\n", ret);
                return ret;
+       }
 
-       dsi->phy = devm_phy_optional_get(dev, "mipi_dphy");
-       if (IS_ERR(dsi->phy)) {
-               ret = PTR_ERR(dsi->phy);
-               dev_err(dev, "failed to get mipi dphy: %d\n", ret);
-               return ret;
+       dev_set_drvdata(dev, dsi);
+
+       pm_runtime_enable(dev);
+       if (dsi->slave)
+               pm_runtime_enable(dsi->slave->dev);
+
+       return ret;
+}
+
+static void dw_mipi_dsi_unbind(struct device *dev, struct device *master,
+       void *data)
+{
+       struct dw_mipi_dsi *dsi = dev_get_drvdata(dev);
+
+       pm_runtime_disable(dev);
+       if (dsi->slave)
+               pm_runtime_disable(dsi->slave->dev);
+}
+
+static const struct component_ops dw_mipi_dsi_ops = {
+       .bind   = dw_mipi_dsi_bind,
+       .unbind = dw_mipi_dsi_unbind,
+};
+
+static int rockchip_dsi_get_reset_handle(struct dw_mipi_dsi *dsi)
+{
+       struct device *dev = dsi->dev;
+
+       dsi->rst = devm_reset_control_get_optional(dev, "apb");
+       if (IS_ERR(dsi->rst)) {
+               dev_info(dev, "no reset control specified\n");
+               dsi->rst = NULL;
        }
 
-       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       if (!res)
-               return -ENODEV;
+       return 0;
+}
 
-       dsi->base = devm_ioremap_resource(dev, res);
-       if (IS_ERR(dsi->base))
-               return PTR_ERR(dsi->base);
+static int rockchip_dsi_grf_regmap(struct dw_mipi_dsi *dsi)
+{
+       struct device_node *np = dsi->dev->of_node;
+
+       dsi->grf_regmap = syscon_regmap_lookup_by_phandle(np, "rockchip,grf");
+       if (IS_ERR(dsi->grf_regmap)) {
+               dev_err(dsi->dev, "Unable to get rockchip,grf\n");
+               return PTR_ERR(dsi->grf_regmap);
+       }
+
+       return 0;
+}
+
+static int rockchip_dsi_clk_get(struct dw_mipi_dsi *dsi)
+{
+       struct device *dev = dsi->dev;
+       int ret;
 
        dsi->pclk = devm_clk_get(dev, "pclk");
        if (IS_ERR(dsi->pclk)) {
@@ -1258,62 +1526,70 @@ static int dw_mipi_dsi_bind(struct device *dev, struct device *master,
                return ret;
        }
 
-       /* optional */
-       dsi->pllref_clk = devm_clk_get(dev, "ref");
-       if (IS_ERR(dsi->pllref_clk)) {
-               dev_info(dev, "No PHY reference clock specified\n");
-               dsi->pllref_clk = NULL;
-       }
+       return 0;
+}
 
-       /* optional */
-       dsi->phy_cfg_clk = devm_clk_get(dev, "phy_cfg");
-       if (IS_ERR(dsi->phy_cfg_clk)) {
-               dev_info(dev, "No PHY APB clock specified\n");
-               dsi->phy_cfg_clk = NULL;
-       }
+static int rockchip_dsi_dphy_parse(struct dw_mipi_dsi *dsi)
+{
+       struct device *dev = dsi->dev;
+       int ret;
 
-       ret = clk_prepare_enable(dsi->pllref_clk);
-       if (ret) {
-               dev_err(dev, "%s: Failed to enable pllref_clk\n", __func__);
+       dsi->dphy.phy = devm_phy_optional_get(dev, "mipi_dphy");
+       if (IS_ERR(dsi->dphy.phy)) {
+               ret = PTR_ERR(dsi->dphy.phy);
+               dev_err(dev, "failed to get mipi dphy: %d\n", ret);
                return ret;
        }
 
-       dsi->rst = devm_reset_control_get_optional(dev, "apb");
-       if (IS_ERR(dsi->rst)) {
-               dev_info(dev, "no reset control specified\n");
-               dsi->rst = NULL;
-       }
+       if (dsi->dphy.phy) {
+               dev_dbg(dev, "Use Non-SNPS PHY\n");
 
-       ret = dw_mipi_dsi_register(drm, dsi);
-       if (ret) {
-               dev_err(dev, "Failed to register mipi_dsi: %d\n", ret);
-               goto err_pllref;
-       }
+               dsi->dphy.hs_clk = devm_clk_get(dev, "hs_clk");
+               if (IS_ERR(dsi->dphy.hs_clk)) {
+                       dev_err(dev, "failed to get PHY high-speed clock\n");
+                       return PTR_ERR(dsi->dphy.hs_clk);
+               }
+       } else {
+               dev_dbg(dev, "Use SNPS PHY\n");
 
-       dev_set_drvdata(dev, dsi);
+               dsi->dphy.ref_clk = devm_clk_get(dev, "ref");
+               if (IS_ERR(dsi->dphy.ref_clk)) {
+                       dev_err(dev, "failed to get PHY reference clock\n");
+                       return PTR_ERR(dsi->dphy.ref_clk);
+               }
 
-       pm_runtime_enable(dev);
+               /* Check if cfg_clk provided */
+               dsi->dphy.cfg_clk = devm_clk_get(dev, "phy_cfg");
+               if (IS_ERR(dsi->dphy.cfg_clk)) {
+                       if (PTR_ERR(dsi->dphy.cfg_clk) != -ENOENT) {
+                               dev_err(dev, "failed to get PHY config clk\n");
+                               return PTR_ERR(dsi->dphy.cfg_clk);
+                       }
 
-       return 0;
+                       /* Otherwise mark the cfg_clk pointer to NULL */
+                       dsi->dphy.cfg_clk = NULL;
+               }
+       }
 
-err_pllref:
-       clk_disable_unprepare(dsi->pllref_clk);
-       return ret;
+       return 0;
 }
 
-static void dw_mipi_dsi_unbind(struct device *dev, struct device *master,
-       void *data)
+static int rockchip_dsi_ioremap_resource(struct platform_device *pdev,
+                                        struct dw_mipi_dsi *dsi)
 {
-       struct dw_mipi_dsi *dsi = dev_get_drvdata(dev);
+       struct device *dev = &pdev->dev;
+       struct resource *res;
 
-       pm_runtime_disable(dev);
-       clk_disable_unprepare(dsi->pllref_clk);
-}
+       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+       if (!res)
+               return -ENODEV;
 
-static const struct component_ops dw_mipi_dsi_ops = {
-       .bind   = dw_mipi_dsi_bind,
-       .unbind = dw_mipi_dsi_unbind,
-};
+       dsi->base = devm_ioremap_resource(dev, res);
+       if (IS_ERR(dsi->base))
+               return PTR_ERR(dsi->base);
+
+       return 0;
+}
 
 static int dw_mipi_dsi_probe(struct platform_device *pdev)
 {
@@ -1322,14 +1598,28 @@ static int dw_mipi_dsi_probe(struct platform_device *pdev)
                        of_match_device(dw_mipi_dsi_dt_ids, dev);
        const struct dw_mipi_dsi_plat_data *pdata = of_id->data;
        struct dw_mipi_dsi *dsi;
+       struct device_node *np = dev->of_node;
        int ret;
+       int dsi_id;
 
        dsi = devm_kzalloc(&pdev->dev, sizeof(*dsi), GFP_KERNEL);
        if (!dsi)
                return -ENOMEM;
 
+       dsi_id = of_alias_get_id(np, "dsi");
+       if (dsi_id < 0)
+               dsi_id = 0;
+
+       dsi->id = dsi_id;
        dsi->dev = dev;
        dsi->pdata = pdata;
+
+       rockchip_dsi_ioremap_resource(pdev, dsi);
+       rockchip_dsi_clk_get(dsi);
+       rockchip_dsi_dphy_parse(dsi);
+       rockchip_dsi_grf_regmap(dsi);
+       rockchip_dsi_get_reset_handle(dsi);
+
        dsi->dsi_host.ops = &dw_mipi_dsi_host_ops;
        dsi->dsi_host.dev = &pdev->dev;