[PATCH -next v2] [media] davinci: vpif_capture: fix error return code in vpif_probe()

2013-09-02 Thread Wei Yongjun 
From: Wei Yongjun yongjun_...@trendmicro.com.cn

Fix to return -ENODEV in the subdevice register error handling
case instead of 0, as done elsewhere in this function.

Introduced by commit 873229e4fdf34196aa5d707957c59ba54c25eaba
([media] media: davinci: vpif: capture: add V4L2-async support)

Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn
---
v1 - v2: fix mistake using -ENOMEM
---
 drivers/media/platform/davinci/vpif_capture.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/media/platform/davinci/vpif_capture.c 
b/drivers/media/platform/davinci/vpif_capture.c
index 7fbde6d..e4b6a26 100644
--- a/drivers/media/platform/davinci/vpif_capture.c
+++ b/drivers/media/platform/davinci/vpif_capture.c
@@ -2160,6 +2160,7 @@ static __init int vpif_probe(struct platform_device *pdev)
 
if (!vpif_obj.sd[i]) {
vpif_err(Error registering v4l2 subdevice\n);
+   err = -ENODEV;
goto probe_subdev_out;
}
v4l2_info(vpif_obj.v4l2_dev,


--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH -next v2] [media] davinci: vpif_capture: fix error return code in vpif_probe()

2013-09-02 Thread Prabhakar Lad 
On Mon, Sep 2, 2013 at 2:36 PM, Wei Yongjun weiyj...@gmail.com wrote:
 From: Wei Yongjun yongjun_...@trendmicro.com.cn

 Fix to return -ENODEV in the subdevice register error handling
 case instead of 0, as done elsewhere in this function.

 Introduced by commit 873229e4fdf34196aa5d707957c59ba54c25eaba
 ([media] media: davinci: vpif: capture: add V4L2-async support)

 Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn

Acked-by: Lad, Prabhakar prabhakar.cse...@gmail.com

Regards,
--Prabhakar Lad
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html