Skip to content

Commit

Permalink
V4L/DVB (7127): tuner: remove dependency of tuner-core on tuner-types
Browse files Browse the repository at this point in the history
This patch fully removes the dependency of tuner-core on
tuner-types.

There is no longer any need to pass struct tunertype in attach-time
config structure - instead pass the tuner type ID.

Signed-off-by: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
  • Loading branch information
Michael Krufky authored and Mauro Carvalho Chehab committed Apr 24, 2008
1 parent 65e8d29 commit 060a5bd
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 22 deletions.
11 changes: 3 additions & 8 deletions drivers/media/video/tuner-core.c
Original file line number Diff line number Diff line change
Expand Up @@ -313,18 +313,13 @@ static void tuner_i2c_address_check(struct tuner *t)
tuner_warn("output to v4l-dvb-maintainer@linuxtv.org\n");
tuner_warn("Please use subject line: \"obsolete tuner i2c address.\"\n");
tuner_warn("driver: %s, addr: 0x%02x, type: %d (%s)\n",
t->i2c->adapter->name, t->i2c->addr, t->type,
tuners[t->type].name);
t->i2c->adapter->name, t->i2c->addr, t->type, t->i2c->name);
tuner_warn("====================== WARNING! ======================\n");
}

static void attach_simple_tuner(struct tuner *t)
static inline void attach_simple_tuner(struct tuner *t)
{
struct simple_tuner_config cfg = {
.type = t->type,
.tun = &tuners[t->type]
};
simple_tuner_attach(&t->fe, t->i2c->adapter, t->i2c->addr, &cfg);
simple_tuner_attach(&t->fe, t->i2c->adapter, t->i2c->addr, t->type);
}

static void attach_tda829x(struct tuner *t)
Expand Down
10 changes: 5 additions & 5 deletions drivers/media/video/tuner-simple.c
Original file line number Diff line number Diff line change
Expand Up @@ -684,7 +684,7 @@ static struct dvb_tuner_ops simple_tuner_ops = {
struct dvb_frontend *simple_tuner_attach(struct dvb_frontend *fe,
struct i2c_adapter *i2c_adap,
u8 i2c_addr,
struct simple_tuner_config *cfg)
unsigned int type)
{
struct tuner_simple_priv *priv = NULL;

Expand All @@ -701,15 +701,15 @@ struct dvb_frontend *simple_tuner_attach(struct dvb_frontend *fe,

priv->i2c_props.addr = i2c_addr;
priv->i2c_props.adap = i2c_adap;
priv->type = cfg->type;
priv->tun = cfg->tun;
priv->type = type;
priv->tun = &tuners[type];

memcpy(&fe->ops.tuner_ops, &simple_tuner_ops,
sizeof(struct dvb_tuner_ops));

tuner_info("type set to %d (%s)\n", cfg->type, cfg->tun->name);
tuner_info("type set to %d (%s)\n", priv->type, priv->tun->name);

strlcpy(fe->ops.tuner_ops.info.name, cfg->tun->name,
strlcpy(fe->ops.tuner_ops.info.name, priv->tun->name,
sizeof(fe->ops.tuner_ops.info.name));

return fe;
Expand Down
11 changes: 2 additions & 9 deletions drivers/media/video/tuner-simple.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,23 +20,16 @@
#include <linux/i2c.h>
#include "dvb_frontend.h"

struct simple_tuner_config
{
/* chip type */
unsigned int type;
struct tunertype *tun;
};

#if defined(CONFIG_TUNER_SIMPLE) || (defined(CONFIG_TUNER_SIMPLE_MODULE) && defined(MODULE))
extern struct dvb_frontend *simple_tuner_attach(struct dvb_frontend *fe,
struct i2c_adapter *i2c_adap,
u8 i2c_addr,
struct simple_tuner_config *cfg);
unsigned int type);
#else
static inline struct dvb_frontend *simple_tuner_attach(struct dvb_frontend *fe,
struct i2c_adapter *i2c_adap,
u8 i2c_addr,
struct simple_tuner_config *cfg)
unsigned int type)
{
printk(KERN_WARNING "%s: driver disabled by Kconfig\n", __FUNCTION__);
return NULL;
Expand Down

0 comments on commit 060a5bd

Please sign in to comment.