On Wed, Sep 16, 2020 at 10:58:33AM +0200, Michael Tretter wrote:
> On Wed, 16 Sep 2020 09:58:39 +0200, Daniel Vetter wrote:
> > On Fri, Sep 11, 2020 at 03:54:13PM +0200, Michael Tretter wrote:
> > > As the driver is not platform dependent anymore, move it to the drm
> > > bridge driver directory.
> > > 
> > > Signed-off-by: Michael Tretter <m.tret...@pengutronix.de>
> > 
> > So new drm_bridge drivers that still use the component stuff is a bit
> > uncool. We're trying to get away from that everywhere, bridges should be
> > abstracted enough that just going through the of lookup functions to get
> > at your bridge should work.
> > 
> > Is there anything here that prevents this, or could this be included?
> 
> The Exynos drm driver uses the component framework. Maybe I can avoid exposing
> the bind/unbind API in the drm_bridge driver by implementing the bind/unbind
> in the platform specific part. However, completely getting rid of the
> component framework for this drm_bridge is not possible without changing the
> entire Exynos drm driver.

You can combine the component framework for internal stuff with
of_drm_find_bridge() for bridges. That's at least supposed to work.

I guess the of_drm_find_bridge would need to be stuffed in one of the
existing bind hooks (maybe the master one), since having a component for
the bridge and using the of_ function doesn't really make sense I think.

I guess another option would be that instead of adding the bridge
component when you create your match, call of_drm_find_bridge, but at that
point you don't yet have the drm_device nor drm_encoder.
-Daniel

> 
> Michael
> 
> > -Daniel
> > 
> > > ---
> > > v2:
> > > - select DRM_SAMSUNG_DSIM from DRM_EXYNOS_DSI
> > > - add removal of depends on !FB_S3C
> > > ---
> > >  drivers/gpu/drm/bridge/Kconfig                |    9 +
> > >  drivers/gpu/drm/bridge/Makefile               |    1 +
> > >  drivers/gpu/drm/bridge/samsung-dsim.c         | 1790 ++++++++++++++++
> > >  drivers/gpu/drm/exynos/Kconfig                |    5 +-
> > >  drivers/gpu/drm/exynos/Makefile               |    2 +-
> > >  drivers/gpu/drm/exynos/exynos_drm_dsi.c       | 1896 ++---------------
> > >  drivers/gpu/drm/exynos/exynos_drm_dsi_pltfm.c |  333 ---
> > >  .../drm/bridge/samsung-dsim.h                 |   20 +-
> > >  8 files changed, 2037 insertions(+), 2019 deletions(-)
> > >  create mode 100644 drivers/gpu/drm/bridge/samsung-dsim.c
> > >  delete mode 100644 drivers/gpu/drm/exynos/exynos_drm_dsi_pltfm.c
> > >  rename drivers/gpu/drm/exynos/exynos_drm_dsi.h => 
> > > include/drm/bridge/samsung-dsim.h (69%)
> > > 
> > > diff --git a/drivers/gpu/drm/bridge/Kconfig 
> > > b/drivers/gpu/drm/bridge/Kconfig
> > > index 3e11af4e9f63..55ab5030c6cf 100644
> > > --- a/drivers/gpu/drm/bridge/Kconfig
> > > +++ b/drivers/gpu/drm/bridge/Kconfig
> > > @@ -125,6 +125,15 @@ config DRM_PARADE_PS8640
> > >     The PS8640 is a high-performance and low-power
> > >     MIPI DSI to eDP converter
> > >  
> > > +config DRM_SAMSUNG_DSIM
> > > + tristate "Samsung MIPI DSI bridge"
> > > + depends on OF
> > > + select DRM_KMS_HELPER
> > > + select DRM_MIPI_DSI
> > > + select DRM_PANEL
> > > + help
> > > +   Samsung MIPI DSI bridge driver.
> > > +
> > >  config DRM_SIL_SII8620
> > >   tristate "Silicon Image SII8620 HDMI/MHL bridge"
> > >   depends on OF
> > > diff --git a/drivers/gpu/drm/bridge/Makefile 
> > > b/drivers/gpu/drm/bridge/Makefile
> > > index c589a6a7cbe1..5ac7a5c413dc 100644
> > > --- a/drivers/gpu/drm/bridge/Makefile
> > > +++ b/drivers/gpu/drm/bridge/Makefile
> > > @@ -8,6 +8,7 @@ obj-$(CONFIG_DRM_MEGACHIPS_STDPXXXX_GE_B850V3_FW) += 
> > > megachips-stdpxxxx-ge-b850v
> > >  obj-$(CONFIG_DRM_NXP_PTN3460) += nxp-ptn3460.o
> > >  obj-$(CONFIG_DRM_PARADE_PS8622) += parade-ps8622.o
> > >  obj-$(CONFIG_DRM_PARADE_PS8640) += parade-ps8640.o
> > > +obj-$(CONFIG_DRM_SAMSUNG_DSIM) += samsung-dsim.o
> > >  obj-$(CONFIG_DRM_SIL_SII8620) += sil-sii8620.o
> > >  obj-$(CONFIG_DRM_SII902X) += sii902x.o
> > >  obj-$(CONFIG_DRM_SII9234) += sii9234.o
> > > diff --git a/drivers/gpu/drm/bridge/samsung-dsim.c 
> > > b/drivers/gpu/drm/bridge/samsung-dsim.c
> > > new file mode 100644
> > > index 000000000000..6d2d8dc027de
> > > --- /dev/null
> > > +++ b/drivers/gpu/drm/bridge/samsung-dsim.c
> > > @@ -0,0 +1,1790 @@
> > > +// SPDX-License-Identifier: GPL-2.0-only
> > > +/*
> > > + * Samsung SoC MIPI DSI Master driver.
> > > + *
> > > + * Copyright (c) 2014 Samsung Electronics Co., Ltd
> > > + *
> > > + * Contacts: Tomasz Figa <t.f...@samsung.com>
> > > +*/
> > > +
> > > +#include <linux/clk.h>
> > > +#include <linux/delay.h>
> > > +#include <linux/component.h>
> > > +#include <linux/gpio/consumer.h>
> > > +#include <linux/irq.h>
> > > +#include <linux/of_device.h>
> > > +#include <linux/of_gpio.h>
> > > +#include <linux/of_graph.h>
> > > +#include <linux/phy/phy.h>
> > > +#include <linux/regulator/consumer.h>
> > > +
> > > +#include <asm/unaligned.h>
> > > +
> > > +#include <video/mipi_display.h>
> > > +#include <video/videomode.h>
> > > +
> > > +#include <drm/bridge/samsung-dsim.h>
> > > +#include <drm/drm_atomic_helper.h>
> > > +#include <drm/drm_bridge.h>
> > > +#include <drm/drm_fb_helper.h>
> > > +#include <drm/drm_mipi_dsi.h>
> > > +#include <drm/drm_panel.h>
> > > +#include <drm/drm_print.h>
> > > +#include <drm/drm_probe_helper.h>
> > > +#include <drm/drm_simple_kms_helper.h>
> > > +
> > > +/* returns true iff both arguments logically differs */
> > > +#define NEQV(a, b) (!(a) ^ !(b))
> > > +
> > > +/* DSIM_STATUS */
> > > +#define DSIM_STOP_STATE_DAT(x)           (((x) & 0xf) << 0)
> > > +#define DSIM_STOP_STATE_CLK              (1 << 8)
> > > +#define DSIM_TX_READY_HS_CLK             (1 << 10)
> > > +#define DSIM_PLL_STABLE                  (1 << 31)
> > > +
> > > +/* DSIM_TIMEOUT */
> > > +#define DSIM_LPDR_TIMEOUT(x)             ((x) << 0)
> > > +#define DSIM_BTA_TIMEOUT(x)              ((x) << 16)
> > > +
> > > +/* DSIM_CLKCTRL */
> > > +#define DSIM_ESC_PRESCALER(x)            (((x) & 0xffff) << 0)
> > > +#define DSIM_ESC_PRESCALER_MASK          (0xffff << 0)
> > > +#define DSIM_LANE_ESC_CLK_EN_CLK (1 << 19)
> > > +#define DSIM_LANE_ESC_CLK_EN_DATA(x)     (((x) & 0xf) << 20)
> > > +#define DSIM_LANE_ESC_CLK_EN_DATA_MASK   (0xf << 20)
> > > +#define DSIM_BYTE_CLKEN                  (1 << 24)
> > > +#define DSIM_BYTE_CLK_SRC(x)             (((x) & 0x3) << 25)
> > > +#define DSIM_BYTE_CLK_SRC_MASK           (0x3 << 25)
> > > +#define DSIM_PLL_BYPASS                  (1 << 27)
> > > +#define DSIM_ESC_CLKEN                   (1 << 28)
> > > +#define DSIM_TX_REQUEST_HSCLK            (1 << 31)
> > > +
> > > +/* DSIM_CONFIG */
> > > +#define DSIM_LANE_EN_CLK         (1 << 0)
> > > +#define DSIM_LANE_EN(x)                  (((x) & 0xf) << 1)
> > > +#define DSIM_NUM_OF_DATA_LANE(x) (((x) & 0x3) << 5)
> > > +#define DSIM_SUB_PIX_FORMAT(x)           (((x) & 0x7) << 8)
> > > +#define DSIM_MAIN_PIX_FORMAT_MASK        (0x7 << 12)
> > > +#define DSIM_MAIN_PIX_FORMAT_RGB888      (0x7 << 12)
> > > +#define DSIM_MAIN_PIX_FORMAT_RGB666      (0x6 << 12)
> > > +#define DSIM_MAIN_PIX_FORMAT_RGB666_P    (0x5 << 12)
> > > +#define DSIM_MAIN_PIX_FORMAT_RGB565      (0x4 << 12)
> > > +#define DSIM_SUB_VC                      (((x) & 0x3) << 16)
> > > +#define DSIM_MAIN_VC                     (((x) & 0x3) << 18)
> > > +#define DSIM_HSA_MODE                    (1 << 20)
> > > +#define DSIM_HBP_MODE                    (1 << 21)
> > > +#define DSIM_HFP_MODE                    (1 << 22)
> > > +#define DSIM_HSE_MODE                    (1 << 23)
> > > +#define DSIM_AUTO_MODE                   (1 << 24)
> > > +#define DSIM_VIDEO_MODE                  (1 << 25)
> > > +#define DSIM_BURST_MODE                  (1 << 26)
> > > +#define DSIM_SYNC_INFORM         (1 << 27)
> > > +#define DSIM_EOT_DISABLE         (1 << 28)
> > > +#define DSIM_MFLUSH_VS                   (1 << 29)
> > > +/* This flag is valid only for exynos3250/3472/5260/5430 */
> > > +#define DSIM_CLKLANE_STOP                (1 << 30)
> > > +
> > > +/* DSIM_ESCMODE */
> > > +#define DSIM_TX_TRIGGER_RST              (1 << 4)
> > > +#define DSIM_TX_LPDT_LP                  (1 << 6)
> > > +#define DSIM_CMD_LPDT_LP         (1 << 7)
> > > +#define DSIM_FORCE_BTA                   (1 << 16)
> > > +#define DSIM_FORCE_STOP_STATE            (1 << 20)
> > > +#define DSIM_STOP_STATE_CNT(x)           (((x) & 0x7ff) << 21)
> > > +#define DSIM_STOP_STATE_CNT_MASK (0x7ff << 21)
> > > +
> > > +/* DSIM_MDRESOL */
> > > +#define DSIM_MAIN_STAND_BY               (1 << 31)
> > > +#define DSIM_MAIN_VRESOL(x, num_bits)    (((x) & ((1 << (num_bits)) - 
> > > 1)) << 16)
> > > +#define DSIM_MAIN_HRESOL(x, num_bits)    (((x) & ((1 << (num_bits)) - 
> > > 1)) << 0)
> > > +
> > > +/* DSIM_MVPORCH */
> > > +#define DSIM_CMD_ALLOW(x)                ((x) << 28)
> > > +#define DSIM_STABLE_VFP(x)               ((x) << 16)
> > > +#define DSIM_MAIN_VBP(x)         ((x) << 0)
> > > +#define DSIM_CMD_ALLOW_MASK              (0xf << 28)
> > > +#define DSIM_STABLE_VFP_MASK             (0x7ff << 16)
> > > +#define DSIM_MAIN_VBP_MASK               (0x7ff << 0)
> > > +
> > > +/* DSIM_MHPORCH */
> > > +#define DSIM_MAIN_HFP(x)         ((x) << 16)
> > > +#define DSIM_MAIN_HBP(x)         ((x) << 0)
> > > +#define DSIM_MAIN_HFP_MASK               ((0xffff) << 16)
> > > +#define DSIM_MAIN_HBP_MASK               ((0xffff) << 0)
> > > +
> > > +/* DSIM_MSYNC */
> > > +#define DSIM_MAIN_VSA(x)         ((x) << 22)
> > > +#define DSIM_MAIN_HSA(x)         ((x) << 0)
> > > +#define DSIM_MAIN_VSA_MASK               ((0x3ff) << 22)
> > > +#define DSIM_MAIN_HSA_MASK               ((0xffff) << 0)
> > > +
> > > +/* DSIM_SDRESOL */
> > > +#define DSIM_SUB_STANDY(x)               ((x) << 31)
> > > +#define DSIM_SUB_VRESOL(x)               ((x) << 16)
> > > +#define DSIM_SUB_HRESOL(x)               ((x) << 0)
> > > +#define DSIM_SUB_STANDY_MASK             ((0x1) << 31)
> > > +#define DSIM_SUB_VRESOL_MASK             ((0x7ff) << 16)
> > > +#define DSIM_SUB_HRESOL_MASK             ((0x7ff) << 0)
> > > +
> > > +/* DSIM_INTSRC */
> > > +#define DSIM_INT_PLL_STABLE              (1 << 31)
> > > +#define DSIM_INT_SW_RST_RELEASE          (1 << 30)
> > > +#define DSIM_INT_SFR_FIFO_EMPTY          (1 << 29)
> > > +#define DSIM_INT_SFR_HDR_FIFO_EMPTY      (1 << 28)
> > > +#define DSIM_INT_BTA                     (1 << 25)
> > > +#define DSIM_INT_FRAME_DONE              (1 << 24)
> > > +#define DSIM_INT_RX_TIMEOUT              (1 << 21)
> > > +#define DSIM_INT_BTA_TIMEOUT             (1 << 20)
> > > +#define DSIM_INT_RX_DONE         (1 << 18)
> > > +#define DSIM_INT_RX_TE                   (1 << 17)
> > > +#define DSIM_INT_RX_ACK                  (1 << 16)
> > > +#define DSIM_INT_RX_ECC_ERR              (1 << 15)
> > > +#define DSIM_INT_RX_CRC_ERR              (1 << 14)
> > > +
> > > +/* DSIM_FIFOCTRL */
> > > +#define DSIM_RX_DATA_FULL                (1 << 25)
> > > +#define DSIM_RX_DATA_EMPTY               (1 << 24)
> > > +#define DSIM_SFR_HEADER_FULL             (1 << 23)
> > > +#define DSIM_SFR_HEADER_EMPTY            (1 << 22)
> > > +#define DSIM_SFR_PAYLOAD_FULL            (1 << 21)
> > > +#define DSIM_SFR_PAYLOAD_EMPTY           (1 << 20)
> > > +#define DSIM_I80_HEADER_FULL             (1 << 19)
> > > +#define DSIM_I80_HEADER_EMPTY            (1 << 18)
> > > +#define DSIM_I80_PAYLOAD_FULL            (1 << 17)
> > > +#define DSIM_I80_PAYLOAD_EMPTY           (1 << 16)
> > > +#define DSIM_SD_HEADER_FULL              (1 << 15)
> > > +#define DSIM_SD_HEADER_EMPTY             (1 << 14)
> > > +#define DSIM_SD_PAYLOAD_FULL             (1 << 13)
> > > +#define DSIM_SD_PAYLOAD_EMPTY            (1 << 12)
> > > +#define DSIM_MD_HEADER_FULL              (1 << 11)
> > > +#define DSIM_MD_HEADER_EMPTY             (1 << 10)
> > > +#define DSIM_MD_PAYLOAD_FULL             (1 << 9)
> > > +#define DSIM_MD_PAYLOAD_EMPTY            (1 << 8)
> > > +#define DSIM_RX_FIFO                     (1 << 4)
> > > +#define DSIM_SFR_FIFO                    (1 << 3)
> > > +#define DSIM_I80_FIFO                    (1 << 2)
> > > +#define DSIM_SD_FIFO                     (1 << 1)
> > > +#define DSIM_MD_FIFO                     (1 << 0)
> > > +
> > > +/* DSIM_PHYACCHR */
> > > +#define DSIM_AFC_EN                      (1 << 14)
> > > +#define DSIM_AFC_CTL(x)                  (((x) & 0x7) << 5)
> > > +
> > > +/* DSIM_PLLCTRL */
> > > +#define DSIM_FREQ_BAND(x)                ((x) << 24)
> > > +#define DSIM_PLL_EN                      (1 << 23)
> > > +#define DSIM_PLL_P(x)                    ((x) << 13)
> > > +#define DSIM_PLL_M(x)                    ((x) << 4)
> > > +#define DSIM_PLL_S(x)                    ((x) << 1)
> > > +
> > > +/* DSIM_PHYCTRL */
> > > +#define DSIM_PHYCTRL_ULPS_EXIT(x)        (((x) & 0x1ff) << 0)
> > > +#define DSIM_PHYCTRL_B_DPHYCTL_VREG_LP   (1 << 30)
> > > +#define DSIM_PHYCTRL_B_DPHYCTL_SLEW_UP   (1 << 14)
> > > +
> > > +/* DSIM_PHYTIMING */
> > > +#define DSIM_PHYTIMING_LPX(x)            ((x) << 8)
> > > +#define DSIM_PHYTIMING_HS_EXIT(x)        ((x) << 0)
> > > +
> > > +/* DSIM_PHYTIMING1 */
> > > +#define DSIM_PHYTIMING1_CLK_PREPARE(x)   ((x) << 24)
> > > +#define DSIM_PHYTIMING1_CLK_ZERO(x)      ((x) << 16)
> > > +#define DSIM_PHYTIMING1_CLK_POST(x)      ((x) << 8)
> > > +#define DSIM_PHYTIMING1_CLK_TRAIL(x)     ((x) << 0)
> > > +
> > > +/* DSIM_PHYTIMING2 */
> > > +#define DSIM_PHYTIMING2_HS_PREPARE(x)    ((x) << 16)
> > > +#define DSIM_PHYTIMING2_HS_ZERO(x)       ((x) << 8)
> > > +#define DSIM_PHYTIMING2_HS_TRAIL(x)      ((x) << 0)
> > > +
> > > +#define DSI_MAX_BUS_WIDTH                4
> > > +#define DSI_NUM_VIRTUAL_CHANNELS 4
> > > +#define DSI_TX_FIFO_SIZE         2048
> > > +#define DSI_RX_FIFO_SIZE         256
> > > +#define DSI_XFER_TIMEOUT_MS              100
> > > +#define DSI_RX_FIFO_EMPTY                0x30800002
> > > +
> > > +#define OLD_SCLK_MIPI_CLK_NAME "pll_clk"
> > > +
> > > +static const char *const clk_names[5] = { "bus_clk", "sclk_mipi",
> > > + "phyclk_mipidphy0_bitclkdiv8", "phyclk_mipidphy0_rxclkesc0",
> > > + "sclk_rgb_vclk_to_dsim0" };
> > > +
> > > +enum samsung_dsim_transfer_type {
> > > + EXYNOS_DSI_TX,
> > > + EXYNOS_DSI_RX,
> > > +};
> > > +
> > > +struct samsung_dsim_transfer {
> > > + struct list_head list;
> > > + struct completion completed;
> > > + int result;
> > > + struct mipi_dsi_packet packet;
> > > + u16 flags;
> > > + u16 tx_done;
> > > +
> > > + u8 *rx_payload;
> > > + u16 rx_len;
> > > + u16 rx_done;
> > > +};
> > > +
> > > +#define DSIM_STATE_ENABLED               BIT(0)
> > > +#define DSIM_STATE_INITIALIZED           BIT(1)
> > > +#define DSIM_STATE_CMD_LPM               BIT(2)
> > > +#define DSIM_STATE_VIDOUT_AVAILABLE      BIT(3)
> > > +
> > > +struct samsung_dsim {
> > > + struct drm_bridge bridge;
> > > + struct mipi_dsi_host dsi_host;
> > > + struct drm_connector connector;
> > > + struct drm_panel *panel;
> > > + struct list_head bridge_chain;
> > > + struct drm_bridge *out_bridge;
> > > + struct device *dev;
> > > +
> > > + void __iomem *reg_base;
> > > + struct phy *phy;
> > > + struct clk **clks;
> > > + struct regulator_bulk_data supplies[2];
> > > + int irq;
> > > + int te_gpio;
> > > +
> > > + u32 pll_clk_rate;
> > > + u32 burst_clk_rate;
> > > + u32 esc_clk_rate;
> > > + u32 lanes;
> > > + u32 mode_flags;
> > > + u32 format;
> > > +
> > > + struct drm_display_mode mode;
> > > +
> > > + int state;
> > > + struct drm_property *brightness;
> > > + struct completion completed;
> > > +
> > > + spinlock_t transfer_lock; /* protects transfer_list */
> > > + struct list_head transfer_list;
> > > +
> > > + const struct samsung_dsim_driver_data *driver_data;
> > > +};
> > > +
> > > +#define host_to_dsi(host) container_of(host, struct samsung_dsim, 
> > > dsi_host)
> > > +#define connector_to_dsi(c) container_of(c, struct samsung_dsim, 
> > > connector)
> > > +
> > > +enum reg_idx {
> > > + DSIM_STATUS_REG,        /* Status register */
> > > + DSIM_SWRST_REG,         /* Software reset register */
> > > + DSIM_CLKCTRL_REG,       /* Clock control register */
> > > + DSIM_TIMEOUT_REG,       /* Time out register */
> > > + DSIM_CONFIG_REG,        /* Configuration register */
> > > + DSIM_ESCMODE_REG,       /* Escape mode register */
> > > + DSIM_MDRESOL_REG,
> > > + DSIM_MVPORCH_REG,       /* Main display Vporch register */
> > > + DSIM_MHPORCH_REG,       /* Main display Hporch register */
> > > + DSIM_MSYNC_REG,         /* Main display sync area register */
> > > + DSIM_INTSRC_REG,        /* Interrupt source register */
> > > + DSIM_INTMSK_REG,        /* Interrupt mask register */
> > > + DSIM_PKTHDR_REG,        /* Packet Header FIFO register */
> > > + DSIM_PAYLOAD_REG,       /* Payload FIFO register */
> > > + DSIM_RXFIFO_REG,        /* Read FIFO register */
> > > + DSIM_FIFOCTRL_REG,      /* FIFO status and control register */
> > > + DSIM_PLLCTRL_REG,       /* PLL control register */
> > > + DSIM_PHYCTRL_REG,
> > > + DSIM_PHYTIMING_REG,
> > > + DSIM_PHYTIMING1_REG,
> > > + DSIM_PHYTIMING2_REG,
> > > + NUM_REGS
> > > +};
> > > +
> > > +static const unsigned int exynos_reg_ofs[] = {
> > > + [DSIM_STATUS_REG] =  0x00,
> > > + [DSIM_SWRST_REG] =  0x04,
> > > + [DSIM_CLKCTRL_REG] =  0x08,
> > > + [DSIM_TIMEOUT_REG] =  0x0c,
> > > + [DSIM_CONFIG_REG] =  0x10,
> > > + [DSIM_ESCMODE_REG] =  0x14,
> > > + [DSIM_MDRESOL_REG] =  0x18,
> > > + [DSIM_MVPORCH_REG] =  0x1c,
> > > + [DSIM_MHPORCH_REG] =  0x20,
> > > + [DSIM_MSYNC_REG] =  0x24,
> > > + [DSIM_INTSRC_REG] =  0x2c,
> > > + [DSIM_INTMSK_REG] =  0x30,
> > > + [DSIM_PKTHDR_REG] =  0x34,
> > > + [DSIM_PAYLOAD_REG] =  0x38,
> > > + [DSIM_RXFIFO_REG] =  0x3c,
> > > + [DSIM_FIFOCTRL_REG] =  0x44,
> > > + [DSIM_PLLCTRL_REG] =  0x4c,
> > > + [DSIM_PHYCTRL_REG] =  0x5c,
> > > + [DSIM_PHYTIMING_REG] =  0x64,
> > > + [DSIM_PHYTIMING1_REG] =  0x68,
> > > + [DSIM_PHYTIMING2_REG] =  0x6c,
> > > +};
> > > +
> > > +static const unsigned int exynos5433_reg_ofs[] = {
> > > + [DSIM_STATUS_REG] = 0x04,
> > > + [DSIM_SWRST_REG] = 0x0C,
> > > + [DSIM_CLKCTRL_REG] = 0x10,
> > > + [DSIM_TIMEOUT_REG] = 0x14,
> > > + [DSIM_CONFIG_REG] = 0x18,
> > > + [DSIM_ESCMODE_REG] = 0x1C,
> > > + [DSIM_MDRESOL_REG] = 0x20,
> > > + [DSIM_MVPORCH_REG] = 0x24,
> > > + [DSIM_MHPORCH_REG] = 0x28,
> > > + [DSIM_MSYNC_REG] = 0x2C,
> > > + [DSIM_INTSRC_REG] = 0x34,
> > > + [DSIM_INTMSK_REG] = 0x38,
> > > + [DSIM_PKTHDR_REG] = 0x3C,
> > > + [DSIM_PAYLOAD_REG] = 0x40,
> > > + [DSIM_RXFIFO_REG] = 0x44,
> > > + [DSIM_FIFOCTRL_REG] = 0x4C,
> > > + [DSIM_PLLCTRL_REG] = 0x94,
> > > + [DSIM_PHYCTRL_REG] = 0xA4,
> > > + [DSIM_PHYTIMING_REG] = 0xB4,
> > > + [DSIM_PHYTIMING1_REG] = 0xB8,
> > > + [DSIM_PHYTIMING2_REG] = 0xBC,
> > > +};
> > > +
> > > +static inline void samsung_dsim_write(struct samsung_dsim *dsi,
> > > +                               enum reg_idx idx, u32 val)
> > > +{
> > > + const unsigned int *reg_ofs;
> > > +
> > > + if (dsi->driver_data->reg_ofs == EXYNOS5433_REG_OFS)
> > > +         reg_ofs = exynos5433_reg_ofs;
> > > + else
> > > +         reg_ofs = exynos_reg_ofs;
> > > +
> > > + writel(val, dsi->reg_base + reg_ofs[idx]);
> > > +}
> > > +
> > > +static inline u32 samsung_dsim_read(struct samsung_dsim *dsi, enum 
> > > reg_idx idx)
> > > +{
> > > + const unsigned int *reg_ofs;
> > > +
> > > + if (dsi->driver_data->reg_ofs == EXYNOS5433_REG_OFS)
> > > +         reg_ofs = exynos5433_reg_ofs;
> > > + else
> > > +         reg_ofs = exynos_reg_ofs;
> > > +
> > > + return readl(dsi->reg_base + reg_ofs[idx]);
> > > +}
> > > +
> > > +static void samsung_dsim_wait_for_reset(struct samsung_dsim *dsi)
> > > +{
> > > + if (wait_for_completion_timeout(&dsi->completed, msecs_to_jiffies(300)))
> > > +         return;
> > > +
> > > + dev_err(dsi->dev, "timeout waiting for reset\n");
> > > +}
> > > +
> > > +static void samsung_dsim_reset(struct samsung_dsim *dsi)
> > > +{
> > > + u32 reset_val = dsi->driver_data->reg_values[RESET_TYPE];
> > > +
> > > + reinit_completion(&dsi->completed);
> > > + samsung_dsim_write(dsi, DSIM_SWRST_REG, reset_val);
> > > +}
> > > +
> > > +#ifndef MHZ
> > > +#define MHZ      (1000*1000)
> > > +#endif
> > > +
> > > +static unsigned long samsung_dsim_pll_find_pms(struct samsung_dsim *dsi,
> > > +                                        unsigned long fin,
> > > +                                        unsigned long fout,
> > > +                                        u8 *p, u16 *m, u8 *s)
> > > +{
> > > + const struct samsung_dsim_driver_data *driver_data = dsi->driver_data;
> > > + unsigned long best_freq = 0;
> > > + u32 min_delta = 0xffffffff;
> > > + u8 p_min, p_max;
> > > + u8 _p, best_p;
> > > + u16 _m, best_m;
> > > + u8 _s, best_s;
> > > +
> > > + p_min = DIV_ROUND_UP(fin, (12 * MHZ));
> > > + p_max = fin / (6 * MHZ);
> > > +
> > > + for (_p = p_min; _p <= p_max; ++_p) {
> > > +         for (_s = 0; _s <= 5; ++_s) {
> > > +                 u64 tmp;
> > > +                 u32 delta;
> > > +
> > > +                 tmp = (u64)fout * (_p << _s);
> > > +                 do_div(tmp, fin);
> > > +                 _m = tmp;
> > > +                 if (_m < 41 || _m > 125)
> > > +                         continue;
> > > +
> > > +                 tmp = (u64)_m * fin;
> > > +                 do_div(tmp, _p);
> > > +                 if (tmp < 500 * MHZ ||
> > > +                                 tmp > driver_data->max_freq * MHZ)
> > > +                         continue;
> > > +
> > > +                 tmp = (u64)_m * fin;
> > > +                 do_div(tmp, _p << _s);
> > > +
> > > +                 delta = abs(fout - tmp);
> > > +                 if (delta < min_delta) {
> > > +                         best_p = _p;
> > > +                         best_m = _m;
> > > +                         best_s = _s;
> > > +                         min_delta = delta;
> > > +                         best_freq = tmp;
> > > +                 }
> > > +         }
> > > + }
> > > +
> > > + if (best_freq) {
> > > +         *p = best_p;
> > > +         *m = best_m;
> > > +         *s = best_s;
> > > + }
> > > +
> > > + return best_freq;
> > > +}
> > > +
> > > +static unsigned long samsung_dsim_set_pll(struct samsung_dsim *dsi,
> > > +                                   unsigned long freq)
> > > +{
> > > + const struct samsung_dsim_driver_data *driver_data = dsi->driver_data;
> > > + unsigned long fin, fout;
> > > + int timeout;
> > > + u8 p, s;
> > > + u16 m;
> > > + u32 reg;
> > > +
> > > + fin = dsi->pll_clk_rate;
> > > + fout = samsung_dsim_pll_find_pms(dsi, fin, freq, &p, &m, &s);
> > > + if (!fout) {
> > > +         dev_err(dsi->dev,
> > > +                 "failed to find PLL PMS for requested frequency\n");
> > > +         return 0;
> > > + }
> > > + dev_dbg(dsi->dev, "PLL freq %lu, (p %d, m %d, s %d)\n", fout, p, m, s);
> > > +
> > > + writel(driver_data->reg_values[PLL_TIMER],
> > > +                 dsi->reg_base + driver_data->plltmr_reg);
> > > +
> > > + reg = DSIM_PLL_EN | DSIM_PLL_P(p) | DSIM_PLL_M(m) | DSIM_PLL_S(s);
> > > +
> > > + if (driver_data->has_freqband) {
> > > +         static const unsigned long freq_bands[] = {
> > > +                 100 * MHZ, 120 * MHZ, 160 * MHZ, 200 * MHZ,
> > > +                 270 * MHZ, 320 * MHZ, 390 * MHZ, 450 * MHZ,
> > > +                 510 * MHZ, 560 * MHZ, 640 * MHZ, 690 * MHZ,
> > > +                 770 * MHZ, 870 * MHZ, 950 * MHZ,
> > > +         };
> > > +         int band;
> > > +
> > > +         for (band = 0; band < ARRAY_SIZE(freq_bands); ++band)
> > > +                 if (fout < freq_bands[band])
> > > +                         break;
> > > +
> > > +         dev_dbg(dsi->dev, "band %d\n", band);
> > > +
> > > +         reg |= DSIM_FREQ_BAND(band);
> > > + }
> > > +
> > > + samsung_dsim_write(dsi, DSIM_PLLCTRL_REG, reg);
> > > +
> > > + timeout = 1000;
> > > + do {
> > > +         if (timeout-- == 0) {
> > > +                 dev_err(dsi->dev, "PLL failed to stabilize\n");
> > > +                 return 0;
> > > +         }
> > > +         reg = samsung_dsim_read(dsi, DSIM_STATUS_REG);
> > > + } while ((reg & DSIM_PLL_STABLE) == 0);
> > > +
> > > + return fout;
> > > +}
> > > +
> > > +static int samsung_dsim_enable_clock(struct samsung_dsim *dsi)
> > > +{
> > > + unsigned long hs_clk, byte_clk, esc_clk;
> > > + unsigned long esc_div;
> > > + u32 reg;
> > > +
> > > + hs_clk = samsung_dsim_set_pll(dsi, dsi->burst_clk_rate);
> > > + if (!hs_clk) {
> > > +         dev_err(dsi->dev, "failed to configure DSI PLL\n");
> > > +         return -EFAULT;
> > > + }
> > > +
> > > + byte_clk = hs_clk / 8;
> > > + esc_div = DIV_ROUND_UP(byte_clk, dsi->esc_clk_rate);
> > > + esc_clk = byte_clk / esc_div;
> > > +
> > > + if (esc_clk > 20 * MHZ) {
> > > +         ++esc_div;
> > > +         esc_clk = byte_clk / esc_div;
> > > + }
> > > +
> > > + dev_dbg(dsi->dev, "hs_clk = %lu, byte_clk = %lu, esc_clk = %lu\n",
> > > +         hs_clk, byte_clk, esc_clk);
> > > +
> > > + reg = samsung_dsim_read(dsi, DSIM_CLKCTRL_REG);
> > > + reg &= ~(DSIM_ESC_PRESCALER_MASK | DSIM_LANE_ESC_CLK_EN_CLK
> > > +                 | DSIM_LANE_ESC_CLK_EN_DATA_MASK | DSIM_PLL_BYPASS
> > > +                 | DSIM_BYTE_CLK_SRC_MASK);
> > > + reg |= DSIM_ESC_CLKEN | DSIM_BYTE_CLKEN
> > > +                 | DSIM_ESC_PRESCALER(esc_div)
> > > +                 | DSIM_LANE_ESC_CLK_EN_CLK
> > > +                 | DSIM_LANE_ESC_CLK_EN_DATA(BIT(dsi->lanes) - 1)
> > > +                 | DSIM_BYTE_CLK_SRC(0)
> > > +                 | DSIM_TX_REQUEST_HSCLK;
> > > + samsung_dsim_write(dsi, DSIM_CLKCTRL_REG, reg);
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static void samsung_dsim_set_phy_ctrl(struct samsung_dsim *dsi)
> > > +{
> > > + const struct samsung_dsim_driver_data *driver_data = dsi->driver_data;
> > > + const unsigned int *reg_values = driver_data->reg_values;
> > > + u32 reg;
> > > +
> > > + if (driver_data->has_freqband)
> > > +         return;
> > > +
> > > + /* B D-PHY: D-PHY Master & Slave Analog Block control */
> > > + reg = DSIM_PHYCTRL_ULPS_EXIT(reg_values[PHYCTRL_ULPS_EXIT]);
> > > + if (reg_values[PHYCTRL_VREG_LP])
> > > +         reg |= DSIM_PHYCTRL_B_DPHYCTL_VREG_LP;
> > > + if (reg_values[PHYCTRL_SLEW_UP])
> > > +         reg |= DSIM_PHYCTRL_B_DPHYCTL_SLEW_UP;
> > > + samsung_dsim_write(dsi, DSIM_PHYCTRL_REG, reg);
> > > +
> > > + /*
> > > +  * T LPX: Transmitted length of any Low-Power state period
> > > +  * T HS-EXIT: Time that the transmitter drives LP-11 following a HS
> > > +  *      burst
> > > +  */
> > > + reg = DSIM_PHYTIMING_LPX(reg_values[PHYTIMING_LPX]) |
> > > +         DSIM_PHYTIMING_HS_EXIT(reg_values[PHYTIMING_HS_EXIT]);
> > > + samsung_dsim_write(dsi, DSIM_PHYTIMING_REG, reg);
> > > +
> > > + /*
> > > +  * T CLK-PREPARE: Time that the transmitter drives the Clock Lane LP-00
> > > +  *      Line state immediately before the HS-0 Line state starting the
> > > +  *      HS transmission
> > > +  * T CLK-ZERO: Time that the transmitter drives the HS-0 state prior to
> > > +  *      transmitting the Clock.
> > > +  * T CLK_POST: Time that the transmitter continues to send HS clock
> > > +  *      after the last associated Data Lane has transitioned to LP Mode
> > > +  *      Interval is defined as the period from the end of T HS-TRAIL to
> > > +  *      the beginning of T CLK-TRAIL
> > > +  * T CLK-TRAIL: Time that the transmitter drives the HS-0 state after
> > > +  *      the last payload clock bit of a HS transmission burst
> > > +  */
> > > + reg = DSIM_PHYTIMING1_CLK_PREPARE(reg_values[PHYTIMING_CLK_PREPARE]) |
> > > +         DSIM_PHYTIMING1_CLK_ZERO(reg_values[PHYTIMING_CLK_ZERO]) |
> > > +         DSIM_PHYTIMING1_CLK_POST(reg_values[PHYTIMING_CLK_POST]) |
> > > +         DSIM_PHYTIMING1_CLK_TRAIL(reg_values[PHYTIMING_CLK_TRAIL]);
> > > + samsung_dsim_write(dsi, DSIM_PHYTIMING1_REG, reg);
> > > +
> > > + /*
> > > +  * T HS-PREPARE: Time that the transmitter drives the Data Lane LP-00
> > > +  *      Line state immediately before the HS-0 Line state starting the
> > > +  *      HS transmission
> > > +  * T HS-ZERO: Time that the transmitter drives the HS-0 state prior to
> > > +  *      transmitting the Sync sequence.
> > > +  * T HS-TRAIL: Time that the transmitter drives the flipped differential
> > > +  *      state after last payload data bit of a HS transmission burst
> > > +  */
> > > + reg = DSIM_PHYTIMING2_HS_PREPARE(reg_values[PHYTIMING_HS_PREPARE]) |
> > > +         DSIM_PHYTIMING2_HS_ZERO(reg_values[PHYTIMING_HS_ZERO]) |
> > > +         DSIM_PHYTIMING2_HS_TRAIL(reg_values[PHYTIMING_HS_TRAIL]);
> > > + samsung_dsim_write(dsi, DSIM_PHYTIMING2_REG, reg);
> > > +}
> > > +
> > > +static void samsung_dsim_disable_clock(struct samsung_dsim *dsi)
> > > +{
> > > + u32 reg;
> > > +
> > > + reg = samsung_dsim_read(dsi, DSIM_CLKCTRL_REG);
> > > + reg &= ~(DSIM_LANE_ESC_CLK_EN_CLK | DSIM_LANE_ESC_CLK_EN_DATA_MASK
> > > +                 | DSIM_ESC_CLKEN | DSIM_BYTE_CLKEN);
> > > + samsung_dsim_write(dsi, DSIM_CLKCTRL_REG, reg);
> > > +
> > > + reg = samsung_dsim_read(dsi, DSIM_PLLCTRL_REG);
> > > + reg &= ~DSIM_PLL_EN;
> > > + samsung_dsim_write(dsi, DSIM_PLLCTRL_REG, reg);
> > > +}
> > > +
> > > +static void samsung_dsim_enable_lane(struct samsung_dsim *dsi, u32 lane)
> > > +{
> > > + u32 reg = samsung_dsim_read(dsi, DSIM_CONFIG_REG);
> > > + reg |= (DSIM_NUM_OF_DATA_LANE(dsi->lanes - 1) | DSIM_LANE_EN_CLK |
> > > +                 DSIM_LANE_EN(lane));
> > > + samsung_dsim_write(dsi, DSIM_CONFIG_REG, reg);
> > > +}
> > > +
> > > +static int samsung_dsim_init_link(struct samsung_dsim *dsi)
> > > +{
> > > + const struct samsung_dsim_driver_data *driver_data = dsi->driver_data;
> > > + int timeout;
> > > + u32 reg;
> > > + u32 lanes_mask;
> > > +
> > > + /* Initialize FIFO pointers */
> > > + reg = samsung_dsim_read(dsi, DSIM_FIFOCTRL_REG);
> > > + reg &= ~0x1f;
> > > + samsung_dsim_write(dsi, DSIM_FIFOCTRL_REG, reg);
> > > +
> > > + usleep_range(9000, 11000);
> > > +
> > > + reg |= 0x1f;
> > > + samsung_dsim_write(dsi, DSIM_FIFOCTRL_REG, reg);
> > > + usleep_range(9000, 11000);
> > > +
> > > + /* DSI configuration */
> > > + reg = 0;
> > > +
> > > + /*
> > > +  * The first bit of mode_flags specifies display configuration.
> > > +  * If this bit is set[= MIPI_DSI_MODE_VIDEO], dsi will support video
> > > +  * mode, otherwise it will support command mode.
> > > +  */
> > > + if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO) {
> > > +         reg |= DSIM_VIDEO_MODE;
> > > +
> > > +         /*
> > > +          * The user manual describes that following bits are ignored in
> > > +          * command mode.
> > > +          */
> > > +         if (!(dsi->mode_flags & MIPI_DSI_MODE_VSYNC_FLUSH))
> > > +                 reg |= DSIM_MFLUSH_VS;
> > > +         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE)
> > > +                 reg |= DSIM_SYNC_INFORM;
> > > +         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_BURST)
> > > +                 reg |= DSIM_BURST_MODE;
> > > +         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_AUTO_VERT)
> > > +                 reg |= DSIM_AUTO_MODE;
> > > +         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HSE)
> > > +                 reg |= DSIM_HSE_MODE;
> > > +         if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HFP))
> > > +                 reg |= DSIM_HFP_MODE;
> > > +         if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HBP))
> > > +                 reg |= DSIM_HBP_MODE;
> > > +         if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HSA))
> > > +                 reg |= DSIM_HSA_MODE;
> > > + }
> > > +
> > > + if (!(dsi->mode_flags & MIPI_DSI_MODE_EOT_PACKET))
> > > +         reg |= DSIM_EOT_DISABLE;
> > > +
> > > + switch (dsi->format) {
> > > + case MIPI_DSI_FMT_RGB888:
> > > +         reg |= DSIM_MAIN_PIX_FORMAT_RGB888;
> > > +         break;
> > > + case MIPI_DSI_FMT_RGB666:
> > > +         reg |= DSIM_MAIN_PIX_FORMAT_RGB666;
> > > +         break;
> > > + case MIPI_DSI_FMT_RGB666_PACKED:
> > > +         reg |= DSIM_MAIN_PIX_FORMAT_RGB666_P;
> > > +         break;
> > > + case MIPI_DSI_FMT_RGB565:
> > > +         reg |= DSIM_MAIN_PIX_FORMAT_RGB565;
> > > +         break;
> > > + default:
> > > +         dev_err(dsi->dev, "invalid pixel format\n");
> > > +         return -EINVAL;
> > > + }
> > > +
> > > + /*
> > > +  * Use non-continuous clock mode if the periparal wants and
> > > +  * host controller supports
> > > +  *
> > > +  * In non-continous clock mode, host controller will turn off
> > > +  * the HS clock between high-speed transmissions to reduce
> > > +  * power consumption.
> > > +  */
> > > + if (driver_data->has_clklane_stop &&
> > > +                 dsi->mode_flags & MIPI_DSI_CLOCK_NON_CONTINUOUS) {
> > > +         reg |= DSIM_CLKLANE_STOP;
> > > + }
> > > + samsung_dsim_write(dsi, DSIM_CONFIG_REG, reg);
> > > +
> > > + lanes_mask = BIT(dsi->lanes) - 1;
> > > + samsung_dsim_enable_lane(dsi, lanes_mask);
> > > +
> > > + /* Check clock and data lane state are stop state */
> > > + timeout = 100;
> > > + do {
> > > +         if (timeout-- == 0) {
> > > +                 dev_err(dsi->dev, "waiting for bus lanes timed out\n");
> > > +                 return -EFAULT;
> > > +         }
> > > +
> > > +         reg = samsung_dsim_read(dsi, DSIM_STATUS_REG);
> > > +         if ((reg & DSIM_STOP_STATE_DAT(lanes_mask))
> > > +             != DSIM_STOP_STATE_DAT(lanes_mask))
> > > +                 continue;
> > > + } while (!(reg & (DSIM_STOP_STATE_CLK | DSIM_TX_READY_HS_CLK)));
> > > +
> > > + reg = samsung_dsim_read(dsi, DSIM_ESCMODE_REG);
> > > + reg &= ~DSIM_STOP_STATE_CNT_MASK;
> > > + reg |= DSIM_STOP_STATE_CNT(driver_data->reg_values[STOP_STATE_CNT]);
> > > + samsung_dsim_write(dsi, DSIM_ESCMODE_REG, reg);
> > > +
> > > + reg = DSIM_BTA_TIMEOUT(0xff) | DSIM_LPDR_TIMEOUT(0xffff);
> > > + samsung_dsim_write(dsi, DSIM_TIMEOUT_REG, reg);
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static void samsung_dsim_set_display_mode(struct samsung_dsim *dsi)
> > > +{
> > > + struct drm_display_mode *m = &dsi->mode;
> > > + unsigned int num_bits_resol = dsi->driver_data->num_bits_resol;
> > > + u32 reg;
> > > +
> > > + if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO) {
> > > +         reg = DSIM_CMD_ALLOW(0xf)
> > > +                 | DSIM_STABLE_VFP(m->vsync_start - m->vdisplay)
> > > +                 | DSIM_MAIN_VBP(m->vtotal - m->vsync_end);
> > > +         samsung_dsim_write(dsi, DSIM_MVPORCH_REG, reg);
> > > +
> > > +         reg = DSIM_MAIN_HFP(m->hsync_start - m->hdisplay)
> > > +                 | DSIM_MAIN_HBP(m->htotal - m->hsync_end);
> > > +         samsung_dsim_write(dsi, DSIM_MHPORCH_REG, reg);
> > > +
> > > +         reg = DSIM_MAIN_VSA(m->vsync_end - m->vsync_start)
> > > +                 | DSIM_MAIN_HSA(m->hsync_end - m->hsync_start);
> > > +         samsung_dsim_write(dsi, DSIM_MSYNC_REG, reg);
> > > + }
> > > + reg =  DSIM_MAIN_HRESOL(m->hdisplay, num_bits_resol) |
> > > +         DSIM_MAIN_VRESOL(m->vdisplay, num_bits_resol);
> > > +
> > > + samsung_dsim_write(dsi, DSIM_MDRESOL_REG, reg);
> > > +
> > > + dev_dbg(dsi->dev, "LCD size = %dx%d\n", m->hdisplay, m->vdisplay);
> > > +}
> > > +
> > > +static void samsung_dsim_set_display_enable(struct samsung_dsim *dsi,
> > > +                                     bool enable)
> > > +{
> > > + u32 reg;
> > > +
> > > + reg = samsung_dsim_read(dsi, DSIM_MDRESOL_REG);
> > > + if (enable)
> > > +         reg |= DSIM_MAIN_STAND_BY;
> > > + else
> > > +         reg &= ~DSIM_MAIN_STAND_BY;
> > > + samsung_dsim_write(dsi, DSIM_MDRESOL_REG, reg);
> > > +}
> > > +
> > > +static int samsung_dsim_wait_for_hdr_fifo(struct samsung_dsim *dsi)
> > > +{
> > > + int timeout = 2000;
> > > +
> > > + do {
> > > +         u32 reg = samsung_dsim_read(dsi, DSIM_FIFOCTRL_REG);
> > > +
> > > +         if (!(reg & DSIM_SFR_HEADER_FULL))
> > > +                 return 0;
> > > +
> > > +         if (!cond_resched())
> > > +                 usleep_range(950, 1050);
> > > + } while (--timeout);
> > > +
> > > + return -ETIMEDOUT;
> > > +}
> > > +
> > > +static void samsung_dsim_set_cmd_lpm(struct samsung_dsim *dsi, bool lpm)
> > > +{
> > > + u32 v = samsung_dsim_read(dsi, DSIM_ESCMODE_REG);
> > > +
> > > + if (lpm)
> > > +         v |= DSIM_CMD_LPDT_LP;
> > > + else
> > > +         v &= ~DSIM_CMD_LPDT_LP;
> > > +
> > > + samsung_dsim_write(dsi, DSIM_ESCMODE_REG, v);
> > > +}
> > > +
> > > +static void samsung_dsim_force_bta(struct samsung_dsim *dsi)
> > > +{
> > > + u32 v = samsung_dsim_read(dsi, DSIM_ESCMODE_REG);
> > > + v |= DSIM_FORCE_BTA;
> > > + samsung_dsim_write(dsi, DSIM_ESCMODE_REG, v);
> > > +}
> > > +
> > > +static void samsung_dsim_send_to_fifo(struct samsung_dsim *dsi,
> > > +                               struct samsung_dsim_transfer *xfer)
> > > +{
> > > + struct device *dev = dsi->dev;
> > > + struct mipi_dsi_packet *pkt = &xfer->packet;
> > > + const u8 *payload = pkt->payload + xfer->tx_done;
> > > + u16 length = pkt->payload_length - xfer->tx_done;
> > > + bool first = !xfer->tx_done;
> > > + u32 reg;
> > > +
> > > + dev_dbg(dev, "< xfer %pK: tx len %u, done %u, rx len %u, done %u\n",
> > > +         xfer, length, xfer->tx_done, xfer->rx_len, xfer->rx_done);
> > > +
> > > + if (length > DSI_TX_FIFO_SIZE)
> > > +         length = DSI_TX_FIFO_SIZE;
> > > +
> > > + xfer->tx_done += length;
> > > +
> > > + /* Send payload */
> > > + while (length >= 4) {
> > > +         reg = get_unaligned_le32(payload);
> > > +         samsung_dsim_write(dsi, DSIM_PAYLOAD_REG, reg);
> > > +         payload += 4;
> > > +         length -= 4;
> > > + }
> > > +
> > > + reg = 0;
> > > + switch (length) {
> > > + case 3:
> > > +         reg |= payload[2] << 16;
> > > +         fallthrough;
> > > + case 2:
> > > +         reg |= payload[1] << 8;
> > > +         fallthrough;
> > > + case 1:
> > > +         reg |= payload[0];
> > > +         samsung_dsim_write(dsi, DSIM_PAYLOAD_REG, reg);
> > > +         break;
> > > + }
> > > +
> > > + /* Send packet header */
> > > + if (!first)
> > > +         return;
> > > +
> > > + reg = get_unaligned_le32(pkt->header);
> > > + if (samsung_dsim_wait_for_hdr_fifo(dsi)) {
> > > +         dev_err(dev, "waiting for header FIFO timed out\n");
> > > +         return;
> > > + }
> > > +
> > > + if (NEQV(xfer->flags & MIPI_DSI_MSG_USE_LPM,
> > > +          dsi->state & DSIM_STATE_CMD_LPM)) {
> > > +         samsung_dsim_set_cmd_lpm(dsi, xfer->flags & 
> > > MIPI_DSI_MSG_USE_LPM);
> > > +         dsi->state ^= DSIM_STATE_CMD_LPM;
> > > + }
> > > +
> > > + samsung_dsim_write(dsi, DSIM_PKTHDR_REG, reg);
> > > +
> > > + if (xfer->flags & MIPI_DSI_MSG_REQ_ACK)
> > > +         samsung_dsim_force_bta(dsi);
> > > +}
> > > +
> > > +static void samsung_dsim_read_from_fifo(struct samsung_dsim *dsi,
> > > +                                 struct samsung_dsim_transfer *xfer)
> > > +{
> > > + u8 *payload = xfer->rx_payload + xfer->rx_done;
> > > + bool first = !xfer->rx_done;
> > > + struct device *dev = dsi->dev;
> > > + u16 length;
> > > + u32 reg;
> > > +
> > > + if (first) {
> > > +         reg = samsung_dsim_read(dsi, DSIM_RXFIFO_REG);
> > > +
> > > +         switch (reg & 0x3f) {
> > > +         case MIPI_DSI_RX_GENERIC_SHORT_READ_RESPONSE_2BYTE:
> > > +         case MIPI_DSI_RX_DCS_SHORT_READ_RESPONSE_2BYTE:
> > > +                 if (xfer->rx_len >= 2) {
> > > +                         payload[1] = reg >> 16;
> > > +                         ++xfer->rx_done;
> > > +                 }
> > > +                 fallthrough;
> > > +         case MIPI_DSI_RX_GENERIC_SHORT_READ_RESPONSE_1BYTE:
> > > +         case MIPI_DSI_RX_DCS_SHORT_READ_RESPONSE_1BYTE:
> > > +                 payload[0] = reg >> 8;
> > > +                 ++xfer->rx_done;
> > > +                 xfer->rx_len = xfer->rx_done;
> > > +                 xfer->result = 0;
> > > +                 goto clear_fifo;
> > > +         case MIPI_DSI_RX_ACKNOWLEDGE_AND_ERROR_REPORT:
> > > +                 dev_err(dev, "DSI Error Report: 0x%04x\n",
> > > +                         (reg >> 8) & 0xffff);
> > > +                 xfer->result = 0;
> > > +                 goto clear_fifo;
> > > +         }
> > > +
> > > +         length = (reg >> 8) & 0xffff;
> > > +         if (length > xfer->rx_len) {
> > > +                 dev_err(dev,
> > > +                         "response too long (%u > %u bytes), 
> > > stripping\n",
> > > +                         xfer->rx_len, length);
> > > +                 length = xfer->rx_len;
> > > +         } else if (length < xfer->rx_len)
> > > +                 xfer->rx_len = length;
> > > + }
> > > +
> > > + length = xfer->rx_len - xfer->rx_done;
> > > + xfer->rx_done += length;
> > > +
> > > + /* Receive payload */
> > > + while (length >= 4) {
> > > +         reg = samsung_dsim_read(dsi, DSIM_RXFIFO_REG);
> > > +         payload[0] = (reg >>  0) & 0xff;
> > > +         payload[1] = (reg >>  8) & 0xff;
> > > +         payload[2] = (reg >> 16) & 0xff;
> > > +         payload[3] = (reg >> 24) & 0xff;
> > > +         payload += 4;
> > > +         length -= 4;
> > > + }
> > > +
> > > + if (length) {
> > > +         reg = samsung_dsim_read(dsi, DSIM_RXFIFO_REG);
> > > +         switch (length) {
> > > +         case 3:
> > > +                 payload[2] = (reg >> 16) & 0xff;
> > > +                 fallthrough;
> > > +         case 2:
> > > +                 payload[1] = (reg >> 8) & 0xff;
> > > +                 fallthrough;
> > > +         case 1:
> > > +                 payload[0] = reg & 0xff;
> > > +         }
> > > + }
> > > +
> > > + if (xfer->rx_done == xfer->rx_len)
> > > +         xfer->result = 0;
> > > +
> > > +clear_fifo:
> > > + length = DSI_RX_FIFO_SIZE / 4;
> > > + do {
> > > +         reg = samsung_dsim_read(dsi, DSIM_RXFIFO_REG);
> > > +         if (reg == DSI_RX_FIFO_EMPTY)
> > > +                 break;
> > > + } while (--length);
> > > +}
> > > +
> > > +static void samsung_dsim_transfer_start(struct samsung_dsim *dsi)
> > > +{
> > > + unsigned long flags;
> > > + struct samsung_dsim_transfer *xfer;
> > > + bool start = false;
> > > +
> > > +again:
> > > + spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > +
> > > + if (list_empty(&dsi->transfer_list)) {
> > > +         spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +         return;
> > > + }
> > > +
> > > + xfer = list_first_entry(&dsi->transfer_list,
> > > +                                 struct samsung_dsim_transfer, list);
> > > +
> > > + spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +
> > > + if (xfer->packet.payload_length &&
> > > +     xfer->tx_done == xfer->packet.payload_length)
> > > +         /* waiting for RX */
> > > +         return;
> > > +
> > > + samsung_dsim_send_to_fifo(dsi, xfer);
> > > +
> > > + if (xfer->packet.payload_length || xfer->rx_len)
> > > +         return;
> > > +
> > > + xfer->result = 0;
> > > + complete(&xfer->completed);
> > > +
> > > + spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > +
> > > + list_del_init(&xfer->list);
> > > + start = !list_empty(&dsi->transfer_list);
> > > +
> > > + spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +
> > > + if (start)
> > > +         goto again;
> > > +}
> > > +
> > > +static bool samsung_dsim_transfer_finish(struct samsung_dsim *dsi)
> > > +{
> > > + struct samsung_dsim_transfer *xfer;
> > > + unsigned long flags;
> > > + bool start = true;
> > > +
> > > + spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > +
> > > + if (list_empty(&dsi->transfer_list)) {
> > > +         spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +         return false;
> > > + }
> > > +
> > > + xfer = list_first_entry(&dsi->transfer_list,
> > > +                                 struct samsung_dsim_transfer, list);
> > > +
> > > + spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +
> > > + dev_dbg(dsi->dev,
> > > +         "> xfer %pK, tx_len %zu, tx_done %u, rx_len %u, rx_done %u\n",
> > > +         xfer, xfer->packet.payload_length, xfer->tx_done, xfer->rx_len,
> > > +         xfer->rx_done);
> > > +
> > > + if (xfer->tx_done != xfer->packet.payload_length)
> > > +         return true;
> > > +
> > > + if (xfer->rx_done != xfer->rx_len)
> > > +         samsung_dsim_read_from_fifo(dsi, xfer);
> > > +
> > > + if (xfer->rx_done != xfer->rx_len)
> > > +         return true;
> > > +
> > > + spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > +
> > > + list_del_init(&xfer->list);
> > > + start = !list_empty(&dsi->transfer_list);
> > > +
> > > + spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +
> > > + if (!xfer->rx_len)
> > > +         xfer->result = 0;
> > > + complete(&xfer->completed);
> > > +
> > > + return start;
> > > +}
> > > +
> > > +static void samsung_dsim_remove_transfer(struct samsung_dsim *dsi,
> > > +                                  struct samsung_dsim_transfer *xfer)
> > > +{
> > > + unsigned long flags;
> > > + bool start;
> > > +
> > > + spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > +
> > > + if (!list_empty(&dsi->transfer_list) &&
> > > +     xfer == list_first_entry(&dsi->transfer_list,
> > > +                              struct samsung_dsim_transfer, list)) {
> > > +         list_del_init(&xfer->list);
> > > +         start = !list_empty(&dsi->transfer_list);
> > > +         spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +         if (start)
> > > +                 samsung_dsim_transfer_start(dsi);
> > > +         return;
> > > + }
> > > +
> > > + list_del_init(&xfer->list);
> > > +
> > > + spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +}
> > > +
> > > +static int samsung_dsim_transfer(struct samsung_dsim *dsi,
> > > +                          struct samsung_dsim_transfer *xfer)
> > > +{
> > > + unsigned long flags;
> > > + bool stopped;
> > > +
> > > + xfer->tx_done = 0;
> > > + xfer->rx_done = 0;
> > > + xfer->result = -ETIMEDOUT;
> > > + init_completion(&xfer->completed);
> > > +
> > > + spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > +
> > > + stopped = list_empty(&dsi->transfer_list);
> > > + list_add_tail(&xfer->list, &dsi->transfer_list);
> > > +
> > > + spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > +
> > > + if (stopped)
> > > +         samsung_dsim_transfer_start(dsi);
> > > +
> > > + wait_for_completion_timeout(&xfer->completed,
> > > +                             msecs_to_jiffies(DSI_XFER_TIMEOUT_MS));
> > > + if (xfer->result == -ETIMEDOUT) {
> > > +         struct mipi_dsi_packet *pkt = &xfer->packet;
> > > +         samsung_dsim_remove_transfer(dsi, xfer);
> > > +         dev_err(dsi->dev, "xfer timed out: %*ph %*ph\n", 4, pkt->header,
> > > +                 (int)pkt->payload_length, pkt->payload);
> > > +         return -ETIMEDOUT;
> > > + }
> > > +
> > > + /* Also covers hardware timeout condition */
> > > + return xfer->result;
> > > +}
> > > +
> > > +static irqreturn_t samsung_dsim_irq(int irq, void *dev_id)
> > > +{
> > > + struct samsung_dsim *dsi = dev_id;
> > > + u32 status;
> > > +
> > > + status = samsung_dsim_read(dsi, DSIM_INTSRC_REG);
> > > + if (!status) {
> > > +         static unsigned long int j;
> > > +         if (printk_timed_ratelimit(&j, 500))
> > > +                 dev_warn(dsi->dev, "spurious interrupt\n");
> > > +         return IRQ_HANDLED;
> > > + }
> > > + samsung_dsim_write(dsi, DSIM_INTSRC_REG, status);
> > > +
> > > + if (status & DSIM_INT_SW_RST_RELEASE) {
> > > +         u32 mask = ~(DSIM_INT_RX_DONE | DSIM_INT_SFR_FIFO_EMPTY |
> > > +                 DSIM_INT_SFR_HDR_FIFO_EMPTY | DSIM_INT_RX_ECC_ERR |
> > > +                 DSIM_INT_SW_RST_RELEASE);
> > > +         samsung_dsim_write(dsi, DSIM_INTMSK_REG, mask);
> > > +         complete(&dsi->completed);
> > > +         return IRQ_HANDLED;
> > > + }
> > > +
> > > + if (!(status & (DSIM_INT_RX_DONE | DSIM_INT_SFR_FIFO_EMPTY |
> > > +                 DSIM_INT_PLL_STABLE)))
> > > +         return IRQ_HANDLED;
> > > +
> > > + if (samsung_dsim_transfer_finish(dsi))
> > > +         samsung_dsim_transfer_start(dsi);
> > > +
> > > + return IRQ_HANDLED;
> > > +}
> > > +
> > > +static irqreturn_t samsung_dsim_te_irq_handler(int irq, void *dev_id)
> > > +{
> > > + struct samsung_dsim *dsi = dev_id;
> > > + const struct samsung_dsim_host_ops *ops = dsi->driver_data->host_ops;
> > > +
> > > + if (ops && ops->te_handler &&
> > > +     (dsi->state & DSIM_STATE_VIDOUT_AVAILABLE))
> > > +         ops->te_handler(dsi->dsi_host.dev);
> > > +
> > > + return IRQ_HANDLED;
> > > +}
> > > +
> > > +static void samsung_dsim_enable_irq(struct samsung_dsim *dsi)
> > > +{
> > > + enable_irq(dsi->irq);
> > > +
> > > + if (gpio_is_valid(dsi->te_gpio))
> > > +         enable_irq(gpio_to_irq(dsi->te_gpio));
> > > +}
> > > +
> > > +static void samsung_dsim_disable_irq(struct samsung_dsim *dsi)
> > > +{
> > > + if (gpio_is_valid(dsi->te_gpio))
> > > +         disable_irq(gpio_to_irq(dsi->te_gpio));
> > > +
> > > + disable_irq(dsi->irq);
> > > +}
> > > +
> > > +static int samsung_dsim_init(struct samsung_dsim *dsi)
> > > +{
> > > + const struct samsung_dsim_driver_data *driver_data = dsi->driver_data;
> > > +
> > > + samsung_dsim_reset(dsi);
> > > + samsung_dsim_enable_irq(dsi);
> > > +
> > > + if (driver_data->reg_values[RESET_TYPE] == DSIM_FUNCRST)
> > > +         samsung_dsim_enable_lane(dsi, BIT(dsi->lanes) - 1);
> > > +
> > > + samsung_dsim_enable_clock(dsi);
> > > + if (driver_data->wait_for_reset)
> > > +         samsung_dsim_wait_for_reset(dsi);
> > > + samsung_dsim_set_phy_ctrl(dsi);
> > > + samsung_dsim_init_link(dsi);
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static int samsung_dsim_register_te_irq(struct samsung_dsim *dsi,
> > > +                                 struct device *panel)
> > > +{
> > > + int ret;
> > > + int te_gpio_irq;
> > > +
> > > + dsi->te_gpio = of_get_named_gpio(panel->of_node, "te-gpios", 0);
> > > + if (dsi->te_gpio == -ENOENT)
> > > +         return 0;
> > > +
> > > + if (!gpio_is_valid(dsi->te_gpio)) {
> > > +         ret = dsi->te_gpio;
> > > +         dev_err(dsi->dev, "cannot get te-gpios, %d\n", ret);
> > > +         goto out;
> > > + }
> > > +
> > > + ret = gpio_request(dsi->te_gpio, "te_gpio");
> > > + if (ret) {
> > > +         dev_err(dsi->dev, "gpio request failed with %d\n", ret);
> > > +         goto out;
> > > + }
> > > +
> > > + te_gpio_irq = gpio_to_irq(dsi->te_gpio);
> > > + irq_set_status_flags(te_gpio_irq, IRQ_NOAUTOEN);
> > > +
> > > + ret = request_threaded_irq(te_gpio_irq, samsung_dsim_te_irq_handler,
> > > +                            NULL, IRQF_TRIGGER_RISING, "TE", dsi);
> > > + if (ret) {
> > > +         dev_err(dsi->dev, "request interrupt failed with %d\n", ret);
> > > +         gpio_free(dsi->te_gpio);
> > > +         goto out;
> > > + }
> > > +
> > > +out:
> > > + return ret;
> > > +}
> > > +
> > > +static void samsung_dsim_unregister_te_irq(struct samsung_dsim *dsi)
> > > +{
> > > + if (gpio_is_valid(dsi->te_gpio)) {
> > > +         free_irq(gpio_to_irq(dsi->te_gpio), dsi);
> > > +         gpio_free(dsi->te_gpio);
> > > +         dsi->te_gpio = -ENOENT;
> > > + }
> > > +}
> > > +
> > > +static void samsung_dsim_enable(struct samsung_dsim *dsi)
> > > +{
> > > + struct drm_bridge *iter;
> > > + int ret;
> > > +
> > > + if (dsi->state & DSIM_STATE_ENABLED)
> > > +         return;
> > > +
> > > + pm_runtime_get_sync(dsi->dev);
> > > + dsi->state |= DSIM_STATE_ENABLED;
> > > +
> > > + if (dsi->panel) {
> > > +         ret = drm_panel_prepare(dsi->panel);
> > > +         if (ret < 0)
> > > +                 goto err_put_sync;
> > > + } else {
> > > +         list_for_each_entry_reverse(iter, &dsi->bridge_chain,
> > > +                                     chain_node) {
> > > +                 if (iter->funcs->pre_enable)
> > > +                         iter->funcs->pre_enable(iter);
> > > +         }
> > > + }
> > > +
> > > + samsung_dsim_set_display_mode(dsi);
> > > + samsung_dsim_set_display_enable(dsi, true);
> > > +
> > > + if (dsi->panel) {
> > > +         ret = drm_panel_enable(dsi->panel);
> > > +         if (ret < 0)
> > > +                 goto err_display_disable;
> > > + } else {
> > > +         list_for_each_entry(iter, &dsi->bridge_chain, chain_node) {
> > > +                 if (iter->funcs->enable)
> > > +                         iter->funcs->enable(iter);
> > > +         }
> > > + }
> > > +
> > > + dsi->state |= DSIM_STATE_VIDOUT_AVAILABLE;
> > > + return;
> > > +
> > > +err_display_disable:
> > > + samsung_dsim_set_display_enable(dsi, false);
> > > + drm_panel_unprepare(dsi->panel);
> > > +
> > > +err_put_sync:
> > > + dsi->state &= ~DSIM_STATE_ENABLED;
> > > + pm_runtime_put(dsi->dev);
> > > +}
> > > +
> > > +static void samsung_dsim_disable(struct samsung_dsim *dsi)
> > > +{
> > > + struct drm_bridge *iter;
> > > +
> > > + if (!(dsi->state & DSIM_STATE_ENABLED))
> > > +         return;
> > > +
> > > + dsi->state &= ~DSIM_STATE_VIDOUT_AVAILABLE;
> > > +
> > > + drm_panel_disable(dsi->panel);
> > > +
> > > + list_for_each_entry_reverse(iter, &dsi->bridge_chain, chain_node) {
> > > +         if (iter->funcs->disable)
> > > +                 iter->funcs->disable(iter);
> > > + }
> > > +
> > > + samsung_dsim_set_display_enable(dsi, false);
> > > + drm_panel_unprepare(dsi->panel);
> > > +
> > > + list_for_each_entry(iter, &dsi->bridge_chain, chain_node) {
> > > +         if (iter->funcs->post_disable)
> > > +                 iter->funcs->post_disable(iter);
> > > + }
> > > +
> > > + dsi->state &= ~DSIM_STATE_ENABLED;
> > > + pm_runtime_put_sync(dsi->dev);
> > > +}
> > > +
> > > +static enum drm_connector_status
> > > +samsung_dsim_detect(struct drm_connector *connector, bool force)
> > > +{
> > > + return connector->status;
> > > +}
> > > +
> > > +static void samsung_dsim_connector_destroy(struct drm_connector 
> > > *connector)
> > > +{
> > > + drm_connector_unregister(connector);
> > > + drm_connector_cleanup(connector);
> > > + connector->dev = NULL;
> > > +}
> > > +
> > > +static const struct drm_connector_funcs samsung_dsim_connector_funcs = {
> > > + .detect = samsung_dsim_detect,
> > > + .fill_modes = drm_helper_probe_single_connector_modes,
> > > + .destroy = samsung_dsim_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 samsung_dsim_get_modes(struct drm_connector *connector)
> > > +{
> > > + struct samsung_dsim *dsi = connector_to_dsi(connector);
> > > +
> > > + if (dsi->panel)
> > > +         return drm_panel_get_modes(dsi->panel, connector);
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static const struct drm_connector_helper_funcs 
> > > samsung_dsim_connector_helper_funcs = {
> > > + .get_modes = samsung_dsim_get_modes,
> > > +};
> > > +
> > > +static int samsung_dsim_create_connector(struct samsung_dsim *dsi)
> > > +{
> > > + struct drm_connector *connector = &dsi->connector;
> > > + struct drm_device *drm = dsi->bridge.dev;
> > > + int ret;
> > > +
> > > + connector->polled = DRM_CONNECTOR_POLL_HPD;
> > > +
> > > + ret = drm_connector_init(drm, connector, &samsung_dsim_connector_funcs,
> > > +                          DRM_MODE_CONNECTOR_DSI);
> > > + if (ret) {
> > > +         DRM_DEV_ERROR(dsi->dev,
> > > +                       "Failed to initialize connector with drm\n");
> > > +         return ret;
> > > + }
> > > +
> > > + connector->status = connector_status_disconnected;
> > > + drm_connector_helper_add(connector, 
> > > &samsung_dsim_connector_helper_funcs);
> > > + drm_connector_attach_encoder(connector, dsi->bridge.encoder);
> > > + if (!drm->registered)
> > > +         return 0;
> > > +
> > > + connector->funcs->reset(connector);
> > > + drm_connector_register(connector);
> > > + return 0;
> > > +}
> > > +
> > > +static int samsung_dsim_bridge_attach(struct drm_bridge *bridge,
> > > +                               enum drm_bridge_attach_flags flags)
> > > +{
> > > + struct samsung_dsim *dsi = bridge->driver_private;
> > > + struct drm_encoder *encoder = bridge->encoder;
> > > + int ret;
> > > +
> > > + if (!dsi->out_bridge && !dsi->panel)
> > > +         return -EPROBE_DEFER;
> > > +
> > > + if (dsi->out_bridge) {
> > > +         ret = drm_bridge_attach(encoder, dsi->out_bridge,
> > > +                                 bridge, flags);
> > > +         if (ret)
> > > +                 return ret;
> > > +         list_splice_init(&encoder->bridge_chain, &dsi->bridge_chain);
> > > + } else {
> > > +         ret = samsung_dsim_create_connector(dsi);
> > > +         if (ret)
> > > +                 return ret;
> > > +
> > > +         if (dsi->panel) {
> > > +                 dsi->connector.status = connector_status_connected;
> > > +         }
> > > + }
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static void samsung_dsim_bridge_detach(struct drm_bridge *bridge)
> > > +{
> > > + struct samsung_dsim *dsi = bridge->driver_private;
> > > + struct drm_encoder *encoder = bridge->encoder;
> > > + struct drm_device *drm = encoder->dev;
> > > +
> > > + if (dsi->panel) {
> > > +         mutex_lock(&drm->mode_config.mutex);
> > > +         samsung_dsim_disable(dsi);
> > > +         dsi->panel = NULL;
> > > +         dsi->connector.status = connector_status_disconnected;
> > > +         mutex_unlock(&drm->mode_config.mutex);
> > > + } else {
> > > +         if (dsi->out_bridge->funcs->detach)
> > > +                 dsi->out_bridge->funcs->detach(dsi->out_bridge);
> > > +         dsi->out_bridge = NULL;
> > > +         INIT_LIST_HEAD(&dsi->bridge_chain);
> > > + }
> > > +}
> > > +
> > > +static void samsung_dsim_bridge_enable(struct drm_bridge *bridge)
> > > +{
> > > + struct samsung_dsim *dsi = bridge->driver_private;
> > > +
> > > + samsung_dsim_enable(dsi);
> > > +}
> > > +
> > > +static void samsung_dsim_bridge_disable(struct drm_bridge *bridge)
> > > +{
> > > + struct samsung_dsim *dsi = bridge->driver_private;
> > > +
> > > + samsung_dsim_disable(dsi);
> > > +}
> > > +
> > > +static void samsung_dsim_bridge_mode_set(struct drm_bridge *bridge,
> > > +                                  const struct drm_display_mode *mode,
> > > +                                  const struct drm_display_mode 
> > > *adjusted_mode)
> > > +{
> > > + struct samsung_dsim *dsi = bridge->driver_private;
> > > +
> > > + /* The mode is set when actually enabling the device. */
> > > + drm_mode_copy(&dsi->mode, adjusted_mode);
> > > +}
> > > +
> > > +static const struct drm_bridge_funcs samsung_dsim_bridge_funcs = {
> > > + .attach = samsung_dsim_bridge_attach,
> > > + .detach = samsung_dsim_bridge_detach,
> > > + .enable = samsung_dsim_bridge_enable,
> > > + .disable = samsung_dsim_bridge_disable,
> > > + .mode_set = samsung_dsim_bridge_mode_set,
> > > +};
> > > +
> > > +static int samsung_dsim_host_attach(struct mipi_dsi_host *host,
> > > +                             struct mipi_dsi_device *device)
> > > +{
> > > + struct samsung_dsim *dsi = host_to_dsi(host);
> > > + const struct samsung_dsim_host_ops *ops = dsi->driver_data->host_ops;
> > > + struct drm_bridge *out_bridge;
> > > +
> > > + out_bridge = of_drm_find_bridge(device->dev.of_node);
> > > + if (out_bridge) {
> > > +         dsi->out_bridge = out_bridge;
> > > + } else {
> > > +         dsi->panel = of_drm_find_panel(device->dev.of_node);
> > > +         if (IS_ERR(dsi->panel))
> > > +                 dsi->panel = NULL;
> > > +         else
> > > +                 dsi->connector.status = connector_status_connected;
> > > + }
> > > +
> > > + /*
> > > +  * This is a temporary solution and should be made by more generic way.
> > > +  *
> > > +  * If attached panel device is for command mode one, dsi should register
> > > +  * TE interrupt handler.
> > > +  */
> > > + if (!(device->mode_flags & MIPI_DSI_MODE_VIDEO)) {
> > > +         int ret = samsung_dsim_register_te_irq(dsi, &device->dev);
> > > +         if (ret)
> > > +                 return ret;
> > > + }
> > > +
> > > + dsi->lanes = device->lanes;
> > > + dsi->format = device->format;
> > > + dsi->mode_flags = device->mode_flags;
> > > +
> > > + if (ops && ops->attach)
> > > +         ops->attach(dsi->dsi_host.dev, device);
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static int samsung_dsim_host_detach(struct mipi_dsi_host *host,
> > > +                             struct mipi_dsi_device *device)
> > > +{
> > > + struct samsung_dsim *dsi = host_to_dsi(host);
> > > + const struct samsung_dsim_host_ops *ops = dsi->driver_data->host_ops;
> > > +
> > > + samsung_dsim_unregister_te_irq(dsi);
> > > +
> > > + if (ops && ops->detach)
> > > +         ops->detach(dsi->dsi_host.dev, device);
> > > +
> > > + samsung_dsim_unregister_te_irq(dsi);
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static ssize_t samsung_dsim_host_transfer(struct mipi_dsi_host *host,
> > > +                                   const struct mipi_dsi_msg *msg)
> > > +{
> > > + struct samsung_dsim *dsi = host_to_dsi(host);
> > > + struct samsung_dsim_transfer xfer;
> > > + int ret;
> > > +
> > > + if (!(dsi->state & DSIM_STATE_ENABLED))
> > > +         return -EINVAL;
> > > +
> > > + if (!(dsi->state & DSIM_STATE_INITIALIZED)) {
> > > +         ret = samsung_dsim_init(dsi);
> > > +         if (ret)
> > > +                 return ret;
> > > +         dsi->state |= DSIM_STATE_INITIALIZED;
> > > + }
> > > +
> > > + ret = mipi_dsi_create_packet(&xfer.packet, msg);
> > > + if (ret < 0)
> > > +         return ret;
> > > +
> > > + xfer.rx_len = msg->rx_len;
> > > + xfer.rx_payload = msg->rx_buf;
> > > + xfer.flags = msg->flags;
> > > +
> > > + ret = samsung_dsim_transfer(dsi, &xfer);
> > > + return (ret < 0) ? ret : xfer.rx_done;
> > > +}
> > > +
> > > +static const struct mipi_dsi_host_ops samsung_dsim_ops = {
> > > + .attach = samsung_dsim_host_attach,
> > > + .detach = samsung_dsim_host_detach,
> > > + .transfer = samsung_dsim_host_transfer,
> > > +};
> > > +
> > > +static int samsung_dsim_of_read_u32(const struct device_node *np,
> > > +                             const char *propname, u32 *out_value)
> > > +{
> > > + int ret = of_property_read_u32(np, propname, out_value);
> > > +
> > > + if (ret < 0)
> > > +         pr_err("%pOF: failed to get '%s' property\n", np, propname);
> > > +
> > > + return ret;
> > > +}
> > > +
> > > +static int samsung_dsim_parse_dt(struct samsung_dsim *dsi)
> > > +{
> > > + struct device *dev = dsi->dev;
> > > + struct device_node *node = dev->of_node;
> > > + int ret;
> > > +
> > > + ret = samsung_dsim_of_read_u32(node, "samsung,pll-clock-frequency",
> > > +                                &dsi->pll_clk_rate);
> > > + if (ret < 0)
> > > +         return ret;
> > > +
> > > + ret = samsung_dsim_of_read_u32(node, "samsung,burst-clock-frequency",
> > > +                                &dsi->burst_clk_rate);
> > > + if (ret < 0)
> > > +         return ret;
> > > +
> > > + ret = samsung_dsim_of_read_u32(node, "samsung,esc-clock-frequency",
> > > +                                &dsi->esc_clk_rate);
> > > + if (ret < 0)
> > > +         return ret;
> > > +
> > > + return 0;
> > > +}
> > > +
> > > +static struct samsung_dsim *__samsung_dsim_probe(struct platform_device 
> > > *pdev)
> > > +{
> > > + struct device *dev = &pdev->dev;
> > > + struct drm_bridge *bridge;
> > > + struct resource *res;
> > > + struct samsung_dsim *dsi;
> > > + int ret, i;
> > > +
> > > + dsi = devm_kzalloc(dev, sizeof(*dsi), GFP_KERNEL);
> > > + if (!dsi)
> > > +         return ERR_PTR(-ENOMEM);
> > > +
> > > + /* To be checked as invalid one */
> > > + dsi->te_gpio = -ENOENT;
> > > +
> > > + init_completion(&dsi->completed);
> > > + spin_lock_init(&dsi->transfer_lock);
> > > + INIT_LIST_HEAD(&dsi->transfer_list);
> > > + INIT_LIST_HEAD(&dsi->bridge_chain);
> > > +
> > > + dsi->dsi_host.ops = &samsung_dsim_ops;
> > > + dsi->dsi_host.dev = dev;
> > > +
> > > + dsi->dev = dev;
> > > + dsi->driver_data = of_device_get_match_data(dev);
> > > +
> > > + dsi->supplies[0].supply = "vddcore";
> > > + dsi->supplies[1].supply = "vddio";
> > > + ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(dsi->supplies),
> > > +                               dsi->supplies);
> > > + if (ret) {
> > > +         if (ret != -EPROBE_DEFER)
> > > +                 dev_info(dev, "failed to get regulators: %d\n", ret);
> > > +         return ERR_PTR(ret);
> > > + }
> > > +
> > > + dsi->clks = devm_kcalloc(dev,
> > > +                 dsi->driver_data->num_clks, sizeof(*dsi->clks),
> > > +                 GFP_KERNEL);
> > > + if (!dsi->clks)
> > > +         return ERR_PTR(-ENOMEM);
> > > +
> > > + for (i = 0; i < dsi->driver_data->num_clks; i++) {
> > > +         dsi->clks[i] = devm_clk_get(dev, clk_names[i]);
> > > +         if (IS_ERR(dsi->clks[i])) {
> > > +                 if (strcmp(clk_names[i], "sclk_mipi") == 0) {
> > > +                         dsi->clks[i] = devm_clk_get(dev,
> > > +                                                 OLD_SCLK_MIPI_CLK_NAME);
> > > +                         if (!IS_ERR(dsi->clks[i]))
> > > +                                 continue;
> > > +                 }
> > > +
> > > +                 dev_info(dev, "failed to get the clock: %s\n",
> > > +                                 clk_names[i]);
> > > +                 return ERR_PTR(PTR_ERR(dsi->clks[i]));
> > > +         }
> > > + }
> > > +
> > > + res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> > > + dsi->reg_base = devm_ioremap_resource(dev, res);
> > > + if (IS_ERR(dsi->reg_base)) {
> > > +         dev_err(dev, "failed to remap io region\n");
> > > +         return dsi->reg_base;
> > > + }
> > > +
> > > + dsi->phy = devm_phy_get(dev, "dsim");
> > > + if (IS_ERR(dsi->phy)) {
> > > +         dev_info(dev, "failed to get dsim phy\n");
> > > +         return ERR_PTR(PTR_ERR(dsi->phy));
> > > + }
> > > +
> > > + dsi->irq = platform_get_irq(pdev, 0);
> > > + if (dsi->irq < 0)
> > > +         return ERR_PTR(dsi->irq);
> > > +
> > > + irq_set_status_flags(dsi->irq, IRQ_NOAUTOEN);
> > > + ret = devm_request_threaded_irq(dev, dsi->irq, NULL,
> > > +                                 samsung_dsim_irq, IRQF_ONESHOT,
> > > +                                 dev_name(dev), dsi);
> > > + if (ret) {
> > > +         dev_err(dev, "failed to request dsi irq\n");
> > > +         return ERR_PTR(ret);
> > > + }
> > > +
> > > + ret = samsung_dsim_parse_dt(dsi);
> > > + if (ret)
> > > +         return ERR_PTR(ret);
> > > +
> > > + ret = mipi_dsi_host_register(&dsi->dsi_host);
> > > + if (ret)
> > > +         return ERR_PTR(ret);
> > > +
> > > + bridge = &dsi->bridge;
> > > + bridge->driver_private = dsi;
> > > + bridge->funcs = &samsung_dsim_bridge_funcs;
> > > + bridge->of_node = dev->of_node;
> > > + drm_bridge_add(bridge);
> > > +
> > > + return dsi;
> > > +}
> > > +
> > > +static void __samsung_dsim_remove(struct samsung_dsim *dsi)
> > > +{
> > > + drm_bridge_remove(&dsi->bridge);
> > > +
> > > + mipi_dsi_host_unregister(&dsi->dsi_host);
> > > +}
> > > +
> > > +/*
> > > + * Probe/remove API, used from platforms based on the DRM bridge API.
> > > + */
> > > +struct samsung_dsim *samsung_dsim_probe(struct platform_device *pdev)
> > > +{
> > > + return __samsung_dsim_probe(pdev);
> > > +}
> > > +EXPORT_SYMBOL_GPL(samsung_dsim_probe);
> > > +
> > > +void samsung_dsim_remove(struct samsung_dsim *dsi)
> > > +{
> > > + return __samsung_dsim_remove(dsi);
> > > +}
> > > +EXPORT_SYMBOL_GPL(samsung_dsim_remove);
> > > +
> > > +/*
> > > + * Bind/unbind API, used from platforms based on the component framework.
> > > + */
> > > +int samsung_dsim_bind(struct samsung_dsim *dsi, struct drm_encoder 
> > > *encoder)
> > > +{
> > > + struct drm_bridge *previous = 
> > > drm_bridge_chain_get_first_bridge(encoder);
> > > +
> > > + return drm_bridge_attach(encoder, &dsi->bridge, previous, 0);
> > > +}
> > > +EXPORT_SYMBOL_GPL(samsung_dsim_bind);
> > > +
> > > +void samsung_dsim_unbind(struct samsung_dsim *dsi)
> > > +{
> > > + samsung_dsim_disable(dsi);
> > > +}
> > > +EXPORT_SYMBOL_GPL(samsung_dsim_unbind);
> > > +
> > > +int samsung_dsim_suspend(struct samsung_dsim *dsi)
> > > +{
> > > + const struct samsung_dsim_driver_data *driver_data = dsi->driver_data;
> > > + int ret, i;
> > > +
> > > + usleep_range(10000, 20000);
> > > +
> > > + if (dsi->state & DSIM_STATE_INITIALIZED) {
> > > +         dsi->state &= ~DSIM_STATE_INITIALIZED;
> > > +
> > > +         samsung_dsim_disable_clock(dsi);
> > > +
> > > +         samsung_dsim_disable_irq(dsi);
> > > + }
> > > +
> > > + dsi->state &= ~DSIM_STATE_CMD_LPM;
> > > +
> > > + phy_power_off(dsi->phy);
> > > +
> > > + for (i = driver_data->num_clks - 1; i > -1; i--)
> > > +         clk_disable_unprepare(dsi->clks[i]);
> > > +
> > > + ret = regulator_bulk_disable(ARRAY_SIZE(dsi->supplies), dsi->supplies);
> > > + if (ret < 0)
> > > +         dev_err(dsi->dev, "cannot disable regulators %d\n", ret);
> > > +
> > > + return 0;
> > > +}
> > > +EXPORT_SYMBOL_GPL(samsung_dsim_suspend);
> > > +
> > > +int samsung_dsim_resume(struct samsung_dsim *dsi)
> > > +{
> > > + const struct samsung_dsim_driver_data *driver_data = dsi->driver_data;
> > > + int ret, i;
> > > +
> > > + ret = regulator_bulk_enable(ARRAY_SIZE(dsi->supplies), dsi->supplies);
> > > + if (ret < 0) {
> > > +         dev_err(dsi->dev, "cannot enable regulators %d\n", ret);
> > > +         return ret;
> > > + }
> > > +
> > > + for (i = 0; i < driver_data->num_clks; i++) {
> > > +         ret = clk_prepare_enable(dsi->clks[i]);
> > > +         if (ret < 0)
> > > +                 goto err_clk;
> > > + }
> > > +
> > > + ret = phy_power_on(dsi->phy);
> > > + if (ret < 0) {
> > > +         dev_err(dsi->dev, "cannot enable phy %d\n", ret);
> > > +         goto err_clk;
> > > + }
> > > +
> > > + return 0;
> > > +
> > > +err_clk:
> > > + while (--i > -1)
> > > +         clk_disable_unprepare(dsi->clks[i]);
> > > + regulator_bulk_disable(ARRAY_SIZE(dsi->supplies), dsi->supplies);
> > > +
> > > + return ret;
> > > +}
> > > +EXPORT_SYMBOL_GPL(samsung_dsim_resume);
> > > +
> > > +MODULE_AUTHOR("Tomasz Figa <t.f...@samsung.com>");
> > > +MODULE_AUTHOR("Andrzej Hajda <a.ha...@samsung.com>");
> > > +MODULE_DESCRIPTION("Samsung SoC MIPI DSI Master");
> > > +MODULE_LICENSE("GPL v2");
> > > diff --git a/drivers/gpu/drm/exynos/Kconfig 
> > > b/drivers/gpu/drm/exynos/Kconfig
> > > index 6417f374b923..3bc321ab5bc8 100644
> > > --- a/drivers/gpu/drm/exynos/Kconfig
> > > +++ b/drivers/gpu/drm/exynos/Kconfig
> > > @@ -16,7 +16,6 @@ comment "CRTCs"
> > >  
> > >  config DRM_EXYNOS_FIMD
> > >   bool "FIMD"
> > > - depends on !FB_S3C
> > >   select MFD_SYSCON
> > >   help
> > >     Choose this option if you want to use Exynos FIMD for DRM.
> > > @@ -28,7 +27,6 @@ config DRM_EXYNOS5433_DECON
> > >  
> > >  config DRM_EXYNOS7_DECON
> > >   bool "DECON on Exynos7"
> > > - depends on !FB_S3C
> > >   help
> > >     Choose this option if you want to use Exynos DECON for DRM.
> > >  
> > > @@ -55,8 +53,7 @@ config DRM_EXYNOS_DPI
> > >  config DRM_EXYNOS_DSI
> > >   bool "MIPI-DSI host"
> > >   depends on DRM_EXYNOS_FIMD || DRM_EXYNOS5433_DECON || DRM_EXYNOS7_DECON
> > > - select DRM_MIPI_DSI
> > > - select DRM_PANEL
> > > + select DRM_SAMSUNG_DSIM
> > >   default n
> > >   help
> > >     This enables support for Exynos MIPI-DSI device.
> > > diff --git a/drivers/gpu/drm/exynos/Makefile 
> > > b/drivers/gpu/drm/exynos/Makefile
> > > index add70b336935..2fd2f3ee4fcf 100644
> > > --- a/drivers/gpu/drm/exynos/Makefile
> > > +++ b/drivers/gpu/drm/exynos/Makefile
> > > @@ -11,7 +11,7 @@ exynosdrm-$(CONFIG_DRM_EXYNOS_FIMD)     += 
> > > exynos_drm_fimd.o
> > >  exynosdrm-$(CONFIG_DRM_EXYNOS5433_DECON) += exynos5433_drm_decon.o
> > >  exynosdrm-$(CONFIG_DRM_EXYNOS7_DECON)    += exynos7_drm_decon.o
> > >  exynosdrm-$(CONFIG_DRM_EXYNOS_DPI)       += exynos_drm_dpi.o
> > > -exynosdrm-$(CONFIG_DRM_EXYNOS_DSI)       += exynos_drm_dsi.o 
> > > exynos_drm_dsi_pltfm.o
> > > +exynosdrm-$(CONFIG_DRM_EXYNOS_DSI)       += exynos_drm_dsi.o
> > >  exynosdrm-$(CONFIG_DRM_EXYNOS_DP)        += exynos_dp.o
> > >  exynosdrm-$(CONFIG_DRM_EXYNOS_MIXER)     += exynos_mixer.o
> > >  exynosdrm-$(CONFIG_DRM_EXYNOS_HDMI)      += exynos_hdmi.o
> > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.c 
> > > b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > > index e8aea9d90c34..17f37fa74718 100644
> > > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > > +++ b/drivers/gpu/drm/exynos/exynos_drm_dsi.c
> > > @@ -5,1774 +5,328 @@
> > >   * Copyright (c) 2014 Samsung Electronics Co., Ltd
> > >   *
> > >   * Contacts: Tomasz Figa <t.f...@samsung.com>
> > > -*/
> > > + */
> > >  
> > > -#include <linux/clk.h>
> > > -#include <linux/delay.h>
> > >  #include <linux/component.h>
> > > -#include <linux/gpio/consumer.h>
> > > -#include <linux/irq.h>
> > >  #include <linux/of_device.h>
> > > -#include <linux/of_gpio.h>
> > >  #include <linux/of_graph.h>
> > > -#include <linux/phy/phy.h>
> > > -#include <linux/regulator/consumer.h>
> > > -
> > > -#include <asm/unaligned.h>
> > > +#include <linux/pm_runtime.h>
> > >  
> > > -#include <video/mipi_display.h>
> > > -#include <video/videomode.h>
> > > -
> > > -#include <drm/drm_atomic_helper.h>
> > > +#include <drm/bridge/samsung-dsim.h>
> > >  #include <drm/drm_bridge.h>
> > > -#include <drm/drm_fb_helper.h>
> > > +#include <drm/drm_encoder.h>
> > >  #include <drm/drm_mipi_dsi.h>
> > > -#include <drm/drm_panel.h>
> > > -#include <drm/drm_print.h>
> > >  #include <drm/drm_probe_helper.h>
> > >  #include <drm/drm_simple_kms_helper.h>
> > >  
> > > -#include "exynos_drm_dsi.h"
> > > -
> > > -/* returns true iff both arguments logically differs */
> > > -#define NEQV(a, b) (!(a) ^ !(b))
> > > -
> > > -/* DSIM_STATUS */
> > > -#define DSIM_STOP_STATE_DAT(x)           (((x) & 0xf) << 0)
> > > -#define DSIM_STOP_STATE_CLK              (1 << 8)
> > > -#define DSIM_TX_READY_HS_CLK             (1 << 10)
> > > -#define DSIM_PLL_STABLE                  (1 << 31)
> > > -
> > > -/* DSIM_TIMEOUT */
> > > -#define DSIM_LPDR_TIMEOUT(x)             ((x) << 0)
> > > -#define DSIM_BTA_TIMEOUT(x)              ((x) << 16)
> > > -
> > > -/* DSIM_CLKCTRL */
> > > -#define DSIM_ESC_PRESCALER(x)            (((x) & 0xffff) << 0)
> > > -#define DSIM_ESC_PRESCALER_MASK          (0xffff << 0)
> > > -#define DSIM_LANE_ESC_CLK_EN_CLK (1 << 19)
> > > -#define DSIM_LANE_ESC_CLK_EN_DATA(x)     (((x) & 0xf) << 20)
> > > -#define DSIM_LANE_ESC_CLK_EN_DATA_MASK   (0xf << 20)
> > > -#define DSIM_BYTE_CLKEN                  (1 << 24)
> > > -#define DSIM_BYTE_CLK_SRC(x)             (((x) & 0x3) << 25)
> > > -#define DSIM_BYTE_CLK_SRC_MASK           (0x3 << 25)
> > > -#define DSIM_PLL_BYPASS                  (1 << 27)
> > > -#define DSIM_ESC_CLKEN                   (1 << 28)
> > > -#define DSIM_TX_REQUEST_HSCLK            (1 << 31)
> > > -
> > > -/* DSIM_CONFIG */
> > > -#define DSIM_LANE_EN_CLK         (1 << 0)
> > > -#define DSIM_LANE_EN(x)                  (((x) & 0xf) << 1)
> > > -#define DSIM_NUM_OF_DATA_LANE(x) (((x) & 0x3) << 5)
> > > -#define DSIM_SUB_PIX_FORMAT(x)           (((x) & 0x7) << 8)
> > > -#define DSIM_MAIN_PIX_FORMAT_MASK        (0x7 << 12)
> > > -#define DSIM_MAIN_PIX_FORMAT_RGB888      (0x7 << 12)
> > > -#define DSIM_MAIN_PIX_FORMAT_RGB666      (0x6 << 12)
> > > -#define DSIM_MAIN_PIX_FORMAT_RGB666_P    (0x5 << 12)
> > > -#define DSIM_MAIN_PIX_FORMAT_RGB565      (0x4 << 12)
> > > -#define DSIM_SUB_VC                      (((x) & 0x3) << 16)
> > > -#define DSIM_MAIN_VC                     (((x) & 0x3) << 18)
> > > -#define DSIM_HSA_MODE                    (1 << 20)
> > > -#define DSIM_HBP_MODE                    (1 << 21)
> > > -#define DSIM_HFP_MODE                    (1 << 22)
> > > -#define DSIM_HSE_MODE                    (1 << 23)
> > > -#define DSIM_AUTO_MODE                   (1 << 24)
> > > -#define DSIM_VIDEO_MODE                  (1 << 25)
> > > -#define DSIM_BURST_MODE                  (1 << 26)
> > > -#define DSIM_SYNC_INFORM         (1 << 27)
> > > -#define DSIM_EOT_DISABLE         (1 << 28)
> > > -#define DSIM_MFLUSH_VS                   (1 << 29)
> > > -/* This flag is valid only for exynos3250/3472/5260/5430 */
> > > -#define DSIM_CLKLANE_STOP                (1 << 30)
> > > -
> > > -/* DSIM_ESCMODE */
> > > -#define DSIM_TX_TRIGGER_RST              (1 << 4)
> > > -#define DSIM_TX_LPDT_LP                  (1 << 6)
> > > -#define DSIM_CMD_LPDT_LP         (1 << 7)
> > > -#define DSIM_FORCE_BTA                   (1 << 16)
> > > -#define DSIM_FORCE_STOP_STATE            (1 << 20)
> > > -#define DSIM_STOP_STATE_CNT(x)           (((x) & 0x7ff) << 21)
> > > -#define DSIM_STOP_STATE_CNT_MASK (0x7ff << 21)
> > > -
> > > -/* DSIM_MDRESOL */
> > > -#define DSIM_MAIN_STAND_BY               (1 << 31)
> > > -#define DSIM_MAIN_VRESOL(x, num_bits)    (((x) & ((1 << (num_bits)) - 
> > > 1)) << 16)
> > > -#define DSIM_MAIN_HRESOL(x, num_bits)    (((x) & ((1 << (num_bits)) - 
> > > 1)) << 0)
> > > -
> > > -/* DSIM_MVPORCH */
> > > -#define DSIM_CMD_ALLOW(x)                ((x) << 28)
> > > -#define DSIM_STABLE_VFP(x)               ((x) << 16)
> > > -#define DSIM_MAIN_VBP(x)         ((x) << 0)
> > > -#define DSIM_CMD_ALLOW_MASK              (0xf << 28)
> > > -#define DSIM_STABLE_VFP_MASK             (0x7ff << 16)
> > > -#define DSIM_MAIN_VBP_MASK               (0x7ff << 0)
> > > -
> > > -/* DSIM_MHPORCH */
> > > -#define DSIM_MAIN_HFP(x)         ((x) << 16)
> > > -#define DSIM_MAIN_HBP(x)         ((x) << 0)
> > > -#define DSIM_MAIN_HFP_MASK               ((0xffff) << 16)
> > > -#define DSIM_MAIN_HBP_MASK               ((0xffff) << 0)
> > > -
> > > -/* DSIM_MSYNC */
> > > -#define DSIM_MAIN_VSA(x)         ((x) << 22)
> > > -#define DSIM_MAIN_HSA(x)         ((x) << 0)
> > > -#define DSIM_MAIN_VSA_MASK               ((0x3ff) << 22)
> > > -#define DSIM_MAIN_HSA_MASK               ((0xffff) << 0)
> > > -
> > > -/* DSIM_SDRESOL */
> > > -#define DSIM_SUB_STANDY(x)               ((x) << 31)
> > > -#define DSIM_SUB_VRESOL(x)               ((x) << 16)
> > > -#define DSIM_SUB_HRESOL(x)               ((x) << 0)
> > > -#define DSIM_SUB_STANDY_MASK             ((0x1) << 31)
> > > -#define DSIM_SUB_VRESOL_MASK             ((0x7ff) << 16)
> > > -#define DSIM_SUB_HRESOL_MASK             ((0x7ff) << 0)
> > > -
> > > -/* DSIM_INTSRC */
> > > -#define DSIM_INT_PLL_STABLE              (1 << 31)
> > > -#define DSIM_INT_SW_RST_RELEASE          (1 << 30)
> > > -#define DSIM_INT_SFR_FIFO_EMPTY          (1 << 29)
> > > -#define DSIM_INT_SFR_HDR_FIFO_EMPTY      (1 << 28)
> > > -#define DSIM_INT_BTA                     (1 << 25)
> > > -#define DSIM_INT_FRAME_DONE              (1 << 24)
> > > -#define DSIM_INT_RX_TIMEOUT              (1 << 21)
> > > -#define DSIM_INT_BTA_TIMEOUT             (1 << 20)
> > > -#define DSIM_INT_RX_DONE         (1 << 18)
> > > -#define DSIM_INT_RX_TE                   (1 << 17)
> > > -#define DSIM_INT_RX_ACK                  (1 << 16)
> > > -#define DSIM_INT_RX_ECC_ERR              (1 << 15)
> > > -#define DSIM_INT_RX_CRC_ERR              (1 << 14)
> > > +#include "exynos_drm_crtc.h"
> > > +#include "exynos_drm_drv.h"
> > >  
> > > -/* DSIM_FIFOCTRL */
> > > -#define DSIM_RX_DATA_FULL                (1 << 25)
> > > -#define DSIM_RX_DATA_EMPTY               (1 << 24)
> > > -#define DSIM_SFR_HEADER_FULL             (1 << 23)
> > > -#define DSIM_SFR_HEADER_EMPTY            (1 << 22)
> > > -#define DSIM_SFR_PAYLOAD_FULL            (1 << 21)
> > > -#define DSIM_SFR_PAYLOAD_EMPTY           (1 << 20)
> > > -#define DSIM_I80_HEADER_FULL             (1 << 19)
> > > -#define DSIM_I80_HEADER_EMPTY            (1 << 18)
> > > -#define DSIM_I80_PAYLOAD_FULL            (1 << 17)
> > > -#define DSIM_I80_PAYLOAD_EMPTY           (1 << 16)
> > > -#define DSIM_SD_HEADER_FULL              (1 << 15)
> > > -#define DSIM_SD_HEADER_EMPTY             (1 << 14)
> > > -#define DSIM_SD_PAYLOAD_FULL             (1 << 13)
> > > -#define DSIM_SD_PAYLOAD_EMPTY            (1 << 12)
> > > -#define DSIM_MD_HEADER_FULL              (1 << 11)
> > > -#define DSIM_MD_HEADER_EMPTY             (1 << 10)
> > > -#define DSIM_MD_PAYLOAD_FULL             (1 << 9)
> > > -#define DSIM_MD_PAYLOAD_EMPTY            (1 << 8)
> > > -#define DSIM_RX_FIFO                     (1 << 4)
> > > -#define DSIM_SFR_FIFO                    (1 << 3)
> > > -#define DSIM_I80_FIFO                    (1 << 2)
> > > -#define DSIM_SD_FIFO                     (1 << 1)
> > > -#define DSIM_MD_FIFO                     (1 << 0)
> > > -
> > > -/* DSIM_PHYACCHR */
> > > -#define DSIM_AFC_EN                      (1 << 14)
> > > -#define DSIM_AFC_CTL(x)                  (((x) & 0x7) << 5)
> > > -
> > > -/* DSIM_PLLCTRL */
> > > -#define DSIM_FREQ_BAND(x)                ((x) << 24)
> > > -#define DSIM_PLL_EN                      (1 << 23)
> > > -#define DSIM_PLL_P(x)                    ((x) << 13)
> > > -#define DSIM_PLL_M(x)                    ((x) << 4)
> > > -#define DSIM_PLL_S(x)                    ((x) << 1)
> > > -
> > > -/* DSIM_PHYCTRL */
> > > -#define DSIM_PHYCTRL_ULPS_EXIT(x)        (((x) & 0x1ff) << 0)
> > > -#define DSIM_PHYCTRL_B_DPHYCTL_VREG_LP   (1 << 30)
> > > -#define DSIM_PHYCTRL_B_DPHYCTL_SLEW_UP   (1 << 14)
> > > -
> > > -/* DSIM_PHYTIMING */
> > > -#define DSIM_PHYTIMING_LPX(x)            ((x) << 8)
> > > -#define DSIM_PHYTIMING_HS_EXIT(x)        ((x) << 0)
> > > -
> > > -/* DSIM_PHYTIMING1 */
> > > -#define DSIM_PHYTIMING1_CLK_PREPARE(x)   ((x) << 24)
> > > -#define DSIM_PHYTIMING1_CLK_ZERO(x)      ((x) << 16)
> > > -#define DSIM_PHYTIMING1_CLK_POST(x)      ((x) << 8)
> > > -#define DSIM_PHYTIMING1_CLK_TRAIL(x)     ((x) << 0)
> > > -
> > > -/* DSIM_PHYTIMING2 */
> > > -#define DSIM_PHYTIMING2_HS_PREPARE(x)    ((x) << 16)
> > > -#define DSIM_PHYTIMING2_HS_ZERO(x)       ((x) << 8)
> > > -#define DSIM_PHYTIMING2_HS_TRAIL(x)      ((x) << 0)
> > > -
> > > -#define DSI_MAX_BUS_WIDTH                4
> > > -#define DSI_NUM_VIRTUAL_CHANNELS 4
> > > -#define DSI_TX_FIFO_SIZE         2048
> > > -#define DSI_RX_FIFO_SIZE         256
> > > -#define DSI_XFER_TIMEOUT_MS              100
> > > -#define DSI_RX_FIFO_EMPTY                0x30800002
> > > -
> > > -#define OLD_SCLK_MIPI_CLK_NAME "pll_clk"
> > > -
> > > -static const char *const clk_names[5] = { "bus_clk", "sclk_mipi",
> > > - "phyclk_mipidphy0_bitclkdiv8", "phyclk_mipidphy0_rxclkesc0",
> > > - "sclk_rgb_vclk_to_dsim0" };
> > > -
> > > -enum exynos_dsi_transfer_type {
> > > - EXYNOS_DSI_TX,
> > > - EXYNOS_DSI_RX,
> > > +enum {
> > > + DSI_PORT_IN,
> > > + DSI_PORT_OUT
> > >  };
> > >  
> > > -struct exynos_dsi_transfer {
> > > - struct list_head list;
> > > - struct completion completed;
> > > - int result;
> > > - struct mipi_dsi_packet packet;
> > > - u16 flags;
> > > - u16 tx_done;
> > > -
> > > - u8 *rx_payload;
> > > - u16 rx_len;
> > > - u16 rx_done;
> > > -};
> > > -
> > > -#define DSIM_STATE_ENABLED               BIT(0)
> > > -#define DSIM_STATE_INITIALIZED           BIT(1)
> > > -#define DSIM_STATE_CMD_LPM               BIT(2)
> > > -#define DSIM_STATE_VIDOUT_AVAILABLE      BIT(3)
> > > -
> > >  struct exynos_dsi {
> > > - struct drm_bridge bridge;
> > > - struct mipi_dsi_host dsi_host;
> > > - struct drm_connector connector;
> > > - struct drm_panel *panel;
> > > - struct list_head bridge_chain;
> > > - struct drm_bridge *out_bridge;
> > > - struct device *dev;
> > > -
> > > - void __iomem *reg_base;
> > > - struct phy *phy;
> > > - struct clk **clks;
> > > - struct regulator_bulk_data supplies[2];
> > > - int irq;
> > > - int te_gpio;
> > > -
> > > - u32 pll_clk_rate;
> > > - u32 burst_clk_rate;
> > > - u32 esc_clk_rate;
> > > - u32 lanes;
> > > - u32 mode_flags;
> > > - u32 format;
> > > -
> > > - struct drm_display_mode mode;
> > > -
> > > - int state;
> > > - struct drm_property *brightness;
> > > - struct completion completed;
> > > -
> > > - spinlock_t transfer_lock; /* protects transfer_list */
> > > - struct list_head transfer_list;
> > > -
> > > - const struct exynos_dsi_driver_data *driver_data;
> > > + struct samsung_dsim *dsi;
> > > + struct drm_encoder encoder;
> > >  };
> > >  
> > > -#define host_to_dsi(host) container_of(host, struct exynos_dsi, dsi_host)
> > > -#define connector_to_dsi(c) container_of(c, struct exynos_dsi, connector)
> > > -
> > > -enum reg_idx {
> > > - DSIM_STATUS_REG,        /* Status register */
> > > - DSIM_SWRST_REG,         /* Software reset register */
> > > - DSIM_CLKCTRL_REG,       /* Clock control register */
> > > - DSIM_TIMEOUT_REG,       /* Time out register */
> > > - DSIM_CONFIG_REG,        /* Configuration register */
> > > - DSIM_ESCMODE_REG,       /* Escape mode register */
> > > - DSIM_MDRESOL_REG,
> > > - DSIM_MVPORCH_REG,       /* Main display Vporch register */
> > > - DSIM_MHPORCH_REG,       /* Main display Hporch register */
> > > - DSIM_MSYNC_REG,         /* Main display sync area register */
> > > - DSIM_INTSRC_REG,        /* Interrupt source register */
> > > - DSIM_INTMSK_REG,        /* Interrupt mask register */
> > > - DSIM_PKTHDR_REG,        /* Packet Header FIFO register */
> > > - DSIM_PAYLOAD_REG,       /* Payload FIFO register */
> > > - DSIM_RXFIFO_REG,        /* Read FIFO register */
> > > - DSIM_FIFOCTRL_REG,      /* FIFO status and control register */
> > > - DSIM_PLLCTRL_REG,       /* PLL control register */
> > > - DSIM_PHYCTRL_REG,
> > > - DSIM_PHYTIMING_REG,
> > > - DSIM_PHYTIMING1_REG,
> > > - DSIM_PHYTIMING2_REG,
> > > - NUM_REGS
> > > +static const unsigned int reg_values[] = {
> > > + [RESET_TYPE] = DSIM_SWRST,
> > > + [PLL_TIMER] = 500,
> > > + [STOP_STATE_CNT] = 0xf,
> > > + [PHYCTRL_ULPS_EXIT] = 0x0af,
> > > + [PHYCTRL_VREG_LP] = 0,
> > > + [PHYCTRL_SLEW_UP] = 0,
> > > + [PHYTIMING_LPX] = 0x06,
> > > + [PHYTIMING_HS_EXIT] = 0x0b,
> > > + [PHYTIMING_CLK_PREPARE] = 0x07,
> > > + [PHYTIMING_CLK_ZERO] = 0x27,
> > > + [PHYTIMING_CLK_POST] = 0x0d,
> > > + [PHYTIMING_CLK_TRAIL] = 0x08,
> > > + [PHYTIMING_HS_PREPARE] = 0x09,
> > > + [PHYTIMING_HS_ZERO] = 0x0d,
> > > + [PHYTIMING_HS_TRAIL] = 0x0b,
> > >  };
> > >  
> > > -static const unsigned int exynos_reg_ofs[] = {
> > > - [DSIM_STATUS_REG] =  0x00,
> > > - [DSIM_SWRST_REG] =  0x04,
> > > - [DSIM_CLKCTRL_REG] =  0x08,
> > > - [DSIM_TIMEOUT_REG] =  0x0c,
> > > - [DSIM_CONFIG_REG] =  0x10,
> > > - [DSIM_ESCMODE_REG] =  0x14,
> > > - [DSIM_MDRESOL_REG] =  0x18,
> > > - [DSIM_MVPORCH_REG] =  0x1c,
> > > - [DSIM_MHPORCH_REG] =  0x20,
> > > - [DSIM_MSYNC_REG] =  0x24,
> > > - [DSIM_INTSRC_REG] =  0x2c,
> > > - [DSIM_INTMSK_REG] =  0x30,
> > > - [DSIM_PKTHDR_REG] =  0x34,
> > > - [DSIM_PAYLOAD_REG] =  0x38,
> > > - [DSIM_RXFIFO_REG] =  0x3c,
> > > - [DSIM_FIFOCTRL_REG] =  0x44,
> > > - [DSIM_PLLCTRL_REG] =  0x4c,
> > > - [DSIM_PHYCTRL_REG] =  0x5c,
> > > - [DSIM_PHYTIMING_REG] =  0x64,
> > > - [DSIM_PHYTIMING1_REG] =  0x68,
> > > - [DSIM_PHYTIMING2_REG] =  0x6c,
> > > +static const unsigned int exynos5422_reg_values[] = {
> > > + [RESET_TYPE] = DSIM_SWRST,
> > > + [PLL_TIMER] = 500,
> > > + [STOP_STATE_CNT] = 0xf,
> > > + [PHYCTRL_ULPS_EXIT] = 0xaf,
> > > + [PHYCTRL_VREG_LP] = 0,
> > > + [PHYCTRL_SLEW_UP] = 0,
> > > + [PHYTIMING_LPX] = 0x08,
> > > + [PHYTIMING_HS_EXIT] = 0x0d,
> > > + [PHYTIMING_CLK_PREPARE] = 0x09,
> > > + [PHYTIMING_CLK_ZERO] = 0x30,
> > > + [PHYTIMING_CLK_POST] = 0x0e,
> > > + [PHYTIMING_CLK_TRAIL] = 0x0a,
> > > + [PHYTIMING_HS_PREPARE] = 0x0c,
> > > + [PHYTIMING_HS_ZERO] = 0x11,
> > > + [PHYTIMING_HS_TRAIL] = 0x0d,
> > >  };
> > >  
> > > -static const unsigned int exynos5433_reg_ofs[] = {
> > > - [DSIM_STATUS_REG] = 0x04,
> > > - [DSIM_SWRST_REG] = 0x0C,
> > > - [DSIM_CLKCTRL_REG] = 0x10,
> > > - [DSIM_TIMEOUT_REG] = 0x14,
> > > - [DSIM_CONFIG_REG] = 0x18,
> > > - [DSIM_ESCMODE_REG] = 0x1C,
> > > - [DSIM_MDRESOL_REG] = 0x20,
> > > - [DSIM_MVPORCH_REG] = 0x24,
> > > - [DSIM_MHPORCH_REG] = 0x28,
> > > - [DSIM_MSYNC_REG] = 0x2C,
> > > - [DSIM_INTSRC_REG] = 0x34,
> > > - [DSIM_INTMSK_REG] = 0x38,
> > > - [DSIM_PKTHDR_REG] = 0x3C,
> > > - [DSIM_PAYLOAD_REG] = 0x40,
> > > - [DSIM_RXFIFO_REG] = 0x44,
> > > - [DSIM_FIFOCTRL_REG] = 0x4C,
> > > - [DSIM_PLLCTRL_REG] = 0x94,
> > > - [DSIM_PHYCTRL_REG] = 0xA4,
> > > - [DSIM_PHYTIMING_REG] = 0xB4,
> > > - [DSIM_PHYTIMING1_REG] = 0xB8,
> > > - [DSIM_PHYTIMING2_REG] = 0xBC,
> > > +static const unsigned int exynos5433_reg_values[] = {
> > > + [RESET_TYPE] = DSIM_FUNCRST,
> > > + [PLL_TIMER] = 22200,
> > > + [STOP_STATE_CNT] = 0xa,
> > > + [PHYCTRL_ULPS_EXIT] = 0x190,
> > > + [PHYCTRL_VREG_LP] = 1,
> > > + [PHYCTRL_SLEW_UP] = 1,
> > > + [PHYTIMING_LPX] = 0x07,
> > > + [PHYTIMING_HS_EXIT] = 0x0c,
> > > + [PHYTIMING_CLK_PREPARE] = 0x09,
> > > + [PHYTIMING_CLK_ZERO] = 0x2d,
> > > + [PHYTIMING_CLK_POST] = 0x0e,
> > > + [PHYTIMING_CLK_TRAIL] = 0x09,
> > > + [PHYTIMING_HS_PREPARE] = 0x0b,
> > > + [PHYTIMING_HS_ZERO] = 0x10,
> > > + [PHYTIMING_HS_TRAIL] = 0x0c,
> > >  };
> > >  
> > > -static inline void exynos_dsi_write(struct exynos_dsi *dsi, enum reg_idx 
> > > idx,
> > > -                             u32 val)
> > > -{
> > > - const unsigned int *reg_ofs;
> > > -
> > > - if (dsi->driver_data->reg_ofs == EXYNOS5433_REG_OFS)
> > > -         reg_ofs = exynos5433_reg_ofs;
> > > - else
> > > -         reg_ofs = exynos_reg_ofs;
> > > -
> > > - writel(val, dsi->reg_base + reg_ofs[idx]);
> > > -}
> > > -
> > > -static inline u32 exynos_dsi_read(struct exynos_dsi *dsi, enum reg_idx 
> > > idx)
> > > -{
> > > - const unsigned int *reg_ofs;
> > > -
> > > - if (dsi->driver_data->reg_ofs == EXYNOS5433_REG_OFS)
> > > -         reg_ofs = exynos5433_reg_ofs;
> > > - else
> > > -         reg_ofs = exynos_reg_ofs;
> > > -
> > > - return readl(dsi->reg_base + reg_ofs[idx]);
> > > -}
> > > -
> > > -static void exynos_dsi_wait_for_reset(struct exynos_dsi *dsi)
> > > -{
> > > - if (wait_for_completion_timeout(&dsi->completed, msecs_to_jiffies(300)))
> > > -         return;
> > > -
> > > - dev_err(dsi->dev, "timeout waiting for reset\n");
> > > -}
> > > -
> > > -static void exynos_dsi_reset(struct exynos_dsi *dsi)
> > > -{
> > > - u32 reset_val = dsi->driver_data->reg_values[RESET_TYPE];
> > > -
> > > - reinit_completion(&dsi->completed);
> > > - exynos_dsi_write(dsi, DSIM_SWRST_REG, reset_val);
> > > -}
> > > -
> > > -#ifndef MHZ
> > > -#define MHZ      (1000*1000)
> > > -#endif
> > > -
> > > -static unsigned long exynos_dsi_pll_find_pms(struct exynos_dsi *dsi,
> > > -         unsigned long fin, unsigned long fout, u8 *p, u16 *m, u8 *s)
> > > -{
> > > - const struct exynos_dsi_driver_data *driver_data = dsi->driver_data;
> > > - unsigned long best_freq = 0;
> > > - u32 min_delta = 0xffffffff;
> > > - u8 p_min, p_max;
> > > - u8 _p, best_p;
> > > - u16 _m, best_m;
> > > - u8 _s, best_s;
> > > -
> > > - p_min = DIV_ROUND_UP(fin, (12 * MHZ));
> > > - p_max = fin / (6 * MHZ);
> > > -
> > > - for (_p = p_min; _p <= p_max; ++_p) {
> > > -         for (_s = 0; _s <= 5; ++_s) {
> > > -                 u64 tmp;
> > > -                 u32 delta;
> > > -
> > > -                 tmp = (u64)fout * (_p << _s);
> > > -                 do_div(tmp, fin);
> > > -                 _m = tmp;
> > > -                 if (_m < 41 || _m > 125)
> > > -                         continue;
> > > -
> > > -                 tmp = (u64)_m * fin;
> > > -                 do_div(tmp, _p);
> > > -                 if (tmp < 500 * MHZ ||
> > > -                                 tmp > driver_data->max_freq * MHZ)
> > > -                         continue;
> > > -
> > > -                 tmp = (u64)_m * fin;
> > > -                 do_div(tmp, _p << _s);
> > > -
> > > -                 delta = abs(fout - tmp);
> > > -                 if (delta < min_delta) {
> > > -                         best_p = _p;
> > > -                         best_m = _m;
> > > -                         best_s = _s;
> > > -                         min_delta = delta;
> > > -                         best_freq = tmp;
> > > -                 }
> > > -         }
> > > - }
> > > -
> > > - if (best_freq) {
> > > -         *p = best_p;
> > > -         *m = best_m;
> > > -         *s = best_s;
> > > - }
> > > -
> > > - return best_freq;
> > > -}
> > > -
> > > -static unsigned long exynos_dsi_set_pll(struct exynos_dsi *dsi,
> > > -                                 unsigned long freq)
> > > -{
> > > - const struct exynos_dsi_driver_data *driver_data = dsi->driver_data;
> > > - unsigned long fin, fout;
> > > - int timeout;
> > > - u8 p, s;
> > > - u16 m;
> > > - u32 reg;
> > > -
> > > - fin = dsi->pll_clk_rate;
> > > - fout = exynos_dsi_pll_find_pms(dsi, fin, freq, &p, &m, &s);
> > > - if (!fout) {
> > > -         dev_err(dsi->dev,
> > > -                 "failed to find PLL PMS for requested frequency\n");
> > > -         return 0;
> > > - }
> > > - dev_dbg(dsi->dev, "PLL freq %lu, (p %d, m %d, s %d)\n", fout, p, m, s);
> > > -
> > > - writel(driver_data->reg_values[PLL_TIMER],
> > > -                 dsi->reg_base + driver_data->plltmr_reg);
> > > -
> > > - reg = DSIM_PLL_EN | DSIM_PLL_P(p) | DSIM_PLL_M(m) | DSIM_PLL_S(s);
> > > -
> > > - if (driver_data->has_freqband) {
> > > -         static const unsigned long freq_bands[] = {
> > > -                 100 * MHZ, 120 * MHZ, 160 * MHZ, 200 * MHZ,
> > > -                 270 * MHZ, 320 * MHZ, 390 * MHZ, 450 * MHZ,
> > > -                 510 * MHZ, 560 * MHZ, 640 * MHZ, 690 * MHZ,
> > > -                 770 * MHZ, 870 * MHZ, 950 * MHZ,
> > > -         };
> > > -         int band;
> > > -
> > > -         for (band = 0; band < ARRAY_SIZE(freq_bands); ++band)
> > > -                 if (fout < freq_bands[band])
> > > -                         break;
> > > -
> > > -         dev_dbg(dsi->dev, "band %d\n", band);
> > > -
> > > -         reg |= DSIM_FREQ_BAND(band);
> > > - }
> > > -
> > > - exynos_dsi_write(dsi, DSIM_PLLCTRL_REG, reg);
> > > -
> > > - timeout = 1000;
> > > - do {
> > > -         if (timeout-- == 0) {
> > > -                 dev_err(dsi->dev, "PLL failed to stabilize\n");
> > > -                 return 0;
> > > -         }
> > > -         reg = exynos_dsi_read(dsi, DSIM_STATUS_REG);
> > > - } while ((reg & DSIM_PLL_STABLE) == 0);
> > > -
> > > - return fout;
> > > -}
> > > -
> > > -static int exynos_dsi_enable_clock(struct exynos_dsi *dsi)
> > > -{
> > > - unsigned long hs_clk, byte_clk, esc_clk;
> > > - unsigned long esc_div;
> > > - u32 reg;
> > > -
> > > - hs_clk = exynos_dsi_set_pll(dsi, dsi->burst_clk_rate);
> > > - if (!hs_clk) {
> > > -         dev_err(dsi->dev, "failed to configure DSI PLL\n");
> > > -         return -EFAULT;
> > > - }
> > > -
> > > - byte_clk = hs_clk / 8;
> > > - esc_div = DIV_ROUND_UP(byte_clk, dsi->esc_clk_rate);
> > > - esc_clk = byte_clk / esc_div;
> > > -
> > > - if (esc_clk > 20 * MHZ) {
> > > -         ++esc_div;
> > > -         esc_clk = byte_clk / esc_div;
> > > - }
> > > -
> > > - dev_dbg(dsi->dev, "hs_clk = %lu, byte_clk = %lu, esc_clk = %lu\n",
> > > -         hs_clk, byte_clk, esc_clk);
> > > -
> > > - reg = exynos_dsi_read(dsi, DSIM_CLKCTRL_REG);
> > > - reg &= ~(DSIM_ESC_PRESCALER_MASK | DSIM_LANE_ESC_CLK_EN_CLK
> > > -                 | DSIM_LANE_ESC_CLK_EN_DATA_MASK | DSIM_PLL_BYPASS
> > > -                 | DSIM_BYTE_CLK_SRC_MASK);
> > > - reg |= DSIM_ESC_CLKEN | DSIM_BYTE_CLKEN
> > > -                 | DSIM_ESC_PRESCALER(esc_div)
> > > -                 | DSIM_LANE_ESC_CLK_EN_CLK
> > > -                 | DSIM_LANE_ESC_CLK_EN_DATA(BIT(dsi->lanes) - 1)
> > > -                 | DSIM_BYTE_CLK_SRC(0)
> > > -                 | DSIM_TX_REQUEST_HSCLK;
> > > - exynos_dsi_write(dsi, DSIM_CLKCTRL_REG, reg);
> > > -
> > > - return 0;
> > > -}
> > > -
> > > -static void exynos_dsi_set_phy_ctrl(struct exynos_dsi *dsi)
> > > -{
> > > - const struct exynos_dsi_driver_data *driver_data = dsi->driver_data;
> > > - const unsigned int *reg_values = driver_data->reg_values;
> > > - u32 reg;
> > > -
> > > - if (driver_data->has_freqband)
> > > -         return;
> > > -
> > > - /* B D-PHY: D-PHY Master & Slave Analog Block control */
> > > - reg = DSIM_PHYCTRL_ULPS_EXIT(reg_values[PHYCTRL_ULPS_EXIT]);
> > > - if (reg_values[PHYCTRL_VREG_LP])
> > > -         reg |= DSIM_PHYCTRL_B_DPHYCTL_VREG_LP;
> > > - if (reg_values[PHYCTRL_SLEW_UP])
> > > -         reg |= DSIM_PHYCTRL_B_DPHYCTL_SLEW_UP;
> > > - exynos_dsi_write(dsi, DSIM_PHYCTRL_REG, reg);
> > > -
> > > - /*
> > > -  * T LPX: Transmitted length of any Low-Power state period
> > > -  * T HS-EXIT: Time that the transmitter drives LP-11 following a HS
> > > -  *      burst
> > > -  */
> > > - reg = DSIM_PHYTIMING_LPX(reg_values[PHYTIMING_LPX]) |
> > > -         DSIM_PHYTIMING_HS_EXIT(reg_values[PHYTIMING_HS_EXIT]);
> > > - exynos_dsi_write(dsi, DSIM_PHYTIMING_REG, reg);
> > > -
> > > - /*
> > > -  * T CLK-PREPARE: Time that the transmitter drives the Clock Lane LP-00
> > > -  *      Line state immediately before the HS-0 Line state starting the
> > > -  *      HS transmission
> > > -  * T CLK-ZERO: Time that the transmitter drives the HS-0 state prior to
> > > -  *      transmitting the Clock.
> > > -  * T CLK_POST: Time that the transmitter continues to send HS clock
> > > -  *      after the last associated Data Lane has transitioned to LP Mode
> > > -  *      Interval is defined as the period from the end of T HS-TRAIL to
> > > -  *      the beginning of T CLK-TRAIL
> > > -  * T CLK-TRAIL: Time that the transmitter drives the HS-0 state after
> > > -  *      the last payload clock bit of a HS transmission burst
> > > -  */
> > > - reg = DSIM_PHYTIMING1_CLK_PREPARE(reg_values[PHYTIMING_CLK_PREPARE]) |
> > > -         DSIM_PHYTIMING1_CLK_ZERO(reg_values[PHYTIMING_CLK_ZERO]) |
> > > -         DSIM_PHYTIMING1_CLK_POST(reg_values[PHYTIMING_CLK_POST]) |
> > > -         DSIM_PHYTIMING1_CLK_TRAIL(reg_values[PHYTIMING_CLK_TRAIL]);
> > > - exynos_dsi_write(dsi, DSIM_PHYTIMING1_REG, reg);
> > > -
> > > - /*
> > > -  * T HS-PREPARE: Time that the transmitter drives the Data Lane LP-00
> > > -  *      Line state immediately before the HS-0 Line state starting the
> > > -  *      HS transmission
> > > -  * T HS-ZERO: Time that the transmitter drives the HS-0 state prior to
> > > -  *      transmitting the Sync sequence.
> > > -  * T HS-TRAIL: Time that the transmitter drives the flipped differential
> > > -  *      state after last payload data bit of a HS transmission burst
> > > -  */
> > > - reg = DSIM_PHYTIMING2_HS_PREPARE(reg_values[PHYTIMING_HS_PREPARE]) |
> > > -         DSIM_PHYTIMING2_HS_ZERO(reg_values[PHYTIMING_HS_ZERO]) |
> > > -         DSIM_PHYTIMING2_HS_TRAIL(reg_values[PHYTIMING_HS_TRAIL]);
> > > - exynos_dsi_write(dsi, DSIM_PHYTIMING2_REG, reg);
> > > -}
> > > -
> > > -static void exynos_dsi_disable_clock(struct exynos_dsi *dsi)
> > > -{
> > > - u32 reg;
> > > -
> > > - reg = exynos_dsi_read(dsi, DSIM_CLKCTRL_REG);
> > > - reg &= ~(DSIM_LANE_ESC_CLK_EN_CLK | DSIM_LANE_ESC_CLK_EN_DATA_MASK
> > > -                 | DSIM_ESC_CLKEN | DSIM_BYTE_CLKEN);
> > > - exynos_dsi_write(dsi, DSIM_CLKCTRL_REG, reg);
> > > -
> > > - reg = exynos_dsi_read(dsi, DSIM_PLLCTRL_REG);
> > > - reg &= ~DSIM_PLL_EN;
> > > - exynos_dsi_write(dsi, DSIM_PLLCTRL_REG, reg);
> > > -}
> > > -
> > > -static void exynos_dsi_enable_lane(struct exynos_dsi *dsi, u32 lane)
> > > -{
> > > - u32 reg = exynos_dsi_read(dsi, DSIM_CONFIG_REG);
> > > - reg |= (DSIM_NUM_OF_DATA_LANE(dsi->lanes - 1) | DSIM_LANE_EN_CLK |
> > > -                 DSIM_LANE_EN(lane));
> > > - exynos_dsi_write(dsi, DSIM_CONFIG_REG, reg);
> > > -}
> > > -
> > > -static int exynos_dsi_init_link(struct exynos_dsi *dsi)
> > > +static int exynos_dsi_host_attach(struct device *dev,
> > > +                           struct mipi_dsi_device *device)
> > >  {
> > > - const struct exynos_dsi_driver_data *driver_data = dsi->driver_data;
> > > - int timeout;
> > > - u32 reg;
> > > - u32 lanes_mask;
> > > -
> > > - /* Initialize FIFO pointers */
> > > - reg = exynos_dsi_read(dsi, DSIM_FIFOCTRL_REG);
> > > - reg &= ~0x1f;
> > > - exynos_dsi_write(dsi, DSIM_FIFOCTRL_REG, reg);
> > > -
> > > - usleep_range(9000, 11000);
> > > -
> > > - reg |= 0x1f;
> > > - exynos_dsi_write(dsi, DSIM_FIFOCTRL_REG, reg);
> > > - usleep_range(9000, 11000);
> > > + struct exynos_dsi *dsi = dev_get_drvdata(dev);
> > > + struct drm_device *drm = dsi->encoder.dev;
> > > + struct exynos_drm_crtc *crtc;
> > >  
> > > - /* DSI configuration */
> > > - reg = 0;
> > > + mutex_lock(&drm->mode_config.mutex);
> > > + crtc = exynos_drm_crtc_get_by_type(drm, EXYNOS_DISPLAY_TYPE_LCD);
> > > + crtc->i80_mode = !(device->mode_flags & MIPI_DSI_MODE_VIDEO);
> > > + mutex_unlock(&drm->mode_config.mutex);
> > >  
> > > - /*
> > > -  * The first bit of mode_flags specifies display configuration.
> > > -  * If this bit is set[= MIPI_DSI_MODE_VIDEO], dsi will support video
> > > -  * mode, otherwise it will support command mode.
> > > -  */
> > > - if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO) {
> > > -         reg |= DSIM_VIDEO_MODE;
> > > -
> > > -         /*
> > > -          * The user manual describes that following bits are ignored in
> > > -          * command mode.
> > > -          */
> > > -         if (!(dsi->mode_flags & MIPI_DSI_MODE_VSYNC_FLUSH))
> > > -                 reg |= DSIM_MFLUSH_VS;
> > > -         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_SYNC_PULSE)
> > > -                 reg |= DSIM_SYNC_INFORM;
> > > -         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_BURST)
> > > -                 reg |= DSIM_BURST_MODE;
> > > -         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_AUTO_VERT)
> > > -                 reg |= DSIM_AUTO_MODE;
> > > -         if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HSE)
> > > -                 reg |= DSIM_HSE_MODE;
> > > -         if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HFP))
> > > -                 reg |= DSIM_HFP_MODE;
> > > -         if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HBP))
> > > -                 reg |= DSIM_HBP_MODE;
> > > -         if (!(dsi->mode_flags & MIPI_DSI_MODE_VIDEO_HSA))
> > > -                 reg |= DSIM_HSA_MODE;
> > > - }
> > > -
> > > - if (!(dsi->mode_flags & MIPI_DSI_MODE_EOT_PACKET))
> > > -         reg |= DSIM_EOT_DISABLE;
> > > -
> > > - switch (dsi->format) {
> > > - case MIPI_DSI_FMT_RGB888:
> > > -         reg |= DSIM_MAIN_PIX_FORMAT_RGB888;
> > > -         break;
> > > - case MIPI_DSI_FMT_RGB666:
> > > -         reg |= DSIM_MAIN_PIX_FORMAT_RGB666;
> > > -         break;
> > > - case MIPI_DSI_FMT_RGB666_PACKED:
> > > -         reg |= DSIM_MAIN_PIX_FORMAT_RGB666_P;
> > > -         break;
> > > - case MIPI_DSI_FMT_RGB565:
> > > -         reg |= DSIM_MAIN_PIX_FORMAT_RGB565;
> > > -         break;
> > > - default:
> > > -         dev_err(dsi->dev, "invalid pixel format\n");
> > > -         return -EINVAL;
> > > - }
> > > -
> > > - /*
> > > -  * Use non-continuous clock mode if the periparal wants and
> > > -  * host controller supports
> > > -  *
> > > -  * In non-continous clock mode, host controller will turn off
> > > -  * the HS clock between high-speed transmissions to reduce
> > > -  * power consumption.
> > > -  */
> > > - if (driver_data->has_clklane_stop &&
> > > -                 dsi->mode_flags & MIPI_DSI_CLOCK_NON_CONTINUOUS) {
> > > -         reg |= DSIM_CLKLANE_STOP;
> > > - }
> > > - exynos_dsi_write(dsi, DSIM_CONFIG_REG, reg);
> > > -
> > > - lanes_mask = BIT(dsi->lanes) - 1;
> > > - exynos_dsi_enable_lane(dsi, lanes_mask);
> > > -
> > > - /* Check clock and data lane state are stop state */
> > > - timeout = 100;
> > > - do {
> > > -         if (timeout-- == 0) {
> > > -                 dev_err(dsi->dev, "waiting for bus lanes timed out\n");
> > > -                 return -EFAULT;
> > > -         }
> > > -
> > > -         reg = exynos_dsi_read(dsi, DSIM_STATUS_REG);
> > > -         if ((reg & DSIM_STOP_STATE_DAT(lanes_mask))
> > > -             != DSIM_STOP_STATE_DAT(lanes_mask))
> > > -                 continue;
> > > - } while (!(reg & (DSIM_STOP_STATE_CLK | DSIM_TX_READY_HS_CLK)));
> > > -
> > > - reg = exynos_dsi_read(dsi, DSIM_ESCMODE_REG);
> > > - reg &= ~DSIM_STOP_STATE_CNT_MASK;
> > > - reg |= DSIM_STOP_STATE_CNT(driver_data->reg_values[STOP_STATE_CNT]);
> > > - exynos_dsi_write(dsi, DSIM_ESCMODE_REG, reg);
> > > -
> > > - reg = DSIM_BTA_TIMEOUT(0xff) | DSIM_LPDR_TIMEOUT(0xffff);
> > > - exynos_dsi_write(dsi, DSIM_TIMEOUT_REG, reg);
> > > + if (drm->mode_config.poll_enabled)
> > > +         drm_kms_helper_hotplug_event(drm);
> > >  
> > >   return 0;
> > >  }
> > >  
> > > -static void exynos_dsi_set_display_mode(struct exynos_dsi *dsi)
> > > -{
> > > - struct drm_display_mode *m = &dsi->mode;
> > > - unsigned int num_bits_resol = dsi->driver_data->num_bits_resol;
> > > - u32 reg;
> > > -
> > > - if (dsi->mode_flags & MIPI_DSI_MODE_VIDEO) {
> > > -         reg = DSIM_CMD_ALLOW(0xf)
> > > -                 | DSIM_STABLE_VFP(m->vsync_start - m->vdisplay)
> > > -                 | DSIM_MAIN_VBP(m->vtotal - m->vsync_end);
> > > -         exynos_dsi_write(dsi, DSIM_MVPORCH_REG, reg);
> > > -
> > > -         reg = DSIM_MAIN_HFP(m->hsync_start - m->hdisplay)
> > > -                 | DSIM_MAIN_HBP(m->htotal - m->hsync_end);
> > > -         exynos_dsi_write(dsi, DSIM_MHPORCH_REG, reg);
> > > -
> > > -         reg = DSIM_MAIN_VSA(m->vsync_end - m->vsync_start)
> > > -                 | DSIM_MAIN_HSA(m->hsync_end - m->hsync_start);
> > > -         exynos_dsi_write(dsi, DSIM_MSYNC_REG, reg);
> > > - }
> > > - reg =  DSIM_MAIN_HRESOL(m->hdisplay, num_bits_resol) |
> > > -         DSIM_MAIN_VRESOL(m->vdisplay, num_bits_resol);
> > > -
> > > - exynos_dsi_write(dsi, DSIM_MDRESOL_REG, reg);
> > > -
> > > - dev_dbg(dsi->dev, "LCD size = %dx%d\n", m->hdisplay, m->vdisplay);
> > > -}
> > > -
> > > -static void exynos_dsi_set_display_enable(struct exynos_dsi *dsi, bool 
> > > enable)
> > > -{
> > > - u32 reg;
> > > -
> > > - reg = exynos_dsi_read(dsi, DSIM_MDRESOL_REG);
> > > - if (enable)
> > > -         reg |= DSIM_MAIN_STAND_BY;
> > > - else
> > > -         reg &= ~DSIM_MAIN_STAND_BY;
> > > - exynos_dsi_write(dsi, DSIM_MDRESOL_REG, reg);
> > > -}
> > > -
> > > -static int exynos_dsi_wait_for_hdr_fifo(struct exynos_dsi *dsi)
> > > -{
> > > - int timeout = 2000;
> > > -
> > > - do {
> > > -         u32 reg = exynos_dsi_read(dsi, DSIM_FIFOCTRL_REG);
> > > -
> > > -         if (!(reg & DSIM_SFR_HEADER_FULL))
> > > -                 return 0;
> > > -
> > > -         if (!cond_resched())
> > > -                 usleep_range(950, 1050);
> > > - } while (--timeout);
> > > -
> > > - return -ETIMEDOUT;
> > > -}
> > > -
> > > -static void exynos_dsi_set_cmd_lpm(struct exynos_dsi *dsi, bool lpm)
> > > -{
> > > - u32 v = exynos_dsi_read(dsi, DSIM_ESCMODE_REG);
> > > -
> > > - if (lpm)
> > > -         v |= DSIM_CMD_LPDT_LP;
> > > - else
> > > -         v &= ~DSIM_CMD_LPDT_LP;
> > > -
> > > - exynos_dsi_write(dsi, DSIM_ESCMODE_REG, v);
> > > -}
> > > -
> > > -static void exynos_dsi_force_bta(struct exynos_dsi *dsi)
> > > -{
> > > - u32 v = exynos_dsi_read(dsi, DSIM_ESCMODE_REG);
> > > - v |= DSIM_FORCE_BTA;
> > > - exynos_dsi_write(dsi, DSIM_ESCMODE_REG, v);
> > > -}
> > > -
> > > -static void exynos_dsi_send_to_fifo(struct exynos_dsi *dsi,
> > > -                                 struct exynos_dsi_transfer *xfer)
> > > -{
> > > - struct device *dev = dsi->dev;
> > > - struct mipi_dsi_packet *pkt = &xfer->packet;
> > > - const u8 *payload = pkt->payload + xfer->tx_done;
> > > - u16 length = pkt->payload_length - xfer->tx_done;
> > > - bool first = !xfer->tx_done;
> > > - u32 reg;
> > > -
> > > - dev_dbg(dev, "< xfer %pK: tx len %u, done %u, rx len %u, done %u\n",
> > > -         xfer, length, xfer->tx_done, xfer->rx_len, xfer->rx_done);
> > > -
> > > - if (length > DSI_TX_FIFO_SIZE)
> > > -         length = DSI_TX_FIFO_SIZE;
> > > -
> > > - xfer->tx_done += length;
> > > -
> > > - /* Send payload */
> > > - while (length >= 4) {
> > > -         reg = get_unaligned_le32(payload);
> > > -         exynos_dsi_write(dsi, DSIM_PAYLOAD_REG, reg);
> > > -         payload += 4;
> > > -         length -= 4;
> > > - }
> > > -
> > > - reg = 0;
> > > - switch (length) {
> > > - case 3:
> > > -         reg |= payload[2] << 16;
> > > -         fallthrough;
> > > - case 2:
> > > -         reg |= payload[1] << 8;
> > > -         fallthrough;
> > > - case 1:
> > > -         reg |= payload[0];
> > > -         exynos_dsi_write(dsi, DSIM_PAYLOAD_REG, reg);
> > > -         break;
> > > - }
> > > -
> > > - /* Send packet header */
> > > - if (!first)
> > > -         return;
> > > -
> > > - reg = get_unaligned_le32(pkt->header);
> > > - if (exynos_dsi_wait_for_hdr_fifo(dsi)) {
> > > -         dev_err(dev, "waiting for header FIFO timed out\n");
> > > -         return;
> > > - }
> > > -
> > > - if (NEQV(xfer->flags & MIPI_DSI_MSG_USE_LPM,
> > > -          dsi->state & DSIM_STATE_CMD_LPM)) {
> > > -         exynos_dsi_set_cmd_lpm(dsi, xfer->flags & MIPI_DSI_MSG_USE_LPM);
> > > -         dsi->state ^= DSIM_STATE_CMD_LPM;
> > > - }
> > > -
> > > - exynos_dsi_write(dsi, DSIM_PKTHDR_REG, reg);
> > > -
> > > - if (xfer->flags & MIPI_DSI_MSG_REQ_ACK)
> > > -         exynos_dsi_force_bta(dsi);
> > > -}
> > > -
> > > -static void exynos_dsi_read_from_fifo(struct exynos_dsi *dsi,
> > > -                                 struct exynos_dsi_transfer *xfer)
> > > -{
> > > - u8 *payload = xfer->rx_payload + xfer->rx_done;
> > > - bool first = !xfer->rx_done;
> > > - struct device *dev = dsi->dev;
> > > - u16 length;
> > > - u32 reg;
> > > -
> > > - if (first) {
> > > -         reg = exynos_dsi_read(dsi, DSIM_RXFIFO_REG);
> > > -
> > > -         switch (reg & 0x3f) {
> > > -         case MIPI_DSI_RX_GENERIC_SHORT_READ_RESPONSE_2BYTE:
> > > -         case MIPI_DSI_RX_DCS_SHORT_READ_RESPONSE_2BYTE:
> > > -                 if (xfer->rx_len >= 2) {
> > > -                         payload[1] = reg >> 16;
> > > -                         ++xfer->rx_done;
> > > -                 }
> > > -                 fallthrough;
> > > -         case MIPI_DSI_RX_GENERIC_SHORT_READ_RESPONSE_1BYTE:
> > > -         case MIPI_DSI_RX_DCS_SHORT_READ_RESPONSE_1BYTE:
> > > -                 payload[0] = reg >> 8;
> > > -                 ++xfer->rx_done;
> > > -                 xfer->rx_len = xfer->rx_done;
> > > -                 xfer->result = 0;
> > > -                 goto clear_fifo;
> > > -         case MIPI_DSI_RX_ACKNOWLEDGE_AND_ERROR_REPORT:
> > > -                 dev_err(dev, "DSI Error Report: 0x%04x\n",
> > > -                         (reg >> 8) & 0xffff);
> > > -                 xfer->result = 0;
> > > -                 goto clear_fifo;
> > > -         }
> > > -
> > > -         length = (reg >> 8) & 0xffff;
> > > -         if (length > xfer->rx_len) {
> > > -                 dev_err(dev,
> > > -                         "response too long (%u > %u bytes), 
> > > stripping\n",
> > > -                         xfer->rx_len, length);
> > > -                 length = xfer->rx_len;
> > > -         } else if (length < xfer->rx_len)
> > > -                 xfer->rx_len = length;
> > > - }
> > > -
> > > - length = xfer->rx_len - xfer->rx_done;
> > > - xfer->rx_done += length;
> > > -
> > > - /* Receive payload */
> > > - while (length >= 4) {
> > > -         reg = exynos_dsi_read(dsi, DSIM_RXFIFO_REG);
> > > -         payload[0] = (reg >>  0) & 0xff;
> > > -         payload[1] = (reg >>  8) & 0xff;
> > > -         payload[2] = (reg >> 16) & 0xff;
> > > -         payload[3] = (reg >> 24) & 0xff;
> > > -         payload += 4;
> > > -         length -= 4;
> > > - }
> > > -
> > > - if (length) {
> > > -         reg = exynos_dsi_read(dsi, DSIM_RXFIFO_REG);
> > > -         switch (length) {
> > > -         case 3:
> > > -                 payload[2] = (reg >> 16) & 0xff;
> > > -                 fallthrough;
> > > -         case 2:
> > > -                 payload[1] = (reg >> 8) & 0xff;
> > > -                 fallthrough;
> > > -         case 1:
> > > -                 payload[0] = reg & 0xff;
> > > -         }
> > > - }
> > > -
> > > - if (xfer->rx_done == xfer->rx_len)
> > > -         xfer->result = 0;
> > > -
> > > -clear_fifo:
> > > - length = DSI_RX_FIFO_SIZE / 4;
> > > - do {
> > > -         reg = exynos_dsi_read(dsi, DSIM_RXFIFO_REG);
> > > -         if (reg == DSI_RX_FIFO_EMPTY)
> > > -                 break;
> > > - } while (--length);
> > > -}
> > > -
> > > -static void exynos_dsi_transfer_start(struct exynos_dsi *dsi)
> > > -{
> > > - unsigned long flags;
> > > - struct exynos_dsi_transfer *xfer;
> > > - bool start = false;
> > > -
> > > -again:
> > > - spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > -
> > > - if (list_empty(&dsi->transfer_list)) {
> > > -         spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -         return;
> > > - }
> > > -
> > > - xfer = list_first_entry(&dsi->transfer_list,
> > > -                                 struct exynos_dsi_transfer, list);
> > > -
> > > - spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -
> > > - if (xfer->packet.payload_length &&
> > > -     xfer->tx_done == xfer->packet.payload_length)
> > > -         /* waiting for RX */
> > > -         return;
> > > -
> > > - exynos_dsi_send_to_fifo(dsi, xfer);
> > > -
> > > - if (xfer->packet.payload_length || xfer->rx_len)
> > > -         return;
> > > -
> > > - xfer->result = 0;
> > > - complete(&xfer->completed);
> > > -
> > > - spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > -
> > > - list_del_init(&xfer->list);
> > > - start = !list_empty(&dsi->transfer_list);
> > > -
> > > - spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -
> > > - if (start)
> > > -         goto again;
> > > -}
> > > -
> > > -static bool exynos_dsi_transfer_finish(struct exynos_dsi *dsi)
> > > -{
> > > - struct exynos_dsi_transfer *xfer;
> > > - unsigned long flags;
> > > - bool start = true;
> > > -
> > > - spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > -
> > > - if (list_empty(&dsi->transfer_list)) {
> > > -         spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -         return false;
> > > - }
> > > -
> > > - xfer = list_first_entry(&dsi->transfer_list,
> > > -                                 struct exynos_dsi_transfer, list);
> > > -
> > > - spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -
> > > - dev_dbg(dsi->dev,
> > > -         "> xfer %pK, tx_len %zu, tx_done %u, rx_len %u, rx_done %u\n",
> > > -         xfer, xfer->packet.payload_length, xfer->tx_done, xfer->rx_len,
> > > -         xfer->rx_done);
> > > -
> > > - if (xfer->tx_done != xfer->packet.payload_length)
> > > -         return true;
> > > -
> > > - if (xfer->rx_done != xfer->rx_len)
> > > -         exynos_dsi_read_from_fifo(dsi, xfer);
> > > -
> > > - if (xfer->rx_done != xfer->rx_len)
> > > -         return true;
> > > -
> > > - spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > -
> > > - list_del_init(&xfer->list);
> > > - start = !list_empty(&dsi->transfer_list);
> > > -
> > > - spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -
> > > - if (!xfer->rx_len)
> > > -         xfer->result = 0;
> > > - complete(&xfer->completed);
> > > -
> > > - return start;
> > > -}
> > > -
> > > -static void exynos_dsi_remove_transfer(struct exynos_dsi *dsi,
> > > -                                 struct exynos_dsi_transfer *xfer)
> > > -{
> > > - unsigned long flags;
> > > - bool start;
> > > -
> > > - spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > -
> > > - if (!list_empty(&dsi->transfer_list) &&
> > > -     xfer == list_first_entry(&dsi->transfer_list,
> > > -                              struct exynos_dsi_transfer, list)) {
> > > -         list_del_init(&xfer->list);
> > > -         start = !list_empty(&dsi->transfer_list);
> > > -         spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -         if (start)
> > > -                 exynos_dsi_transfer_start(dsi);
> > > -         return;
> > > - }
> > > -
> > > - list_del_init(&xfer->list);
> > > -
> > > - spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -}
> > > -
> > > -static int exynos_dsi_transfer(struct exynos_dsi *dsi,
> > > -                                 struct exynos_dsi_transfer *xfer)
> > > -{
> > > - unsigned long flags;
> > > - bool stopped;
> > > -
> > > - xfer->tx_done = 0;
> > > - xfer->rx_done = 0;
> > > - xfer->result = -ETIMEDOUT;
> > > - init_completion(&xfer->completed);
> > > -
> > > - spin_lock_irqsave(&dsi->transfer_lock, flags);
> > > -
> > > - stopped = list_empty(&dsi->transfer_list);
> > > - list_add_tail(&xfer->list, &dsi->transfer_list);
> > > -
> > > - spin_unlock_irqrestore(&dsi->transfer_lock, flags);
> > > -
> > > - if (stopped)
> > > -         exynos_dsi_transfer_start(dsi);
> > > -
> > > - wait_for_completion_timeout(&xfer->completed,
> > > -                             msecs_to_jiffies(DSI_XFER_TIMEOUT_MS));
> > > - if (xfer->result == -ETIMEDOUT) {
> > > -         struct mipi_dsi_packet *pkt = &xfer->packet;
> > > -         exynos_dsi_remove_transfer(dsi, xfer);
> > > -         dev_err(dsi->dev, "xfer timed out: %*ph %*ph\n", 4, pkt->header,
> > > -                 (int)pkt->payload_length, pkt->payload);
> > > -         return -ETIMEDOUT;
> > > - }
> > > -
> > > - /* Also covers hardware timeout condition */
> > > - return xfer->result;
> > > -}
> > > -
> > > -static irqreturn_t exynos_dsi_irq(int irq, void *dev_id)
> > > -{
> > > - struct exynos_dsi *dsi = dev_id;
> > > - u32 status;
> > > -
> > > - status = exynos_dsi_read(dsi, DSIM_INTSRC_REG);
> > > - if (!status) {
> > > -         static unsigned long int j;
> > > -         if (printk_timed_ratelimit(&j, 500))
> > > -                 dev_warn(dsi->dev, "spurious interrupt\n");
> > > -         return IRQ_HANDLED;
> > > - }
> > > - exynos_dsi_write(dsi, DSIM_INTSRC_REG, status);
> > > -
> > > - if (status & DSIM_INT_SW_RST_RELEASE) {
> > > -         u32 mask = ~(DSIM_INT_RX_DONE | DSIM_INT_SFR_FIFO_EMPTY |
> > > -                 DSIM_INT_SFR_HDR_FIFO_EMPTY | DSIM_INT_RX_ECC_ERR |
> > > -                 DSIM_INT_SW_RST_RELEASE);
> > > -         exynos_dsi_write(dsi, DSIM_INTMSK_REG, mask);
> > > -         complete(&dsi->completed);
> > > -         return IRQ_HANDLED;
> > > - }
> > > -
> > > - if (!(status & (DSIM_INT_RX_DONE | DSIM_INT_SFR_FIFO_EMPTY |
> > > -                 DSIM_INT_PLL_STABLE)))
> > > -         return IRQ_HANDLED;
> > > -
> > > - if (exynos_dsi_transfer_finish(dsi))
> > > -         exynos_dsi_transfer_start(dsi);
> > > -
> > > - return IRQ_HANDLED;
> > > -}
> > > -
> > > -static irqreturn_t exynos_dsi_te_irq_handler(int irq, void *dev_id)
> > > -{
> > > - struct exynos_dsi *dsi = dev_id;
> > > - const struct exynos_dsi_host_ops *ops = dsi->driver_data->host_ops;
> > > -
> > > - if (ops && ops->te_handler &&
> > > -     (dsi->state & DSIM_STATE_VIDOUT_AVAILABLE))
> > > -         ops->te_handler(dsi->dsi_host.dev);
> > > -
> > > - return IRQ_HANDLED;
> > > -}
> > > -
> > > -static void exynos_dsi_enable_irq(struct exynos_dsi *dsi)
> > > -{
> > > - enable_irq(dsi->irq);
> > > -
> > > - if (gpio_is_valid(dsi->te_gpio))
> > > -         enable_irq(gpio_to_irq(dsi->te_gpio));
> > > -}
> > > -
> > > -static void exynos_dsi_disable_irq(struct exynos_dsi *dsi)
> > > -{
> > > - if (gpio_is_valid(dsi->te_gpio))
> > > -         disable_irq(gpio_to_irq(dsi->te_gpio));
> > > -
> > > - disable_irq(dsi->irq);
> > > -}
> > > -
> > > -static int exynos_dsi_init(struct exynos_dsi *dsi)
> > > +static int exynos_dsi_host_detach(struct device *dev,
> > > +                           struct mipi_dsi_device *device)
> > >  {
> > > - const struct exynos_dsi_driver_data *driver_data = dsi->driver_data;
> > > -
> > > - exynos_dsi_reset(dsi);
> > > - exynos_dsi_enable_irq(dsi);
> > > + struct exynos_dsi *dsi = dev_get_drvdata(dev);
> > > + struct drm_device *drm = dsi->encoder.dev;
> > >  
> > > - if (driver_data->reg_values[RESET_TYPE] == DSIM_FUNCRST)
> > > -         exynos_dsi_enable_lane(dsi, BIT(dsi->lanes) - 1);
> > > -
> > > - exynos_dsi_enable_clock(dsi);
> > > - if (driver_data->wait_for_reset)
> > > -         exynos_dsi_wait_for_reset(dsi);
> > > - exynos_dsi_set_phy_ctrl(dsi);
> > > - exynos_dsi_init_link(dsi);
> > > + if (drm->mode_config.poll_enabled)
> > > +         drm_kms_helper_hotplug_event(drm);
> > >  
> > >   return 0;
> > >  }
> > >  
> > > -static int exynos_dsi_register_te_irq(struct exynos_dsi *dsi,
> > > -                               struct device *panel)
> > > +static void exynos_dsi_te_handler(struct device *dev)
> > >  {
> > > - int ret;
> > > - int te_gpio_irq;
> > > + struct exynos_dsi *dsi = dev_get_drvdata(dev);
> > >  
> > > - dsi->te_gpio = of_get_named_gpio(panel->of_node, "te-gpios", 0);
> > > - if (dsi->te_gpio == -ENOENT)
> > > -         return 0;
> > > -
> > > - if (!gpio_is_valid(dsi->te_gpio)) {
> > > -         ret = dsi->te_gpio;
> > > -         dev_err(dsi->dev, "cannot get te-gpios, %d\n", ret);
> > > -         goto out;
> > > - }
> > > -
> > > - ret = gpio_request(dsi->te_gpio, "te_gpio");
> > > - if (ret) {
> > > -         dev_err(dsi->dev, "gpio request failed with %d\n", ret);
> > > -         goto out;
> > > - }
> > > -
> > > - te_gpio_irq = gpio_to_irq(dsi->te_gpio);
> > > - irq_set_status_flags(te_gpio_irq, IRQ_NOAUTOEN);
> > > -
> > > - ret = request_threaded_irq(te_gpio_irq, exynos_dsi_te_irq_handler, NULL,
> > > -                                 IRQF_TRIGGER_RISING, "TE", dsi);
> > > - if (ret) {
> > > -         dev_err(dsi->dev, "request interrupt failed with %d\n", ret);
> > > -         gpio_free(dsi->te_gpio);
> > > -         goto out;
> > > - }
> > > -
> > > -out:
> > > - return ret;
> > > + exynos_drm_crtc_te_handler(dsi->encoder.crtc);
> > >  }
> > >  
> > > -static void exynos_dsi_unregister_te_irq(struct exynos_dsi *dsi)
> > > -{
> > > - if (gpio_is_valid(dsi->te_gpio)) {
> > > -         free_irq(gpio_to_irq(dsi->te_gpio), dsi);
> > > -         gpio_free(dsi->te_gpio);
> > > -         dsi->te_gpio = -ENOENT;
> > > - }
> > > -}
> > > -
> > > -static void exynos_dsi_enable(struct exynos_dsi *dsi)
> > > -{
> > > - struct drm_bridge *iter;
> > > - int ret;
> > > -
> > > - if (dsi->state & DSIM_STATE_ENABLED)
> > > -         return;
> > > -
> > > - pm_runtime_get_sync(dsi->dev);
> > > - dsi->state |= DSIM_STATE_ENABLED;
> > > -
> > > - if (dsi->panel) {
> > > -         ret = drm_panel_prepare(dsi->panel);
> > > -         if (ret < 0)
> > > -                 goto err_put_sync;
> > > - } else {
> > > -         list_for_each_entry_reverse(iter, &dsi->bridge_chain,
> > > -                                     chain_node) {
> > > -                 if (iter->funcs->pre_enable)
> > > -                         iter->funcs->pre_enable(iter);
> > > -         }
> > > - }
> > > -
> > > - exynos_dsi_set_display_mode(dsi);
> > > - exynos_dsi_set_display_enable(dsi, true);
> > > -
> > > - if (dsi->panel) {
> > > -         ret = drm_panel_enable(dsi->panel);
> > > -         if (ret < 0)
> > > -                 goto err_display_disable;
> > > - } else {
> > > -         list_for_each_entry(iter, &dsi->bridge_chain, chain_node) {
> > > -                 if (iter->funcs->enable)
> > > -                         iter->funcs->enable(iter);
> > > -         }
> > > - }
> > > -
> > > - dsi->state |= DSIM_STATE_VIDOUT_AVAILABLE;
> > > - return;
> > > -
> > > -err_display_disable:
> > > - exynos_dsi_set_display_enable(dsi, false);
> > > - drm_panel_unprepare(dsi->panel);
> > > -
> > > -err_put_sync:
> > > - dsi->state &= ~DSIM_STATE_ENABLED;
> > > - pm_runtime_put(dsi->dev);
> > > -}
> > > -
> > > -static void exynos_dsi_disable(struct exynos_dsi *dsi)
> > > -{
> > > - struct drm_bridge *iter;
> > > -
> > > - if (!(dsi->state & DSIM_STATE_ENABLED))
> > > -         return;
> > > -
> > > - dsi->state &= ~DSIM_STATE_VIDOUT_AVAILABLE;
> > > -
> > > - drm_panel_disable(dsi->panel);
> > > -
> > > - list_for_each_entry_reverse(iter, &dsi->bridge_chain, chain_node) {
> > > -         if (iter->funcs->disable)
> > > -                 iter->funcs->disable(iter);
> > > - }
> > > -
> > > - exynos_dsi_set_display_enable(dsi, false);
> > > - drm_panel_unprepare(dsi->panel);
> > > -
> > > - list_for_each_entry(iter, &dsi->bridge_chain, chain_node) {
> > > -         if (iter->funcs->post_disable)
> > > -                 iter->funcs->post_disable(iter);
> > > - }
> > > -
> > > - dsi->state &= ~DSIM_STATE_ENABLED;
> > > - pm_runtime_put_sync(dsi->dev);
> > > -}
> > > -
> > > -static enum drm_connector_status
> > > -exynos_dsi_detect(struct drm_connector *connector, bool force)
> > > -{
> > > - return connector->status;
> > > -}
> > > +static const struct samsung_dsim_host_ops exynos_dsi_host_ops = {
> > > + .attach = exynos_dsi_host_attach,
> > > + .detach = exynos_dsi_host_detach,
> > > + .te_handler = exynos_dsi_te_handler,
> > > +};
> > >  
> > > -static void exynos_dsi_connector_destroy(struct drm_connector *connector)
> > > -{
> > > - drm_connector_unregister(connector);
> > > - drm_connector_cleanup(connector);
> > > - connector->dev = NULL;
> > > -}
> > > +static const struct samsung_dsim_driver_data exynos3_dsi_driver_data = {
> > > + .reg_ofs = EXYNOS_REG_OFS,
> > > + .plltmr_reg = 0x50,
> > > + .has_freqband = 1,
> > > + .has_clklane_stop = 1,
> > > + .num_clks = 2,
> > > + .max_freq = 1000,
> > > + .wait_for_reset = 1,
> > > + .num_bits_resol = 11,
> > > + .reg_values = reg_values,
> > > + .host_ops = &exynos_dsi_host_ops,
> > > +};
> > >  
> > > -static const struct drm_connector_funcs exynos_dsi_connector_funcs = {
> > > - .detect = exynos_dsi_detect,
> > > - .fill_modes = drm_helper_probe_single_connector_modes,
> > > - .destroy = exynos_dsi_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 const struct samsung_dsim_driver_data exynos4_dsi_driver_data = {
> > > + .reg_ofs = EXYNOS_REG_OFS,
> > > + .plltmr_reg = 0x50,
> > > + .has_freqband = 1,
> > > + .has_clklane_stop = 1,
> > > + .num_clks = 2,
> > > + .max_freq = 1000,
> > > + .wait_for_reset = 1,
> > > + .num_bits_resol = 11,
> > > + .reg_values = reg_values,
> > > + .host_ops = &exynos_dsi_host_ops,
> > >  };
> > >  
> > > -static int exynos_dsi_get_modes(struct drm_connector *connector)
> > > -{
> > > - struct exynos_dsi *dsi = connector_to_dsi(connector);
> > > +static const struct samsung_dsim_driver_data exynos5_dsi_driver_data = {
> > > + .reg_ofs = EXYNOS_REG_OFS,
> > > + .plltmr_reg = 0x58,
> > > + .num_clks = 2,
> > > + .max_freq = 1000,
> > > + .wait_for_reset = 1,
> > > + .num_bits_resol = 11,
> > > + .reg_values = reg_values,
> > > + .host_ops = &exynos_dsi_host_ops,
> > > +};
> > >  
> > > - if (dsi->panel)
> > > -         return drm_panel_get_modes(dsi->panel, connector);
> > > +static const struct samsung_dsim_driver_data exynos5433_dsi_driver_data 
> > > = {
> > > + .reg_ofs = EXYNOS5433_REG_OFS,
> > > + .plltmr_reg = 0xa0,
> > > + .has_clklane_stop = 1,
> > > + .num_clks = 5,
> > > + .max_freq = 1500,
> > > + .wait_for_reset = 0,
> > > + .num_bits_resol = 12,
> > > + .reg_values = exynos5433_reg_values,
> > > + .host_ops = &exynos_dsi_host_ops,
> > > +};
> > >  
> > > - return 0;
> > > -}
> > > +static const struct samsung_dsim_driver_data exynos5422_dsi_driver_data 
> > > = {
> > > + .reg_ofs = EXYNOS5433_REG_OFS,
> > > + .plltmr_reg = 0xa0,
> > > + .has_clklane_stop = 1,
> > > + .num_clks = 2,
> > > + .max_freq = 1500,
> > > + .wait_for_reset = 1,
> > > + .num_bits_resol = 12,
> > > + .reg_values = exynos5422_reg_values,
> > > + .host_ops = &exynos_dsi_host_ops,
> > > +};
> > >  
> > > -static const struct drm_connector_helper_funcs 
> > > exynos_dsi_connector_helper_funcs = {
> > > - .get_modes = exynos_dsi_get_modes,
> > > +static const struct of_device_id exynos_dsi_of_match[] = {
> > > + { .compatible = "samsung,exynos3250-mipi-dsi",
> > > +   .data = &exynos3_dsi_driver_data },
> > > + { .compatible = "samsung,exynos4210-mipi-dsi",
> > > +   .data = &exynos4_dsi_driver_data },
> > > + { .compatible = "samsung,exynos5410-mipi-dsi",
> > > +   .data = &exynos5_dsi_driver_data },
> > > + { .compatible = "samsung,exynos5422-mipi-dsi",
> > > +   .data = &exynos5422_dsi_driver_data },
> > > + { .compatible = "samsung,exynos5433-mipi-dsi",
> > > +   .data = &exynos5433_dsi_driver_data },
> > > + { }
> > >  };
> > >  
> > > -static int exynos_dsi_create_connector(struct exynos_dsi *dsi)
> > > +static int exynos_dsi_bind(struct device *dev,
> > > +                    struct device *master, void *data)
> > >  {
> > > - struct drm_connector *connector = &dsi->connector;
> > > - struct drm_device *drm = dsi->bridge.dev;
> > > + struct exynos_dsi *dsi = dev_get_drvdata(dev);
> > > + struct drm_encoder *encoder = &dsi->encoder;
> > > + struct drm_device *drm_dev = data;
> > > + struct device_node *in_bridge_node;
> > > + struct drm_bridge *in_bridge;
> > >   int ret;
> > >  
> > > - connector->polled = DRM_CONNECTOR_POLL_HPD;
> > > + drm_simple_encoder_init(drm_dev, encoder, DRM_MODE_ENCODER_TMDS);
> > >  
> > > - ret = drm_connector_init(drm, connector, &exynos_dsi_connector_funcs,
> > > -                          DRM_MODE_CONNECTOR_DSI);
> > > - if (ret) {
> > > -         DRM_DEV_ERROR(dsi->dev,
> > > -                       "Failed to initialize connector with drm\n");
> > > + ret = exynos_drm_set_possible_crtcs(encoder, EXYNOS_DISPLAY_TYPE_LCD);
> > > + if (ret < 0)
> > >           return ret;
> > > - }
> > > -
> > > - connector->status = connector_status_disconnected;
> > > - drm_connector_helper_add(connector, &exynos_dsi_connector_helper_funcs);
> > > - drm_connector_attach_encoder(connector, dsi->bridge.encoder);
> > > - if (!drm->registered)
> > > -         return 0;
> > > -
> > > - connector->funcs->reset(connector);
> > > - drm_connector_register(connector);
> > > - return 0;
> > > -}
> > > -
> > > -static int exynos_dsi_bridge_attach(struct drm_bridge *bridge,
> > > -                             enum drm_bridge_attach_flags flags)
> > > -{
> > > - struct exynos_dsi *dsi = bridge->driver_private;
> > > - struct drm_encoder *encoder = bridge->encoder;
> > > - int ret;
> > > -
> > > - if (!dsi->out_bridge && !dsi->panel)
> > > -         return -EPROBE_DEFER;
> > > -
> > > - if (dsi->out_bridge) {
> > > -         ret = drm_bridge_attach(encoder, dsi->out_bridge,
> > > -                                 bridge, flags);
> > > -         if (ret)
> > > -                 return ret;
> > > -         list_splice_init(&encoder->bridge_chain, &dsi->bridge_chain);
> > > - } else {
> > > -         ret = exynos_dsi_create_connector(dsi);
> > > -         if (ret)
> > > -                 return ret;
> > > -
> > > -         if (dsi->panel) {
> > > -                 dsi->connector.status = connector_status_connected;
> > > -         }
> > > - }
> > >  
> > > - return 0;
> > > -}
> > > -
> > > -static void exynos_dsi_bridge_detach(struct drm_bridge *bridge)
> > > -{
> > > - struct exynos_dsi *dsi = bridge->driver_private;
> > > - struct drm_encoder *encoder = bridge->encoder;
> > > - struct drm_device *drm = encoder->dev;
> > > -
> > > - if (dsi->panel) {
> > > -         mutex_lock(&drm->mode_config.mutex);
> > > -         exynos_dsi_disable(dsi);
> > > -         dsi->panel = NULL;
> > > -         dsi->connector.status = connector_status_disconnected;
> > > -         mutex_unlock(&drm->mode_config.mutex);
> > > - } else {
> > > -         if (dsi->out_bridge->funcs->detach)
> > > -                 dsi->out_bridge->funcs->detach(dsi->out_bridge);
> > > -         dsi->out_bridge = NULL;
> > > -         INIT_LIST_HEAD(&dsi->bridge_chain);
> > > + in_bridge_node = of_graph_get_remote_node(dev->of_node, DSI_PORT_IN, 0);
> > > + if (in_bridge_node) {
> > > +         in_bridge = of_drm_find_bridge(in_bridge_node);
> > > +         if (in_bridge)
> > > +                 drm_bridge_attach(encoder, in_bridge, NULL, 0);
> > > +         of_node_put(in_bridge_node);
> > >   }
> > > -}
> > > -
> > > -static void exynos_dsi_bridge_enable(struct drm_bridge *bridge)
> > > -{
> > > - struct exynos_dsi *dsi = bridge->driver_private;
> > > -
> > > - exynos_dsi_enable(dsi);
> > > -}
> > > -
> > > -static void exynos_dsi_bridge_disable(struct drm_bridge *bridge)
> > > -{
> > > - struct exynos_dsi *dsi = bridge->driver_private;
> > > -
> > > - exynos_dsi_disable(dsi);
> > > -}
> > >  
> > > -static void exynos_dsi_bridge_mode_set(struct drm_bridge *bridge,
> > > -                                const struct drm_display_mode *mode,
> > > -                                const struct drm_display_mode 
> > > *adjusted_mode)
> > > -{
> > > - struct exynos_dsi *dsi = bridge->driver_private;
> > > -
> > > - /* The mode is set when actually enabling the device. */
> > > - drm_mode_copy(&dsi->mode, adjusted_mode);
> > > -}
> > > -
> > > -static const struct drm_bridge_funcs exynos_dsi_bridge_funcs = {
> > > - .attach = exynos_dsi_bridge_attach,
> > > - .detach = exynos_dsi_bridge_detach,
> > > - .enable = exynos_dsi_bridge_enable,
> > > - .disable = exynos_dsi_bridge_disable,
> > > - .mode_set = exynos_dsi_bridge_mode_set,
> > > -};
> > > -
> > > -static int exynos_dsi_host_attach(struct mipi_dsi_host *host,
> > > -                           struct mipi_dsi_device *device)
> > > -{
> > > - struct exynos_dsi *dsi = host_to_dsi(host);
> > > - const struct exynos_dsi_host_ops *ops = dsi->driver_data->host_ops;
> > > - struct drm_bridge *out_bridge;
> > > -
> > > - out_bridge = of_drm_find_bridge(device->dev.of_node);
> > > - if (out_bridge) {
> > > -         dsi->out_bridge = out_bridge;
> > > - } else {
> > > -         dsi->panel = of_drm_find_panel(device->dev.of_node);
> > > -         if (IS_ERR(dsi->panel))
> > > -                 dsi->panel = NULL;
> > > -         else
> > > -                 dsi->connector.status = connector_status_connected;
> > > - }
> > > -
> > > - /*
> > > -  * This is a temporary solution and should be made by more generic way.
> > > -  *
> > > -  * If attached panel device is for command mode one, dsi should register
> > > -  * TE interrupt handler.
> > > -  */
> > > - if (!(device->mode_flags & MIPI_DSI_MODE_VIDEO)) {
> > > -         int ret = exynos_dsi_register_te_irq(dsi, &device->dev);
> > > -         if (ret)
> > > -                 return ret;
> > > - }
> > > -
> > > - dsi->lanes = device->lanes;
> > > - dsi->format = device->format;
> > > - dsi->mode_flags = device->mode_flags;
> > > -
> > > - if (ops && ops->attach)
> > > -         ops->attach(dsi->dsi_host.dev, device);
> > > + ret = samsung_dsim_bind(dsi->dsi, encoder);
> > > + if (ret)
> > > +         goto err;
> > >  
> > >   return 0;
> > > -}
> > > -
> > > -static int exynos_dsi_host_detach(struct mipi_dsi_host *host,
> > > -                           struct mipi_dsi_device *device)
> > > -{
> > > - struct exynos_dsi *dsi = host_to_dsi(host);
> > > - const struct exynos_dsi_host_ops *ops = dsi->driver_data->host_ops;
> > >  
> > > - if (ops && ops->detach)
> > > -         ops->detach(dsi->dsi_host.dev, device);
> > > -
> > > - exynos_dsi_unregister_te_irq(dsi);
> > > -
> > > - return 0;
> > > +err:
> > > + drm_encoder_cleanup(encoder);
> > > + return ret;
> > >  }
> > >  
> > > -static ssize_t exynos_dsi_host_transfer(struct mipi_dsi_host *host,
> > > -                                  const struct mipi_dsi_msg *msg)
> > > +static void exynos_dsi_unbind(struct device *dev,
> > > +                       struct device *master, void *data)
> > >  {
> > > - struct exynos_dsi *dsi = host_to_dsi(host);
> > > - struct exynos_dsi_transfer xfer;
> > > - int ret;
> > > + struct exynos_dsi *dsi = dev_get_drvdata(dev);
> > > + struct drm_encoder *encoder = &dsi->encoder;
> > >  
> > > - if (!(dsi->state & DSIM_STATE_ENABLED))
> > > -         return -EINVAL;
> > > -
> > > - if (!(dsi->state & DSIM_STATE_INITIALIZED)) {
> > > -         ret = exynos_dsi_init(dsi);
> > > -         if (ret)
> > > -                 return ret;
> > > -         dsi->state |= DSIM_STATE_INITIALIZED;
> > > - }
> > > -
> > > - ret = mipi_dsi_create_packet(&xfer.packet, msg);
> > > - if (ret < 0)
> > > -         return ret;
> > > + samsung_dsim_unbind(dsi->dsi);
> > >  
> > > - xfer.rx_len = msg->rx_len;
> > > - xfer.rx_payload = msg->rx_buf;
> > > - xfer.flags = msg->flags;
> > > -
> > > - ret = exynos_dsi_transfer(dsi, &xfer);
> > > - return (ret < 0) ? ret : xfer.rx_done;
> > > + drm_encoder_cleanup(encoder);
> > >  }
> > >  
> > > -static const struct mipi_dsi_host_ops exynos_dsi_ops = {
> > > - .attach = exynos_dsi_host_attach,
> > > - .detach = exynos_dsi_host_detach,
> > > - .transfer = exynos_dsi_host_transfer,
> > > +static const struct component_ops exynos_dsi_component_ops = {
> > > + .bind   = exynos_dsi_bind,
> > > + .unbind = exynos_dsi_unbind,
> > >  };
> > >  
> > > -static int exynos_dsi_of_read_u32(const struct device_node *np,
> > > -                           const char *propname, u32 *out_value)
> > > -{
> > > - int ret = of_property_read_u32(np, propname, out_value);
> > > -
> > > - if (ret < 0)
> > > -         pr_err("%pOF: failed to get '%s' property\n", np, propname);
> > > -
> > > - return ret;
> > > -}
> > > -
> > > -static int exynos_dsi_parse_dt(struct exynos_dsi *dsi)
> > > -{
> > > - struct device *dev = dsi->dev;
> > > - struct device_node *node = dev->of_node;
> > > - int ret;
> > > -
> > > - ret = exynos_dsi_of_read_u32(node, "samsung,pll-clock-frequency",
> > > -                              &dsi->pll_clk_rate);
> > > - if (ret < 0)
> > > -         return ret;
> > > -
> > > - ret = exynos_dsi_of_read_u32(node, "samsung,burst-clock-frequency",
> > > -                              &dsi->burst_clk_rate);
> > > - if (ret < 0)
> > > -         return ret;
> > > -
> > > - ret = exynos_dsi_of_read_u32(node, "samsung,esc-clock-frequency",
> > > -                              &dsi->esc_clk_rate);
> > > - if (ret < 0)
> > > -         return ret;
> > > -
> > > - return 0;
> > > -}
> > > -
> > > -static struct exynos_dsi *__exynos_dsi_probe(struct platform_device 
> > > *pdev)
> > > +static int exynos_dsi_probe(struct platform_device *pdev)
> > >  {
> > > - struct device *dev = &pdev->dev;
> > > - struct drm_bridge *bridge;
> > > - struct resource *res;
> > >   struct exynos_dsi *dsi;
> > > - int ret, i;
> > > + struct device *dev = &pdev->dev;
> > > + int ret;
> > >  
> > >   dsi = devm_kzalloc(dev, sizeof(*dsi), GFP_KERNEL);
> > >   if (!dsi)
> > > -         return ERR_PTR(-ENOMEM);
> > > +         return -ENOMEM;
> > > + platform_set_drvdata(pdev, dsi);
> > >  
> > > - /* To be checked as invalid one */
> > > - dsi->te_gpio = -ENOENT;
> > > + dsi->dsi = samsung_dsim_probe(pdev);
> > > + if (IS_ERR(dsi->dsi))
> > > +         return PTR_ERR(dsi->dsi);
> > >  
> > > - init_completion(&dsi->completed);
> > > - spin_lock_init(&dsi->transfer_lock);
> > > - INIT_LIST_HEAD(&dsi->transfer_list);
> > > - INIT_LIST_HEAD(&dsi->bridge_chain);
> > > + pm_runtime_enable(dev);
> > >  
> > > - dsi->dsi_host.ops = &exynos_dsi_ops;
> > > - dsi->dsi_host.dev = dev;
> > > -
> > > - dsi->dev = dev;
> > > - dsi->driver_data = of_device_get_match_data(dev);
> > > -
> > > - dsi->supplies[0].supply = "vddcore";
> > > - dsi->supplies[1].supply = "vddio";
> > > - ret = devm_regulator_bulk_get(dev, ARRAY_SIZE(dsi->supplies),
> > > -                               dsi->supplies);
> > > - if (ret) {
> > > -         if (ret != -EPROBE_DEFER)
> > > -                 dev_info(dev, "failed to get regulators: %d\n", ret);
> > > -         return ERR_PTR(ret);
> > > - }
> > > -
> > > - dsi->clks = devm_kcalloc(dev,
> > > -                 dsi->driver_data->num_clks, sizeof(*dsi->clks),
> > > -                 GFP_KERNEL);
> > > - if (!dsi->clks)
> > > -         return ERR_PTR(-ENOMEM);
> > > -
> > > - for (i = 0; i < dsi->driver_data->num_clks; i++) {
> > > -         dsi->clks[i] = devm_clk_get(dev, clk_names[i]);
> > > -         if (IS_ERR(dsi->clks[i])) {
> > > -                 if (strcmp(clk_names[i], "sclk_mipi") == 0) {
> > > -                         dsi->clks[i] = devm_clk_get(dev,
> > > -                                                 OLD_SCLK_MIPI_CLK_NAME);
> > > -                         if (!IS_ERR(dsi->clks[i]))
> > > -                                 continue;
> > > -                 }
> > > -
> > > -                 dev_info(dev, "failed to get the clock: %s\n",
> > > -                                 clk_names[i]);
> > > -                 return ERR_PTR(PTR_ERR(dsi->clks[i]));
> > > -         }
> > > - }
> > > -
> > > - res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> > > - dsi->reg_base = devm_ioremap_resource(dev, res);
> > > - if (IS_ERR(dsi->reg_base)) {
> > > -         dev_err(dev, "failed to remap io region\n");
> > > -         return dsi->reg_base;
> > > - }
> > > -
> > > - dsi->phy = devm_phy_get(dev, "dsim");
> > > - if (IS_ERR(dsi->phy)) {
> > > -         dev_info(dev, "failed to get dsim phy\n");
> > > -         return ERR_PTR(PTR_ERR(dsi->phy));
> > > - }
> > > -
> > > - dsi->irq = platform_get_irq(pdev, 0);
> > > - if (dsi->irq < 0)
> > > -         return ERR_PTR(dsi->irq);
> > > -
> > > - irq_set_status_flags(dsi->irq, IRQ_NOAUTOEN);
> > > - ret = devm_request_threaded_irq(dev, dsi->irq, NULL,
> > > -                                 exynos_dsi_irq, IRQF_ONESHOT,
> > > -                                 dev_name(dev), dsi);
> > > - if (ret) {
> > > -         dev_err(dev, "failed to request dsi irq\n");
> > > -         return ERR_PTR(ret);
> > > - }
> > > -
> > > - ret = exynos_dsi_parse_dt(dsi);
> > > + ret = component_add(dev, &exynos_dsi_component_ops);
> > >   if (ret)
> > > -         return ERR_PTR(ret);
> > > +         goto err_disable_runtime;
> > >  
> > > - ret = mipi_dsi_host_register(&dsi->dsi_host);
> > > - if (ret)
> > > -         return ERR_PTR(ret);
> > > + return 0;
> > >  
> > > - bridge = &dsi->bridge;
> > > - bridge->driver_private = dsi;
> > > - bridge->funcs = &exynos_dsi_bridge_funcs;
> > > - bridge->of_node = dev->of_node;
> > > - drm_bridge_add(bridge);
> > > +err_disable_runtime:
> > > + pm_runtime_disable(dev);
> > >  
> > > - return dsi;
> > > + return ret;
> > >  }
> > >  
> > > -static void __exynos_dsi_remove(struct exynos_dsi *dsi)
> > > +static int exynos_dsi_remove(struct platform_device *pdev)
> > >  {
> > > - drm_bridge_remove(&dsi->bridge);
> > > + struct exynos_dsi *dsi = platform_get_drvdata(pdev);
> > >  
> > > - mipi_dsi_host_unregister(&dsi->dsi_host);
> > > -}
> > > -
> > > -/*
> > > - * Probe/remove API, used from platforms based on the DRM bridge API.
> > > - */
> > > -struct exynos_dsi *exynos_dsi_probe(struct platform_device *pdev)
> > > -{
> > > - return __exynos_dsi_probe(pdev);
> > > -}
> > > + pm_runtime_disable(&pdev->dev);
> > >  
> > > -void exynos_dsi_remove(struct exynos_dsi *dsi)
> > > -{
> > > - return __exynos_dsi_remove(dsi);
> > > -}
> > > + samsung_dsim_remove(dsi->dsi);
> > >  
> > > -/*
> > > - * Bind/unbind API, used from platforms based on the component framework.
> > > - */
> > > -int exynos_dsi_bind(struct exynos_dsi *dsi, struct drm_encoder *encoder)
> > > -{
> > > - struct drm_bridge *previous = 
> > > drm_bridge_chain_get_first_bridge(encoder);
> > > + component_del(&pdev->dev, &exynos_dsi_component_ops);
> > >  
> > > - return drm_bridge_attach(encoder, &dsi->bridge, previous, 0);
> > > -}
> > > -
> > > -void exynos_dsi_unbind(struct exynos_dsi *dsi)
> > > -{
> > > - exynos_dsi_disable(dsi);
> > > + return 0;
> > >  }
> > >  
> > > -int exynos_dsi_suspend(struct exynos_dsi *dsi)
> > > +static int __maybe_unused exynos_dsi_suspend(struct device *dev)
> > >  {
> > > - const struct exynos_dsi_driver_data *driver_data = dsi->driver_data;
> > > - int ret, i;
> > > -
> > > - usleep_range(10000, 20000);
> > > -
> > > - if (dsi->state & DSIM_STATE_INITIALIZED) {
> > > -         dsi->state &= ~DSIM_STATE_INITIALIZED;
> > > -
> > > -         exynos_dsi_disable_clock(dsi);
> > > -
> > > -         exynos_dsi_disable_irq(dsi);
> > > - }
> > > -
> > > - dsi->state &= ~DSIM_STATE_CMD_LPM;
> > > -
> > > - phy_power_off(dsi->phy);
> > > -
> > > - for (i = driver_data->num_clks - 1; i > -1; i--)
> > > -         clk_disable_unprepare(dsi->clks[i]);
> > > -
> > > - ret = regulator_bulk_disable(ARRAY_SIZE(dsi->supplies), dsi->supplies);
> > > - if (ret < 0)
> > > -         dev_err(dsi->dev, "cannot disable regulators %d\n", ret);
> > > + struct exynos_dsi *dsi = dev_get_drvdata(dev);
> > >  
> > > - return 0;
> > > + return samsung_dsim_suspend(dsi->dsi);
> > >  }
> > >  
> > > -int exynos_dsi_resume(struct exynos_dsi *dsi)
> > > +static int __maybe_unused exynos_dsi_resume(struct device *dev)
> > >  {
> > > - const struct exynos_dsi_driver_data *driver_data = dsi->driver_data;
> > > - int ret, i;
> > > -
> > > - ret = regulator_bulk_enable(ARRAY_SIZE(dsi->supplies), dsi->supplies);
> > > - if (ret < 0) {
> > > -         dev_err(dsi->dev, "cannot enable regulators %d\n", ret);
> > > -         return ret;
> > > - }
> > > -
> > > - for (i = 0; i < driver_data->num_clks; i++) {
> > > -         ret = clk_prepare_enable(dsi->clks[i]);
> > > -         if (ret < 0)
> > > -                 goto err_clk;
> > > - }
> > > -
> > > - ret = phy_power_on(dsi->phy);
> > > - if (ret < 0) {
> > > -         dev_err(dsi->dev, "cannot enable phy %d\n", ret);
> > > -         goto err_clk;
> > > - }
> > > + struct exynos_dsi *dsi = dev_get_drvdata(dev);
> > >  
> > > - return 0;
> > > + return samsung_dsim_resume(dsi->dsi);
> > > +}
> > >  
> > > -err_clk:
> > > - while (--i > -1)
> > > -         clk_disable_unprepare(dsi->clks[i]);
> > > - regulator_bulk_disable(ARRAY_SIZE(dsi->supplies), dsi->supplies);
> > > +static const struct dev_pm_ops exynos_dsi_pm_ops = {
> > > + SET_RUNTIME_PM_OPS(exynos_dsi_suspend, exynos_dsi_resume, NULL)
> > > + SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
> > > +                         pm_runtime_force_resume)
> > > +};
> > >  
> > > - return ret;
> > > -}
> > > +struct platform_driver dsi_driver = {
> > > + .probe = exynos_dsi_probe,
> > > + .remove = exynos_dsi_remove,
> > > + .driver = {
> > > +            .name = "exynos-dsi",
> > > +            .owner = THIS_MODULE,
> > > +            .pm = &exynos_dsi_pm_ops,
> > > +            .of_match_table = exynos_dsi_of_match,
> > > + },
> > > +};
> > >  
> > >  MODULE_AUTHOR("Tomasz Figa <t.f...@samsung.com>");
> > >  MODULE_AUTHOR("Andrzej Hajda <a.ha...@samsung.com>");
> > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi_pltfm.c 
> > > b/drivers/gpu/drm/exynos/exynos_drm_dsi_pltfm.c
> > > deleted file mode 100644
> > > index 79d9ec6ade45..000000000000
> > > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi_pltfm.c
> > > +++ /dev/null
> > > @@ -1,333 +0,0 @@
> > > -// SPDX-License-Identifier: GPL-2.0-only
> > > -/*
> > > - * Samsung SoC MIPI DSI Master driver.
> > > - *
> > > - * Copyright (c) 2014 Samsung Electronics Co., Ltd
> > > - *
> > > - * Contacts: Tomasz Figa <t.f...@samsung.com>
> > > - */
> > > -
> > > -#include <linux/component.h>
> > > -#include <linux/of_device.h>
> > > -#include <linux/of_graph.h>
> > > -#include <linux/pm_runtime.h>
> > > -
> > > -#include <drm/drm_bridge.h>
> > > -#include <drm/drm_encoder.h>
> > > -#include <drm/drm_mipi_dsi.h>
> > > -#include <drm/drm_probe_helper.h>
> > > -#include <drm/drm_simple_kms_helper.h>
> > > -
> > > -#include "exynos_drm_crtc.h"
> > > -#include "exynos_drm_drv.h"
> > > -#include "exynos_drm_dsi.h"
> > > -
> > > -enum {
> > > - DSI_PORT_IN,
> > > - DSI_PORT_OUT
> > > -};
> > > -
> > > -struct exynos_dsi_pltfm {
> > > - struct exynos_dsi *dsi;
> > > - struct drm_encoder encoder;
> > > -};
> > > -
> > > -static const unsigned int reg_values[] = {
> > > - [RESET_TYPE] = DSIM_SWRST,
> > > - [PLL_TIMER] = 500,
> > > - [STOP_STATE_CNT] = 0xf,
> > > - [PHYCTRL_ULPS_EXIT] = 0x0af,
> > > - [PHYCTRL_VREG_LP] = 0,
> > > - [PHYCTRL_SLEW_UP] = 0,
> > > - [PHYTIMING_LPX] = 0x06,
> > > - [PHYTIMING_HS_EXIT] = 0x0b,
> > > - [PHYTIMING_CLK_PREPARE] = 0x07,
> > > - [PHYTIMING_CLK_ZERO] = 0x27,
> > > - [PHYTIMING_CLK_POST] = 0x0d,
> > > - [PHYTIMING_CLK_TRAIL] = 0x08,
> > > - [PHYTIMING_HS_PREPARE] = 0x09,
> > > - [PHYTIMING_HS_ZERO] = 0x0d,
> > > - [PHYTIMING_HS_TRAIL] = 0x0b,
> > > -};
> > > -
> > > -static const unsigned int exynos5422_reg_values[] = {
> > > - [RESET_TYPE] = DSIM_SWRST,
> > > - [PLL_TIMER] = 500,
> > > - [STOP_STATE_CNT] = 0xf,
> > > - [PHYCTRL_ULPS_EXIT] = 0xaf,
> > > - [PHYCTRL_VREG_LP] = 0,
> > > - [PHYCTRL_SLEW_UP] = 0,
> > > - [PHYTIMING_LPX] = 0x08,
> > > - [PHYTIMING_HS_EXIT] = 0x0d,
> > > - [PHYTIMING_CLK_PREPARE] = 0x09,
> > > - [PHYTIMING_CLK_ZERO] = 0x30,
> > > - [PHYTIMING_CLK_POST] = 0x0e,
> > > - [PHYTIMING_CLK_TRAIL] = 0x0a,
> > > - [PHYTIMING_HS_PREPARE] = 0x0c,
> > > - [PHYTIMING_HS_ZERO] = 0x11,
> > > - [PHYTIMING_HS_TRAIL] = 0x0d,
> > > -};
> > > -
> > > -static const unsigned int exynos5433_reg_values[] = {
> > > - [RESET_TYPE] = DSIM_FUNCRST,
> > > - [PLL_TIMER] = 22200,
> > > - [STOP_STATE_CNT] = 0xa,
> > > - [PHYCTRL_ULPS_EXIT] = 0x190,
> > > - [PHYCTRL_VREG_LP] = 1,
> > > - [PHYCTRL_SLEW_UP] = 1,
> > > - [PHYTIMING_LPX] = 0x07,
> > > - [PHYTIMING_HS_EXIT] = 0x0c,
> > > - [PHYTIMING_CLK_PREPARE] = 0x09,
> > > - [PHYTIMING_CLK_ZERO] = 0x2d,
> > > - [PHYTIMING_CLK_POST] = 0x0e,
> > > - [PHYTIMING_CLK_TRAIL] = 0x09,
> > > - [PHYTIMING_HS_PREPARE] = 0x0b,
> > > - [PHYTIMING_HS_ZERO] = 0x10,
> > > - [PHYTIMING_HS_TRAIL] = 0x0c,
> > > -};
> > > -
> > > -static int __exynos_dsi_host_attach(struct device *dev,
> > > -                             struct mipi_dsi_device *device)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = dev_get_drvdata(dev);
> > > - struct drm_device *drm = dsi->encoder.dev;
> > > - struct exynos_drm_crtc *crtc;
> > > -
> > > - mutex_lock(&drm->mode_config.mutex);
> > > - crtc = exynos_drm_crtc_get_by_type(drm, EXYNOS_DISPLAY_TYPE_LCD);
> > > - crtc->i80_mode = !(device->mode_flags & MIPI_DSI_MODE_VIDEO);
> > > - mutex_unlock(&drm->mode_config.mutex);
> > > -
> > > - if (drm->mode_config.poll_enabled)
> > > -         drm_kms_helper_hotplug_event(drm);
> > > -
> > > - return 0;
> > > -}
> > > -
> > > -static int __exynos_dsi_host_detach(struct device *dev,
> > > -                             struct mipi_dsi_device *device)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = dev_get_drvdata(dev);
> > > - struct drm_device *drm = dsi->encoder.dev;
> > > -
> > > - if (drm->mode_config.poll_enabled)
> > > -         drm_kms_helper_hotplug_event(drm);
> > > -
> > > - return 0;
> > > -}
> > > -
> > > -static void __exynos_dsi_te_handler(struct device *dev)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = dev_get_drvdata(dev);
> > > -
> > > - exynos_drm_crtc_te_handler(dsi->encoder.crtc);
> > > -}
> > > -
> > > -static const struct exynos_dsi_host_ops exynos_dsi_host_ops = {
> > > - .attach = __exynos_dsi_host_attach,
> > > - .detach = __exynos_dsi_host_detach,
> > > - .te_handler = __exynos_dsi_te_handler,
> > > -};
> > > -
> > > -static const struct exynos_dsi_driver_data exynos3_dsi_driver_data = {
> > > - .reg_ofs = EXYNOS_REG_OFS,
> > > - .plltmr_reg = 0x50,
> > > - .has_freqband = 1,
> > > - .has_clklane_stop = 1,
> > > - .num_clks = 2,
> > > - .max_freq = 1000,
> > > - .wait_for_reset = 1,
> > > - .num_bits_resol = 11,
> > > - .reg_values = reg_values,
> > > - .host_ops = &exynos_dsi_host_ops,
> > > -};
> > > -
> > > -static const struct exynos_dsi_driver_data exynos4_dsi_driver_data = {
> > > - .reg_ofs = EXYNOS_REG_OFS,
> > > - .plltmr_reg = 0x50,
> > > - .has_freqband = 1,
> > > - .has_clklane_stop = 1,
> > > - .num_clks = 2,
> > > - .max_freq = 1000,
> > > - .wait_for_reset = 1,
> > > - .num_bits_resol = 11,
> > > - .reg_values = reg_values,
> > > - .host_ops = &exynos_dsi_host_ops,
> > > -};
> > > -
> > > -static const struct exynos_dsi_driver_data exynos5_dsi_driver_data = {
> > > - .reg_ofs = EXYNOS_REG_OFS,
> > > - .plltmr_reg = 0x58,
> > > - .num_clks = 2,
> > > - .max_freq = 1000,
> > > - .wait_for_reset = 1,
> > > - .num_bits_resol = 11,
> > > - .reg_values = reg_values,
> > > - .host_ops = &exynos_dsi_host_ops,
> > > -};
> > > -
> > > -static const struct exynos_dsi_driver_data exynos5433_dsi_driver_data = {
> > > - .reg_ofs = EXYNOS5433_REG_OFS,
> > > - .plltmr_reg = 0xa0,
> > > - .has_clklane_stop = 1,
> > > - .num_clks = 5,
> > > - .max_freq = 1500,
> > > - .wait_for_reset = 0,
> > > - .num_bits_resol = 12,
> > > - .reg_values = exynos5433_reg_values,
> > > - .host_ops = &exynos_dsi_host_ops,
> > > -};
> > > -
> > > -static const struct exynos_dsi_driver_data exynos5422_dsi_driver_data = {
> > > - .reg_ofs = EXYNOS5433_REG_OFS,
> > > - .plltmr_reg = 0xa0,
> > > - .has_clklane_stop = 1,
> > > - .num_clks = 2,
> > > - .max_freq = 1500,
> > > - .wait_for_reset = 1,
> > > - .num_bits_resol = 12,
> > > - .reg_values = exynos5422_reg_values,
> > > - .host_ops = &exynos_dsi_host_ops,
> > > -};
> > > -
> > > -static const struct of_device_id exynos_dsi_of_match[] = {
> > > - { .compatible = "samsung,exynos3250-mipi-dsi",
> > > -   .data = &exynos3_dsi_driver_data },
> > > - { .compatible = "samsung,exynos4210-mipi-dsi",
> > > -   .data = &exynos4_dsi_driver_data },
> > > - { .compatible = "samsung,exynos5410-mipi-dsi",
> > > -   .data = &exynos5_dsi_driver_data },
> > > - { .compatible = "samsung,exynos5422-mipi-dsi",
> > > -   .data = &exynos5422_dsi_driver_data },
> > > - { .compatible = "samsung,exynos5433-mipi-dsi",
> > > -   .data = &exynos5433_dsi_driver_data },
> > > - { }
> > > -};
> > > -
> > > -static int exynos_dsi_pltfm_bind(struct device *dev, struct device 
> > > *master, void *data)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = dev_get_drvdata(dev);
> > > - struct drm_encoder *encoder = &dsi->encoder;
> > > - struct drm_device *drm_dev = data;
> > > - struct device_node *in_bridge_node;
> > > - struct drm_bridge *in_bridge;
> > > - int ret;
> > > -
> > > - drm_simple_encoder_init(drm_dev, encoder, DRM_MODE_ENCODER_TMDS);
> > > -
> > > - ret = exynos_drm_set_possible_crtcs(encoder, EXYNOS_DISPLAY_TYPE_LCD);
> > > - if (ret < 0)
> > > -         return ret;
> > > -
> > > - in_bridge_node = of_graph_get_remote_node(dev->of_node, DSI_PORT_IN, 0);
> > > - if (in_bridge_node) {
> > > -         in_bridge = of_drm_find_bridge(in_bridge_node);
> > > -         if (in_bridge)
> > > -                 drm_bridge_attach(encoder, in_bridge, NULL, 0);
> > > -         of_node_put(in_bridge_node);
> > > - }
> > > -
> > > - ret = exynos_dsi_bind(dsi->dsi, encoder);
> > > - if (ret)
> > > -         goto err;
> > > -
> > > - return 0;
> > > -
> > > -err:
> > > - drm_encoder_cleanup(encoder);
> > > - return ret;
> > > -}
> > > -
> > > -static void exynos_dsi_pltfm_unbind(struct device *dev, struct device 
> > > *master,
> > > -                             void *data)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = dev_get_drvdata(dev);
> > > - struct drm_encoder *encoder = &dsi->encoder;
> > > -
> > > - exynos_dsi_unbind(dsi->dsi);
> > > -
> > > - drm_encoder_cleanup(encoder);
> > > -}
> > > -
> > > -static const struct component_ops exynos_dsi_pltfm_component_ops = {
> > > - .bind   = exynos_dsi_pltfm_bind,
> > > - .unbind = exynos_dsi_pltfm_unbind,
> > > -};
> > > -
> > > -static int exynos_dsi_pltfm_probe(struct platform_device *pdev)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi;
> > > - struct device *dev = &pdev->dev;
> > > - int ret;
> > > -
> > > - dsi = devm_kzalloc(dev, sizeof(*dsi), GFP_KERNEL);
> > > - if (!dsi)
> > > -         return -ENOMEM;
> > > - platform_set_drvdata(pdev, dsi);
> > > -
> > > - dsi->dsi = exynos_dsi_probe(pdev);
> > > - if (IS_ERR(dsi->dsi))
> > > -         return PTR_ERR(dsi->dsi);
> > > -
> > > - pm_runtime_enable(dev);
> > > -
> > > - ret = component_add(dev, &exynos_dsi_pltfm_component_ops);
> > > - if (ret)
> > > -         goto err_disable_runtime;
> > > -
> > > - return 0;
> > > -
> > > -err_disable_runtime:
> > > - pm_runtime_disable(dev);
> > > -
> > > - return ret;
> > > -}
> > > -
> > > -static int exynos_dsi_pltfm_remove(struct platform_device *pdev)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = platform_get_drvdata(pdev);
> > > -
> > > - pm_runtime_disable(&pdev->dev);
> > > -
> > > - exynos_dsi_remove(dsi->dsi);
> > > -
> > > - component_del(&pdev->dev, &exynos_dsi_pltfm_component_ops);
> > > -
> > > - return 0;
> > > -}
> > > -
> > > -static int __maybe_unused exynos_dsi_pltfm_suspend(struct device *dev)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = dev_get_drvdata(dev);
> > > -
> > > - return exynos_dsi_suspend(dsi->dsi);
> > > -}
> > > -
> > > -static int __maybe_unused exynos_dsi_pltfm_resume(struct device *dev)
> > > -{
> > > - struct exynos_dsi_pltfm *dsi = dev_get_drvdata(dev);
> > > -
> > > - return exynos_dsi_resume(dsi->dsi);
> > > -}
> > > -
> > > -static const struct dev_pm_ops exynos_dsi_pm_ops = {
> > > - SET_RUNTIME_PM_OPS(exynos_dsi_pltfm_suspend, exynos_dsi_pltfm_resume, 
> > > NULL)
> > > - SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
> > > -                         pm_runtime_force_resume)
> > > -};
> > > -
> > > -struct platform_driver dsi_driver = {
> > > - .probe = exynos_dsi_pltfm_probe,
> > > - .remove = exynos_dsi_pltfm_remove,
> > > - .driver = {
> > > -            .name = "exynos-dsi",
> > > -            .owner = THIS_MODULE,
> > > -            .pm = &exynos_dsi_pm_ops,
> > > -            .of_match_table = exynos_dsi_of_match,
> > > - },
> > > -};
> > > -
> > > -MODULE_AUTHOR("Tomasz Figa <t.f...@samsung.com>");
> > > -MODULE_AUTHOR("Andrzej Hajda <a.ha...@samsung.com>");
> > > -MODULE_DESCRIPTION("Samsung SoC MIPI DSI Master");
> > > -MODULE_LICENSE("GPL v2");
> > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dsi.h 
> > > b/include/drm/bridge/samsung-dsim.h
> > > similarity index 69%
> > > rename from drivers/gpu/drm/exynos/exynos_drm_dsi.h
> > > rename to include/drm/bridge/samsung-dsim.h
> > > index 8fa3276889de..be8b4913aa9c 100644
> > > --- a/drivers/gpu/drm/exynos/exynos_drm_dsi.h
> > > +++ b/include/drm/bridge/samsung-dsim.h
> > > @@ -3,7 +3,7 @@
> > >  #define __EXYNOS_DRM_DSI__
> > >  
> > >  struct drm_encoder;
> > > -struct exynos_dsi;
> > > +struct samsung_dsim;
> > >  struct platform_device;
> > >  struct mipi_dsi_device;
> > >  
> > > @@ -12,13 +12,13 @@ enum exynos_reg_offset {
> > >   EXYNOS5433_REG_OFS
> > >  };
> > >  
> > > -struct exynos_dsi *exynos_dsi_probe(struct platform_device *pdev);
> > > -void exynos_dsi_remove(struct exynos_dsi *dsi);
> > > -int exynos_dsi_bind(struct exynos_dsi *dsi, struct drm_encoder *encoder);
> > > -void exynos_dsi_unbind(struct exynos_dsi *dsi);
> > > +struct samsung_dsim *samsung_dsim_probe(struct platform_device *pdev);
> > > +void samsung_dsim_remove(struct samsung_dsim *dsi);
> > > +int samsung_dsim_bind(struct samsung_dsim *dsi, struct drm_encoder 
> > > *encoder);
> > > +void samsung_dsim_unbind(struct samsung_dsim *dsi);
> > >  
> > > -int exynos_dsi_suspend(struct exynos_dsi *dsi);
> > > -int exynos_dsi_resume(struct exynos_dsi *dsi);
> > > +int samsung_dsim_suspend(struct samsung_dsim *dsi);
> > > +int samsung_dsim_resume(struct samsung_dsim *dsi);
> > >  
> > >  enum reg_value_idx {
> > >   RESET_TYPE,
> > > @@ -42,13 +42,13 @@ enum reg_value_idx {
> > >  #define DSIM_FUNCRST                     (1 << 16)
> > >  #define DSIM_SWRST                       (1 << 0)
> > >  
> > > -struct exynos_dsi_host_ops {
> > > +struct samsung_dsim_host_ops {
> > >   int (*attach)(struct device *dev, struct mipi_dsi_device *device);
> > >   int (*detach)(struct device *dev, struct mipi_dsi_device *device);
> > >   void (*te_handler)(struct device *dev);
> > >  };
> > >  
> > > -struct exynos_dsi_driver_data {
> > > +struct samsung_dsim_driver_data {
> > >   enum exynos_reg_offset reg_ofs;
> > >   unsigned int plltmr_reg;
> > >   unsigned int has_freqband:1;
> > > @@ -58,7 +58,7 @@ struct exynos_dsi_driver_data {
> > >   unsigned int wait_for_reset;
> > >   unsigned int num_bits_resol;
> > >   const unsigned int *reg_values;
> > > - const struct exynos_dsi_host_ops *host_ops;
> > > + const struct samsung_dsim_host_ops *host_ops;
> > >  };
> > >  
> > >  #endif /* __EXYNOS_DRM_DSI__ */
> > > -- 
> > > 2.20.1
> > > 
> > > _______________________________________________
> > > dri-devel mailing list
> > > dri-devel@lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> > 
> > -- 
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
> > 
> 
> -- 
> Pengutronix e.K.                           | Michael Tretter             |
> Steuerwalder Str. 21                       | https://www.pengutronix.de/ |
> 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

-- 
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to