Hi Wei,

Thank you for your patch.

Best wishes,
-- 
Kamil Debski
Linux Platform Group
Samsung Poland R&D Center

> -----Original Message-----
> From: linux-media-ow...@vger.kernel.org [mailto:linux-media-
> ow...@vger.kernel.org] On Behalf Of Wei Yongjun
> Sent: Sunday, December 02, 2012 1:17 PM
> To: kyungmin.p...@samsung.com; k.deb...@samsung.com;
> jtp.p...@samsung.com; mche...@redhat.com
> Cc: yongjun_...@trendmicro.com.cn; linux-arm-ker...@lists.infradead.org;
> linux-media@vger.kernel.org
> Subject: [PATCH -next] [media] s5p-mfc: remove unused variable
> 
> From: Wei Yongjun <yongjun_...@trendmicro.com.cn>
> 
> The variable index is initialized but never used otherwise, so remove
> the unused variable.
> 
> Signed-off-by: Wei Yongjun <yongjun_...@trendmicro.com.cn>

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

> ---
>  drivers/media/platform/s5p-mfc/s5p_mfc.c        | 5 -----
>  drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c | 6 ------
>  2 files changed, 11 deletions(-)
> 
> diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc.c
> b/drivers/media/platform/s5p-mfc/s5p_mfc.c
> index 3afe879..856cf00 100644
> --- a/drivers/media/platform/s5p-mfc/s5p_mfc.c
> +++ b/drivers/media/platform/s5p-mfc/s5p_mfc.c
> @@ -273,7 +273,6 @@ static void s5p_mfc_handle_frame_new(struct
> s5p_mfc_ctx *ctx, unsigned int err)
>       struct s5p_mfc_buf  *dst_buf;
>       size_t dspl_y_addr;
>       unsigned int frame_type;
> -     unsigned int index;
> 
>       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); @@ -310,7 +309,6 @@ static void s5p_mfc_handle_frame_new(struct
> s5p_mfc_ctx *ctx, unsigned int err)
>                       vb2_buffer_done(dst_buf->b,
>                               err ? VB2_BUF_STATE_ERROR :
VB2_BUF_STATE_DONE);
> 
> -                     index = dst_buf->b->v4l2_buf.index;
>                       break;
>               }
>       }
> @@ -326,8 +324,6 @@ static void s5p_mfc_handle_frame(struct s5p_mfc_ctx
> *ctx,
>       unsigned long flags;
>       unsigned int res_change;
> 
> -     unsigned int index;
> -
>       dst_frame_status = s5p_mfc_hw_call(dev->mfc_ops, get_dspl_status,
> dev)
>                               & S5P_FIMV_DEC_STATUS_DECODING_STATUS_MASK;
>       res_change = (s5p_mfc_hw_call(dev->mfc_ops, get_dspl_status, dev)
> @@ -387,7 +383,6 @@ static void s5p_mfc_handle_frame(struct s5p_mfc_ctx
> *ctx,
>                       mfc_debug(2, "Running again the same buffer\n");
>                       ctx->after_packed_pb = 1;
>               } else {
> -                     index = src_buf->b->v4l2_buf.index;
>                       mfc_debug(2, "MFC needs next buffer\n");
>                       ctx->consumed_stream = 0;
>                       list_del(&src_buf->list);
> 
> diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c
> b/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c
> index 3a8cfd9..bf4d2f4 100644
> --- a/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c
> +++ b/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c
> @@ -1408,7 +1408,6 @@ static inline int s5p_mfc_run_dec_frame(struct
> s5p_mfc_ctx *ctx)
>       struct s5p_mfc_buf *temp_vb;
>       unsigned long flags;
>       int last_frame = 0;
> -     unsigned int index;
> 
>       spin_lock_irqsave(&dev->irqlock, flags);
> 
> @@ -1427,8 +1426,6 @@ static inline int s5p_mfc_run_dec_frame(struct
> s5p_mfc_ctx *ctx)
>                       temp_vb->b->v4l2_planes[0].bytesused);
>       spin_unlock_irqrestore(&dev->irqlock, flags);
> 
> -     index = temp_vb->b->v4l2_buf.index;
> -
>       dev->curr_ctx = ctx->num;
>       s5p_mfc_clean_ctx_int_flags(ctx);
>       if (temp_vb->b->v4l2_planes[0].bytesused == 0) { @@ -1452,7
> +1449,6 @@ static inline int s5p_mfc_run_enc_frame(struct s5p_mfc_ctx
> *ctx)
>       unsigned int src_y_size, src_c_size;
>       */
>       unsigned int dst_size;
> -     unsigned int index;
> 
>       spin_lock_irqsave(&dev->irqlock, flags);
> 
> @@ -1487,8 +1483,6 @@ static inline int s5p_mfc_run_enc_frame(struct
> s5p_mfc_ctx *ctx)
> 
>       spin_unlock_irqrestore(&dev->irqlock, flags);
> 
> -     index = src_mb->b->v4l2_buf.index;
> -
>       dev->curr_ctx = ctx->num;
>       s5p_mfc_clean_ctx_int_flags(ctx);
>       s5p_mfc_encode_one_frame_v6(ctx);
> 
> --
> 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


--
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