[media] em28xx: move struct v4l2_clk *clk from struct em28xx to struct v4l2

The current clock definition applies only to the V4L2 side of the
driver. Move its struct pointer to the v4l2_dev, where it belongs.

Signed-off-by: Frank Schäfer <fschaefer.oss@googlemail.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>

diff --git a/drivers/media/usb/em28xx/em28xx-camera.c b/drivers/media/usb/em28xx/em28xx-camera.c
index daebef39769fe3dc4affe089d31ec28de6dccf99..c2672b4f45403febca4b238a4699b9d747e3d5d3 100644 (file)
--- a/drivers/media/usb/em28xx/em28xx-camera.c
+++ b/drivers/media/usb/em28xx/em28xx-camera.c
@@ -330,13 +330,14 @@ int em28xx_init_camera(struct em28xx *dev)
        char clk_name[V4L2_SUBDEV_NAME_SIZE];
        struct i2c_client *client = &dev->i2c_client[dev->def_i2c_bus];
        struct i2c_adapter *adap = &dev->i2c_adap[dev->def_i2c_bus];
+       struct em28xx_v4l2 *v4l2 = dev->v4l2;
        int ret = 0;
 
        v4l2_clk_name_i2c(clk_name, sizeof(clk_name),
                          i2c_adapter_id(adap), client->addr);
-       dev->clk = v4l2_clk_register_fixed(clk_name, "mclk", -EINVAL);
-       if (IS_ERR(dev->clk))
-               return PTR_ERR(dev->clk);
+       v4l2->clk = v4l2_clk_register_fixed(clk_name, "mclk", -EINVAL);
+       if (IS_ERR(v4l2->clk))
+               return PTR_ERR(v4l2->clk);
 
        switch (dev->em28xx_sensor) {
        case EM28XX_MT9V011:
@@ -448,8 +449,8 @@ int em28xx_init_camera(struct em28xx *dev)
        }
 
        if (ret < 0) {
-               v4l2_clk_unregister_fixed(dev->clk);
-               dev->clk = NULL;
+               v4l2_clk_unregister_fixed(v4l2->clk);
+               v4l2->clk = NULL;
        }
 
        return ret;

diff --git a/drivers/media/usb/em28xx/em28xx-video.c b/drivers/media/usb/em28xx/em28xx-video.c
index a8ddc4cada95979a9157697978e7954b75fb9357..ffb95da7a2cfc199dcc985f2975744121f84ba27 100644 (file)
--- a/drivers/media/usb/em28xx/em28xx-video.c
+++ b/drivers/media/usb/em28xx/em28xx-video.c
@@ -1970,9 +1970,9 @@ static int em28xx_v4l2_fini(struct em28xx *dev)
        v4l2_ctrl_handler_free(&v4l2->ctrl_handler);
        v4l2_device_unregister(&v4l2->v4l2_dev);
 
-       if (dev->clk) {
-               v4l2_clk_unregister_fixed(dev->clk);
-               dev->clk = NULL;
+       if (v4l2->clk) {
+               v4l2_clk_unregister_fixed(v4l2->clk);
+               v4l2->clk = NULL;
        }
 
        kref_put(&v4l2->ref, em28xx_free_v4l2);

diff --git a/drivers/media/usb/em28xx/em28xx.h b/drivers/media/usb/em28xx/em28xx.h
index 24b2a9a73366c8d60c91430ba869fb5774d99550..3d17872d2b8cdcedd5b76851b76bc9d19c41bbce 100644 (file)
--- a/drivers/media/usb/em28xx/em28xx.h
+++ b/drivers/media/usb/em28xx/em28xx.h
@@ -504,6 +504,7 @@ struct em28xx_v4l2 {
 
        struct v4l2_device v4l2_dev;
        struct v4l2_ctrl_handler ctrl_handler;
+       struct v4l2_clk *clk;
 };
 
 struct em28xx_audio {
@@ -569,7 +570,6 @@ struct em28xx {
        unsigned int has_alsa_audio:1;
        unsigned int is_audio_only:1;
 
-       struct v4l2_clk *clk;
        struct em28xx_board board;
 
        /* Webcam specific fields */