Hi Laurent, Thanks for your kindly comments on this. Will remove those two camera sensor driver from Medfield ISP.
Thanks Wen >-----Original Message----- >From: Laurent Pinchart [mailto:[email protected]] >Sent: 2010年12月20日 20:05 >To: [email protected] >Cc: Wang, Wen W; [email protected] >Subject: Re: [Meego-kernel] [MFLD Camera - PATCH v5 1/8] iCDK secondary >camera (ov2720) sensor driver based on v4l2 subdev > >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
