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 majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
