Hi Sachin, Thank you for your patch. Best wishes, -- Kamil Debski Linux Platform Group Samsung Poland R&D Center > -----Original Message----- > From: linux-media-owner@xxxxxxxxxxxxxxx [mailto:linux-media- > owner@xxxxxxxxxxxxxxx] On Behalf Of Sachin Kamat > Sent: Friday, December 28, 2012 11:18 AM > To: linux-media@xxxxxxxxxxxxxxx > Cc: k.debski@xxxxxxxxxxx; s.nawrocki@xxxxxxxxxxx; > sylvester.nawrocki@xxxxxxxxx; sachin.kamat@xxxxxxxxxx; > patches@xxxxxxxxxx > Subject: [PATCH 2/3] [media] s5p-mfc: Remove redundant 'break' > > The code returns before this statement. Hence not required. > Silences the following smatch message: > drivers/media/platform/s5p-mfc/s5p_mfc_opr_v5.c:525 > s5p_mfc_set_dec_frame_buffer_v5() info: ignoring unreachable code. > > Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> > --- > drivers/media/platform/s5p-mfc/s5p_mfc_opr_v5.c | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) > > diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v5.c > b/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v5.c > index bb99d3d..b0f277e 100644 > --- a/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v5.c > +++ b/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v5.c > @@ -522,7 +522,6 @@ int s5p_mfc_set_dec_frame_buffer_v5(struct > s5p_mfc_ctx *ctx) > mfc_err("Unknown codec for decoding (%x)\n", > ctx->codec_mode); > return -EINVAL; > - break; > } > frame_size = ctx->luma_size; > frame_size_ch = ctx->chroma_size; > -- > 1.7.4.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-media" > in the body of a message to majordomo@xxxxxxxxxxxxxxx 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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html