V4L/DVB (5632): Dvb-pll: pass dvb_frontend_parameters to generic set() function

Rename dvb_pll_desc.setbw() to set(), and accept struct dvb_frontend_parameters
instead of passing both freq and bandwidth, so that this may be used as a
generic function.

In order to do this, dvb_pll_configure must also be altered in the same manner,
to take struct dvb_frontend_parameters instead of freq and bandwidth.

Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Trent Piepho <xyzzy@speakeasy.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>

6 files changed, 49 insertions, 62 deletions

diff --git a/drivers/media/dvb/dvb-usb/dvb-usb-i2c.c b/drivers/media/dvb/dvb-usb/dvb-usb-i2c.c
index 088b6dee3a7f..4c80823d8d07 100644
--- a/drivers/media/dvb/dvb-usb/dvb-usb-i2c.c
+++ b/drivers/media/dvb/dvb-usb/dvb-usb-i2c.c
@@ -90,7 +90,7 @@ int dvb_usb_tuner_calc_regs(struct dvb_frontend *fe, struct dvb_frontend_paramet
         deb_pll("pll addr: %x, freq: %d %p\n",adap->pll_addr, fep->frequency, adap->pll_desc);
 
         b[0] = adap->pll_addr;
-        dvb_pll_configure(adap->pll_desc, &b[1], fep->frequency, fep->u.ofdm.bandwidth);
+        dvb_pll_configure(adap->pll_desc, &b[1], fep);
 
         deb_pll("pll-buf: %x %x %x %x %x\n",b[0],b[1],b[2],b[3],b[4]);
 
diff --git a/drivers/media/dvb/frontends/dvb-pll.c b/drivers/media/dvb/frontends/dvb-pll.c
index 5f96ffda91ad..0e3195f95756 100644
--- a/drivers/media/dvb/frontends/dvb-pll.c
+++ b/drivers/media/dvb/frontends/dvb-pll.c
@@ -68,9 +68,10 @@ struct dvb_pll_desc dvb_pll_thomson_dtt7610 = {
 };
 EXPORT_SYMBOL(dvb_pll_thomson_dtt7610);
 
-static void thomson_dtt759x_bw(u8 *buf, u32 freq, int bandwidth)
+static void thomson_dtt759x_bw(u8 *buf,
+                               const struct dvb_frontend_parameters *params)
 {
-        if (BANDWIDTH_7_MHZ == bandwidth)
+        if (BANDWIDTH_7_MHZ == params->u.ofdm.bandwidth)
                 buf[3] |= 0x10;
 }
 
@@ -78,7 +79,7 @@ struct dvb_pll_desc dvb_pll_thomson_dtt759x = {
         .name  = "Thomson dtt759x",
         .min   = 177000000,
         .max   = 896000000,
-        .setbw = thomson_dtt759x_bw,
+        .set   = thomson_dtt759x_bw,
         .iffreq= 36166667,
         .sleepdata = (u8[]){ 2, 0x84, 0x03 },
         .count = 5,
@@ -195,9 +196,9 @@ EXPORT_SYMBOL(dvb_pll_env57h1xd5);
 /* Philips TDA6650/TDA6651
  * used in Panasonic ENV77H11D5
  */
-static void tda665x_bw(u8 *buf, u32 freq, int bandwidth)
+static void tda665x_bw(u8 *buf, const struct dvb_frontend_parameters *params)
 {
-        if (bandwidth == BANDWIDTH_8_MHZ)
+        if (params->u.ofdm.bandwidth == BANDWIDTH_8_MHZ)
                 buf[3] |= 0x08;
 }
 
@@ -205,7 +206,7 @@ struct dvb_pll_desc dvb_pll_tda665x = {
         .name  = "Philips TDA6650/TDA6651",
         .min   =  44250000,
         .max   = 858000000,
-        .setbw = tda665x_bw,
+        .set   = tda665x_bw,
         .iffreq= 36166667,
         .count = 12,
         .entries = {
@@ -228,9 +229,9 @@ EXPORT_SYMBOL(dvb_pll_tda665x);
 /* Infineon TUA6034
  * used in LG TDTP E102P
  */
-static void tua6034_bw(u8 *buf, u32 freq, int bandwidth)
+static void tua6034_bw(u8 *buf, const struct dvb_frontend_parameters *params)
 {
-        if (BANDWIDTH_7_MHZ != bandwidth)
+        if (BANDWIDTH_7_MHZ != params->u.ofdm.bandwidth)
                 buf[3] |= 0x08;
 }
 
@@ -240,7 +241,7 @@ struct dvb_pll_desc dvb_pll_tua6034 = {
         .max   = 858000000,
         .iffreq= 36166667,
         .count = 3,
-        .setbw = tua6034_bw,
+        .set   = tua6034_bw,
         .entries = {
                 {  174500000, 62500, 0xce, 0x01 },
                 {  230000000, 62500, 0xce, 0x02 },
@@ -270,9 +271,10 @@ EXPORT_SYMBOL(dvb_pll_lg_tdvs_h06xf);
 /* Philips FMD1216ME
  * used in Medion Hybrid PCMCIA card and USB Box
  */
-static void fmd1216me_bw(u8 *buf, u32 freq, int bandwidth)
+static void fmd1216me_bw(u8 *buf, const struct dvb_frontend_parameters *params)
 {
-        if (bandwidth == BANDWIDTH_8_MHZ && freq >= 158870000)
+        if (params->u.ofdm.bandwidth == BANDWIDTH_8_MHZ &&
+            params->frequency >= 158870000)
                 buf[3] |= 0x08;
 }
 
@@ -281,7 +283,7 @@ struct dvb_pll_desc dvb_pll_fmd1216me = {
         .min = 50870000,
         .max = 858000000,
         .iffreq= 36125000,
-        .setbw = fmd1216me_bw,
+        .set   = fmd1216me_bw,
         .count = 7,
         .entries = {
                 { 143870000, 166667, 0xbc, 0x41 },
@@ -298,9 +300,9 @@ EXPORT_SYMBOL(dvb_pll_fmd1216me);
 /* ALPS TDED4
  * used in Nebula-Cards and USB boxes
  */
-static void tded4_bw(u8 *buf, u32 freq, int bandwidth)
+static void tded4_bw(u8 *buf, const struct dvb_frontend_parameters *params)
 {
-        if (bandwidth == BANDWIDTH_8_MHZ)
+        if (params->u.ofdm.bandwidth == BANDWIDTH_8_MHZ)
                 buf[3] |= 0x04;
 }
 
@@ -309,7 +311,7 @@ struct dvb_pll_desc dvb_pll_tded4 = {
         .min = 47000000,
         .max = 863000000,
         .iffreq= 36166667,
-        .setbw = tded4_bw,
+        .set   = tded4_bw,
         .count = 4,
         .entries = {
                 { 153000000, 166667, 0x85, 0x01 },
@@ -396,14 +398,14 @@ EXPORT_SYMBOL(dvb_pll_philips_sd1878_tda8261);
 /*
  * Philips TD1316 Tuner.
  */
-static void td1316_bw(u8 *buf, u32 freq, int bandwidth)
+static void td1316_bw(u8 *buf, const struct dvb_frontend_parameters *params)
 {
         u8 band;
 
         /* determine band */
-        if (freq < 161000000)
+        if (params->frequency < 161000000)
                 band = 1;
-        else if (freq < 444000000)
+        else if (params->frequency < 444000000)
                 band = 2;
         else
                 band = 4;
@@ -411,7 +413,7 @@ static void td1316_bw(u8 *buf, u32 freq, int bandwidth)
         buf[3] |= band;
 
         /* setup PLL filter */
-        if (bandwidth == BANDWIDTH_8_MHZ)
+        if (params->u.ofdm.bandwidth == BANDWIDTH_8_MHZ)
                 buf[3] |= 1 << 3;
 }
 
@@ -420,7 +422,7 @@ struct dvb_pll_desc dvb_pll_philips_td1316 = {
         .min   =  87000000,
         .max   = 895000000,
         .iffreq= 36166667,
-        .setbw = td1316_bw,
+        .set   = td1316_bw,
         .count = 9,
         .entries = {
                 {  93834000, 166667, 0xca, 0x60},
@@ -451,9 +453,9 @@ struct dvb_pll_desc dvb_pll_thomson_fe6600 = {
         }
 };
 EXPORT_SYMBOL(dvb_pll_thomson_fe6600);
-static void opera1_bw(u8 *buf, u32 freq, int bandwidth)
+static void opera1_bw(u8 *buf, const struct dvb_frontend_parameters *params)
 {
-        if (bandwidth == BANDWIDTH_8_MHZ)
+        if (params->u.ofdm.bandwidth == BANDWIDTH_8_MHZ)
                 buf[2] |= 0x08;
 }
 
@@ -462,7 +464,7 @@ struct dvb_pll_desc dvb_pll_opera1 = {
         .min   =  900000,
         .max   = 2250000,
         .iffreq= 0,
-        .setbw = opera1_bw,
+        .set   = opera1_bw,
         .count = 8,
         .entries = {
                 { 1064000, 500, 0xe5, 0xc6 },
@@ -498,34 +500,36 @@ module_param(debug, int, 0644);
 MODULE_PARM_DESC(debug, "enable verbose debug messages");
 
 int dvb_pll_configure(struct dvb_pll_desc *desc, u8 *buf,
-                      u32 freq, int bandwidth)
+                      const struct dvb_frontend_parameters *params)
 {
         u32 div;
         int i;
 
-        if (freq != 0 && (freq < desc->min || freq > desc->max))
-            return -EINVAL;
+        if (params->frequency != 0 && (params->frequency < desc->min ||
+                                       params->frequency > desc->max))
+                return -EINVAL;
 
         for (i = 0; i < desc->count; i++) {
-                if (freq > desc->entries[i].limit)
+                if (params->frequency > desc->entries[i].limit)
                         continue;
                 break;
         }
+
         if (debug)
-                printk("pll: %s: freq=%d bw=%d | i=%d/%d\n",
-                       desc->name, freq, bandwidth, i, desc->count);
+                printk("pll: %s: freq=%d | i=%d/%d\n", desc->name,
+                       params->frequency, i, desc->count);
         if (i == desc->count)
                 return -EINVAL;
 
-        div = (freq + desc->iffreq + desc->entries[i].stepsize/2) /
-              desc->entries[i].stepsize;
+        div = (params->frequency + desc->iffreq +
+               desc->entries[i].stepsize/2) / desc->entries[i].stepsize;
         buf[0] = div >> 8;
         buf[1] = div & 0xff;
         buf[2] = desc->entries[i].config;
         buf[3] = desc->entries[i].cb;
 
-        if (desc->setbw)
-                desc->setbw(buf, freq, bandwidth);
+        if (desc->set)
+                desc->set(buf, params);
 
         if (debug)
                 printk("pll: %s: div=%d | buf=0x%02x,0x%02x,0x%02x,0x%02x\n",
@@ -578,18 +582,12 @@ static int dvb_pll_set_params(struct dvb_frontend *fe,
                 { .addr = priv->pll_i2c_address, .flags = 0,
                   .buf = buf, .len = sizeof(buf) };
         int result;
-        u32 bandwidth = 0, frequency = 0;
+        u32 frequency = 0;
 
         if (priv->i2c == NULL)
                 return -EINVAL;
 
-        // DVBT bandwidth only just now
-        if (fe->ops.info.type == FE_OFDM) {
-                bandwidth = params->u.ofdm.bandwidth;
-        }
-
-        if ((result = dvb_pll_configure(priv->pll_desc, buf,
-                                        params->frequency, bandwidth)) < 0)
+        if ((result = dvb_pll_configure(priv->pll_desc, buf, params)) < 0)
                 return result;
         else
                 frequency = result;
@@ -601,7 +599,7 @@ static int dvb_pll_set_params(struct dvb_frontend *fe,
         }
 
         priv->frequency = frequency;
-        priv->bandwidth = bandwidth;
+        priv->bandwidth = (fe->ops.info.type == FE_OFDM) ? params->u.ofdm.bandwidth : 0;
 
         return 0;
 }
@@ -612,18 +610,12 @@ static int dvb_pll_calc_regs(struct dvb_frontend *fe,
 {
         struct dvb_pll_priv *priv = fe->tuner_priv;
         int result;
-        u32 bandwidth = 0, frequency = 0;
+        u32 frequency = 0;
 
         if (buf_len < 5)
                 return -EINVAL;
 
-        // DVBT bandwidth only just now
-        if (fe->ops.info.type == FE_OFDM) {
-                bandwidth = params->u.ofdm.bandwidth;
-        }
-
-        if ((result = dvb_pll_configure(priv->pll_desc, buf+1,
-                                        params->frequency, bandwidth)) < 0)
+        if ((result = dvb_pll_configure(priv->pll_desc, buf+1, params)) < 0)
                 return result;
         else
                 frequency = result;
@@ -631,7 +623,7 @@ static int dvb_pll_calc_regs(struct dvb_frontend *fe,
         buf[0] = priv->pll_i2c_address;
 
         priv->frequency = frequency;
-        priv->bandwidth = bandwidth;
+        priv->bandwidth = (fe->ops.info.type == FE_OFDM) ? params->u.ofdm.bandwidth : 0;
 
         return 5;
 }
diff --git a/drivers/media/dvb/frontends/dvb-pll.h b/drivers/media/dvb/frontends/dvb-pll.h
index 5209f46f0893..83f1279da77f 100644
--- a/drivers/media/dvb/frontends/dvb-pll.h
+++ b/drivers/media/dvb/frontends/dvb-pll.h
@@ -13,7 +13,7 @@ struct dvb_pll_desc {
         u32  min;
         u32  max;
         u32  iffreq;
-        void (*setbw)(u8 *buf, u32 freq, int bandwidth);
+        void (*set)(u8 *buf, const struct dvb_frontend_parameters *params);
         u8   *initdata;
         u8   *sleepdata;
         int  count;
@@ -51,7 +51,7 @@ extern struct dvb_pll_desc dvb_pll_thomson_fe6600;
 extern struct dvb_pll_desc dvb_pll_opera1;
 
 extern int dvb_pll_configure(struct dvb_pll_desc *desc, u8 *buf,
-                             u32 freq, int bandwidth);
+                             const struct dvb_frontend_parameters *params);
 
 /**
  * Attach a dvb-pll to the supplied frontend structure.
diff --git a/drivers/media/dvb/ttpci/budget-av.c b/drivers/media/dvb/ttpci/budget-av.c
index 0e817d6f1ce5..8de19cefb24d 100644
--- a/drivers/media/dvb/ttpci/budget-av.c
+++ b/drivers/media/dvb/ttpci/budget-av.c
@@ -839,8 +839,7 @@ static int philips_sd1878_tda8261_tuner_set_params(struct dvb_frontend *fe,
         if((params->frequency < 950000) || (params->frequency > 2150000))
                 return -EINVAL;
 
-        rc=dvb_pll_configure(&dvb_pll_philips_sd1878_tda8261, buf,
-                             params->frequency, 0);
+        rc=dvb_pll_configure(&dvb_pll_philips_sd1878_tda8261, buf, params);
         if(rc < 0) return rc;
 
         if (fe->ops.i2c_gate_ctrl)
diff --git a/drivers/media/video/cx88/cx88-dvb.c b/drivers/media/video/cx88/cx88-dvb.c
index 796083f00910..770ea64369ef 100644
--- a/drivers/media/video/cx88/cx88-dvb.c
+++ b/drivers/media/video/cx88/cx88-dvb.c
@@ -260,9 +260,7 @@ static int dntv_live_dvbt_pro_tuner_set_params(struct dvb_frontend* fe,
                 return err;
 
         /* Tune PLL */
-        dvb_pll_configure(dev->core->pll_desc, buf,
-                          params->frequency,
-                          params->u.ofdm.bandwidth);
+        dvb_pll_configure(dev->core->pll_desc, buf, params);
         if (fe->ops.i2c_gate_ctrl)
                 fe->ops.i2c_gate_ctrl(fe, 1);
         if ((err = i2c_transfer(&dev->core->i2c_adap, &msg, 1)) != 1) {
diff --git a/drivers/media/video/saa7134/saa7134-dvb.c b/drivers/media/video/saa7134/saa7134-dvb.c
index e0eec80088c7..d6dd6d55b595 100644
--- a/drivers/media/video/saa7134/saa7134-dvb.c
+++ b/drivers/media/video/saa7134/saa7134-dvb.c
@@ -181,9 +181,7 @@ static int mt352_aver777_tuner_calc_regs(struct dvb_frontend *fe, struct dvb_fro
                 return -EINVAL;
 
         pllbuf[0] = 0x61;
-        dvb_pll_configure(&dvb_pll_philips_td1316, pllbuf+1,
-                          params->frequency,
-                          params->u.ofdm.bandwidth);
+        dvb_pll_configure(&dvb_pll_philips_td1316, pllbuf+1, params);
         return 5;
 }