[media] tvp5150: store dev id and rom version
authorJavier Martinez Canillas <javier@osg.samsung.com>
Fri, 5 Feb 2016 19:09:54 +0000 (17:09 -0200)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Wed, 10 Feb 2016 11:38:41 +0000 (09:38 -0200)
Not all tvp5150 variants support the same, for example some have an
internal signal generator that can output a black screen.

So the device id and rom version have to be stored in the driver's
state to know what variant is a given device.

While being there, remove some redundant comments about the device
version since there is already calls to v4l2_info() with that info.

Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/i2c/tvp5150.c

index c7eeb59a999bcaf37b256dc8a95681e77f467681..093ff80f944ccce72de967e3ef292791504ac02b 100644 (file)
@@ -49,6 +49,9 @@ struct tvp5150 {
        u32 output;
        int enable;
 
+       u16 dev_id;
+       u16 rom_ver;
+
        enum v4l2_mbus_type mbus_type;
 };
 
@@ -1180,8 +1183,6 @@ static int tvp5150_detect_version(struct tvp5150 *core)
        struct v4l2_subdev *sd = &core->sd;
        struct i2c_client *c = v4l2_get_subdevdata(sd);
        unsigned int i;
-       u16 dev_id;
-       u16 rom_ver;
        u8 regs[4];
        int res;
 
@@ -1196,23 +1197,25 @@ static int tvp5150_detect_version(struct tvp5150 *core)
                regs[i] = res;
        }
 
-       dev_id = (regs[0] << 8) | regs[1];
-       rom_ver = (regs[2] << 8) | regs[3];
+       core->dev_id = (regs[0] << 8) | regs[1];
+       core->rom_ver = (regs[2] << 8) | regs[3];
 
        v4l2_info(sd, "tvp%04x (%u.%u) chip found @ 0x%02x (%s)\n",
-                 dev_id, regs[2], regs[3], c->addr << 1, c->adapter->name);
+                 core->dev_id, regs[2], regs[3], c->addr << 1,
+                 c->adapter->name);
 
-       if (dev_id == 0x5150 && rom_ver == 0x0321) { /* TVP51510A */
+       if (core->dev_id == 0x5150 && core->rom_ver == 0x0321) {
                v4l2_info(sd, "tvp5150a detected.\n");
-       } else if (dev_id == 0x5150 && rom_ver == 0x0400) { /* TVP5150AM1 */
+       } else if (core->dev_id == 0x5150 && core->rom_ver == 0x0400) {
                v4l2_info(sd, "tvp5150am1 detected.\n");
 
                /* ITU-T BT.656.4 timing */
                tvp5150_write(sd, TVP5150_REV_SELECT, 0);
-       } else if (dev_id == 0x5151 && rom_ver == 0x0100) { /* TVP5151 */
+       } else if (core->dev_id == 0x5151 && core->rom_ver == 0x0100) {
                v4l2_info(sd, "tvp5151 detected.\n");
        } else {
-               v4l2_info(sd, "*** unknown tvp%04x chip detected.\n", dev_id);
+               v4l2_info(sd, "*** unknown tvp%04x chip detected.\n",
+                         core->dev_id);
        }
 
        return 0;