diff options
author | Colin Ian King <colin.king@canonical.com> | 2017-03-11 19:48:19 +0000 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-03-12 14:57:00 +0100 |
commit | b8ff636cd7d72334951a7fa1dfe9f21ba4d2107b (patch) | |
tree | 476455c17aa5aa88269a1ab50d811ba9e2392b14 /drivers/staging/media | |
parent | 9573878611f4dd4d7be2e649814bef3270f4e5bd (diff) | |
download | op-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.c | 2 |
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++) ; |