From patchwork Tue Feb 26 22:02:11 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arun Kumar K X-Patchwork-Id: 2185461 Return-Path: X-Original-To: patchwork-linux-media@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id B8E033FCF2 for ; Tue, 26 Feb 2013 13:08:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756928Ab3BZNIL (ORCPT ); Tue, 26 Feb 2013 08:08:11 -0500 Received: from mailout3.samsung.com ([203.254.224.33]:33992 "EHLO mailout3.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756759Ab3BZNIJ (ORCPT ); Tue, 26 Feb 2013 08:08:09 -0500 Received: from epcpsbgr3.samsung.com (u143.gpu120.samsung.co.kr [203.254.230.143]) by mailout3.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTP id <0MIT00E96WH5RO70@mailout3.samsung.com> for linux-media@vger.kernel.org; Tue, 26 Feb 2013 22:08:08 +0900 (KST) Received: from epcpsbgm1.samsung.com ( [172.20.52.125]) by epcpsbgr3.samsung.com (EPCPMTA) with SMTP id E0.F4.18793.7B3BC215; Tue, 26 Feb 2013 22:08:07 +0900 (KST) X-AuditID: cbfee68f-b7f6a6d000004969-e7-512cb3b753c6 Received: from epmmp2 ( [203.254.227.17]) by epcpsbgm1.samsung.com (EPCPMTA) with SMTP id AE.84.03918.7B3BC215; Tue, 26 Feb 2013 22:08:07 +0900 (KST) Received: from ubuntu.sisodomain.com ([107.108.73.176]) by mmp2.samsung.com (Oracle Communications Messaging Server 7u4-24.01 (7.0.4.24.0) 64bit (built Nov 17 2011)) with ESMTPA id <0MIT008U1WHER6N0@mmp2.samsung.com>; Tue, 26 Feb 2013 22:08:07 +0900 (KST) From: Arun Kumar K To: linux-media@vger.kernel.org Cc: k.debski@samsung.com, jtp.park@samsung.com, s.nawrocki@samsung.com, arun.kk@samsung.com, arun.m@samsung.com Subject: [PATCH] [media] s5p-mfc: Fix frame skip bug Date: Wed, 27 Feb 2013 03:32:11 +0530 Message-id: <1361916131-1717-1-git-send-email-arun.kk@samsung.com> X-Mailer: git-send-email 1.7.9.5 X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrGLMWRmVeSWpSXmKPExsWyRsSkVnf7Zp1Ag123rCw+nrrNavHq3yV2 iyNrrzJZ/Hh9gc2iZ8NWVovDb9pZHdg8+rasYvT4vEkugCmKyyYlNSezLLVI3y6BK6Oz+yp7 wXqOiv9TBRoYJ7B3MXJySAiYSEx+f5EJwhaTuHBvPRuILSSwlFFi3StHmJrrd1sYIeLTGSWW TbXqYuQCstuZJN7+XM4KkmAT0JC4d+IxmC0iIC/xpPcG0CAODmaBQon2V6kgYWGgOUuXrgSb zyKgKjG/ZRoLiM0r4CSxd8cndpByCQEFiTmTbEDGSwg8ZpN4/+INM0S9gMS3yYdYIGpkJTYd YIY4TVLi4IobLBMYBRcwMqxiFE0tSC4oTkovMtYrTswtLs1L10vOz93ECAzD0/+e9e9gvHvA +hBjMtC4icxSosn5wDDOK4k3NDYxNzU2NTOytLQ0JU1YSZxX/pJMoJBAemJJanZqakFqUXxR aU5q8SFGJg5OqQZG1Y/1hZ91sptn1Oq+z+M+3rrw3rcHnTxXXV+WB3Ks0N6cWRe+2Mgg9dCX E8JulsvD2q+tlXDpnXxYa92SqeWZrKHmaVcuL/jKrm/ryreJP0o3LKL+qy/HhgW8HQ+OltXq Noa/eXXm6efnl3qXvH70ZpfAeeEvl8+E37J29Zmgs1quoZHh74O9SizFGYmGWsxFxYkAeifI XFkCAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrCIsWRmVeSWpSXmKPExsVy+t9jQd3tm3UCDZZvNbL4eOo2q8Wrf5fY LY6svcpk8eP1BTaLng1bWS0Ov2lndWDz6NuyitHj8ya5AKaoBkabjNTElNQihdS85PyUzLx0 WyXv4HjneFMzA0NdQ0sLcyWFvMTcVFslF58AXbfMHKCdSgpliTmlQKGAxOJiJX07TBNCQ9x0 LWAaI3R9Q4LgeowM0EDCGsaMzu6r7AXrOSr+TxVoYJzA3sXIySEhYCJx/W4LI4QtJnHh3no2 EFtIYDqjxLKpVl2MXEB2O5PE25/LWUESbAIaEvdOPAazRQTkJZ703gBq4OBgFiiUaH+VChIW Bpq5dOlKsDksAqoS81umsYDYvAJOEnt3fGIHKZcQUJCYM8lmAiP3AkaGVYyiqQXJBcVJ6bmG esWJucWleel6yfm5mxjBYf5MagfjygaLQ4wCHIxKPLwJLDqBQqyJZcWVuYcYJTiYlUR4mTYB hXhTEiurUovy44tKc1KLDzEmAy2fyCwlmpwPjMG8knhDYxNzU2NTSxMLEzNL0oSVxHkZTz0J EBJITyxJzU5NLUgtgtnCxMEp1cCoFmGZN+n13H+syfNOrPyn+/jhN2/HXQzCdy7f2Ws293Z4 QOmNRobFzUtZBdYf32XvsNJz//or9qJOWoWv5d5E1id3T/uV+MV15ZJF/7/aMe+tZ3GNOv5Z P5qzJ7zWpO3PyguPH5f+7bn3ZKledabA1GuRlze3rFS03rWL21h23ny9XX+vh8mkK7EUZyQa ajEXFScCACfzI5W3AgAA DLP-Filter: Pass X-MTR: 20000000000000000@CPGS X-CFilter-Loop: Reflected Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org The issue was seen in VP8 decoding where the last frame was skipped by the driver. This patch gets the correct frame_type value to fix this bug. Signed-off-by: Arun Kumar K Signed-off-by: Arun Mankuzhi --- drivers/media/platform/s5p-mfc/s5p_mfc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc.c b/drivers/media/platform/s5p-mfc/s5p_mfc.c index ac69e9b..64c55cf 100644 --- a/drivers/media/platform/s5p-mfc/s5p_mfc.c +++ b/drivers/media/platform/s5p-mfc/s5p_mfc.c @@ -276,7 +276,7 @@ static void s5p_mfc_handle_frame_new(struct s5p_mfc_ctx *ctx, unsigned int err) unsigned int frame_type; dspl_y_addr = s5p_mfc_hw_call(dev->mfc_ops, get_dspl_y_adr, dev); - frame_type = s5p_mfc_hw_call(dev->mfc_ops, get_dec_frame_type, dev); + frame_type = s5p_mfc_hw_call(dev->mfc_ops, get_disp_frame_type, ctx); /* If frame is same as previous then skip and do not dequeue */ if (frame_type == S5P_FIMV_DECODE_FRAME_SKIPPED) {