Re: [PATCH,v1] media: mtk-vcodec: fix vp9 content playback error with show exist frame

[gtk_ruiwang]

Dear Hans, all,

Could you please help to review this patch?

This is a fix about vp9 show+AF8-existing+AF8-frames.

Sincerely thanks,
BR

On Mon, 2019-04-01 at 16:18 +-0800, maoguang.meng+AEA-mediatek.com wrote:
+AD4 From: Maoguang Meng +ADw-maoguang.meng+AEA-mediatek.com+AD4
+AD4 
+AD4 Update referenced frame buffer's reference count when playing vp9
+AD4 content which has show+AF8-existing+AF8-frame flag, and copy enough buffer
+AD4 data to current shown frame.
+AD4 
+AD4 Signed-off-by: Maoguang Meng +ADw-maoguang.meng+AEA-mediatek.com+AD4
+AD4 ---
+AD4  .../media/platform/mtk-vcodec/vdec/vdec+AF8-vp9+AF8-if.c   +AHw   16 +-+-+-+-+-+-+----------
+AD4  1 file changed, 7 insertions(+-), 9 deletions(-)
+AD4 
+AD4 diff --git a/drivers/media/platform/mtk-vcodec/vdec/vdec+AF8-vp9+AF8-if.c b/drivers/media/platform/mtk-vcodec/vdec/vdec+AF8-vp9+AF8-if.c
+AD4 index bc8349b..d950837 100644
+AD4 --- a/drivers/media/platform/mtk-vcodec/vdec/vdec+AF8-vp9+AF8-if.c
+AD4 +-+-+- b/drivers/media/platform/mtk-vcodec/vdec/vdec+AF8-vp9+AF8-if.c
+AD4 +AEAAQA -481,15 +-481,15 +AEAAQA static void vp9+AF8-swap+AF8-frm+AF8-bufs(struct vdec+AF8-vp9+AF8-inst +ACo-inst)
+AD4  		 +ACo-/
+AD4  		if ((frm+AF8-to+AF8-show-+AD4-fb +ACEAPQ NULL) +ACYAJg
+AD4  			(inst-+AD4-cur+AF8-fb-+AD4-base+AF8-y.size +AD4APQ
+AD4 -			frm+AF8-to+AF8-show-+AD4-fb-+AD4-base+AF8-y.size)) +AHs
+AD4 +-			frm+AF8-to+AF8-show-+AD4-fb-+AD4-base+AF8-y.size) +ACYAJg
+AD4 +-			(inst-+AD4-cur+AF8-fb-+AD4-base+AF8-c.size +AD4APQ
+AD4 +-			frm+AF8-to+AF8-show-+AD4-fb-+AD4-base+AF8-c.size)) +AHs
+AD4  			memcpy((void +ACo)inst-+AD4-cur+AF8-fb-+AD4-base+AF8-y.va,
+AD4  				(void +ACo)frm+AF8-to+AF8-show-+AD4-fb-+AD4-base+AF8-y.va,
+AD4 -				vsi-+AD4-buf+AF8-w +ACo
+AD4 -				vsi-+AD4-buf+AF8-h)+ADs
+AD4 +-				frm+AF8-to+AF8-show-+AD4-fb-+AD4-base+AF8-y.size)+ADs
+AD4  			memcpy((void +ACo)inst-+AD4-cur+AF8-fb-+AD4-base+AF8-c.va,
+AD4  				(void +ACo)frm+AF8-to+AF8-show-+AD4-fb-+AD4-base+AF8-c.va,
+AD4 -				vsi-+AD4-buf+AF8-w +ACo
+AD4 -				vsi-+AD4-buf+AF8-h / 2)+ADs
+AD4 +-				frm+AF8-to+AF8-show-+AD4-fb-+AD4-base+AF8-c.size)+ADs
+AD4  		+AH0 else +AHs
+AD4  			/+ACo After resolution change case, current CAPTURE buffer
+AD4  			 +ACo may have less buffer size than frm+AF8-to+AF8-show buffer
+AD4 +AEAAQA -895,7 +-895,7 +AEAAQA static int vdec+AF8-vp9+AF8-decode(unsigned long h+AF8-vdec, struct mtk+AF8-vcodec+AF8-mem +ACo-bs,
+AD4  
+AD4  		if (vsi-+AD4-resolution+AF8-changed) +AHs
+AD4  			if (+ACE-vp9+AF8-alloc+AF8-work+AF8-buf(inst)) +AHs
+AD4 -				ret +AD0 -EINVAL+ADs
+AD4 +-				ret +AD0 -EIO+ADs
+AD4  				goto DECODE+AF8-ERROR+ADs
+AD4  			+AH0
+AD4  		+AH0
+AD4 +AEAAQA -924,14 +-924,12 +AEAAQA static int vdec+AF8-vp9+AF8-decode(unsigned long h+AF8-vdec, struct mtk+AF8-vcodec+AF8-mem +ACo-bs,
+AD4  
+AD4  		if (vsi-+AD4-show+AF8-existing+AF8-frame +ACYAJg (vsi-+AD4-frm+AF8-to+AF8-show+AF8-idx +ADw
+AD4  					VP9+AF8-MAX+AF8-FRM+AF8-BUF+AF8-NUM)) +AHs
+AD4 -			mtk+AF8-vcodec+AF8-err(inst,
+AD4 +-			mtk+AF8-vcodec+AF8-debug(inst,
+AD4  				+ACI-Skip Decode drv-+AD4-new+AF8-fb+AF8-idx+AD0AJQ-d, drv-+AD4-frm+AF8-to+AF8-show+AF8-idx+AD0AJQ-d+ACI,
+AD4  				vsi-+AD4-new+AF8-fb+AF8-idx, vsi-+AD4-frm+AF8-to+AF8-show+AF8-idx)+ADs
+AD4  
+AD4  			vp9+AF8-ref+AF8-cnt+AF8-fb(inst, +ACY-vsi-+AD4-new+AF8-fb+AF8-idx,
+AD4  					vsi-+AD4-frm+AF8-to+AF8-show+AF8-idx)+ADs
+AD4 -			ret +AD0 -EINVAL+ADs
+AD4 -			goto DECODE+AF8-ERROR+ADs
+AD4  		+AH0
+AD4  
+AD4  		/+ACo VPU assign the buffer pointer in its address space,