V4L/DVB: gspca_sonixb: Add support for camera button
[safe/jmp/linux-2.6] / drivers / media / video / mt9m111.c
index 4844486..d35f536 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Driver for MT9M111 CMOS Image Sensor from Micron
+ * Driver for MT9M111/MT9M112 CMOS Image Sensor from Micron
  *
  * Copyright (C) 2008, Robert Jarzmik <robert.jarzmik@free.fr>
  *
@@ -19,7 +19,7 @@
 #include <media/soc_camera.h>
 
 /*
- * mt9m111 i2c address is 0x5d or 0x48 (depending on SAddr pin)
+ * mt9m111 and mt9m112 i2c address is 0x5d or 0x48 (depending on SAddr pin)
  * The platform has to define i2c_board_info and call i2c_register_board_info()
  */
 
@@ -90,7 +90,7 @@
 #define MT9M111_OUTPUT_FORMAT_CTRL2_B  0x19b
 
 #define MT9M111_OPMODE_AUTOEXPO_EN     (1 << 14)
-
+#define MT9M111_OPMODE_AUTOWHITEBAL_EN (1 << 1)
 
 #define MT9M111_OUTFMT_PROCESSED_BAYER (1 << 14)
 #define MT9M111_OUTFMT_BYPASS_IFP      (1 << 10)
  * mt9m111: Camera control register addresses (0x200..0x2ff not implemented)
  */
 
-#define reg_read(reg) mt9m111_reg_read(icd, MT9M111_##reg)
-#define reg_write(reg, val) mt9m111_reg_write(icd, MT9M111_##reg, (val))
-#define reg_set(reg, val) mt9m111_reg_set(icd, MT9M111_##reg, (val))
-#define reg_clear(reg, val) mt9m111_reg_clear(icd, MT9M111_##reg, (val))
+#define reg_read(reg) mt9m111_reg_read(client, MT9M111_##reg)
+#define reg_write(reg, val) mt9m111_reg_write(client, MT9M111_##reg, (val))
+#define reg_set(reg, val) mt9m111_reg_set(client, MT9M111_##reg, (val))
+#define reg_clear(reg, val) mt9m111_reg_clear(client, MT9M111_##reg, (val))
 
 #define MT9M111_MIN_DARK_ROWS  8
 #define MT9M111_MIN_DARK_COLS  24
 #define MT9M111_MAX_HEIGHT     1024
 #define MT9M111_MAX_WIDTH      1280
 
-#define COL_FMT(_name, _depth, _fourcc, _colorspace) \
-       { .name = _name, .depth = _depth, .fourcc = _fourcc, \
-       .colorspace = _colorspace }
-#define RGB_FMT(_name, _depth, _fourcc) \
-       COL_FMT(_name, _depth, _fourcc, V4L2_COLORSPACE_SRGB)
-
-static const struct soc_camera_data_format mt9m111_colour_formats[] = {
-       COL_FMT("YCrYCb 8 bit", 8, V4L2_PIX_FMT_YUYV, V4L2_COLORSPACE_JPEG),
-       RGB_FMT("RGB 565", 16, V4L2_PIX_FMT_RGB565),
-       RGB_FMT("RGB 555", 16, V4L2_PIX_FMT_RGB555),
-       RGB_FMT("Bayer (sRGB) 10 bit", 10, V4L2_PIX_FMT_SBGGR16),
-       RGB_FMT("Bayer (sRGB) 8 bit", 8, V4L2_PIX_FMT_SBGGR8),
+/* MT9M111 has only one fixed colorspace per pixelcode */
+struct mt9m111_datafmt {
+       enum v4l2_mbus_pixelcode        code;
+       enum v4l2_colorspace            colorspace;
+};
+
+/* Find a data format by a pixel code in an array */
+static const struct mt9m111_datafmt *mt9m111_find_datafmt(
+       enum v4l2_mbus_pixelcode code, const struct mt9m111_datafmt *fmt,
+       int n)
+{
+       int i;
+       for (i = 0; i < n; i++)
+               if (fmt[i].code == code)
+                       return fmt + i;
+
+       return NULL;
+}
+
+static const struct mt9m111_datafmt mt9m111_colour_fmts[] = {
+       {V4L2_MBUS_FMT_YUYV8_2X8_LE, V4L2_COLORSPACE_JPEG},
+       {V4L2_MBUS_FMT_YVYU8_2X8_LE, V4L2_COLORSPACE_JPEG},
+       {V4L2_MBUS_FMT_YUYV8_2X8_BE, V4L2_COLORSPACE_JPEG},
+       {V4L2_MBUS_FMT_YVYU8_2X8_BE, V4L2_COLORSPACE_JPEG},
+       {V4L2_MBUS_FMT_RGB555_2X8_PADHI_LE, V4L2_COLORSPACE_SRGB},
+       {V4L2_MBUS_FMT_RGB565_2X8_LE, V4L2_COLORSPACE_SRGB},
+       {V4L2_MBUS_FMT_SBGGR8_1X8, V4L2_COLORSPACE_SRGB},
+       {V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_LE, V4L2_COLORSPACE_SRGB},
 };
 
 enum mt9m111_context {
@@ -143,12 +159,12 @@ enum mt9m111_context {
 };
 
 struct mt9m111 {
-       struct i2c_client *client;
-       struct soc_camera_device icd;
-       int model;      /* V4L2_IDENT_MT9M111* codes from v4l2-chip-ident.h */
+       struct v4l2_subdev subdev;
+       int model;      /* V4L2_IDENT_MT9M11x* codes from v4l2-chip-ident.h */
        enum mt9m111_context context;
-       unsigned int left, top, width, height;
-       u32 pixfmt;
+       struct v4l2_rect rect;
+       const struct mt9m111_datafmt *fmt;
+       unsigned int gain;
        unsigned char autoexposure;
        unsigned char datawidth;
        unsigned int powered:1;
@@ -158,8 +174,14 @@ struct mt9m111 {
        unsigned int swap_rgb_red_blue:1;
        unsigned int swap_yuv_y_chromas:1;
        unsigned int swap_yuv_cb_cr:1;
+       unsigned int autowhitebalance:1;
 };
 
+static struct mt9m111 *to_mt9m111(const struct i2c_client *client)
+{
+       return container_of(i2c_get_clientdata(client), struct mt9m111, subdev);
+}
+
 static int reg_page_map_set(struct i2c_client *client, const u16 reg)
 {
        int ret;
@@ -178,56 +200,52 @@ static int reg_page_map_set(struct i2c_client *client, const u16 reg)
        return ret;
 }
 
-static int mt9m111_reg_read(struct soc_camera_device *icd, const u16 reg)
+static int mt9m111_reg_read(struct i2c_client *client, const u16 reg)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
-       struct i2c_client *client = mt9m111->client;
        int ret;
 
        ret = reg_page_map_set(client, reg);
        if (!ret)
-               ret = swab16(i2c_smbus_read_word_data(client, (reg & 0xff)));
+               ret = swab16(i2c_smbus_read_word_data(client, reg & 0xff));
 
-       dev_dbg(&icd->dev, "read  reg.%03x -> %04x\n", reg, ret);
+       dev_dbg(&client->dev, "read  reg.%03x -> %04x\n", reg, ret);
        return ret;
 }
 
-static int mt9m111_reg_write(struct soc_camera_device *icd, const u16 reg,
+static int mt9m111_reg_write(struct i2c_client *client, const u16 reg,
                             const u16 data)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
-       struct i2c_client *client = mt9m111->client;
        int ret;
 
        ret = reg_page_map_set(client, reg);
        if (!ret)
-               ret = i2c_smbus_write_word_data(mt9m111->client, (reg & 0xff),
+               ret = i2c_smbus_write_word_data(client, reg & 0xff,
                                                swab16(data));
-       dev_dbg(&icd->dev, "write reg.%03x = %04x -> %d\n", reg, data, ret);
+       dev_dbg(&client->dev, "write reg.%03x = %04x -> %d\n", reg, data, ret);
        return ret;
 }
 
-static int mt9m111_reg_set(struct soc_camera_device *icd, const u16 reg,
+static int mt9m111_reg_set(struct i2c_client *client, const u16 reg,
                           const u16 data)
 {
        int ret;
 
-       ret = mt9m111_reg_read(icd, reg);
+       ret = mt9m111_reg_read(client, reg);
        if (ret >= 0)
-               ret = mt9m111_reg_write(icd, reg, ret | data);
+               ret = mt9m111_reg_write(client, reg, ret | data);
        return ret;
 }
 
-static int mt9m111_reg_clear(struct soc_camera_device *icd, const u16 reg,
+static int mt9m111_reg_clear(struct i2c_client *client, const u16 reg,
                             const u16 data)
 {
        int ret;
 
-       ret = mt9m111_reg_read(icd, reg);
-       return mt9m111_reg_write(icd, reg, ret & ~data);
+       ret = mt9m111_reg_read(client, reg);
+       return mt9m111_reg_write(client, reg, ret & ~data);
 }
 
-static int mt9m111_set_context(struct soc_camera_device *icd,
+static int mt9m111_set_context(struct i2c_client *client,
                               enum mt9m111_context ctxt)
 {
        int valB = MT9M111_CTXT_CTRL_RESTART | MT9M111_CTXT_CTRL_DEFECTCOR_B
@@ -243,22 +261,23 @@ static int mt9m111_set_context(struct soc_camera_device *icd,
                return reg_write(CONTEXT_CONTROL, valA);
 }
 
-static int mt9m111_setup_rect(struct soc_camera_device *icd)
+static int mt9m111_setup_rect(struct i2c_client *client,
+                             struct v4l2_rect *rect)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret, is_raw_format;
-       int width = mt9m111->width;
-       int height = mt9m111->height;
+       int width = rect->width;
+       int height = rect->height;
 
-       if ((mt9m111->pixfmt == V4L2_PIX_FMT_SBGGR8)
-           || (mt9m111->pixfmt == V4L2_PIX_FMT_SBGGR16))
+       if (mt9m111->fmt->code == V4L2_MBUS_FMT_SBGGR8_1X8 ||
+           mt9m111->fmt->code == V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_LE)
                is_raw_format = 1;
        else
                is_raw_format = 0;
 
-       ret = reg_write(COLUMN_START, mt9m111->left);
+       ret = reg_write(COLUMN_START, rect->left);
        if (!ret)
-               ret = reg_write(ROW_START, mt9m111->top);
+               ret = reg_write(ROW_START, rect->top);
 
        if (is_raw_format) {
                if (!ret)
@@ -287,7 +306,7 @@ static int mt9m111_setup_rect(struct soc_camera_device *icd)
        return ret;
 }
 
-static int mt9m111_setup_pixfmt(struct soc_camera_device *icd, u16 outfmt)
+static int mt9m111_setup_pixfmt(struct i2c_client *client, u16 outfmt)
 {
        int ret;
 
@@ -297,19 +316,20 @@ static int mt9m111_setup_pixfmt(struct soc_camera_device *icd, u16 outfmt)
        return ret;
 }
 
-static int mt9m111_setfmt_bayer8(struct soc_camera_device *icd)
+static int mt9m111_setfmt_bayer8(struct i2c_client *client)
 {
-       return mt9m111_setup_pixfmt(icd, MT9M111_OUTFMT_PROCESSED_BAYER);
+       return mt9m111_setup_pixfmt(client, MT9M111_OUTFMT_PROCESSED_BAYER |
+                                   MT9M111_OUTFMT_RGB);
 }
 
-static int mt9m111_setfmt_bayer10(struct soc_camera_device *icd)
+static int mt9m111_setfmt_bayer10(struct i2c_client *client)
 {
-       return mt9m111_setup_pixfmt(icd, MT9M111_OUTFMT_BYPASS_IFP);
+       return mt9m111_setup_pixfmt(client, MT9M111_OUTFMT_BYPASS_IFP);
 }
 
-static int mt9m111_setfmt_rgb565(struct soc_camera_device *icd)
+static int mt9m111_setfmt_rgb565(struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int val = 0;
 
        if (mt9m111->swap_rgb_red_blue)
@@ -318,12 +338,12 @@ static int mt9m111_setfmt_rgb565(struct soc_camera_device *icd)
                val |= MT9M111_OUTFMT_SWAP_RGB_EVEN;
        val |= MT9M111_OUTFMT_RGB | MT9M111_OUTFMT_RGB565;
 
-       return mt9m111_setup_pixfmt(icd, val);
+       return mt9m111_setup_pixfmt(client, val);
 }
 
-static int mt9m111_setfmt_rgb555(struct soc_camera_device *icd)
+static int mt9m111_setfmt_rgb555(struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int val = 0;
 
        if (mt9m111->swap_rgb_red_blue)
@@ -332,12 +352,12 @@ static int mt9m111_setfmt_rgb555(struct soc_camera_device *icd)
                val |= MT9M111_OUTFMT_SWAP_RGB_EVEN;
        val |= MT9M111_OUTFMT_RGB | MT9M111_OUTFMT_RGB555;
 
-       return mt9m111_setup_pixfmt(icd, val);
+       return mt9m111_setup_pixfmt(client, val);
 }
 
-static int mt9m111_setfmt_yuv(struct soc_camera_device *icd)
+static int mt9m111_setfmt_yuv(struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int val = 0;
 
        if (mt9m111->swap_yuv_cb_cr)
@@ -345,154 +365,260 @@ static int mt9m111_setfmt_yuv(struct soc_camera_device *icd)
        if (mt9m111->swap_yuv_y_chromas)
                val |= MT9M111_OUTFMT_SWAP_YCbCr_C_Y;
 
-       return mt9m111_setup_pixfmt(icd, val);
+       return mt9m111_setup_pixfmt(client, val);
 }
 
-static int mt9m111_enable(struct soc_camera_device *icd)
+static int mt9m111_enable(struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
-       struct soc_camera_link *icl = mt9m111->client->dev.platform_data;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret;
 
-       if (icl->power) {
-               ret = icl->power(&mt9m111->client->dev, 1);
-               if (ret < 0) {
-                       dev_err(icd->vdev->parent,
-                               "Platform failed to power-on the camera.\n");
-                       return ret;
-               }
-       }
-
        ret = reg_set(RESET, MT9M111_RESET_CHIP_ENABLE);
        if (!ret)
                mt9m111->powered = 1;
        return ret;
 }
 
-static int mt9m111_disable(struct soc_camera_device *icd)
+static int mt9m111_reset(struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
-       struct soc_camera_link *icl = mt9m111->client->dev.platform_data;
        int ret;
 
-       ret = reg_clear(RESET, MT9M111_RESET_CHIP_ENABLE);
+       ret = reg_set(RESET, MT9M111_RESET_RESET_MODE);
        if (!ret)
-               mt9m111->powered = 0;
-
-       if (icl->power)
-               icl->power(&mt9m111->client->dev, 0);
+               ret = reg_set(RESET, MT9M111_RESET_RESET_SOC);
+       if (!ret)
+               ret = reg_clear(RESET, MT9M111_RESET_RESET_MODE
+                               | MT9M111_RESET_RESET_SOC);
 
        return ret;
 }
 
-static int mt9m111_reset(struct soc_camera_device *icd)
+static unsigned long mt9m111_query_bus_param(struct soc_camera_device *icd)
 {
+       struct soc_camera_link *icl = to_soc_camera_link(icd);
+       unsigned long flags = SOCAM_MASTER | SOCAM_PCLK_SAMPLE_RISING |
+               SOCAM_HSYNC_ACTIVE_HIGH | SOCAM_VSYNC_ACTIVE_HIGH |
+               SOCAM_DATA_ACTIVE_HIGH | SOCAM_DATAWIDTH_8;
+
+       return soc_camera_apply_sensor_flags(icl, flags);
+}
+
+static int mt9m111_set_bus_param(struct soc_camera_device *icd, unsigned long f)
+{
+       return 0;
+}
+
+static int mt9m111_make_rect(struct i2c_client *client,
+                            struct v4l2_rect *rect)
+{
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+
+       if (mt9m111->fmt->code == V4L2_MBUS_FMT_SBGGR8_1X8 ||
+           mt9m111->fmt->code == V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_LE) {
+               /* Bayer format - even size lengths */
+               rect->width     = ALIGN(rect->width, 2);
+               rect->height    = ALIGN(rect->height, 2);
+               /* Let the user play with the starting pixel */
+       }
+
+       /* FIXME: the datasheet doesn't specify minimum sizes */
+       soc_camera_limit_side(&rect->left, &rect->width,
+                    MT9M111_MIN_DARK_COLS, 2, MT9M111_MAX_WIDTH);
+
+       soc_camera_limit_side(&rect->top, &rect->height,
+                    MT9M111_MIN_DARK_ROWS, 2, MT9M111_MAX_HEIGHT);
+
+       return mt9m111_setup_rect(client, rect);
+}
+
+static int mt9m111_s_crop(struct v4l2_subdev *sd, struct v4l2_crop *a)
+{
+       struct v4l2_rect rect = a->c;
+       struct i2c_client *client = sd->priv;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret;
 
-       ret = reg_set(RESET, MT9M111_RESET_RESET_MODE);
-       if (!ret)
-               ret = reg_set(RESET, MT9M111_RESET_RESET_SOC);
+       dev_dbg(&client->dev, "%s left=%d, top=%d, width=%d, height=%d\n",
+               __func__, rect.left, rect.top, rect.width, rect.height);
+
+       ret = mt9m111_make_rect(client, &rect);
        if (!ret)
-               ret = reg_clear(RESET, MT9M111_RESET_RESET_MODE
-                               | MT9M111_RESET_RESET_SOC);
+               mt9m111->rect = rect;
        return ret;
 }
 
-static int mt9m111_start_capture(struct soc_camera_device *icd)
+static int mt9m111_g_crop(struct v4l2_subdev *sd, struct v4l2_crop *a)
 {
+       struct i2c_client *client = sd->priv;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+
+       a->c    = mt9m111->rect;
+       a->type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
+
        return 0;
 }
 
-static int mt9m111_stop_capture(struct soc_camera_device *icd)
+static int mt9m111_cropcap(struct v4l2_subdev *sd, struct v4l2_cropcap *a)
 {
+       a->bounds.left                  = MT9M111_MIN_DARK_COLS;
+       a->bounds.top                   = MT9M111_MIN_DARK_ROWS;
+       a->bounds.width                 = MT9M111_MAX_WIDTH;
+       a->bounds.height                = MT9M111_MAX_HEIGHT;
+       a->defrect                      = a->bounds;
+       a->type                         = V4L2_BUF_TYPE_VIDEO_CAPTURE;
+       a->pixelaspect.numerator        = 1;
+       a->pixelaspect.denominator      = 1;
+
        return 0;
 }
 
-static unsigned long mt9m111_query_bus_param(struct soc_camera_device *icd)
+static int mt9m111_g_fmt(struct v4l2_subdev *sd,
+                        struct v4l2_mbus_framefmt *mf)
 {
-       return SOCAM_MASTER | SOCAM_PCLK_SAMPLE_RISING |
-               SOCAM_HSYNC_ACTIVE_HIGH | SOCAM_VSYNC_ACTIVE_HIGH |
-               SOCAM_DATAWIDTH_8;
-}
+       struct i2c_client *client = sd->priv;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+
+       mf->width       = mt9m111->rect.width;
+       mf->height      = mt9m111->rect.height;
+       mf->code        = mt9m111->fmt->code;
+       mf->field       = V4L2_FIELD_NONE;
 
-static int mt9m111_set_bus_param(struct soc_camera_device *icd, unsigned long f)
-{
        return 0;
 }
 
-static int mt9m111_set_pixfmt(struct soc_camera_device *icd, u32 pixfmt)
+static int mt9m111_set_pixfmt(struct i2c_client *client,
+                             enum v4l2_mbus_pixelcode code)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret;
 
-       switch (pixfmt) {
-       case V4L2_PIX_FMT_SBGGR8:
-               ret = mt9m111_setfmt_bayer8(icd);
+       switch (code) {
+       case V4L2_MBUS_FMT_SBGGR8_1X8:
+               ret = mt9m111_setfmt_bayer8(client);
                break;
-       case V4L2_PIX_FMT_SBGGR16:
-               ret = mt9m111_setfmt_bayer10(icd);
+       case V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_LE:
+               ret = mt9m111_setfmt_bayer10(client);
                break;
-       case V4L2_PIX_FMT_RGB555:
-               ret = mt9m111_setfmt_rgb555(icd);
+       case V4L2_MBUS_FMT_RGB555_2X8_PADHI_LE:
+               ret = mt9m111_setfmt_rgb555(client);
                break;
-       case V4L2_PIX_FMT_RGB565:
-               ret = mt9m111_setfmt_rgb565(icd);
+       case V4L2_MBUS_FMT_RGB565_2X8_LE:
+               ret = mt9m111_setfmt_rgb565(client);
                break;
-       case V4L2_PIX_FMT_YUYV:
-               ret = mt9m111_setfmt_yuv(icd);
+       case V4L2_MBUS_FMT_YUYV8_2X8_BE:
+               mt9m111->swap_yuv_y_chromas = 0;
+               mt9m111->swap_yuv_cb_cr = 0;
+               ret = mt9m111_setfmt_yuv(client);
+               break;
+       case V4L2_MBUS_FMT_YVYU8_2X8_BE:
+               mt9m111->swap_yuv_y_chromas = 0;
+               mt9m111->swap_yuv_cb_cr = 1;
+               ret = mt9m111_setfmt_yuv(client);
+               break;
+       case V4L2_MBUS_FMT_YUYV8_2X8_LE:
+               mt9m111->swap_yuv_y_chromas = 1;
+               mt9m111->swap_yuv_cb_cr = 0;
+               ret = mt9m111_setfmt_yuv(client);
+               break;
+       case V4L2_MBUS_FMT_YVYU8_2X8_LE:
+               mt9m111->swap_yuv_y_chromas = 1;
+               mt9m111->swap_yuv_cb_cr = 1;
+               ret = mt9m111_setfmt_yuv(client);
                break;
        default:
-               dev_err(&icd->dev, "Pixel format not handled : %x\n", pixfmt);
+               dev_err(&client->dev, "Pixel format not handled : %x\n",
+                       code);
                ret = -EINVAL;
        }
 
-       if (!ret)
-               mt9m111->pixfmt = pixfmt;
-
        return ret;
 }
 
-static int mt9m111_set_fmt_cap(struct soc_camera_device *icd,
-                              __u32 pixfmt, struct v4l2_rect *rect)
+static int mt9m111_s_fmt(struct v4l2_subdev *sd,
+                        struct v4l2_mbus_framefmt *mf)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct i2c_client *client = sd->priv;
+       const struct mt9m111_datafmt *fmt;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+       struct v4l2_rect rect = {
+               .left   = mt9m111->rect.left,
+               .top    = mt9m111->rect.top,
+               .width  = mf->width,
+               .height = mf->height,
+       };
        int ret;
 
-       mt9m111->left = rect->left;
-       mt9m111->top = rect->top;
-       mt9m111->width = rect->width;
-       mt9m111->height = rect->height;
+       fmt = mt9m111_find_datafmt(mf->code, mt9m111_colour_fmts,
+                                  ARRAY_SIZE(mt9m111_colour_fmts));
+       if (!fmt)
+               return -EINVAL;
 
-       dev_dbg(&icd->dev, "%s fmt=%x left=%d, top=%d, width=%d, height=%d\n",
-               __func__, pixfmt, mt9m111->left, mt9m111->top, mt9m111->width,
-               mt9m111->height);
+       dev_dbg(&client->dev,
+               "%s code=%x left=%d, top=%d, width=%d, height=%d\n", __func__,
+               mf->code, rect.left, rect.top, rect.width, rect.height);
 
-       ret = mt9m111_setup_rect(icd);
+       ret = mt9m111_make_rect(client, &rect);
        if (!ret)
-               ret = mt9m111_set_pixfmt(icd, pixfmt);
+               ret = mt9m111_set_pixfmt(client, mf->code);
+       if (!ret) {
+               mt9m111->rect   = rect;
+               mt9m111->fmt    = fmt;
+               mf->colorspace  = fmt->colorspace;
+       }
+
        return ret;
 }
 
-static int mt9m111_try_fmt_cap(struct soc_camera_device *icd,
-                              struct v4l2_format *f)
+static int mt9m111_try_fmt(struct v4l2_subdev *sd,
+                          struct v4l2_mbus_framefmt *mf)
 {
-       if (f->fmt.pix.height > MT9M111_MAX_HEIGHT)
-               f->fmt.pix.height = MT9M111_MAX_HEIGHT;
-       if (f->fmt.pix.width > MT9M111_MAX_WIDTH)
-               f->fmt.pix.width = MT9M111_MAX_WIDTH;
+       struct i2c_client *client = sd->priv;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+       const struct mt9m111_datafmt *fmt;
+       bool bayer = mf->code == V4L2_MBUS_FMT_SBGGR8_1X8 ||
+               mf->code == V4L2_MBUS_FMT_SBGGR10_2X8_PADHI_LE;
+
+       fmt = mt9m111_find_datafmt(mf->code, mt9m111_colour_fmts,
+                                  ARRAY_SIZE(mt9m111_colour_fmts));
+       if (!fmt) {
+               fmt = mt9m111->fmt;
+               mf->code = fmt->code;
+       }
+
+       /*
+        * With Bayer format enforce even side lengths, but let the user play
+        * with the starting pixel
+        */
+
+       if (mf->height > MT9M111_MAX_HEIGHT)
+               mf->height = MT9M111_MAX_HEIGHT;
+       else if (mf->height < 2)
+               mf->height = 2;
+       else if (bayer)
+               mf->height = ALIGN(mf->height, 2);
+
+       if (mf->width > MT9M111_MAX_WIDTH)
+               mf->width = MT9M111_MAX_WIDTH;
+       else if (mf->width < 2)
+               mf->width = 2;
+       else if (bayer)
+               mf->width = ALIGN(mf->width, 2);
+
+       mf->colorspace = fmt->colorspace;
 
        return 0;
 }
 
-static int mt9m111_get_chip_id(struct soc_camera_device *icd,
-                              struct v4l2_chip_ident *id)
+static int mt9m111_g_chip_ident(struct v4l2_subdev *sd,
+                               struct v4l2_dbg_chip_ident *id)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct i2c_client *client = sd->priv;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
 
-       if (id->match_type != V4L2_CHIP_MATCH_I2C_ADDR)
+       if (id->match.type != V4L2_CHIP_MATCH_I2C_ADDR)
                return -EINVAL;
 
-       if (id->match_chip != mt9m111->client->addr)
+       if (id->match.addr != client->addr)
                return -ENODEV;
 
        id->ident       = mt9m111->model;
@@ -502,19 +628,19 @@ static int mt9m111_get_chip_id(struct soc_camera_device *icd,
 }
 
 #ifdef CONFIG_VIDEO_ADV_DEBUG
-static int mt9m111_get_register(struct soc_camera_device *icd,
-                               struct v4l2_register *reg)
+static int mt9m111_g_register(struct v4l2_subdev *sd,
+                             struct v4l2_dbg_register *reg)
 {
+       struct i2c_client *client = sd->priv;
        int val;
 
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
-
-       if (reg->match_type != V4L2_CHIP_MATCH_I2C_ADDR || reg->reg > 0x2ff)
+       if (reg->match.type != V4L2_CHIP_MATCH_I2C_ADDR || reg->reg > 0x2ff)
                return -EINVAL;
-       if (reg->match_chip != mt9m111->client->addr)
+       if (reg->match.addr != client->addr)
                return -ENODEV;
 
-       val = mt9m111_reg_read(icd, reg->reg);
+       val = mt9m111_reg_read(client, reg->reg);
+       reg->size = 2;
        reg->val = (u64)val;
 
        if (reg->val > 0xffff)
@@ -523,18 +649,18 @@ static int mt9m111_get_register(struct soc_camera_device *icd,
        return 0;
 }
 
-static int mt9m111_set_register(struct soc_camera_device *icd,
-                               struct v4l2_register *reg)
+static int mt9m111_s_register(struct v4l2_subdev *sd,
+                             struct v4l2_dbg_register *reg)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct i2c_client *client = sd->priv;
 
-       if (reg->match_type != V4L2_CHIP_MATCH_I2C_ADDR || reg->reg > 0x2ff)
+       if (reg->match.type != V4L2_CHIP_MATCH_I2C_ADDR || reg->reg > 0x2ff)
                return -EINVAL;
 
-       if (reg->match_chip != mt9m111->client->addr)
+       if (reg->match.addr != client->addr)
                return -ENODEV;
 
-       if (mt9m111_reg_write(icd, reg->reg, reg->val) < 0)
+       if (mt9m111_reg_write(client, reg->reg, reg->val) < 0)
                return -EIO;
 
        return 0;
@@ -578,43 +704,21 @@ static const struct v4l2_queryctrl mt9m111_controls[] = {
        }
 };
 
-static int mt9m111_video_probe(struct soc_camera_device *);
-static void mt9m111_video_remove(struct soc_camera_device *);
-static int mt9m111_get_control(struct soc_camera_device *,
-                              struct v4l2_control *);
-static int mt9m111_set_control(struct soc_camera_device *,
-                              struct v4l2_control *);
 static int mt9m111_resume(struct soc_camera_device *icd);
-static int mt9m111_init(struct soc_camera_device *icd);
-static int mt9m111_release(struct soc_camera_device *icd);
+static int mt9m111_suspend(struct soc_camera_device *icd, pm_message_t state);
 
 static struct soc_camera_ops mt9m111_ops = {
-       .owner                  = THIS_MODULE,
-       .probe                  = mt9m111_video_probe,
-       .remove                 = mt9m111_video_remove,
-       .init                   = mt9m111_init,
+       .suspend                = mt9m111_suspend,
        .resume                 = mt9m111_resume,
-       .release                = mt9m111_release,
-       .start_capture          = mt9m111_start_capture,
-       .stop_capture           = mt9m111_stop_capture,
-       .set_fmt_cap            = mt9m111_set_fmt_cap,
-       .try_fmt_cap            = mt9m111_try_fmt_cap,
        .query_bus_param        = mt9m111_query_bus_param,
        .set_bus_param          = mt9m111_set_bus_param,
        .controls               = mt9m111_controls,
        .num_controls           = ARRAY_SIZE(mt9m111_controls),
-       .get_control            = mt9m111_get_control,
-       .set_control            = mt9m111_set_control,
-       .get_chip_id            = mt9m111_get_chip_id,
-#ifdef CONFIG_VIDEO_ADV_DEBUG
-       .get_register           = mt9m111_get_register,
-       .set_register           = mt9m111_set_register,
-#endif
 };
 
-static int mt9m111_set_flip(struct soc_camera_device *icd, int flip, int mask)
+static int mt9m111_set_flip(struct i2c_client *client, int flip, int mask)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret;
 
        if (mt9m111->context == HIGHPOWER) {
@@ -632,28 +736,26 @@ static int mt9m111_set_flip(struct soc_camera_device *icd, int flip, int mask)
        return ret;
 }
 
-static int mt9m111_get_global_gain(struct soc_camera_device *icd)
+static int mt9m111_get_global_gain(struct i2c_client *client)
 {
-       unsigned int data, gain;
+       int data;
 
        data = reg_read(GLOBAL_GAIN);
        if (data >= 0)
-               gain = ((data & (1 << 10)) * 2)
-                       | ((data & (1 << 9)) * 2)
-                       | (data & 0x2f);
-       else
-               gain = data;
-
-       return gain;
+               return (data & 0x2f) * (1 << ((data >> 10) & 1)) *
+                       (1 << ((data >> 9) & 1));
+       return data;
 }
-static int mt9m111_set_global_gain(struct soc_camera_device *icd, int gain)
+
+static int mt9m111_set_global_gain(struct i2c_client *client, int gain)
 {
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        u16 val;
 
        if (gain > 63 * 2 * 2)
                return -EINVAL;
 
-       icd->gain = gain;
+       mt9m111->gain = gain;
        if ((gain >= 64 * 2) && (gain < 63 * 2 * 2))
                val = (1 << 10) | (1 << 9) | (gain / 4);
        else if ((gain >= 64) && (gain < 64 * 2))
@@ -664,9 +766,9 @@ static int mt9m111_set_global_gain(struct soc_camera_device *icd, int gain)
        return reg_write(GLOBAL_GAIN, val);
 }
 
-static int mt9m111_set_autoexposure(struct soc_camera_device *icd, int on)
+static int mt9m111_set_autoexposure(struct i2c_client *client, int on)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret;
 
        if (on)
@@ -679,10 +781,27 @@ static int mt9m111_set_autoexposure(struct soc_camera_device *icd, int on)
 
        return ret;
 }
-static int mt9m111_get_control(struct soc_camera_device *icd,
-                              struct v4l2_control *ctrl)
+
+static int mt9m111_set_autowhitebalance(struct i2c_client *client, int on)
+{
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+       int ret;
+
+       if (on)
+               ret = reg_set(OPER_MODE_CTRL, MT9M111_OPMODE_AUTOWHITEBAL_EN);
+       else
+               ret = reg_clear(OPER_MODE_CTRL, MT9M111_OPMODE_AUTOWHITEBAL_EN);
+
+       if (!ret)
+               mt9m111->autowhitebalance = on;
+
+       return ret;
+}
+
+static int mt9m111_g_ctrl(struct v4l2_subdev *sd, struct v4l2_control *ctrl)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct i2c_client *client = sd->priv;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int data;
 
        switch (ctrl->id) {
@@ -707,7 +826,7 @@ static int mt9m111_get_control(struct soc_camera_device *icd,
                ctrl->value = !!(data & MT9M111_RMB_MIRROR_COLS);
                break;
        case V4L2_CID_GAIN:
-               data = mt9m111_get_global_gain(icd);
+               data = mt9m111_get_global_gain(client);
                if (data < 0)
                        return data;
                ctrl->value = data;
@@ -715,38 +834,43 @@ static int mt9m111_get_control(struct soc_camera_device *icd,
        case V4L2_CID_EXPOSURE_AUTO:
                ctrl->value = mt9m111->autoexposure;
                break;
+       case V4L2_CID_AUTO_WHITE_BALANCE:
+               ctrl->value = mt9m111->autowhitebalance;
+               break;
        }
        return 0;
 }
 
-static int mt9m111_set_control(struct soc_camera_device *icd,
-                              struct v4l2_control *ctrl)
+static int mt9m111_s_ctrl(struct v4l2_subdev *sd, struct v4l2_control *ctrl)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct i2c_client *client = sd->priv;
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        const struct v4l2_queryctrl *qctrl;
        int ret;
 
        qctrl = soc_camera_find_qctrl(&mt9m111_ops, ctrl->id);
-
        if (!qctrl)
                return -EINVAL;
 
        switch (ctrl->id) {
        case V4L2_CID_VFLIP:
                mt9m111->vflip = ctrl->value;
-               ret = mt9m111_set_flip(icd, ctrl->value,
+               ret = mt9m111_set_flip(client, ctrl->value,
                                        MT9M111_RMB_MIRROR_ROWS);
                break;
        case V4L2_CID_HFLIP:
                mt9m111->hflip = ctrl->value;
-               ret = mt9m111_set_flip(icd, ctrl->value,
+               ret = mt9m111_set_flip(client, ctrl->value,
                                        MT9M111_RMB_MIRROR_COLS);
                break;
        case V4L2_CID_GAIN:
-               ret = mt9m111_set_global_gain(icd, ctrl->value);
+               ret = mt9m111_set_global_gain(client, ctrl->value);
                break;
        case V4L2_CID_EXPOSURE_AUTO:
-               ret =  mt9m111_set_autoexposure(icd, ctrl->value);
+               ret =  mt9m111_set_autoexposure(client, ctrl->value);
+               break;
+       case V4L2_CID_AUTO_WHITE_BALANCE:
+               ret =  mt9m111_set_autowhitebalance(client, ctrl->value);
                break;
        default:
                ret = -EINVAL;
@@ -755,61 +879,62 @@ static int mt9m111_set_control(struct soc_camera_device *icd,
        return ret;
 }
 
-int mt9m111_restore_state(struct soc_camera_device *icd)
+static int mt9m111_suspend(struct soc_camera_device *icd, pm_message_t state)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct i2c_client *client = to_i2c_client(to_soc_camera_control(icd));
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+
+       mt9m111->gain = mt9m111_get_global_gain(client);
 
-       mt9m111_set_context(icd, mt9m111->context);
-       mt9m111_set_pixfmt(icd, mt9m111->pixfmt);
-       mt9m111_setup_rect(icd);
-       mt9m111_set_flip(icd, mt9m111->hflip, MT9M111_RMB_MIRROR_COLS);
-       mt9m111_set_flip(icd, mt9m111->vflip, MT9M111_RMB_MIRROR_ROWS);
-       mt9m111_set_global_gain(icd, icd->gain);
-       mt9m111_set_autoexposure(icd, mt9m111->autoexposure);
+       return 0;
+}
+
+static int mt9m111_restore_state(struct i2c_client *client)
+{
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+
+       mt9m111_set_context(client, mt9m111->context);
+       mt9m111_set_pixfmt(client, mt9m111->fmt->code);
+       mt9m111_setup_rect(client, &mt9m111->rect);
+       mt9m111_set_flip(client, mt9m111->hflip, MT9M111_RMB_MIRROR_COLS);
+       mt9m111_set_flip(client, mt9m111->vflip, MT9M111_RMB_MIRROR_ROWS);
+       mt9m111_set_global_gain(client, mt9m111->gain);
+       mt9m111_set_autoexposure(client, mt9m111->autoexposure);
+       mt9m111_set_autowhitebalance(client, mt9m111->autowhitebalance);
        return 0;
 }
 
 static int mt9m111_resume(struct soc_camera_device *icd)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct i2c_client *client = to_i2c_client(to_soc_camera_control(icd));
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret = 0;
 
        if (mt9m111->powered) {
-               ret = mt9m111_enable(icd);
+               ret = mt9m111_enable(client);
                if (!ret)
-                       ret = mt9m111_reset(icd);
+                       ret = mt9m111_reset(client);
                if (!ret)
-                       ret = mt9m111_restore_state(icd);
+                       ret = mt9m111_restore_state(client);
        }
        return ret;
 }
 
-static int mt9m111_init(struct soc_camera_device *icd)
+static int mt9m111_init(struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        int ret;
 
        mt9m111->context = HIGHPOWER;
-       ret = mt9m111_enable(icd);
+       ret = mt9m111_enable(client);
        if (!ret)
-               ret = mt9m111_reset(icd);
+               ret = mt9m111_reset(client);
        if (!ret)
-               ret = mt9m111_set_context(icd, mt9m111->context);
+               ret = mt9m111_set_context(client, mt9m111->context);
        if (!ret)
-               ret = mt9m111_set_autoexposure(icd, mt9m111->autoexposure);
+               ret = mt9m111_set_autoexposure(client, mt9m111->autoexposure);
        if (ret)
-               dev_err(&icd->dev, "mt9m111 init failed: %d\n", ret);
-       return ret;
-}
-
-static int mt9m111_release(struct soc_camera_device *icd)
-{
-       int ret;
-
-       ret = mt9m111_disable(icd);
-       if (ret < 0)
-               dev_err(&icd->dev, "mt9m111 release failed: %d\n", ret);
-
+               dev_err(&client->dev, "mt9m11x init failed: %d\n", ret);
        return ret;
 }
 
@@ -817,9 +942,10 @@ static int mt9m111_release(struct soc_camera_device *icd)
  * Interface active, can use i2c. If it fails, it can indeed mean, that
  * this wasn't our capture interface, so, we wait for the right one
  */
-static int mt9m111_video_probe(struct soc_camera_device *icd)
+static int mt9m111_video_probe(struct soc_camera_device *icd,
+                              struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
        s32 data;
        int ret;
 
@@ -831,65 +957,90 @@ static int mt9m111_video_probe(struct soc_camera_device *icd)
            to_soc_camera_host(icd->dev.parent)->nr != icd->iface)
                return -ENODEV;
 
-       ret = mt9m111_enable(icd);
-       if (ret)
-               goto ei2c;
-       ret = mt9m111_reset(icd);
+       mt9m111->autoexposure = 1;
+       mt9m111->autowhitebalance = 1;
+
+       mt9m111->swap_rgb_even_odd = 1;
+       mt9m111->swap_rgb_red_blue = 1;
+
+       ret = mt9m111_init(client);
        if (ret)
                goto ei2c;
 
        data = reg_read(CHIP_VERSION);
 
        switch (data) {
-       case 0x143a:
+       case 0x143a: /* MT9M111 */
                mt9m111->model = V4L2_IDENT_MT9M111;
-               icd->formats = mt9m111_colour_formats;
-               icd->num_formats = ARRAY_SIZE(mt9m111_colour_formats);
+               break;
+       case 0x148c: /* MT9M112 */
+               mt9m111->model = V4L2_IDENT_MT9M112;
                break;
        default:
                ret = -ENODEV;
-               dev_err(&icd->dev,
-                       "No MT9M111 chip detected, register read %x\n", data);
+               dev_err(&client->dev,
+                       "No MT9M11x chip detected, register read %x\n", data);
                goto ei2c;
        }
 
-       dev_info(&icd->dev, "Detected a MT9M111 chip ID 0x143a\n");
-
-       ret = soc_camera_video_start(icd);
-       if (ret)
-               goto eisis;
+       dev_info(&client->dev, "Detected a MT9M11x chip ID %x\n", data);
 
-       mt9m111->autoexposure = 1;
-
-       mt9m111->swap_rgb_even_odd = 1;
-       mt9m111->swap_rgb_red_blue = 1;
-
-       return 0;
-eisis:
 ei2c:
        return ret;
 }
 
-static void mt9m111_video_remove(struct soc_camera_device *icd)
+static struct v4l2_subdev_core_ops mt9m111_subdev_core_ops = {
+       .g_ctrl         = mt9m111_g_ctrl,
+       .s_ctrl         = mt9m111_s_ctrl,
+       .g_chip_ident   = mt9m111_g_chip_ident,
+#ifdef CONFIG_VIDEO_ADV_DEBUG
+       .g_register     = mt9m111_g_register,
+       .s_register     = mt9m111_s_register,
+#endif
+};
+
+static int mt9m111_enum_fmt(struct v4l2_subdev *sd, int index,
+                           enum v4l2_mbus_pixelcode *code)
 {
-       struct mt9m111 *mt9m111 = container_of(icd, struct mt9m111, icd);
+       if ((unsigned int)index >= ARRAY_SIZE(mt9m111_colour_fmts))
+               return -EINVAL;
 
-       dev_dbg(&icd->dev, "Video %x removed: %p, %p\n", mt9m111->client->addr,
-               mt9m111->icd.dev.parent, mt9m111->icd.vdev);
-       soc_camera_video_stop(&mt9m111->icd);
+       *code = mt9m111_colour_fmts[index].code;
+       return 0;
 }
 
+static struct v4l2_subdev_video_ops mt9m111_subdev_video_ops = {
+       .s_mbus_fmt     = mt9m111_s_fmt,
+       .g_mbus_fmt     = mt9m111_g_fmt,
+       .try_mbus_fmt   = mt9m111_try_fmt,
+       .s_crop         = mt9m111_s_crop,
+       .g_crop         = mt9m111_g_crop,
+       .cropcap        = mt9m111_cropcap,
+       .enum_mbus_fmt  = mt9m111_enum_fmt,
+};
+
+static struct v4l2_subdev_ops mt9m111_subdev_ops = {
+       .core   = &mt9m111_subdev_core_ops,
+       .video  = &mt9m111_subdev_video_ops,
+};
+
 static int mt9m111_probe(struct i2c_client *client,
                         const struct i2c_device_id *did)
 {
        struct mt9m111 *mt9m111;
-       struct soc_camera_device *icd;
+       struct soc_camera_device *icd = client->dev.platform_data;
        struct i2c_adapter *adapter = to_i2c_adapter(client->dev.parent);
-       struct soc_camera_link *icl = client->dev.platform_data;
+       struct soc_camera_link *icl;
        int ret;
 
+       if (!icd) {
+               dev_err(&client->dev, "MT9M11x: missing soc-camera data!\n");
+               return -EINVAL;
+       }
+
+       icl = to_soc_camera_link(icd);
        if (!icl) {
-               dev_err(&client->dev, "MT9M111 driver needs platform data\n");
+               dev_err(&client->dev, "MT9M11x driver needs platform data\n");
                return -EINVAL;
        }
 
@@ -903,38 +1054,35 @@ static int mt9m111_probe(struct i2c_client *client,
        if (!mt9m111)
                return -ENOMEM;
 
-       mt9m111->client = client;
-       i2c_set_clientdata(client, mt9m111);
+       v4l2_i2c_subdev_init(&mt9m111->subdev, client, &mt9m111_subdev_ops);
 
        /* Second stage probe - when a capture adapter is there */
-       icd             = &mt9m111->icd;
-       icd->ops        = &mt9m111_ops;
-       icd->control    = &client->dev;
-       icd->x_min      = MT9M111_MIN_DARK_COLS;
-       icd->y_min      = MT9M111_MIN_DARK_ROWS;
-       icd->x_current  = icd->x_min;
-       icd->y_current  = icd->y_min;
-       icd->width_min  = MT9M111_MIN_DARK_ROWS;
-       icd->width_max  = MT9M111_MAX_WIDTH;
-       icd->height_min = MT9M111_MIN_DARK_COLS;
-       icd->height_max = MT9M111_MAX_HEIGHT;
-       icd->y_skip_top = 0;
-       icd->iface      = icl->bus_id;
-
-       ret = soc_camera_device_register(icd);
-       if (ret)
-               goto eisdr;
-       return 0;
+       icd->ops                = &mt9m111_ops;
+
+       mt9m111->rect.left      = MT9M111_MIN_DARK_COLS;
+       mt9m111->rect.top       = MT9M111_MIN_DARK_ROWS;
+       mt9m111->rect.width     = MT9M111_MAX_WIDTH;
+       mt9m111->rect.height    = MT9M111_MAX_HEIGHT;
+       mt9m111->fmt            = &mt9m111_colour_fmts[0];
+
+       ret = mt9m111_video_probe(icd, client);
+       if (ret) {
+               icd->ops = NULL;
+               i2c_set_clientdata(client, NULL);
+               kfree(mt9m111);
+       }
 
-eisdr:
-       kfree(mt9m111);
        return ret;
 }
 
 static int mt9m111_remove(struct i2c_client *client)
 {
-       struct mt9m111 *mt9m111 = i2c_get_clientdata(client);
-       soc_camera_device_unregister(&mt9m111->icd);
+       struct mt9m111 *mt9m111 = to_mt9m111(client);
+       struct soc_camera_device *icd = client->dev.platform_data;
+
+       icd->ops = NULL;
+       i2c_set_clientdata(client, NULL);
+       client->driver = NULL;
        kfree(mt9m111);
 
        return 0;
@@ -968,6 +1116,6 @@ static void __exit mt9m111_mod_exit(void)
 module_init(mt9m111_mod_init);
 module_exit(mt9m111_mod_exit);
 
-MODULE_DESCRIPTION("Micron MT9M111 Camera driver");
+MODULE_DESCRIPTION("Micron MT9M111/MT9M112 Camera driver");
 MODULE_AUTHOR("Robert Jarzmik");
 MODULE_LICENSE("GPL");