diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2014-09-22 05:00:08 -0300 |
|---|---|---|
| committer |   Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-09-23 16:13:45 -0300 |
| commit | 692f637522f0ae230fe49aab3776603d5e2c5f31 (patch) | |
| tree | 844d2ec8496eee42fb460d6f7883312dbde3e841 /drivers/media/platform/davinci | |
| parent | [media] as102: remove some unneeded checks (diff) | |
| download | linux-dev-692f637522f0ae230fe49aab3776603d5e2c5f31.tar.xz linux-dev-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>
Diffstat (limited to 'drivers/media/platform/davinci')
| -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) |