'break' after return statement is redundant. Remove it. Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> Cc: Paul Mundt <lethal@xxxxxxxxxxxx> --- drivers/video/kyro/fbdev.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/video/kyro/fbdev.c b/drivers/video/kyro/fbdev.c index c82f040..79bfb5c 100644 --- a/drivers/video/kyro/fbdev.c +++ b/drivers/video/kyro/fbdev.c @@ -623,7 +623,6 @@ static int kyrofb_ioctl(struct fb_info *info, "command instead.\n"); return -EINVAL; } - break; case KYRO_IOCTL_UVSTRIDE: if (copy_to_user(argp, &deviceInfo.ulOverlayUVStride, sizeof(unsigned long))) return -EFAULT; -- 1.7.9.5 -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html