dean_go Zhang wrote:
> I'm reading to documentation in linux-2.6.30,and ov7670 driver ,and I'm
> trying to make a driver for ov9650.
> 
>     struct soc_camera_ops provides .probe and .remove methods, which are
>     called by
>     the soc-camera core, when a camera is matched against or removed
>     from a camera
>     host bus, .init, .release, .suspend, and .resume are called from the
>     camera host
>     driver as discussed above. Other members of this struct provide
>     respective V4L2
>     functionality.
> 
>  
> This is from the documentation,but I didn't find any thing about
> soc_camera_host_device in ov7670's driver code.
> Does anyone know how to make a v4l2 soc camera driver? 
This isn't really an arm related question.  Should really be asked on
linux-media. (now cc'd along with Guennadi)

Having said that...

The reason you aren't finding soc camera related stuff in the ov7670 driver
is that it isn't currently a soc camera driver.  There is ongoing work to
move the soc-camera framework fully over to using v4l2-subdevs thus allowing
drivers like this one to work both with soc-camera interfaces and others.

There are still a few elements being cleaned up (primarily to do with
negotiation of image formats) that make it tricky for a single driver to
directly support use through soc camera and without it. 

I would suggest looking in the linux-media archive for
Guennanadi Liakhovetski's latest imagebus patches for what still needs doing.

In meantime, the following patch against 2.6.32-rc5 adds soc camera support to 
the
ov7670 driver.  I've been posting updates tracking Guennadi's changes to soc 
camera
to linux-media (though I haven't had a chance to do the recent imagebus changes 
yet).

Unfortunately omnivision aren't exactly free with datasheets (I can get the 
ov9650 from
google but not the ov9640), so I can't check, but based purely on numbering how 
does
this chip compare to the ov9640 which as a driver in kernel (probably in a queue
for next merge window? - it's certainly in the tree Guennadi is using and has 
been
posted to linux-media)

Google did however give me this hit, which mentions an ov9650 driver
http://marex-hnd.blogspot.com/2009/08/omnivision-ov9640-hacking-part-iv.html
Perhaps Guennadi has more info on this?


>From 408902c5584796924f8f9903f6c7338db4a0fd0f Mon Sep 17 00:00:00 2001
From: Jonathan Cameron <ji...@cam.ac.uk>
Date: Sat, 4 Jul 2009 13:25:06 +0000
Subject: [PATCH 02/10] ov7670: Temporary soc-camera support

Signed-off-by: Jonathan Cameron <ji...@cam.ac.uk>
---
 drivers/media/video/ov7670.c |   50 ++++++++++++++++++++++++++++++++++++++++++
 1 files changed, 50 insertions(+), 0 deletions(-)

diff --git a/drivers/media/video/ov7670.c b/drivers/media/video/ov7670.c
index 0e2184e..910a499 100644
--- a/drivers/media/video/ov7670.c
+++ b/drivers/media/video/ov7670.c
@@ -19,6 +19,8 @@
 #include <media/v4l2-chip-ident.h>
 #include <media/v4l2-i2c-drv.h>
 
+#include <media/soc_camera.h>
+#include <linux/autoconf.h>
 
 MODULE_AUTHOR("Jonathan Corbet <cor...@lwn.net>");
 MODULE_DESCRIPTION("A low-level driver for OmniVision ov7670 sensors");
@@ -745,6 +747,10 @@ static int ov7670_s_fmt(struct v4l2_subdev *sd, struct 
v4l2_format *fmt)
        struct ov7670_info *info = to_state(sd);
        unsigned char com7, clkrc = 0;
 
+       ret = ov7670_init(sd, 0);
+       if (ret)
+               return ret;
+
        ret = ov7670_try_fmt_internal(sd, fmt, &ovfmt, &wsize);
        if (ret)
                return ret;
@@ -1239,6 +1245,41 @@ static const struct v4l2_subdev_ops ov7670_ops = {
 };
 
 /* ----------------------------------------------------------------------- */
+static unsigned long ov7670_soc_query_bus_param(struct soc_camera_device *icd)
+{
+       struct soc_camera_link *icl = to_soc_camera_link(icd);
+
+       unsigned long flags = SOCAM_PCLK_SAMPLE_RISING | SOCAM_MASTER |
+               SOCAM_VSYNC_ACTIVE_HIGH | SOCAM_HSYNC_ACTIVE_HIGH |
+               SOCAM_DATAWIDTH_8 | SOCAM_DATA_ACTIVE_HIGH;
+
+       return soc_camera_apply_sensor_flags(icl, flags);
+}
+
+/* This device only supports one bus option */
+static int ov7670_soc_set_bus_param(struct soc_camera_device *icd,
+                                   unsigned long flags)
+{
+       return 0;
+}
+
+static struct soc_camera_ops ov7670_soc_ops = {
+       .set_bus_param = ov7670_soc_set_bus_param,
+       .query_bus_param = ov7670_soc_query_bus_param,
+};
+
+#define SETFOURCC(type) .name = (#type), .fourcc = (V4L2_PIX_FMT_ ## type)
+static const struct soc_camera_data_format ov7670_soc_fmt_lists[] = {
+       {
+               SETFOURCC(YUYV),
+               .depth = 16,
+               .colorspace = V4L2_COLORSPACE_JPEG,
+       }, {
+               SETFOURCC(RGB565),
+               .depth = 16,
+               .colorspace = V4L2_COLORSPACE_SRGB,
+       },
+};
 
 static int ov7670_probe(struct i2c_client *client,
                        const struct i2c_device_id *id)
@@ -1246,7 +1287,16 @@ static int ov7670_probe(struct i2c_client *client,
        struct v4l2_subdev *sd;
        struct ov7670_info *info;
        int ret;
+       struct soc_camera_device *icd = client->dev.platform_data;
+
+       if (!icd) {
+               dev_err(&client->dev, "OV7670: missing soc-camera data!\n");
+               return -EINVAL;
+       }
 
+       icd->ops = &ov7670_soc_ops;
+       icd->formats = ov7670_soc_fmt_lists;
+       icd->num_formats = ARRAY_SIZE(ov7670_soc_fmt_lists);
        info = kzalloc(sizeof(struct ov7670_info), GFP_KERNEL);
        if (info == NULL)
                return -ENOMEM;
-- 
1.6.3.3



--
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

Reply via email to