> From: Arun Kumar K [mailto:arun...@samsung.com]
> Sent: Wednesday, November 14, 2012 2:27 PM
> To: linux-media@vger.kernel.org
> Cc: k.deb...@samsung.com; jtp.p...@samsung.com; aru...@samsung.com;
> arun...@samsung.com
> Subject: [PATCH] [media] s5p-mfc: Handle multi-frame input buffer
> 
> When one input buffer has multiple frames, it should be fed again to
> the hardware with the remaining bytes. Removed the check for P frame in
> this scenario as this condition can come with all frame types.
> 
> Signed-off-by: Arun Kumar K <arun...@samsung.com>
> Signed-off-by: ARUN MANKUZHI <aru...@samsung.com>

Acked-by: Kamil Debski <k.deb...@samsung.com>

> ---
>  drivers/media/platform/s5p-mfc/s5p_mfc.c |    7 ++-----
>  1 files changed, 2 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc.c
> b/drivers/media/platform/s5p-mfc/s5p_mfc.c
> index 0ca8dbb..d3cd738 100644
> --- a/drivers/media/platform/s5p-mfc/s5p_mfc.c
> +++ b/drivers/media/platform/s5p-mfc/s5p_mfc.c
> @@ -382,11 +382,8 @@ static void s5p_mfc_handle_frame(struct
> s5p_mfc_ctx *ctx,
>               ctx->consumed_stream += s5p_mfc_hw_call(dev->mfc_ops,
>                                               get_consumed_stream, dev);
>               if (ctx->codec_mode != S5P_MFC_CODEC_H264_DEC &&
> -                     s5p_mfc_hw_call(dev->mfc_ops,
> -                             get_dec_frame_type, dev) ==
> -                                     S5P_FIMV_DECODE_FRAME_P_FRAME
> -                                     && ctx->consumed_stream + STUFF_BYTE
<
> -
src_buf->b->v4l2_planes[0].bytesused) {
> +                     ctx->consumed_stream + STUFF_BYTE <
> +                     src_buf->b->v4l2_planes[0].bytesused) {
>                       /* Run MFC again on the same buffer */
>                       mfc_debug(2, "Running again the same buffer\n");
>                       ctx->after_packed_pb = 1;
> --
> 1.7.0.4


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to