[PATCH] [media] vim2m: return error if driver registration fails

2015-12-25 Thread Niklas Söderlund
From: Niklas Söderlund 

Signed-off-by: Niklas Söderlund 
---
 drivers/media/platform/vim2m.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/vim2m.c b/drivers/media/platform/vim2m.c
index e18fb9f..da37a59 100644
--- a/drivers/media/platform/vim2m.c
+++ b/drivers/media/platform/vim2m.c
@@ -1079,7 +1079,7 @@ static int __init vim2m_init(void)
if (ret)
platform_device_unregister(_pdev);
 
-   return 0;
+   return ret;
 }
 
 module_init(vim2m_init);
-- 
2.6.3

--
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: dt3155 as a reference

2015-12-25 Thread Hans Verkuil

On 2015-12-24 21:52, Ran Shalit wrote:

On Wed, Dec 23, 2015 at 3:09 PM, hverkuil  wrote:

On 2015-12-23 13:55, Ran Shalit wrote:


Hello,

I think to use dt3155 as a reference for new pci express driver ,
becuase it is highly simple as a starting point, and contains only
single file :)
The driver I'll develop will eventually be used for multiple video
capture and single video output.

I just wanted to ask if you recommend it as a starting point.



Yeah, that would work.

Regards,

Hans



Hi,

Is there a chance anyone have the HW reference for dt3155 ?
I can't  find it anywhere.


It's a saa7116-based device. Google for 'saa7116 datasheet' to download 
the datasheet.


Regards,

Hans
--
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