summaryrefslogtreecommitdiffstats
path: root/drivers/media/v4l2-core/v4l2-mc.c
diff options
context:
space:
mode:
authorMauro Carvalho Chehab <mchehab@osg.samsung.com>2016-02-12 08:07:59 -0200
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2016-02-16 08:55:58 -0200
commit153d41a1d0a47dfd4d773d8b9db192a62f92355e (patch)
tree967e028aa0900c76ed363dab255761e64b769537 /drivers/media/v4l2-core/v4l2-mc.c
parent6168309acd6afe420108dba20a66e6781b933f89 (diff)
downloadlinux-153d41a1d0a47dfd4d773d8b9db192a62f92355e.tar.bz2
[media] v4l2-mc: remove the unused sensor var
This fixes this warning: v4l2-mc.c: In function 'v4l2_mc_create_media_graph': v4l2-mc.c:60:69: warning: variable 'sensor' set but not used [-Wunused-but-set-variable] We could solve it the other way: don't do the second loop for webcams. However, that would fail if a chip would have two sensors plugged. This is not the current case, but it doesn't hurt to be future-safe here, specially since this code runs only once during device probe. So, performance is not an issue here. Reported-by: Hans Verkuil <hverkuil@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/v4l2-core/v4l2-mc.c')
-rw-r--r--drivers/media/v4l2-core/v4l2-mc.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/media/v4l2-core/v4l2-mc.c b/drivers/media/v4l2-core/v4l2-mc.c
index 97b2e1e64d2e..9d79d599d979 100644
--- a/drivers/media/v4l2-core/v4l2-mc.c
+++ b/drivers/media/v4l2-core/v4l2-mc.c
@@ -96,7 +96,7 @@ int v4l2_mc_create_media_graph(struct media_device *mdev)
{
struct media_entity *entity;
- struct media_entity *if_vid = NULL, *if_aud = NULL, *sensor = NULL;
+ struct media_entity *if_vid = NULL, *if_aud = NULL;
struct media_entity *tuner = NULL, *decoder = NULL;
struct media_entity *io_v4l = NULL, *io_vbi = NULL, *io_swradio = NULL;
bool is_webcam = false;
@@ -130,7 +130,6 @@ int v4l2_mc_create_media_graph(struct media_device *mdev)
io_swradio = entity;
break;
case MEDIA_ENT_F_CAM_SENSOR:
- sensor = entity;
is_webcam = true;
break;
}