tree: git://linuxtv.org/media_tree.git master
head: 2a9ecc17ed9f076ff199a4bf4ebd22b41badb505
commit: 01c2819330b1e0ec6b53dcfac76ad75ff2c8ba4f [395/499] [media] em28xx: make
em28xx-video to be a separate module
reproduce: make C=1 CF=-D__CHECK_ENDIAN__
sparse warnings: (new ones prefixed by >>)
>> drivers/media/usb/em28xx/em28xx-video.c:1151:28: sparse: symbol
>> 'em28xx_ctrl_ops' was not declared. Should it be static?
--
>> drivers/media/usb/em28xx/em28xx-cards.c:2164:36: sparse: cannot size
>> expression
Please consider folding the attached diff :-)
---
0-DAY kernel build testing backend Open Source Technology Center
http://lists.01.org/mailman/listinfo/kbuild Intel Corporation
From: Fengguang Wu <fengguang...@intel.com>
Subject: [PATCH linuxtv-media] em28xx: em28xx_ctrl_ops can be static
TO: Mauro Carvalho Chehab <m.che...@samsung.com>
CC: linux-media@vger.kernel.org
CC: linux-media@vger.kernel.org
CC: linux-ker...@vger.kernel.org
CC: Mauro Carvalho Chehab <m.che...@samsung.com>
CC: linux-media@vger.kernel.org
Signed-off-by: Fengguang Wu <fengguang...@intel.com>
---
em28xx-video.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/usb/em28xx/em28xx-video.c b/drivers/media/usb/em28xx/em28xx-video.c
index 7d11a16..7a3a514 100644
--- a/drivers/media/usb/em28xx/em28xx-video.c
+++ b/drivers/media/usb/em28xx/em28xx-video.c
@@ -1148,7 +1148,7 @@ static int em28xx_s_ctrl(struct v4l2_ctrl *ctrl)
return (ret < 0) ? ret : 0;
}
-const struct v4l2_ctrl_ops em28xx_ctrl_ops = {
+static const struct v4l2_ctrl_ops em28xx_ctrl_ops = {
.s_ctrl = em28xx_s_ctrl,
};
