drm/rockchip: vop: use crtc_[h/v]display for vop
[firefly-linux-kernel-4.4.55.git] / drivers / gpu / drm / rockchip / rockchip_drm_vop.c
index 80194f7ae7eb45e3cc7aece9c86431a78985921a..e21b8e28485b65a4c7c7d53466dcb0306e5d1afa 100644 (file)
@@ -19,6 +19,7 @@
 #include <drm/drm_crtc_helper.h>
 #include <drm/drm_plane_helper.h>
 
+#include <linux/iopoll.h>
 #include <linux/kernel.h>
 #include <linux/module.h>
 #include <linux/platform_device.h>
 #include <linux/reset.h>
 #include <linux/delay.h>
 #include <linux/sort.h>
+#include <uapi/drm/rockchip_drm.h>
 
 #include "rockchip_drm_drv.h"
 #include "rockchip_drm_gem.h"
 #include "rockchip_drm_fb.h"
 #include "rockchip_drm_vop.h"
 
-#define __REG_SET_RELAXED(x, off, mask, shift, v, write_mask) \
-               vop_mask_write(x, off, mask, shift, v, write_mask, true)
+#define VOP_REG_SUPPORT(vop, reg) \
+               (!reg.major || (reg.major == VOP_MAJOR(vop->data->version) && \
+               reg.begin_minor <= VOP_MINOR(vop->data->version) && \
+               reg.end_minor >= VOP_MINOR(vop->data->version) && \
+               reg.mask))
 
-#define __REG_SET_NORMAL(x, off, mask, shift, v, write_mask) \
-               vop_mask_write(x, off, mask, shift, v, write_mask, false)
+#define VOP_WIN_SUPPORT(vop, win, name) \
+               VOP_REG_SUPPORT(vop, win->phy->name)
 
-#define REG_SET(x, off, reg, v, mode) \
-               __REG_SET_##mode(x, off + reg.offset, \
-                                reg.mask, reg.shift, v, reg.write_mask)
-#define REG_SET_MASK(x, off, reg, mask, v, mode) \
-               __REG_SET_##mode(x, off + reg.offset, \
-                                mask, reg.shift, v, reg.write_mask)
+#define VOP_CTRL_SUPPORT(vop, name) \
+               VOP_REG_SUPPORT(vop, vop->data->ctrl->name)
+
+#define VOP_INTR_SUPPORT(vop, name) \
+               VOP_REG_SUPPORT(vop, vop->data->intr->name)
+
+#define __REG_SET(x, off, mask, shift, v, write_mask, relaxed) \
+               vop_mask_write(x, off, mask, shift, v, write_mask, relaxed)
+
+#define _REG_SET(vop, name, off, reg, mask, v, relaxed) \
+       do { \
+               if (VOP_REG_SUPPORT(vop, reg)) \
+                       __REG_SET(vop, off + reg.offset, mask, reg.shift, \
+                                 v, reg.write_mask, relaxed); \
+               else \
+                       dev_dbg(vop->dev, "Warning: not support "#name"\n"); \
+       } while(0)
+
+#define REG_SET(x, name, off, reg, v, relaxed) \
+               _REG_SET(x, name, off, reg, reg.mask, v, relaxed)
+#define REG_SET_MASK(x, name, off, reg, mask, v, relaxed) \
+               _REG_SET(x, name, off, reg, reg.mask & mask, v, relaxed)
 
 #define VOP_WIN_SET(x, win, name, v) \
-               REG_SET(x, win->offset, VOP_WIN_NAME(win, name), v, RELAXED)
+               REG_SET(x, name, win->offset, VOP_WIN_NAME(win, name), v, true)
+#define VOP_WIN_SET_EXT(x, win, ext, name, v) \
+               REG_SET(x, name, 0, win->ext->name, v, true)
 #define VOP_SCL_SET(x, win, name, v) \
-               REG_SET(x, win->offset, win->phy->scl->name, v, RELAXED)
+               REG_SET(x, name, win->offset, win->phy->scl->name, v, true)
 #define VOP_SCL_SET_EXT(x, win, name, v) \
-               REG_SET(x, win->offset, win->phy->scl->ext->name, v, RELAXED)
+               REG_SET(x, name, win->offset, win->phy->scl->ext->name, v, true)
 
 #define VOP_CTRL_SET(x, name, v) \
-               REG_SET(x, 0, (x)->data->ctrl->name, v, NORMAL)
+               REG_SET(x, name, 0, (x)->data->ctrl->name, v, false)
 
 #define VOP_INTR_GET(vop, name) \
                vop_read_reg(vop, 0, &vop->data->ctrl->name)
 
-#define VOP_INTR_SET(vop, name, mask, v) \
-               REG_SET_MASK(vop, 0, vop->data->intr->name, mask, v, NORMAL)
+#define VOP_INTR_SET(vop, name, v) \
+               REG_SET(vop, name, 0, vop->data->intr->name, \
+                       v, false)
+#define VOP_INTR_SET_MASK(vop, name, mask, v) \
+               REG_SET_MASK(vop, name, 0, vop->data->intr->name, \
+                            mask, v, false)
+
 #define VOP_INTR_SET_TYPE(vop, name, type, v) \
        do { \
                int i, reg = 0, mask = 0; \
                                mask |= 1 << i; \
                        } \
                } \
-               VOP_INTR_SET(vop, name, mask, reg); \
+               VOP_INTR_SET_MASK(vop, name, mask, reg); \
        } while (0)
 #define VOP_INTR_GET_TYPE(vop, name, type) \
                vop_get_intr_type(vop, &vop->data->intr->name, type)
 
+#define VOP_CTRL_GET(x, name) \
+               vop_read_reg(x, 0, &vop->data->ctrl->name)
+
 #define VOP_WIN_GET(x, win, name) \
                vop_read_reg(x, win->offset, &VOP_WIN_NAME(win, name))
 
 #define VOP_WIN_GET_YRGBADDR(vop, win) \
                vop_readl(vop, win->offset + VOP_WIN_NAME(win, yrgb_mst).offset)
 
-#define VOP_WIN_SUPPORT(win, name) \
-               (win->phy->name.mask ? true : false)
-
 #define to_vop(x) container_of(x, struct vop, crtc)
 #define to_vop_win(x) container_of(x, struct vop_win, base)
 #define to_vop_plane_state(x) container_of(x, struct vop_plane_state, base)
@@ -107,6 +135,10 @@ struct vop_plane_state {
        struct drm_rect src;
        struct drm_rect dest;
        dma_addr_t yrgb_mst;
+       dma_addr_t uv_mst;
+       const uint32_t *y2r_table;
+       const uint32_t *r2r_table;
+       const uint32_t *r2y_table;
        bool enable;
 };
 
@@ -119,6 +151,7 @@ struct vop_win {
        uint32_t offset;
        enum drm_plane_type type;
        const struct vop_win_phy *phy;
+       const struct vop_csc *csc;
        const uint32_t *data_formats;
        uint32_t nformats;
        struct vop *vop;
@@ -132,15 +165,22 @@ struct vop {
        struct device *dev;
        struct drm_device *drm_dev;
        struct drm_property *plane_zpos_prop;
+       struct drm_property *plane_feature_prop;
+       struct drm_property *feature_prop;
+       bool is_iommu_enabled;
+       bool is_iommu_needed;
        bool is_enabled;
 
        /* mutex vsync_ work */
        struct mutex vsync_mutex;
        bool vsync_work_pending;
+       bool loader_protect;
        struct completion dsp_hold_completion;
        struct completion wait_update_complete;
        struct drm_pending_vblank_event *event;
 
+       struct completion line_flag_completion;
+
        const struct vop_data *data;
        int num_wins;
 
@@ -232,11 +272,51 @@ static inline uint32_t vop_get_intr_type(struct vop *vop,
        return ret;
 }
 
+static void vop_load_csc_table(struct vop *vop, u32 offset, const u32 *table)
+{
+       int i;
+
+       if (!table)
+               return;
+
+       for (i = 0; i < 8; i++)
+               vop_writel(vop, offset + i * 4, table[i]);
+}
+
 static inline void vop_cfg_done(struct vop *vop)
 {
        VOP_CTRL_SET(vop, cfg_done, 1);
 }
 
+static bool vop_is_allwin_disabled(struct vop *vop)
+{
+       int i;
+
+       for (i = 0; i < vop->num_wins; i++) {
+               struct vop_win *win = &vop->win[i];
+
+               if (VOP_WIN_GET(vop, win, enable) != 0)
+                       return false;
+       }
+
+       return true;
+}
+
+static bool vop_is_cfg_done_complete(struct vop *vop)
+{
+       return VOP_CTRL_GET(vop, cfg_done) ? false : true;
+}
+
+static bool vop_fs_irq_is_active(struct vop *vop)
+{
+       return VOP_INTR_GET_TYPE(vop, status, FS_INTR);
+}
+
+static bool vop_line_flag_is_active(struct vop *vop)
+{
+       return VOP_INTR_GET_TYPE(vop, status, LINE_FLAG_INTR);
+}
+
 static bool has_rb_swapped(uint32_t format)
 {
        switch (format) {
@@ -265,10 +345,13 @@ static enum vop_data_format vop_convert_format(uint32_t format)
        case DRM_FORMAT_BGR565:
                return VOP_FMT_RGB565;
        case DRM_FORMAT_NV12:
+       case DRM_FORMAT_NV12_10:
                return VOP_FMT_YUV420SP;
        case DRM_FORMAT_NV16:
+       case DRM_FORMAT_NV16_10:
                return VOP_FMT_YUV422SP;
        case DRM_FORMAT_NV24:
+       case DRM_FORMAT_NV24_10:
                return VOP_FMT_YUV444SP;
        default:
                DRM_ERROR("unsupport format[%08x]\n", format);
@@ -276,12 +359,38 @@ static enum vop_data_format vop_convert_format(uint32_t format)
        }
 }
 
+static bool is_yuv_output(uint32_t bus_format)
+{
+       switch (bus_format) {
+       case MEDIA_BUS_FMT_YUV8_1X24:
+       case MEDIA_BUS_FMT_YUV10_1X30:
+               return true;
+       default:
+               return false;
+       }
+}
+
 static bool is_yuv_support(uint32_t format)
 {
        switch (format) {
        case DRM_FORMAT_NV12:
+       case DRM_FORMAT_NV12_10:
        case DRM_FORMAT_NV16:
+       case DRM_FORMAT_NV16_10:
        case DRM_FORMAT_NV24:
+       case DRM_FORMAT_NV24_10:
+               return true;
+       default:
+               return false;
+       }
+}
+
+static bool is_yuv_10bit(uint32_t format)
+{
+       switch (format) {
+       case DRM_FORMAT_NV12_10:
+       case DRM_FORMAT_NV16_10:
+       case DRM_FORMAT_NV24_10:
                return true;
        default:
                return false;
@@ -350,11 +459,6 @@ static void scl_vop_cal_scl_fac(struct vop *vop, struct vop_win *win,
        if (!win->phy->scl)
                return;
 
-       if (dst_w > 3840) {
-               DRM_ERROR("Maximum destination width (3840) exceeded\n");
-               return;
-       }
-
        if (!win->phy->scl->ext) {
                VOP_SCL_SET(vop, win, scale_yrgb_x,
                            scl_cal_scale2(src_w, dst_w));
@@ -419,6 +523,8 @@ static void scl_vop_cal_scl_fac(struct vop *vop, struct vop_win *win,
        VOP_SCL_SET_EXT(vop, win, yrgb_vsd_mode, SCALE_DOWN_BIL);
        VOP_SCL_SET_EXT(vop, win, yrgb_vsu_mode, vsu_mode);
        if (is_yuv) {
+               vskiplines = 0;
+
                val = scl_vop_cal_scale(cbcr_hor_scl_mode, cbcr_src_w,
                                        dst_w, true, 0, NULL);
                VOP_SCL_SET(vop, win, scale_cbcr_x, val);
@@ -436,21 +542,218 @@ static void scl_vop_cal_scl_fac(struct vop *vop, struct vop_win *win,
        }
 }
 
+/*
+ * rk3399 colorspace path:
+ *      Input        Win csc                     Output
+ * 1. YUV(2020)  --> Y2R->2020To709->R2Y   --> YUV_OUTPUT(601/709)
+ *    RGB        --> R2Y                  __/
+ *
+ * 2. YUV(2020)  --> bypasss               --> YUV_OUTPUT(2020)
+ *    RGB        --> 709To2020->R2Y       __/
+ *
+ * 3. YUV(2020)  --> Y2R->2020To709        --> RGB_OUTPUT(709)
+ *    RGB        --> R2Y                  __/
+ *
+ * 4. YUV(601/709)-> Y2R->709To2020->R2Y   --> YUV_OUTPUT(2020)
+ *    RGB        --> 709To2020->R2Y       __/
+ *
+ * 5. YUV(601/709)-> bypass                --> YUV_OUTPUT(709)
+ *    RGB        --> R2Y                  __/
+ *
+ * 6. YUV(601/709)-> bypass                --> YUV_OUTPUT(601)
+ *    RGB        --> R2Y(601)             __/
+ *
+ * 7. YUV        --> Y2R(709)              --> RGB_OUTPUT(709)
+ *    RGB        --> bypass               __/
+ *
+ * 8. RGB        --> 709To2020->R2Y        --> YUV_OUTPUT(2020)
+ *
+ * 9. RGB        --> R2Y(709)              --> YUV_OUTPUT(709)
+ *
+ * 10. RGB       --> R2Y(601)              --> YUV_OUTPUT(601)
+ *
+ * 11. RGB       --> bypass                --> RGB_OUTPUT(709)
+ */
+static int vop_csc_setup(const struct vop_csc_table *csc_table,
+                        bool is_input_yuv, bool is_output_yuv,
+                        int input_csc, int output_csc,
+                        const uint32_t **y2r_table,
+                        const uint32_t **r2r_table,
+                        const uint32_t **r2y_table)
+{
+       *y2r_table = NULL;
+       *r2r_table = NULL;
+       *r2y_table = NULL;
+
+       if (is_output_yuv) {
+               if (output_csc == CSC_BT2020) {
+                       if (is_input_yuv) {
+                               if (input_csc == CSC_BT2020)
+                                       return 0;
+                               *y2r_table = csc_table->y2r_bt709;
+                       }
+                       if (input_csc != CSC_BT2020)
+                               *r2r_table = csc_table->r2r_bt709_to_bt2020;
+                       *r2y_table = csc_table->r2y_bt2020;
+               } else {
+                       if (is_input_yuv && input_csc == CSC_BT2020)
+                               *y2r_table = csc_table->y2r_bt2020;
+                       if (input_csc == CSC_BT2020)
+                               *r2r_table = csc_table->r2r_bt2020_to_bt709;
+                       if (!is_input_yuv || *y2r_table) {
+                               if (output_csc == CSC_BT709)
+                                       *r2y_table = csc_table->r2y_bt709;
+                               else
+                                       *r2y_table = csc_table->r2y_bt601;
+                       }
+               }
+       } else {
+               if (!is_input_yuv)
+                       return 0;
+
+               /*
+                * is possible use bt2020 on rgb mode?
+                */
+               if (WARN_ON(output_csc == CSC_BT2020))
+                       return -EINVAL;
+
+               if (input_csc == CSC_BT2020)
+                       *y2r_table = csc_table->y2r_bt2020;
+               else if (input_csc == CSC_BT709)
+                       *y2r_table = csc_table->y2r_bt709;
+               else
+                       *y2r_table = csc_table->y2r_bt601;
+
+               if (input_csc == CSC_BT2020)
+                       /*
+                        * We don't have bt601 to bt709 table, force use bt709.
+                        */
+                       *r2r_table = csc_table->r2r_bt2020_to_bt709;
+       }
+
+       return 0;
+}
+
+static int vop_csc_atomic_check(struct drm_crtc *crtc,
+                               struct drm_crtc_state *crtc_state)
+{
+       struct vop *vop = to_vop(crtc);
+       struct drm_atomic_state *state = crtc_state->state;
+       struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc_state);
+       const struct vop_csc_table *csc_table = vop->data->csc_table;
+       struct drm_plane_state *pstate;
+       struct drm_plane *plane;
+       bool is_input_yuv, is_output_yuv;
+       int ret;
+
+       if (!csc_table)
+               return 0;
+
+       is_output_yuv = is_yuv_output(s->bus_format);
+
+       drm_atomic_crtc_state_for_each_plane(plane, crtc_state) {
+               struct vop_plane_state *vop_plane_state;
+
+               pstate = drm_atomic_get_plane_state(state, plane);
+               if (IS_ERR(pstate))
+                       return PTR_ERR(pstate);
+               vop_plane_state = to_vop_plane_state(pstate);
+
+               if (!pstate->fb)
+                       continue;
+               is_input_yuv = is_yuv_support(pstate->fb->pixel_format);
+
+               /*
+                * TODO: force set input and output csc mode.
+                */
+               ret = vop_csc_setup(csc_table, is_input_yuv, is_output_yuv,
+                                   CSC_BT709, CSC_BT709,
+                                   &vop_plane_state->y2r_table,
+                                   &vop_plane_state->r2r_table,
+                                   &vop_plane_state->r2y_table);
+               if (ret)
+                       return ret;
+       }
+
+       return 0;
+}
+
 static void vop_dsp_hold_valid_irq_enable(struct vop *vop)
 {
        unsigned long flags;
 
+       spin_lock_irqsave(&vop->irq_lock, flags);
+
+       VOP_INTR_SET_TYPE(vop, clear, DSP_HOLD_VALID_INTR, 1);
+       VOP_INTR_SET_TYPE(vop, enable, DSP_HOLD_VALID_INTR, 1);
+
+       spin_unlock_irqrestore(&vop->irq_lock, flags);
+}
+
+static void vop_dsp_hold_valid_irq_disable(struct vop *vop)
+{
+       unsigned long flags;
+
+       spin_lock_irqsave(&vop->irq_lock, flags);
+
+       VOP_INTR_SET_TYPE(vop, enable, DSP_HOLD_VALID_INTR, 0);
+
+       spin_unlock_irqrestore(&vop->irq_lock, flags);
+}
+
+/*
+ * (1) each frame starts at the start of the Vsync pulse which is signaled by
+ *     the "FRAME_SYNC" interrupt.
+ * (2) the active data region of each frame ends at dsp_vact_end
+ * (3) we should program this same number (dsp_vact_end) into dsp_line_frag_num,
+ *      to get "LINE_FLAG" interrupt at the end of the active on screen data.
+ *
+ * VOP_INTR_CTRL0.dsp_line_frag_num = VOP_DSP_VACT_ST_END.dsp_vact_end
+ * Interrupts
+ * LINE_FLAG -------------------------------+
+ * FRAME_SYNC ----+                         |
+ *                |                         |
+ *                v                         v
+ *                | Vsync | Vbp |  Vactive  | Vfp |
+ *                        ^     ^           ^     ^
+ *                        |     |           |     |
+ *                        |     |           |     |
+ * dsp_vs_end ------------+     |           |     |   VOP_DSP_VTOTAL_VS_END
+ * dsp_vact_start --------------+           |     |   VOP_DSP_VACT_ST_END
+ * dsp_vact_end ----------------------------+     |   VOP_DSP_VACT_ST_END
+ * dsp_total -------------------------------------+   VOP_DSP_VTOTAL_VS_END
+ */
+static bool vop_line_flag_irq_is_enabled(struct vop *vop)
+{
+       uint32_t line_flag_irq;
+       unsigned long flags;
+
+       spin_lock_irqsave(&vop->irq_lock, flags);
+
+       line_flag_irq = VOP_INTR_GET_TYPE(vop, enable, LINE_FLAG_INTR);
+
+       spin_unlock_irqrestore(&vop->irq_lock, flags);
+
+       return !!line_flag_irq;
+}
+
+static void vop_line_flag_irq_enable(struct vop *vop, int line_num)
+{
+       unsigned long flags;
+
        if (WARN_ON(!vop->is_enabled))
                return;
 
        spin_lock_irqsave(&vop->irq_lock, flags);
 
-       VOP_INTR_SET_TYPE(vop, enable, DSP_HOLD_VALID_INTR, 1);
+       VOP_INTR_SET(vop, line_flag_num[0], line_num);
+       VOP_INTR_SET_TYPE(vop, clear, LINE_FLAG_INTR, 1);
+       VOP_INTR_SET_TYPE(vop, enable, LINE_FLAG_INTR, 1);
 
        spin_unlock_irqrestore(&vop->irq_lock, flags);
 }
 
-static void vop_dsp_hold_valid_irq_disable(struct vop *vop)
+static void vop_line_flag_irq_disable(struct vop *vop)
 {
        unsigned long flags;
 
@@ -459,32 +762,29 @@ static void vop_dsp_hold_valid_irq_disable(struct vop *vop)
 
        spin_lock_irqsave(&vop->irq_lock, flags);
 
-       VOP_INTR_SET_TYPE(vop, enable, DSP_HOLD_VALID_INTR, 0);
+       VOP_INTR_SET_TYPE(vop, enable, LINE_FLAG_INTR, 0);
 
        spin_unlock_irqrestore(&vop->irq_lock, flags);
 }
 
-static void vop_enable(struct drm_crtc *crtc)
+static void vop_power_enable(struct drm_crtc *crtc)
 {
        struct vop *vop = to_vop(crtc);
        int ret;
 
-       if (vop->is_enabled)
-               return;
-
-       ret = clk_enable(vop->hclk);
+       ret = clk_prepare_enable(vop->hclk);
        if (ret < 0) {
                dev_err(vop->dev, "failed to enable hclk - %d\n", ret);
                return;
        }
 
-       ret = clk_enable(vop->dclk);
+       ret = clk_prepare_enable(vop->dclk);
        if (ret < 0) {
                dev_err(vop->dev, "failed to enable dclk - %d\n", ret);
                goto err_disable_hclk;
        }
 
-       ret = clk_enable(vop->aclk);
+       ret = clk_prepare_enable(vop->aclk);
        if (ret < 0) {
                dev_err(vop->dev, "failed to enable aclk - %d\n", ret);
                goto err_disable_dclk;
@@ -496,64 +796,53 @@ static void vop_enable(struct drm_crtc *crtc)
                return;
        }
 
-       /*
-        * Slave iommu shares power, irq and clock with vop.  It was associated
-        * automatically with this master device via common driver code.
-        * Now that we have enabled the clock we attach it to the shared drm
-        * mapping.
-        */
-       ret = rockchip_drm_dma_attach_device(vop->drm_dev, vop->dev);
-       if (ret) {
-               dev_err(vop->dev, "failed to attach dma mapping, %d\n", ret);
-               goto err_disable_aclk;
-       }
+       memcpy(vop->regsbak, vop->regs, vop->len);
 
-       memcpy(vop->regs, vop->regsbak, vop->len);
-       /*
-        * At here, vop clock & iommu is enable, R/W vop regs would be safe.
-        */
        vop->is_enabled = true;
 
-       spin_lock(&vop->reg_lock);
-
-       VOP_CTRL_SET(vop, standby, 0);
-
-       spin_unlock(&vop->reg_lock);
-
-       enable_irq(vop->irq);
-
-       drm_crtc_vblank_on(crtc);
-
        return;
 
-err_disable_aclk:
-       clk_disable(vop->aclk);
 err_disable_dclk:
-       clk_disable(vop->dclk);
+       clk_disable_unprepare(vop->dclk);
 err_disable_hclk:
-       clk_disable(vop->hclk);
+       clk_disable_unprepare(vop->hclk);
 }
 
-static void vop_crtc_disable(struct drm_crtc *crtc)
+static void vop_initial(struct drm_crtc *crtc)
 {
        struct vop *vop = to_vop(crtc);
        int i;
 
-       if (!vop->is_enabled)
-               return;
+       vop_power_enable(crtc);
+
+       VOP_CTRL_SET(vop, global_regdone_en, 1);
+       VOP_CTRL_SET(vop, dsp_blank, 0);
 
        /*
-        * We need to make sure that all windows are disabled before we
-        * disable that crtc. Otherwise we might try to scan from a destroyed
+        * We need to make sure that all windows are disabled before resume
+        * the crtc. Otherwise we might try to scan from a destroyed
         * buffer later.
         */
        for (i = 0; i < vop->num_wins; i++) {
                struct vop_win *win = &vop->win[i];
-
-               spin_lock(&vop->reg_lock);
+               int channel = i * 2 + 1;
+
+               VOP_WIN_SET(vop, win, channel, (channel + 1) << 4 | channel);
+               if (win->phy->scl && win->phy->scl->ext) {
+                       VOP_SCL_SET_EXT(vop, win, yrgb_hor_scl_mode, SCALE_NONE);
+                       VOP_SCL_SET_EXT(vop, win, yrgb_ver_scl_mode, SCALE_NONE);
+                       VOP_SCL_SET_EXT(vop, win, cbcr_hor_scl_mode, SCALE_NONE);
+                       VOP_SCL_SET_EXT(vop, win, cbcr_ver_scl_mode, SCALE_NONE);
+               }
                VOP_WIN_SET(vop, win, enable, 0);
-               spin_unlock(&vop->reg_lock);
+               VOP_WIN_SET(vop, win, gate, 1);
        }
+       VOP_CTRL_SET(vop, afbdc_en, 0);
+}
+
+static void vop_crtc_disable(struct drm_crtc *crtc)
+{
+       struct vop *vop = to_vop(crtc);
 
        drm_crtc_vblank_off(crtc);
 
@@ -573,23 +862,26 @@ static void vop_crtc_disable(struct drm_crtc *crtc)
 
        spin_unlock(&vop->reg_lock);
 
-       wait_for_completion(&vop->dsp_hold_completion);
+       WARN_ON(!wait_for_completion_timeout(&vop->dsp_hold_completion,
+                                            msecs_to_jiffies(50)));
 
        vop_dsp_hold_valid_irq_disable(vop);
 
        disable_irq(vop->irq);
 
        vop->is_enabled = false;
-
-       /*
-        * vop standby complete, so iommu detach is safe.
-        */
-       rockchip_drm_dma_detach_device(vop->drm_dev, vop->dev);
+       if (vop->is_iommu_enabled) {
+               /*
+                * vop standby complete, so iommu detach is safe.
+                */
+               rockchip_drm_dma_detach_device(vop->drm_dev, vop->dev);
+               vop->is_iommu_enabled = false;
+       }
 
        pm_runtime_put(vop->dev);
-       clk_disable(vop->dclk);
-       clk_disable(vop->aclk);
-       clk_disable(vop->hclk);
+       clk_disable_unprepare(vop->dclk);
+       clk_disable_unprepare(vop->aclk);
+       clk_disable_unprepare(vop->hclk);
 }
 
 static void vop_plane_destroy(struct drm_plane *plane)
@@ -621,6 +913,8 @@ static int vop_plane_atomic_check(struct drm_plane *plane,
        struct vop_win *win = to_vop_win(plane);
        struct vop_plane_state *vop_plane_state = to_vop_plane_state(state);
        struct drm_crtc_state *crtc_state;
+       const struct vop_data *vop_data;
+       struct vop *vop;
        bool visible;
        int ret;
        struct drm_rect *dest = &vop_plane_state->dest;
@@ -630,6 +924,9 @@ static int vop_plane_atomic_check(struct drm_plane *plane,
                                        DRM_PLANE_HELPER_NO_SCALING;
        int max_scale = win->phy->scl ? FRAC_16_16(8, 1) :
                                        DRM_PLANE_HELPER_NO_SCALING;
+       unsigned long offset;
+       dma_addr_t dma_addr;
+       u16 vdisplay;
 
        crtc = crtc ? crtc : plane->state->crtc;
        /*
@@ -651,10 +948,14 @@ static int vop_plane_atomic_check(struct drm_plane *plane,
        dest->x2 = state->crtc_x + state->crtc_w;
        dest->y2 = state->crtc_y + state->crtc_h;
 
+       vdisplay = crtc_state->adjusted_mode.crtc_vdisplay;
+       if (crtc_state->adjusted_mode.flags & DRM_MODE_FLAG_INTERLACE)
+               vdisplay *= 2;
+
        clip.x1 = 0;
        clip.y1 = 0;
-       clip.x2 = crtc_state->mode.hdisplay;
-       clip.y2 = crtc_state->mode.vdisplay;
+       clip.x2 = crtc_state->adjusted_mode.crtc_hdisplay;
+       clip.y2 = vdisplay;
 
        ret = drm_plane_helper_check_update(plane, crtc, state->fb,
                                            src, dest, &clip,
@@ -671,12 +972,48 @@ static int vop_plane_atomic_check(struct drm_plane *plane,
        if (vop_plane_state->format < 0)
                return vop_plane_state->format;
 
+       vop = to_vop(crtc);
+       vop_data = vop->data;
+
+       if (drm_rect_width(src) >> 16 > vop_data->max_input.width ||
+           drm_rect_height(src) >> 16 > vop_data->max_input.height) {
+               DRM_ERROR("Invalid source: %dx%d. max input: %dx%d\n",
+                         drm_rect_width(src) >> 16,
+                         drm_rect_height(src) >> 16,
+                         vop_data->max_input.width,
+                         vop_data->max_input.height);
+               return -EINVAL;
+       }
+
        /*
         * Src.x1 can be odd when do clip, but yuv plane start point
         * need align with 2 pixel.
         */
-       if (is_yuv_support(fb->pixel_format) && ((src->x1 >> 16) % 2))
+       if (is_yuv_support(fb->pixel_format) && ((src->x1 >> 16) % 2)) {
+               DRM_ERROR("Invalid Source: Yuv format Can't support odd xpos\n");
                return -EINVAL;
+       }
+
+       offset = (src->x1 >> 16) * drm_format_plane_bpp(fb->pixel_format, 0) / 8;
+       if (state->rotation & BIT(DRM_REFLECT_Y))
+               offset += ((src->y2 >> 16) - 1) * fb->pitches[0];
+       else
+               offset += (src->y1 >> 16) * fb->pitches[0];
+
+       dma_addr = rockchip_fb_get_dma_addr(fb, 0);
+       vop_plane_state->yrgb_mst = dma_addr + offset + fb->offsets[0];
+       if (is_yuv_support(fb->pixel_format)) {
+               int hsub = drm_format_horz_chroma_subsampling(fb->pixel_format);
+               int vsub = drm_format_vert_chroma_subsampling(fb->pixel_format);
+               int bpp = drm_format_plane_bpp(fb->pixel_format, 1);
+
+               offset = (src->x1 >> 16) * bpp / hsub / 8;
+               offset += (src->y1 >> 16) * fb->pitches[1] / vsub;
+
+               dma_addr = rockchip_fb_get_dma_addr(fb, 1);
+               dma_addr += offset + fb->offsets[1];
+               vop_plane_state->uv_mst = dma_addr;
+       }
 
        vop_plane_state->enable = true;
 
@@ -699,6 +1036,16 @@ static void vop_plane_atomic_disable(struct drm_plane *plane,
 
        spin_lock(&vop->reg_lock);
 
+       /*
+        * FIXUP: some of the vop scale would be abnormal after windows power
+        * on/off so deinit scale to scale_none mode.
+        */
+       if (win->phy->scl && win->phy->scl->ext) {
+               VOP_SCL_SET_EXT(vop, win, yrgb_hor_scl_mode, SCALE_NONE);
+               VOP_SCL_SET_EXT(vop, win, yrgb_ver_scl_mode, SCALE_NONE);
+               VOP_SCL_SET_EXT(vop, win, cbcr_hor_scl_mode, SCALE_NONE);
+               VOP_SCL_SET_EXT(vop, win, cbcr_ver_scl_mode, SCALE_NONE);
+       }
        VOP_WIN_SET(vop, win, enable, 0);
 
        spin_unlock(&vop->reg_lock);
@@ -713,18 +1060,17 @@ static void vop_plane_atomic_update(struct drm_plane *plane,
        struct drm_crtc *crtc = state->crtc;
        struct vop_win *win = to_vop_win(plane);
        struct vop_plane_state *vop_plane_state = to_vop_plane_state(state);
-       struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc->state);
-       struct vop *vop = to_vop(state->crtc);
+       struct rockchip_crtc_state *s;
+       struct vop *vop;
        struct drm_framebuffer *fb = state->fb;
        unsigned int actual_w, actual_h;
        unsigned int dsp_stx, dsp_sty;
        uint32_t act_info, dsp_info, dsp_st;
        struct drm_rect *src = &vop_plane_state->src;
        struct drm_rect *dest = &vop_plane_state->dest;
-       struct drm_gem_object *obj, *uv_obj;
-       struct rockchip_gem_object *rk_obj, *rk_uv_obj;
-       unsigned long offset;
-       dma_addr_t dma_addr;
+       const uint32_t *y2r_table = vop_plane_state->y2r_table;
+       const uint32_t *r2r_table = vop_plane_state->r2r_table;
+       const uint32_t *r2y_table = vop_plane_state->r2y_table;
        int ymirror, xmirror;
        uint32_t val;
        bool rb_swap;
@@ -735,17 +1081,11 @@ static void vop_plane_atomic_update(struct drm_plane *plane,
        if (!crtc)
                return;
 
-       if (WARN_ON(!vop->is_enabled))
-               return;
-
        if (!vop_plane_state->enable) {
                vop_plane_atomic_disable(plane, old_state);
                return;
        }
 
-       obj = rockchip_fb_get_gem_obj(fb, 0);
-       rk_obj = to_rockchip_obj(obj);
-
        actual_w = drm_rect_width(src) >> 16;
        actual_h = drm_rect_height(src) >> 16;
        act_info = (actual_h - 1) << 16 | ((actual_w - 1) & 0xffff);
@@ -757,16 +1097,12 @@ static void vop_plane_atomic_update(struct drm_plane *plane,
        dsp_sty = dest->y1 + crtc->mode.vtotal - crtc->mode.vsync_start;
        dsp_st = dsp_sty << 16 | (dsp_stx & 0xffff);
 
-       offset = (src->x1 >> 16) * drm_format_plane_cpp(fb->pixel_format, 0);
-       if (state->rotation & BIT(DRM_REFLECT_Y))
-               offset += ((src->y2 >> 16) - 1) * fb->pitches[0];
-       else
-               offset += (src->y1 >> 16) * fb->pitches[0];
-       vop_plane_state->yrgb_mst = rk_obj->dma_addr + offset + fb->offsets[0];
-
        ymirror = !!(state->rotation & BIT(DRM_REFLECT_Y));
        xmirror = !!(state->rotation & BIT(DRM_REFLECT_X));
 
+       vop = to_vop(state->crtc);
+       s = to_rockchip_crtc_state(crtc->state);
+
        spin_lock(&vop->reg_lock);
 
        VOP_WIN_SET(vop, win, xmirror, xmirror);
@@ -775,20 +1111,10 @@ static void vop_plane_atomic_update(struct drm_plane *plane,
        VOP_WIN_SET(vop, win, yrgb_vir, fb->pitches[0] >> 2);
        VOP_WIN_SET(vop, win, yrgb_mst, vop_plane_state->yrgb_mst);
        if (is_yuv_support(fb->pixel_format)) {
-               int hsub = drm_format_horz_chroma_subsampling(fb->pixel_format);
-               int vsub = drm_format_vert_chroma_subsampling(fb->pixel_format);
-               int bpp = drm_format_plane_cpp(fb->pixel_format, 1);
-
-               uv_obj = rockchip_fb_get_gem_obj(fb, 1);
-               rk_uv_obj = to_rockchip_obj(uv_obj);
-
-               offset = (src->x1 >> 16) * bpp / hsub;
-               offset += (src->y1 >> 16) * fb->pitches[1] / vsub;
-
-               dma_addr = rk_uv_obj->dma_addr + offset + fb->offsets[1];
                VOP_WIN_SET(vop, win, uv_vir, fb->pitches[1] >> 2);
-               VOP_WIN_SET(vop, win, uv_mst, dma_addr);
+               VOP_WIN_SET(vop, win, uv_mst, vop_plane_state->uv_mst);
        }
+       VOP_WIN_SET(vop, win, fmt_10, is_yuv_10bit(fb->pixel_format));
 
        scl_vop_cal_scl_fac(vop, win, actual_w, actual_h,
                            drm_rect_width(dest), drm_rect_height(dest),
@@ -818,8 +1144,17 @@ static void vop_plane_atomic_update(struct drm_plane *plane,
                VOP_WIN_SET(vop, win, alpha_en, 0);
        }
 
+       if (win->csc) {
+               vop_load_csc_table(vop, win->csc->y2r_offset, y2r_table);
+               vop_load_csc_table(vop, win->csc->r2r_offset, r2r_table);
+               vop_load_csc_table(vop, win->csc->r2y_offset, r2y_table);
+               VOP_WIN_SET_EXT(vop, win, csc, y2r_en, !!y2r_table);
+               VOP_WIN_SET_EXT(vop, win, csc, r2r_en, !!r2r_table);
+               VOP_WIN_SET_EXT(vop, win, csc, r2y_en, !!r2y_table);
+       }
        VOP_WIN_SET(vop, win, enable, 1);
        spin_unlock(&vop->reg_lock);
+       vop->is_iommu_needed = true;
 }
 
 static const struct drm_plane_helper_funcs plane_helper_funcs = {
@@ -940,11 +1275,12 @@ static int vop_crtc_enable_vblank(struct drm_crtc *crtc)
        struct vop *vop = to_vop(crtc);
        unsigned long flags;
 
-       if (WARN_ON(!vop->is_enabled))
+       if (!vop->is_enabled)
                return -EPERM;
 
        spin_lock_irqsave(&vop->irq_lock, flags);
 
+       VOP_INTR_SET_TYPE(vop, clear, FS_INTR, 1);
        VOP_INTR_SET_TYPE(vop, enable, FS_INTR, 1);
 
        spin_unlock_irqrestore(&vop->irq_lock, flags);
@@ -957,7 +1293,7 @@ static void vop_crtc_disable_vblank(struct drm_crtc *crtc)
        struct vop *vop = to_vop(crtc);
        unsigned long flags;
 
-       if (WARN_ON(!vop->is_enabled))
+       if (!vop->is_enabled)
                return;
 
        spin_lock_irqsave(&vop->irq_lock, flags);
@@ -994,83 +1330,184 @@ static void vop_crtc_cancel_pending_vblank(struct drm_crtc *crtc,
        spin_unlock_irqrestore(&drm->event_lock, flags);
 }
 
-static const struct rockchip_crtc_funcs private_crtc_funcs = {
-       .enable_vblank = vop_crtc_enable_vblank,
-       .disable_vblank = vop_crtc_disable_vblank,
-       .wait_for_update = vop_crtc_wait_for_update,
-       .cancel_pending_vblank = vop_crtc_cancel_pending_vblank,
-};
-
-static bool vop_crtc_mode_fixup(struct drm_crtc *crtc,
-                               const struct drm_display_mode *mode,
-                               struct drm_display_mode *adjusted_mode)
+static int vop_crtc_loader_protect(struct drm_crtc *crtc, bool on)
 {
        struct vop *vop = to_vop(crtc);
 
-       adjusted_mode->clock =
-               clk_round_rate(vop->dclk, mode->clock * 1000) / 1000;
+       if (on == vop->loader_protect)
+               return 0;
 
-       return true;
+       if (on) {
+               vop_power_enable(crtc);
+               enable_irq(vop->irq);
+               drm_crtc_vblank_on(crtc);
+               vop->loader_protect = true;
+       } else {
+               vop_crtc_disable(crtc);
+
+               vop->loader_protect = false;
+       }
+
+       return 0;
 }
 
-static void vop_crtc_enable(struct drm_crtc *crtc)
+static int vop_plane_info_dump(struct seq_file *s, struct drm_plane *plane)
 {
-       struct vop *vop = to_vop(crtc);
-       const struct vop_data *vop_data = vop->data;
-       struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc->state);
-       struct drm_display_mode *adjusted_mode = &crtc->state->adjusted_mode;
-       u16 hsync_len = adjusted_mode->hsync_end - adjusted_mode->hsync_start;
-       u16 hdisplay = adjusted_mode->hdisplay;
-       u16 htotal = adjusted_mode->htotal;
-       u16 hact_st = adjusted_mode->htotal - adjusted_mode->hsync_start;
-       u16 hact_end = hact_st + hdisplay;
-       u16 vdisplay = adjusted_mode->vdisplay;
-       u16 vtotal = adjusted_mode->vtotal;
-       u16 vsync_len = adjusted_mode->vsync_end - adjusted_mode->vsync_start;
-       u16 vact_st = adjusted_mode->vtotal - adjusted_mode->vsync_start;
-       u16 vact_end = vact_st + vdisplay;
-       uint32_t val;
+       struct vop_win *win = to_vop_win(plane);
+       struct drm_plane_state *state = plane->state;
+       struct vop_plane_state *pstate = to_vop_plane_state(state);
+       struct drm_rect *src, *dest;
+       struct drm_framebuffer *fb = state->fb;
+       int i;
+
+       seq_printf(s, "    win%d-%d: %s\n", win->win_id, win->area_id,
+                  pstate->enable ? "ACTIVE" : "DISABLED");
+       if (!fb)
+               return 0;
+
+       src = &pstate->src;
+       dest = &pstate->dest;
+
+       seq_printf(s, "\tformat: %s%s\n", drm_get_format_name(fb->pixel_format),
+                  fb->modifier[0] == DRM_FORMAT_MOD_ARM_AFBC ? "[AFBC]" : "");
+       seq_printf(s, "\tzpos: %d\n", pstate->zpos);
+       seq_printf(s, "\tsrc: pos[%dx%d] rect[%dx%d]\n", src->x1 >> 16,
+                  src->y1 >> 16, drm_rect_width(src) >> 16,
+                  drm_rect_height(src) >> 16);
+       seq_printf(s, "\tdst: pos[%dx%d] rect[%dx%d]\n", dest->x1, dest->y1,
+                  drm_rect_width(dest), drm_rect_height(dest));
+
+       for (i = 0; i < drm_format_num_planes(fb->pixel_format); i++) {
+               dma_addr_t fb_addr = rockchip_fb_get_dma_addr(fb, i);
+               seq_printf(s, "\tbuf[%d]: addr: %pad pitch: %d offset: %d\n",
+                          i, &fb_addr, fb->pitches[i], fb->offsets[i]);
+       }
+
+       return 0;
+}
+
+static int vop_crtc_debugfs_dump(struct drm_crtc *crtc, struct seq_file *s)
+{
+       struct vop *vop = to_vop(crtc);
+       struct drm_crtc_state *crtc_state = crtc->state;
+       struct drm_display_mode *mode = &crtc->state->adjusted_mode;
+       struct rockchip_crtc_state *state = to_rockchip_crtc_state(crtc->state);
+       bool interlaced = !!(mode->flags & DRM_MODE_FLAG_INTERLACE);
+       struct drm_plane *plane;
+       int i;
+
+       seq_printf(s, "VOP [%s]: %s\n", dev_name(vop->dev),
+                  crtc_state->active ? "ACTIVE" : "DISABLED");
+
+       if (!crtc_state->active)
+               return 0;
+
+       seq_printf(s, "    Connector: %s\n",
+                  drm_get_connector_name(state->output_type));
+       seq_printf(s, "\tbus_format[%x] output_mode[%x]\n",
+                  state->bus_format, state->output_mode);
+       seq_printf(s, "    Display mode: %dx%d%s%d\n",
+                  mode->hdisplay, mode->vdisplay, interlaced ? "i" : "p",
+                  drm_mode_vrefresh(mode));
+       seq_printf(s, "\tclk[%d] real_clk[%d] type[%x] flag[%x]\n",
+                  mode->clock, mode->crtc_clock, mode->type, mode->flags);
+       seq_printf(s, "\tH: %d %d %d %d\n", mode->hdisplay, mode->hsync_start,
+                  mode->hsync_end, mode->htotal);
+       seq_printf(s, "\tV: %d %d %d %d\n", mode->vdisplay, mode->vsync_start,
+                  mode->vsync_end, mode->vtotal);
+
+       for (i = 0; i < vop->num_wins; i++) {
+               plane = &vop->win[i].base;
+               vop_plane_info_dump(s, plane);
+       }
+
+       return 0;
+}
+
+static enum drm_mode_status
+vop_crtc_mode_valid(struct drm_crtc *crtc, const struct drm_display_mode *mode,
+                   int output_type)
+{
+       struct vop *vop = to_vop(crtc);
+       const struct vop_data *vop_data = vop->data;
+       int request_clock = mode->clock;
+       int clock;
+
+       if (mode->hdisplay > vop_data->max_output.width)
+               return MODE_BAD_HVALUE;
+       if (mode->vdisplay > vop_data->max_output.height)
+               return MODE_BAD_VVALUE;
+
+       if (mode->flags & DRM_MODE_FLAG_DBLCLK)
+               request_clock *= 2;
+       clock = clk_round_rate(vop->dclk, request_clock * 1000) / 1000;
 
-       vop_enable(crtc);
        /*
-        * If dclk rate is zero, mean that scanout is stop,
-        * we don't need wait any more.
+        * Hdmi or DisplayPort request a Accurate clock.
         */
-       if (clk_get_rate(vop->dclk)) {
-               /*
-                * Rk3288 vop timing register is immediately, when configure
-                * display timing on display time, may cause tearing.
-                *
-                * Vop standby will take effect at end of current frame,
-                * if dsp hold valid irq happen, it means standby complete.
-                *
-                * mode set:
-                *    standby and wait complete --> |----
-                *                                  | display time
-                *                                  |----
-                *                                  |---> dsp hold irq
-                *     configure display timing --> |
-                *         standby exit             |
-                *                                  | new frame start.
-                */
+       if (output_type == DRM_MODE_CONNECTOR_HDMIA ||
+           output_type == DRM_MODE_CONNECTOR_DisplayPort)
+               if (clock != request_clock)
+                       return MODE_CLOCK_RANGE;
 
-               reinit_completion(&vop->dsp_hold_completion);
-               vop_dsp_hold_valid_irq_enable(vop);
+       return MODE_OK;
+}
 
-               spin_lock(&vop->reg_lock);
+static const struct rockchip_crtc_funcs private_crtc_funcs = {
+       .loader_protect = vop_crtc_loader_protect,
+       .enable_vblank = vop_crtc_enable_vblank,
+       .disable_vblank = vop_crtc_disable_vblank,
+       .wait_for_update = vop_crtc_wait_for_update,
+       .cancel_pending_vblank = vop_crtc_cancel_pending_vblank,
+       .debugfs_dump = vop_crtc_debugfs_dump,
+       .mode_valid = vop_crtc_mode_valid,
+};
 
-               VOP_CTRL_SET(vop, standby, 1);
+static bool vop_crtc_mode_fixup(struct drm_crtc *crtc,
+                               const struct drm_display_mode *mode,
+                               struct drm_display_mode *adjusted_mode)
+{
+       struct vop *vop = to_vop(crtc);
+       const struct vop_data *vop_data = vop->data;
+       int request_clock = mode->clock;
 
-               spin_unlock(&vop->reg_lock);
+       if (mode->hdisplay > vop_data->max_output.width ||
+           mode->vdisplay > vop_data->max_output.height)
+               return false;
 
-               wait_for_completion(&vop->dsp_hold_completion);
+       if (mode->flags & DRM_MODE_FLAG_DBLCLK)
+               request_clock *= 2;
+       adjusted_mode->crtc_clock =
+               clk_round_rate(vop->dclk, request_clock * 1000) / 1000;
 
-               vop_dsp_hold_valid_irq_disable(vop);
-       }
+       return true;
+}
+
+static void vop_crtc_enable(struct drm_crtc *crtc)
+{
+       struct vop *vop = to_vop(crtc);
+       const struct vop_data *vop_data = vop->data;
+       struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc->state);
+       struct drm_display_mode *adjusted_mode = &crtc->state->adjusted_mode;
+       u16 hsync_len = adjusted_mode->crtc_hsync_end - adjusted_mode->crtc_hsync_start;
+       u16 hdisplay = adjusted_mode->crtc_hdisplay;
+       u16 htotal = adjusted_mode->crtc_htotal;
+       u16 hact_st = adjusted_mode->crtc_htotal - adjusted_mode->crtc_hsync_start;
+       u16 hact_end = hact_st + hdisplay;
+       u16 vdisplay = adjusted_mode->crtc_vdisplay;
+       u16 vtotal = adjusted_mode->crtc_vtotal;
+       u16 vsync_len = adjusted_mode->crtc_vsync_end - adjusted_mode->crtc_vsync_start;
+       u16 vact_st = adjusted_mode->crtc_vtotal - adjusted_mode->crtc_vsync_start;
+       u16 vact_end = vact_st + vdisplay;
+       uint32_t val;
+
+       vop_initial(crtc);
 
-       val = 0x8;
-       val |= (adjusted_mode->flags & DRM_MODE_FLAG_NHSYNC) ? 0 : 1;
-       val |= (adjusted_mode->flags & DRM_MODE_FLAG_NVSYNC) ? 0 : (1 << 1);
+       val = BIT(DCLK_INVERT);
+       val |= (adjusted_mode->flags & DRM_MODE_FLAG_NHSYNC) ?
+                  0 : BIT(HSYNC_POSITIVE);
+       val |= (adjusted_mode->flags & DRM_MODE_FLAG_NVSYNC) ?
+                  0 : BIT(VSYNC_POSITIVE);
        VOP_CTRL_SET(vop, pin_pol, val);
        switch (s->output_type) {
        case DRM_MODE_CONNECTOR_LVDS:
@@ -1089,6 +1526,11 @@ static void vop_crtc_enable(struct drm_crtc *crtc)
                VOP_CTRL_SET(vop, mipi_en, 1);
                VOP_CTRL_SET(vop, mipi_pin_pol, val);
                break;
+       case DRM_MODE_CONNECTOR_DisplayPort:
+               val &= ~BIT(DCLK_INVERT);
+               VOP_CTRL_SET(vop, dp_pin_pol, val);
+               VOP_CTRL_SET(vop, dp_en, 1);
+               break;
        default:
                DRM_ERROR("unsupport connector_type[%d]\n", s->output_type);
        }
@@ -1098,6 +1540,38 @@ static void vop_crtc_enable(struct drm_crtc *crtc)
                s->output_mode = ROCKCHIP_OUT_MODE_P888;
 
        VOP_CTRL_SET(vop, out_mode, s->output_mode);
+       switch (s->bus_format) {
+       case MEDIA_BUS_FMT_RGB565_1X16:
+               val = DITHER_DOWN_EN(1) | DITHER_DOWN_MODE(RGB888_TO_RGB565);
+               break;
+       case MEDIA_BUS_FMT_RGB666_1X18:
+       case MEDIA_BUS_FMT_RGB666_1X24_CPADHI:
+               val = DITHER_DOWN_EN(1) | DITHER_DOWN_MODE(RGB888_TO_RGB666);
+               break;
+       case MEDIA_BUS_FMT_YUV8_1X24:
+               val = DITHER_DOWN_EN(0) | PRE_DITHER_DOWN_EN(1);
+               break;
+       case MEDIA_BUS_FMT_YUV10_1X30:
+               val = DITHER_DOWN_EN(0) | PRE_DITHER_DOWN_EN(0);
+               break;
+       case MEDIA_BUS_FMT_RGB888_1X24:
+       default:
+               val = DITHER_DOWN_EN(0) | PRE_DITHER_DOWN_EN(0);
+               break;
+       }
+
+       if (s->output_mode == ROCKCHIP_OUT_MODE_AAAA)
+               val |= PRE_DITHER_DOWN_EN(0);
+       else
+               val |= PRE_DITHER_DOWN_EN(1);
+       val |= DITHER_DOWN_MODE_SEL(DITHER_DOWN_ALLEGRO);
+       VOP_CTRL_SET(vop, dither_down, val);
+       VOP_CTRL_SET(vop, dclk_ddr,
+                    s->output_mode == ROCKCHIP_OUT_MODE_YUV420 ? 1 : 0);
+       VOP_CTRL_SET(vop, overlay_mode, is_yuv_output(s->bus_format));
+       VOP_CTRL_SET(vop, dsp_out_yuv, is_yuv_output(s->bus_format));
+       VOP_CTRL_SET(vop, dsp_background,
+                    is_yuv_output(s->bus_format) ? 0x20010200 : 0);
 
        VOP_CTRL_SET(vop, htotal_pw, (htotal << 16) | hsync_len);
        val = hact_st << 16;
@@ -1105,15 +1579,42 @@ static void vop_crtc_enable(struct drm_crtc *crtc)
        VOP_CTRL_SET(vop, hact_st_end, val);
        VOP_CTRL_SET(vop, hpost_st_end, val);
 
-       VOP_CTRL_SET(vop, vtotal_pw, (vtotal << 16) | vsync_len);
        val = vact_st << 16;
        val |= vact_end;
        VOP_CTRL_SET(vop, vact_st_end, val);
        VOP_CTRL_SET(vop, vpost_st_end, val);
+       if (adjusted_mode->flags & DRM_MODE_FLAG_INTERLACE) {
+               u16 vact_st_f1 = vtotal + vact_st + 1;
+               u16 vact_end_f1 = vact_st_f1 + vdisplay;
+
+               val = vact_st_f1 << 16 | vact_end_f1;
+               VOP_CTRL_SET(vop, vact_st_end_f1, val);
+               VOP_CTRL_SET(vop, vpost_st_end_f1, val);
+
+               val = vtotal << 16 | (vtotal + vsync_len);
+               VOP_CTRL_SET(vop, vs_st_end_f1, val);
+               VOP_CTRL_SET(vop, dsp_interlace, 1);
+               VOP_CTRL_SET(vop, p2i_en, 1);
+               vtotal = vtotal + 1;
+       } else {
+               VOP_CTRL_SET(vop, dsp_interlace, 0);
+               VOP_CTRL_SET(vop, p2i_en, 0);
+       }
+       VOP_CTRL_SET(vop, vtotal_pw, vtotal << 16 | vsync_len);
+
+       VOP_CTRL_SET(vop, core_dclk_div,
+                    !!(adjusted_mode->flags & DRM_MODE_FLAG_DBLCLK));
 
-       clk_set_rate(vop->dclk, adjusted_mode->clock * 1000);
+       clk_set_rate(vop->dclk, adjusted_mode->crtc_clock * 1000);
 
+       vop_cfg_done(vop);
+       /*
+        * enable vop, all the register would take effect when vop exit standby
+        */
        VOP_CTRL_SET(vop, standby, 0);
+
+       enable_irq(vop->irq);
+       drm_crtc_vblank_on(crtc);
 }
 
 static int vop_zpos_cmp(const void *a, const void *b)
@@ -1124,6 +1625,78 @@ static int vop_zpos_cmp(const void *a, const void *b)
        return pa->zpos - pb->zpos;
 }
 
+static int vop_afbdc_atomic_check(struct drm_crtc *crtc,
+                                 struct drm_crtc_state *crtc_state)
+{
+       struct vop *vop = to_vop(crtc);
+       const struct vop_data *vop_data = vop->data;
+       struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc_state);
+       struct drm_atomic_state *state = crtc_state->state;
+       struct drm_plane *plane;
+       struct drm_plane_state *pstate;
+       struct vop_plane_state *plane_state;
+       struct vop_win *win;
+       int afbdc_format;
+       int i;
+
+       s->afbdc_en = 0;
+
+       for_each_plane_in_state(state, plane, pstate, i) {
+               struct drm_framebuffer *fb = pstate->fb;
+               struct drm_rect *src;
+
+               win = to_vop_win(plane);
+               plane_state = to_vop_plane_state(pstate);
+
+               if (pstate->crtc != crtc || !fb)
+                       continue;
+
+               if (fb->modifier[0] != DRM_FORMAT_MOD_ARM_AFBC)
+                       continue;
+
+               if (!(vop_data->feature & VOP_FEATURE_AFBDC)) {
+                       DRM_ERROR("not support afbdc\n");
+                       return -EINVAL;
+               }
+
+               switch (plane_state->format) {
+               case VOP_FMT_ARGB8888:
+                       afbdc_format = AFBDC_FMT_U8U8U8U8;
+                       break;
+               case VOP_FMT_RGB888:
+                       afbdc_format = AFBDC_FMT_U8U8U8;
+                       break;
+               case VOP_FMT_RGB565:
+                       afbdc_format = AFBDC_FMT_RGB565;
+                       break;
+               default:
+                       return -EINVAL;
+               }
+
+               if (s->afbdc_en) {
+                       DRM_ERROR("vop only support one afbc layer\n");
+                       return -EINVAL;
+               }
+
+               src = &plane_state->src;
+               if (src->x1 || src->y1 || fb->offsets[0]) {
+                       DRM_ERROR("win[%d] afbdc not support offset display\n",
+                                 win->win_id);
+                       DRM_ERROR("xpos=%d, ypos=%d, offset=%d\n",
+                                 src->x1, src->y1, fb->offsets[0]);
+                       return -EINVAL;
+               }
+               s->afbdc_win_format = afbdc_format;
+               s->afbdc_win_width = pstate->fb->width - 1;
+               s->afbdc_win_height = (drm_rect_height(src) >> 16) - 1;
+               s->afbdc_win_id = win->win_id;
+               s->afbdc_win_ptr = plane_state->yrgb_mst;
+               s->afbdc_en = 1;
+       }
+
+       return 0;
+}
+
 static int vop_crtc_atomic_check(struct drm_crtc *crtc,
                                 struct drm_crtc_state *crtc_state)
 {
@@ -1138,6 +1711,14 @@ static int vop_crtc_atomic_check(struct drm_crtc *crtc,
        int dsp_layer_sel = 0;
        int i, j, cnt = 0, ret = 0;
 
+       ret = vop_afbdc_atomic_check(crtc, crtc_state);
+       if (ret)
+               return ret;
+
+       ret = vop_csc_atomic_check(crtc, crtc_state);
+       if (ret)
+               return ret;
+
        pzpos = kmalloc_array(vop_data->win_size, sizeof(*pzpos), GFP_KERNEL);
        if (!pzpos)
                return -ENOMEM;
@@ -1195,24 +1776,123 @@ err_free_pzpos:
        return ret;
 }
 
-static void vop_crtc_atomic_flush(struct drm_crtc *crtc,
-                                 struct drm_crtc_state *old_crtc_state)
+static void vop_post_config(struct drm_crtc *crtc)
+{
+       struct vop *vop = to_vop(crtc);
+       struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc->state);
+       struct drm_display_mode *mode = &crtc->state->adjusted_mode;
+       u16 vtotal = mode->crtc_vtotal;
+       u16 hdisplay = mode->crtc_hdisplay;
+       u16 hact_st = mode->crtc_htotal - mode->crtc_hsync_start;
+       u16 vdisplay = mode->crtc_vdisplay;
+       u16 vact_st = mode->crtc_vtotal - mode->crtc_vsync_start;
+       u16 hsize = hdisplay * (s->left_margin + s->right_margin) / 200;
+       u16 vsize = vdisplay * (s->top_margin + s->bottom_margin) / 200;
+       u16 hact_end, vact_end;
+       u32 val;
+
+       hact_st += hdisplay * (100 - s->left_margin) / 200;
+       hact_end = hact_st + hsize;
+       val = hact_st << 16;
+       val |= hact_end;
+       VOP_CTRL_SET(vop, hpost_st_end, val);
+       vact_st += vdisplay * (100 - s->top_margin) / 200;
+       vact_end = vact_st + vsize;
+       val = vact_st << 16;
+       val |= vact_end;
+       VOP_CTRL_SET(vop, vpost_st_end, val);
+       val = scl_cal_scale2(vdisplay, vsize) << 16;
+       val |= scl_cal_scale2(hdisplay, hsize);
+       VOP_CTRL_SET(vop, post_scl_factor, val);
+       VOP_CTRL_SET(vop, post_scl_ctrl, 0x3);
+       if (mode->flags & DRM_MODE_FLAG_INTERLACE) {
+               u16 vact_st_f1 = vtotal + vact_st + 1;
+               u16 vact_end_f1 = vact_st_f1 + vsize;
+
+               val = vact_st_f1 << 16 | vact_end_f1;
+               VOP_CTRL_SET(vop, vpost_st_end_f1, val);
+       }
+}
+
+static void vop_cfg_update(struct drm_crtc *crtc,
+                          struct drm_crtc_state *old_crtc_state)
 {
        struct rockchip_crtc_state *s =
                        to_rockchip_crtc_state(crtc->state);
        struct vop *vop = to_vop(crtc);
 
-       if (WARN_ON(!vop->is_enabled))
-               return;
-
        spin_lock(&vop->reg_lock);
 
+       if (s->afbdc_en) {
+               uint32_t pic_size;
+
+               VOP_CTRL_SET(vop, afbdc_format, s->afbdc_win_format | 1 << 4);
+               VOP_CTRL_SET(vop, afbdc_hreg_block_split, 0);
+               VOP_CTRL_SET(vop, afbdc_sel, s->afbdc_win_id);
+               VOP_CTRL_SET(vop, afbdc_hdr_ptr, s->afbdc_win_ptr);
+               pic_size = (s->afbdc_win_width & 0xffff);
+               pic_size |= s->afbdc_win_height << 16;
+               VOP_CTRL_SET(vop, afbdc_pic_size, pic_size);
+       }
+
+       VOP_CTRL_SET(vop, afbdc_en, s->afbdc_en);
        VOP_CTRL_SET(vop, dsp_layer_sel, s->dsp_layer_sel);
-       vop_cfg_done(vop);
+       vop_post_config(crtc);
 
        spin_unlock(&vop->reg_lock);
 }
 
+static void vop_crtc_atomic_flush(struct drm_crtc *crtc,
+                                 struct drm_crtc_state *old_crtc_state)
+{
+       struct vop *vop = to_vop(crtc);
+
+       vop_cfg_update(crtc, old_crtc_state);
+
+       if (!vop->is_iommu_enabled && vop->is_iommu_needed) {
+               bool need_wait_vblank = !vop_is_allwin_disabled(vop);
+               int ret;
+
+               if (need_wait_vblank) {
+                       bool active;
+
+                       disable_irq(vop->irq);
+                       drm_crtc_vblank_get(crtc);
+                       VOP_INTR_SET_TYPE(vop, enable, LINE_FLAG_INTR, 1);
+
+                       ret = readx_poll_timeout_atomic(vop_fs_irq_is_active,
+                                                       vop, active, active,
+                                                       0, 50 * 1000);
+                       if (ret)
+                               dev_err(vop->dev, "wait fs irq timeout\n");
+
+                       VOP_INTR_SET_TYPE(vop, clear, LINE_FLAG_INTR, 1);
+                       vop_cfg_done(vop);
+
+                       ret = readx_poll_timeout_atomic(vop_line_flag_is_active,
+                                                       vop, active, active,
+                                                       0, 50 * 1000);
+                       if (ret)
+                               dev_err(vop->dev, "wait line flag timeout\n");
+
+                       enable_irq(vop->irq);
+               }
+               ret = rockchip_drm_dma_attach_device(vop->drm_dev, vop->dev);
+               if (ret)
+                       dev_err(vop->dev, "failed to attach dma mapping, %d\n",
+                               ret);
+
+               if (need_wait_vblank) {
+                       VOP_INTR_SET_TYPE(vop, enable, LINE_FLAG_INTR, 0);
+                       drm_crtc_vblank_put(crtc);
+               }
+
+               vop->is_iommu_enabled = true;
+       }
+
+       vop_cfg_done(vop);
+}
+
 static void vop_crtc_atomic_begin(struct drm_crtc *crtc,
                                  struct drm_crtc_state *old_crtc_state)
 {
@@ -1240,11 +1920,32 @@ static void vop_crtc_destroy(struct drm_crtc *crtc)
        drm_crtc_cleanup(crtc);
 }
 
+static void vop_crtc_reset(struct drm_crtc *crtc)
+{
+       struct rockchip_crtc_state *s = to_rockchip_crtc_state(crtc->state);
+
+       if (crtc->state) {
+               __drm_atomic_helper_crtc_destroy_state(crtc, crtc->state);
+               kfree(s);
+       }
+
+       s = kzalloc(sizeof(*s), GFP_KERNEL);
+       if (!s)
+               return;
+       crtc->state = &s->base;
+       crtc->state->crtc = crtc;
+       s->left_margin = 100;
+       s->right_margin = 100;
+       s->top_margin = 100;
+       s->bottom_margin = 100;
+}
+
 static struct drm_crtc_state *vop_crtc_duplicate_state(struct drm_crtc *crtc)
 {
-       struct rockchip_crtc_state *rockchip_state;
+       struct rockchip_crtc_state *rockchip_state, *old_state;
 
-       rockchip_state = kzalloc(sizeof(*rockchip_state), GFP_KERNEL);
+       old_state = to_rockchip_crtc_state(crtc->state);
+       rockchip_state = kmemdup(old_state, sizeof(*old_state), GFP_KERNEL);
        if (!rockchip_state)
                return NULL;
 
@@ -1261,40 +1962,91 @@ static void vop_crtc_destroy_state(struct drm_crtc *crtc,
        kfree(s);
 }
 
+static int vop_crtc_atomic_get_property(struct drm_crtc *crtc,
+                                       const struct drm_crtc_state *state,
+                                       struct drm_property *property,
+                                       uint64_t *val)
+{
+       struct drm_device *drm_dev = crtc->dev;
+       struct drm_mode_config *mode_config = &drm_dev->mode_config;
+       struct rockchip_crtc_state *s = to_rockchip_crtc_state(state);
+
+       if (property == mode_config->tv_left_margin_property) {
+               *val = s->left_margin;
+               return 0;
+       }
+
+       if (property == mode_config->tv_right_margin_property) {
+               *val = s->right_margin;
+               return 0;
+       }
+
+       if (property == mode_config->tv_top_margin_property) {
+               *val = s->top_margin;
+               return 0;
+       }
+
+       if (property == mode_config->tv_bottom_margin_property) {
+               *val = s->bottom_margin;
+               return 0;
+       }
+
+       DRM_ERROR("failed to get vop crtc property\n");
+       return -EINVAL;
+}
+
+static int vop_crtc_atomic_set_property(struct drm_crtc *crtc,
+                                       struct drm_crtc_state *state,
+                                       struct drm_property *property,
+                                       uint64_t val)
+{
+       struct drm_device *drm_dev = crtc->dev;
+       struct drm_mode_config *mode_config = &drm_dev->mode_config;
+       struct rockchip_crtc_state *s = to_rockchip_crtc_state(state);
+
+       if (property == mode_config->tv_left_margin_property) {
+               s->left_margin = val;
+               return 0;
+       }
+
+       if (property == mode_config->tv_right_margin_property) {
+               s->right_margin = val;
+               return 0;
+       }
+
+       if (property == mode_config->tv_top_margin_property) {
+               s->top_margin = val;
+               return 0;
+       }
+
+       if (property == mode_config->tv_bottom_margin_property) {
+               s->bottom_margin = val;
+               return 0;
+       }
+
+       DRM_ERROR("failed to set vop crtc property\n");
+       return -EINVAL;
+}
+
 static const struct drm_crtc_funcs vop_crtc_funcs = {
        .set_config = drm_atomic_helper_set_config,
        .page_flip = drm_atomic_helper_page_flip,
        .destroy = vop_crtc_destroy,
-       .reset = drm_atomic_helper_crtc_reset,
+       .reset = vop_crtc_reset,
+       .atomic_get_property = vop_crtc_atomic_get_property,
+       .atomic_set_property = vop_crtc_atomic_set_property,
        .atomic_duplicate_state = vop_crtc_duplicate_state,
        .atomic_destroy_state = vop_crtc_destroy_state,
 };
 
-static bool vop_win_pending_is_complete(struct vop_win *vop_win)
-{
-       struct drm_plane *plane = &vop_win->base;
-       struct vop_plane_state *state = to_vop_plane_state(plane->state);
-       dma_addr_t yrgb_mst;
-
-       if (!state->enable)
-               return VOP_WIN_GET(vop_win->vop, vop_win, enable) == 0;
-
-       yrgb_mst = VOP_WIN_GET_YRGBADDR(vop_win->vop, vop_win);
-
-       return yrgb_mst == state->yrgb_mst;
-}
-
 static void vop_handle_vblank(struct vop *vop)
 {
        struct drm_device *drm = vop->drm_dev;
        struct drm_crtc *crtc = &vop->crtc;
        unsigned long flags;
-       int i;
 
-       for (i = 0; i < vop->num_wins; i++) {
-               if (!vop_win_pending_is_complete(&vop->win[i]))
-                       return;
-       }
+       if (!vop_is_cfg_done_complete(vop))
+               return;
 
        if (vop->event) {
                spin_lock_irqsave(&drm->event_lock, flags);
@@ -1340,6 +2092,12 @@ static irqreturn_t vop_isr(int irq, void *data)
                ret = IRQ_HANDLED;
        }
 
+       if (active_irqs & LINE_FLAG_INTR) {
+               complete(&vop->line_flag_completion);
+               active_irqs &= ~LINE_FLAG_INTR;
+               ret = IRQ_HANDLED;
+       }
+
        if (active_irqs & FS_INTR) {
                drm_crtc_handle_vblank(crtc);
                vop_handle_vblank(vop);
@@ -1360,6 +2118,7 @@ static int vop_plane_init(struct vop *vop, struct vop_win *win,
        struct drm_plane *share = NULL;
        unsigned int rotations = 0;
        struct drm_property *prop;
+       uint64_t feature = 0;
        int ret;
 
        if (win->parent)
@@ -1376,10 +2135,10 @@ static int vop_plane_init(struct vop *vop, struct vop_win *win,
        drm_object_attach_property(&win->base.base,
                                   vop->plane_zpos_prop, win->win_id);
 
-       if (VOP_WIN_SUPPORT(win, xmirror))
+       if (VOP_WIN_SUPPORT(vop, win, xmirror))
                rotations |= BIT(DRM_REFLECT_X);
 
-       if (VOP_WIN_SUPPORT(win, ymirror))
+       if (VOP_WIN_SUPPORT(vop, win, ymirror))
                rotations |= BIT(DRM_REFLECT_Y);
 
        if (rotations) {
@@ -1394,6 +2153,14 @@ static int vop_plane_init(struct vop *vop, struct vop_win *win,
                                           BIT(DRM_ROTATE_0));
                win->rotation_prop = prop;
        }
+       if (win->phy->scl)
+               feature |= BIT(ROCKCHIP_DRM_PLANE_FEATURE_SCALE);
+       if (VOP_WIN_SUPPORT(vop, win, src_alpha_ctl) ||
+           VOP_WIN_SUPPORT(vop, win, alpha_en))
+               feature |= BIT(ROCKCHIP_DRM_PLANE_FEATURE_ALPHA);
+
+       drm_object_attach_property(&win->base.base, vop->plane_feature_prop,
+                                  feature);
 
        return 0;
 }
@@ -1401,10 +2168,12 @@ static int vop_plane_init(struct vop *vop, struct vop_win *win,
 static int vop_create_crtc(struct vop *vop)
 {
        struct device *dev = vop->dev;
+       const struct vop_data *vop_data = vop->data;
        struct drm_device *drm_dev = vop->drm_dev;
        struct drm_plane *primary = NULL, *cursor = NULL, *plane, *tmp;
        struct drm_crtc *crtc = &vop->crtc;
        struct device_node *port;
+       uint64_t feature = 0;
        int ret;
        int i;
 
@@ -1465,11 +2234,31 @@ static int vop_create_crtc(struct vop *vop)
 
        init_completion(&vop->dsp_hold_completion);
        init_completion(&vop->wait_update_complete);
+       init_completion(&vop->line_flag_completion);
        crtc->port = port;
        rockchip_register_crtc_funcs(crtc, &private_crtc_funcs);
 
+       ret = drm_mode_create_tv_properties(drm_dev, 0, NULL);
+       if (ret)
+               goto err_unregister_crtc_funcs;
+#define VOP_ATTACH_MODE_CONFIG_PROP(prop, v) \
+       drm_object_attach_property(&crtc->base, drm_dev->mode_config.prop, v)
+
+       VOP_ATTACH_MODE_CONFIG_PROP(tv_left_margin_property, 100);
+       VOP_ATTACH_MODE_CONFIG_PROP(tv_right_margin_property, 100);
+       VOP_ATTACH_MODE_CONFIG_PROP(tv_top_margin_property, 100);
+       VOP_ATTACH_MODE_CONFIG_PROP(tv_bottom_margin_property, 100);
+#undef VOP_ATTACH_MODE_CONFIG_PROP
+
+       if (vop_data->feature & VOP_FEATURE_AFBDC)
+               feature |= BIT(ROCKCHIP_DRM_CRTC_FEATURE_AFBDC);
+       drm_object_attach_property(&crtc->base, vop->feature_prop,
+                                  feature);
+
        return 0;
 
+err_unregister_crtc_funcs:
+       rockchip_unregister_crtc_funcs(crtc);
 err_cleanup_crtc:
        drm_crtc_cleanup(crtc);
 err_cleanup_planes:
@@ -1507,103 +2296,6 @@ static void vop_destroy_crtc(struct vop *vop)
        drm_crtc_cleanup(crtc);
 }
 
-static int vop_initial(struct vop *vop)
-{
-       const struct vop_data *vop_data = vop->data;
-       const struct vop_reg_data *init_table = vop_data->init_table;
-       struct reset_control *ahb_rst;
-       int i, ret;
-
-       vop->hclk = devm_clk_get(vop->dev, "hclk_vop");
-       if (IS_ERR(vop->hclk)) {
-               dev_err(vop->dev, "failed to get hclk source\n");
-               return PTR_ERR(vop->hclk);
-       }
-       vop->aclk = devm_clk_get(vop->dev, "aclk_vop");
-       if (IS_ERR(vop->aclk)) {
-               dev_err(vop->dev, "failed to get aclk source\n");
-               return PTR_ERR(vop->aclk);
-       }
-       vop->dclk = devm_clk_get(vop->dev, "dclk_vop");
-       if (IS_ERR(vop->dclk)) {
-               dev_err(vop->dev, "failed to get dclk source\n");
-               return PTR_ERR(vop->dclk);
-       }
-
-       ret = clk_prepare(vop->dclk);
-       if (ret < 0) {
-               dev_err(vop->dev, "failed to prepare dclk\n");
-               return ret;
-       }
-
-       /* Enable both the hclk and aclk to setup the vop */
-       ret = clk_prepare_enable(vop->hclk);
-       if (ret < 0) {
-               dev_err(vop->dev, "failed to prepare/enable hclk\n");
-               goto err_unprepare_dclk;
-       }
-
-       ret = clk_prepare_enable(vop->aclk);
-       if (ret < 0) {
-               dev_err(vop->dev, "failed to prepare/enable aclk\n");
-               goto err_disable_hclk;
-       }
-
-       /*
-        * do hclk_reset, reset all vop registers.
-        */
-       ahb_rst = devm_reset_control_get(vop->dev, "ahb");
-       if (IS_ERR(ahb_rst)) {
-               dev_err(vop->dev, "failed to get ahb reset\n");
-               ret = PTR_ERR(ahb_rst);
-               goto err_disable_aclk;
-       }
-       reset_control_assert(ahb_rst);
-       usleep_range(10, 20);
-       reset_control_deassert(ahb_rst);
-
-       memcpy(vop->regsbak, vop->regs, vop->len);
-
-       for (i = 0; i < vop_data->table_size; i++)
-               vop_writel(vop, init_table[i].offset, init_table[i].value);
-
-       for (i = 0; i < vop->num_wins; i++) {
-               struct vop_win *win = &vop->win[i];
-
-               VOP_WIN_SET(vop, win, enable, 0);
-       }
-
-       vop_cfg_done(vop);
-
-       /*
-        * do dclk_reset, let all config take affect.
-        */
-       vop->dclk_rst = devm_reset_control_get(vop->dev, "dclk");
-       if (IS_ERR(vop->dclk_rst)) {
-               dev_err(vop->dev, "failed to get dclk reset\n");
-               ret = PTR_ERR(vop->dclk_rst);
-               goto err_disable_aclk;
-       }
-       reset_control_assert(vop->dclk_rst);
-       usleep_range(10, 20);
-       reset_control_deassert(vop->dclk_rst);
-
-       clk_disable(vop->hclk);
-       clk_disable(vop->aclk);
-
-       vop->is_enabled = false;
-
-       return 0;
-
-err_disable_aclk:
-       clk_disable_unprepare(vop->aclk);
-err_disable_hclk:
-       clk_disable_unprepare(vop->hclk);
-err_unprepare_dclk:
-       clk_unprepare(vop->dclk);
-       return ret;
-}
-
 /*
  * Initialize the vop->win array elements.
  */
@@ -1613,6 +2305,13 @@ static int vop_win_init(struct vop *vop)
        unsigned int i, j;
        unsigned int num_wins = 0;
        struct drm_property *prop;
+       static const struct drm_prop_enum_list props[] = {
+               { ROCKCHIP_DRM_PLANE_FEATURE_SCALE, "scale" },
+               { ROCKCHIP_DRM_PLANE_FEATURE_ALPHA, "alpha" },
+       };
+       static const struct drm_prop_enum_list crtc_props[] = {
+               { ROCKCHIP_DRM_CRTC_FEATURE_AFBDC, "afbdc" },
+       };
 
        for (i = 0; i < vop_data->win_size; i++) {
                struct vop_win *vop_win = &vop->win[num_wins];
@@ -1622,6 +2321,7 @@ static int vop_win_init(struct vop *vop)
                        continue;
 
                vop_win->phy = win_data->phy;
+               vop_win->csc = win_data->csc;
                vop_win->offset = win_data->base;
                vop_win->type = win_data->type;
                vop_win->data_formats = win_data->phy->data_formats;
@@ -1658,8 +2358,70 @@ static int vop_win_init(struct vop *vop)
        }
        vop->plane_zpos_prop = prop;
 
+       vop->plane_feature_prop = drm_property_create_bitmask(vop->drm_dev,
+                               DRM_MODE_PROP_IMMUTABLE, "FEATURE",
+                               props, ARRAY_SIZE(props),
+                               BIT(ROCKCHIP_DRM_PLANE_FEATURE_SCALE) |
+                               BIT(ROCKCHIP_DRM_PLANE_FEATURE_ALPHA));
+       if (!vop->plane_feature_prop) {
+               DRM_ERROR("failed to create feature property\n");
+               return -EINVAL;
+       }
+
+       vop->feature_prop = drm_property_create_bitmask(vop->drm_dev,
+                               DRM_MODE_PROP_IMMUTABLE, "FEATURE",
+                               crtc_props, ARRAY_SIZE(crtc_props),
+                               BIT(ROCKCHIP_DRM_CRTC_FEATURE_AFBDC));
+       if (!vop->feature_prop) {
+               DRM_ERROR("failed to create vop feature property\n");
+               return -EINVAL;
+       }
+
+       return 0;
+}
+
+/**
+ * rockchip_drm_wait_line_flag - acqiure the give line flag event
+ * @crtc: CRTC to enable line flag
+ * @line_num: interested line number
+ * @mstimeout: millisecond for timeout
+ *
+ * Driver would hold here until the interested line flag interrupt have
+ * happened or timeout to wait.
+ *
+ * Returns:
+ * Zero on success, negative errno on failure.
+ */
+int rockchip_drm_wait_line_flag(struct drm_crtc *crtc, unsigned int line_num,
+                               unsigned int mstimeout)
+{
+       struct vop *vop = to_vop(crtc);
+       unsigned long jiffies_left;
+
+       if (!crtc || !vop->is_enabled)
+               return -ENODEV;
+
+       if (line_num > crtc->mode.vtotal || mstimeout <= 0)
+               return -EINVAL;
+
+       if (vop_line_flag_irq_is_enabled(vop))
+               return -EBUSY;
+
+       reinit_completion(&vop->line_flag_completion);
+       vop_line_flag_irq_enable(vop, line_num);
+
+       jiffies_left = wait_for_completion_timeout(&vop->line_flag_completion,
+                                                  msecs_to_jiffies(mstimeout));
+       vop_line_flag_irq_disable(vop);
+
+       if (jiffies_left == 0) {
+               dev_err(vop->dev, "Timeout waiting for IRQ\n");
+               return -ETIMEDOUT;
+       }
+
        return 0;
 }
+EXPORT_SYMBOL(rockchip_drm_wait_line_flag);
 
 static int vop_bind(struct device *dev, struct device *master, void *data)
 {
@@ -1708,10 +2470,20 @@ static int vop_bind(struct device *dev, struct device *master, void *data)
        if (!vop->regsbak)
                return -ENOMEM;
 
-       ret = vop_initial(vop);
-       if (ret < 0) {
-               dev_err(&pdev->dev, "cannot initial vop dev - err %d\n", ret);
-               return ret;
+       vop->hclk = devm_clk_get(vop->dev, "hclk_vop");
+       if (IS_ERR(vop->hclk)) {
+               dev_err(vop->dev, "failed to get hclk source\n");
+               return PTR_ERR(vop->hclk);
+       }
+       vop->aclk = devm_clk_get(vop->dev, "aclk_vop");
+       if (IS_ERR(vop->aclk)) {
+               dev_err(vop->dev, "failed to get aclk source\n");
+               return PTR_ERR(vop->aclk);
+       }
+       vop->dclk = devm_clk_get(vop->dev, "dclk_vop");
+       if (IS_ERR(vop->dclk)) {
+               dev_err(vop->dev, "failed to get dclk source\n");
+               return PTR_ERR(vop->dclk);
        }
 
        irq = platform_get_irq(pdev, 0);