summaryrefslogtreecommitdiffstats
path: root/drivers/media/i2c/tc358743.c
diff options
context:
space:
mode:
authorNathan Chancellor <natechancellor@gmail.com>2018-10-09 00:11:28 +0200
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>2018-11-06 11:19:49 +0100
commit0ede1794b9667d9cd35d1ac27f2eeff52d5733d5 (patch)
tree11f45d2c931313b7b5013dbd55b03353f4fb250c /drivers/media/i2c/tc358743.c
parentMerge tag 'v4.20-rc1' into patchwork (diff)
downloadlinux-0ede1794b9667d9cd35d1ac27f2eeff52d5733d5.tar.xz
linux-0ede1794b9667d9cd35d1ac27f2eeff52d5733d5.zip
media: tc358743: Remove unnecessary self assignment
Clang warns when a variable is assigned to itself. drivers/media/i2c/tc358743.c:1921:7: warning: explicitly assigning value of variable of type 'int' to itself [-Wself-assign] ret = ret; ~~~ ^ ~~~ 1 warning generated. Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com> Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Diffstat (limited to 'drivers/media/i2c/tc358743.c')
-rw-r--r--drivers/media/i2c/tc358743.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/media/i2c/tc358743.c b/drivers/media/i2c/tc358743.c
index ca5d92942820..41d470d9ca94 100644
--- a/drivers/media/i2c/tc358743.c
+++ b/drivers/media/i2c/tc358743.c
@@ -1918,7 +1918,6 @@ static int tc358743_probe_of(struct tc358743_state *state)
ret = v4l2_fwnode_endpoint_alloc_parse(of_fwnode_handle(ep), &endpoint);
if (ret) {
dev_err(dev, "failed to parse endpoint\n");
- ret = ret;
goto put_node;
}