From: Jihong Sui <jihong.sui@amlogic.com>
Date: Wed, 28 Nov 2018 07:06:15 +0000 (+0800)
Subject: deinterlace: reduce the screen flash when fast forward [1/1]
X-Git-Tag: khadas-vims-v0.9.6-release~1056
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0bc8b4d6c36c0c6b34f00d03ef318e7adbfe0432;p=platform%2Fkernel%2Flinux-amlogic.git

deinterlace: reduce the screen flash when fast forward [1/1]

PD#SWPL-2188

Problem:
1.fast forward/rewind operation, the screen flashes

Solution:
1.add function to update MCDI_MCVECRD_CTRL[9]

Verify:
1.txl

Change-Id: I1bf8583901fa49c518cca74e7716632447adf32f
Signed-off-by: Jihong Sui <jihong.sui@amlogic.com>
---

diff --git a/drivers/amlogic/media/deinterlace/deinterlace.c b/drivers/amlogic/media/deinterlace/deinterlace.c
index bfb1378..b9eeeab 100644
--- a/drivers/amlogic/media/deinterlace/deinterlace.c
+++ b/drivers/amlogic/media/deinterlace/deinterlace.c
@@ -129,7 +129,7 @@ static di_dev_t *de_devp;
 static dev_t di_devno;
 static struct class *di_clsp;
 
-static const char version_s[] = "2018-11-28b";
+static const char version_s[] = "2018-12-04a";
 
 static int bypass_state = 1;
 static int bypass_all;
@@ -3383,6 +3383,8 @@ static unsigned char pre_de_buf_config(void)
 #endif
 		    )
 			return 0;
+
+		di_patch_post_update_mc_sw(DI_MC_SW_OTHER, false);
 	} else if (di_pre_stru.prog_proc_type == 2) {
 		di_linked_buf_idx = peek_free_linked_buf();
 		if (di_linked_buf_idx == -1 &&
@@ -3923,14 +3925,17 @@ jiffies_to_msecs(jiffies_64 - vframe->ready_jiffies64));
 		di_pre_stru.madi_enable = 0;
 		di_pre_stru.mcdi_enable = 0;
 		di_buf->post_proc_flag = 0;
+		di_patch_post_update_mc_sw(DI_MC_SW_OTHER, false);
 	} else if (bypass_post_state) {
 		di_pre_stru.madi_enable = 0;
 		di_pre_stru.mcdi_enable = 0;
 		di_buf->post_proc_flag = 0;
+		di_patch_post_update_mc_sw(DI_MC_SW_OTHER, false);
 	} else {
 		di_pre_stru.madi_enable = (pre_enable_mask&1);
 		di_pre_stru.mcdi_enable = ((pre_enable_mask>>1)&1);
 		di_buf->post_proc_flag = 1;
+		di_patch_post_update_mc_sw(DI_MC_SW_OTHER, mcpre_en);//en
 	}
 	if ((di_pre_stru.di_mem_buf_dup_p == di_pre_stru.di_wr_buf) ||
 	    (di_pre_stru.di_chan2_buf_dup_p == di_pre_stru.di_wr_buf)) {
@@ -5903,7 +5908,7 @@ static void di_unreg_process_irq(void)
 #if (defined ENABLE_SPIN_LOCK_ALWAYS)
 	spin_unlock_irqrestore(&plist_lock, flags);
 #endif
-
+	di_patch_post_update_mc_sw(DI_MC_SW_REG, false);
 	di_pre_stru.force_unreg_req_flag = 0;
 	di_pre_stru.disable_req_flag = 0;
 	recovery_flag = 0;
@@ -6100,6 +6105,7 @@ static void di_reg_process_irq(void)
 				vframe->width, vframe->height, vframe->type);
 			}
 			di_pre_stru.bypass_flag = true;
+			di_patch_post_update_mc_sw(DI_MC_SW_OTHER, false);
 			return;
 		} else {
 			di_pre_stru.bypass_flag = false;
@@ -6198,6 +6204,8 @@ static void di_reg_process_irq(void)
 					(vframe->source_type),
 					is_progressive(vframe),
 					vframe->sig_fmt);
+
+		di_patch_post_update_mc_sw(DI_MC_SW_REG, true);
 		if (de_devp->flags & DI_LOAD_REG_FLAG)
 			up(&di_sema);
 		init_flag = 1;
@@ -6474,6 +6482,7 @@ static enum hrtimer_restart di_pre_hrtimer_func(struct hrtimer *timer)
 	if (!di_pre_stru.bypass_flag)
 		di_pre_trigger_work(&di_pre_stru);
 	hrtimer_forward_now(&di_pre_hrtimer, ms_to_ktime(10));
+	di_patch_post_update_mc();
 	return HRTIMER_RESTART;
 }
 /*
@@ -7630,6 +7639,7 @@ static int di_probe(struct platform_device *pdev)
 	if (IS_ERR(di_devp->task))
 		pr_err("%s create kthread error.\n", __func__);
 	di_debugfs_init();	/*2018-07-18 add debugfs*/
+	di_patch_post_update_mc_sw(DI_MC_SW_IC, true);
 fail_kmalloc_dev:
 	return ret;
 }
diff --git a/drivers/amlogic/media/deinterlace/deinterlace_hw.c b/drivers/amlogic/media/deinterlace/deinterlace_hw.c
index c597258..7bc25f2 100644
--- a/drivers/amlogic/media/deinterlace/deinterlace_hw.c
+++ b/drivers/amlogic/media/deinterlace/deinterlace_hw.c
@@ -2453,6 +2453,52 @@ mif->chroma_x_end0 - mif->chroma_x_start0 + 1, /* c length */
 	}
 }
 
+static unsigned int di_mc_update;
+void di_patch_post_update_mc(void)
+{
+	if (di_mc_update == DI_MC_SW_ON_MASK) {
+//	if (di_mc_update) {
+		DI_VSYNC_WR_MPEG_REG_BITS(MCDI_MCVECRD_CTRL, 1, 9, 1);
+	}
+}
+
+
+void di_patch_post_update_mc_sw(unsigned int cmd, bool on)
+{
+	unsigned int l_flg = di_mc_update;
+
+	switch (cmd) {
+	case DI_MC_SW_IC:
+		if (is_meson_gxtvbb_cpu()   ||
+			is_meson_txl_cpu()  ||
+			is_meson_txlx_cpu() ||
+			is_meson_txhd_cpu()) {
+			di_mc_update |= DI_MC_SW_IC;
+		}
+		break;
+	case DI_MC_SW_REG:
+		if (on) {
+			di_mc_update |= cmd;
+			di_mc_update &= ~DI_MC_SW_OTHER;
+		} else {
+			di_mc_update &= ~(cmd|DI_MC_SW_OTHER);
+		}
+		break;
+	case DI_MC_SW_OTHER:
+
+//	case DI_MC_SW_POST:
+		if (on)
+			di_mc_update |= cmd;
+		else
+			di_mc_update &= ~cmd;
+
+		break;
+	}
+
+	if (l_flg !=  di_mc_update)
+		pr_debug("%s:0x%x->0x%x\n", __func__, l_flg, di_mc_update);
+
+}
 void initial_di_post_2(int hsize_post, int vsize_post,
 	int hold_line, bool post_write_en)
 {
diff --git a/drivers/amlogic/media/deinterlace/deinterlace_hw.h b/drivers/amlogic/media/deinterlace/deinterlace_hw.h
index 8d8b95d..d6b44d5 100644
--- a/drivers/amlogic/media/deinterlace/deinterlace_hw.h
+++ b/drivers/amlogic/media/deinterlace/deinterlace_hw.h
@@ -187,4 +187,14 @@ extern u8 *di_vmap(ulong addr, u32 size, bool *bflg);
 extern void di_unmap_phyaddr(u8 *vaddr);
 extern int di_print(const char *fmt, ...);
 
+#define DI_MC_SW_OTHER	(1<<0)
+#define DI_MC_SW_REG	(1<<1)
+//#define DI_MC_SW_POST	(1<<2)
+#define DI_MC_SW_IC	(1<<2)
+
+#define DI_MC_SW_ON_MASK	(DI_MC_SW_REG | DI_MC_SW_OTHER | DI_MC_SW_IC)
+
+extern void di_patch_post_update_mc(void);
+extern void di_patch_post_update_mc_sw(unsigned int cmd, bool on);
+
 #endif