Hi Wen, 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 <[email protected]> > 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 > framework > 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. -- Regards, Laurent Pinchart _______________________________________________ MeeGo-kernel mailing list [email protected] http://lists.meego.com/listinfo/meego-kernel
