V4L/DVB (6611): Change xc2028_attach method to make easier for DVB
Removes uneeded parameters and adds an structure for passing the parameters
This patch is co-authored by Mauro Carvalho Chehab.
Signed-off-by: Michel Ludwig <michel.ludwig@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
diff --git a/drivers/media/video/tuner-core.c b/drivers/media/video/tuner-core.c
index 1b0d28a..e56a419 100644
--- a/drivers/media/video/tuner-core.c
+++ b/drivers/media/video/tuner-core.c
@@ -336,10 +336,13 @@
break;
case TUNER_XC2028:
{
- int rc=xc2028_attach(&t->fe, t->i2c->adapter, t->i2c->addr,
- &c->dev, c->adapter->algo_data,
- t->tuner_callback);
- if (rc<0) {
+ struct xc2028_config cfg = {
+ .i2c_adap = t->i2c->adapter,
+ .i2c_addr = t->i2c->addr,
+ .video_dev = c->adapter->algo_data,
+ .callback = t->tuner_callback,
+ };
+ if (!xc2028_attach(&t->fe, &cfg)) {
t->type = TUNER_ABSENT;
t->mode_mask = T_UNINITIALIZED;
return;
diff --git a/drivers/media/video/tuner-xc2028.c b/drivers/media/video/tuner-xc2028.c
index c64eae2..0a2ffe4 100644
--- a/drivers/media/video/tuner-xc2028.c
+++ b/drivers/media/video/tuner-xc2028.c
@@ -56,7 +56,6 @@
struct tuner_i2c_props i2c_props;
int (*tuner_callback) (void *dev,
int command, int arg);
- struct device *dev;
void *video_dev;
int count;
__u32 frequency;
@@ -240,7 +239,8 @@
tuner_dbg("%s called\n", __FUNCTION__);
tuner_info("Reading firmware %s\n", priv->ctrl.fname);
- rc = request_firmware(&fw, priv->ctrl.fname, priv->dev);
+ rc = request_firmware(&fw, priv->ctrl.fname,
+ &priv->i2c_props.adap->dev);
if (rc < 0) {
if (rc == -ENOENT)
tuner_err("Error: firmware %s not found.\n",
@@ -546,8 +546,10 @@
tuner_dbg("%s called\n", __FUNCTION__);
if (!priv->firm) {
- if (!priv->ctrl.fname)
+ if (!priv->ctrl.fname) {
+ tuner_info("xc2028/3028 firmware name not set!\n");
return -EINVAL;
+ }
rc = load_all_firmwares(fe);
if (rc < 0)
@@ -882,53 +884,51 @@
};
-int xc2028_attach(struct dvb_frontend *fe, struct i2c_adapter *i2c_adap,
- u8 i2c_addr, struct device *dev, void *video_dev,
- int (*tuner_callback) (void *dev, int command, int arg))
+void *xc2028_attach(struct dvb_frontend *fe, struct xc2028_config *cfg)
{
struct xc2028_data *priv;
+ void *video_dev;
if (debug)
printk(KERN_DEBUG PREFIX "Xcv2028/3028 init called!\n");
- if (NULL == dev)
- return -ENODEV;
+ if (NULL == cfg->video_dev)
+ return NULL;
- if (NULL == video_dev)
- return -ENODEV;
-
- if (!tuner_callback) {
- printk(KERN_ERR PREFIX "No tuner callback!\n");
- return -EINVAL;
+ if (!fe) {
+ printk(KERN_ERR PREFIX "No frontend!\n");
+ return NULL;
}
+ video_dev = cfg->video_dev;
+
list_for_each_entry(priv, &xc2028_list, xc2028_list) {
- if (priv->dev == dev)
- dev = NULL;
+ if (priv->video_dev == cfg->video_dev) {
+ video_dev = NULL;
+ break;
+ }
}
- if (dev) {
+ if (video_dev) {
priv = kzalloc(sizeof(*priv), GFP_KERNEL);
if (priv == NULL)
- return -ENOMEM;
-
- fe->tuner_priv = priv;
+ return NULL;
priv->bandwidth = BANDWIDTH_6_MHZ;
priv->need_load_generic = 1;
priv->mode = T_UNINITIALIZED;
- priv->i2c_props.addr = i2c_addr;
- priv->i2c_props.adap = i2c_adap;
- priv->dev = dev;
+ priv->i2c_props.addr = cfg->i2c_addr;
+ priv->i2c_props.adap = cfg->i2c_adap;
priv->video_dev = video_dev;
- priv->tuner_callback = tuner_callback;
+ priv->tuner_callback = cfg->callback;
priv->max_len = 13;
-
mutex_init(&priv->lock);
list_add_tail(&priv->xc2028_list, &xc2028_list);
}
+
+ fe->tuner_priv = priv;
priv->count++;
memcpy(&fe->ops.tuner_ops, &xc2028_dvb_tuner_ops,
@@ -936,8 +936,9 @@
tuner_info("type set to %s\n", "XCeive xc2028/xc3028 tuner");
- return 0;
+ return fe;
}
+
EXPORT_SYMBOL(xc2028_attach);
MODULE_DESCRIPTION("Xceive xc2028/xc3028 tuner driver");
diff --git a/drivers/media/video/tuner-xc2028.h b/drivers/media/video/tuner-xc2028.h
index e04611e..a20eeb4 100644
--- a/drivers/media/video/tuner-xc2028.h
+++ b/drivers/media/video/tuner-xc2028.h
@@ -22,22 +22,22 @@
int max_len;
};
+struct xc2028_config {
+ struct i2c_adapter *i2c_adap;
+ u8 i2c_addr;
+ void *video_dev;
+ int (*callback) (void *dev, int command, int arg);
+};
+
/* xc2028 commands for callback */
#define XC2028_TUNER_RESET 0
#define XC2028_RESET_CLK 1
#if defined(CONFIG_TUNER_XC2028) || (defined(CONFIG_TUNER_XC2028_MODULE) && defined(MODULE))
-int xc2028_attach(struct dvb_frontend *fe, struct i2c_adapter *i2c_adap,
- u8 i2c_addr, struct device *dev, void *video_dev,
- int (*tuner_callback) (void *dev, int command, int arg));
-
+void *xc2028_attach(struct dvb_frontend *fe, struct xc2028_config *cfg);
#else
-static inline int xc2028_attach(struct dvb_frontend *fe,
- struct i2c_adapter *i2c_adap,
- u8 i2c_addr, struct device *dev,
- void *video_dev,
- int (*tuner_callback) (void *dev, int command,
- int arg))
+void *xc2028_attach(struct dvb_frontend *fe,
+ struct xc2028_config *cfg)
{
printk(KERN_INFO "%s: not probed - driver disabled by Kconfig\n",
__FUNCTION__);