Hi Sangwook, On 08/03/2012 04:24 PM, Sangwook Lee wrote: > I was thinking about this, but this seems to be is a bit time-consuming because > I have to do this just due to lack of s5k4ecgx hardware information. > let me try it later once > this patch is accepted. I've converted this driver to use function calls instead of the register arrays. It can be pulled, along with a couple of minor fixes/improvements, from following git tree: git://linuxtv.org/snawrocki/media.git s5k4ecgx (gitweb: http://git.linuxtv.org/snawrocki/media.git/s5k4ecgx) I don't own any Origen board thus it's untested. Could you give it a try ? The register write sequence should be identical as in the case of using the arrays. Regarding support for still (JPEG) capture features of S5K4ECGX, it should be possible to make this work with the mainline s5p-fimc driver, it supports V4L2_PIX_FMT_JPEG/V4L2_MBUS_FMT_JPEG_1X8 formats. There is only missing an API for preallocating proper memory buffer for the snapshot frame. Currently s5p-fimc calculates buffer's size from pixel resolution, using some fixed coefficient. I'm planning on adding new V4L2_CID_FRAMESIZE control that could be a replacement for V4L2_CID_CAM_JPEG_MEMSIZE, as found in this driver: https://android.googlesource.com/kernel/samsung.git/+/3b0c5d2887fca99cab7dd506817b1049d38198a1/drivers/media/video/s5k4ecgx.c Except that, there would be needed a new V4L2_CID_SNAPSHOT control in place of custom V4L2_CID_CAM_CAPTURE. I might try to add that and document in near future. You won't find much regarding the face detection features in V4L2, unfortunately. _Maybe_ I'll try to address this as well on some day, for now private controls might be your only solution. Unless you feel like adding face detection features support to V4L2.. ;) BTW, are your requirements to support both EVT1.0 and EVT1.1 S5K4ECGX revisions ? -- Regards, Sylwester -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
