V4L/DVB (3436): move config byte from tuner_params to tuner_range struct.
[firefly-linux-kernel-4.4.55.git] / drivers / media / video / tuner-simple.c
index feaed92e2a69d6e8b88aa9d1b73cf7252cbdba02..6f0d376f0d1802b964a2cbfa9b10a53116fb115c 100644 (file)
@@ -8,6 +8,10 @@
 #include <linux/videodev.h>
 #include <media/tuner.h>
 
+static int offset = 0;
+module_param(offset, int, 0666);
+MODULE_PARM_DESC(offset,"Allows to specify an offset for tuner");
+
 /* ---------------------------------------------------------------------- */
 
 /* tv standard selection for Temic 4046 FM5
 #define TUNER_PLL_LOCKED   0x40
 #define TUNER_STEREO_MK3   0x04
 
-/* ---------------------------------------------------------------------- */
-
-struct tunertype
-{
-       char *name;
-
-       unsigned short thresh1;  /*  band switch VHF_LO <=> VHF_HI  */
-       unsigned short thresh2;  /*  band switch VHF_HI <=> UHF     */
-       unsigned char VHF_L;
-       unsigned char VHF_H;
-       unsigned char UHF;
-       unsigned char config;
-};
-
-/*
- *     The floats in the tuner struct are computed at compile time
- *     by gcc and cast back to integers. Thus we don't violate the
- *     "no float in kernel" rule.
+#define TUNER_PARAM_ANALOG 0  /* to be removed */
+/* FIXME:
+ * Right now, all tuners are using the first tuner_params[] array element
+ * for analog mode. In the future, we will be merging similar tuner
+ * definitions together, such that each tuner definition will have a
+ * tuner_params struct for each available video standard. At that point,
+ * TUNER_PARAM_ANALOG will be removed, and the tuner_params[] array
+ * element will be chosen based on the video standard in use.
+ *
  */
-static struct tunertype tuners[] = {
-       /* 0-9 */
-       { "Temic PAL (4002 FH5)",                         /* TEMIC PAL */
-         16*140.25, 16*463.25, 0x02, 0x04, 0x01, 0x8e},
-       { "Philips PAL_I (FI1246 and compatibles)",       /* Philips PAL_I */
-         16*140.25, 16*463.25, 0xa0, 0x90, 0x30, 0x8e},
-       { "Philips NTSC (FI1236,FM1236 and compatibles)", /* Philips NTSC */
-         16*157.25, 16*451.25, 0xA0, 0x90, 0x30, 0x8e},
-       { "Philips (SECAM+PAL_BG) (FI1216MF, FM1216MF, FR1216MF)",/* Philips SECAM */
-         16*168.25, 16*447.25, 0xA7, 0x97, 0x37, 0x8e},
-       { "NoTuner",                                      /* NoTuner NOTUNER */
-         0, 0, 0x00, 0x00, 0x00, 0x00},
-       { "Philips PAL_BG (FI1216 and compatibles)",      /* Philips PAL */
-         16*168.25, 16*447.25, 0xA0, 0x90, 0x30, 0x8e},
-       { "Temic NTSC (4032 FY5)",                        /* TEMIC NTSC */
-         16*157.25, 16*463.25, 0x02, 0x04, 0x01, 0x8e},
-       { "Temic PAL_I (4062 FY5)",                       /* TEMIC PAL_I */
-         16*170.00, 16*450.00, 0x02, 0x04, 0x01, 0x8e},
-       { "Temic NTSC (4036 FY5)",                        /* TEMIC NTSC */
-         16*157.25, 16*463.25, 0xa0, 0x90, 0x30, 0x8e},
-       { "Alps HSBH1",                                   /* TEMIC NTSC */
-         16*137.25, 16*385.25, 0x01, 0x02, 0x08, 0x8e},
-
-       /* 10-19 */
-       { "Alps TSBE1",                                   /* TEMIC PAL */
-         16*137.25, 16*385.25, 0x01, 0x02, 0x08, 0x8e},
-       { "Alps TSBB5",                                   /* Alps PAL_I */
-         16*133.25, 16*351.25, 0x01, 0x02, 0x08, 0x8e},
-       { "Alps TSBE5",                                   /* Alps PAL */
-         16*133.25, 16*351.25, 0x01, 0x02, 0x08, 0x8e},
-       { "Alps TSBC5",                                   /* Alps PAL */
-         16*133.25, 16*351.25, 0x01, 0x02, 0x08, 0x8e},
-       { "Temic PAL_BG (4006FH5)",                       /* TEMIC PAL */
-         16*170.00, 16*450.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "Alps TSCH6",                                   /* Alps NTSC */
-         16*137.25, 16*385.25, 0x14, 0x12, 0x11, 0x8e},
-       { "Temic PAL_DK (4016 FY5)",                      /* TEMIC PAL */
-         16*168.25, 16*456.25, 0xa0, 0x90, 0x30, 0x8e},
-       { "Philips NTSC_M (MK2)",                         /* Philips NTSC */
-         16*160.00, 16*454.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "Temic PAL_I (4066 FY5)",                       /* TEMIC PAL_I */
-         16*169.00,  16*454.00,  0xa0, 0x90, 0x30, 0x8e},
-       { "Temic PAL* auto (4006 FN5)",                   /* TEMIC PAL */
-         16*169.00,  16*454.00,  0xa0, 0x90, 0x30, 0x8e},
-
-       /* 20-29 */
-       { "Temic PAL_BG (4009 FR5) or PAL_I (4069 FR5)",  /* TEMIC PAL */
-         16*141.00,  16*464.00,  0xa0, 0x90, 0x30, 0x8e},
-       { "Temic NTSC (4039 FR5)",                        /* TEMIC NTSC */
-         16*158.00,  16*453.00,  0xa0, 0x90, 0x30, 0x8e},
-       { "Temic PAL/SECAM multi (4046 FM5)",             /* TEMIC PAL */
-         16*169.00,  16*454.00,  0xa0, 0x90, 0x30, 0x8e},
-       { "Philips PAL_DK (FI1256 and compatibles)",      /* Philips PAL */
-         16*170.00, 16*450.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "Philips PAL/SECAM multi (FQ1216ME)",           /* Philips PAL */
-         16*170.00, 16*450.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "LG PAL_I+FM (TAPC-I001D)",                     /* LGINNOTEK PAL_I */
-         16*170.00, 16*450.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "LG PAL_I (TAPC-I701D)",                        /* LGINNOTEK PAL_I */
-         16*170.00, 16*450.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "LG NTSC+FM (TPI8NSR01F)",                      /* LGINNOTEK NTSC */
-         16*210.00, 16*497.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "LG PAL_BG+FM (TPI8PSB01D)",                    /* LGINNOTEK PAL */
-         16*170.00, 16*450.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "LG PAL_BG (TPI8PSB11D)",                       /* LGINNOTEK PAL */
-         16*170.00, 16*450.00, 0xa0, 0x90, 0x30, 0x8e},
-
-       /* 30-39 */
-       { "Temic PAL* auto + FM (4009 FN5)",              /* TEMIC PAL */
-         16*141.00,  16*464.00,  0xa0, 0x90, 0x30, 0x8e},
-       { "SHARP NTSC_JP (2U5JF5540)",                    /* SHARP NTSC */
-         16*137.25, 16*317.25, 0x01, 0x02, 0x08, 0x8e},
-       { "Samsung PAL TCPM9091PD27",                     /* Samsung PAL */
-         16*169, 16*464, 0xA0, 0x90, 0x30, 0x8e},
-       { "MT20xx universal",                             /* Microtune PAL|NTSC */
-         /* see mt20xx.c for details */ },
-       { "Temic PAL_BG (4106 FH5)",                      /* TEMIC PAL */
-         16*141.00,  16*464.00,  0xa0, 0x90, 0x30, 0x8e},
-       { "Temic PAL_DK/SECAM_L (4012 FY5)",              /* TEMIC PAL */
-         16*140.25,  16*463.25,  0x02, 0x04, 0x01, 0x8e},
-       { "Temic NTSC (4136 FY5)",                        /* TEMIC NTSC */
-         16*158.00,  16*453.00,  0xa0, 0x90, 0x30, 0x8e},
-       { "LG PAL (newer TAPC series)",                   /* LGINNOTEK PAL */
-         16*170.00,  16*450.00,  0x01, 0x02, 0x08, 0x8e},
-       { "Philips PAL/SECAM multi (FM1216ME MK3)",       /* Philips PAL */
-         16*158.00, 16*442.00, 0x01, 0x02, 0x04, 0x8e},
-       { "LG NTSC (newer TAPC series)",                  /* LGINNOTEK NTSC */
-         16*170.00,  16*450.00,  0x01, 0x02, 0x08, 0x8e},
-
-       /* 40-49 */
-       { "HITACHI V7-J180AT",                            /* HITACHI NTSC */
-         16*170.00,  16*450.00,  0x01, 0x02, 0x08, 0x8e},
-       { "Philips PAL_MK (FI1216 MK)",                   /* Philips PAL */
-         16*140.25, 16*463.25, 0x01, 0xc2, 0xcf, 0x8e},
-       { "Philips 1236D ATSC/NTSC daul in",              /* Philips ATSC */
-         16*157.25, 16*454.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "Philips NTSC MK3 (FM1236MK3 or FM1236/F)",     /* Philips NTSC */
-         16*160.00, 16*442.00, 0x01, 0x02, 0x04, 0x8e},
-       { "Philips 4 in 1 (ATI TV Wonder Pro/Conexant)",  /* Philips NTSC */
-         16*160.00, 16*442.00, 0x01, 0x02, 0x04, 0x8e},
-       { "Microtune 4049 FM5",                           /* Microtune PAL */
-         16*141.00, 16*464.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "Panasonic VP27s/ENGE4324D",                    /* Panasonic NTSC */
-         16*160.00, 16*454.00, 0x01, 0x02, 0x08, 0xce},
-       { "LG NTSC (TAPE series)",                        /* LGINNOTEK NTSC */
-         16*160.00, 16*442.00, 0x01, 0x02, 0x04, 0x8e},
-       { "Tenna TNF 8831 BGFF)",                         /* Philips PAL */
-         16*161.25, 16*463.25, 0xa0, 0x90, 0x30, 0x8e},
-       { "Microtune 4042 FI5 ATSC/NTSC dual in",         /* Microtune NTSC */
-         16*162.00, 16*457.00, 0xa2, 0x94, 0x31, 0x8e},
-
-       /* 50-59 */
-       { "TCL 2002N",                                    /* TCL NTSC */
-         16*172.00, 16*448.00, 0x01, 0x02, 0x08, 0x8e},
-       { "Philips PAL/SECAM_D (FM 1256 I-H3)",           /* Philips PAL */
-         16*160.00, 16*442.00, 0x01, 0x02, 0x04, 0x8e},
-       { "Thomson DDT 7610 (ATSC/NTSC)",                 /* THOMSON ATSC */
-         16*157.25, 16*454.00, 0x39, 0x3a, 0x3c, 0x8e},
-       { "Philips FQ1286",                               /* Philips NTSC */
-         16*160.00, 16*454.00, 0x41, 0x42, 0x04, 0x8e},
-       { "tda8290+75",                                   /* Philips PAL|NTSC */
-         /* see tda8290.c for details */ },
-       { "TCL 2002MB",                                   /* TCL PAL */
-         16*170.00,  16*450.00,  0x01, 0x02, 0x08, 0xce},
-       { "Philips PAL/SECAM multi (FQ1216AME MK4)",      /* Philips PAL */
-         16*160.00, 16*442.00, 0x01, 0x02, 0x04, 0xce},
-       { "Philips FQ1236A MK4",                          /* Philips NTSC */
-         16*160.00, 16*442.00, 0x01, 0x02, 0x04, 0x8e},
-       { "Ymec TVision TVF-8531MF/8831MF/8731MF",        /* Philips NTSC */
-         16*160.00, 16*454.00, 0xa0, 0x90, 0x30, 0x8e},
-       { "Ymec TVision TVF-5533MF",                      /* Philips NTSC */
-         16*160.00, 16*454.00, 0x01, 0x02, 0x04, 0x8e},
-
-       /* 60-69 */
-       { "Thomson DDT 7611 (ATSC/NTSC)",                 /* THOMSON ATSC */
-         16*157.25, 16*454.00, 0x39, 0x3a, 0x3c, 0x8e},
-       { "Tena TNF9533-D/IF/TNF9533-B/DF",               /* Philips PAL */
-         16*160.25, 16*464.25, 0x01, 0x02, 0x04, 0x8e},
-       { "Philips TEA5767HN FM Radio",                   /* Philips RADIO */
-         /* see tea5767.c for details */},
-       { "Philips FMD1216ME MK3 Hybrid Tuner",           /* Philips PAL */
-         16*160.00, 16*442.00, 0x51, 0x52, 0x54, 0x86},
-       { "LG TDVS-H062F/TUA6034",                        /* LGINNOTEK ATSC */
-         16*160.00, 16*455.00, 0x01, 0x02, 0x04, 0x8e},
-       { "Ymec TVF66T5-B/DFF",                           /* Philips PAL */
-         16*160.25, 16*464.25, 0x01, 0x02, 0x08, 0x8e},
-       { "LG NTSC (TALN mini series)",                   /* LGINNOTEK NTSC */
-         16*137.25, 16*373.25, 0x01, 0x02, 0x08, 0x8e},
-       { "Philips TD1316 Hybrid Tuner",                  /* Philips PAL */
-         16*160.00, 16*442.00, 0xa1, 0xa2, 0xa4, 0xc8},
-       { "Philips TUV1236D ATSC/NTSC dual in",           /* Philips ATSC */
-         16*157.25, 16*454.00, 0x01, 0x02, 0x04, 0xce},
-       { "Tena TNF 5335 MF",                             /* Philips NTSC */
-         16*157.25, 16*454.00, 0x01, 0x02, 0x04, 0x8e},
-};
-
-unsigned const int tuner_count = ARRAY_SIZE(tuners);
 
 /* ---------------------------------------------------------------------- */
 
@@ -295,24 +133,31 @@ static int tuner_stereo(struct i2c_client *c)
 static void default_set_tv_freq(struct i2c_client *c, unsigned int freq)
 {
        struct tuner *t = i2c_get_clientdata(c);
-       u8 config, tuneraddr;
+       u8 config, cb, tuneraddr;
        u16 div;
        struct tunertype *tun;
-       unsigned char buffer[4];
-       int rc, IFPCoff;
+       u8 buffer[4];
+       int rc, IFPCoff, i, j;
 
        tun = &tuners[t->type];
-       if (freq < tun->thresh1) {
-               config = tun->VHF_L;
-               tuner_dbg("tv: VHF lowrange\n");
-       } else if (freq < tun->thresh2) {
-               config = tun->VHF_H;
-               tuner_dbg("tv: VHF high range\n");
-       } else {
-               config = tun->UHF;
-               tuner_dbg("tv: UHF range\n");
-       }
+       j = TUNER_PARAM_ANALOG;
 
+       for (i = 0; i < tun->params[j].count; i++) {
+               if (freq > tun->params[j].ranges[i].limit)
+                       continue;
+               break;
+       }
+       if (i == tun->params[j].count) {
+               tuner_dbg("TV frequency out of range (%d > %d)",
+                               freq, tun->params[j].ranges[i - 1].limit);
+               freq = tun->params[j].ranges[--i].limit;
+       }
+       config = tun->params[j].ranges[i].config;
+       cb     = tun->params[j].ranges[i].cb;
+       /*  i == 0 -> VHF_LO  */
+       /*  i == 1 -> VHF_HI  */
+       /*  i == 2 -> UHF     */
+       tuner_dbg("tv: range %d\n",i);
 
        /* tv norm specific stuff for multi-norm tuners */
        switch (t->type) {
@@ -320,40 +165,40 @@ static void default_set_tv_freq(struct i2c_client *c, unsigned int freq)
                /* 0x01 -> ??? no change ??? */
                /* 0x02 -> PAL BDGHI / SECAM L */
                /* 0x04 -> ??? PAL others / SECAM others ??? */
-               config &= ~0x02;
+               cb &= ~0x02;
                if (t->std & V4L2_STD_SECAM)
-                       config |= 0x02;
+                       cb |= 0x02;
                break;
 
        case TUNER_TEMIC_4046FM5:
-               config &= ~0x0f;
+               cb &= ~0x0f;
 
                if (t->std & V4L2_STD_PAL_BG) {
-                       config |= TEMIC_SET_PAL_BG;
+                       cb |= TEMIC_SET_PAL_BG;
 
                } else if (t->std & V4L2_STD_PAL_I) {
-                       config |= TEMIC_SET_PAL_I;
+                       cb |= TEMIC_SET_PAL_I;
 
                } else if (t->std & V4L2_STD_PAL_DK) {
-                       config |= TEMIC_SET_PAL_DK;
+                       cb |= TEMIC_SET_PAL_DK;
 
                } else if (t->std & V4L2_STD_SECAM_L) {
-                       config |= TEMIC_SET_PAL_L;
+                       cb |= TEMIC_SET_PAL_L;
 
                }
                break;
 
        case TUNER_PHILIPS_FQ1216ME:
-               config &= ~0x0f;
+               cb &= ~0x0f;
 
                if (t->std & (V4L2_STD_PAL_BG|V4L2_STD_PAL_DK)) {
-                       config |= PHILIPS_SET_PAL_BGDK;
+                       cb |= PHILIPS_SET_PAL_BGDK;
 
                } else if (t->std & V4L2_STD_PAL_I) {
-                       config |= PHILIPS_SET_PAL_I;
+                       cb |= PHILIPS_SET_PAL_I;
 
                } else if (t->std & V4L2_STD_SECAM_L) {
-                       config |= PHILIPS_SET_PAL_L;
+                       cb |= PHILIPS_SET_PAL_L;
 
                }
                break;
@@ -363,15 +208,15 @@ static void default_set_tv_freq(struct i2c_client *c, unsigned int freq)
                /* 0x01 -> ATSC antenna input 2 */
                /* 0x02 -> NTSC antenna input 1 */
                /* 0x03 -> NTSC antenna input 2 */
-               config &= ~0x03;
+               cb &= ~0x03;
                if (!(t->std & V4L2_STD_ATSC))
-                       config |= 2;
+                       cb |= 2;
                /* FIXME: input */
                break;
 
        case TUNER_MICROTUNE_4042FI5:
                /* Set the charge pump for fast tuning */
-               tun->config |= TUNER_CHARGE_PUMP;
+               config |= TUNER_CHARGE_PUMP;
                break;
 
        case TUNER_PHILIPS_TUV1236D:
@@ -383,9 +228,9 @@ static void default_set_tv_freq(struct i2c_client *c, unsigned int freq)
                buffer[1] = 0x00;
                buffer[2] = 0x17;
                buffer[3] = 0x00;
-               config &= ~0x40;
+               cb &= ~0x40;
                if (t->std & V4L2_STD_ATSC) {
-                       config |= 0x40;
+                       cb |= 0x40;
                        buffer[1] = 0x04;
                }
                /* set to the correct mode (analog or digital) */
@@ -400,20 +245,6 @@ static void default_set_tv_freq(struct i2c_client *c, unsigned int freq)
                break;
        }
 
-       /*
-        * Philips FI1216MK2 remark from specification :
-        * for channel selection involving band switching, and to ensure
-        * smooth tuning to the desired channel without causing
-        * unnecessary charge pump action, it is recommended to consider
-        * the difference between wanted channel frequency and the
-        * current channel frequency.  Unnecessary charge pump action
-        * will result in very low tuning voltage which may drive the
-        * oscillator to extreme conditions.
-        *
-        * Progfou: specification says to send config data before
-        * frequency in case (wanted frequency < current frequency).
-        */
-
        /* IFPCoff = Video Intermediate Frequency - Vif:
                940  =16*58.75  NTSC/J (Japan)
                732  =16*45.75  M/N STD
@@ -426,28 +257,37 @@ static void default_set_tv_freq(struct i2c_client *c, unsigned int freq)
                171.2=16*10.70  FM Radio (at set_radio_freq)
        */
 
-       if (t->std & V4L2_STD_NTSC_M_JP) {
+       if (t->std == V4L2_STD_NTSC_M_JP) {
                IFPCoff = 940;
-       } else if (t->std & V4L2_STD_MN) {
+       } else if ((t->std & V4L2_STD_MN) &&
+                 !(t->std & ~V4L2_STD_MN)) {
                IFPCoff = 732;
-       } else if (t->std & V4L2_STD_SECAM_LC) {
+       } else if (t->std == V4L2_STD_SECAM_LC) {
                IFPCoff = 543;
        } else {
                IFPCoff = 623;
        }
 
-       div=freq + IFPCoff;
-       if (t->type == TUNER_PHILIPS_SECAM && freq < t->freq) {
-               buffer[0] = tun->config;
-               buffer[1] = config;
+       div=freq + IFPCoff + offset;
+
+       tuner_dbg("Freq= %d.%02d MHz, V_IF=%d.%02d MHz, Offset=%d.%02d MHz, div=%0d\n",
+                                       freq / 16, freq % 16 * 100 / 16,
+                                       IFPCoff / 16, IFPCoff % 16 * 100 / 16,
+                                       offset / 16, offset % 16 * 100 / 16,
+                                       div);
+
+       if (tuners[t->type].params->cb_first_if_lower_freq && div < t->last_div) {
+               buffer[0] = config;
+               buffer[1] = cb;
                buffer[2] = (div>>8) & 0x7f;
                buffer[3] = div      & 0xff;
        } else {
                buffer[0] = (div>>8) & 0x7f;
                buffer[1] = div      & 0xff;
-               buffer[2] = tun->config;
-               buffer[3] = config;
+               buffer[2] = config;
+               buffer[3] = cb;
        }
+       t->last_div = div;
        tuner_dbg("tv 0x%02x 0x%02x 0x%02x 0x%02x\n",
                  buffer[0],buffer[1],buffer[2],buffer[3]);
 
@@ -473,11 +313,11 @@ static void default_set_tv_freq(struct i2c_client *c, unsigned int freq)
                }
 
                /* Set the charge pump for optimized phase noise figure */
-               tun->config &= ~TUNER_CHARGE_PUMP;
+               config &= ~TUNER_CHARGE_PUMP;
                buffer[0] = (div>>8) & 0x7f;
                buffer[1] = div      & 0xff;
-               buffer[2] = tun->config;
-               buffer[3] = config;
+               buffer[2] = config;
+               buffer[3] = cb;
                tuner_dbg("tv 0x%02x 0x%02x 0x%02x 0x%02x\n",
                       buffer[0],buffer[1],buffer[2],buffer[3]);
 
@@ -490,13 +330,15 @@ static void default_set_radio_freq(struct i2c_client *c, unsigned int freq)
 {
        struct tunertype *tun;
        struct tuner *t = i2c_get_clientdata(c);
-       unsigned char buffer[4];
-       unsigned div;
-       int rc;
+       u8 buffer[4];
+       u16 div;
+       int rc, j;
 
        tun = &tuners[t->type];
+       j = TUNER_PARAM_ANALOG;
+
        div = (20 * freq / 16000) + (int)(20*10.7); /* IF 10.7 MHz */
-       buffer[2] = (tun->config & ~TUNER_RATIO_MASK) | TUNER_RATIO_SELECT_50; /* 50 kHz step */
+       buffer[2] = (tun->params[j].ranges[0].config & ~TUNER_RATIO_MASK) | TUNER_RATIO_SELECT_50; /* 50 kHz step */
 
        switch (t->type) {
        case TUNER_TENA_9533_DI:
@@ -525,9 +367,19 @@ static void default_set_radio_freq(struct i2c_client *c, unsigned int freq)
        }
        buffer[0] = (div>>8) & 0x7f;
        buffer[1] = div      & 0xff;
+       if (tuners[t->type].params->cb_first_if_lower_freq && div < t->last_div) {
+               buffer[0] = buffer[2];
+               buffer[1] = buffer[3];
+               buffer[2] = (div>>8) & 0x7f;
+               buffer[3] = div      & 0xff;
+       } else {
+               buffer[0] = (div>>8) & 0x7f;
+               buffer[1] = div      & 0xff;
+       }
 
        tuner_dbg("radio 0x%02x 0x%02x 0x%02x 0x%02x\n",
               buffer[0],buffer[1],buffer[2],buffer[3]);
+       t->last_div = div;
 
        if (4 != (rc = i2c_master_send(c,buffer,4)))
                tuner_warn("i2c i/o error: rc == %d (should be 4)\n",rc);
@@ -541,10 +393,10 @@ int default_tuner_init(struct i2c_client *c)
                   t->type, tuners[t->type].name);
        strlcpy(c->name, tuners[t->type].name, sizeof(c->name));
 
-       t->tv_freq    = default_set_tv_freq;
-       t->radio_freq = default_set_radio_freq;
+       t->set_tv_freq = default_set_tv_freq;
+       t->set_radio_freq = default_set_radio_freq;
        t->has_signal = tuner_signal;
-       t->is_stereo  = tuner_stereo;
+       t->is_stereo = tuner_stereo;
        t->standby = NULL;
 
        return 0;