summaryrefslogtreecommitdiffstats
path: root/drivers/staging/media
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2017-03-11 19:48:19 +0000
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-03-12 14:57:00 +0100
commitb8ff636cd7d72334951a7fa1dfe9f21ba4d2107b (patch)
tree476455c17aa5aa88269a1ab50d811ba9e2392b14 /drivers/staging/media
parent9573878611f4dd4d7be2e649814bef3270f4e5bd (diff)
downloadop-kernel-dev-b8ff636cd7d72334951a7fa1dfe9f21ba4d2107b.zip
op-kernel-dev-b8ff636cd7d72334951a7fa1dfe9f21ba4d2107b.tar.gz
staging: atomisp: remove redundant check for client being null
The previous statement checks if client is null, so the null check when assigning dev is redundant and can be removed. Detected by CoverityScan, CID#1416555 ("Logically Dead Code") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/media')
-rw-r--r--drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c b/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
index 65513ca..2929492 100644
--- a/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
+++ b/drivers/staging/media/atomisp/platform/intel-mid/atomisp_gmin_platform.c
@@ -354,7 +354,7 @@ static struct gmin_subdev *gmin_subdev_add(struct v4l2_subdev *subdev)
if (!client)
return NULL;
- dev = client ? &client->dev : NULL;
+ dev = &client->dev;
for (i=0; i < MAX_SUBDEVS && gmin_subdevs[i].subdev; i++)
;
OpenPOWER on IntegriCloud