V4L/DVB (6131): tea5761: convert from tuner sub-driver into dvb_frontend module

Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Acked-by: Hans Verkuil <hverkuil@xs4all.nl>
Acked-by: Mike Isely <isely@pobox.com>
Acked-by: Steven Toth <stoth@hauppauge.com>
Acked-by: Patrick Boettcher <pb@linuxtv.org>
Acked-by: Jarod Wilson <jwilson@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>

diff --git a/drivers/media/video/tea5761.c b/drivers/media/video/tea5761.c
index 9965ba4..8cdaf47 100644 (file)
--- a/drivers/media/video/tea5761.c
+++ b/drivers/media/video/tea5761.c
@@ -11,15 +11,19 @@
 #include <linux/delay.h>
 #include <linux/videodev.h>
 #include <media/tuner.h>
-#include "tuner-driver.h"
+#include "tuner-i2c.h"
+#include "tea5761.h"
 
-#define PREFIX "TEA5761 "
+static int debug = 0;
+module_param(debug, int, 0644);
+MODULE_PARM_DESC(debug, "enable verbose debug messages");
 
-/* from tuner-core.c */
-extern int tuner_debug;
+#define PREFIX "tea5761 "
 
 struct tea5761_priv {
        struct tuner_i2c_props i2c_props;
+
+       u32 frequency;
 };
 
 /*****************************************************************************/
@@ -118,11 +122,6 @@ struct tea5761_priv {
 
 /*****************************************************************************/
 
-static void set_tv_freq(struct tuner *t, unsigned int freq)
-{
-       tuner_warn("This tuner doesn't support TV freq.\n");
-}
-
 #define FREQ_OFFSET 0 /* for TEA5767, it is 700 to give the right freq */
 static void tea5761_status_dump(unsigned char *buffer)
 {
@@ -137,16 +136,18 @@ static void tea5761_status_dump(unsigned char *buffer)
 }
 
 /* Freq should be specifyed at 62.5 Hz */
-static void set_radio_freq(struct tuner *t, unsigned int frq)
+static int set_radio_freq(struct dvb_frontend *fe,
+                         struct analog_parameters *params)
 {
-       struct tea5761_priv *priv = t->priv;
+       struct tea5761_priv *priv = fe->tuner_priv;
+       unsigned int frq = params->frequency;
        unsigned char buffer[7] = {0, 0, 0, 0, 0, 0, 0 };
        unsigned div;
        int rc;
 
-       tuner_dbg (PREFIX "radio freq counter %d\n", frq);
+       tuner_dbg("radio freq counter %d\n", frq);
 
-       if (t->mode == T_STANDBY) {
+       if (params->mode == T_STANDBY) {
                tuner_dbg("TEA5761 set to standby mode\n");
                buffer[5] |= TEA5761_TNCTRL_MU;
        } else {
@@ -154,10 +155,9 @@ static void set_radio_freq(struct tuner *t, unsigned int frq)
        }
 
 
-       if (t->audmode == V4L2_TUNER_MODE_MONO) {
+       if (params->audmode == V4L2_TUNER_MODE_MONO) {
                tuner_dbg("TEA5761 set to mono\n");
                buffer[5] |= TEA5761_TNCTRL_MST;
-;
        } else {
                tuner_dbg("TEA5761 set to stereo\n");
        }
@@ -166,18 +166,22 @@ static void set_radio_freq(struct tuner *t, unsigned int frq)
        buffer[1] = (div >> 8) & 0x3f;
        buffer[2] = div & 0xff;
 
-       if (tuner_debug)
+       if (debug)
                tea5761_status_dump(buffer);
 
        if (7 != (rc = tuner_i2c_xfer_send(&priv->i2c_props, buffer, 7)))
                tuner_warn("i2c i/o error: rc == %d (should be 5)\n", rc);
+
+       priv->frequency = frq * 125 / 2;
+
+       return 0;
 }
 
-static int tea5761_signal(struct tuner *t)
+static int tea5761_signal(struct dvb_frontend *fe)
 {
        unsigned char buffer[16];
        int rc;
-       struct tea5761_priv *priv = t->priv;
+       struct tea5761_priv *priv = fe->tuner_priv;
 
        memset(buffer, 0, sizeof(buffer));
        if (16 != (rc = tuner_i2c_xfer_recv(&priv->i2c_props, buffer, 16)))
@@ -186,11 +190,11 @@ static int tea5761_signal(struct tuner *t)
        return ((buffer[9] & TEA5761_TUNCHECK_LEV_MASK) << (13 - 4));
 }
 
-static int tea5761_stereo(struct tuner *t)
+static int tea5761_stereo(struct dvb_frontend *fe)
 {
        unsigned char buffer[16];
        int rc;
-       struct tea5761_priv *priv = t->priv;
+       struct tea5761_priv *priv = fe->tuner_priv;
 
        memset(buffer, 0, sizeof(buffer));
        if (16 != (rc = tuner_i2c_xfer_recv(&priv->i2c_props, buffer, 16)))
@@ -203,58 +207,96 @@ static int tea5761_stereo(struct tuner *t)
        return (rc ? V4L2_TUNER_SUB_STEREO : 0);
 }
 
-int tea5761_autodetection(struct tuner *t)
+static int tea5761_get_status(struct dvb_frontend *fe, u32 *status)
+{
+       struct tea5761_priv *priv = fe->tuner_priv;
+       int signal = tea5761_signal(fe);
+
+       *status = 0;
+
+       if (signal)
+               *status = TUNER_STATUS_LOCKED;
+       if (tea5761_stereo(fe))
+               *status |= TUNER_STATUS_STEREO;
+
+       tuner_dbg("tea5761: Signal strength: %d\n", signal);
+
+       return 0;
+}
+
+int tea5761_autodetection(struct i2c_adapter* i2c_adap, u8 i2c_addr)
 {
        unsigned char buffer[16];
        int rc;
-       struct tuner_i2c_props i2c = { .adap = t->i2c.adapter, .addr = t->i2c.addr };
+       struct tuner_i2c_props i2c = { .adap = i2c_adap, .addr = i2c_addr };
 
        if (16 != (rc = tuner_i2c_xfer_recv(&i2c, buffer, 16))) {
-               tuner_warn("it is not a TEA5761. Received %i chars.\n", rc);
+               printk(KERN_WARNING "it is not a TEA5761. Received %i chars.\n", rc);
                return EINVAL;
        }
 
        if (!((buffer[13] != 0x2b) || (buffer[14] != 0x57) || (buffer[15] != 0x061))) {
-               tuner_warn("Manufacturer ID= 0x%02x, Chip ID = %02x%02x. It is not a TEA5761\n",buffer[13],buffer[14],buffer[15]);
+               printk(KERN_WARNING "Manufacturer ID= 0x%02x, Chip ID = %02x%02x. It is not a TEA5761\n",buffer[13],buffer[14],buffer[15]);
                return EINVAL;
        }
-       tuner_warn("TEA5761 detected.\n");
+       printk(KERN_WARNING "TEA5761 detected.\n");
+       return 0;
+}
+
+static int tea5761_release(struct dvb_frontend *fe)
+{
+       kfree(fe->tuner_priv);
+       fe->tuner_priv = NULL;
+
        return 0;
 }
 
-static void tea5761_release(struct tuner *t)
+static int tea5761_get_frequency(struct dvb_frontend *fe, u32 *frequency)
 {
-       kfree(t->priv);
-       t->priv = NULL;
+       struct tea5761_priv *priv = fe->tuner_priv;
+       *frequency = priv->frequency;
+       return 0;
 }
 
-static struct tuner_operations tea5761_tuner_ops = {
-       .set_tv_freq    = set_tv_freq,
-       .set_radio_freq = set_radio_freq,
-       .has_signal     = tea5761_signal,
-       .is_stereo      = tea5761_stereo,
-       .release        = tea5761_release,
+static struct dvb_tuner_ops tea5761_tuner_ops = {
+       .info = {
+               .name           = "tea5761", // Philips TEA5761HN FM Radio
+       },
+       .set_analog_params = set_radio_freq,
+       .release           = tea5761_release,
+       .get_frequency     = tea5761_get_frequency,
+       .get_status        = tea5761_get_status,
 };
 
-int tea5761_tuner_init(struct tuner *t)
+struct dvb_frontend *tea5761_attach(struct dvb_frontend *fe,
+                                   struct i2c_adapter* i2c_adap,
+                                   u8 i2c_addr)
 {
        struct tea5761_priv *priv = NULL;
 
-       if (tea5761_autodetection(t) == EINVAL)
-               return EINVAL;
+       if (tea5761_autodetection(i2c_adap, i2c_addr) == EINVAL)
+               return NULL;
 
        priv = kzalloc(sizeof(struct tea5761_priv), GFP_KERNEL);
        if (priv == NULL)
-               return -ENOMEM;
-       t->priv = priv;
+               return NULL;
+       fe->tuner_priv = priv;
 
-       priv->i2c_props.addr = t->i2c.addr;
-       priv->i2c_props.adap = t->i2c.adapter;
+       priv->i2c_props.addr = i2c_addr;
+       priv->i2c_props.adap = i2c_adap;
 
-       tuner_info("type set to %d (%s)\n", t->type, "Philips TEA5761HN FM Radio");
-       strlcpy(t->i2c.name, "tea5761", sizeof(t->i2c.name));
+       memcpy(&fe->ops.tuner_ops, &tea5761_tuner_ops,
+              sizeof(struct dvb_tuner_ops));
 
-       memcpy(&t->ops, &tea5761_tuner_ops, sizeof(struct tuner_operations));
+       tuner_info("type set to %s\n", "Philips TEA5761HN FM Radio");
 
-       return (0);
+       return fe;
 }
+
+
+EXPORT_SYMBOL_GPL(tea5761_attach);
+EXPORT_SYMBOL_GPL(tea5761_autodetection);
+
+MODULE_DESCRIPTION("Philips TEA5761 FM tuner driver");
+MODULE_AUTHOR("Mauro Carvalho Chehab <mchehab@infradead.org>");
+MODULE_LICENSE("GPL");

diff --git a/drivers/media/video/tea5761.h b/drivers/media/video/tea5761.h
new file mode 100644 (file)
index 0000000..f287c02
--- /dev/null
+++ b/drivers/media/video/tea5761.h
@@ -0,0 +1,28 @@
+/*
+    This program is free software; you can redistribute it and/or modify
+    it under the terms of the GNU General Public License as published by
+    the Free Software Foundation; either version 2 of the License, or
+    (at your option) any later version.
+
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+    GNU General Public License for more details.
+
+    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.
+*/
+
+#ifndef __TEA5761_H__
+#define __TEA5761_H__
+
+#include <linux/i2c.h>
+#include "dvb_frontend.h"
+
+extern int tea5761_autodetection(struct i2c_adapter* i2c_adap, u8 i2c_addr);
+extern struct dvb_frontend *tea5761_attach(struct dvb_frontend *fe,
+                                          struct i2c_adapter* i2c_adap,
+                                          u8 i2c_addr);
+
+#endif /* __TEA5761_H__ */

diff --git a/drivers/media/video/tuner-core.c b/drivers/media/video/tuner-core.c
index f24ec1a..848ee64 100644 (file)
--- a/drivers/media/video/tuner-core.c
+++ b/drivers/media/video/tuner-core.c
@@ -21,6 +21,7 @@
 #include "tuner-driver.h"
 #include "mt20xx.h"
 #include "tda8290.h"
+#include "tea5761.h"
 
 #define UNSET (-1U)
 
@@ -270,7 +271,7 @@ static void set_type(struct i2c_client *c, unsigned int type,
                break;
 #ifdef CONFIG_TUNER_TEA5761
        case TUNER_TEA5761:
-               if (tea5761_tuner_init(t) == EINVAL) {
+               if (tea5761_attach(&t->fe, t->i2c.adapter, t->i2c.addr) == NULL) {
                        t->type = TUNER_ABSENT;
                        t->mode_mask = T_UNINITIALIZED;
                        return;
@@ -571,7 +572,7 @@ static int tuner_attach(struct i2c_adapter *adap, int addr, int kind)
                switch (addr) {
 #ifdef CONFIG_TUNER_TEA5761
                case 0x10:
-                       if (tea5761_autodetection(t) != EINVAL) {
+                       if (tea5761_autodetection(t->i2c.adapter, t->i2c.addr) != EINVAL) {
                                t->type = TUNER_TEA5761;
                                t->mode_mask = T_RADIO;
                                t->mode = T_STANDBY;

diff --git a/drivers/media/video/tuner-driver.h b/drivers/media/video/tuner-driver.h
index 87d937f..bcb6a61 100644 (file)
--- a/drivers/media/video/tuner-driver.h
+++ b/drivers/media/video/tuner-driver.h
@@ -76,9 +76,6 @@ extern int default_tuner_init(struct tuner *t);
 
 extern int tda9887_tuner_init(struct tuner *t);
 
-extern int tea5761_tuner_init(struct tuner *t);
-extern int tea5761_autodetection(struct tuner *t);
-
 extern int tea5767_autodetection(struct tuner *t);
 extern int tea5767_tuner_init(struct tuner *t);