Commit 0e584cc2 authored by Antti Palosaari's avatar Antti Palosaari Committed by Mauro Carvalho Chehab

[media] tda18212: convert driver to I2C binding

Convert driver from DVB proprietary model to common I2C model.
Signed-off-by: default avatarAntti Palosaari <crope@iki.fi>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@osg.samsung.com>
parent ac7a24fb
...@@ -24,8 +24,8 @@ ...@@ -24,8 +24,8 @@
#define MAX_XFER_SIZE 64 #define MAX_XFER_SIZE 64
struct tda18212_priv { struct tda18212_priv {
struct tda18212_config *cfg; struct tda18212_config cfg;
struct i2c_adapter *i2c; struct i2c_client *client;
u32 if_frequency; u32 if_frequency;
}; };
...@@ -38,7 +38,7 @@ static int tda18212_wr_regs(struct tda18212_priv *priv, u8 reg, u8 *val, ...@@ -38,7 +38,7 @@ static int tda18212_wr_regs(struct tda18212_priv *priv, u8 reg, u8 *val,
u8 buf[MAX_XFER_SIZE]; u8 buf[MAX_XFER_SIZE];
struct i2c_msg msg[1] = { struct i2c_msg msg[1] = {
{ {
.addr = priv->cfg->i2c_address, .addr = priv->client->addr,
.flags = 0, .flags = 0,
.len = 1 + len, .len = 1 + len,
.buf = buf, .buf = buf,
...@@ -46,7 +46,7 @@ static int tda18212_wr_regs(struct tda18212_priv *priv, u8 reg, u8 *val, ...@@ -46,7 +46,7 @@ static int tda18212_wr_regs(struct tda18212_priv *priv, u8 reg, u8 *val,
}; };
if (1 + len > sizeof(buf)) { if (1 + len > sizeof(buf)) {
dev_warn(&priv->i2c->dev, dev_warn(&priv->client->dev,
"%s: i2c wr reg=%04x: len=%d is too big!\n", "%s: i2c wr reg=%04x: len=%d is too big!\n",
KBUILD_MODNAME, reg, len); KBUILD_MODNAME, reg, len);
return -EINVAL; return -EINVAL;
...@@ -55,12 +55,12 @@ static int tda18212_wr_regs(struct tda18212_priv *priv, u8 reg, u8 *val, ...@@ -55,12 +55,12 @@ static int tda18212_wr_regs(struct tda18212_priv *priv, u8 reg, u8 *val,
buf[0] = reg; buf[0] = reg;
memcpy(&buf[1], val, len); memcpy(&buf[1], val, len);
ret = i2c_transfer(priv->i2c, msg, 1); ret = i2c_transfer(priv->client->adapter, msg, 1);
if (ret == 1) { if (ret == 1) {
ret = 0; ret = 0;
} else { } else {
dev_warn(&priv->i2c->dev, "%s: i2c wr failed=%d reg=%02x " \ dev_warn(&priv->client->dev, "%s: i2c wr failed=%d reg=%02x len=%d\n",
"len=%d\n", KBUILD_MODNAME, ret, reg, len); KBUILD_MODNAME, ret, reg, len);
ret = -EREMOTEIO; ret = -EREMOTEIO;
} }
return ret; return ret;
...@@ -74,12 +74,12 @@ static int tda18212_rd_regs(struct tda18212_priv *priv, u8 reg, u8 *val, ...@@ -74,12 +74,12 @@ static int tda18212_rd_regs(struct tda18212_priv *priv, u8 reg, u8 *val,
u8 buf[MAX_XFER_SIZE]; u8 buf[MAX_XFER_SIZE];
struct i2c_msg msg[2] = { struct i2c_msg msg[2] = {
{ {
.addr = priv->cfg->i2c_address, .addr = priv->client->addr,
.flags = 0, .flags = 0,
.len = 1, .len = 1,
.buf = &reg, .buf = &reg,
}, { }, {
.addr = priv->cfg->i2c_address, .addr = priv->client->addr,
.flags = I2C_M_RD, .flags = I2C_M_RD,
.len = len, .len = len,
.buf = buf, .buf = buf,
...@@ -87,19 +87,19 @@ static int tda18212_rd_regs(struct tda18212_priv *priv, u8 reg, u8 *val, ...@@ -87,19 +87,19 @@ static int tda18212_rd_regs(struct tda18212_priv *priv, u8 reg, u8 *val,
}; };
if (len > sizeof(buf)) { if (len > sizeof(buf)) {
dev_warn(&priv->i2c->dev, dev_warn(&priv->client->dev,
"%s: i2c rd reg=%04x: len=%d is too big!\n", "%s: i2c rd reg=%04x: len=%d is too big!\n",
KBUILD_MODNAME, reg, len); KBUILD_MODNAME, reg, len);
return -EINVAL; return -EINVAL;
} }
ret = i2c_transfer(priv->i2c, msg, 2); ret = i2c_transfer(priv->client->adapter, msg, 2);
if (ret == 2) { if (ret == 2) {
memcpy(val, buf, len); memcpy(val, buf, len);
ret = 0; ret = 0;
} else { } else {
dev_warn(&priv->i2c->dev, "%s: i2c rd failed=%d reg=%02x " \ dev_warn(&priv->client->dev, "%s: i2c rd failed=%d reg=%02x len=%d\n",
"len=%d\n", KBUILD_MODNAME, ret, reg, len); KBUILD_MODNAME, ret, reg, len);
ret = -EREMOTEIO; ret = -EREMOTEIO;
} }
...@@ -166,7 +166,7 @@ static int tda18212_set_params(struct dvb_frontend *fe) ...@@ -166,7 +166,7 @@ static int tda18212_set_params(struct dvb_frontend *fe)
[ATSC_QAM] = { 0x7d, 0x20, 0x63 }, [ATSC_QAM] = { 0x7d, 0x20, 0x63 },
}; };
dev_dbg(&priv->i2c->dev, dev_dbg(&priv->client->dev,
"%s: delivery_system=%d frequency=%d bandwidth_hz=%d\n", "%s: delivery_system=%d frequency=%d bandwidth_hz=%d\n",
__func__, c->delivery_system, c->frequency, __func__, c->delivery_system, c->frequency,
c->bandwidth_hz); c->bandwidth_hz);
...@@ -176,25 +176,25 @@ static int tda18212_set_params(struct dvb_frontend *fe) ...@@ -176,25 +176,25 @@ static int tda18212_set_params(struct dvb_frontend *fe)
switch (c->delivery_system) { switch (c->delivery_system) {
case SYS_ATSC: case SYS_ATSC:
if_khz = priv->cfg->if_atsc_vsb; if_khz = priv->cfg.if_atsc_vsb;
i = ATSC_VSB; i = ATSC_VSB;
break; break;
case SYS_DVBC_ANNEX_B: case SYS_DVBC_ANNEX_B:
if_khz = priv->cfg->if_atsc_qam; if_khz = priv->cfg.if_atsc_qam;
i = ATSC_QAM; i = ATSC_QAM;
break; break;
case SYS_DVBT: case SYS_DVBT:
switch (c->bandwidth_hz) { switch (c->bandwidth_hz) {
case 6000000: case 6000000:
if_khz = priv->cfg->if_dvbt_6; if_khz = priv->cfg.if_dvbt_6;
i = DVBT_6; i = DVBT_6;
break; break;
case 7000000: case 7000000:
if_khz = priv->cfg->if_dvbt_7; if_khz = priv->cfg.if_dvbt_7;
i = DVBT_7; i = DVBT_7;
break; break;
case 8000000: case 8000000:
if_khz = priv->cfg->if_dvbt_8; if_khz = priv->cfg.if_dvbt_8;
i = DVBT_8; i = DVBT_8;
break; break;
default: default:
...@@ -205,15 +205,15 @@ static int tda18212_set_params(struct dvb_frontend *fe) ...@@ -205,15 +205,15 @@ static int tda18212_set_params(struct dvb_frontend *fe)
case SYS_DVBT2: case SYS_DVBT2:
switch (c->bandwidth_hz) { switch (c->bandwidth_hz) {
case 6000000: case 6000000:
if_khz = priv->cfg->if_dvbt2_6; if_khz = priv->cfg.if_dvbt2_6;
i = DVBT2_6; i = DVBT2_6;
break; break;
case 7000000: case 7000000:
if_khz = priv->cfg->if_dvbt2_7; if_khz = priv->cfg.if_dvbt2_7;
i = DVBT2_7; i = DVBT2_7;
break; break;
case 8000000: case 8000000:
if_khz = priv->cfg->if_dvbt2_8; if_khz = priv->cfg.if_dvbt2_8;
i = DVBT2_8; i = DVBT2_8;
break; break;
default: default:
...@@ -223,7 +223,7 @@ static int tda18212_set_params(struct dvb_frontend *fe) ...@@ -223,7 +223,7 @@ static int tda18212_set_params(struct dvb_frontend *fe)
break; break;
case SYS_DVBC_ANNEX_A: case SYS_DVBC_ANNEX_A:
case SYS_DVBC_ANNEX_C: case SYS_DVBC_ANNEX_C:
if_khz = priv->cfg->if_dvbc; if_khz = priv->cfg.if_dvbc;
i = DVBC_8; i = DVBC_8;
break; break;
default: default:
...@@ -266,7 +266,7 @@ static int tda18212_set_params(struct dvb_frontend *fe) ...@@ -266,7 +266,7 @@ static int tda18212_set_params(struct dvb_frontend *fe)
return ret; return ret;
error: error:
dev_dbg(&priv->i2c->dev, "%s: failed=%d\n", __func__, ret); dev_dbg(&priv->client->dev, "%s: failed=%d\n", __func__, ret);
goto exit; goto exit;
} }
...@@ -279,13 +279,6 @@ static int tda18212_get_if_frequency(struct dvb_frontend *fe, u32 *frequency) ...@@ -279,13 +279,6 @@ static int tda18212_get_if_frequency(struct dvb_frontend *fe, u32 *frequency)
return 0; return 0;
} }
static int tda18212_release(struct dvb_frontend *fe)
{
kfree(fe->tuner_priv);
fe->tuner_priv = NULL;
return 0;
}
static const struct dvb_tuner_ops tda18212_tuner_ops = { static const struct dvb_tuner_ops tda18212_tuner_ops = {
.info = { .info = {
.name = "NXP TDA18212", .name = "NXP TDA18212",
...@@ -295,34 +288,36 @@ static const struct dvb_tuner_ops tda18212_tuner_ops = { ...@@ -295,34 +288,36 @@ static const struct dvb_tuner_ops tda18212_tuner_ops = {
.frequency_step = 1000, .frequency_step = 1000,
}, },
.release = tda18212_release,
.set_params = tda18212_set_params, .set_params = tda18212_set_params,
.get_if_frequency = tda18212_get_if_frequency, .get_if_frequency = tda18212_get_if_frequency,
}; };
struct dvb_frontend *tda18212_attach(struct dvb_frontend *fe, static int tda18212_probe(struct i2c_client *client,
struct i2c_adapter *i2c, struct tda18212_config *cfg) const struct i2c_device_id *id)
{ {
struct tda18212_priv *priv = NULL; struct tda18212_config *cfg = client->dev.platform_data;
struct dvb_frontend *fe = cfg->fe;
struct tda18212_priv *priv;
int ret; int ret;
u8 chip_id = chip_id; u8 chip_id = chip_id;
char *version; char *version;
priv = kzalloc(sizeof(struct tda18212_priv), GFP_KERNEL); priv = kzalloc(sizeof(*priv), GFP_KERNEL);
if (priv == NULL) if (!priv) {
return NULL; ret = -ENOMEM;
dev_err(&client->dev, "%s: kzalloc() failed\n", KBUILD_MODNAME);
goto err;
}
priv->cfg = cfg; memcpy(&priv->cfg, cfg, sizeof(struct tda18212_config));
priv->i2c = i2c; priv->client = client;
fe->tuner_priv = priv;
/* check if the tuner is there */
if (fe->ops.i2c_gate_ctrl) if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 1); /* open I2C-gate */ fe->ops.i2c_gate_ctrl(fe, 1); /* open I2C-gate */
/* check if the tuner is there */
ret = tda18212_rd_reg(priv, 0x00, &chip_id); ret = tda18212_rd_reg(priv, 0x00, &chip_id);
dev_dbg(&priv->i2c->dev, "%s: chip_id=%02x\n", __func__, chip_id); dev_dbg(&priv->client->dev, "%s: chip_id=%02x\n", __func__, chip_id);
if (fe->ops.i2c_gate_ctrl) if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 0); /* close I2C-gate */ fe->ops.i2c_gate_ctrl(fe, 0); /* close I2C-gate */
...@@ -338,23 +333,57 @@ struct dvb_frontend *tda18212_attach(struct dvb_frontend *fe, ...@@ -338,23 +333,57 @@ struct dvb_frontend *tda18212_attach(struct dvb_frontend *fe,
version = "S"; /* slave */ version = "S"; /* slave */
break; break;
default: default:
ret = -ENODEV;
goto err; goto err;
} }
dev_info(&priv->i2c->dev, dev_info(&priv->client->dev,
"%s: NXP TDA18212HN/%s successfully identified\n", "%s: NXP TDA18212HN/%s successfully identified\n",
KBUILD_MODNAME, version); KBUILD_MODNAME, version);
fe->tuner_priv = priv;
memcpy(&fe->ops.tuner_ops, &tda18212_tuner_ops, memcpy(&fe->ops.tuner_ops, &tda18212_tuner_ops,
sizeof(struct dvb_tuner_ops)); sizeof(struct dvb_tuner_ops));
i2c_set_clientdata(client, priv);
return fe; return 0;
err: err:
dev_dbg(&i2c->dev, "%s: failed=%d\n", __func__, ret); dev_dbg(&client->dev, "%s: failed=%d\n", __func__, ret);
kfree(priv); kfree(priv);
return NULL; return ret;
} }
EXPORT_SYMBOL(tda18212_attach);
static int tda18212_remove(struct i2c_client *client)
{
struct tda18212_priv *priv = i2c_get_clientdata(client);
struct dvb_frontend *fe = priv->cfg.fe;
dev_dbg(&client->dev, "%s:\n", __func__);
memset(&fe->ops.tuner_ops, 0, sizeof(struct dvb_tuner_ops));
fe->tuner_priv = NULL;
kfree(priv);
return 0;
}
static const struct i2c_device_id tda18212_id[] = {
{"tda18212", 0},
{}
};
MODULE_DEVICE_TABLE(i2c, tda18212_id);
static struct i2c_driver tda18212_driver = {
.driver = {
.owner = THIS_MODULE,
.name = "tda18212",
},
.probe = tda18212_probe,
.remove = tda18212_remove,
.id_table = tda18212_id,
};
module_i2c_driver(tda18212_driver);
MODULE_DESCRIPTION("NXP TDA18212HN silicon tuner driver"); MODULE_DESCRIPTION("NXP TDA18212HN silicon tuner driver");
MODULE_AUTHOR("Antti Palosaari <crope@iki.fi>"); MODULE_AUTHOR("Antti Palosaari <crope@iki.fi>");
......
...@@ -25,8 +25,6 @@ ...@@ -25,8 +25,6 @@
#include "dvb_frontend.h" #include "dvb_frontend.h"
struct tda18212_config { struct tda18212_config {
u8 i2c_address;
u16 if_dvbt_6; u16 if_dvbt_6;
u16 if_dvbt_7; u16 if_dvbt_7;
u16 if_dvbt_8; u16 if_dvbt_8;
...@@ -44,16 +42,4 @@ struct tda18212_config { ...@@ -44,16 +42,4 @@ struct tda18212_config {
struct dvb_frontend *fe; struct dvb_frontend *fe;
}; };
#if IS_ENABLED(CONFIG_MEDIA_TUNER_TDA18212)
extern struct dvb_frontend *tda18212_attach(struct dvb_frontend *fe,
struct i2c_adapter *i2c, struct tda18212_config *cfg);
#else
static inline struct dvb_frontend *tda18212_attach(struct dvb_frontend *fe,
struct i2c_adapter *i2c, struct tda18212_config *cfg)
{
printk(KERN_WARNING "%s: driver disabled by Kconfig\n", __func__);
return NULL;
}
#endif
#endif #endif
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment