[Meego-kernel] [MFLD Camera - PATCH v5 1/8] iCDK secondary camera (ov2720) sensor driver based on v4l2 subdev
Wang, Wen W
wen.w.wang at intel.com
Mon Dec 20 18:07:38 PST 2010
Thanks for your kindly comments on this. Will remove those two camera sensor driver from Medfield ISP.
>From: Laurent Pinchart [mailto:laurent.pinchart at ideasonboard.com]
>Sent: 2010年12月20日 20:05
>To: meego-kernel at lists.meego.com
>Cc: Wang, Wen W; meego-kernel at meego.com
>Subject: Re: [Meego-kernel] [MFLD Camera - PATCH v5 1/8] iCDK secondary
>camera (ov2720) sensor driver based on v4l2 subdev
>Thanks for the patch.
>On Friday 17 December 2010 11:53:29 Wang, Wen W wrote:
>> From a645f19882830fa38d3d81573d10b53f48f56553 Mon Sep 17 00:00:00 2001
>> From: Wen Wang <wen.w.wang at intel.com>
>> Date: Sat, 18 Dec 2010 00:05:16 +0800
>> Subject: [PATCH] iCDK secondary camera (ov2720) sensor driver based on v4l2
>> subdev framework
>> This patch enables OminiVision 2720 camera sensor based on V4L2 subdev
>> diff --git a/drivers/media/video/ov2720/Kconfig
>> b/drivers/media/video/ov2720/Kconfig new file mode 100644
>> index 0000000..129986f
>> --- /dev/null
>> +++ b/drivers/media/video/ov2720/Kconfig
>> @@ -0,0 +1,9 @@
>> +config VIDEO_OV2720
>> + tristate "Medifiled OV2720 RAW Sensor"
>> + depends on I2C
>When I asked you to remove the dependency on the Medfield ISP, I didn't mean
>just removing from Kconfig. You need to make the ov2720 (as well as the other
>sensor driver) completely independent of the Medfield ISP. They can't use
>*any* Medfield-related data structure.
>Please fix the two sensor drivers and send them on their own (I really want to
>emphasize that there must be absolutely no reference to anything Medfield
>specific in the sensor drivers) for review to the linux-media mailing list.
>They must compile and run fine without the Medfield ISP driver.
More information about the MeeGo-kernel