V4L/DVB (6785): tda8290: remove dependency on struct tuner
[pandora-kernel.git] / drivers / media / video / tda8290.c
index 59cff5a..d0d13bc 100644 (file)
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
    Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+
+   This "tda8290" module was split apart from the original "tuner" module.
 */
 
 #include <linux/i2c.h>
-#include <linux/videodev.h>
 #include <linux/delay.h>
+#include <linux/videodev.h>
 #include "tuner-driver.h"
+#include "tuner-i2c.h"
+#include "tda8290.h"
+#include "tda827x.h"
+#include "tda18271.h"
+
+static int debug;
+module_param(debug, int, 0644);
+MODULE_PARM_DESC(debug, "enable verbose debug messages");
+
+#define PREFIX "tda8290"
 
 /* ---------------------------------------------------------------------- */
 
 struct tda8290_priv {
+       struct tuner_i2c_props i2c_props;
+
        unsigned char tda8290_easy_mode;
-       unsigned char tda827x_lpsel;
+
        unsigned char tda827x_addr;
-       unsigned char tda827x_ver;
-       unsigned int sgIF;
-};
 
-/* ---------------------------------------------------------------------- */
+       unsigned char ver;
+#define TDA8290   1
+#define TDA8295   2
+#define TDA8275   4
+#define TDA8275A  8
+#define TDA18271 16
 
-struct tda827x_data {
-       u32 lomax;
-       u8  spd;
-       u8  bs;
-       u8  bp;
-       u8  cp;
-       u8  gc3;
-       u8 div1p5;
+       struct tda827x_config cfg;
 };
 
-     /* Note lomax entry is lo / 62500 */
-
-static struct tda827x_data tda827x_analog[] = {
-       { .lomax =   992, .spd = 3, .bs = 2, .bp = 0, .cp = 0, .gc3 = 3, .div1p5 = 1}, /*  62 MHz */
-       { .lomax =  1056, .spd = 3, .bs = 3, .bp = 0, .cp = 0, .gc3 = 3, .div1p5 = 1}, /*  66 MHz */
-       { .lomax =  1216, .spd = 3, .bs = 1, .bp = 0, .cp = 0, .gc3 = 3, .div1p5 = 0}, /*  76 MHz */
-       { .lomax =  1344, .spd = 3, .bs = 2, .bp = 0, .cp = 0, .gc3 = 3, .div1p5 = 0}, /*  84 MHz */
-       { .lomax =  1488, .spd = 3, .bs = 2, .bp = 0, .cp = 0, .gc3 = 1, .div1p5 = 0}, /*  93 MHz */
-       { .lomax =  1568, .spd = 3, .bs = 3, .bp = 0, .cp = 0, .gc3 = 1, .div1p5 = 0}, /*  98 MHz */
-       { .lomax =  1744, .spd = 3, .bs = 3, .bp = 1, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 109 MHz */
-       { .lomax =  1968, .spd = 2, .bs = 2, .bp = 1, .cp = 0, .gc3 = 1, .div1p5 = 1}, /* 123 MHz */
-       { .lomax =  2128, .spd = 2, .bs = 3, .bp = 1, .cp = 0, .gc3 = 1, .div1p5 = 1}, /* 133 MHz */
-       { .lomax =  2416, .spd = 2, .bs = 1, .bp = 1, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 151 MHz */
-       { .lomax =  2464, .spd = 2, .bs = 2, .bp = 1, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 154 MHz */
-       { .lomax =  2896, .spd = 2, .bs = 2, .bp = 1, .cp = 0, .gc3 = 0, .div1p5 = 0}, /* 181 MHz */
-       { .lomax =  2960, .spd = 2, .bs = 2, .bp = 2, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 185 MHz */
-       { .lomax =  3472, .spd = 2, .bs = 3, .bp = 2, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 217 MHz */
-       { .lomax =  3904, .spd = 1, .bs = 2, .bp = 2, .cp = 0, .gc3 = 1, .div1p5 = 1}, /* 244 MHz */
-       { .lomax =  4240, .spd = 1, .bs = 3, .bp = 2, .cp = 0, .gc3 = 1, .div1p5 = 1}, /* 265 MHz */
-       { .lomax =  4832, .spd = 1, .bs = 1, .bp = 2, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 302 MHz */
-       { .lomax =  5184, .spd = 1, .bs = 2, .bp = 2, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 324 MHz */
-       { .lomax =  5920, .spd = 1, .bs = 2, .bp = 3, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 370 MHz */
-       { .lomax =  7264, .spd = 1, .bs = 3, .bp = 3, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 454 MHz */
-       { .lomax =  7888, .spd = 0, .bs = 2, .bp = 3, .cp = 0, .gc3 = 1, .div1p5 = 1}, /* 493 MHz */
-       { .lomax =  8480, .spd = 0, .bs = 3, .bp = 3, .cp = 0, .gc3 = 1, .div1p5 = 1}, /* 530 MHz */
-       { .lomax =  8864, .spd = 0, .bs = 1, .bp = 3, .cp = 0, .gc3 = 1, .div1p5 = 0}, /* 554 MHz */
-       { .lomax =  9664, .spd = 0, .bs = 1, .bp = 4, .cp = 0, .gc3 = 0, .div1p5 = 0}, /* 604 MHz */
-       { .lomax = 11088, .spd = 0, .bs = 2, .bp = 4, .cp = 0, .gc3 = 0, .div1p5 = 0}, /* 696 MHz */
-       { .lomax = 11840, .spd = 0, .bs = 2, .bp = 4, .cp = 1, .gc3 = 0, .div1p5 = 0}, /* 740 MHz */
-       { .lomax = 13120, .spd = 0, .bs = 3, .bp = 4, .cp = 0, .gc3 = 0, .div1p5 = 0}, /* 820 MHz */
-       { .lomax = 13840, .spd = 0, .bs = 3, .bp = 4, .cp = 1, .gc3 = 0, .div1p5 = 0}, /* 865 MHz */
-       { .lomax =     0, .spd = 0, .bs = 0, .bp = 0, .cp = 0, .gc3 = 0, .div1p5 = 0}  /* End      */
-};
+/*---------------------------------------------------------------------*/
 
-static void tda827x_tune(struct i2c_client *c, u16 ifc, unsigned int freq)
+static int tda8290_i2c_bridge(struct dvb_frontend *fe, int close)
 {
-       unsigned char tuner_reg[8];
-       unsigned char reg2[2];
-       u32 N;
-       int i;
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
-       struct i2c_msg msg = {.addr = priv->tda827x_addr, .flags = 0};
+       struct tda8290_priv *priv = fe->analog_demod_priv;
 
-       if (t->mode == V4L2_TUNER_RADIO)
-               freq = freq / 1000;
+       unsigned char  enable[2] = { 0x21, 0xC0 };
+       unsigned char disable[2] = { 0x21, 0x00 };
+       unsigned char *msg;
 
-       N = freq + ifc;
-       i = 0;
-       while (tda827x_analog[i].lomax < N) {
-               if(tda827x_analog[i + 1].lomax == 0)
-                       break;
-               i++;
+       if (close) {
+               msg = enable;
+               tuner_i2c_xfer_send(&priv->i2c_props, msg, 2);
+               /* let the bridge stabilize */
+               msleep(20);
+       } else {
+               msg = disable;
+               tuner_i2c_xfer_send(&priv->i2c_props, msg, 2);
        }
 
-       N = N << tda827x_analog[i].spd;
-
-       tuner_reg[0] = 0;
-       tuner_reg[1] = (unsigned char)(N>>8);
-       tuner_reg[2] = (unsigned char) N;
-       tuner_reg[3] = 0x40;
-       tuner_reg[4] = 0x52 + (priv->tda827x_lpsel << 5);
-       tuner_reg[5] = (tda827x_analog[i].spd   << 6) + (tda827x_analog[i].div1p5 <<5) +
-                      (tda827x_analog[i].bs     <<3) +  tda827x_analog[i].bp;
-       tuner_reg[6] = 0x8f + (tda827x_analog[i].gc3 << 4);
-       tuner_reg[7] = 0x8f;
-
-       msg.buf = tuner_reg;
-       msg.len = 8;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       msg.buf= reg2;
-       msg.len = 2;
-       reg2[0] = 0x80;
-       reg2[1] = 0;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       reg2[0] = 0x60;
-       reg2[1] = 0xbf;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       reg2[0] = 0x30;
-       reg2[1] = tuner_reg[4] + 0x80;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       msleep(1);
-       reg2[0] = 0x30;
-       reg2[1] = tuner_reg[4] + 4;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       msleep(1);
-       reg2[0] = 0x30;
-       reg2[1] = tuner_reg[4];
-       i2c_transfer(c->adapter, &msg, 1);
-
-       msleep(550);
-       reg2[0] = 0x30;
-       reg2[1] = (tuner_reg[4] & 0xfc) + tda827x_analog[i].cp ;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       reg2[0] = 0x60;
-       reg2[1] = 0x3f;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       reg2[0] = 0x80;
-       reg2[1] = 0x08;   // Vsync en
-       i2c_transfer(c->adapter, &msg, 1);
+       return 0;
 }
 
-static void tda827x_agcf(struct i2c_client *c)
+static int tda8295_i2c_bridge(struct dvb_frontend *fe, int close)
 {
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
-       unsigned char data[] = {0x80, 0x0c};
-       struct i2c_msg msg = {.addr = priv->tda827x_addr, .buf = data,
-                             .flags = 0, .len = 2};
-       i2c_transfer(c->adapter, &msg, 1);
-}
-
-/* ---------------------------------------------------------------------- */
-
-struct tda827xa_data {
-       u32 lomax;
-       u8  svco;
-       u8  spd;
-       u8  scr;
-       u8  sbs;
-       u8  gc3;
-};
-
-static struct tda827xa_data tda827xa_analog[] = {
-       { .lomax =   910, .svco = 3, .spd = 4, .scr = 0, .sbs = 0, .gc3 = 3},  /*  56.875 MHz */
-       { .lomax =  1076, .svco = 0, .spd = 3, .scr = 0, .sbs = 0, .gc3 = 3},  /*  67.25 MHz */
-       { .lomax =  1300, .svco = 1, .spd = 3, .scr = 0, .sbs = 0, .gc3 = 3},  /*  81.25 MHz */
-       { .lomax =  1560, .svco = 2, .spd = 3, .scr = 0, .sbs = 0, .gc3 = 3},  /*  97.5  MHz */
-       { .lomax =  1820, .svco = 3, .spd = 3, .scr = 0, .sbs = 1, .gc3 = 1},  /* 113.75 MHz */
-       { .lomax =  2152, .svco = 0, .spd = 2, .scr = 0, .sbs = 1, .gc3 = 1},  /* 134.5 MHz */
-       { .lomax =  2464, .svco = 1, .spd = 2, .scr = 0, .sbs = 1, .gc3 = 1},  /* 154   MHz */
-       { .lomax =  2600, .svco = 1, .spd = 2, .scr = 0, .sbs = 1, .gc3 = 1},  /* 162.5 MHz */
-       { .lomax =  2928, .svco = 2, .spd = 2, .scr = 0, .sbs = 1, .gc3 = 1},  /* 183   MHz */
-       { .lomax =  3120, .svco = 2, .spd = 2, .scr = 0, .sbs = 2, .gc3 = 1},  /* 195   MHz */
-       { .lomax =  3640, .svco = 3, .spd = 2, .scr = 0, .sbs = 2, .gc3 = 3},  /* 227.5 MHz */
-       { .lomax =  4304, .svco = 0, .spd = 1, .scr = 0, .sbs = 2, .gc3 = 3},  /* 269   MHz */
-       { .lomax =  5200, .svco = 1, .spd = 1, .scr = 0, .sbs = 2, .gc3 = 1},  /* 325   MHz */
-       { .lomax =  6240, .svco = 2, .spd = 1, .scr = 0, .sbs = 3, .gc3 = 3},  /* 390   MHz */
-       { .lomax =  7280, .svco = 3, .spd = 1, .scr = 0, .sbs = 3, .gc3 = 3},  /* 455   MHz */
-       { .lomax =  8320, .svco = 0, .spd = 0, .scr = 0, .sbs = 3, .gc3 = 1},  /* 520   MHz */
-       { .lomax =  8608, .svco = 0, .spd = 0, .scr = 1, .sbs = 3, .gc3 = 1},  /* 538   MHz */
-       { .lomax =  8864, .svco = 1, .spd = 0, .scr = 0, .sbs = 3, .gc3 = 1},  /* 554   MHz */
-       { .lomax =  9920, .svco = 1, .spd = 0, .scr = 0, .sbs = 4, .gc3 = 0},  /* 620   MHz */
-       { .lomax = 10400, .svco = 1, .spd = 0, .scr = 1, .sbs = 4, .gc3 = 0},  /* 650   MHz */
-       { .lomax = 11200, .svco = 2, .spd = 0, .scr = 0, .sbs = 4, .gc3 = 0},  /* 700   MHz */
-       { .lomax = 12480, .svco = 2, .spd = 0, .scr = 1, .sbs = 4, .gc3 = 0},  /* 780   MHz */
-       { .lomax = 13120, .svco = 3, .spd = 0, .scr = 0, .sbs = 4, .gc3 = 0},  /* 820   MHz */
-       { .lomax = 13920, .svco = 3, .spd = 0, .scr = 1, .sbs = 4, .gc3 = 0},  /* 870   MHz */
-       { .lomax = 14576, .svco = 3, .spd = 0, .scr = 2, .sbs = 4, .gc3 = 0},  /* 911   MHz */
-       { .lomax =     0, .svco = 0, .spd = 0, .scr = 0, .sbs = 0, .gc3 = 0}   /* End */
-};
+       struct tda8290_priv *priv = fe->analog_demod_priv;
 
-static void tda827xa_lna_gain(struct i2c_client *c, int high)
-{
-       struct tuner *t = i2c_get_clientdata(c);
-       unsigned char buf[] = {0x22, 0x01};
-       int arg;
-       struct i2c_msg msg = {.addr = c->addr, .flags = 0, .buf = buf, .len = sizeof(buf)};
-       if (t->config) {
-               if (high)
-                       tuner_dbg("setting LNA to high gain\n");
-               else
-                       tuner_dbg("setting LNA to low gain\n");
-       }
-       switch (t->config) {
-       case 0: /* no LNA */
-               break;
-       case 1: /* switch is GPIO 0 of tda8290 */
-       case 2:
-               /* turn Vsync on */
-               if (t->std & V4L2_STD_MN)
-                       arg = 1;
-               else
-                       arg = 0;
-               if (t->tuner_callback)
-                       t->tuner_callback(c->adapter->algo_data, 1, arg);
-               buf[1] = high ? 0 : 1;
-               if (t->config == 2)
-                       buf[1] = high ? 1 : 0;
-               i2c_transfer(c->adapter, &msg, 1);
-               break;
-       case 3: /* switch with GPIO of saa713x */
-               if (t->tuner_callback)
-                       t->tuner_callback(c->adapter->algo_data, 0, high);
-               break;
-       }
-}
-
-static void tda827xa_tune(struct i2c_client *c, u16 ifc, unsigned int freq)
-{
-       unsigned char tuner_reg[11];
-       u32 N;
-       int i;
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
-       struct i2c_msg msg = {.addr = priv->tda827x_addr, .flags = 0, .buf = tuner_reg};
+       unsigned char  enable[2] = { 0x45, 0xc1 };
+       unsigned char disable[2] = { 0x46, 0x00 };
+       unsigned char buf[3] = { 0x45, 0x01, 0x00 };
+       unsigned char *msg;
 
-       tda827xa_lna_gain( c, 1);
-       msleep(10);
+       if (close) {
+               msg = enable;
+               tuner_i2c_xfer_send(&priv->i2c_props, msg, 2);
+               /* let the bridge stabilize */
+               msleep(20);
+       } else {
+               msg = disable;
+               tuner_i2c_xfer_send(&priv->i2c_props, msg, 1);
+               tuner_i2c_xfer_recv(&priv->i2c_props, &msg[1], 1);
 
-       if (t->mode == V4L2_TUNER_RADIO)
-               freq = freq / 1000;
+               buf[2] = msg[1];
+               buf[2] &= ~0x04;
+               tuner_i2c_xfer_send(&priv->i2c_props, buf, 3);
+               msleep(5);
 
-       N = freq + ifc;
-       i = 0;
-       while (tda827xa_analog[i].lomax < N) {
-               if(tda827xa_analog[i + 1].lomax == 0)
-                       break;
-               i++;
+               msg[1] |= 0x04;
+               tuner_i2c_xfer_send(&priv->i2c_props, msg, 2);
        }
 
-       N = N << tda827xa_analog[i].spd;
-
-       tuner_reg[0] = 0;
-       tuner_reg[1] = (unsigned char)(N>>8);
-       tuner_reg[2] = (unsigned char) N;
-       tuner_reg[3] = 0;
-       tuner_reg[4] = 0x16;
-       tuner_reg[5] = (tda827xa_analog[i].spd << 5) + (tda827xa_analog[i].svco << 3) +
-                       tda827xa_analog[i].sbs;
-       tuner_reg[6] = 0x8b + (tda827xa_analog[i].gc3 << 4);
-       tuner_reg[7] = 0x1c;
-       tuner_reg[8] = 4;
-       tuner_reg[9] = 0x20;
-       tuner_reg[10] = 0x00;
-       msg.len = 11;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       tuner_reg[0] = 0x90;
-       tuner_reg[1] = 0xff;
-       tuner_reg[2] = 0xe0;
-       tuner_reg[3] = 0;
-       tuner_reg[4] = 0x99 + (priv->tda827x_lpsel << 1);
-       msg.len = 5;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       tuner_reg[0] = 0xa0;
-       tuner_reg[1] = 0xc0;
-       msg.len = 2;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       tuner_reg[0] = 0x30;
-       tuner_reg[1] = 0x10 + tda827xa_analog[i].scr;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       msg.flags = I2C_M_RD;
-       i2c_transfer(c->adapter, &msg, 1);
-       msg.flags = 0;
-       tuner_reg[1] >>= 4;
-       tuner_dbg("AGC2 gain is: %d\n", tuner_reg[1]);
-       if (tuner_reg[1] < 1)
-               tda827xa_lna_gain( c, 0);
-
-       msleep(100);
-       tuner_reg[0] = 0x60;
-       tuner_reg[1] = 0x3c;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       msleep(163);
-       tuner_reg[0] = 0x50;
-       tuner_reg[1] = 0x8f + (tda827xa_analog[i].gc3 << 4);
-       i2c_transfer(c->adapter, &msg, 1);
-
-       tuner_reg[0] = 0x80;
-       tuner_reg[1] = 0x28;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       tuner_reg[0] = 0xb0;
-       tuner_reg[1] = 0x01;
-       i2c_transfer(c->adapter, &msg, 1);
-
-       tuner_reg[0] = 0xc0;
-       tuner_reg[1] = 0x19 + (priv->tda827x_lpsel << 1);
-       i2c_transfer(c->adapter, &msg, 1);
-}
-
-static void tda827xa_agcf(struct i2c_client *c)
-{
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
-       unsigned char data[] = {0x80, 0x2c};
-       struct i2c_msg msg = {.addr = priv->tda827x_addr, .buf = data,
-                             .flags = 0, .len = 2};
-       i2c_transfer(c->adapter, &msg, 1);
+       return 0;
 }
 
 /*---------------------------------------------------------------------*/
 
-static void tda8290_i2c_bridge(struct i2c_client *c, int close)
+static void set_audio(struct dvb_frontend *fe,
+                     struct analog_parameters *params)
 {
-       unsigned char  enable[2] = { 0x21, 0xC0 };
-       unsigned char disable[2] = { 0x21, 0x00 };
-       unsigned char *msg;
-       if(close) {
-               msg = enable;
-               i2c_master_send(c, msg, 2);
-               /* let the bridge stabilize */
-               msleep(20);
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+       char* mode;
+
+       if (params->std & V4L2_STD_MN) {
+               priv->tda8290_easy_mode = 0x01;
+               mode = "MN";
+       } else if (params->std & V4L2_STD_B) {
+               priv->tda8290_easy_mode = 0x02;
+               mode = "B";
+       } else if (params->std & V4L2_STD_GH) {
+               priv->tda8290_easy_mode = 0x04;
+               mode = "GH";
+       } else if (params->std & V4L2_STD_PAL_I) {
+               priv->tda8290_easy_mode = 0x08;
+               mode = "I";
+       } else if (params->std & V4L2_STD_DK) {
+               priv->tda8290_easy_mode = 0x10;
+               mode = "DK";
+       } else if (params->std & V4L2_STD_SECAM_L) {
+               priv->tda8290_easy_mode = 0x20;
+               mode = "L";
+       } else if (params->std & V4L2_STD_SECAM_LC) {
+               priv->tda8290_easy_mode = 0x40;
+               mode = "LC";
        } else {
-               msg = disable;
-               i2c_master_send(c, msg, 2);
+               priv->tda8290_easy_mode = 0x10;
+               mode = "xx";
        }
-}
 
-/*---------------------------------------------------------------------*/
+       tuner_dbg("setting tda829x to system %s\n", mode);
+}
 
-static int tda8290_tune(struct i2c_client *c, u16 ifc, unsigned int freq)
+static void tda8290_set_params(struct dvb_frontend *fe,
+                              struct analog_parameters *params)
 {
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+
        unsigned char soft_reset[]  = { 0x00, 0x00 };
        unsigned char easy_mode[]   = { 0x01, priv->tda8290_easy_mode };
        unsigned char expert_mode[] = { 0x01, 0x80 };
@@ -384,35 +172,38 @@ static int tda8290_tune(struct i2c_client *c, u16 ifc, unsigned int freq)
                      pll_stat;
        int i;
 
-       tuner_dbg("tda827xa config is 0x%02x\n", t->config);
-       i2c_master_send(c, easy_mode, 2);
-       i2c_master_send(c, agc_out_on, 2);
-       i2c_master_send(c, soft_reset, 2);
+       set_audio(fe, params);
+
+       if (priv->cfg.config)
+               tuner_dbg("tda827xa config is 0x%02x\n", *priv->cfg.config);
+       tuner_i2c_xfer_send(&priv->i2c_props, easy_mode, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, agc_out_on, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, soft_reset, 2);
        msleep(1);
 
        expert_mode[1] = priv->tda8290_easy_mode + 0x80;
-       i2c_master_send(c, expert_mode, 2);
-       i2c_master_send(c, gainset_off, 2);
-       i2c_master_send(c, if_agc_spd, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, expert_mode, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, gainset_off, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, if_agc_spd, 2);
        if (priv->tda8290_easy_mode & 0x60)
-               i2c_master_send(c, adc_head_9, 2);
+               tuner_i2c_xfer_send(&priv->i2c_props, adc_head_9, 2);
        else
-               i2c_master_send(c, adc_head_6, 2);
-       i2c_master_send(c, pll_bw_nom, 2);
+               tuner_i2c_xfer_send(&priv->i2c_props, adc_head_6, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, pll_bw_nom, 2);
+
+       tda8290_i2c_bridge(fe, 1);
+
+       if (fe->ops.tuner_ops.set_analog_params)
+               fe->ops.tuner_ops.set_analog_params(fe, params);
 
-       tda8290_i2c_bridge(c, 1);
-       if (priv->tda827x_ver != 0)
-               tda827xa_tune(c, ifc, freq);
-       else
-               tda827x_tune(c, ifc, freq);
        for (i = 0; i < 3; i++) {
-               i2c_master_send(c, &addr_pll_stat, 1);
-               i2c_master_recv(c, &pll_stat, 1);
+               tuner_i2c_xfer_send(&priv->i2c_props, &addr_pll_stat, 1);
+               tuner_i2c_xfer_recv(&priv->i2c_props, &pll_stat, 1);
                if (pll_stat & 0x80) {
-                       i2c_master_send(c, &addr_adc_sat, 1);
-                       i2c_master_recv(c, &adc_sat, 1);
-                       i2c_master_send(c, &addr_agc_stat, 1);
-                       i2c_master_recv(c, &agc_stat, 1);
+                       tuner_i2c_xfer_send(&priv->i2c_props, &addr_adc_sat, 1);
+                       tuner_i2c_xfer_recv(&priv->i2c_props, &adc_sat, 1);
+                       tuner_i2c_xfer_send(&priv->i2c_props, &addr_agc_stat, 1);
+                       tuner_i2c_xfer_recv(&priv->i2c_props, &agc_stat, 1);
                        tuner_dbg("tda8290 is locked, AGC: %d\n", agc_stat);
                        break;
                } else {
@@ -424,28 +215,26 @@ static int tda8290_tune(struct i2c_client *c, u16 ifc, unsigned int freq)
        if ((agc_stat > 115) || (!(pll_stat & 0x80) && (adc_sat < 20))) {
                tuner_dbg("adjust gain, step 1. Agc: %d, ADC stat: %d, lock: %d\n",
                           agc_stat, adc_sat, pll_stat & 0x80);
-               i2c_master_send(c, gainset_2, 2);
+               tuner_i2c_xfer_send(&priv->i2c_props, gainset_2, 2);
                msleep(100);
-               i2c_master_send(c, &addr_agc_stat, 1);
-               i2c_master_recv(c, &agc_stat, 1);
-               i2c_master_send(c, &addr_pll_stat, 1);
-               i2c_master_recv(c, &pll_stat, 1);
+               tuner_i2c_xfer_send(&priv->i2c_props, &addr_agc_stat, 1);
+               tuner_i2c_xfer_recv(&priv->i2c_props, &agc_stat, 1);
+               tuner_i2c_xfer_send(&priv->i2c_props, &addr_pll_stat, 1);
+               tuner_i2c_xfer_recv(&priv->i2c_props, &pll_stat, 1);
                if ((agc_stat > 115) || !(pll_stat & 0x80)) {
                        tuner_dbg("adjust gain, step 2. Agc: %d, lock: %d\n",
                                   agc_stat, pll_stat & 0x80);
-                       if (priv->tda827x_ver != 0)
-                               tda827xa_agcf(c);
-                       else
-                               tda827x_agcf(c);
+                       if (priv->cfg.agcf)
+                               priv->cfg.agcf(fe);
                        msleep(100);
-                       i2c_master_send(c, &addr_agc_stat, 1);
-                       i2c_master_recv(c, &agc_stat, 1);
-                       i2c_master_send(c, &addr_pll_stat, 1);
-                       i2c_master_recv(c, &pll_stat, 1);
+                       tuner_i2c_xfer_send(&priv->i2c_props, &addr_agc_stat, 1);
+                       tuner_i2c_xfer_recv(&priv->i2c_props, &agc_stat, 1);
+                       tuner_i2c_xfer_send(&priv->i2c_props, &addr_pll_stat, 1);
+                       tuner_i2c_xfer_recv(&priv->i2c_props, &pll_stat, 1);
                        if((agc_stat > 115) || !(pll_stat & 0x80)) {
                                tuner_dbg("adjust gain, step 3. Agc: %d\n", agc_stat);
-                               i2c_master_send(c, adc_head_12, 2);
-                               i2c_master_send(c, pll_bw_low, 2);
+                               tuner_i2c_xfer_send(&priv->i2c_props, adc_head_12, 2);
+                               tuner_i2c_xfer_send(&priv->i2c_props, pll_bw_low, 2);
                                msleep(100);
                        }
                }
@@ -453,185 +242,294 @@ static int tda8290_tune(struct i2c_client *c, u16 ifc, unsigned int freq)
 
        /* l/ l' deadlock? */
        if(priv->tda8290_easy_mode & 0x60) {
-               i2c_master_send(c, &addr_adc_sat, 1);
-               i2c_master_recv(c, &adc_sat, 1);
-               i2c_master_send(c, &addr_pll_stat, 1);
-               i2c_master_recv(c, &pll_stat, 1);
+               tuner_i2c_xfer_send(&priv->i2c_props, &addr_adc_sat, 1);
+               tuner_i2c_xfer_recv(&priv->i2c_props, &adc_sat, 1);
+               tuner_i2c_xfer_send(&priv->i2c_props, &addr_pll_stat, 1);
+               tuner_i2c_xfer_recv(&priv->i2c_props, &pll_stat, 1);
                if ((adc_sat > 20) || !(pll_stat & 0x80)) {
                        tuner_dbg("trying to resolve SECAM L deadlock\n");
-                       i2c_master_send(c, agc_rst_on, 2);
+                       tuner_i2c_xfer_send(&priv->i2c_props, agc_rst_on, 2);
                        msleep(40);
-                       i2c_master_send(c, agc_rst_off, 2);
+                       tuner_i2c_xfer_send(&priv->i2c_props, agc_rst_off, 2);
                }
        }
 
-       tda8290_i2c_bridge(c, 0);
-       i2c_master_send(c, if_agc_set, 2);
-       return 0;
+       tda8290_i2c_bridge(fe, 0);
+       tuner_i2c_xfer_send(&priv->i2c_props, if_agc_set, 2);
 }
 
 /*---------------------------------------------------------------------*/
 
-static void set_audio(struct tuner *t)
+static void tda8295_power(struct dvb_frontend *fe, int enable)
 {
-       struct tda8290_priv *priv = t->priv;
-       char* mode;
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+       unsigned char buf[] = { 0x30, 0x00 }; /* clb_stdbt */
 
-       priv->tda827x_lpsel = 0;
-       if (t->std & V4L2_STD_MN) {
-               priv->sgIF = 92;
-               priv->tda8290_easy_mode = 0x01;
-               priv->tda827x_lpsel = 1;
-               mode = "MN";
-       } else if (t->std & V4L2_STD_B) {
-               priv->sgIF = 108;
-               priv->tda8290_easy_mode = 0x02;
-               mode = "B";
-       } else if (t->std & V4L2_STD_GH) {
-               priv->sgIF = 124;
-               priv->tda8290_easy_mode = 0x04;
-               mode = "GH";
-       } else if (t->std & V4L2_STD_PAL_I) {
-               priv->sgIF = 124;
-               priv->tda8290_easy_mode = 0x08;
-               mode = "I";
-       } else if (t->std & V4L2_STD_DK) {
-               priv->sgIF = 124;
-               priv->tda8290_easy_mode = 0x10;
-               mode = "DK";
-       } else if (t->std & V4L2_STD_SECAM_L) {
-               priv->sgIF = 124;
-               priv->tda8290_easy_mode = 0x20;
-               mode = "L";
-       } else if (t->std & V4L2_STD_SECAM_LC) {
-               priv->sgIF = 20;
-               priv->tda8290_easy_mode = 0x40;
-               mode = "LC";
-       } else {
-               priv->sgIF = 124;
-               priv->tda8290_easy_mode = 0x10;
-               mode = "xx";
+       tuner_i2c_xfer_send(&priv->i2c_props, &buf[0], 1);
+       tuner_i2c_xfer_recv(&priv->i2c_props, &buf[1], 1);
+
+       if (enable)
+               buf[1] = 0x01;
+       else
+               buf[1] = 0x03;
+
+       tuner_i2c_xfer_send(&priv->i2c_props, buf, 2);
+}
+
+static void tda8295_set_easy_mode(struct dvb_frontend *fe, int enable)
+{
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+       unsigned char buf[] = { 0x01, 0x00 };
+
+       tuner_i2c_xfer_send(&priv->i2c_props, &buf[0], 1);
+       tuner_i2c_xfer_recv(&priv->i2c_props, &buf[1], 1);
+
+       if (enable)
+               buf[1] = 0x01; /* rising edge sets regs 0x02 - 0x23 */
+       else
+               buf[1] = 0x00; /* reset active bit */
+
+       tuner_i2c_xfer_send(&priv->i2c_props, buf, 2);
+}
+
+static void tda8295_set_video_std(struct dvb_frontend *fe)
+{
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+       unsigned char buf[] = { 0x00, priv->tda8290_easy_mode };
+
+       tuner_i2c_xfer_send(&priv->i2c_props, buf, 2);
+
+       tda8295_set_easy_mode(fe, 1);
+       msleep(20);
+       tda8295_set_easy_mode(fe, 0);
+}
+
+/*---------------------------------------------------------------------*/
+
+static void tda8295_agc1_out(struct dvb_frontend *fe, int enable)
+{
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+       unsigned char buf[] = { 0x02, 0x00 }; /* DIV_FUNC */
+
+       tuner_i2c_xfer_send(&priv->i2c_props, &buf[0], 1);
+       tuner_i2c_xfer_recv(&priv->i2c_props, &buf[1], 1);
+
+       if (enable)
+               buf[1] &= ~0x40;
+       else
+               buf[1] |= 0x40;
+
+       tuner_i2c_xfer_send(&priv->i2c_props, buf, 2);
+}
+
+static void tda8295_agc2_out(struct dvb_frontend *fe, int enable)
+{
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+       unsigned char set_gpio_cf[]    = { 0x44, 0x00 };
+       unsigned char set_gpio_val[]   = { 0x46, 0x00 };
+
+       tuner_i2c_xfer_send(&priv->i2c_props, &set_gpio_cf[0], 1);
+       tuner_i2c_xfer_recv(&priv->i2c_props, &set_gpio_cf[1], 1);
+       tuner_i2c_xfer_send(&priv->i2c_props, &set_gpio_val[0], 1);
+       tuner_i2c_xfer_recv(&priv->i2c_props, &set_gpio_val[1], 1);
+
+       set_gpio_cf[1] &= 0xf0; /* clear GPIO_0 bits 3-0 */
+
+       if (enable) {
+               set_gpio_cf[1]  |= 0x01; /* config GPIO_0 as Open Drain Out */
+               set_gpio_val[1] &= 0xfe; /* set GPIO_0 pin low */
        }
-       tuner_dbg("setting tda8290 to system %s\n", mode);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_gpio_cf, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_gpio_val, 2);
 }
 
-static void set_tv_freq(struct i2c_client *c, unsigned int freq)
+static int tda8295_has_signal(struct dvb_frontend *fe)
 {
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+
+       unsigned char hvpll_stat = 0x26;
+       unsigned char ret;
 
-       set_audio(t);
-       tda8290_tune(c, priv->sgIF, freq);
+       tuner_i2c_xfer_send(&priv->i2c_props, &hvpll_stat, 1);
+       tuner_i2c_xfer_recv(&priv->i2c_props, &ret, 1);
+       return (ret & 0x01) ? 65535 : 0;
 }
 
-static void set_radio_freq(struct i2c_client *c, unsigned int freq)
+/*---------------------------------------------------------------------*/
+
+static void tda8295_set_params(struct dvb_frontend *fe,
+                              struct analog_parameters *params)
 {
-       /* if frequency is 5.5 MHz */
-       tda8290_tune(c, 88, freq);
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+
+       unsigned char blanking_mode[]     = { 0x1d, 0x00 };
+
+       set_audio(fe, params);
+
+       tuner_dbg("%s: freq = %d\n", __FUNCTION__, params->frequency);
+
+       tda8295_power(fe, 1);
+       tda8295_agc1_out(fe, 1);
+
+       tuner_i2c_xfer_send(&priv->i2c_props, &blanking_mode[0], 1);
+       tuner_i2c_xfer_recv(&priv->i2c_props, &blanking_mode[1], 1);
+
+       tda8295_set_video_std(fe);
+
+       blanking_mode[1] = 0x03;
+       tuner_i2c_xfer_send(&priv->i2c_props, blanking_mode, 2);
+       msleep(20);
+
+       tda8295_i2c_bridge(fe, 1);
+
+       if (fe->ops.tuner_ops.set_analog_params)
+               fe->ops.tuner_ops.set_analog_params(fe, params);
+
+       if (priv->cfg.agcf)
+               priv->cfg.agcf(fe);
+
+       if (tda8295_has_signal(fe))
+               tuner_dbg("tda8295 is locked\n");
+       else
+               tuner_dbg("tda8295 not locked, no signal?\n");
+
+       tda8295_i2c_bridge(fe, 0);
 }
 
-static int has_signal(struct i2c_client *c)
+/*---------------------------------------------------------------------*/
+
+static int tda8290_has_signal(struct dvb_frontend *fe)
 {
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+
        unsigned char i2c_get_afc[1] = { 0x1B };
        unsigned char afc = 0;
 
-       i2c_master_send(c, i2c_get_afc, ARRAY_SIZE(i2c_get_afc));
-       i2c_master_recv(c, &afc, 1);
+       tuner_i2c_xfer_send(&priv->i2c_props, i2c_get_afc, ARRAY_SIZE(i2c_get_afc));
+       tuner_i2c_xfer_recv(&priv->i2c_props, &afc, 1);
        return (afc & 0x80)? 65535:0;
 }
 
 /*---------------------------------------------------------------------*/
 
-static void standby(struct i2c_client *c)
+static void tda8290_standby(struct dvb_frontend *fe)
 {
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+
        unsigned char cb1[] = { 0x30, 0xD0 };
        unsigned char tda8290_standby[] = { 0x00, 0x02 };
        unsigned char tda8290_agc_tri[] = { 0x02, 0x20 };
        struct i2c_msg msg = {.addr = priv->tda827x_addr, .flags=0, .buf=cb1, .len = 2};
 
-       tda8290_i2c_bridge(c, 1);
-       if (priv->tda827x_ver != 0)
+       tda8290_i2c_bridge(fe, 1);
+       if (priv->ver & TDA8275A)
                cb1[1] = 0x90;
-       i2c_transfer(c->adapter, &msg, 1);
-       tda8290_i2c_bridge(c, 0);
-       i2c_master_send(c, tda8290_agc_tri, 2);
-       i2c_master_send(c, tda8290_standby, 2);
+       i2c_transfer(priv->i2c_props.adap, &msg, 1);
+       tda8290_i2c_bridge(fe, 0);
+       tuner_i2c_xfer_send(&priv->i2c_props, tda8290_agc_tri, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, tda8290_standby, 2);
 }
 
+static void tda8295_standby(struct dvb_frontend *fe)
+{
+       tda8295_agc1_out(fe, 0); /* Put AGC in tri-state */
 
-static void tda8290_init_if(struct i2c_client *c)
+       tda8295_power(fe, 0);
+}
+
+static void tda8290_init_if(struct dvb_frontend *fe)
 {
-       struct tuner *t = i2c_get_clientdata(c);
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+
        unsigned char set_VS[] = { 0x30, 0x6F };
        unsigned char set_GP00_CF[] = { 0x20, 0x01 };
        unsigned char set_GP01_CF[] = { 0x20, 0x0B };
 
-       if ((t->config == 1) || (t->config == 2))
-               i2c_master_send(c, set_GP00_CF, 2);
+       if ((priv->cfg.config) &&
+           ((*priv->cfg.config == 1) || (*priv->cfg.config == 2)))
+               tuner_i2c_xfer_send(&priv->i2c_props, set_GP00_CF, 2);
        else
-               i2c_master_send(c, set_GP01_CF, 2);
-       i2c_master_send(c, set_VS, 2);
+               tuner_i2c_xfer_send(&priv->i2c_props, set_GP01_CF, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_VS, 2);
 }
 
-static void tda8290_init_tuner(struct i2c_client *c)
+static void tda8295_init_if(struct dvb_frontend *fe)
 {
-       struct tuner *t = i2c_get_clientdata(c);
-       struct tda8290_priv *priv = t->priv;
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+
+       static unsigned char set_adc_ctl[]       = { 0x33, 0x14 };
+       static unsigned char set_adc_ctl2[]      = { 0x34, 0x00 };
+       static unsigned char set_pll_reg6[]      = { 0x3e, 0x63 };
+       static unsigned char set_pll_reg0[]      = { 0x38, 0x23 };
+       static unsigned char set_pll_reg7[]      = { 0x3f, 0x01 };
+       static unsigned char set_pll_reg10[]     = { 0x42, 0x61 };
+       static unsigned char set_gpio_reg0[]     = { 0x44, 0x0b };
+
+       tda8295_power(fe, 1);
+
+       tda8295_set_easy_mode(fe, 0);
+       tda8295_set_video_std(fe);
+
+       tuner_i2c_xfer_send(&priv->i2c_props, set_adc_ctl, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_adc_ctl2, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_pll_reg6, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_pll_reg0, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_pll_reg7, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_pll_reg10, 2);
+       tuner_i2c_xfer_send(&priv->i2c_props, set_gpio_reg0, 2);
+
+       tda8295_agc1_out(fe, 0);
+       tda8295_agc2_out(fe, 0);
+}
+
+static void tda8290_init_tuner(struct dvb_frontend *fe)
+{
+       struct tda8290_priv *priv = fe->analog_demod_priv;
        unsigned char tda8275_init[]  = { 0x00, 0x00, 0x00, 0x40, 0xdC, 0x04, 0xAf,
                                          0x3F, 0x2A, 0x04, 0xFF, 0x00, 0x00, 0x40 };
        unsigned char tda8275a_init[] = { 0x00, 0x00, 0x00, 0x00, 0xdC, 0x05, 0x8b,
                                          0x0c, 0x04, 0x20, 0xFF, 0x00, 0x00, 0x4b };
        struct i2c_msg msg = {.addr = priv->tda827x_addr, .flags=0,
                              .buf=tda8275_init, .len = 14};
-       if (priv->tda827x_ver != 0)
+       if (priv->ver & TDA8275A)
                msg.buf = tda8275a_init;
 
-       tda8290_i2c_bridge(c, 1);
-       i2c_transfer(c->adapter, &msg, 1);
-       tda8290_i2c_bridge(c, 0);
+       tda8290_i2c_bridge(fe, 1);
+       i2c_transfer(priv->i2c_props.adap, &msg, 1);
+       tda8290_i2c_bridge(fe, 0);
 }
 
 /*---------------------------------------------------------------------*/
 
-static void tda8290_release(struct i2c_client *c)
+static void tda829x_release(struct dvb_frontend *fe)
 {
-       struct tuner *t = i2c_get_clientdata(c);
+       if (fe->ops.tuner_ops.release)
+               fe->ops.tuner_ops.release(fe);
 
-       kfree(t->priv);
-       t->priv = NULL;
+       kfree(fe->analog_demod_priv);
+       fe->analog_demod_priv = NULL;
 }
 
-static struct tuner_operations tda8290_tuner_ops = {
-       .set_tv_freq    = set_tv_freq,
-       .set_radio_freq = set_radio_freq,
-       .has_signal     = has_signal,
-       .standby        = standby,
-       .release        = tda8290_release,
-};
-
-int tda8290_init(struct i2c_client *c)
+static int tda829x_find_tuner(struct dvb_frontend *fe)
 {
-       struct tda8290_priv *priv = NULL;
-       struct tuner *t = i2c_get_clientdata(c);
-       u8 data;
+       struct tda8290_priv *priv = fe->analog_demod_priv;
+       struct analog_tuner_ops *ops = fe->ops.analog_demod_ops;
        int i, ret, tuners_found;
        u32 tuner_addrs;
-       struct i2c_msg msg = {.flags=I2C_M_RD, .buf=&data, .len = 1};
+       u8 data;
+       struct i2c_msg msg = { .flags = I2C_M_RD, .buf = &data, .len = 1 };
 
-       priv = kzalloc(sizeof(struct tda8290_priv), GFP_KERNEL);
-       if (priv == NULL)
-               return -ENOMEM;
-       t->priv = priv;
+       if (NULL == ops)
+               return -EINVAL;
+
+       ops->i2c_gate_ctrl(fe, 1);
 
-       tda8290_i2c_bridge(c, 1);
        /* probe for tuner chip */
        tuners_found = 0;
        tuner_addrs = 0;
-       for (i=0x60; i<= 0x63; i++) {
+       for (i = 0x60; i <= 0x63; i++) {
                msg.addr = i;
-               ret = i2c_transfer(c->adapter, &msg, 1);
+               ret = i2c_transfer(priv->i2c_props.adap, &msg, 1);
                if (ret == 1) {
                        tuners_found++;
                        tuner_addrs = (tuner_addrs << 8) + i;
@@ -641,75 +539,244 @@ int tda8290_init(struct i2c_client *c)
           behind the bridge and we choose the highest address that doesn't
           give a response now
         */
-       tda8290_i2c_bridge(c, 0);
-       if(tuners_found > 1)
+
+       ops->i2c_gate_ctrl(fe, 0);
+
+       if (tuners_found > 1)
                for (i = 0; i < tuners_found; i++) {
                        msg.addr = tuner_addrs  & 0xff;
-                       ret = i2c_transfer(c->adapter, &msg, 1);
-                       if(ret == 1)
+                       ret = i2c_transfer(priv->i2c_props.adap, &msg, 1);
+                       if (ret == 1)
                                tuner_addrs = tuner_addrs >> 8;
                        else
                                break;
                }
+
        if (tuner_addrs == 0) {
-               tuner_addrs = 0x61;
-               tuner_info ("could not clearly identify tuner address, defaulting to %x\n",
-                            tuner_addrs);
+               tuner_addrs = 0x60;
+               tuner_info("could not clearly identify tuner address, "
+                          "defaulting to %x\n", tuner_addrs);
        } else {
                tuner_addrs = tuner_addrs & 0xff;
-               tuner_info ("setting tuner address to %x\n", tuner_addrs);
+               tuner_info("setting tuner address to %x\n", tuner_addrs);
        }
        priv->tda827x_addr = tuner_addrs;
        msg.addr = tuner_addrs;
 
-       tda8290_i2c_bridge(c, 1);
-       ret = i2c_transfer(c->adapter, &msg, 1);
-       if( ret != 1)
-               tuner_warn ("TDA827x access failed!\n");
-       if ((data & 0x3c) == 0) {
-               strlcpy(c->name, "tda8290+75", sizeof(c->name));
-               priv->tda827x_ver = 0;
+       ops->i2c_gate_ctrl(fe, 1);
+       ret = i2c_transfer(priv->i2c_props.adap, &msg, 1);
+
+       if (ret != 1) {
+               tuner_warn("tuner access failed!\n");
+               return -EREMOTEIO;
+       }
+
+       if (data == 0x83) {
+               priv->ver |= TDA18271;
+               tda18271_attach(fe, priv->tda827x_addr,
+                               priv->i2c_props.adap);
        } else {
-               strlcpy(c->name, "tda8290+75a", sizeof(c->name));
-               priv->tda827x_ver = 2;
+               if ((data & 0x3c) == 0)
+                       priv->ver |= TDA8275;
+               else
+                       priv->ver |= TDA8275A;
+
+               tda827x_attach(fe, priv->tda827x_addr,
+                              priv->i2c_props.adap, &priv->cfg);
        }
-       tuner_info("type set to %s\n", c->name);
+       if (fe->ops.tuner_ops.init)
+               fe->ops.tuner_ops.init(fe);
 
-       memcpy(&t->ops, &tda8290_tuner_ops, sizeof(struct tuner_operations));
+       if (fe->ops.tuner_ops.sleep)
+               fe->ops.tuner_ops.sleep(fe);
 
-       priv->tda827x_lpsel = 0;
-       t->mode = V4L2_TUNER_ANALOG_TV;
+       ops->i2c_gate_ctrl(fe, 0);
 
-       tda8290_init_tuner(c);
-       tda8290_init_if(c);
        return 0;
 }
 
-int tda8290_probe(struct i2c_client *c)
+static int tda8290_probe(struct tuner_i2c_props *i2c_props)
+{
+#define TDA8290_ID 0x89
+       unsigned char tda8290_id[] = { 0x1f, 0x00 };
+
+       /* detect tda8290 */
+       tuner_i2c_xfer_send(i2c_props, &tda8290_id[0], 1);
+       tuner_i2c_xfer_recv(i2c_props, &tda8290_id[1], 1);
+
+       if (tda8290_id[1] == TDA8290_ID) {
+               if (debug)
+                       printk(KERN_DEBUG "%s: tda8290 detected @ %d-%04x\n",
+                              __FUNCTION__, i2c_adapter_id(i2c_props->adap),
+                              i2c_props->addr);
+               return 0;
+       }
+
+       return -ENODEV;
+}
+
+static int tda8295_probe(struct tuner_i2c_props *i2c_props)
+{
+#define TDA8295_ID 0x8a
+       unsigned char tda8295_id[] = { 0x2f, 0x00 };
+
+       /* detect tda8295 */
+       tuner_i2c_xfer_send(i2c_props, &tda8295_id[0], 1);
+       tuner_i2c_xfer_recv(i2c_props, &tda8295_id[1], 1);
+
+       if (tda8295_id[1] == TDA8295_ID) {
+               if (debug)
+                       printk(KERN_DEBUG "%s: tda8295 detected @ %d-%04x\n",
+                              __FUNCTION__, i2c_adapter_id(i2c_props->adap),
+                              i2c_props->addr);
+               return 0;
+       }
+
+       return -ENODEV;
+}
+
+static struct analog_tuner_ops tda8290_tuner_ops = {
+       .set_params     = tda8290_set_params,
+       .has_signal     = tda8290_has_signal,
+       .standby        = tda8290_standby,
+       .release        = tda829x_release,
+       .i2c_gate_ctrl  = tda8290_i2c_bridge,
+};
+
+static struct analog_tuner_ops tda8295_tuner_ops = {
+       .set_params     = tda8295_set_params,
+       .has_signal     = tda8295_has_signal,
+       .standby        = tda8295_standby,
+       .release        = tda829x_release,
+       .i2c_gate_ctrl  = tda8295_i2c_bridge,
+};
+
+struct dvb_frontend *tda829x_attach(struct dvb_frontend *fe,
+                                   struct i2c_adapter *i2c_adap, u8 i2c_addr,
+                                   struct tda829x_config *cfg)
+{
+       struct tda8290_priv *priv = NULL;
+       char *name;
+
+       priv = kzalloc(sizeof(struct tda8290_priv), GFP_KERNEL);
+       if (priv == NULL)
+               return NULL;
+       fe->analog_demod_priv = priv;
+
+       priv->i2c_props.addr     = i2c_addr;
+       priv->i2c_props.adap     = i2c_adap;
+       if (cfg) {
+               priv->cfg.config         = cfg->lna_cfg;
+               priv->cfg.tuner_callback = cfg->tuner_callback;
+       }
+
+       if (tda8290_probe(&priv->i2c_props) == 0) {
+               priv->ver = TDA8290;
+               fe->ops.analog_demod_ops = &tda8290_tuner_ops;
+       }
+
+       if (tda8295_probe(&priv->i2c_props) == 0) {
+               priv->ver = TDA8295;
+               fe->ops.analog_demod_ops = &tda8295_tuner_ops;
+       }
+
+       if (tda829x_find_tuner(fe) < 0)
+               goto fail;
+
+       switch (priv->ver) {
+       case TDA8290 | TDA8275:
+               name = "tda8290+75";
+               break;
+       case TDA8295 | TDA8275:
+               name = "tda8295+75";
+               break;
+       case TDA8290 | TDA8275A:
+               name = "tda8290+75a";
+               break;
+       case TDA8295 | TDA8275A:
+               name = "tda8295+75a";
+               break;
+       case TDA8290 | TDA18271:
+               name = "tda8290+18271";
+               break;
+       case TDA8295 | TDA18271:
+               name = "tda8295+18271";
+               break;
+       default:
+               goto fail;
+       }
+       tuner_info("type set to %s\n", name);
+
+       if (priv->ver & TDA8290) {
+               tda8290_init_tuner(fe);
+               tda8290_init_if(fe);
+       } else if (priv->ver & TDA8295)
+               tda8295_init_if(fe);
+
+       return fe;
+
+fail:
+       tda829x_release(fe);
+       fe->ops.analog_demod_ops = NULL;
+       return NULL;
+}
+EXPORT_SYMBOL_GPL(tda829x_attach);
+
+int tda829x_probe(struct i2c_adapter *i2c_adap, u8 i2c_addr)
 {
+       struct tuner_i2c_props i2c_props = {
+               .adap = i2c_adap,
+               .addr = i2c_addr,
+       };
+
        unsigned char soft_reset[]   = { 0x00, 0x00 };
        unsigned char easy_mode_b[]  = { 0x01, 0x02 };
        unsigned char easy_mode_g[]  = { 0x01, 0x04 };
        unsigned char restore_9886[] = { 0x00, 0xd6, 0x30 };
        unsigned char addr_dto_lsb = 0x07;
        unsigned char data;
+#define PROBE_BUFFER_SIZE 8
+       unsigned char buf[PROBE_BUFFER_SIZE];
+       int i;
+
+       /* rule out tda9887, which would return the same byte repeatedly */
+       tuner_i2c_xfer_send(&i2c_props, soft_reset, 1);
+       tuner_i2c_xfer_recv(&i2c_props, buf, PROBE_BUFFER_SIZE);
+       for (i = 1; i < PROBE_BUFFER_SIZE; i++) {
+               if (buf[i] != buf[0])
+                       break;
+       }
 
-       i2c_master_send(c, easy_mode_b, 2);
-       i2c_master_send(c, soft_reset, 2);
-       i2c_master_send(c, &addr_dto_lsb, 1);
-       i2c_master_recv(c, &data, 1);
+       /* all bytes are equal, not a tda829x - probably a tda9887 */
+       if (i == PROBE_BUFFER_SIZE)
+               return -ENODEV;
+
+       if ((tda8290_probe(&i2c_props) == 0) ||
+           (tda8295_probe(&i2c_props) == 0))
+               return 0;
+
+       /* fall back to old probing method */
+       tuner_i2c_xfer_send(&i2c_props, easy_mode_b, 2);
+       tuner_i2c_xfer_send(&i2c_props, soft_reset, 2);
+       tuner_i2c_xfer_send(&i2c_props, &addr_dto_lsb, 1);
+       tuner_i2c_xfer_recv(&i2c_props, &data, 1);
        if (data == 0) {
-               i2c_master_send(c, easy_mode_g, 2);
-               i2c_master_send(c, soft_reset, 2);
-               i2c_master_send(c, &addr_dto_lsb, 1);
-               i2c_master_recv(c, &data, 1);
+               tuner_i2c_xfer_send(&i2c_props, easy_mode_g, 2);
+               tuner_i2c_xfer_send(&i2c_props, soft_reset, 2);
+               tuner_i2c_xfer_send(&i2c_props, &addr_dto_lsb, 1);
+               tuner_i2c_xfer_recv(&i2c_props, &data, 1);
                if (data == 0x7b) {
                        return 0;
                }
        }
-       i2c_master_send(c, restore_9886, 3);
-       return -1;
+       tuner_i2c_xfer_send(&i2c_props, restore_9886, 3);
+       return -ENODEV;
 }
+EXPORT_SYMBOL_GPL(tda829x_probe);
+
+MODULE_DESCRIPTION("Philips/NXP TDA8290/TDA8295 analog IF demodulator driver");
+MODULE_AUTHOR("Gerd Knorr, Hartmut Hackmann, Michael Krufky");
+MODULE_LICENSE("GPL");
 
 /*
  * Overrides for Emacs so that we follow Linus's tabbing style.