The case statement where c is zero is redundant because the previous while loop will only exit if c is non-zero or non-0xff, so c can never be zero by the time the switch statement is reaced. Clean up the code by removing it. Signed-off-by: Colin Ian King <colin.i.king@xxxxxxxxx> --- drivers/media/platform/renesas/rcar_jpu.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/media/platform/renesas/rcar_jpu.c b/drivers/media/platform/renesas/rcar_jpu.c index 81038df71bb5..6af154b41eb4 100644 --- a/drivers/media/platform/renesas/rcar_jpu.c +++ b/drivers/media/platform/renesas/rcar_jpu.c @@ -643,8 +643,6 @@ static u8 jpu_parse_hdr(void *buffer, unsigned long size, unsigned int *width, return 0; skip(&jpeg_buffer, (long)word - 2); break; - case 0: - break; default: return 0; } -- 2.49.0
