From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Date: Fri, 11 Jan 2013 06:48:41 +0000 Subject: [patch] [media] cx231xx: add a missing break statement Message-Id: <20130111064841.GB13207@elgon.mountain> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Mauro Carvalho Chehab Cc: Hans Verkuil , Hans de Goede , Sakari Ailus , Jesper Juhl , linux-media@vger.kernel.org, kernel-janitors@vger.kernel.org My static checker complains about the fall through here. From the context it looks like we should add a break statement. Signed-off-by: Dan Carpenter --- Untested. diff --git a/drivers/media/usb/cx231xx/cx231xx-video.c b/drivers/media/usb/cx231xx/cx231xx-video.c index 93dfc18..06376d9 100644 --- a/drivers/media/usb/cx231xx/cx231xx-video.c +++ b/drivers/media/usb/cx231xx/cx231xx-video.c @@ -1751,6 +1751,7 @@ static int vidioc_s_register(struct file *file, void *priv, 0x02, (u16)reg->reg, 1, value, 1, 2); + break; case 0x322: ret cx231xx_write_i2c_master(dev,