[Meego-kernel] [MFLD Camera - PATCH v5 1/8] iCDK secondary camera (ov2720) sensor driver based on v4l2 subdev
laurent.pinchart at ideasonboard.com
Mon Dec 20 04:04:30 PST 2010
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