[media] em28xx: remove field tuner_addr from struct em28xx
authorFrank Schaefer <fschaefer.oss@googlemail.com>
Mon, 24 Mar 2014 19:33:24 +0000 (16:33 -0300)
committerMauro Carvalho Chehab <m.chehab@samsung.com>
Fri, 23 May 2014 16:43:09 +0000 (13:43 -0300)
The tuner address is only used by the v4l submodule and at tuner setup and
can be obtained from the board data directly (if specified).

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>
drivers/media/usb/em28xx/em28xx-cards.c
drivers/media/usb/em28xx/em28xx-video.c
drivers/media/usb/em28xx/em28xx.h

index 98b82afdf5cdd830c428847bbb041efbe9ff044d..15ad470455532dfbae8879418a430515ea925e8e 100644 (file)
@@ -2741,8 +2741,6 @@ static void em28xx_card_setup(struct em28xx *dev)
                    dev->board.name, dev->model);
 
        dev->tuner_type = em28xx_boards[dev->model].tuner_type;
-       if (em28xx_boards[dev->model].tuner_addr)
-               dev->tuner_addr = em28xx_boards[dev->model].tuner_addr;
 
        /* request some modules */
        switch (dev->model) {
index 10c45f5e2fdd6ae1e95611248f28d4c658475595..b9b9775aa9c432c3c86332a6a2a86a04f996bba6 100644 (file)
@@ -2219,16 +2219,13 @@ static struct video_device *em28xx_vdev_init(struct em28xx *dev,
        return vfd;
 }
 
-static void em28xx_tuner_setup(struct em28xx *dev)
+static void em28xx_tuner_setup(struct em28xx *dev, unsigned short tuner_addr)
 {
        struct em28xx_v4l2      *v4l2 = dev->v4l2;
        struct v4l2_device      *v4l2_dev = &v4l2->v4l2_dev;
        struct tuner_setup      tun_setup;
        struct v4l2_frequency   f;
 
-       if (dev->tuner_type == TUNER_ABSENT)
-               return;
-
        memset(&tun_setup, 0, sizeof(tun_setup));
 
        tun_setup.mode_mask = T_ANALOG_TV | T_RADIO;
@@ -2244,7 +2241,7 @@ static void em28xx_tuner_setup(struct em28xx *dev)
 
        if ((dev->tuner_type != TUNER_ABSENT) && (dev->tuner_type)) {
                tun_setup.type   = dev->tuner_type;
-               tun_setup.addr   = dev->tuner_addr;
+               tun_setup.addr   = tuner_addr;
 
                v4l2_device_call_all(v4l2_dev,
                                     0, tuner, s_type_addr, &tun_setup);
@@ -2360,6 +2357,7 @@ static int em28xx_v4l2_init(struct em28xx *dev)
        /* Initialize tuner and camera */
 
        if (dev->board.tuner_type != TUNER_ABSENT) {
+               unsigned short tuner_addr = dev->board.tuner_addr;
                int has_demod = (dev->board.tda9887_conf & TDA9887_PRESENT);
 
                if (dev->board.radio.type)
@@ -2371,7 +2369,7 @@ static int em28xx_v4l2_init(struct em28xx *dev)
                        v4l2_i2c_new_subdev(&v4l2->v4l2_dev,
                                &dev->i2c_adap[dev->def_i2c_bus], "tuner",
                                0, v4l2_i2c_tuner_addrs(ADDRS_DEMOD));
-               if (dev->tuner_addr == 0) {
+               if (tuner_addr == 0) {
                        enum v4l2_i2c_tuner_type type =
                                has_demod ? ADDRS_TV_WITH_DEMOD : ADDRS_TV;
                        struct v4l2_subdev *sd;
@@ -2381,15 +2379,16 @@ static int em28xx_v4l2_init(struct em28xx *dev)
                                0, v4l2_i2c_tuner_addrs(type));
 
                        if (sd)
-                               dev->tuner_addr = v4l2_i2c_subdev_addr(sd);
+                               tuner_addr = v4l2_i2c_subdev_addr(sd);
                } else {
                        v4l2_i2c_new_subdev(&v4l2->v4l2_dev,
                                            &dev->i2c_adap[dev->def_i2c_bus],
-                                           "tuner", dev->tuner_addr, NULL);
+                                           "tuner", tuner_addr, NULL);
                }
+
+               em28xx_tuner_setup(dev, tuner_addr);
        }
 
-       em28xx_tuner_setup(dev);
        if (dev->em28xx_sensor != EM28XX_NOSENSOR)
                em28xx_init_camera(dev);
 
index c3758d4a9af8e99ac61afb5a23cd13c7825e3deb..55d8bca907d623685530959d61413bfb78834e49 100644 (file)
@@ -633,7 +633,6 @@ struct em28xx {
        struct em28xx_audio_mode audio_mode;
 
        int tuner_type;         /* type of the tuner */
-       int tuner_addr;         /* tuner address */
 
        /* i2c i/o */
        struct i2c_adapter i2c_adap[NUM_I2C_BUSES];