El Tue, Nov 28, 2017 at 07:20:05PM +0800 Nickey Yang ha dit:

> Add the ROCKCHIP DSI controller driver that uses the Synopsys DesignWare
> MIPI DSI host controller bridge.
> 
> v2:
>    add err_pllref, remove unnecessary encoder.enable & disable
>    correct spelling mistakes
> v3:
>    call dw_mipi_dsi_unbind() in dw_mipi_dsi_rockchip_unbind()
>    fix typo, use of_device_get_match_data(),
>    change some ‘bind()’ logic into 'probe()'
>    add 'dev_set_drvdata()'
> 
> Signed-off-by: Nickey Yang <nickey.y...@rock-chips.com>
> ---
>  drivers/gpu/drm/rockchip/Kconfig                |    2 +-
>  drivers/gpu/drm/rockchip/Makefile               |    2 +-
>  drivers/gpu/drm/rockchip/dw-mipi-dsi.c          | 1349 
> -----------------------
>  drivers/gpu/drm/rockchip/dw-mipi-dsi_rockchip.c |  764 +++++++++++++
>  drivers/gpu/drm/rockchip/rockchip_drm_drv.c     |    2 +-
>  drivers/gpu/drm/rockchip/rockchip_drm_drv.h     |    2 +-
>  6 files changed, 768 insertions(+), 1353 deletions(-)
>  delete mode 100644 drivers/gpu/drm/rockchip/dw-mipi-dsi.c
>  create mode 100644 drivers/gpu/drm/rockchip/dw-mipi-dsi_rockchip.c
> 
> diff --git a/drivers/gpu/drm/rockchip/Kconfig 
> b/drivers/gpu/drm/rockchip/Kconfig
> index 0ccc762..9eb4795 100644
> --- a/drivers/gpu/drm/rockchip/Kconfig
> +++ b/drivers/gpu/drm/rockchip/Kconfig
> @@ -7,7 +7,7 @@ config DRM_ROCKCHIP
>       select VIDEOMODE_HELPERS
>       select DRM_ANALOGIX_DP if ROCKCHIP_ANALOGIX_DP
>       select DRM_DW_HDMI if ROCKCHIP_DW_HDMI
> -     select DRM_MIPI_DSI if ROCKCHIP_DW_MIPI_DSI
> +     select DRM_DW_MIPI_DSI if ROCKCHIP_DW_MIPI_DSI
>       select SND_SOC_HDMI_CODEC if ROCKCHIP_CDN_DP && SND_SOC
>       help
>         Choose this option if you have a Rockchip soc chipset.
> diff --git a/drivers/gpu/drm/rockchip/Makefile 
> b/drivers/gpu/drm/rockchip/Makefile
> index a314e21..c05fe47 100644
> --- a/drivers/gpu/drm/rockchip/Makefile
> +++ b/drivers/gpu/drm/rockchip/Makefile
> @@ -11,7 +11,7 @@ rockchipdrm-$(CONFIG_DRM_FBDEV_EMULATION) += 
> rockchip_drm_fbdev.o
>  rockchipdrm-$(CONFIG_ROCKCHIP_ANALOGIX_DP) += analogix_dp-rockchip.o
>  rockchipdrm-$(CONFIG_ROCKCHIP_CDN_DP) += cdn-dp-core.o cdn-dp-reg.o
>  rockchipdrm-$(CONFIG_ROCKCHIP_DW_HDMI) += dw_hdmi-rockchip.o
> -rockchipdrm-$(CONFIG_ROCKCHIP_DW_MIPI_DSI) += dw-mipi-dsi.o
> +rockchipdrm-$(CONFIG_ROCKCHIP_DW_MIPI_DSI) += dw-mipi-dsi_rockchip.o
>  rockchipdrm-$(CONFIG_ROCKCHIP_INNO_HDMI) += inno_hdmi.o
>  rockchipdrm-$(CONFIG_ROCKCHIP_LVDS) += rockchip_lvds.o
>  
> diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c 
> b/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
> deleted file mode 100644
> index b15755b..0000000
> --- a/drivers/gpu/drm/rockchip/dw-mipi-dsi.c
> +++ /dev/null
> @@ -1,1349 +0,0 @@
> -/*
> - * Copyright (c) 2016, Fuzhou Rockchip Electronics Co., Ltd
> - *
> - * This program is free software; you can redistribute it and/or modify
> - * it under the terms of the GNU General Public License as published by
> - * the Free Software Foundation; either version 2 of the License, or
> - * (at your option) any later version.
> - */
> -#include <linux/clk.h>
> -#include <linux/component.h>
> -#include <linux/iopoll.h>
> -#include <linux/math64.h>
> -#include <linux/module.h>
> -#include <linux/of_device.h>
> -#include <linux/pm_runtime.h>
> -#include <linux/regmap.h>
> -#include <linux/reset.h>
> -#include <linux/mfd/syscon.h>
> -#include <drm/drm_atomic_helper.h>
> -#include <drm/drm_crtc.h>
> -#include <drm/drm_crtc_helper.h>
> -#include <drm/drm_mipi_dsi.h>
> -#include <drm/drm_of.h>
> -#include <drm/drm_panel.h>
> -#include <drm/drmP.h>
> -#include <video/mipi_display.h>
> -
> -#include "rockchip_drm_drv.h"
> -#include "rockchip_drm_vop.h"
> -
> -#define DRIVER_NAME    "dw-mipi-dsi"
> -
> -#define RK3288_GRF_SOC_CON6          0x025c
> -#define RK3288_DSI0_SEL_VOP_LIT              BIT(6)
> -#define RK3288_DSI1_SEL_VOP_LIT              BIT(9)
> -
> -#define RK3399_GRF_SOC_CON20         0x6250
> -#define RK3399_DSI0_SEL_VOP_LIT              BIT(0)
> -#define RK3399_DSI1_SEL_VOP_LIT              BIT(4)
> -
> -/* disable turnrequest, turndisable, forcetxstopmode, forcerxmode */
> -#define RK3399_GRF_SOC_CON22         0x6258
> -#define RK3399_GRF_DSI_MODE          0xffff0000
> -
> -#define DSI_VERSION                  0x00
> -#define DSI_PWR_UP                   0x04
> -#define RESET                                0
> -#define POWERUP                              BIT(0)
> -
> -#define DSI_CLKMGR_CFG                       0x08
> -#define TO_CLK_DIVIDSION(div)                (((div) & 0xff) << 8)
> -#define TX_ESC_CLK_DIVIDSION(div)    (((div) & 0xff) << 0)
> -
> -#define DSI_DPI_VCID                 0x0c
> -#define DPI_VID(vid)                 (((vid) & 0x3) << 0)
> -
> -#define DSI_DPI_COLOR_CODING         0x10
> -#define EN18_LOOSELY                 BIT(8)
> -#define DPI_COLOR_CODING_16BIT_1     0x0
> -#define DPI_COLOR_CODING_16BIT_2     0x1
> -#define DPI_COLOR_CODING_16BIT_3     0x2
> -#define DPI_COLOR_CODING_18BIT_1     0x3
> -#define DPI_COLOR_CODING_18BIT_2     0x4
> -#define DPI_COLOR_CODING_24BIT               0x5
> -
> -#define DSI_DPI_CFG_POL                      0x14
> -#define COLORM_ACTIVE_LOW            BIT(4)
> -#define SHUTD_ACTIVE_LOW             BIT(3)
> -#define HSYNC_ACTIVE_LOW             BIT(2)
> -#define VSYNC_ACTIVE_LOW             BIT(1)
> -#define DATAEN_ACTIVE_LOW            BIT(0)
> -
> -#define DSI_DPI_LP_CMD_TIM           0x18
> -#define OUTVACT_LPCMD_TIME(p)                (((p) & 0xff) << 16)
> -#define INVACT_LPCMD_TIME(p)         ((p) & 0xff)
> -
> -#define DSI_DBI_CFG                  0x20
> -#define DSI_DBI_CMDSIZE                      0x28
> -
> -#define DSI_PCKHDL_CFG                       0x2c
> -#define EN_CRC_RX                    BIT(4)
> -#define EN_ECC_RX                    BIT(3)
> -#define EN_BTA                               BIT(2)
> -#define EN_EOTP_RX                   BIT(1)
> -#define EN_EOTP_TX                   BIT(0)
> -
> -#define DSI_MODE_CFG                 0x34
> -#define ENABLE_VIDEO_MODE            0
> -#define ENABLE_CMD_MODE                      BIT(0)
> -
> -#define DSI_VID_MODE_CFG             0x38
> -#define FRAME_BTA_ACK                        BIT(14)
> -#define ENABLE_LOW_POWER             (0x3f << 8)
> -#define ENABLE_LOW_POWER_MASK                (0x3f << 8)
> -#define VID_MODE_TYPE_NON_BURST_SYNC_PULSES  0x0
> -#define VID_MODE_TYPE_NON_BURST_SYNC_EVENTS  0x1
> -#define VID_MODE_TYPE_BURST                  0x2
> -#define VID_MODE_TYPE_MASK                   0x3
> -
> -#define DSI_VID_PKT_SIZE             0x3c
> -#define VID_PKT_SIZE(p)                      (((p) & 0x3fff) << 0)
> -#define VID_PKT_MAX_SIZE             0x3fff
> -
> -#define DSI_VID_HSA_TIME             0x48
> -#define DSI_VID_HBP_TIME             0x4c
> -#define DSI_VID_HLINE_TIME           0x50
> -#define DSI_VID_VSA_LINES            0x54
> -#define DSI_VID_VBP_LINES            0x58
> -#define DSI_VID_VFP_LINES            0x5c
> -#define DSI_VID_VACTIVE_LINES                0x60
> -#define DSI_CMD_MODE_CFG             0x68
> -#define MAX_RD_PKT_SIZE_LP           BIT(24)
> -#define DCS_LW_TX_LP                 BIT(19)
> -#define DCS_SR_0P_TX_LP                      BIT(18)
> -#define DCS_SW_1P_TX_LP                      BIT(17)
> -#define DCS_SW_0P_TX_LP                      BIT(16)
> -#define GEN_LW_TX_LP                 BIT(14)
> -#define GEN_SR_2P_TX_LP                      BIT(13)
> -#define GEN_SR_1P_TX_LP                      BIT(12)
> -#define GEN_SR_0P_TX_LP                      BIT(11)
> -#define GEN_SW_2P_TX_LP                      BIT(10)
> -#define GEN_SW_1P_TX_LP                      BIT(9)
> -#define GEN_SW_0P_TX_LP                      BIT(8)
> -#define EN_ACK_RQST                  BIT(1)
> -#define EN_TEAR_FX                   BIT(0)
> -
> -#define CMD_MODE_ALL_LP                      (MAX_RD_PKT_SIZE_LP | \
> -                                      DCS_LW_TX_LP | \
> -                                      DCS_SR_0P_TX_LP | \
> -                                      DCS_SW_1P_TX_LP | \
> -                                      DCS_SW_0P_TX_LP | \
> -                                      GEN_LW_TX_LP | \
> -                                      GEN_SR_2P_TX_LP | \
> -                                      GEN_SR_1P_TX_LP | \
> -                                      GEN_SR_0P_TX_LP | \
> -                                      GEN_SW_2P_TX_LP | \
> -                                      GEN_SW_1P_TX_LP | \
> -                                      GEN_SW_0P_TX_LP)
> -
> -#define DSI_GEN_HDR                  0x6c
> -#define GEN_HDATA(data)                      (((data) & 0xffff) << 8)
> -#define GEN_HDATA_MASK                       (0xffff << 8)
> -#define GEN_HTYPE(type)                      (((type) & 0xff) << 0)
> -#define GEN_HTYPE_MASK                       0xff
> -
> -#define DSI_GEN_PLD_DATA             0x70
> -
> -#define DSI_CMD_PKT_STATUS           0x74
> -#define GEN_CMD_EMPTY                        BIT(0)
> -#define GEN_CMD_FULL                 BIT(1)
> -#define GEN_PLD_W_EMPTY                      BIT(2)
> -#define GEN_PLD_W_FULL                       BIT(3)
> -#define GEN_PLD_R_EMPTY                      BIT(4)
> -#define GEN_PLD_R_FULL                       BIT(5)
> -#define GEN_RD_CMD_BUSY                      BIT(6)
> -
> -#define DSI_TO_CNT_CFG                       0x78
> -#define HSTX_TO_CNT(p)                       (((p) & 0xffff) << 16)
> -#define LPRX_TO_CNT(p)                       ((p) & 0xffff)
> -
> -#define DSI_BTA_TO_CNT                       0x8c
> -#define DSI_LPCLK_CTRL                       0x94
> -#define AUTO_CLKLANE_CTRL            BIT(1)
> -#define PHY_TXREQUESTCLKHS           BIT(0)
> -
> -#define DSI_PHY_TMR_LPCLK_CFG                0x98
> -#define PHY_CLKHS2LP_TIME(lbcc)              (((lbcc) & 0x3ff) << 16)
> -#define PHY_CLKLP2HS_TIME(lbcc)              ((lbcc) & 0x3ff)
> -
> -#define DSI_PHY_TMR_CFG                      0x9c
> -#define PHY_HS2LP_TIME(lbcc)         (((lbcc) & 0xff) << 24)
> -#define PHY_LP2HS_TIME(lbcc)         (((lbcc) & 0xff) << 16)
> -#define MAX_RD_TIME(lbcc)            ((lbcc) & 0x7fff)
> -
> -#define DSI_PHY_RSTZ                 0xa0
> -#define PHY_DISFORCEPLL                      0
> -#define PHY_ENFORCEPLL                       BIT(3)
> -#define PHY_DISABLECLK                       0
> -#define PHY_ENABLECLK                        BIT(2)
> -#define PHY_RSTZ                     0
> -#define PHY_UNRSTZ                   BIT(1)
> -#define PHY_SHUTDOWNZ                        0
> -#define PHY_UNSHUTDOWNZ                      BIT(0)
> -
> -#define DSI_PHY_IF_CFG                       0xa4
> -#define N_LANES(n)                   ((((n) - 1) & 0x3) << 0)
> -#define PHY_STOP_WAIT_TIME(cycle)    (((cycle) & 0xff) << 8)
> -
> -#define DSI_PHY_STATUS                       0xb0
> -#define LOCK                         BIT(0)
> -#define STOP_STATE_CLK_LANE          BIT(2)
> -
> -#define DSI_PHY_TST_CTRL0            0xb4
> -#define PHY_TESTCLK                  BIT(1)
> -#define PHY_UNTESTCLK                        0
> -#define PHY_TESTCLR                  BIT(0)
> -#define PHY_UNTESTCLR                        0
> -
> -#define DSI_PHY_TST_CTRL1            0xb8
> -#define PHY_TESTEN                   BIT(16)
> -#define PHY_UNTESTEN                 0
> -#define PHY_TESTDOUT(n)                      (((n) & 0xff) << 8)
> -#define PHY_TESTDIN(n)                       (((n) & 0xff) << 0)
> -
> -#define DSI_INT_ST0                  0xbc
> -#define DSI_INT_ST1                  0xc0
> -#define DSI_INT_MSK0                 0xc4
> -#define DSI_INT_MSK1                 0xc8
> -
> -#define PHY_STATUS_TIMEOUT_US                10000
> -#define CMD_PKT_STATUS_TIMEOUT_US    20000
> -
> -#define BYPASS_VCO_RANGE     BIT(7)
> -#define VCO_RANGE_CON_SEL(val)       (((val) & 0x7) << 3)
> -#define VCO_IN_CAP_CON_DEFAULT       (0x0 << 1)
> -#define VCO_IN_CAP_CON_LOW   (0x1 << 1)
> -#define VCO_IN_CAP_CON_HIGH  (0x2 << 1)
> -#define REF_BIAS_CUR_SEL     BIT(0)
> -
> -#define CP_CURRENT_3MA               BIT(3)
> -#define CP_PROGRAM_EN                BIT(7)
> -#define LPF_PROGRAM_EN               BIT(6)
> -#define LPF_RESISTORS_20_KOHM        0
> -
> -#define HSFREQRANGE_SEL(val) (((val) & 0x3f) << 1)
> -
> -#define INPUT_DIVIDER(val)   (((val) - 1) & 0x7f)
> -#define LOW_PROGRAM_EN               0
> -#define HIGH_PROGRAM_EN              BIT(7)
> -#define LOOP_DIV_LOW_SEL(val)        (((val) - 1) & 0x1f)
> -#define LOOP_DIV_HIGH_SEL(val)       ((((val) - 1) >> 5) & 0x1f)
> -#define PLL_LOOP_DIV_EN              BIT(5)
> -#define PLL_INPUT_DIV_EN     BIT(4)
> -
> -#define POWER_CONTROL                BIT(6)
> -#define INTERNAL_REG_CURRENT BIT(3)
> -#define BIAS_BLOCK_ON                BIT(2)
> -#define BANDGAP_ON           BIT(0)
> -
> -#define TER_RESISTOR_HIGH    BIT(7)
> -#define      TER_RESISTOR_LOW        0
> -#define LEVEL_SHIFTERS_ON    BIT(6)
> -#define TER_CAL_DONE         BIT(5)
> -#define SETRD_MAX            (0x7 << 2)
> -#define POWER_MANAGE         BIT(1)
> -#define TER_RESISTORS_ON     BIT(0)
> -
> -#define BIASEXTR_SEL(val)    ((val) & 0x7)
> -#define BANDGAP_SEL(val)     ((val) & 0x7)
> -#define TLP_PROGRAM_EN               BIT(7)
> -#define THS_PRE_PROGRAM_EN   BIT(7)
> -#define THS_ZERO_PROGRAM_EN  BIT(6)
> -
> -#define DW_MIPI_NEEDS_PHY_CFG_CLK    BIT(0)
> -#define DW_MIPI_NEEDS_GRF_CLK                BIT(1)
> -
> -enum {
> -     BANDGAP_97_07,
> -     BANDGAP_98_05,
> -     BANDGAP_99_02,
> -     BANDGAP_100_00,
> -     BANDGAP_93_17,
> -     BANDGAP_94_15,
> -     BANDGAP_95_12,
> -     BANDGAP_96_10,
> -};
> -
> -enum {
> -     BIASEXTR_87_1,
> -     BIASEXTR_91_5,
> -     BIASEXTR_95_9,
> -     BIASEXTR_100,
> -     BIASEXTR_105_94,
> -     BIASEXTR_111_88,
> -     BIASEXTR_118_8,
> -     BIASEXTR_127_7,
> -};
> -
> -struct dw_mipi_dsi_plat_data {
> -     u32 dsi0_en_bit;
> -     u32 dsi1_en_bit;
> -     u32 grf_switch_reg;
> -     u32 grf_dsi0_mode;
> -     u32 grf_dsi0_mode_reg;
> -     unsigned int flags;
> -     unsigned int max_data_lanes;
> -};
> -
> -struct dw_mipi_dsi {
> -     struct drm_encoder encoder;
> -     struct drm_connector connector;
> -     struct mipi_dsi_host dsi_host;
> -     struct drm_panel *panel;
> -     struct device *dev;
> -     struct regmap *grf_regmap;
> -     void __iomem *base;
> -
> -     struct clk *grf_clk;
> -     struct clk *pllref_clk;
> -     struct clk *pclk;
> -     struct clk *phy_cfg_clk;
> -
> -     int dpms_mode;
> -     unsigned int lane_mbps; /* per lane */
> -     u32 channel;
> -     u32 lanes;
> -     u32 format;
> -     u16 input_div;
> -     u16 feedback_div;
> -     unsigned long mode_flags;
> -
> -     const struct dw_mipi_dsi_plat_data *pdata;
> -};
> -
> -enum dw_mipi_dsi_mode {
> -     DW_MIPI_DSI_CMD_MODE,
> -     DW_MIPI_DSI_VID_MODE,
> -};
> -
> -struct dphy_pll_testdin_map {
> -     unsigned int max_mbps;
> -     u8 testdin;
> -};
> -
> -/* The table is based on 27MHz DPHY pll reference clock. */
> -static const struct dphy_pll_testdin_map dptdin_map[] = {
> -     {  90, 0x00}, { 100, 0x10}, { 110, 0x20}, { 130, 0x01},
> -     { 140, 0x11}, { 150, 0x21}, { 170, 0x02}, { 180, 0x12},
> -     { 200, 0x22}, { 220, 0x03}, { 240, 0x13}, { 250, 0x23},
> -     { 270, 0x04}, { 300, 0x14}, { 330, 0x05}, { 360, 0x15},
> -     { 400, 0x25}, { 450, 0x06}, { 500, 0x16}, { 550, 0x07},
> -     { 600, 0x17}, { 650, 0x08}, { 700, 0x18}, { 750, 0x09},
> -     { 800, 0x19}, { 850, 0x29}, { 900, 0x39}, { 950, 0x0a},
> -     {1000, 0x1a}, {1050, 0x2a}, {1100, 0x3a}, {1150, 0x0b},
> -     {1200, 0x1b}, {1250, 0x2b}, {1300, 0x3b}, {1350, 0x0c},
> -     {1400, 0x1c}, {1450, 0x2c}, {1500, 0x3c}
> -};
> -
> -static int max_mbps_to_testdin(unsigned int max_mbps)
> -{
> -     int i;
> -
> -     for (i = 0; i < ARRAY_SIZE(dptdin_map); i++)
> -             if (dptdin_map[i].max_mbps > max_mbps)
> -                     return dptdin_map[i].testdin;
> -
> -     return -EINVAL;
> -}
> -
> -/*
> - * The controller should generate 2 frames before
> - * preparing the peripheral.
> - */
> -static void dw_mipi_dsi_wait_for_two_frames(struct drm_display_mode *mode)
> -{
> -     int refresh, two_frames;
> -
> -     refresh = drm_mode_vrefresh(mode);
> -     two_frames = DIV_ROUND_UP(MSEC_PER_SEC, refresh) * 2;
> -     msleep(two_frames);
> -}
> -
> -static inline struct dw_mipi_dsi *host_to_dsi(struct mipi_dsi_host *host)
> -{
> -     return container_of(host, struct dw_mipi_dsi, dsi_host);
> -}
> -
> -static inline struct dw_mipi_dsi *con_to_dsi(struct drm_connector *con)
> -{
> -     return container_of(con, struct dw_mipi_dsi, connector);
> -}
> -
> -static inline struct dw_mipi_dsi *encoder_to_dsi(struct drm_encoder *encoder)
> -{
> -     return container_of(encoder, struct dw_mipi_dsi, encoder);
> -}
> -
> -static inline void dsi_write(struct dw_mipi_dsi *dsi, u32 reg, u32 val)
> -{
> -     writel(val, dsi->base + reg);
> -}
> -
> -static inline u32 dsi_read(struct dw_mipi_dsi *dsi, u32 reg)
> -{
> -     return readl(dsi->base + reg);
> -}
> -
> -static void dw_mipi_dsi_phy_write(struct dw_mipi_dsi *dsi, u8 test_code,
> -                               u8 test_data)
> -{
> -     /*
> -      * With the falling edge on TESTCLK, the TESTDIN[7:0] signal content
> -      * is latched internally as the current test code. Test data is
> -      * programmed internally by rising edge on TESTCLK.
> -      */
> -     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_TESTCLK | PHY_UNTESTCLR);
> -
> -     dsi_write(dsi, DSI_PHY_TST_CTRL1, PHY_TESTEN | PHY_TESTDOUT(0) |
> -                                       PHY_TESTDIN(test_code));
> -
> -     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_UNTESTCLK | PHY_UNTESTCLR);
> -
> -     dsi_write(dsi, DSI_PHY_TST_CTRL1, PHY_UNTESTEN | PHY_TESTDOUT(0) |
> -                                       PHY_TESTDIN(test_data));
> -
> -     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_TESTCLK | PHY_UNTESTCLR);
> -}
> -
> -/**
> - * ns2bc - Nanoseconds to byte clock cycles
> - */
> -static inline unsigned int ns2bc(struct dw_mipi_dsi *dsi, int ns)
> -{
> -     return DIV_ROUND_UP(ns * dsi->lane_mbps / 8, 1000);
> -}
> -
> -/**
> - * ns2ui - Nanoseconds to UI time periods
> - */
> -static inline unsigned int ns2ui(struct dw_mipi_dsi *dsi, int ns)
> -{
> -     return DIV_ROUND_UP(ns * dsi->lane_mbps, 1000);
> -}
> -
> -static int dw_mipi_dsi_phy_init(struct dw_mipi_dsi *dsi)
> -{
> -     int ret, testdin, vco, val;
> -
> -     vco = (dsi->lane_mbps < 200) ? 0 : (dsi->lane_mbps + 100) / 200;
> -
> -     testdin = max_mbps_to_testdin(dsi->lane_mbps);
> -     if (testdin < 0) {
> -             DRM_DEV_ERROR(dsi->dev,
> -                           "failed to get testdin for %dmbps lane clock\n",
> -                           dsi->lane_mbps);
> -             return testdin;
> -     }
> -
> -     /* Start by clearing PHY state */
> -     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_UNTESTCLR);
> -     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_TESTCLR);
> -     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_UNTESTCLR);
> -
> -     ret = clk_prepare_enable(dsi->phy_cfg_clk);
> -     if (ret) {
> -             DRM_DEV_ERROR(dsi->dev, "Failed to enable phy_cfg_clk\n");
> -             return ret;
> -     }
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x10, BYPASS_VCO_RANGE |
> -                                      VCO_RANGE_CON_SEL(vco) |
> -                                      VCO_IN_CAP_CON_LOW |
> -                                      REF_BIAS_CUR_SEL);
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x11, CP_CURRENT_3MA);
> -     dw_mipi_dsi_phy_write(dsi, 0x12, CP_PROGRAM_EN | LPF_PROGRAM_EN |
> -                                      LPF_RESISTORS_20_KOHM);
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x44, HSFREQRANGE_SEL(testdin));
> -
> -     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, 0x19, PLL_LOOP_DIV_EN | PLL_INPUT_DIV_EN);
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x22, LOW_PROGRAM_EN |
> -                                      BIASEXTR_SEL(BIASEXTR_127_7));
> -     dw_mipi_dsi_phy_write(dsi, 0x22, HIGH_PROGRAM_EN |
> -                                      BANDGAP_SEL(BANDGAP_96_10));
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x20, POWER_CONTROL | INTERNAL_REG_CURRENT |
> -                                      BIAS_BLOCK_ON | BANDGAP_ON);
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x21, TER_RESISTOR_LOW | TER_CAL_DONE |
> -                                      SETRD_MAX | TER_RESISTORS_ON);
> -     dw_mipi_dsi_phy_write(dsi, 0x21, TER_RESISTOR_HIGH | LEVEL_SHIFTERS_ON |
> -                                      SETRD_MAX | POWER_MANAGE |
> -                                      TER_RESISTORS_ON);
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x60, TLP_PROGRAM_EN | ns2bc(dsi, 500));
> -     dw_mipi_dsi_phy_write(dsi, 0x61, THS_PRE_PROGRAM_EN | ns2ui(dsi, 40));
> -     dw_mipi_dsi_phy_write(dsi, 0x62, THS_ZERO_PROGRAM_EN | ns2bc(dsi, 300));
> -     dw_mipi_dsi_phy_write(dsi, 0x63, THS_PRE_PROGRAM_EN | ns2ui(dsi, 100));
> -     dw_mipi_dsi_phy_write(dsi, 0x64, BIT(5) | ns2bc(dsi, 100));
> -     dw_mipi_dsi_phy_write(dsi, 0x65, BIT(5) | (ns2bc(dsi, 60) + 7));
> -
> -     dw_mipi_dsi_phy_write(dsi, 0x70, TLP_PROGRAM_EN | ns2bc(dsi, 500));
> -     dw_mipi_dsi_phy_write(dsi, 0x71,
> -                           THS_PRE_PROGRAM_EN | (ns2ui(dsi, 50) + 5));
> -     dw_mipi_dsi_phy_write(dsi, 0x72,
> -                           THS_ZERO_PROGRAM_EN | (ns2bc(dsi, 140) + 2));
> -     dw_mipi_dsi_phy_write(dsi, 0x73,
> -                           THS_PRE_PROGRAM_EN | (ns2ui(dsi, 60) + 8));
> -     dw_mipi_dsi_phy_write(dsi, 0x74, BIT(5) | ns2bc(dsi, 100));
> -
> -     dsi_write(dsi, DSI_PHY_RSTZ, PHY_ENFORCEPLL | PHY_ENABLECLK |
> -                                  PHY_UNRSTZ | PHY_UNSHUTDOWNZ);
> -
> -     ret = readl_poll_timeout(dsi->base + DSI_PHY_STATUS,
> -                              val, val & LOCK, 1000, PHY_STATUS_TIMEOUT_US);
> -     if (ret < 0) {
> -             DRM_DEV_ERROR(dsi->dev, "failed to wait for phy lock state\n");
> -             goto phy_init_end;
> -     }
> -
> -     ret = readl_poll_timeout(dsi->base + DSI_PHY_STATUS,
> -                              val, val & STOP_STATE_CLK_LANE, 1000,
> -                              PHY_STATUS_TIMEOUT_US);
> -     if (ret < 0)
> -             DRM_DEV_ERROR(dsi->dev,
> -                           "failed to wait for phy clk lane stop state\n");
> -
> -phy_init_end:
> -     clk_disable_unprepare(dsi->phy_cfg_clk);
> -
> -     return ret;
> -}
> -
> -static int dw_mipi_dsi_get_lane_bps(struct dw_mipi_dsi *dsi,
> -                                 struct drm_display_mode *mode)
> -{
> -     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;
> -
> -     bpp = mipi_dsi_pixel_format_to_bpp(dsi->format);
> -     if (bpp < 0) {
> -             DRM_DEV_ERROR(dsi->dev,
> -                           "failed to get bpp for pixel format %d\n",
> -                           dsi->format);
> -             return bpp;
> -     }
> -
> -     mpclk = DIV_ROUND_UP(mode->clock, MSEC_PER_SEC);
> -     if (mpclk) {
> -             /* take 1 / 0.8, since mbps must big than bandwidth of RGB */
> -             tmp = mpclk * (bpp / dsi->lanes) * 10 / 8;
> -             if (tmp < max_mbps)
> -                     target_mbps = tmp;
> -             else
> -                     DRM_DEV_ERROR(dsi->dev,
> -                                   "DPHY clock frequency is out of range\n");
> -     }
> -
> -     pllref = DIV_ROUND_UP(clk_get_rate(dsi->pllref_clk), USEC_PER_SEC);
> -     tmp = pllref;
> -
> -     /*
> -      * The limits on the PLL divisor are:
> -      *
> -      *      5MHz <= (pllref / n) <= 40MHz
> -      *
> -      * we walk over these values in descreasing order so that if we hit
> -      * an exact match for target_mbps it is more likely that "m" will be
> -      * even.
> -      *
> -      * TODO: ensure that "m" is even after this loop.
> -      */
> -     for (i = pllref / 5; i > (pllref / 40); i--) {
> -             pre = pllref / i;
> -             if ((tmp > (target_mbps % pre)) && (target_mbps / pre < 512)) {
> -                     tmp = target_mbps % pre;
> -                     n = i;
> -                     m = target_mbps / pre;
> -             }
> -             if (tmp == 0)
> -                     break;
> -     }
> -
> -     dsi->lane_mbps = pllref / n * m;
> -     dsi->input_div = n;
> -     dsi->feedback_div = m;
> -
> -     return 0;
> -}
> -
> -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);
> -
> -     if (device->lanes > dsi->pdata->max_data_lanes) {
> -             DRM_DEV_ERROR(dsi->dev,
> -                           "the number of data lanes(%u) is too many\n",
> -                           device->lanes);
> -             return -EINVAL;
> -     }
> -
> -     dsi->lanes = device->lanes;
> -     dsi->channel = device->channel;
> -     dsi->format = device->format;
> -     dsi->mode_flags = device->mode_flags;
> -     dsi->panel = of_drm_find_panel(device->dev.of_node);
> -     if (dsi->panel)
> -             return drm_panel_attach(dsi->panel, &dsi->connector);
> -
> -     return -EINVAL;
> -}
> -
> -static int dw_mipi_dsi_host_detach(struct mipi_dsi_host *host,
> -                                struct mipi_dsi_device *device)
> -{
> -     struct dw_mipi_dsi *dsi = host_to_dsi(host);
> -
> -     drm_panel_detach(dsi->panel);
> -
> -     return 0;
> -}
> -
> -static void dw_mipi_message_config(struct dw_mipi_dsi *dsi,
> -                                const struct mipi_dsi_msg *msg)
> -{
> -     bool lpm = msg->flags & MIPI_DSI_MSG_USE_LPM;
> -     u32 val = 0;
> -
> -     if (msg->flags & MIPI_DSI_MSG_REQ_ACK)
> -             val |= EN_ACK_RQST;
> -     if (lpm)
> -             val |= CMD_MODE_ALL_LP;
> -
> -     dsi_write(dsi, DSI_LPCLK_CTRL, lpm ? 0 : PHY_TXREQUESTCLKHS);
> -     dsi_write(dsi, DSI_CMD_MODE_CFG, val);
> -}
> -
> -static int dw_mipi_dsi_gen_pkt_hdr_write(struct dw_mipi_dsi *dsi, u32 
> hdr_val)
> -{
> -     int ret;
> -     u32 val, mask;
> -
> -     ret = readl_poll_timeout(dsi->base + DSI_CMD_PKT_STATUS,
> -                              val, !(val & GEN_CMD_FULL), 1000,
> -                              CMD_PKT_STATUS_TIMEOUT_US);
> -     if (ret < 0) {
> -             DRM_DEV_ERROR(dsi->dev,
> -                           "failed to get available command FIFO\n");
> -             return ret;
> -     }
> -
> -     dsi_write(dsi, DSI_GEN_HDR, hdr_val);
> -
> -     mask = GEN_CMD_EMPTY | GEN_PLD_W_EMPTY;
> -     ret = readl_poll_timeout(dsi->base + DSI_CMD_PKT_STATUS,
> -                              val, (val & mask) == mask,
> -                              1000, CMD_PKT_STATUS_TIMEOUT_US);
> -     if (ret < 0) {
> -             DRM_DEV_ERROR(dsi->dev, "failed to write command FIFO\n");
> -             return ret;
> -     }
> -
> -     return 0;
> -}
> -
> -static int dw_mipi_dsi_dcs_short_write(struct dw_mipi_dsi *dsi,
> -                                    const struct mipi_dsi_msg *msg)
> -{
> -     const u8 *tx_buf = msg->tx_buf;
> -     u16 data = 0;
> -     u32 val;
> -
> -     if (msg->tx_len > 0)
> -             data |= tx_buf[0];
> -     if (msg->tx_len > 1)
> -             data |= tx_buf[1] << 8;
> -
> -     if (msg->tx_len > 2) {
> -             DRM_DEV_ERROR(dsi->dev,
> -                           "too long tx buf length %zu for short write\n",
> -                           msg->tx_len);
> -             return -EINVAL;
> -     }
> -
> -     val = GEN_HDATA(data) | GEN_HTYPE(msg->type);
> -     return dw_mipi_dsi_gen_pkt_hdr_write(dsi, val);
> -}
> -
> -static int dw_mipi_dsi_dcs_long_write(struct dw_mipi_dsi *dsi,
> -                                   const struct mipi_dsi_msg *msg)
> -{
> -     const u8 *tx_buf = msg->tx_buf;
> -     int len = msg->tx_len, pld_data_bytes = sizeof(u32), ret;
> -     u32 hdr_val = GEN_HDATA(msg->tx_len) | GEN_HTYPE(msg->type);
> -     u32 remainder;
> -     u32 val;
> -
> -     if (msg->tx_len < 3) {
> -             DRM_DEV_ERROR(dsi->dev,
> -                           "wrong tx buf length %zu for long write\n",
> -                           msg->tx_len);
> -             return -EINVAL;
> -     }
> -
> -     while (DIV_ROUND_UP(len, pld_data_bytes)) {
> -             if (len < pld_data_bytes) {
> -                     remainder = 0;
> -                     memcpy(&remainder, tx_buf, len);
> -                     dsi_write(dsi, DSI_GEN_PLD_DATA, remainder);
> -                     len = 0;
> -             } else {
> -                     memcpy(&remainder, tx_buf, pld_data_bytes);
> -                     dsi_write(dsi, DSI_GEN_PLD_DATA, remainder);
> -                     tx_buf += pld_data_bytes;
> -                     len -= pld_data_bytes;
> -             }
> -
> -             ret = readl_poll_timeout(dsi->base + DSI_CMD_PKT_STATUS,
> -                                      val, !(val & GEN_PLD_W_FULL), 1000,
> -                                      CMD_PKT_STATUS_TIMEOUT_US);
> -             if (ret < 0) {
> -                     DRM_DEV_ERROR(dsi->dev,
> -                                   "failed to get available write payload 
> FIFO\n");
> -                     return ret;
> -             }
> -     }
> -
> -     return dw_mipi_dsi_gen_pkt_hdr_write(dsi, hdr_val);
> -}
> -
> -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);
> -     int ret;
> -
> -     dw_mipi_message_config(dsi, msg);
> -
> -     switch (msg->type) {
> -     case MIPI_DSI_DCS_SHORT_WRITE:
> -     case MIPI_DSI_DCS_SHORT_WRITE_PARAM:
> -     case MIPI_DSI_SET_MAXIMUM_RETURN_PACKET_SIZE:
> -             ret = dw_mipi_dsi_dcs_short_write(dsi, msg);
> -             break;
> -     case MIPI_DSI_DCS_LONG_WRITE:
> -             ret = dw_mipi_dsi_dcs_long_write(dsi, msg);
> -             break;
> -     default:
> -             DRM_DEV_ERROR(dsi->dev, "unsupported message type 0x%02x\n",
> -                           msg->type);
> -             ret = -EINVAL;
> -     }
> -
> -     return ret;
> -}
> -
> -static const struct mipi_dsi_host_ops dw_mipi_dsi_host_ops = {
> -     .attach = dw_mipi_dsi_host_attach,
> -     .detach = dw_mipi_dsi_host_detach,
> -     .transfer = dw_mipi_dsi_host_transfer,
> -};
> -
> -static void dw_mipi_dsi_video_mode_config(struct dw_mipi_dsi *dsi)
> -{
> -     u32 val;
> -
> -     val = 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_NON_BURST_SYNC_PULSES;
> -     else
> -             val |= VID_MODE_TYPE_NON_BURST_SYNC_EVENTS;
> -
> -     dsi_write(dsi, DSI_VID_MODE_CFG, val);
> -}
> -
> -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);
> -             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_MODE_CFG, ENABLE_VIDEO_MODE);
> -             dw_mipi_dsi_video_mode_config(dsi);
> -             dsi_write(dsi, DSI_LPCLK_CTRL, PHY_TXREQUESTCLKHS);
> -             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)
> -{
> -     /*
> -      * The maximum permitted escape clock is 20MHz and it is derived from
> -      * lanebyteclk, which is running at "lane_mbps / 8".  Thus we want:
> -      *
> -      *     (lane_mbps >> 3) / esc_clk_division < 20
> -      * which is:
> -      *     (lane_mbps >> 3) / 20 > esc_clk_division
> -      */
> -     u32 esc_clk_division = (dsi->lane_mbps >> 3) / 20 + 1;
> -
> -     dsi_write(dsi, DSI_PWR_UP, RESET);
> -     dsi_write(dsi, DSI_PHY_RSTZ, PHY_DISFORCEPLL | PHY_DISABLECLK
> -               | PHY_RSTZ | PHY_SHUTDOWNZ);
> -     dsi_write(dsi, DSI_CLKMGR_CFG, TO_CLK_DIVIDSION(10) |
> -               TX_ESC_CLK_DIVIDSION(esc_clk_division));
> -}
> -
> -static void dw_mipi_dsi_dpi_config(struct dw_mipi_dsi *dsi,
> -                                struct drm_display_mode *mode)
> -{
> -     u32 val = 0, color = 0;
> -
> -     switch (dsi->format) {
> -     case MIPI_DSI_FMT_RGB888:
> -             color = DPI_COLOR_CODING_24BIT;
> -             break;
> -     case MIPI_DSI_FMT_RGB666:
> -             color = DPI_COLOR_CODING_18BIT_2 | EN18_LOOSELY;
> -             break;
> -     case MIPI_DSI_FMT_RGB666_PACKED:
> -             color = DPI_COLOR_CODING_18BIT_1;
> -             break;
> -     case MIPI_DSI_FMT_RGB565:
> -             color = DPI_COLOR_CODING_16BIT_1;
> -             break;
> -     }
> -
> -     if (mode->flags & DRM_MODE_FLAG_NVSYNC)
> -             val |= VSYNC_ACTIVE_LOW;
> -     if (mode->flags & DRM_MODE_FLAG_NHSYNC)
> -             val |= HSYNC_ACTIVE_LOW;
> -
> -     dsi_write(dsi, DSI_DPI_VCID, DPI_VID(dsi->channel));
> -     dsi_write(dsi, DSI_DPI_COLOR_CODING, color);
> -     dsi_write(dsi, DSI_DPI_CFG_POL, val);
> -     dsi_write(dsi, DSI_DPI_LP_CMD_TIM, OUTVACT_LPCMD_TIME(4)
> -               | INVACT_LPCMD_TIME(4));
> -}
> -
> -static void dw_mipi_dsi_packet_handler_config(struct dw_mipi_dsi *dsi)
> -{
> -     dsi_write(dsi, DSI_PCKHDL_CFG, EN_CRC_RX | EN_ECC_RX | EN_BTA);
> -}
> -
> -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));
> -}
> -
> -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,
> -                                        struct drm_display_mode *mode,
> -                                        u32 hcomponent)
> -{
> -     u32 frac, lbcc;
> -
> -     lbcc = hcomponent * dsi->lane_mbps * MSEC_PER_SEC / 8;
> -
> -     frac = lbcc % mode->clock;
> -     lbcc = lbcc / mode->clock;
> -     if (frac)
> -             lbcc++;
> -
> -     return lbcc;
> -}
> -
> -static void dw_mipi_dsi_line_timer_config(struct dw_mipi_dsi *dsi,
> -                                       struct drm_display_mode *mode)
> -{
> -     u32 htotal, hsa, hbp, lbcc;
> -
> -     htotal = mode->htotal;
> -     hsa = mode->hsync_end - mode->hsync_start;
> -     hbp = mode->htotal - mode->hsync_end;
> -
> -     lbcc = dw_mipi_dsi_get_hcomponent_lbcc(dsi, mode, htotal);
> -     dsi_write(dsi, DSI_VID_HLINE_TIME, lbcc);
> -
> -     lbcc = dw_mipi_dsi_get_hcomponent_lbcc(dsi, mode, hsa);
> -     dsi_write(dsi, DSI_VID_HSA_TIME, lbcc);
> -
> -     lbcc = dw_mipi_dsi_get_hcomponent_lbcc(dsi, mode, hbp);
> -     dsi_write(dsi, DSI_VID_HBP_TIME, lbcc);
> -}
> -
> -static void dw_mipi_dsi_vertical_timing_config(struct dw_mipi_dsi *dsi,
> -                                            struct drm_display_mode *mode)
> -{
> -     u32 vactive, vsa, vfp, vbp;
> -
> -     vactive = mode->vdisplay;
> -     vsa = mode->vsync_end - mode->vsync_start;
> -     vfp = mode->vsync_start - mode->vdisplay;
> -     vbp = mode->vtotal - mode->vsync_end;
> -
> -     dsi_write(dsi, DSI_VID_VACTIVE_LINES, vactive);
> -     dsi_write(dsi, DSI_VID_VSA_LINES, vsa);
> -     dsi_write(dsi, DSI_VID_VFP_LINES, vfp);
> -     dsi_write(dsi, DSI_VID_VBP_LINES, vbp);
> -}
> -
> -static void dw_mipi_dsi_dphy_timing_config(struct dw_mipi_dsi *dsi)
> -{
> -     dsi_write(dsi, DSI_PHY_TMR_CFG, PHY_HS2LP_TIME(0x40)
> -               | PHY_LP2HS_TIME(0x40) | MAX_RD_TIME(10000));
> -
> -     dsi_write(dsi, DSI_PHY_TMR_LPCLK_CFG, PHY_CLKHS2LP_TIME(0x40)
> -               | PHY_CLKLP2HS_TIME(0x40));
> -}
> -
> -static void dw_mipi_dsi_dphy_interface_config(struct dw_mipi_dsi *dsi)
> -{
> -     dsi_write(dsi, DSI_PHY_IF_CFG, PHY_STOP_WAIT_TIME(0x20) |
> -               N_LANES(dsi->lanes));
> -}
> -
> -static void dw_mipi_dsi_clear_err(struct dw_mipi_dsi *dsi)
> -{
> -     dsi_read(dsi, DSI_INT_ST0);
> -     dsi_read(dsi, DSI_INT_ST1);
> -     dsi_write(dsi, DSI_INT_MSK0, 0);
> -     dsi_write(dsi, DSI_INT_MSK1, 0);
> -}
> -
> -static void dw_mipi_dsi_encoder_disable(struct drm_encoder *encoder)
> -{
> -     struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
> -
> -     if (dsi->dpms_mode != DRM_MODE_DPMS_ON)
> -             return;
> -
> -     if (clk_prepare_enable(dsi->pclk)) {
> -             DRM_DEV_ERROR(dsi->dev, "Failed to enable pclk\n");
> -             return;
> -     }
> -
> -     drm_panel_disable(dsi->panel);
> -
> -     dw_mipi_dsi_set_mode(dsi, DW_MIPI_DSI_CMD_MODE);
> -     drm_panel_unprepare(dsi->panel);
> -
> -     dw_mipi_dsi_disable(dsi);
> -     pm_runtime_put(dsi->dev);
> -     clk_disable_unprepare(dsi->pclk);
> -     dsi->dpms_mode = DRM_MODE_DPMS_OFF;
> -}
> -
> -static void dw_mipi_dsi_encoder_enable(struct drm_encoder *encoder)
> -{
> -     struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
> -     struct drm_display_mode *mode = &encoder->crtc->state->adjusted_mode;
> -     const struct dw_mipi_dsi_plat_data *pdata = dsi->pdata;
> -     int mux = drm_of_encoder_active_endpoint_id(dsi->dev->of_node, encoder);
> -     u32 val;
> -     int ret;
> -
> -     ret = dw_mipi_dsi_get_lane_bps(dsi, mode);
> -     if (ret < 0)
> -             return;
> -
> -     if (dsi->dpms_mode == DRM_MODE_DPMS_ON)
> -             return;
> -
> -     if (clk_prepare_enable(dsi->pclk)) {
> -             DRM_DEV_ERROR(dsi->dev, "Failed to enable pclk\n");
> -             return;
> -     }
> -
> -     pm_runtime_get_sync(dsi->dev);
> -     dw_mipi_dsi_init(dsi);
> -     dw_mipi_dsi_dpi_config(dsi, mode);
> -     dw_mipi_dsi_packet_handler_config(dsi);
> -     dw_mipi_dsi_video_mode_config(dsi);
> -     dw_mipi_dsi_video_packet_config(dsi, mode);
> -     dw_mipi_dsi_command_mode_config(dsi);
> -     dw_mipi_dsi_line_timer_config(dsi, mode);
> -     dw_mipi_dsi_vertical_timing_config(dsi, mode);
> -     dw_mipi_dsi_dphy_timing_config(dsi);
> -     dw_mipi_dsi_dphy_interface_config(dsi);
> -     dw_mipi_dsi_clear_err(dsi);
> -
> -     /*
> -      * For the RK3399, the clk of grf must be enabled before writing grf
> -      * register. And for RK3288 or other soc, this grf_clk must be NULL,
> -      * the clk_prepare_enable return true directly.
> -      */
> -     ret = clk_prepare_enable(dsi->grf_clk);
> -     if (ret) {
> -             DRM_DEV_ERROR(dsi->dev, "Failed to enable grf_clk: %d\n", ret);
> -             return;
> -     }
> -
> -     if (pdata->grf_dsi0_mode_reg)
> -             regmap_write(dsi->grf_regmap, pdata->grf_dsi0_mode_reg,
> -                          pdata->grf_dsi0_mode);
> -
> -     dw_mipi_dsi_phy_init(dsi);
> -     dw_mipi_dsi_wait_for_two_frames(mode);
> -
> -     dw_mipi_dsi_set_mode(dsi, DW_MIPI_DSI_CMD_MODE);
> -     if (drm_panel_prepare(dsi->panel))
> -             DRM_DEV_ERROR(dsi->dev, "failed to prepare panel\n");
> -
> -     dw_mipi_dsi_set_mode(dsi, DW_MIPI_DSI_VID_MODE);
> -     drm_panel_enable(dsi->panel);
> -
> -     clk_disable_unprepare(dsi->pclk);
> -
> -     if (mux)
> -             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);
> -     DRM_DEV_DEBUG(dsi->dev,
> -                   "vop %s output to dsi0\n", (mux) ? "LIT" : "BIG");
> -     dsi->dpms_mode = DRM_MODE_DPMS_ON;
> -
> -     clk_disable_unprepare(dsi->grf_clk);
> -}
> -
> -static int
> -dw_mipi_dsi_encoder_atomic_check(struct drm_encoder *encoder,
> -                              struct drm_crtc_state *crtc_state,
> -                              struct drm_connector_state *conn_state)
> -{
> -     struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc_state);
> -     struct dw_mipi_dsi *dsi = encoder_to_dsi(encoder);
> -
> -     switch (dsi->format) {
> -     case MIPI_DSI_FMT_RGB888:
> -             s->output_mode = ROCKCHIP_OUT_MODE_P888;
> -             break;
> -     case MIPI_DSI_FMT_RGB666:
> -             s->output_mode = ROCKCHIP_OUT_MODE_P666;
> -             break;
> -     case MIPI_DSI_FMT_RGB565:
> -             s->output_mode = ROCKCHIP_OUT_MODE_P565;
> -             break;
> -     default:
> -             WARN_ON(1);
> -             return -EINVAL;
> -     }
> -
> -     s->output_type = DRM_MODE_CONNECTOR_DSI;
> -
> -     return 0;
> -}
> -
> -static const struct drm_encoder_helper_funcs
> -dw_mipi_dsi_encoder_helper_funcs = {
> -     .enable = dw_mipi_dsi_encoder_enable,
> -     .disable = dw_mipi_dsi_encoder_disable,
> -     .atomic_check = dw_mipi_dsi_encoder_atomic_check,
> -};
> -
> -static const struct drm_encoder_funcs dw_mipi_dsi_encoder_funcs = {
> -     .destroy = drm_encoder_cleanup,
> -};
> -
> -static int dw_mipi_dsi_connector_get_modes(struct drm_connector *connector)
> -{
> -     struct dw_mipi_dsi *dsi = con_to_dsi(connector);
> -
> -     return drm_panel_get_modes(dsi->panel);
> -}
> -
> -static struct drm_connector_helper_funcs dw_mipi_dsi_connector_helper_funcs 
> = {
> -     .get_modes = dw_mipi_dsi_connector_get_modes,
> -};
> -
> -static void dw_mipi_dsi_drm_connector_destroy(struct drm_connector 
> *connector)
> -{
> -     drm_connector_unregister(connector);
> -     drm_connector_cleanup(connector);
> -}
> -
> -static const struct drm_connector_funcs dw_mipi_dsi_atomic_connector_funcs = 
> {
> -     .fill_modes = drm_helper_probe_single_connector_modes,
> -     .destroy = dw_mipi_dsi_drm_connector_destroy,
> -     .reset = drm_atomic_helper_connector_reset,
> -     .atomic_duplicate_state = drm_atomic_helper_connector_duplicate_state,
> -     .atomic_destroy_state = drm_atomic_helper_connector_destroy_state,
> -};
> -
> -static int dw_mipi_dsi_register(struct drm_device *drm,
> -                             struct dw_mipi_dsi *dsi)
> -{
> -     struct drm_encoder *encoder = &dsi->encoder;
> -     struct drm_connector *connector = &dsi->connector;
> -     struct device *dev = dsi->dev;
> -     int ret;
> -
> -     encoder->possible_crtcs = drm_of_find_possible_crtcs(drm,
> -                                                          dev->of_node);
> -     /*
> -      * If we failed to find the CRTC(s) which this encoder is
> -      * supposed to be connected to, it's because the CRTC has
> -      * not been registered yet.  Defer probing, and hope that
> -      * the required CRTC is added later.
> -      */
> -     if (encoder->possible_crtcs == 0)
> -             return -EPROBE_DEFER;
> -
> -     drm_encoder_helper_add(&dsi->encoder,
> -                            &dw_mipi_dsi_encoder_helper_funcs);
> -     ret = drm_encoder_init(drm, &dsi->encoder, &dw_mipi_dsi_encoder_funcs,
> -                            DRM_MODE_ENCODER_DSI, NULL);
> -     if (ret) {
> -             DRM_DEV_ERROR(dev, "Failed to initialize encoder with drm\n");
> -             return ret;
> -     }
> -
> -     drm_connector_helper_add(connector,
> -                              &dw_mipi_dsi_connector_helper_funcs);
> -
> -     drm_connector_init(drm, &dsi->connector,
> -                        &dw_mipi_dsi_atomic_connector_funcs,
> -                        DRM_MODE_CONNECTOR_DSI);
> -
> -     drm_mode_connector_attach_encoder(connector, encoder);
> -
> -     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)) {
> -             DRM_DEV_ERROR(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,
> -     .max_data_lanes = 4,
> -};
> -
> -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_CON20,
> -     .grf_dsi0_mode = RK3399_GRF_DSI_MODE,
> -     .grf_dsi0_mode_reg = RK3399_GRF_SOC_CON22,
> -     .flags = DW_MIPI_NEEDS_PHY_CFG_CLK | DW_MIPI_NEEDS_GRF_CLK,
> -     .max_data_lanes = 4,
> -};
> -
> -static const struct of_device_id dw_mipi_dsi_dt_ids[] = {
> -     {
> -      .compatible = "rockchip,rk3288-mipi-dsi",
> -      .data = &rk3288_mipi_dsi_drv_data,
> -     }, {
> -      .compatible = "rockchip,rk3399-mipi-dsi",
> -      .data = &rk3399_mipi_dsi_drv_data,
> -     },
> -     { /* sentinel */ }
> -};
> -MODULE_DEVICE_TABLE(of, dw_mipi_dsi_dt_ids);
> -
> -static int dw_mipi_dsi_bind(struct device *dev, struct device *master,
> -                         void *data)
> -{
> -     const struct of_device_id *of_id =
> -                     of_match_device(dw_mipi_dsi_dt_ids, dev);
> -     const struct dw_mipi_dsi_plat_data *pdata = of_id->data;
> -     struct platform_device *pdev = to_platform_device(dev);
> -     struct reset_control *apb_rst;
> -     struct drm_device *drm = data;
> -     struct dw_mipi_dsi *dsi;
> -     struct resource *res;
> -     int ret;
> -
> -     dsi = devm_kzalloc(dev, sizeof(*dsi), GFP_KERNEL);
> -     if (!dsi)
> -             return -ENOMEM;
> -
> -     dsi->dev = dev;
> -     dsi->pdata = pdata;
> -     dsi->dpms_mode = DRM_MODE_DPMS_OFF;
> -
> -     ret = rockchip_mipi_parse_dt(dsi);
> -     if (ret)
> -             return ret;
> -
> -     res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -     if (!res)
> -             return -ENODEV;
> -
> -     dsi->base = devm_ioremap_resource(dev, res);
> -     if (IS_ERR(dsi->base))
> -             return PTR_ERR(dsi->base);
> -
> -     dsi->pllref_clk = devm_clk_get(dev, "ref");
> -     if (IS_ERR(dsi->pllref_clk)) {
> -             ret = PTR_ERR(dsi->pllref_clk);
> -             DRM_DEV_ERROR(dev,
> -                           "Unable to get pll reference clock: %d\n", ret);
> -             return ret;
> -     }
> -
> -     dsi->pclk = devm_clk_get(dev, "pclk");
> -     if (IS_ERR(dsi->pclk)) {
> -             ret = PTR_ERR(dsi->pclk);
> -             DRM_DEV_ERROR(dev, "Unable to get pclk: %d\n", ret);
> -             return ret;
> -     }
> -
> -     /*
> -      * Note that the reset was not defined in the initial device tree, so
> -      * we have to be prepared for it not being found.
> -      */
> -     apb_rst = devm_reset_control_get(dev, "apb");
> -     if (IS_ERR(apb_rst)) {
> -             ret = PTR_ERR(apb_rst);
> -             if (ret == -ENOENT) {
> -                     apb_rst = NULL;
> -             } else {
> -                     DRM_DEV_ERROR(dev,
> -                                   "Unable to get reset control: %d\n", ret);
> -                     return ret;
> -             }
> -     }
> -
> -     if (apb_rst) {
> -             ret = clk_prepare_enable(dsi->pclk);
> -             if (ret) {
> -                     DRM_DEV_ERROR(dev, "Failed to enable pclk\n");
> -                     return ret;
> -             }
> -
> -             reset_control_assert(apb_rst);
> -             usleep_range(10, 20);
> -             reset_control_deassert(apb_rst);
> -
> -             clk_disable_unprepare(dsi->pclk);
> -     }
> -
> -     if (pdata->flags & DW_MIPI_NEEDS_PHY_CFG_CLK) {
> -             dsi->phy_cfg_clk = devm_clk_get(dev, "phy_cfg");
> -             if (IS_ERR(dsi->phy_cfg_clk)) {
> -                     ret = PTR_ERR(dsi->phy_cfg_clk);
> -                     DRM_DEV_ERROR(dev,
> -                                   "Unable to get phy_cfg_clk: %d\n", ret);
> -                     return ret;
> -             }
> -     }
> -
> -     if (pdata->flags & DW_MIPI_NEEDS_GRF_CLK) {
> -             dsi->grf_clk = devm_clk_get(dev, "grf");
> -             if (IS_ERR(dsi->grf_clk)) {
> -                     ret = PTR_ERR(dsi->grf_clk);
> -                     DRM_DEV_ERROR(dev, "Unable to get grf_clk: %d\n", ret);
> -                     return ret;
> -             }
> -     }
> -
> -     ret = clk_prepare_enable(dsi->pllref_clk);
> -     if (ret) {
> -             DRM_DEV_ERROR(dev, "Failed to enable pllref_clk\n");
> -             return ret;
> -     }
> -
> -     ret = dw_mipi_dsi_register(drm, dsi);
> -     if (ret) {
> -             DRM_DEV_ERROR(dev, "Failed to register mipi_dsi: %d\n", ret);
> -             goto err_pllref;
> -     }
> -
> -     pm_runtime_enable(dev);
> -
> -     dsi->dsi_host.ops = &dw_mipi_dsi_host_ops;
> -     dsi->dsi_host.dev = dev;
> -     ret = mipi_dsi_host_register(&dsi->dsi_host);
> -     if (ret) {
> -             DRM_DEV_ERROR(dev, "Failed to register MIPI host: %d\n", ret);
> -             goto err_cleanup;
> -     }
> -
> -     if (!dsi->panel) {
> -             ret = -EPROBE_DEFER;
> -             goto err_mipi_dsi_host;
> -     }
> -
> -     dev_set_drvdata(dev, dsi);
> -     return 0;
> -
> -err_mipi_dsi_host:
> -     mipi_dsi_host_unregister(&dsi->dsi_host);
> -err_cleanup:
> -     drm_encoder_cleanup(&dsi->encoder);
> -     drm_connector_cleanup(&dsi->connector);
> -err_pllref:
> -     clk_disable_unprepare(dsi->pllref_clk);
> -     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);
> -
> -     mipi_dsi_host_unregister(&dsi->dsi_host);
> -     pm_runtime_disable(dev);
> -     clk_disable_unprepare(dsi->pllref_clk);
> -}
> -
> -static const struct component_ops dw_mipi_dsi_ops = {
> -     .bind   = dw_mipi_dsi_bind,
> -     .unbind = dw_mipi_dsi_unbind,
> -};
> -
> -static int dw_mipi_dsi_probe(struct platform_device *pdev)
> -{
> -     return component_add(&pdev->dev, &dw_mipi_dsi_ops);
> -}
> -
> -static int dw_mipi_dsi_remove(struct platform_device *pdev)
> -{
> -     component_del(&pdev->dev, &dw_mipi_dsi_ops);
> -     return 0;
> -}
> -
> -struct platform_driver dw_mipi_dsi_driver = {
> -     .probe          = dw_mipi_dsi_probe,
> -     .remove         = dw_mipi_dsi_remove,
> -     .driver         = {
> -             .of_match_table = dw_mipi_dsi_dt_ids,
> -             .name   = DRIVER_NAME,
> -     },
> -};
> diff --git a/drivers/gpu/drm/rockchip/dw-mipi-dsi_rockchip.c 
> b/drivers/gpu/drm/rockchip/dw-mipi-dsi_rockchip.c
> new file mode 100644
> index 0000000..c682ed2
> --- /dev/null
> +++ b/drivers/gpu/drm/rockchip/dw-mipi-dsi_rockchip.c
> @@ -0,0 +1,764 @@
> +/*
> + * Copyright (C) Fuzhou Rockchip Electronics Co.Ltd
> + * Author:
> + *      Chris Zhong <z...@rock-chips.com>
> + *      Nickey Yang <nickey.y...@rock-chips.com>
> + *
> + * This software is licensed under the terms of the GNU General Public
> + * License version 2, as published by the Free Software Foundation, and
> + * may be copied, distributed, and modified under those terms.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> + *
> + * License terms:  GNU General Public License (GPL), version 2
> + */
> +
> +#include <linux/clk.h>
> +#include <linux/iopoll.h>
> +#include <linux/math64.h>
> +#include <linux/module.h>
> +#include <linux/of_device.h>
> +#include <drm/drmP.h>
> +#include <drm/drm_mipi_dsi.h>
> +#include <drm/bridge/dw_mipi_dsi.h>
> +#include <video/mipi_display.h>
> +#include <linux/regmap.h>
> +#include <drm/drm_of.h>
> +#include <linux/mfd/syscon.h>
> +
> +#include "rockchip_drm_drv.h"
> +#include "rockchip_drm_vop.h"
> +
> +#define DSI_PHY_TST_CTRL0            0xb4
> +#define PHY_TESTCLK                  BIT(1)
> +#define PHY_UNTESTCLK                        0
> +#define PHY_TESTCLR                  BIT(0)
> +#define PHY_UNTESTCLR                        0
> +
> +#define DSI_PHY_TST_CTRL1            0xb8
> +#define PHY_TESTEN                   BIT(16)
> +#define PHY_UNTESTEN                 0
> +#define PHY_TESTDOUT(n)                      (((n) & 0xff) << 8)
> +#define PHY_TESTDIN(n)                       (((n) & 0xff) << 0)
> +
> +#define BYPASS_VCO_RANGE     BIT(7)
> +#define VCO_RANGE_CON_SEL(val)       (((val) & 0x7) << 3)
> +#define VCO_IN_CAP_CON_DEFAULT       (0x0 << 1)
> +#define VCO_IN_CAP_CON_LOW   (0x1 << 1)
> +#define VCO_IN_CAP_CON_HIGH  (0x2 << 1)
> +#define REF_BIAS_CUR_SEL     BIT(0)
> +
> +#define CP_CURRENT_3UA       0x1
> +#define CP_CURRENT_4_5UA     0x2
> +#define CP_CURRENT_7_5UA     0x6
> +#define CP_CURRENT_6UA       0x9
> +#define CP_CURRENT_12UA      0xb
> +#define CP_CURRENT_SEL(val)  ((val) & 0xf)
> +#define CP_PROGRAM_EN                BIT(7)
> +
> +#define LPF_RESISTORS_15_5KOHM       0x1
> +#define LPF_RESISTORS_13KOHM 0x2
> +#define LPF_RESISTORS_11_5KOHM       0x4
> +#define LPF_RESISTORS_10_5KOHM       0x8
> +#define LPF_RESISTORS_8KOHM  0x10
> +#define LPF_PROGRAM_EN               BIT(6)
> +#define LPF_RESISTORS_SEL(val)       ((val) & 0x3f)
> +
> +#define HSFREQRANGE_SEL(val) (((val) & 0x3f) << 1)
> +
> +#define INPUT_DIVIDER(val)   (((val) - 1) & 0x7f)
> +#define LOW_PROGRAM_EN               0
> +#define HIGH_PROGRAM_EN              BIT(7)
> +#define LOOP_DIV_LOW_SEL(val)        (((val) - 1) & 0x1f)
> +#define LOOP_DIV_HIGH_SEL(val)       ((((val) - 1) >> 5) & 0xf)
> +#define PLL_LOOP_DIV_EN              BIT(5)
> +#define PLL_INPUT_DIV_EN     BIT(4)
> +
> +#define POWER_CONTROL                BIT(6)
> +#define INTERNAL_REG_CURRENT BIT(3)
> +#define BIAS_BLOCK_ON                BIT(2)
> +#define BANDGAP_ON           BIT(0)
> +
> +#define TER_RESISTOR_HIGH    BIT(7)
> +#define      TER_RESISTOR_LOW        0
> +#define LEVEL_SHIFTERS_ON    BIT(6)
> +#define TER_CAL_DONE         BIT(5)
> +#define SETRD_MAX            (0x7 << 2)
> +#define POWER_MANAGE         BIT(1)
> +#define TER_RESISTORS_ON     BIT(0)
> +
> +#define BIASEXTR_SEL(val)    ((val) & 0x7)
> +#define BANDGAP_SEL(val)     ((val) & 0x7)
> +#define TLP_PROGRAM_EN               BIT(7)
> +#define THS_PRE_PROGRAM_EN   BIT(7)
> +#define THS_ZERO_PROGRAM_EN  BIT(6)
> +
> +#define PLL_BIAS_CUR_SEL_CAP_VCO_CONTROL     0x10
> +#define PLL_CP_CONTROL_PLL_LOCK_BYPASS       0x11
> +#define PLL_LPF_AND_CP_CONTROL       0x12
> +#define PLL_INPUT_DIVIDER_RATIO      0x17
> +#define PLL_LOOP_DIVIDER_RATIO       0x18
> +#define PLL_INPUT_AND_LOOP_DIVIDER_RATIOS_CONTROL    0x19
> +#define BANDGAP_AND_BIAS_CONTROL     0x20
> +#define TERMINATION_RESISTER_CONTROL 0x21
> +#define AFE_BIAS_BANDGAP_ANALOG_PROGRAMMABILITY      0x22
> +#define HS_RX_CONTROL_OF_LANE_0      0x44
> +#define HS_TX_CLOCK_LANE_REQUEST_STATE_TIME_CONTROL  0x60
> +#define HS_TX_CLOCK_LANE_PREPARE_STATE_TIME_CONTROL  0x61
> +#define HS_TX_CLOCK_LANE_HS_ZERO_STATE_TIME_CONTROL  0x62
> +#define HS_TX_CLOCK_LANE_TRAIL_STATE_TIME_CONTROL    0x63
> +#define HS_TX_CLOCK_LANE_EXIT_STATE_TIME_CONTROL     0x64
> +#define HS_TX_CLOCK_LANE_POST_TIME_CONTROL   0x65
> +#define HS_TX_DATA_LANE_REQUEST_STATE_TIME_CONTROL   0x70
> +#define HS_TX_DATA_LANE_PREPARE_STATE_TIME_CONTROL   0x71
> +#define HS_TX_DATA_LANE_HS_ZERO_STATE_TIME_CONTROL   0x72
> +#define HS_TX_DATA_LANE_TRAIL_STATE_TIME_CONTROL     0x73
> +#define HS_TX_DATA_LANE_EXIT_STATE_TIME_CONTROL      0x74
> +
> +#define DW_MIPI_NEEDS_PHY_CFG_CLK    BIT(0)
> +#define DW_MIPI_NEEDS_GRF_CLK                BIT(1)
> +
> +#define RK3288_GRF_SOC_CON6          0x025c
> +#define RK3288_DSI0_SEL_VOP_LIT              BIT(6)
> +#define RK3288_DSI1_SEL_VOP_LIT              BIT(9)
> +
> +#define RK3399_GRF_SOC_CON20         0x6250
> +#define RK3399_DSI0_SEL_VOP_LIT              BIT(0)
> +#define RK3399_DSI1_SEL_VOP_LIT              BIT(4)
> +
> +/* disable turnrequest, turndisable, forcetxstopmode, forcerxmode */
> +#define RK3399_GRF_SOC_CON22         0x6258
> +#define RK3399_GRF_DSI_MODE          0xffff0000
> +
> +#define to_dsi(nm)   container_of(nm, struct dw_mipi_dsi_rockchip, nm)
> +
> +enum {
> +     BANDGAP_97_07,
> +     BANDGAP_98_05,
> +     BANDGAP_99_02,
> +     BANDGAP_100_00,
> +     BANDGAP_93_17,
> +     BANDGAP_94_15,
> +     BANDGAP_95_12,
> +     BANDGAP_96_10,
> +};
> +
> +enum {
> +     BIASEXTR_87_1,
> +     BIASEXTR_91_5,
> +     BIASEXTR_95_9,
> +     BIASEXTR_100,
> +     BIASEXTR_105_94,
> +     BIASEXTR_111_88,
> +     BIASEXTR_118_8,
> +     BIASEXTR_127_7,
> +};
> +
> +struct rockchip_dw_dsi_chip_data {
> +     u32 dsi0_en_bit;
> +     u32 dsi1_en_bit;
> +     u32 grf_switch_reg;
> +     u32 grf_dsi0_mode;
> +     u32 grf_dsi0_mode_reg;
> +     unsigned int flags;
> +     unsigned int max_data_lanes;
> +};
> +
> +struct dw_mipi_dsi_rockchip {
> +     struct device *dev;
> +     struct drm_encoder encoder;
> +     void __iomem *base;
> +
> +     struct regmap *grf_regmap;
> +     struct clk *pllref_clk;
> +     struct clk *grf_clk;
> +     struct clk *phy_cfg_clk;
> +
> +     unsigned int lane_mbps; /* per lane */
> +     u16 input_div;
> +     u16 feedback_div;
> +     u32 format;
> +
> +     struct dw_mipi_dsi *dmd;
> +     const struct rockchip_dw_dsi_chip_data *cdata;
> +     struct dw_mipi_dsi_plat_data pdata;
> +};
> +
> +struct dphy_pll_parameter_map {
> +     unsigned int max_mbps;
> +     u8 hsfreqrange;
> +     u8 icpctrl;
> +     u8 lpfctrl;
> +};
> +
> +/* The table is based on 27MHz DPHY pll reference clock. */
> +static const struct dphy_pll_parameter_map dppa_map[] = {
> +     {  89, 0x00, CP_CURRENT_3UA, LPF_RESISTORS_13KOHM },
> +     {  99, 0x10, CP_CURRENT_3UA, LPF_RESISTORS_13KOHM },
> +     { 109, 0x20, CP_CURRENT_3UA, LPF_RESISTORS_13KOHM },
> +     { 129, 0x01, CP_CURRENT_3UA, LPF_RESISTORS_15_5KOHM },
> +     { 139, 0x11, CP_CURRENT_3UA, LPF_RESISTORS_15_5KOHM },
> +     { 149, 0x21, CP_CURRENT_3UA, LPF_RESISTORS_15_5KOHM },
> +     { 169, 0x02, CP_CURRENT_6UA, LPF_RESISTORS_13KOHM },
> +     { 179, 0x12, CP_CURRENT_6UA, LPF_RESISTORS_13KOHM },
> +     { 199, 0x22, CP_CURRENT_6UA, LPF_RESISTORS_13KOHM },
> +     { 219, 0x03, CP_CURRENT_4_5UA, LPF_RESISTORS_13KOHM },
> +     { 239, 0x13, CP_CURRENT_4_5UA, LPF_RESISTORS_13KOHM },
> +     { 249, 0x23, CP_CURRENT_4_5UA, LPF_RESISTORS_13KOHM },
> +     { 269, 0x04, CP_CURRENT_6UA, LPF_RESISTORS_11_5KOHM },
> +     { 299, 0x14, CP_CURRENT_6UA, LPF_RESISTORS_11_5KOHM },
> +     { 329, 0x05, CP_CURRENT_3UA, LPF_RESISTORS_15_5KOHM },
> +     { 359, 0x15, CP_CURRENT_3UA, LPF_RESISTORS_15_5KOHM },
> +     { 399, 0x25, CP_CURRENT_3UA, LPF_RESISTORS_15_5KOHM },
> +     { 449, 0x06, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 499, 0x16, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 549, 0x07, CP_CURRENT_7_5UA, LPF_RESISTORS_10_5KOHM },
> +     { 599, 0x17, CP_CURRENT_7_5UA, LPF_RESISTORS_10_5KOHM },
> +     { 649, 0x08, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 699, 0x18, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 749, 0x09, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 799, 0x19, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 849, 0x29, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 899, 0x39, CP_CURRENT_7_5UA, LPF_RESISTORS_11_5KOHM },
> +     { 949, 0x0a, CP_CURRENT_12UA, LPF_RESISTORS_8KOHM },
> +     { 999, 0x1a, CP_CURRENT_12UA, LPF_RESISTORS_8KOHM },
> +     {1049, 0x2a, CP_CURRENT_12UA, LPF_RESISTORS_8KOHM },
> +     {1099, 0x3a, CP_CURRENT_12UA, LPF_RESISTORS_8KOHM },
> +     {1149, 0x0b, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM },
> +     {1199, 0x1b, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM },
> +     {1249, 0x2b, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM },
> +     {1299, 0x3b, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM },
> +     {1349, 0x0c, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM },
> +     {1399, 0x1c, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM },
> +     {1449, 0x2c, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM },
> +     {1500, 0x3c, CP_CURRENT_12UA, LPF_RESISTORS_10_5KOHM }
> +};
> +
> +static int max_mbps_to_parameter(unsigned int max_mbps)
> +{
> +     int i;
> +
> +     for (i = 0; i < ARRAY_SIZE(dppa_map); i++)
> +             if (dppa_map[i].max_mbps >= max_mbps)
> +                     return i;
> +
> +     return -EINVAL;
> +}
> +
> +static inline void dsi_write(struct dw_mipi_dsi_rockchip *dsi, u32 reg, u32 
> val)
> +{
> +     writel(val, dsi->base + reg);
> +}
> +
> +static inline u32 dsi_read(struct dw_mipi_dsi_rockchip *dsi, u32 reg)
> +{
> +     return readl(dsi->base + reg);
> +}
> +
> +static inline void dsi_set(struct dw_mipi_dsi_rockchip *dsi, u32 reg, u32 
> mask)
> +{
> +     dsi_write(dsi, reg, dsi_read(dsi, reg) | mask);
> +}
> +
> +static inline void dsi_update_bits(struct dw_mipi_dsi_rockchip *dsi, u32 reg,
> +                                u32 mask, u32 val)
> +{
> +     dsi_write(dsi, reg, (dsi_read(dsi, reg) & ~mask) | val);
> +}
> +
> +static void dw_mipi_dsi_phy_write(struct dw_mipi_dsi_rockchip *dsi,
> +                               u8 test_code,
> +                               u8 test_data)
> +{
> +     /*
> +      * With the falling edge on TESTCLK, the TESTDIN[7:0] signal content
> +      * is latched internally as the current test code. Test data is
> +      * programmed internally by rising edge on TESTCLK.
> +      */
> +     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_TESTCLK | PHY_UNTESTCLR);
> +
> +     dsi_write(dsi, DSI_PHY_TST_CTRL1, PHY_TESTEN | PHY_TESTDOUT(0) |
> +                                       PHY_TESTDIN(test_code));
> +
> +     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_UNTESTCLK | PHY_UNTESTCLR);
> +
> +     dsi_write(dsi, DSI_PHY_TST_CTRL1, PHY_UNTESTEN | PHY_TESTDOUT(0) |
> +                                       PHY_TESTDIN(test_data));
> +
> +     dsi_write(dsi, DSI_PHY_TST_CTRL0, PHY_TESTCLK | PHY_UNTESTCLR);
> +}
> +
> +/**
> + * ns2bc - Nanoseconds to byte clock cycles
> + */
> +static inline unsigned int ns2bc(struct dw_mipi_dsi_rockchip *dsi, int ns)
> +{
> +     return DIV_ROUND_UP(ns * dsi->lane_mbps / 8, 1000);
> +}
> +
> +/**
> + * ns2ui - Nanoseconds to UI time periods
> + */
> +static inline unsigned int ns2ui(struct dw_mipi_dsi_rockchip *dsi, int ns)
> +{
> +     return DIV_ROUND_UP(ns * dsi->lane_mbps, 1000);
> +}
> +
> +static int dw_mipi_dsi_phy_init(void *priv_data)
> +{
> +     struct dw_mipi_dsi_rockchip *dsi = priv_data;
> +     int ret, i, vco;
> +
> +     vco = (dsi->lane_mbps < 200) ? 0 : (dsi->lane_mbps + 100) / 200;

Please add a clarifying comment as requested by Sean on
https://chromium-review.googlesource.com/#/c/chromiumos/third_party/kernel/+/780120/

> +     i = max_mbps_to_parameter(dsi->lane_mbps);
> +     if (i < 0) {
> +             DRM_DEV_ERROR(dsi->dev,
> +                           "failed to get parameter for %dmbps clock\n",
> +                           dsi->lane_mbps);
> +             return i;
> +     }
> +
> +     ret = clk_prepare_enable(dsi->phy_cfg_clk);
> +     if (ret) {
> +             DRM_DEV_ERROR(dsi->dev, "Failed to enable phy_cfg_clk\n");
> +             return ret;
> +     }
> +
> +     dw_mipi_dsi_phy_write(dsi, PLL_BIAS_CUR_SEL_CAP_VCO_CONTROL,
> +                           BYPASS_VCO_RANGE |
> +                           VCO_RANGE_CON_SEL(vco) |
> +                           VCO_IN_CAP_CON_LOW |
> +                           REF_BIAS_CUR_SEL);
> +
> +     dw_mipi_dsi_phy_write(dsi, PLL_CP_CONTROL_PLL_LOCK_BYPASS,
> +                           CP_CURRENT_SEL(dppa_map[i].icpctrl));
> +     dw_mipi_dsi_phy_write(dsi, PLL_LPF_AND_CP_CONTROL,
> +                           CP_PROGRAM_EN | LPF_PROGRAM_EN |
> +                           LPF_RESISTORS_SEL(dppa_map[i].lpfctrl));
> +
> +     dw_mipi_dsi_phy_write(dsi, HS_RX_CONTROL_OF_LANE_0,
> +                           HSFREQRANGE_SEL(dppa_map[i].hsfreqrange));
> +
> +     dw_mipi_dsi_phy_write(dsi, PLL_INPUT_DIVIDER_RATIO,
> +                           INPUT_DIVIDER(dsi->input_div));
> +     dw_mipi_dsi_phy_write(dsi, PLL_LOOP_DIVIDER_RATIO,
> +                           LOOP_DIV_LOW_SEL(dsi->feedback_div) |
> +                           LOW_PROGRAM_EN);
> +     /*
> +      * We need set PLL_INPUT_AND_LOOP_DIVIDER_RATIOS_CONTROL immediately
> +      * to make the configured LSB effective according to IP simulation
> +      * and lab test results.
> +      * Only in this way can we get correct mipi phy pll frequency.
> +      */
> +     dw_mipi_dsi_phy_write(dsi, PLL_INPUT_AND_LOOP_DIVIDER_RATIOS_CONTROL,
> +                           PLL_LOOP_DIV_EN | PLL_INPUT_DIV_EN);
> +     dw_mipi_dsi_phy_write(dsi, PLL_LOOP_DIVIDER_RATIO,
> +                           LOOP_DIV_HIGH_SEL(dsi->feedback_div) |
> +                           HIGH_PROGRAM_EN);
> +     dw_mipi_dsi_phy_write(dsi, PLL_INPUT_AND_LOOP_DIVIDER_RATIOS_CONTROL,
> +                           PLL_LOOP_DIV_EN | PLL_INPUT_DIV_EN);
> +
> +     dw_mipi_dsi_phy_write(dsi, AFE_BIAS_BANDGAP_ANALOG_PROGRAMMABILITY,
> +                           LOW_PROGRAM_EN | BIASEXTR_SEL(BIASEXTR_127_7));
> +     dw_mipi_dsi_phy_write(dsi, AFE_BIAS_BANDGAP_ANALOG_PROGRAMMABILITY,
> +                           HIGH_PROGRAM_EN | BANDGAP_SEL(BANDGAP_96_10));
> +
> +     dw_mipi_dsi_phy_write(dsi, BANDGAP_AND_BIAS_CONTROL,
> +                           POWER_CONTROL | INTERNAL_REG_CURRENT |
> +                           BIAS_BLOCK_ON | BANDGAP_ON);
> +
> +     dw_mipi_dsi_phy_write(dsi, TERMINATION_RESISTER_CONTROL,
> +                           TER_RESISTOR_LOW | TER_CAL_DONE |
> +                           SETRD_MAX | TER_RESISTORS_ON);
> +     dw_mipi_dsi_phy_write(dsi, TERMINATION_RESISTER_CONTROL,
> +                           TER_RESISTOR_HIGH | LEVEL_SHIFTERS_ON |
> +                           SETRD_MAX | POWER_MANAGE |
> +                           TER_RESISTORS_ON);
> +
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_CLOCK_LANE_REQUEST_STATE_TIME_CONTROL,
> +                           TLP_PROGRAM_EN | ns2bc(dsi, 500));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_CLOCK_LANE_PREPARE_STATE_TIME_CONTROL,
> +                           THS_PRE_PROGRAM_EN | ns2ui(dsi, 40));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_CLOCK_LANE_HS_ZERO_STATE_TIME_CONTROL,
> +                           THS_ZERO_PROGRAM_EN | ns2bc(dsi, 300));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_CLOCK_LANE_TRAIL_STATE_TIME_CONTROL,
> +                           THS_PRE_PROGRAM_EN | ns2ui(dsi, 100));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_CLOCK_LANE_EXIT_STATE_TIME_CONTROL,
> +                           BIT(5) | ns2bc(dsi, 100));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_CLOCK_LANE_POST_TIME_CONTROL,
> +                           BIT(5) | (ns2bc(dsi, 60) + 7));
> +
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_DATA_LANE_REQUEST_STATE_TIME_CONTROL,
> +                           TLP_PROGRAM_EN | ns2bc(dsi, 500));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_DATA_LANE_PREPARE_STATE_TIME_CONTROL,
> +                           THS_PRE_PROGRAM_EN | (ns2ui(dsi, 50) + 20));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_DATA_LANE_HS_ZERO_STATE_TIME_CONTROL,
> +                           THS_ZERO_PROGRAM_EN | (ns2bc(dsi, 140) + 2));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_DATA_LANE_TRAIL_STATE_TIME_CONTROL,
> +                           THS_PRE_PROGRAM_EN | (ns2ui(dsi, 60) + 8));
> +     dw_mipi_dsi_phy_write(dsi, HS_TX_DATA_LANE_EXIT_STATE_TIME_CONTROL,
> +                           BIT(5) | ns2bc(dsi, 100));
> +
> +     clk_disable_unprepare(dsi->phy_cfg_clk);
> +
> +     return ret;
> +}
> +
> +static int
> +dw_mipi_dsi_get_lane_mbps(void *priv_data, struct drm_display_mode *mode,
> +                       unsigned long mode_flags, u32 lanes, u32 format,
> +                       unsigned int *lane_mbps)
> +{
> +     struct dw_mipi_dsi_rockchip *dsi = priv_data;
> +     int bpp;
> +     unsigned long mpclk, tmp;
> +     unsigned int target_mbps = 1000;
> +     unsigned int max_mbps = dppa_map[ARRAY_SIZE(dppa_map) - 1].max_mbps;
> +     unsigned long best_freq = 0;
> +     unsigned long fvco_min, fvco_max, fin, fout;
> +     unsigned int min_prediv, max_prediv;
> +     unsigned int _prediv, uninitialized_var(best_prediv);
> +     unsigned long _fbdiv, uninitialized_var(best_fbdiv);
> +     unsigned long min_delta = ULONG_MAX;
> +
> +     dsi->format = format;
> +     bpp = mipi_dsi_pixel_format_to_bpp(dsi->format);
> +     if (bpp < 0) {
> +             DRM_DEV_ERROR(dsi->dev,
> +                           "failed to get bpp for pixel format %d\n",
> +                           dsi->format);
> +             return bpp;
> +     }
> +
> +     mpclk = DIV_ROUND_UP(mode->clock, MSEC_PER_SEC);
> +     if (mpclk) {
> +             /* take 1 / 0.8, since mbps must big than bandwidth of RGB */
> +             tmp = mpclk * (bpp / lanes) * 10 / 8;
> +             if (tmp < max_mbps)
> +                     target_mbps = tmp;
> +             else
> +                     DRM_DEV_ERROR(dsi->dev,
> +                                   "DPHY clock frequency is out of range\n");
> +     }
> +
> +     fin = clk_get_rate(dsi->pllref_clk);
> +     fout = target_mbps * USEC_PER_SEC;
> +
> +     /* constraint: 5Mhz <= Fref / N <= 40MHz */
> +     min_prediv = DIV_ROUND_UP(fin, 40 * USEC_PER_SEC);
> +     max_prediv = fin / (5 * USEC_PER_SEC);
> +
> +     /* constraint: 80MHz <= Fvco <= 1500Mhz */
> +     fvco_min = 80 * USEC_PER_SEC;
> +     fvco_max = 1500 * USEC_PER_SEC;
> +
> +     for (_prediv = min_prediv; _prediv <= max_prediv; _prediv++) {
> +             u64 tmp;
> +             u32 delta;
> +             /* Fvco = Fref * M / N */
> +             tmp = (u64)fout * _prediv;
> +             do_div(tmp, fin);
> +             _fbdiv = tmp;
> +             /*
> +              * Due to the use of a "by 2 pre-scaler," the range of the
> +              * feedback multiplication value M is limited to even division
> +              * numbers, and m must be greater than 6, less than 512.
> +              */

It seems this should be "not bigger than 512" or something similar.

> +             if (_fbdiv < 6 || _fbdiv > 512)
> +                     continue;
> +
> +             _fbdiv += _fbdiv % 2;
> +
> +             tmp = (u64)_fbdiv * fin;
> +             do_div(tmp, _prediv);

Should we bail out early if min_prediv == 0 due to some bogus
configuration of pllref_clk?

> +             if (tmp < fvco_min || tmp > fvco_max)
> +                     continue;
> +
> +             delta = abs(fout - tmp);
> +             if (delta < min_delta) {
> +                     best_prediv = _prediv;
> +                     best_fbdiv = _fbdiv;
> +                     min_delta = delta;
> +                     best_freq = tmp;
> +             }
> +     }
> +
> +     if (best_freq) {
> +             dsi->lane_mbps = DIV_ROUND_UP(best_freq, USEC_PER_SEC);
> +             *lane_mbps = dsi->lane_mbps;
> +             dsi->input_div = best_prediv;
> +             dsi->feedback_div = best_fbdiv;
> +     } else {
> +             DRM_DEV_ERROR(dsi->dev, "Can not find best_freq for DPHY\n");

return -1;

> +     }
> +
> +     return 0;
> +}
> +
> +static const struct dw_mipi_dsi_phy_ops dw_mipi_dsi_rockchip_phy_ops = {
> +     .init = dw_mipi_dsi_phy_init,
> +     .get_lane_mbps = dw_mipi_dsi_get_lane_mbps,
> +};
> +
> +static void dw_mipi_dsi_encoder_mode_set(struct drm_encoder *encoder,
> +                                      struct drm_display_mode *mode,
> +                                      struct drm_display_mode *adjusted)
> +{
> +     struct dw_mipi_dsi_rockchip *dsi = to_dsi(encoder);
> +     const struct rockchip_dw_dsi_chip_data *cdata = dsi->cdata;
> +     int val, ret, mux;
> +
> +     mux = drm_of_encoder_active_endpoint_id(dsi->dev->of_node,
> +                                             &dsi->encoder);
> +     if (mux < 0)
> +             return;
> +     /*
> +      * For the RK3399, the clk of grf must be enabled before writing grf
> +      * register. And for RK3288 or other soc, this grf_clk must be NULL,
> +      * the clk_prepare_enable return true directly.
> +      */
> +     ret = clk_prepare_enable(dsi->grf_clk);
> +     if (ret) {
> +             DRM_DEV_ERROR(dsi->dev, "Failed to enable grf_clk: %d\n", ret);
> +             return;
> +     }
> +
> +     val = cdata->dsi0_en_bit << 16;
> +     if (mux)
> +             val |= cdata->dsi0_en_bit;
> +     regmap_write(dsi->grf_regmap, cdata->grf_switch_reg, val);
> +
> +     if (cdata->grf_dsi0_mode_reg)
> +             regmap_write(dsi->grf_regmap, cdata->grf_dsi0_mode_reg,
> +                          cdata->grf_dsi0_mode);
> +
> +     clk_disable_unprepare(dsi->grf_clk);
> +}
> +
> +static int
> +dw_mipi_dsi_encoder_atomic_check(struct drm_encoder *encoder,
> +                              struct drm_crtc_state *crtc_state,
> +                              struct drm_connector_state *conn_state)
> +{
> +     struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc_state);
> +     struct dw_mipi_dsi_rockchip *dsi = to_dsi(encoder);
> +
> +     switch (dsi->format) {
> +     case MIPI_DSI_FMT_RGB888:
> +             s->output_mode = ROCKCHIP_OUT_MODE_P888;
> +             break;
> +     case MIPI_DSI_FMT_RGB666:
> +             s->output_mode = ROCKCHIP_OUT_MODE_P666;
> +             break;
> +     case MIPI_DSI_FMT_RGB565:
> +             s->output_mode = ROCKCHIP_OUT_MODE_P565;
> +             break;
> +     default:
> +             WARN_ON(1);
> +             return -EINVAL;
> +     }
> +
> +     s->output_type = DRM_MODE_CONNECTOR_DSI;
> +
> +     return 0;
> +}
> +
> +static const struct drm_encoder_helper_funcs
> +dw_mipi_dsi_encoder_helper_funcs = {
> +     .mode_set = dw_mipi_dsi_encoder_mode_set,
> +     .atomic_check = dw_mipi_dsi_encoder_atomic_check,
> +};
> +
> +static const struct drm_encoder_funcs dw_mipi_dsi_encoder_funcs = {
> +     .destroy = drm_encoder_cleanup,
> +};
> +
> +static int rockchip_dsi_drm_create_encoder(struct dw_mipi_dsi_rockchip *dsi,
> +                                        struct drm_device *drm_dev)
> +{
> +     struct drm_encoder *encoder = &dsi->encoder;
> +     int ret;
> +
> +     encoder->possible_crtcs = drm_of_find_possible_crtcs(drm_dev,
> +                                                          dsi->dev->of_node);
> +
> +     ret = drm_encoder_init(drm_dev, encoder, &dw_mipi_dsi_encoder_funcs,
> +                            DRM_MODE_ENCODER_DSI, NULL);
> +     if (ret) {
> +             DRM_ERROR("Failed to initialize encoder with drm\n");
> +             return ret;
> +     }
> +
> +     drm_encoder_helper_add(encoder, &dw_mipi_dsi_encoder_helper_funcs);
> +
> +     return 0;
> +}
> +
> +static int dw_mipi_dsi_rockchip_bind(struct device *dev,
> +                                  struct device *master,
> +                                  void *data)
> +{
> +     struct dw_mipi_dsi_rockchip *dsi = dev_get_drvdata(dev);
> +     struct platform_device *pdev = to_platform_device(dev);
> +     struct drm_device *drm_dev = data;
> +     int ret;
> +
> +     dsi->pdata.base = dsi->base;
> +     dsi->pdata.max_data_lanes = dsi->cdata->max_data_lanes;
> +     dsi->pdata.phy_ops = &dw_mipi_dsi_rockchip_phy_ops;
> +     dsi->pdata.priv_data = dsi;
> +
> +     ret = clk_prepare_enable(dsi->pllref_clk);
> +     if (ret) {
> +             DRM_DEV_ERROR(dev,
> +                           "%s: Failed to enable pllref_clk\n", __func__);
> +             return ret;
> +     }
> +
> +     ret = rockchip_dsi_drm_create_encoder(dsi, drm_dev);
> +     if (ret) {
> +             DRM_ERROR("Failed to create drm encoder\n");
> +             goto err_pllref;
> +     }
> +
> +     dsi->dmd = dw_mipi_dsi_bind(pdev, &dsi->encoder, &dsi->pdata);
> +     if (IS_ERR(dsi->dmd)) {
> +             DRM_ERROR("Failed to bind\n");
> +             ret = PTR_ERR(dsi->dmd);
> +             goto err_pllref;
> +     }
> +     return 0;
> +
> +err_pllref:
> +     clk_disable_unprepare(dsi->pllref_clk);
> +     return ret;
> +}
> +
> +static void dw_mipi_dsi_rockchip_unbind(struct device *dev,
> +                                     struct device *master,
> +                                     void *data)
> +{
> +     struct dw_mipi_dsi_rockchip *dsi = dev_get_drvdata(dev);
> +
> +     dw_mipi_dsi_unbind(dsi->dmd);
> +     clk_disable_unprepare(dsi->pllref_clk);
> +}
> +
> +static const struct component_ops dw_mipi_dsi_rockchip_ops = {
> +     .bind   = dw_mipi_dsi_rockchip_bind,
> +     .unbind = dw_mipi_dsi_rockchip_unbind,
> +};
> +
> +static int dw_mipi_dsi_rockchip_probe(struct platform_device *pdev)
> +{
> +     struct device *dev = &pdev->dev;
> +     struct device_node *np = dev->of_node;
> +     struct dw_mipi_dsi_rockchip *dsi;
> +     struct resource *res;
> +     const struct rockchip_dw_dsi_chip_data *cdata =
> +                             of_device_get_match_data(dev);
> +     int ret;
> +
> +     dsi = devm_kzalloc(dev, sizeof(*dsi), GFP_KERNEL);
> +     if (!dsi)
> +             return -ENOMEM;
> +
> +     res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> +     if (!res) {
> +             DRM_ERROR("Unable to get resource\n");
> +             return -ENODEV;
> +     }
> +
> +     dsi->base = devm_ioremap_resource(dev, res);
> +     if (IS_ERR(dsi->base)) {
> +             DRM_ERROR("Unable to get dsi registers\n");
> +             return PTR_ERR(dsi->base);
> +     }
> +
> +     dsi->pllref_clk = devm_clk_get(dev, "ref");
> +     if (IS_ERR(dsi->pllref_clk)) {
> +             ret = PTR_ERR(dsi->pllref_clk);
> +             DRM_DEV_ERROR(dev,
> +                           "Unable to get pll reference clock: %d\n", ret);
> +             return ret;
> +     }
> +
> +     if (cdata->flags & DW_MIPI_NEEDS_PHY_CFG_CLK) {
> +             dsi->phy_cfg_clk = devm_clk_get(dev, "phy_cfg");
> +             if (IS_ERR(dsi->phy_cfg_clk)) {
> +                     ret = PTR_ERR(dsi->phy_cfg_clk);
> +                     DRM_DEV_ERROR(dev,
> +                                   "Unable to get phy_cfg_clk: %d\n", ret);
> +                     return ret;
> +             }
> +     }
> +
> +     if (cdata->flags & DW_MIPI_NEEDS_GRF_CLK) {
> +             dsi->grf_clk = devm_clk_get(dev, "grf");
> +             if (IS_ERR(dsi->grf_clk)) {
> +                     ret = PTR_ERR(dsi->grf_clk);
> +                     DRM_DEV_ERROR(dev, "Unable to get grf_clk: %d\n", ret);
> +                     return ret;
> +             }
> +     }
> +
> +     dsi->grf_regmap = syscon_regmap_lookup_by_phandle(np, "rockchip,grf");
> +     if (IS_ERR(dsi->grf_regmap)) {
> +             DRM_DEV_ERROR(dsi->dev, "Unable to get rockchip,grf\n");
> +             return PTR_ERR(dsi->grf_regmap);
> +     }
> +
> +     dsi->cdata = cdata;
> +     dsi->dev = dev;
> +     dev_set_drvdata(dev, dsi);
> +
> +     return component_add(&pdev->dev, &dw_mipi_dsi_rockchip_ops);
> +}
> +
> +static int dw_mipi_dsi_rockchip_remove(struct platform_device *pdev)
> +{
> +     component_del(&pdev->dev, &dw_mipi_dsi_rockchip_ops);
> +
> +     return 0;
> +}
> +
> +static struct rockchip_dw_dsi_chip_data rk3288_chip_data = {
> +     .dsi0_en_bit = RK3288_DSI0_SEL_VOP_LIT,
> +     .dsi1_en_bit = RK3288_DSI1_SEL_VOP_LIT,
> +     .grf_switch_reg = RK3288_GRF_SOC_CON6,
> +     .max_data_lanes = 4,
> +};
> +
> +static struct rockchip_dw_dsi_chip_data rk3399_chip_data = {
> +     .dsi0_en_bit = RK3399_DSI0_SEL_VOP_LIT,
> +     .dsi1_en_bit = RK3399_DSI1_SEL_VOP_LIT,
> +     .grf_switch_reg = RK3399_GRF_SOC_CON20,
> +     .grf_dsi0_mode = RK3399_GRF_DSI_MODE,
> +     .grf_dsi0_mode_reg = RK3399_GRF_SOC_CON22,
> +     .flags = DW_MIPI_NEEDS_PHY_CFG_CLK | DW_MIPI_NEEDS_GRF_CLK,
> +     .max_data_lanes = 4,
> +};
> +
> +static const struct of_device_id dw_mipi_dsi_rockchip_dt_ids[] = {
> +     {
> +      .compatible = "rockchip,rk3288-mipi-dsi",
> +      .data = &rk3288_chip_data,
> +     }, {
> +      .compatible = "rockchip,rk3399-mipi-dsi",
> +      .data = &rk3399_chip_data,
> +     },
> +     { /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, dw_mipi_dsi_rockchip_dt_ids);
> +
> +struct platform_driver dw_mipi_dsi_rockchip_driver = {
> +     .probe          = dw_mipi_dsi_rockchip_probe,
> +     .remove         = dw_mipi_dsi_rockchip_remove,
> +     .driver         = {
> +             .of_match_table = dw_mipi_dsi_rockchip_dt_ids,
> +             .name   = "dw-mipi-dsi-rockchip",
> +     },
> +};
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c 
> b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c
> index 76d63de..d40cc39 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.c
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.c
> @@ -462,7 +462,7 @@ static int __init rockchip_drm_init(void)
>       ADD_ROCKCHIP_SUB_DRIVER(cdn_dp_driver, CONFIG_ROCKCHIP_CDN_DP);
>       ADD_ROCKCHIP_SUB_DRIVER(dw_hdmi_rockchip_pltfm_driver,
>                               CONFIG_ROCKCHIP_DW_HDMI);
> -     ADD_ROCKCHIP_SUB_DRIVER(dw_mipi_dsi_driver,
> +     ADD_ROCKCHIP_SUB_DRIVER(dw_mipi_dsi_rockchip_driver,
>                               CONFIG_ROCKCHIP_DW_MIPI_DSI);
>       ADD_ROCKCHIP_SUB_DRIVER(inno_hdmi_driver, CONFIG_ROCKCHIP_INNO_HDMI);
>  
> diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_drv.h 
> b/drivers/gpu/drm/rockchip/rockchip_drm_drv.h
> index 498dfbc..af235b2 100644
> --- a/drivers/gpu/drm/rockchip/rockchip_drm_drv.h
> +++ b/drivers/gpu/drm/rockchip/rockchip_drm_drv.h
> @@ -66,7 +66,7 @@ void rockchip_drm_dma_detach_device(struct drm_device 
> *drm_dev,
>  
>  extern struct platform_driver cdn_dp_driver;
>  extern struct platform_driver dw_hdmi_rockchip_pltfm_driver;
> -extern struct platform_driver dw_mipi_dsi_driver;
> +extern struct platform_driver dw_mipi_dsi_rockchip_driver;
>  extern struct platform_driver inno_hdmi_driver;
>  extern struct platform_driver rockchip_dp_driver;
>  extern struct platform_driver rockchip_lvds_driver;

Reply via email to