[REVIEW PATCH 01/16] e4000: convert DVB tuner to I2C driver model

2014-02-10 Thread Antti Palosaari
Driver conversion from proprietary DVB tuner model to more
general I2C driver model.

Cc: Jean Delvare 
Cc: Mauro Carvalho Chehab 
Signed-off-by: Antti Palosaari 
---
 drivers/media/tuners/e4000.c| 115 
 drivers/media/tuners/e4000.h|  21 ++
 drivers/media/tuners/e4000_priv.h   |   5 +-
 drivers/media/usb/dvb-usb-v2/rtl28xxu.c |  41 +---
 drivers/media/usb/dvb-usb-v2/rtl28xxu.h |   1 +
 5 files changed, 111 insertions(+), 72 deletions(-)

diff --git a/drivers/media/tuners/e4000.c b/drivers/media/tuners/e4000.c
index 40c1da7..0153169 100644
--- a/drivers/media/tuners/e4000.c
+++ b/drivers/media/tuners/e4000.c
@@ -31,7 +31,7 @@ static int e4000_wr_regs(struct e4000_priv *priv, u8 reg, u8 
*val, int len)
u8 buf[MAX_XFER_SIZE];
struct i2c_msg msg[1] = {
{
-   .addr = priv->cfg->i2c_addr,
+   .addr = priv->client->addr,
.flags = 0,
.len = 1 + len,
.buf = buf,
@@ -39,7 +39,7 @@ static int e4000_wr_regs(struct e4000_priv *priv, u8 reg, u8 
*val, int len)
};
 
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",
 KBUILD_MODNAME, reg, len);
return -EINVAL;
@@ -48,11 +48,11 @@ static int e4000_wr_regs(struct e4000_priv *priv, u8 reg, 
u8 *val, int len)
buf[0] = reg;
memcpy(&buf[1], val, len);
 
-   ret = i2c_transfer(priv->i2c, msg, 1);
+   ret = i2c_transfer(priv->client->adapter, msg, 1);
if (ret == 1) {
ret = 0;
} else {
-   dev_warn(&priv->i2c->dev,
+   dev_warn(&priv->client->dev,
"%s: i2c wr failed=%d reg=%02x len=%d\n",
KBUILD_MODNAME, ret, reg, len);
ret = -EREMOTEIO;
@@ -67,12 +67,12 @@ static int e4000_rd_regs(struct e4000_priv *priv, u8 reg, 
u8 *val, int len)
u8 buf[MAX_XFER_SIZE];
struct i2c_msg msg[2] = {
{
-   .addr = priv->cfg->i2c_addr,
+   .addr = priv->client->addr,
.flags = 0,
.len = 1,
.buf = ®,
}, {
-   .addr = priv->cfg->i2c_addr,
+   .addr = priv->client->addr,
.flags = I2C_M_RD,
.len = len,
.buf = buf,
@@ -80,18 +80,18 @@ static int e4000_rd_regs(struct e4000_priv *priv, u8 reg, 
u8 *val, int len)
};
 
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",
 KBUILD_MODNAME, reg, len);
return -EINVAL;
}
 
-   ret = i2c_transfer(priv->i2c, msg, 2);
+   ret = i2c_transfer(priv->client->adapter, msg, 2);
if (ret == 2) {
memcpy(val, buf, len);
ret = 0;
} else {
-   dev_warn(&priv->i2c->dev,
+   dev_warn(&priv->client->dev,
"%s: i2c rd failed=%d reg=%02x len=%d\n",
KBUILD_MODNAME, ret, reg, len);
ret = -EREMOTEIO;
@@ -117,7 +117,7 @@ static int e4000_init(struct dvb_frontend *fe)
struct e4000_priv *priv = fe->tuner_priv;
int ret;
 
-   dev_dbg(&priv->i2c->dev, "%s:\n", __func__);
+   dev_dbg(&priv->client->dev, "%s:\n", __func__);
 
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 1);
@@ -186,7 +186,7 @@ err:
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 0);
 
-   dev_dbg(&priv->i2c->dev, "%s: failed=%d\n", __func__, ret);
+   dev_dbg(&priv->client->dev, "%s: failed=%d\n", __func__, ret);
return ret;
 }
 
@@ -195,7 +195,7 @@ static int e4000_sleep(struct dvb_frontend *fe)
struct e4000_priv *priv = fe->tuner_priv;
int ret;
 
-   dev_dbg(&priv->i2c->dev, "%s:\n", __func__);
+   dev_dbg(&priv->client->dev, "%s:\n", __func__);
 
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 1);
@@ -212,7 +212,7 @@ err:
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 0);
 
-   dev_dbg(&priv->i2c->dev, "%s: failed=%d\n", __func__, ret);
+   dev_dbg(&priv->client->dev, "%s: failed=%d\n", __func__, ret);
return ret;
 }
 
@@ -224,7 +224,7 @@ static int e4000_set_params(struct dvb_frontend *fe)
unsigned int f_vco;
u8 buf[5], i_data[4], q_data[4];
 
-   dev_dbg(&priv->i2c->dev,
+   dev_dbg(&priv->client->dev,
"%s: deliv

[REVIEW PATCH 01/16] e4000: convert DVB tuner to I2C driver model

2014-02-26 Thread Antti Palosaari
Driver conversion from proprietary DVB tuner model to more
general I2C driver model.

Cc: Jean Delvare 
Cc: Mauro Carvalho Chehab 
Signed-off-by: Antti Palosaari 
---
 drivers/media/tuners/e4000.c| 115 
 drivers/media/tuners/e4000.h|  21 ++
 drivers/media/tuners/e4000_priv.h   |   5 +-
 drivers/media/usb/dvb-usb-v2/rtl28xxu.c |  41 +---
 drivers/media/usb/dvb-usb-v2/rtl28xxu.h |   1 +
 5 files changed, 111 insertions(+), 72 deletions(-)

diff --git a/drivers/media/tuners/e4000.c b/drivers/media/tuners/e4000.c
index 40c1da7..0153169 100644
--- a/drivers/media/tuners/e4000.c
+++ b/drivers/media/tuners/e4000.c
@@ -31,7 +31,7 @@ static int e4000_wr_regs(struct e4000_priv *priv, u8 reg, u8 
*val, int len)
u8 buf[MAX_XFER_SIZE];
struct i2c_msg msg[1] = {
{
-   .addr = priv->cfg->i2c_addr,
+   .addr = priv->client->addr,
.flags = 0,
.len = 1 + len,
.buf = buf,
@@ -39,7 +39,7 @@ static int e4000_wr_regs(struct e4000_priv *priv, u8 reg, u8 
*val, int len)
};
 
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",
 KBUILD_MODNAME, reg, len);
return -EINVAL;
@@ -48,11 +48,11 @@ static int e4000_wr_regs(struct e4000_priv *priv, u8 reg, 
u8 *val, int len)
buf[0] = reg;
memcpy(&buf[1], val, len);
 
-   ret = i2c_transfer(priv->i2c, msg, 1);
+   ret = i2c_transfer(priv->client->adapter, msg, 1);
if (ret == 1) {
ret = 0;
} else {
-   dev_warn(&priv->i2c->dev,
+   dev_warn(&priv->client->dev,
"%s: i2c wr failed=%d reg=%02x len=%d\n",
KBUILD_MODNAME, ret, reg, len);
ret = -EREMOTEIO;
@@ -67,12 +67,12 @@ static int e4000_rd_regs(struct e4000_priv *priv, u8 reg, 
u8 *val, int len)
u8 buf[MAX_XFER_SIZE];
struct i2c_msg msg[2] = {
{
-   .addr = priv->cfg->i2c_addr,
+   .addr = priv->client->addr,
.flags = 0,
.len = 1,
.buf = ®,
}, {
-   .addr = priv->cfg->i2c_addr,
+   .addr = priv->client->addr,
.flags = I2C_M_RD,
.len = len,
.buf = buf,
@@ -80,18 +80,18 @@ static int e4000_rd_regs(struct e4000_priv *priv, u8 reg, 
u8 *val, int len)
};
 
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",
 KBUILD_MODNAME, reg, len);
return -EINVAL;
}
 
-   ret = i2c_transfer(priv->i2c, msg, 2);
+   ret = i2c_transfer(priv->client->adapter, msg, 2);
if (ret == 2) {
memcpy(val, buf, len);
ret = 0;
} else {
-   dev_warn(&priv->i2c->dev,
+   dev_warn(&priv->client->dev,
"%s: i2c rd failed=%d reg=%02x len=%d\n",
KBUILD_MODNAME, ret, reg, len);
ret = -EREMOTEIO;
@@ -117,7 +117,7 @@ static int e4000_init(struct dvb_frontend *fe)
struct e4000_priv *priv = fe->tuner_priv;
int ret;
 
-   dev_dbg(&priv->i2c->dev, "%s:\n", __func__);
+   dev_dbg(&priv->client->dev, "%s:\n", __func__);
 
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 1);
@@ -186,7 +186,7 @@ err:
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 0);
 
-   dev_dbg(&priv->i2c->dev, "%s: failed=%d\n", __func__, ret);
+   dev_dbg(&priv->client->dev, "%s: failed=%d\n", __func__, ret);
return ret;
 }
 
@@ -195,7 +195,7 @@ static int e4000_sleep(struct dvb_frontend *fe)
struct e4000_priv *priv = fe->tuner_priv;
int ret;
 
-   dev_dbg(&priv->i2c->dev, "%s:\n", __func__);
+   dev_dbg(&priv->client->dev, "%s:\n", __func__);
 
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 1);
@@ -212,7 +212,7 @@ err:
if (fe->ops.i2c_gate_ctrl)
fe->ops.i2c_gate_ctrl(fe, 0);
 
-   dev_dbg(&priv->i2c->dev, "%s: failed=%d\n", __func__, ret);
+   dev_dbg(&priv->client->dev, "%s: failed=%d\n", __func__, ret);
return ret;
 }
 
@@ -224,7 +224,7 @@ static int e4000_set_params(struct dvb_frontend *fe)
unsigned int f_vco;
u8 buf[5], i_data[4], q_data[4];
 
-   dev_dbg(&priv->i2c->dev,
+   dev_dbg(&priv->client->dev,
"%s: deliv