Messages in this thread | | | Date | Fri, 13 Jan 2012 14:33:54 +0900 | Subject | Re: [PATCH 4/4] drivers/media/video/s5p-mfc/s5p_mfc.c: adjust double test | From | Kyungmin Park <> |
| |
All patches.
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
To Marek, Please collect all patches and send git request pull.
Thank you, Kyungmin Park
On 1/13/12, Julia Lawall <Julia.Lawall@lip6.fr> wrote: > From: Julia Lawall <Julia.Lawall@lip6.fr> > > Rewrite a duplicated test to test the correct value > > The semantic match that finds this problem is as follows: > (http://coccinelle.lip6.fr/) > > // <smpl> > @@ > expression E; > @@ > > ( > * E > || ... || E > | > * E > && ... && E > ) > // </smpl> > > Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr> > > --- > drivers/media/video/s5p-mfc/s5p_mfc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/video/s5p-mfc/s5p_mfc.c > b/drivers/media/video/s5p-mfc/s5p_mfc.c > index 8be8b54..53126f2 100644 > --- a/drivers/media/video/s5p-mfc/s5p_mfc.c > +++ b/drivers/media/video/s5p-mfc/s5p_mfc.c > @@ -475,7 +475,7 @@ static void s5p_mfc_handle_seq_done(struct s5p_mfc_ctx > *ctx, > ctx->mv_size = 0; > } > ctx->dpb_count = s5p_mfc_get_dpb_count(); > - if (ctx->img_width == 0 || ctx->img_width == 0) > + if (ctx->img_width == 0 || ctx->img_height == 0) > ctx->state = MFCINST_ERROR; > else > ctx->state = MFCINST_HEAD_PARSED; > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >
| |