include cleanup: Update gfp.h and slab.h includes to prepare for breaking implicit...
[safe/jmp/linux-2.6] / drivers / media / video / soc_camera.c
index b017490..a24174d 100644 (file)
  * published by the Free Software Foundation.
  */
 
-#include <linux/module.h>
-#include <linux/init.h>
 #include <linux/device.h>
-#include <linux/list.h>
 #include <linux/err.h>
+#include <linux/i2c.h>
+#include <linux/init.h>
+#include <linux/list.h>
 #include <linux/mutex.h>
+#include <linux/module.h>
+#include <linux/platform_device.h>
+#include <linux/slab.h>
 #include <linux/vmalloc.h>
 
+#include <media/soc_camera.h>
 #include <media/v4l2-common.h>
 #include <media/v4l2-ioctl.h>
 #include <media/v4l2-dev.h>
 #include <media/videobuf-core.h>
-#include <media/soc_camera.h>
+#include <media/soc_mediabus.h>
+
+/* Default to VGA resolution */
+#define DEFAULT_WIDTH  640
+#define DEFAULT_HEIGHT 480
 
 static LIST_HEAD(hosts);
 static LIST_HEAD(devices);
-static DEFINE_MUTEX(list_lock);
-static DEFINE_MUTEX(video_lock);
+static DEFINE_MUTEX(list_lock);                /* Protects the list of hosts */
 
-const static struct soc_camera_data_format*
-format_by_fourcc(struct soc_camera_device *icd, unsigned int fourcc)
+const struct soc_camera_format_xlate *soc_camera_xlate_by_fourcc(
+       struct soc_camera_device *icd, unsigned int fourcc)
 {
        unsigned int i;
 
-       for (i = 0; i < icd->num_formats; i++)
-               if (icd->formats[i].fourcc == fourcc)
-                       return icd->formats + i;
+       for (i = 0; i < icd->num_user_formats; i++)
+               if (icd->user_formats[i].host_fmt->fourcc == fourcc)
+                       return icd->user_formats + i;
        return NULL;
 }
+EXPORT_SYMBOL(soc_camera_xlate_by_fourcc);
 
-static int soc_camera_try_fmt_vid_cap(struct file *file, void *priv,
-                                 struct v4l2_format *f)
+/**
+ * soc_camera_apply_sensor_flags() - apply platform SOCAM_SENSOR_INVERT_* flags
+ * @icl:       camera platform parameters
+ * @flags:     flags to be inverted according to platform configuration
+ * @return:    resulting flags
+ */
+unsigned long soc_camera_apply_sensor_flags(struct soc_camera_link *icl,
+                                           unsigned long flags)
 {
-       struct soc_camera_file *icf = file->private_data;
-       struct soc_camera_device *icd = icf->icd;
-       struct soc_camera_host *ici =
-               to_soc_camera_host(icd->dev.parent);
-       enum v4l2_field field;
-       const struct soc_camera_data_format *fmt;
-       int ret;
+       unsigned long f;
 
-       WARN_ON(priv != file->private_data);
-
-       fmt = format_by_fourcc(icd, f->fmt.pix.pixelformat);
-       if (!fmt) {
-               dev_dbg(&icd->dev, "invalid format 0x%08x\n",
-                       f->fmt.pix.pixelformat);
-               return -EINVAL;
+       /* If only one of the two polarities is supported, switch to the opposite */
+       if (icl->flags & SOCAM_SENSOR_INVERT_HSYNC) {
+               f = flags & (SOCAM_HSYNC_ACTIVE_HIGH | SOCAM_HSYNC_ACTIVE_LOW);
+               if (f == SOCAM_HSYNC_ACTIVE_HIGH || f == SOCAM_HSYNC_ACTIVE_LOW)
+                       flags ^= SOCAM_HSYNC_ACTIVE_HIGH | SOCAM_HSYNC_ACTIVE_LOW;
        }
 
-       dev_dbg(&icd->dev, "fmt: 0x%08x\n", fmt->fourcc);
-
-       field = f->fmt.pix.field;
+       if (icl->flags & SOCAM_SENSOR_INVERT_VSYNC) {
+               f = flags & (SOCAM_VSYNC_ACTIVE_HIGH | SOCAM_VSYNC_ACTIVE_LOW);
+               if (f == SOCAM_VSYNC_ACTIVE_HIGH || f == SOCAM_VSYNC_ACTIVE_LOW)
+                       flags ^= SOCAM_VSYNC_ACTIVE_HIGH | SOCAM_VSYNC_ACTIVE_LOW;
+       }
 
-       if (field == V4L2_FIELD_ANY) {
-               field = V4L2_FIELD_NONE;
-       } else if (V4L2_FIELD_NONE != field) {
-               dev_err(&icd->dev, "Field type invalid.\n");
-               return -EINVAL;
+       if (icl->flags & SOCAM_SENSOR_INVERT_PCLK) {
+               f = flags & (SOCAM_PCLK_SAMPLE_RISING | SOCAM_PCLK_SAMPLE_FALLING);
+               if (f == SOCAM_PCLK_SAMPLE_RISING || f == SOCAM_PCLK_SAMPLE_FALLING)
+                       flags ^= SOCAM_PCLK_SAMPLE_RISING | SOCAM_PCLK_SAMPLE_FALLING;
        }
 
-       /* test physical bus parameters */
-       ret = ici->ops->try_bus_param(icd, f->fmt.pix.pixelformat);
-       if (ret)
-               return ret;
+       return flags;
+}
+EXPORT_SYMBOL(soc_camera_apply_sensor_flags);
 
-       /* limit format to hardware capabilities */
-       ret = ici->ops->try_fmt_cap(icd, f);
+static int soc_camera_try_fmt_vid_cap(struct file *file, void *priv,
+                                     struct v4l2_format *f)
+{
+       struct soc_camera_file *icf = file->private_data;
+       struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
 
-       /* calculate missing fields */
-       f->fmt.pix.field = field;
-       f->fmt.pix.bytesperline =
-               (f->fmt.pix.width * fmt->depth) >> 3;
-       f->fmt.pix.sizeimage =
-               f->fmt.pix.height * f->fmt.pix.bytesperline;
+       WARN_ON(priv != file->private_data);
 
-       return ret;
+       /* limit format to hardware capabilities */
+       return ici->ops->try_fmt(icd, f);
 }
 
 static int soc_camera_enum_input(struct file *file, void *priv,
                                 struct v4l2_input *inp)
 {
+       struct soc_camera_file *icf = file->private_data;
+       struct soc_camera_device *icd = icf->icd;
+       int ret = 0;
+
        if (inp->index != 0)
                return -EINVAL;
 
-       inp->type = V4L2_INPUT_TYPE_CAMERA;
-       inp->std = V4L2_STD_UNKNOWN;
-       strcpy(inp->name, "Camera");
+       if (icd->ops->enum_input)
+               ret = icd->ops->enum_input(icd, inp);
+       else {
+               /* default is camera */
+               inp->type = V4L2_INPUT_TYPE_CAMERA;
+               inp->std  = V4L2_STD_UNKNOWN;
+               strcpy(inp->name, "Camera");
+       }
 
-       return 0;
+       return ret;
 }
 
 static int soc_camera_g_input(struct file *file, void *priv, unsigned int *i)
@@ -125,7 +140,11 @@ static int soc_camera_s_input(struct file *file, void *priv, unsigned int i)
 
 static int soc_camera_s_std(struct file *file, void *priv, v4l2_std_id *a)
 {
-       return 0;
+       struct soc_camera_file *icf = file->private_data;
+       struct soc_camera_device *icd = icf->icd;
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+
+       return v4l2_subdev_call(sd, core, s_std, *a);
 }
 
 static int soc_camera_reqbufs(struct file *file, void *priv,
@@ -134,13 +153,10 @@ static int soc_camera_reqbufs(struct file *file, void *priv,
        int ret;
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
-       struct soc_camera_host *ici =
-               to_soc_camera_host(icd->dev.parent);
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
 
        WARN_ON(priv != file->private_data);
 
-       dev_dbg(&icd->dev, "%s: %d\n", __func__, p->memory);
-
        ret = videobuf_reqbufs(&icf->vb_vidq, p);
        if (ret < 0)
                return ret;
@@ -178,30 +194,153 @@ static int soc_camera_dqbuf(struct file *file, void *priv,
        return videobuf_dqbuf(&icf->vb_vidq, p, file->f_flags & O_NONBLOCK);
 }
 
-static int soc_camera_open(struct inode *inode, struct file *file)
+/* Always entered with .video_lock held */
+static int soc_camera_init_user_formats(struct soc_camera_device *icd)
 {
-       struct video_device *vdev;
-       struct soc_camera_device *icd;
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       int i, fmts = 0, raw_fmts = 0, ret;
+       enum v4l2_mbus_pixelcode code;
+
+       while (!v4l2_subdev_call(sd, video, enum_mbus_fmt, raw_fmts, &code))
+               raw_fmts++;
+
+       if (!ici->ops->get_formats)
+               /*
+                * Fallback mode - the host will have to serve all
+                * sensor-provided formats one-to-one to the user
+                */
+               fmts = raw_fmts;
+       else
+               /*
+                * First pass - only count formats this host-sensor
+                * configuration can provide
+                */
+               for (i = 0; i < raw_fmts; i++) {
+                       ret = ici->ops->get_formats(icd, i, NULL);
+                       if (ret < 0)
+                               return ret;
+                       fmts += ret;
+               }
+
+       if (!fmts)
+               return -ENXIO;
+
+       icd->user_formats =
+               vmalloc(fmts * sizeof(struct soc_camera_format_xlate));
+       if (!icd->user_formats)
+               return -ENOMEM;
+
+       icd->num_user_formats = fmts;
+
+       dev_dbg(&icd->dev, "Found %d supported formats.\n", fmts);
+
+       /* Second pass - actually fill data formats */
+       fmts = 0;
+       for (i = 0; i < raw_fmts; i++)
+               if (!ici->ops->get_formats) {
+                       v4l2_subdev_call(sd, video, enum_mbus_fmt, i, &code);
+                       icd->user_formats[i].host_fmt =
+                               soc_mbus_get_fmtdesc(code);
+                       icd->user_formats[i].code = code;
+               } else {
+                       ret = ici->ops->get_formats(icd, i,
+                                                   &icd->user_formats[fmts]);
+                       if (ret < 0)
+                               goto egfmt;
+                       fmts += ret;
+               }
+
+       icd->current_fmt = &icd->user_formats[0];
+
+       return 0;
+
+egfmt:
+       icd->num_user_formats = 0;
+       vfree(icd->user_formats);
+       return ret;
+}
+
+/* Always entered with .video_lock held */
+static void soc_camera_free_user_formats(struct soc_camera_device *icd)
+{
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+
+       if (ici->ops->put_formats)
+               ici->ops->put_formats(icd);
+       icd->current_fmt = NULL;
+       icd->num_user_formats = 0;
+       vfree(icd->user_formats);
+       icd->user_formats = NULL;
+}
+
+#define pixfmtstr(x) (x) & 0xff, ((x) >> 8) & 0xff, ((x) >> 16) & 0xff, \
+       ((x) >> 24) & 0xff
+
+/* Called with .vb_lock held, or from the first open(2), see comment there */
+static int soc_camera_set_fmt(struct soc_camera_file *icf,
+                             struct v4l2_format *f)
+{
+       struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       struct v4l2_pix_format *pix = &f->fmt.pix;
+       int ret;
+
+       dev_dbg(&icd->dev, "S_FMT(%c%c%c%c, %ux%u)\n",
+               pixfmtstr(pix->pixelformat), pix->width, pix->height);
+
+       /* We always call try_fmt() before set_fmt() or set_crop() */
+       ret = ici->ops->try_fmt(icd, f);
+       if (ret < 0)
+               return ret;
+
+       ret = ici->ops->set_fmt(icd, f);
+       if (ret < 0) {
+               return ret;
+       } else if (!icd->current_fmt ||
+                  icd->current_fmt->host_fmt->fourcc != pix->pixelformat) {
+               dev_err(&icd->dev,
+                       "Host driver hasn't set up current format correctly!\n");
+               return -EINVAL;
+       }
+
+       icd->user_width         = pix->width;
+       icd->user_height        = pix->height;
+       icd->colorspace         = pix->colorspace;
+       icf->vb_vidq.field      =
+               icd->field      = pix->field;
+
+       if (f->type != V4L2_BUF_TYPE_VIDEO_CAPTURE)
+               dev_warn(&icd->dev, "Attention! Wrong buf-type %d\n",
+                        f->type);
+
+       dev_dbg(&icd->dev, "set width: %d height: %d\n",
+               icd->user_width, icd->user_height);
+
+       /* set physical bus parameters */
+       return ici->ops->set_bus_param(icd, pix->pixelformat);
+}
+
+static int soc_camera_open(struct file *file)
+{
+       struct video_device *vdev = video_devdata(file);
+       struct soc_camera_device *icd = container_of(vdev->parent,
+                                                    struct soc_camera_device,
+                                                    dev);
+       struct soc_camera_link *icl = to_soc_camera_link(icd);
        struct soc_camera_host *ici;
        struct soc_camera_file *icf;
        int ret;
 
-       icf = vmalloc(sizeof(*icf));
-       if (!icf)
-               return -ENOMEM;
-
-       /* Protect against icd->remove() until we module_get() both drivers. */
-       mutex_lock(&video_lock);
+       if (!icd->ops)
+               /* No device driver attached */
+               return -ENODEV;
 
-       vdev = video_devdata(file);
-       icd = container_of(vdev->parent, struct soc_camera_device, dev);
        ici = to_soc_camera_host(icd->dev.parent);
 
-       if (!try_module_get(icd->ops->owner)) {
-               dev_err(&icd->dev, "Couldn't lock sensor driver.\n");
-               ret = -EINVAL;
-               goto emgd;
-       }
+       icf = vmalloc(sizeof(*icf));
+       if (!icf)
+               return -ENOMEM;
 
        if (!try_module_get(ici->ops->owner)) {
                dev_err(&icd->dev, "Couldn't lock capture bus driver.\n");
@@ -209,70 +348,119 @@ static int soc_camera_open(struct inode *inode, struct file *file)
                goto emgi;
        }
 
+       /*
+        * Protect against icd->ops->remove() until we module_get() both
+        * drivers.
+        */
+       mutex_lock(&icd->video_lock);
+
        icf->icd = icd;
        icd->use_count++;
 
        /* Now we really have to activate the camera */
        if (icd->use_count == 1) {
+               /* Restore parameters before the last close() per V4L2 API */
+               struct v4l2_format f = {
+                       .type = V4L2_BUF_TYPE_VIDEO_CAPTURE,
+                       .fmt.pix = {
+                               .width          = icd->user_width,
+                               .height         = icd->user_height,
+                               .field          = icd->field,
+                               .colorspace     = icd->colorspace,
+                               .pixelformat    =
+                                       icd->current_fmt->host_fmt->fourcc,
+                       },
+               };
+
+               if (icl->power) {
+                       ret = icl->power(icd->pdev, 1);
+                       if (ret < 0)
+                               goto epower;
+               }
+
+               /* The camera could have been already on, try to reset */
+               if (icl->reset)
+                       icl->reset(icd->pdev);
+
                ret = ici->ops->add(icd);
                if (ret < 0) {
                        dev_err(&icd->dev, "Couldn't activate the camera: %d\n", ret);
-                       icd->use_count--;
                        goto eiciadd;
                }
-       }
 
-       mutex_unlock(&video_lock);
+               /*
+                * Try to configure with default parameters. Notice: this is the
+                * very first open, so, we cannot race against other calls,
+                * apart from someone else calling open() simultaneously, but
+                * .video_lock is protecting us against it.
+                */
+               ret = soc_camera_set_fmt(icf, &f);
+               if (ret < 0)
+                       goto esfmt;
+       }
 
        file->private_data = icf;
        dev_dbg(&icd->dev, "camera device open\n");
 
        ici->ops->init_videobuf(&icf->vb_vidq, icd);
 
+       mutex_unlock(&icd->video_lock);
+
        return 0;
 
-       /* All errors are entered with the video_lock held */
+       /*
+        * First five errors are entered with the .video_lock held
+        * and use_count == 1
+        */
+esfmt:
+       ici->ops->remove(icd);
 eiciadd:
+       if (icl->power)
+               icl->power(icd->pdev, 0);
+epower:
+       icd->use_count--;
+       mutex_unlock(&icd->video_lock);
        module_put(ici->ops->owner);
 emgi:
-       module_put(icd->ops->owner);
-emgd:
-       mutex_unlock(&video_lock);
        vfree(icf);
        return ret;
 }
 
-static int soc_camera_close(struct inode *inode, struct file *file)
+static int soc_camera_close(struct file *file)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
        struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
-       struct video_device *vdev = icd->vdev;
 
-       mutex_lock(&video_lock);
+       mutex_lock(&icd->video_lock);
        icd->use_count--;
-       if (!icd->use_count)
+       if (!icd->use_count) {
+               struct soc_camera_link *icl = to_soc_camera_link(icd);
+
                ici->ops->remove(icd);
-       module_put(icd->ops->owner);
+               if (icl->power)
+                       icl->power(icd->pdev, 0);
+       }
+
+       mutex_unlock(&icd->video_lock);
+
        module_put(ici->ops->owner);
-       mutex_unlock(&video_lock);
 
        vfree(icf);
 
-       dev_dbg(vdev->parent, "camera device close\n");
+       dev_dbg(&icd->dev, "camera device close\n");
 
        return 0;
 }
 
 static ssize_t soc_camera_read(struct file *file, char __user *buf,
-                          size_t count, loff_t *ppos)
+                              size_t count, loff_t *ppos)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
-       struct video_device *vdev = icd->vdev;
        int err = -EINVAL;
 
-       dev_err(vdev->parent, "camera device read not implemented\n");
+       dev_err(&icd->dev, "camera device read not implemented\n");
 
        return err;
 }
@@ -299,8 +487,7 @@ static unsigned int soc_camera_poll(struct file *file, poll_table *pt)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
-       struct soc_camera_host *ici =
-               to_soc_camera_host(icd->dev.parent);
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
 
        if (list_empty(&icf->vb_vidq.stream)) {
                dev_err(&icd->dev, "Trying to poll with no queued buffers!\n");
@@ -310,8 +497,7 @@ static unsigned int soc_camera_poll(struct file *file, poll_table *pt)
        return ici->ops->poll(file, pt);
 }
 
-
-static struct file_operations soc_camera_fops = {
+static struct v4l2_file_operations soc_camera_fops = {
        .owner          = THIS_MODULE,
        .open           = soc_camera_open,
        .release        = soc_camera_close,
@@ -319,94 +505,74 @@ static struct file_operations soc_camera_fops = {
        .read           = soc_camera_read,
        .mmap           = soc_camera_mmap,
        .poll           = soc_camera_poll,
-       .llseek         = no_llseek,
 };
 
-
 static int soc_camera_s_fmt_vid_cap(struct file *file, void *priv,
-                               struct v4l2_format *f)
+                                   struct v4l2_format *f)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
-       struct soc_camera_host *ici =
-               to_soc_camera_host(icd->dev.parent);
        int ret;
-       struct v4l2_rect rect;
-       const static struct soc_camera_data_format *data_fmt;
 
        WARN_ON(priv != file->private_data);
 
-       data_fmt = format_by_fourcc(icd, f->fmt.pix.pixelformat);
-       if (!data_fmt)
-               return -EINVAL;
-
-       /* buswidth may be further adjusted by the ici */
-       icd->buswidth = data_fmt->depth;
-
-       ret = soc_camera_try_fmt_vid_cap(file, icf, f);
-       if (ret < 0)
-               return ret;
+       mutex_lock(&icf->vb_vidq.vb_lock);
 
-       rect.left       = icd->x_current;
-       rect.top        = icd->y_current;
-       rect.width      = f->fmt.pix.width;
-       rect.height     = f->fmt.pix.height;
-       ret = ici->ops->set_fmt_cap(icd, f->fmt.pix.pixelformat, &rect);
-       if (ret < 0)
-               return ret;
+       if (icf->vb_vidq.bufs[0]) {
+               dev_err(&icd->dev, "S_FMT denied: queue initialised\n");
+               ret = -EBUSY;
+               goto unlock;
+       }
 
-       icd->current_fmt        = data_fmt;
-       icd->width              = rect.width;
-       icd->height             = rect.height;
-       icf->vb_vidq.field      = f->fmt.pix.field;
-       if (V4L2_BUF_TYPE_VIDEO_CAPTURE != f->type)
-               dev_warn(&icd->dev, "Attention! Wrong buf-type %d\n",
-                        f->type);
+       ret = soc_camera_set_fmt(icf, f);
 
-       dev_dbg(&icd->dev, "set width: %d height: %d\n",
-               icd->width, icd->height);
+unlock:
+       mutex_unlock(&icf->vb_vidq.vb_lock);
 
-       /* set physical bus parameters */
-       return ici->ops->set_bus_param(icd, f->fmt.pix.pixelformat);
+       return ret;
 }
 
 static int soc_camera_enum_fmt_vid_cap(struct file *file, void  *priv,
-                                  struct v4l2_fmtdesc *f)
+                                      struct v4l2_fmtdesc *f)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
-       const struct soc_camera_data_format *format;
+       const struct soc_mbus_pixelfmt *format;
 
        WARN_ON(priv != file->private_data);
 
-       if (f->index >= icd->num_formats)
+       if (f->index >= icd->num_user_formats)
                return -EINVAL;
 
-       format = &icd->formats[f->index];
+       format = icd->user_formats[f->index].host_fmt;
 
-       strlcpy(f->description, format->name, sizeof(f->description));
+       if (format->name)
+               strlcpy(f->description, format->name, sizeof(f->description));
        f->pixelformat = format->fourcc;
        return 0;
 }
 
 static int soc_camera_g_fmt_vid_cap(struct file *file, void *priv,
-                               struct v4l2_format *f)
+                                   struct v4l2_format *f)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct v4l2_pix_format *pix = &f->fmt.pix;
 
        WARN_ON(priv != file->private_data);
 
-       f->fmt.pix.width        = icd->width;
-       f->fmt.pix.height       = icd->height;
-       f->fmt.pix.field        = icf->vb_vidq.field;
-       f->fmt.pix.pixelformat  = icd->current_fmt->fourcc;
-       f->fmt.pix.bytesperline =
-               (f->fmt.pix.width * icd->current_fmt->depth) >> 3;
-       f->fmt.pix.sizeimage    =
-               f->fmt.pix.height * f->fmt.pix.bytesperline;
+       pix->width              = icd->user_width;
+       pix->height             = icd->user_height;
+       pix->field              = icf->vb_vidq.field;
+       pix->pixelformat        = icd->current_fmt->host_fmt->fourcc;
+       pix->bytesperline       = soc_mbus_bytes_per_line(pix->width,
+                                               icd->current_fmt->host_fmt);
+       pix->colorspace         = icd->colorspace;
+       if (pix->bytesperline < 0)
+               return pix->bytesperline;
+       pix->sizeimage          = pix->height * pix->bytesperline;
        dev_dbg(&icd->dev, "current_fmt->fourcc: 0x%08x\n",
-               icd->current_fmt->fourcc);
+               icd->current_fmt->host_fmt->fourcc);
        return 0;
 }
 
@@ -415,8 +581,7 @@ static int soc_camera_querycap(struct file *file, void  *priv,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
-       struct soc_camera_host *ici =
-               to_soc_camera_host(icd->dev.parent);
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
 
        WARN_ON(priv != file->private_data);
 
@@ -429,18 +594,24 @@ static int soc_camera_streamon(struct file *file, void *priv,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+       int ret;
 
        WARN_ON(priv != file->private_data);
 
-       dev_dbg(&icd->dev, "%s\n", __func__);
-
        if (i != V4L2_BUF_TYPE_VIDEO_CAPTURE)
                return -EINVAL;
 
-       icd->ops->start_capture(icd);
+       mutex_lock(&icd->video_lock);
+
+       v4l2_subdev_call(sd, video, s_stream, 1);
 
        /* This calls buf_queue from host driver's videobuf_queue_ops */
-       return videobuf_streamon(&icf->vb_vidq);
+       ret = videobuf_streamon(&icf->vb_vidq);
+
+       mutex_unlock(&icd->video_lock);
+
+       return ret;
 }
 
 static int soc_camera_streamoff(struct file *file, void *priv,
@@ -448,19 +619,24 @@ static int soc_camera_streamoff(struct file *file, void *priv,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
 
        WARN_ON(priv != file->private_data);
 
-       dev_dbg(&icd->dev, "%s\n", __func__);
-
        if (i != V4L2_BUF_TYPE_VIDEO_CAPTURE)
                return -EINVAL;
 
-       /* This calls buf_release from host driver's videobuf_queue_ops for all
-        * remaining buffers. When the last buffer is freed, stop capture */
+       mutex_lock(&icd->video_lock);
+
+       /*
+        * This calls buf_release from host driver's videobuf_queue_ops for all
+        * remaining buffers. When the last buffer is freed, stop capture
+        */
        videobuf_streamoff(&icf->vb_vidq);
 
-       icd->ops->stop_capture(icd);
+       v4l2_subdev_call(sd, video, s_stream, 0);
+
+       mutex_unlock(&icd->video_lock);
 
        return 0;
 }
@@ -470,6 +646,7 @@ static int soc_camera_queryctrl(struct file *file, void *priv,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
        int i;
 
        WARN_ON(priv != file->private_data);
@@ -477,6 +654,15 @@ static int soc_camera_queryctrl(struct file *file, void *priv,
        if (!qc->id)
                return -EINVAL;
 
+       /* First check host controls */
+       for (i = 0; i < ici->ops->num_controls; i++)
+               if (qc->id == ici->ops->controls[i].id) {
+                       memcpy(qc, &(ici->ops->controls[i]),
+                               sizeof(*qc));
+                       return 0;
+               }
+
+       /* Then device controls */
        for (i = 0; i < icd->ops->num_controls; i++)
                if (qc->id == icd->ops->controls[i].id) {
                        memcpy(qc, &(icd->ops->controls[i]),
@@ -492,25 +678,19 @@ static int soc_camera_g_ctrl(struct file *file, void *priv,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+       int ret;
 
        WARN_ON(priv != file->private_data);
 
-       switch (ctrl->id) {
-       case V4L2_CID_GAIN:
-               if (icd->gain == (unsigned short)~0)
-                       return -EINVAL;
-               ctrl->value = icd->gain;
-               return 0;
-       case V4L2_CID_EXPOSURE:
-               if (icd->exposure == (unsigned short)~0)
-                       return -EINVAL;
-               ctrl->value = icd->exposure;
-               return 0;
+       if (ici->ops->get_ctrl) {
+               ret = ici->ops->get_ctrl(icd, ctrl);
+               if (ret != -ENOIOCTLCMD)
+                       return ret;
        }
 
-       if (icd->ops->get_control)
-               return icd->ops->get_control(icd, ctrl);
-       return -EINVAL;
+       return v4l2_subdev_call(sd, core, g_ctrl, ctrl);
 }
 
 static int soc_camera_s_ctrl(struct file *file, void *priv,
@@ -518,12 +698,19 @@ static int soc_camera_s_ctrl(struct file *file, void *priv,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+       int ret;
 
        WARN_ON(priv != file->private_data);
 
-       if (icd->ops->set_control)
-               return icd->ops->set_control(icd, ctrl);
-       return -EINVAL;
+       if (ici->ops->set_ctrl) {
+               ret = ici->ops->set_ctrl(icd, ctrl);
+               if (ret != -ENOIOCTLCMD)
+                       return ret;
+       }
+
+       return v4l2_subdev_call(sd, core, s_ctrl, ctrl);
 }
 
 static int soc_camera_cropcap(struct file *file, void *fh,
@@ -531,20 +718,9 @@ static int soc_camera_cropcap(struct file *file, void *fh,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
 
-       a->type                         = V4L2_BUF_TYPE_VIDEO_CAPTURE;
-       a->bounds.left                  = icd->x_min;
-       a->bounds.top                   = icd->y_min;
-       a->bounds.width                 = icd->width_max;
-       a->bounds.height                = icd->height_max;
-       a->defrect.left                 = icd->x_min;
-       a->defrect.top                  = icd->y_min;
-       a->defrect.width                = 640;
-       a->defrect.height               = 480;
-       a->pixelaspect.numerator        = 1;
-       a->pixelaspect.denominator      = 1;
-
-       return 0;
+       return ici->ops->cropcap(icd, a);
 }
 
 static int soc_camera_g_crop(struct file *file, void *fh,
@@ -552,96 +728,117 @@ static int soc_camera_g_crop(struct file *file, void *fh,
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       int ret;
 
-       a->type         = V4L2_BUF_TYPE_VIDEO_CAPTURE;
-       a->c.left       = icd->x_current;
-       a->c.top        = icd->y_current;
-       a->c.width      = icd->width;
-       a->c.height     = icd->height;
+       mutex_lock(&icf->vb_vidq.vb_lock);
+       ret = ici->ops->get_crop(icd, a);
+       mutex_unlock(&icf->vb_vidq.vb_lock);
 
-       return 0;
+       return ret;
 }
 
+/*
+ * According to the V4L2 API, drivers shall not update the struct v4l2_crop
+ * argument with the actual geometry, instead, the user shall use G_CROP to
+ * retrieve it. However, we expect camera host and client drivers to update
+ * the argument, which we then use internally, but do not return to the user.
+ */
 static int soc_camera_s_crop(struct file *file, void *fh,
                             struct v4l2_crop *a)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
-       struct soc_camera_host *ici =
-               to_soc_camera_host(icd->dev.parent);
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       struct v4l2_rect *rect = &a->c;
+       struct v4l2_crop current_crop;
        int ret;
 
        if (a->type != V4L2_BUF_TYPE_VIDEO_CAPTURE)
                return -EINVAL;
 
-       ret = ici->ops->set_fmt_cap(icd, 0, &a->c);
-       if (!ret) {
-               icd->width      = a->c.width;
-               icd->height     = a->c.height;
-               icd->x_current  = a->c.left;
-               icd->y_current  = a->c.top;
+       dev_dbg(&icd->dev, "S_CROP(%ux%u@%u:%u)\n",
+               rect->width, rect->height, rect->left, rect->top);
+
+       /* Cropping is allowed during a running capture, guard consistency */
+       mutex_lock(&icf->vb_vidq.vb_lock);
+
+       /* If get_crop fails, we'll let host and / or client drivers decide */
+       ret = ici->ops->get_crop(icd, &current_crop);
+
+       /* Prohibit window size change with initialised buffers */
+       if (icf->vb_vidq.bufs[0] && !ret &&
+           (a->c.width != current_crop.c.width ||
+            a->c.height != current_crop.c.height)) {
+               dev_err(&icd->dev,
+                       "S_CROP denied: queue initialised and sizes differ\n");
+               ret = -EBUSY;
+       } else {
+               ret = ici->ops->set_crop(icd, a);
        }
 
+       mutex_unlock(&icf->vb_vidq.vb_lock);
+
        return ret;
 }
 
-static int soc_camera_g_chip_ident(struct file *file, void *fh,
-                                  struct v4l2_chip_ident *id)
+static int soc_camera_g_parm(struct file *file, void *fh,
+                            struct v4l2_streamparm *a)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
 
-       if (!icd->ops->get_chip_id)
-               return -EINVAL;
+       if (ici->ops->get_parm)
+               return ici->ops->get_parm(icd, a);
 
-       return icd->ops->get_chip_id(icd, id);
+       return -ENOIOCTLCMD;
 }
 
-#ifdef CONFIG_VIDEO_ADV_DEBUG
-static int soc_camera_g_register(struct file *file, void *fh,
-                                struct v4l2_register *reg)
+static int soc_camera_s_parm(struct file *file, void *fh,
+                            struct v4l2_streamparm *a)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
 
-       if (!icd->ops->get_register)
-               return -EINVAL;
+       if (ici->ops->set_parm)
+               return ici->ops->set_parm(icd, a);
 
-       return icd->ops->get_register(icd, reg);
+       return -ENOIOCTLCMD;
 }
 
-static int soc_camera_s_register(struct file *file, void *fh,
-                                struct v4l2_register *reg)
+static int soc_camera_g_chip_ident(struct file *file, void *fh,
+                                  struct v4l2_dbg_chip_ident *id)
 {
        struct soc_camera_file *icf = file->private_data;
        struct soc_camera_device *icd = icf->icd;
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
 
-       if (!icd->ops->set_register)
-               return -EINVAL;
+       return v4l2_subdev_call(sd, core, g_chip_ident, id);
+}
 
-       return icd->ops->set_register(icd, reg);
+#ifdef CONFIG_VIDEO_ADV_DEBUG
+static int soc_camera_g_register(struct file *file, void *fh,
+                                struct v4l2_dbg_register *reg)
+{
+       struct soc_camera_file *icf = file->private_data;
+       struct soc_camera_device *icd = icf->icd;
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+
+       return v4l2_subdev_call(sd, core, g_register, reg);
 }
-#endif
 
-static int device_register_link(struct soc_camera_device *icd)
+static int soc_camera_s_register(struct file *file, void *fh,
+                                struct v4l2_dbg_register *reg)
 {
-       int ret = device_register(&icd->dev);
+       struct soc_camera_file *icf = file->private_data;
+       struct soc_camera_device *icd = icf->icd;
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
 
-       if (ret < 0) {
-               /* Prevent calling device_unregister() */
-               icd->dev.parent = NULL;
-               dev_err(&icd->dev, "Cannot register device: %d\n", ret);
-       /* Even if probe() was unsuccessful for all registered drivers,
-        * device_register() returns 0, and we add the link, just to
-        * document this camera's control device */
-       } else if (icd->control)
-               /* Have to sysfs_remove_link() before device_unregister()? */
-               if (sysfs_create_link(&icd->dev.kobj, &icd->control->kobj,
-                                     "control"))
-                       dev_warn(&icd->dev,
-                                "Failed creating the control symlink\n");
-       return ret;
+       return v4l2_subdev_call(sd, core, s_register, reg);
 }
+#endif
 
 /* So far this function cannot fail */
 static void scan_add_host(struct soc_camera_host *ici)
@@ -652,90 +849,258 @@ static void scan_add_host(struct soc_camera_host *ici)
 
        list_for_each_entry(icd, &devices, list) {
                if (icd->iface == ici->nr) {
-                       icd->dev.parent = &ici->dev;
-                       device_register_link(icd);
+                       int ret;
+                       icd->dev.parent = ici->v4l2_dev.dev;
+                       dev_set_name(&icd->dev, "%u-%u", icd->iface,
+                                    icd->devnum);
+                       ret = device_register(&icd->dev);
+                       if (ret < 0) {
+                               icd->dev.parent = NULL;
+                               dev_err(&icd->dev,
+                                       "Cannot register device: %d\n", ret);
+                       }
                }
        }
 
        mutex_unlock(&list_lock);
 }
 
-/* return: 0 if no match found or a match found and
- * device_register() successful, error code otherwise */
-static int scan_add_device(struct soc_camera_device *icd)
+#ifdef CONFIG_I2C_BOARDINFO
+static int soc_camera_init_i2c(struct soc_camera_device *icd,
+                              struct soc_camera_link *icl)
 {
-       struct soc_camera_host *ici;
-       int ret = 0;
+       struct i2c_client *client;
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       struct i2c_adapter *adap = i2c_get_adapter(icl->i2c_adapter_id);
+       struct v4l2_subdev *subdev;
 
-       mutex_lock(&list_lock);
+       if (!adap) {
+               dev_err(&icd->dev, "Cannot get I2C adapter #%d. No driver?\n",
+                       icl->i2c_adapter_id);
+               goto ei2cga;
+       }
 
-       list_add_tail(&icd->list, &devices);
+       icl->board_info->platform_data = icd;
 
-       /* Watch out for class_for_each_device / class_find_device API by
-        * Dave Young <hidave.darkstar@gmail.com> */
-       list_for_each_entry(ici, &hosts, list) {
-               if (icd->iface == ici->nr) {
-                       ret = 1;
-                       icd->dev.parent = &ici->dev;
-                       break;
-               }
-       }
+       subdev = v4l2_i2c_new_subdev_board(&ici->v4l2_dev, adap,
+                               icl->module_name, icl->board_info, NULL);
+       if (!subdev)
+               goto ei2cnd;
 
-       mutex_unlock(&list_lock);
+       client = subdev->priv;
 
-       if (ret)
-               ret = device_register_link(icd);
+       /* Use to_i2c_client(dev) to recover the i2c client */
+       dev_set_drvdata(&icd->dev, &client->dev);
 
-       return ret;
+       return 0;
+ei2cnd:
+       i2c_put_adapter(adap);
+ei2cga:
+       return -ENODEV;
+}
+
+static void soc_camera_free_i2c(struct soc_camera_device *icd)
+{
+       struct i2c_client *client =
+               to_i2c_client(to_soc_camera_control(icd));
+       dev_set_drvdata(&icd->dev, NULL);
+       v4l2_device_unregister_subdev(i2c_get_clientdata(client));
+       i2c_unregister_device(client);
+       i2c_put_adapter(client->adapter);
 }
+#else
+#define soc_camera_init_i2c(icd, icl)  (-ENODEV)
+#define soc_camera_free_i2c(icd)       do {} while (0)
+#endif
 
+static int soc_camera_video_start(struct soc_camera_device *icd);
+static int video_dev_create(struct soc_camera_device *icd);
+/* Called during host-driver probe */
 static int soc_camera_probe(struct device *dev)
 {
        struct soc_camera_device *icd = to_soc_camera_dev(dev);
-       struct soc_camera_host *ici =
-               to_soc_camera_host(icd->dev.parent);
+       struct soc_camera_host *ici = to_soc_camera_host(dev->parent);
+       struct soc_camera_link *icl = to_soc_camera_link(icd);
+       struct device *control = NULL;
+       struct v4l2_subdev *sd;
+       struct v4l2_mbus_framefmt mf;
        int ret;
 
-       if (!icd->ops->probe)
-               return -ENODEV;
+       dev_info(dev, "Probing %s\n", dev_name(dev));
+
+       if (icl->power) {
+               ret = icl->power(icd->pdev, 1);
+               if (ret < 0) {
+                       dev_err(dev,
+                               "Platform failed to power-on the camera.\n");
+                       goto epower;
+               }
+       }
+
+       /* The camera could have been already on, try to reset */
+       if (icl->reset)
+               icl->reset(icd->pdev);
 
-       /* We only call ->add() here to activate and probe the camera.
-        * We shall ->remove() and deactivate it immediately afterwards. */
        ret = ici->ops->add(icd);
        if (ret < 0)
-               return ret;
+               goto eadd;
+
+       /* Must have icd->vdev before registering the device */
+       ret = video_dev_create(icd);
+       if (ret < 0)
+               goto evdc;
+
+       /* Non-i2c cameras, e.g., soc_camera_platform, have no board_info */
+       if (icl->board_info) {
+               ret = soc_camera_init_i2c(icd, icl);
+               if (ret < 0)
+                       goto eadddev;
+       } else if (!icl->add_device || !icl->del_device) {
+               ret = -EINVAL;
+               goto eadddev;
+       } else {
+               if (icl->module_name)
+                       ret = request_module(icl->module_name);
+
+               ret = icl->add_device(icl, &icd->dev);
+               if (ret < 0)
+                       goto eadddev;
+
+               /*
+                * FIXME: this is racy, have to use driver-binding notification,
+                * when it is available
+                */
+               control = to_soc_camera_control(icd);
+               if (!control || !control->driver || !dev_get_drvdata(control) ||
+                   !try_module_get(control->driver->owner)) {
+                       icl->del_device(icl);
+                       goto enodrv;
+               }
+       }
+
+       /* At this point client .probe() should have run already */
+       ret = soc_camera_init_user_formats(icd);
+       if (ret < 0)
+               goto eiufmt;
+
+       icd->field = V4L2_FIELD_ANY;
 
-       ret = icd->ops->probe(icd);
-       if (ret >= 0) {
-               const struct v4l2_queryctrl *qctrl;
+       /* ..._video_start() will create a device node, so we have to protect */
+       mutex_lock(&icd->video_lock);
 
-               qctrl = soc_camera_find_qctrl(icd->ops, V4L2_CID_GAIN);
-               icd->gain = qctrl ? qctrl->default_value : (unsigned short)~0;
-               qctrl = soc_camera_find_qctrl(icd->ops, V4L2_CID_EXPOSURE);
-               icd->exposure = qctrl ? qctrl->default_value :
-                       (unsigned short)~0;
+       ret = soc_camera_video_start(icd);
+       if (ret < 0)
+               goto evidstart;
+
+       /* Try to improve our guess of a reasonable window format */
+       sd = soc_camera_to_subdev(icd);
+       if (!v4l2_subdev_call(sd, video, g_mbus_fmt, &mf)) {
+               icd->user_width         = mf.width;
+               icd->user_height        = mf.height;
+               icd->colorspace         = mf.colorspace;
+               icd->field              = mf.field;
        }
+
+       /* Do we have to sysfs_remove_link() before device_unregister()? */
+       if (sysfs_create_link(&icd->dev.kobj, &to_soc_camera_control(icd)->kobj,
+                             "control"))
+               dev_warn(&icd->dev, "Failed creating the control symlink\n");
+
        ici->ops->remove(icd);
 
+       if (icl->power)
+               icl->power(icd->pdev, 0);
+
+       mutex_unlock(&icd->video_lock);
+
+       return 0;
+
+evidstart:
+       mutex_unlock(&icd->video_lock);
+       soc_camera_free_user_formats(icd);
+eiufmt:
+       if (icl->board_info) {
+               soc_camera_free_i2c(icd);
+       } else {
+               icl->del_device(icl);
+               module_put(control->driver->owner);
+       }
+enodrv:
+eadddev:
+       video_device_release(icd->vdev);
+evdc:
+       ici->ops->remove(icd);
+eadd:
+       if (icl->power)
+               icl->power(icd->pdev, 0);
+epower:
        return ret;
 }
 
-/* This is called on device_unregister, which only means we have to disconnect
- * from the host, but not remove ourselves from the device list */
+/*
+ * This is called on device_unregister, which only means we have to disconnect
+ * from the host, but not remove ourselves from the device list
+ */
 static int soc_camera_remove(struct device *dev)
 {
        struct soc_camera_device *icd = to_soc_camera_dev(dev);
+       struct soc_camera_link *icl = to_soc_camera_link(icd);
+       struct video_device *vdev = icd->vdev;
 
-       if (icd->ops->remove)
-               icd->ops->remove(icd);
+       BUG_ON(!dev->parent);
+
+       if (vdev) {
+               mutex_lock(&icd->video_lock);
+               video_unregister_device(vdev);
+               icd->vdev = NULL;
+               mutex_unlock(&icd->video_lock);
+       }
+
+       if (icl->board_info) {
+               soc_camera_free_i2c(icd);
+       } else {
+               struct device_driver *drv = to_soc_camera_control(icd) ?
+                       to_soc_camera_control(icd)->driver : NULL;
+               if (drv) {
+                       icl->del_device(icl);
+                       module_put(drv->owner);
+               }
+       }
+       soc_camera_free_user_formats(icd);
 
        return 0;
 }
 
+static int soc_camera_suspend(struct device *dev, pm_message_t state)
+{
+       struct soc_camera_device *icd = to_soc_camera_dev(dev);
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       int ret = 0;
+
+       if (ici->ops->suspend)
+               ret = ici->ops->suspend(icd, state);
+
+       return ret;
+}
+
+static int soc_camera_resume(struct device *dev)
+{
+       struct soc_camera_device *icd = to_soc_camera_dev(dev);
+       struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
+       int ret = 0;
+
+       if (ici->ops->resume)
+               ret = ici->ops->resume(icd);
+
+       return ret;
+}
+
 static struct bus_type soc_camera_bus_type = {
        .name           = "soc-camera",
        .probe          = soc_camera_probe,
        .remove         = soc_camera_remove,
+       .suspend        = soc_camera_suspend,
+       .resume         = soc_camera_resume,
 };
 
 static struct device_driver ic_drv = {
@@ -748,44 +1113,78 @@ static void dummy_release(struct device *dev)
 {
 }
 
+static int default_cropcap(struct soc_camera_device *icd,
+                          struct v4l2_cropcap *a)
+{
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+       return v4l2_subdev_call(sd, video, cropcap, a);
+}
+
+static int default_g_crop(struct soc_camera_device *icd, struct v4l2_crop *a)
+{
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+       return v4l2_subdev_call(sd, video, g_crop, a);
+}
+
+static int default_s_crop(struct soc_camera_device *icd, struct v4l2_crop *a)
+{
+       struct v4l2_subdev *sd = soc_camera_to_subdev(icd);
+       return v4l2_subdev_call(sd, video, s_crop, a);
+}
+
+static void soc_camera_device_init(struct device *dev, void *pdata)
+{
+       dev->platform_data      = pdata;
+       dev->bus                = &soc_camera_bus_type;
+       dev->release            = dummy_release;
+}
+
 int soc_camera_host_register(struct soc_camera_host *ici)
 {
-       int ret;
        struct soc_camera_host *ix;
+       int ret;
 
-       if (!ici->ops->init_videobuf || !ici->ops->add || !ici->ops->remove)
+       if (!ici || !ici->ops ||
+           !ici->ops->try_fmt ||
+           !ici->ops->set_fmt ||
+           !ici->ops->set_bus_param ||
+           !ici->ops->querycap ||
+           !ici->ops->init_videobuf ||
+           !ici->ops->reqbufs ||
+           !ici->ops->add ||
+           !ici->ops->remove ||
+           !ici->ops->poll ||
+           !ici->v4l2_dev.dev)
                return -EINVAL;
 
-       /* Number might be equal to the platform device ID */
-       sprintf(ici->dev.bus_id, "camera_host%d", ici->nr);
+       if (!ici->ops->set_crop)
+               ici->ops->set_crop = default_s_crop;
+       if (!ici->ops->get_crop)
+               ici->ops->get_crop = default_g_crop;
+       if (!ici->ops->cropcap)
+               ici->ops->cropcap = default_cropcap;
 
        mutex_lock(&list_lock);
        list_for_each_entry(ix, &hosts, list) {
                if (ix->nr == ici->nr) {
-                       mutex_unlock(&list_lock);
-                       return -EBUSY;
+                       ret = -EBUSY;
+                       goto edevreg;
                }
        }
 
+       ret = v4l2_device_register(ici->v4l2_dev.dev, &ici->v4l2_dev);
+       if (ret < 0)
+               goto edevreg;
+
        list_add_tail(&ici->list, &hosts);
        mutex_unlock(&list_lock);
 
-       ici->dev.release = dummy_release;
-
-       ret = device_register(&ici->dev);
-
-       if (ret)
-               goto edevr;
-
        scan_add_host(ici);
 
        return 0;
 
-edevr:
-       mutex_lock(&list_lock);
-       list_del(&ici->list);
+edevreg:
        mutex_unlock(&list_lock);
-
        return ret;
 }
 EXPORT_SYMBOL(soc_camera_host_register);
@@ -800,32 +1199,38 @@ void soc_camera_host_unregister(struct soc_camera_host *ici)
        list_del(&ici->list);
 
        list_for_each_entry(icd, &devices, list) {
-               if (icd->dev.parent == &ici->dev) {
+               if (icd->iface == ici->nr) {
+                       void *pdata = icd->dev.platform_data;
+                       /* The bus->remove will be called */
                        device_unregister(&icd->dev);
-                       /* Not before device_unregister(), .remove
-                        * needs parent to call ici->ops->remove() */
-                       icd->dev.parent = NULL;
-                       memset(&icd->dev.kobj, 0, sizeof(icd->dev.kobj));
+                       /*
+                        * Not before device_unregister(), .remove
+                        * needs parent to call ici->ops->remove().
+                        * If the host module is loaded again, device_register()
+                        * would complain "already initialised," since 2.6.32
+                        * this is also needed to prevent use-after-free of the
+                        * device private data.
+                        */
+                       memset(&icd->dev, 0, sizeof(icd->dev));
+                       soc_camera_device_init(&icd->dev, pdata);
                }
        }
 
        mutex_unlock(&list_lock);
 
-       device_unregister(&ici->dev);
+       v4l2_device_unregister(&ici->v4l2_dev);
 }
 EXPORT_SYMBOL(soc_camera_host_unregister);
 
 /* Image capture device */
-int soc_camera_device_register(struct soc_camera_device *icd)
+static int soc_camera_device_register(struct soc_camera_device *icd)
 {
        struct soc_camera_device *ix;
        int num = -1, i;
 
-       if (!icd)
-               return -EINVAL;
-
        for (i = 0; i < 256 && num < 0; i++) {
                num = i;
+               /* Check if this index is available on this interface */
                list_for_each_entry(ix, &devices, list) {
                        if (ix->iface == icd->iface && ix->devnum == i) {
                                num = -1;
@@ -835,116 +1240,164 @@ int soc_camera_device_register(struct soc_camera_device *icd)
        }
 
        if (num < 0)
-               /* ok, we have 256 cameras on this host...
-                * man, stay reasonable... */
+               /*
+                * ok, we have 256 cameras on this host...
+                * man, stay reasonable...
+                */
                return -ENOMEM;
 
-       icd->devnum = num;
-       icd->dev.bus = &soc_camera_bus_type;
-       snprintf(icd->dev.bus_id, sizeof(icd->dev.bus_id),
-                "%u-%u", icd->iface, icd->devnum);
+       icd->devnum             = num;
+       icd->use_count          = 0;
+       icd->host_priv          = NULL;
+       mutex_init(&icd->video_lock);
 
-       icd->dev.release = dummy_release;
+       list_add_tail(&icd->list, &devices);
 
-       return scan_add_device(icd);
+       return 0;
 }
-EXPORT_SYMBOL(soc_camera_device_register);
 
-void soc_camera_device_unregister(struct soc_camera_device *icd)
+static void soc_camera_device_unregister(struct soc_camera_device *icd)
 {
-       mutex_lock(&list_lock);
        list_del(&icd->list);
-
-       /* The bus->remove will be eventually called */
-       if (icd->dev.parent)
-               device_unregister(&icd->dev);
-       mutex_unlock(&list_lock);
 }
-EXPORT_SYMBOL(soc_camera_device_unregister);
 
-int soc_camera_video_start(struct soc_camera_device *icd)
+static const struct v4l2_ioctl_ops soc_camera_ioctl_ops = {
+       .vidioc_querycap         = soc_camera_querycap,
+       .vidioc_g_fmt_vid_cap    = soc_camera_g_fmt_vid_cap,
+       .vidioc_enum_fmt_vid_cap = soc_camera_enum_fmt_vid_cap,
+       .vidioc_s_fmt_vid_cap    = soc_camera_s_fmt_vid_cap,
+       .vidioc_enum_input       = soc_camera_enum_input,
+       .vidioc_g_input          = soc_camera_g_input,
+       .vidioc_s_input          = soc_camera_s_input,
+       .vidioc_s_std            = soc_camera_s_std,
+       .vidioc_reqbufs          = soc_camera_reqbufs,
+       .vidioc_try_fmt_vid_cap  = soc_camera_try_fmt_vid_cap,
+       .vidioc_querybuf         = soc_camera_querybuf,
+       .vidioc_qbuf             = soc_camera_qbuf,
+       .vidioc_dqbuf            = soc_camera_dqbuf,
+       .vidioc_streamon         = soc_camera_streamon,
+       .vidioc_streamoff        = soc_camera_streamoff,
+       .vidioc_queryctrl        = soc_camera_queryctrl,
+       .vidioc_g_ctrl           = soc_camera_g_ctrl,
+       .vidioc_s_ctrl           = soc_camera_s_ctrl,
+       .vidioc_cropcap          = soc_camera_cropcap,
+       .vidioc_g_crop           = soc_camera_g_crop,
+       .vidioc_s_crop           = soc_camera_s_crop,
+       .vidioc_g_parm           = soc_camera_g_parm,
+       .vidioc_s_parm           = soc_camera_s_parm,
+       .vidioc_g_chip_ident     = soc_camera_g_chip_ident,
+#ifdef CONFIG_VIDEO_ADV_DEBUG
+       .vidioc_g_register       = soc_camera_g_register,
+       .vidioc_s_register       = soc_camera_s_register,
+#endif
+};
+
+static int video_dev_create(struct soc_camera_device *icd)
 {
        struct soc_camera_host *ici = to_soc_camera_host(icd->dev.parent);
-       int err = -ENOMEM;
-       struct video_device *vdev;
-
-       if (!icd->dev.parent)
-               return -ENODEV;
+       struct video_device *vdev = video_device_alloc();
 
-       vdev = video_device_alloc();
        if (!vdev)
-               goto evidallocd;
-       dev_dbg(&ici->dev, "Allocated video_device %p\n", vdev);
+               return -ENOMEM;
 
        strlcpy(vdev->name, ici->drv_name, sizeof(vdev->name));
-       /* Maybe better &ici->dev */
+
        vdev->parent            = &icd->dev;
-       vdev->type              = VID_TYPE_CAPTURE;
        vdev->current_norm      = V4L2_STD_UNKNOWN;
        vdev->fops              = &soc_camera_fops;
+       vdev->ioctl_ops         = &soc_camera_ioctl_ops;
        vdev->release           = video_device_release;
-       vdev->minor             = -1;
-       vdev->tvnorms           = V4L2_STD_UNKNOWN,
-       vdev->vidioc_querycap   = soc_camera_querycap;
-       vdev->vidioc_g_fmt_vid_cap = soc_camera_g_fmt_vid_cap;
-       vdev->vidioc_enum_fmt_vid_cap = soc_camera_enum_fmt_vid_cap;
-       vdev->vidioc_s_fmt_vid_cap = soc_camera_s_fmt_vid_cap;
-       vdev->vidioc_enum_input = soc_camera_enum_input;
-       vdev->vidioc_g_input    = soc_camera_g_input;
-       vdev->vidioc_s_input    = soc_camera_s_input;
-       vdev->vidioc_s_std      = soc_camera_s_std;
-       vdev->vidioc_reqbufs    = soc_camera_reqbufs;
-       vdev->vidioc_try_fmt_vid_cap = soc_camera_try_fmt_vid_cap;
-       vdev->vidioc_querybuf   = soc_camera_querybuf;
-       vdev->vidioc_qbuf       = soc_camera_qbuf;
-       vdev->vidioc_dqbuf      = soc_camera_dqbuf;
-       vdev->vidioc_streamon   = soc_camera_streamon;
-       vdev->vidioc_streamoff  = soc_camera_streamoff;
-       vdev->vidioc_queryctrl  = soc_camera_queryctrl;
-       vdev->vidioc_g_ctrl     = soc_camera_g_ctrl;
-       vdev->vidioc_s_ctrl     = soc_camera_s_ctrl;
-       vdev->vidioc_cropcap    = soc_camera_cropcap;
-       vdev->vidioc_g_crop     = soc_camera_g_crop;
-       vdev->vidioc_s_crop     = soc_camera_s_crop;
-       vdev->vidioc_g_chip_ident = soc_camera_g_chip_ident;
-#ifdef CONFIG_VIDEO_ADV_DEBUG
-       vdev->vidioc_g_register = soc_camera_g_register;
-       vdev->vidioc_s_register = soc_camera_s_register;
-#endif
+       vdev->tvnorms           = V4L2_STD_UNKNOWN;
+
+       icd->vdev = vdev;
+
+       return 0;
+}
 
-       icd->current_fmt = &icd->formats[0];
+/*
+ * Called from soc_camera_probe() above (with .video_lock held???)
+ */
+static int soc_camera_video_start(struct soc_camera_device *icd)
+{
+       int ret;
 
-       err = video_register_device(vdev, VFL_TYPE_GRABBER, vdev->minor);
-       if (err < 0) {
-               dev_err(vdev->parent, "video_register_device failed\n");
-               goto evidregd;
+       if (!icd->dev.parent)
+               return -ENODEV;
+
+       if (!icd->ops ||
+           !icd->ops->query_bus_param ||
+           !icd->ops->set_bus_param)
+               return -EINVAL;
+
+       ret = video_register_device(icd->vdev, VFL_TYPE_GRABBER, -1);
+       if (ret < 0) {
+               dev_err(&icd->dev, "video_register_device failed: %d\n", ret);
+               return ret;
        }
-       icd->vdev = vdev;
 
        return 0;
+}
 
-evidregd:
-       video_device_release(vdev);
-evidallocd:
-       return err;
+static int __devinit soc_camera_pdrv_probe(struct platform_device *pdev)
+{
+       struct soc_camera_link *icl = pdev->dev.platform_data;
+       struct soc_camera_device *icd;
+       int ret;
+
+       if (!icl)
+               return -EINVAL;
+
+       icd = kzalloc(sizeof(*icd), GFP_KERNEL);
+       if (!icd)
+               return -ENOMEM;
+
+       icd->iface = icl->bus_id;
+       icd->pdev = &pdev->dev;
+       platform_set_drvdata(pdev, icd);
+
+       ret = soc_camera_device_register(icd);
+       if (ret < 0)
+               goto escdevreg;
+
+       soc_camera_device_init(&icd->dev, icl);
+
+       icd->user_width         = DEFAULT_WIDTH;
+       icd->user_height        = DEFAULT_HEIGHT;
+
+       return 0;
+
+escdevreg:
+       kfree(icd);
+
+       return ret;
 }
-EXPORT_SYMBOL(soc_camera_video_start);
 
-void soc_camera_video_stop(struct soc_camera_device *icd)
+/*
+ * Only called on rmmod for each platform device, since they are not
+ * hot-pluggable. Now we know, that all our users - hosts and devices have
+ * been unloaded already
+ */
+static int __devexit soc_camera_pdrv_remove(struct platform_device *pdev)
 {
-       struct video_device *vdev = icd->vdev;
+       struct soc_camera_device *icd = platform_get_drvdata(pdev);
+
+       if (!icd)
+               return -EINVAL;
 
-       dev_dbg(&icd->dev, "%s\n", __func__);
+       soc_camera_device_unregister(icd);
 
-       if (!icd->dev.parent || !vdev)
-               return;
+       kfree(icd);
 
-       mutex_lock(&video_lock);
-       video_unregister_device(vdev);
-       icd->vdev = NULL;
-       mutex_unlock(&video_lock);
+       return 0;
 }
-EXPORT_SYMBOL(soc_camera_video_stop);
+
+static struct platform_driver __refdata soc_camera_pdrv = {
+       .remove  = __devexit_p(soc_camera_pdrv_remove),
+       .driver  = {
+               .name   = "soc-camera-pdrv",
+               .owner  = THIS_MODULE,
+       },
+};
 
 static int __init soc_camera_init(void)
 {
@@ -955,8 +1408,14 @@ static int __init soc_camera_init(void)
        if (ret)
                goto edrvr;
 
+       ret = platform_driver_probe(&soc_camera_pdrv, soc_camera_pdrv_probe);
+       if (ret)
+               goto epdr;
+
        return 0;
 
+epdr:
+       driver_unregister(&ic_drv);
 edrvr:
        bus_unregister(&soc_camera_bus_type);
        return ret;
@@ -964,6 +1423,7 @@ edrvr:
 
 static void __exit soc_camera_exit(void)
 {
+       platform_driver_unregister(&soc_camera_pdrv);
        driver_unregister(&ic_drv);
        bus_unregister(&soc_camera_bus_type);
 }
@@ -974,3 +1434,4 @@ module_exit(soc_camera_exit);
 MODULE_DESCRIPTION("Image capture bus driver");
 MODULE_AUTHOR("Guennadi Liakhovetski <kernel@pengutronix.de>");
 MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:soc-camera-pdrv");