diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-09-22 10:00:08 +0200 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-09-23 21:13:45 +0200 |
commit | 692f637522f0ae230fe49aab3776603d5e2c5f31 (patch) | |
tree | 844d2ec8496eee42fb460d6f7883312dbde3e841 | |
parent | [media] as102: remove some unneeded checks (diff) | |
download | linux-692f637522f0ae230fe49aab3776603d5e2c5f31.tar.xz linux-692f637522f0ae230fe49aab3776603d5e2c5f31.zip |
[media] davinci: remove an unneeded check
We don't need to check "ret", we know it's zero.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
-rw-r--r-- | drivers/media/platform/davinci/vpfe_capture.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/drivers/media/platform/davinci/vpfe_capture.c b/drivers/media/platform/davinci/vpfe_capture.c index 3ff817b80f22..de55f47a77db 100644 --- a/drivers/media/platform/davinci/vpfe_capture.c +++ b/drivers/media/platform/davinci/vpfe_capture.c @@ -442,11 +442,10 @@ static int vpfe_config_image_format(struct vpfe_device *vpfe_dev, return ret; /* Update the values of sizeimage and bytesperline */ - if (!ret) { - pix->bytesperline = ccdc_dev->hw_ops.get_line_length(); - pix->sizeimage = pix->bytesperline * pix->height; - } - return ret; + pix->bytesperline = ccdc_dev->hw_ops.get_line_length(); + pix->sizeimage = pix->bytesperline * pix->height; + + return 0; } static int vpfe_initialize_device(struct vpfe_device *vpfe_dev) |