Merge branch 'perf/urgent' into perf/core, to resolve a conflict
[firefly-linux-kernel-4.4.55.git] / drivers / media / dvb-frontends / tda10071.c
index 39a4197a34534a63dc2e096e4e9417bca570485c..119d47596ac81ace7dacf073c4024ca65b4fa38a 100644 (file)
 
 #include "tda10071_priv.h"
 
-/* Max transfer size done by I2C transfer functions */
-#define MAX_XFER_SIZE  64
-
 static struct dvb_frontend_ops tda10071_ops;
 
-/* write multiple registers */
-static int tda10071_wr_regs(struct tda10071_dev *dev, u8 reg, u8 *val,
-       int len)
-{
-       struct i2c_client *client = dev->client;
-       int ret;
-       u8 buf[MAX_XFER_SIZE];
-       struct i2c_msg msg[1] = {
-               {
-                       .addr = client->addr,
-                       .flags = 0,
-                       .len = 1 + len,
-                       .buf = buf,
-               }
-       };
-
-       if (1 + len > sizeof(buf)) {
-               dev_warn(&client->dev, "i2c wr reg=%04x: len=%d is too big!\n",
-                               reg, len);
-               return -EINVAL;
-       }
-
-       buf[0] = reg;
-       memcpy(&buf[1], val, len);
-
-       ret = i2c_transfer(client->adapter, msg, 1);
-       if (ret == 1) {
-               ret = 0;
-       } else {
-               dev_warn(&client->dev, "i2c wr failed=%d reg=%02x len=%d\n",
-                               ret, reg, len);
-               ret = -EREMOTEIO;
-       }
-       return ret;
-}
-
-/* read multiple registers */
-static int tda10071_rd_regs(struct tda10071_dev *dev, u8 reg, u8 *val,
-       int len)
-{
-       struct i2c_client *client = dev->client;
-       int ret;
-       u8 buf[MAX_XFER_SIZE];
-       struct i2c_msg msg[2] = {
-               {
-                       .addr = client->addr,
-                       .flags = 0,
-                       .len = 1,
-                       .buf = &reg,
-               }, {
-                       .addr = client->addr,
-                       .flags = I2C_M_RD,
-                       .len = len,
-                       .buf = buf,
-               }
-       };
-
-       if (len > sizeof(buf)) {
-               dev_warn(&client->dev, "i2c wr reg=%04x: len=%d is too big!\n",
-                               reg, len);
-               return -EINVAL;
-       }
-
-       ret = i2c_transfer(client->adapter, msg, 2);
-       if (ret == 2) {
-               memcpy(val, buf, len);
-               ret = 0;
-       } else {
-               dev_warn(&client->dev, "i2c rd failed=%d reg=%02x len=%d\n",
-                               ret, reg, len);
-               ret = -EREMOTEIO;
-       }
-       return ret;
-}
-
-/* write single register */
-static int tda10071_wr_reg(struct tda10071_dev *dev, u8 reg, u8 val)
-{
-       return tda10071_wr_regs(dev, reg, &val, 1);
-}
-
-/* read single register */
-static int tda10071_rd_reg(struct tda10071_dev *dev, u8 reg, u8 *val)
-{
-       return tda10071_rd_regs(dev, reg, val, 1);
-}
-
+/*
+ * XXX: regmap_update_bits() does not fit our needs as it does not support
+ * partially volatile registers. Also it performs register read even mask is as
+ * wide as register value.
+ */
 /* write single register with mask */
 static int tda10071_wr_reg_mask(struct tda10071_dev *dev,
                                u8 reg, u8 val, u8 mask)
@@ -121,7 +36,7 @@ static int tda10071_wr_reg_mask(struct tda10071_dev *dev,
 
        /* no need for read if whole reg is written */
        if (mask != 0xff) {
-               ret = tda10071_rd_regs(dev, reg, &tmp, 1);
+               ret = regmap_bulk_read(dev->regmap, reg, &tmp, 1);
                if (ret)
                        return ret;
 
@@ -130,30 +45,7 @@ static int tda10071_wr_reg_mask(struct tda10071_dev *dev,
                val |= tmp;
        }
 
-       return tda10071_wr_regs(dev, reg, &val, 1);
-}
-
-/* read single register with mask */
-static int tda10071_rd_reg_mask(struct tda10071_dev *dev,
-                               u8 reg, u8 *val, u8 mask)
-{
-       int ret, i;
-       u8 tmp;
-
-       ret = tda10071_rd_regs(dev, reg, &tmp, 1);
-       if (ret)
-               return ret;
-
-       tmp &= mask;
-
-       /* find position of the first bit */
-       for (i = 0; i < 8; i++) {
-               if ((mask >> i) & 0x01)
-                       break;
-       }
-       *val = tmp >> i;
-
-       return 0;
+       return regmap_bulk_write(dev->regmap, reg, &val, 1);
 }
 
 /* execute firmware command */
@@ -162,32 +54,35 @@ static int tda10071_cmd_execute(struct tda10071_dev *dev,
 {
        struct i2c_client *client = dev->client;
        int ret, i;
-       utmp;
+       unsigned int uitmp;
 
        if (!dev->warm) {
                ret = -EFAULT;
                goto error;
        }
 
+       mutex_lock(&dev->cmd_execute_mutex);
+
        /* write cmd and args for firmware */
-       ret = tda10071_wr_regs(dev, 0x00, cmd->args, cmd->len);
+       ret = regmap_bulk_write(dev->regmap, 0x00, cmd->args, cmd->len);
        if (ret)
-               goto error;
+               goto error_mutex_unlock;
 
        /* start cmd execution */
-       ret = tda10071_wr_reg(dev, 0x1f, 1);
+       ret = regmap_write(dev->regmap, 0x1f, 1);
        if (ret)
-               goto error;
+               goto error_mutex_unlock;
 
        /* wait cmd execution terminate */
-       for (i = 1000, tmp = 1; i && tmp; i--) {
-               ret = tda10071_rd_reg(dev, 0x1f, &tmp);
+       for (i = 1000, uitmp = 1; i && uitmp; i--) {
+               ret = regmap_read(dev->regmap, 0x1f, &uitmp);
                if (ret)
-                       goto error;
+                       goto error_mutex_unlock;
 
                usleep_range(200, 5000);
        }
 
+       mutex_unlock(&dev->cmd_execute_mutex);
        dev_dbg(&client->dev, "loop=%d\n", i);
 
        if (i == 0) {
@@ -196,6 +91,8 @@ static int tda10071_cmd_execute(struct tda10071_dev *dev,
        }
 
        return ret;
+error_mutex_unlock:
+       mutex_unlock(&dev->cmd_execute_mutex);
 error:
        dev_dbg(&client->dev, "failed=%d\n", ret);
        return ret;
@@ -299,7 +196,7 @@ static int tda10071_diseqc_send_master_cmd(struct dvb_frontend *fe,
        struct i2c_client *client = dev->client;
        struct tda10071_cmd cmd;
        int ret, i;
-       utmp;
+       unsigned int uitmp;
 
        if (!dev->warm) {
                ret = -EFAULT;
@@ -314,11 +211,11 @@ static int tda10071_diseqc_send_master_cmd(struct dvb_frontend *fe,
        }
 
        /* wait LNB TX */
-       for (i = 500, tmp = 0; i && !tmp; i--) {
-               ret = tda10071_rd_reg_mask(dev, 0x47, &tmp, 0x01);
+       for (i = 500, uitmp = 0; i && !uitmp; i--) {
+               ret = regmap_read(dev->regmap, 0x47, &uitmp);
                if (ret)
                        goto error;
-
+               uitmp = (uitmp >> 0) & 1;
                usleep_range(10000, 20000);
        }
 
@@ -329,7 +226,7 @@ static int tda10071_diseqc_send_master_cmd(struct dvb_frontend *fe,
                goto error;
        }
 
-       ret = tda10071_wr_reg_mask(dev, 0x47, 0x00, 0x01);
+       ret = regmap_update_bits(dev->regmap, 0x47, 0x01, 0x00);
        if (ret)
                goto error;
 
@@ -359,7 +256,7 @@ static int tda10071_diseqc_recv_slave_reply(struct dvb_frontend *fe,
        struct i2c_client *client = dev->client;
        struct tda10071_cmd cmd;
        int ret, i;
-       utmp;
+       unsigned int uitmp;
 
        if (!dev->warm) {
                ret = -EFAULT;
@@ -369,11 +266,11 @@ static int tda10071_diseqc_recv_slave_reply(struct dvb_frontend *fe,
        dev_dbg(&client->dev, "\n");
 
        /* wait LNB RX */
-       for (i = 500, tmp = 0; i && !tmp; i--) {
-               ret = tda10071_rd_reg_mask(dev, 0x47, &tmp, 0x02);
+       for (i = 500, uitmp = 0; i && !uitmp; i--) {
+               ret = regmap_read(dev->regmap, 0x47, &uitmp);
                if (ret)
                        goto error;
-
+               uitmp = (uitmp >> 1) & 1;
                usleep_range(10000, 20000);
        }
 
@@ -385,11 +282,11 @@ static int tda10071_diseqc_recv_slave_reply(struct dvb_frontend *fe,
        }
 
        /* reply len */
-       ret = tda10071_rd_reg(dev, 0x46, &tmp);
+       ret = regmap_read(dev->regmap, 0x46, &uitmp);
        if (ret)
                goto error;
 
-       reply->msg_len = tmp & 0x1f; /* [4:0] */
+       reply->msg_len = uitmp & 0x1f; /* [4:0] */
        if (reply->msg_len > sizeof(reply->msg))
                reply->msg_len = sizeof(reply->msg); /* truncate API max */
 
@@ -401,7 +298,8 @@ static int tda10071_diseqc_recv_slave_reply(struct dvb_frontend *fe,
        if (ret)
                goto error;
 
-       ret = tda10071_rd_regs(dev, cmd.len, reply->msg, reply->msg_len);
+       ret = regmap_bulk_read(dev->regmap, cmd.len, reply->msg,
+                              reply->msg_len);
        if (ret)
                goto error;
 
@@ -418,7 +316,8 @@ static int tda10071_diseqc_send_burst(struct dvb_frontend *fe,
        struct i2c_client *client = dev->client;
        struct tda10071_cmd cmd;
        int ret, i;
-       u8 tmp, burst;
+       unsigned int uitmp;
+       u8 burst;
 
        if (!dev->warm) {
                ret = -EFAULT;
@@ -441,11 +340,11 @@ static int tda10071_diseqc_send_burst(struct dvb_frontend *fe,
        }
 
        /* wait LNB TX */
-       for (i = 500, tmp = 0; i && !tmp; i--) {
-               ret = tda10071_rd_reg_mask(dev, 0x47, &tmp, 0x01);
+       for (i = 500, uitmp = 0; i && !uitmp; i--) {
+               ret = regmap_read(dev->regmap, 0x47, &uitmp);
                if (ret)
                        goto error;
-
+               uitmp = (uitmp >> 0) & 1;
                usleep_range(10000, 20000);
        }
 
@@ -456,7 +355,7 @@ static int tda10071_diseqc_send_burst(struct dvb_frontend *fe,
                goto error;
        }
 
-       ret = tda10071_wr_reg_mask(dev, 0x47, 0x00, 0x01);
+       ret = regmap_update_bits(dev->regmap, 0x47, 0x01, 0x00);
        if (ret)
                goto error;
 
@@ -478,8 +377,11 @@ static int tda10071_read_status(struct dvb_frontend *fe, enum fe_status *status)
 {
        struct tda10071_dev *dev = fe->demodulator_priv;
        struct i2c_client *client = dev->client;
+       struct dtv_frontend_properties *c = &fe->dtv_property_cache;
+       struct tda10071_cmd cmd;
        int ret;
-       u8 tmp;
+       unsigned int uitmp;
+       u8 buf[8];
 
        *status = 0;
 
@@ -488,85 +390,119 @@ static int tda10071_read_status(struct dvb_frontend *fe, enum fe_status *status)
                goto error;
        }
 
-       ret = tda10071_rd_reg(dev, 0x39, &tmp);
+       ret = regmap_read(dev->regmap, 0x39, &uitmp);
        if (ret)
                goto error;
 
        /* 0x39[0] tuner PLL */
-       if (tmp & 0x02) /* demod PLL */
+       if (uitmp & 0x02) /* demod PLL */
                *status |= FE_HAS_SIGNAL | FE_HAS_CARRIER;
-       if (tmp & 0x04) /* viterbi or LDPC*/
+       if (uitmp & 0x04) /* viterbi or LDPC*/
                *status |= FE_HAS_VITERBI;
-       if (tmp & 0x08) /* RS or BCH */
+       if (uitmp & 0x08) /* RS or BCH */
                *status |= FE_HAS_SYNC | FE_HAS_LOCK;
 
        dev->fe_status = *status;
 
-       return ret;
-error:
-       dev_dbg(&client->dev, "failed=%d\n", ret);
-       return ret;
-}
+       /* signal strength */
+       if (dev->fe_status & FE_HAS_SIGNAL) {
+               cmd.args[0] = CMD_GET_AGCACC;
+               cmd.args[1] = 0;
+               cmd.len = 2;
+               ret = tda10071_cmd_execute(dev, &cmd);
+               if (ret)
+                       goto error;
 
-static int tda10071_read_snr(struct dvb_frontend *fe, u16 *snr)
-{
-       struct tda10071_dev *dev = fe->demodulator_priv;
-       struct i2c_client *client = dev->client;
-       int ret;
-       u8 buf[2];
+               /* input power estimate dBm */
+               ret = regmap_read(dev->regmap, 0x50, &uitmp);
+               if (ret)
+                       goto error;
 
-       if (!dev->warm || !(dev->fe_status & FE_HAS_LOCK)) {
-               *snr = 0;
-               ret = 0;
-               goto error;
+               c->strength.stat[0].scale = FE_SCALE_DECIBEL;
+               c->strength.stat[0].svalue = (int) (uitmp - 256) * 1000;
+       } else {
+               c->strength.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
        }
 
-       ret = tda10071_rd_regs(dev, 0x3a, buf, 2);
-       if (ret)
-               goto error;
+       /* CNR */
+       if (dev->fe_status & FE_HAS_VITERBI) {
+               /* Es/No */
+               ret = regmap_bulk_read(dev->regmap, 0x3a, buf, 2);
+               if (ret)
+                       goto error;
 
-       /* Es/No dBx10 */
-       *snr = buf[0] << 8 | buf[1];
+               c->cnr.stat[0].scale = FE_SCALE_DECIBEL;
+               c->cnr.stat[0].svalue = (buf[0] << 8 | buf[1] << 0) * 100;
+       } else {
+               c->cnr.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+       }
 
-       return ret;
-error:
-       dev_dbg(&client->dev, "failed=%d\n", ret);
-       return ret;
-}
+       /* UCB/PER/BER */
+       if (dev->fe_status & FE_HAS_LOCK) {
+               /* TODO: report total bits/packets */
+               u8 delivery_system, reg, len;
 
-static int tda10071_read_signal_strength(struct dvb_frontend *fe, u16 *strength)
-{
-       struct tda10071_dev *dev = fe->demodulator_priv;
-       struct i2c_client *client = dev->client;
-       struct tda10071_cmd cmd;
-       int ret;
-       u8 tmp;
+               switch (dev->delivery_system) {
+               case SYS_DVBS:
+                       reg = 0x4c;
+                       len = 8;
+                       delivery_system = 1;
+                       break;
+               case SYS_DVBS2:
+                       reg = 0x4d;
+                       len = 4;
+                       delivery_system = 0;
+                       break;
+               default:
+                       ret = -EINVAL;
+                       goto error;
+               }
 
-       if (!dev->warm || !(dev->fe_status & FE_HAS_LOCK)) {
-               *strength = 0;
-               ret = 0;
-               goto error;
-       }
+               ret = regmap_read(dev->regmap, reg, &uitmp);
+               if (ret)
+                       goto error;
 
-       cmd.args[0] = CMD_GET_AGCACC;
-       cmd.args[1] = 0;
-       cmd.len = 2;
-       ret = tda10071_cmd_execute(dev, &cmd);
-       if (ret)
-               goto error;
+               if (dev->meas_count == uitmp) {
+                       dev_dbg(&client->dev, "meas not ready=%02x\n", uitmp);
+                       ret = 0;
+                       goto error;
+               } else {
+                       dev->meas_count = uitmp;
+               }
 
-       /* input power estimate dBm */
-       ret = tda10071_rd_reg(dev, 0x50, &tmp);
-       if (ret)
-               goto error;
+               cmd.args[0] = CMD_BER_UPDATE_COUNTERS;
+               cmd.args[1] = 0;
+               cmd.args[2] = delivery_system;
+               cmd.len = 3;
+               ret = tda10071_cmd_execute(dev, &cmd);
+               if (ret)
+                       goto error;
 
-       if (tmp < 181)
-               tmp = 181; /* -75 dBm */
-       else if (tmp > 236)
-               tmp = 236; /* -20 dBm */
+               ret = regmap_bulk_read(dev->regmap, cmd.len, buf, len);
+               if (ret)
+                       goto error;
 
-       /* scale value to 0x0000-0xffff */
-       *strength = (tmp-181) * 0xffff / (236-181);
+               if (dev->delivery_system == SYS_DVBS) {
+                       dev->dvbv3_ber = buf[0] << 24 | buf[1] << 16 |
+                                        buf[2] << 8 | buf[3] << 0;
+                       dev->post_bit_error += buf[0] << 24 | buf[1] << 16 |
+                                              buf[2] << 8 | buf[3] << 0;
+                       c->post_bit_error.stat[0].scale = FE_SCALE_COUNTER;
+                       c->post_bit_error.stat[0].uvalue = dev->post_bit_error;
+                       dev->block_error += buf[4] << 8 | buf[5] << 0;
+                       c->block_error.stat[0].scale = FE_SCALE_COUNTER;
+                       c->block_error.stat[0].uvalue = dev->block_error;
+               } else {
+                       dev->dvbv3_ber = buf[0] << 8 | buf[1] << 0;
+                       dev->post_bit_error += buf[0] << 8 | buf[1] << 0;
+                       c->post_bit_error.stat[0].scale = FE_SCALE_COUNTER;
+                       c->post_bit_error.stat[0].uvalue = dev->post_bit_error;
+                       c->block_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+               }
+       } else {
+               c->post_bit_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+               c->block_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+       }
 
        return ret;
 error:
@@ -574,93 +510,50 @@ error:
        return ret;
 }
 
-static int tda10071_read_ber(struct dvb_frontend *fe, u32 *ber)
+static int tda10071_read_snr(struct dvb_frontend *fe, u16 *snr)
 {
-       struct tda10071_dev *dev = fe->demodulator_priv;
-       struct i2c_client *client = dev->client;
-       struct tda10071_cmd cmd;
-       int ret, i, len;
-       u8 tmp, reg, buf[8];
-
-       if (!dev->warm || !(dev->fe_status & FE_HAS_LOCK)) {
-               *ber = dev->ber = 0;
-               ret = 0;
-               goto error;
-       }
+       struct dtv_frontend_properties *c = &fe->dtv_property_cache;
 
-       switch (dev->delivery_system) {
-       case SYS_DVBS:
-               reg = 0x4c;
-               len = 8;
-               i = 1;
-               break;
-       case SYS_DVBS2:
-               reg = 0x4d;
-               len = 4;
-               i = 0;
-               break;
-       default:
-               *ber = dev->ber = 0;
-               return 0;
-       }
+       if (c->cnr.stat[0].scale == FE_SCALE_DECIBEL)
+               *snr = div_s64(c->cnr.stat[0].svalue, 100);
+       else
+               *snr = 0;
+       return 0;
+}
 
-       ret = tda10071_rd_reg(dev, reg, &tmp);
-       if (ret)
-               goto error;
+static int tda10071_read_signal_strength(struct dvb_frontend *fe, u16 *strength)
+{
+       struct dtv_frontend_properties *c = &fe->dtv_property_cache;
+       unsigned int uitmp;
 
-       if (dev->meas_count[i] == tmp) {
-               dev_dbg(&client->dev, "meas not ready=%02x\n", tmp);
-               *ber = dev->ber;
-               return 0;
+       if (c->strength.stat[0].scale == FE_SCALE_DECIBEL) {
+               uitmp = div_s64(c->strength.stat[0].svalue, 1000) + 256;
+               uitmp = clamp(uitmp, 181U, 236U); /* -75dBm - -20dBm */
+               /* scale value to 0x0000-0xffff */
+               *strength = (uitmp-181) * 0xffff / (236-181);
        } else {
-               dev->meas_count[i] = tmp;
+               *strength = 0;
        }
+       return 0;
+}
 
-       cmd.args[0] = CMD_BER_UPDATE_COUNTERS;
-       cmd.args[1] = 0;
-       cmd.args[2] = i;
-       cmd.len = 3;
-       ret = tda10071_cmd_execute(dev, &cmd);
-       if (ret)
-               goto error;
-
-       ret = tda10071_rd_regs(dev, cmd.len, buf, len);
-       if (ret)
-               goto error;
-
-       if (dev->delivery_system == SYS_DVBS) {
-               *ber = (buf[0] << 24) | (buf[1] << 16) | (buf[2] << 8) | buf[3];
-               dev->ucb += (buf[4] << 8) | buf[5];
-       } else {
-               *ber = (buf[0] << 8) | buf[1];
-       }
-       dev->ber = *ber;
+static int tda10071_read_ber(struct dvb_frontend *fe, u32 *ber)
+{
+       struct tda10071_dev *dev = fe->demodulator_priv;
 
-       return ret;
-error:
-       dev_dbg(&client->dev, "failed=%d\n", ret);
-       return ret;
+       *ber = dev->dvbv3_ber;
+       return 0;
 }
 
 static int tda10071_read_ucblocks(struct dvb_frontend *fe, u32 *ucblocks)
 {
-       struct tda10071_dev *dev = fe->demodulator_priv;
-       struct i2c_client *client = dev->client;
-       int ret = 0;
+       struct dtv_frontend_properties *c = &fe->dtv_property_cache;
 
-       if (!dev->warm || !(dev->fe_status & FE_HAS_LOCK)) {
+       if (c->block_error.stat[0].scale == FE_SCALE_COUNTER)
+               *ucblocks = c->block_error.stat[0].uvalue;
+       else
                *ucblocks = 0;
-               goto error;
-       }
-
-       /* UCB is updated when BER is read. Assume BER is read anyway. */
-
-       *ucblocks = dev->ucb;
-
-       return ret;
-error:
-       dev_dbg(&client->dev, "failed=%d\n", ret);
-       return ret;
+       return 0;
 }
 
 static int tda10071_set_frontend(struct dvb_frontend *fe)
@@ -772,11 +665,11 @@ static int tda10071_set_frontend(struct dvb_frontend *fe)
        else
                div = 4;
 
-       ret = tda10071_wr_reg(dev, 0x81, div);
+       ret = regmap_write(dev->regmap, 0x81, div);
        if (ret)
                goto error;
 
-       ret = tda10071_wr_reg(dev, 0xe3, div);
+       ret = regmap_write(dev->regmap, 0xe3, div);
        if (ret)
                goto error;
 
@@ -817,11 +710,11 @@ static int tda10071_get_frontend(struct dvb_frontend *fe)
        u8 buf[5], tmp;
 
        if (!dev->warm || !(dev->fe_status & FE_HAS_LOCK)) {
-               ret = -EFAULT;
+               ret = 0;
                goto error;
        }
 
-       ret = tda10071_rd_regs(dev, 0x30, buf, 5);
+       ret = regmap_bulk_read(dev->regmap, 0x30, buf, 5);
        if (ret)
                goto error;
 
@@ -854,7 +747,7 @@ static int tda10071_get_frontend(struct dvb_frontend *fe)
 
        c->frequency = (buf[2] << 16) | (buf[3] << 8) | (buf[4] << 0);
 
-       ret = tda10071_rd_regs(dev, 0x52, buf, 3);
+       ret = regmap_bulk_read(dev->regmap, 0x52, buf, 3);
        if (ret)
                goto error;
 
@@ -870,8 +763,10 @@ static int tda10071_init(struct dvb_frontend *fe)
 {
        struct tda10071_dev *dev = fe->demodulator_priv;
        struct i2c_client *client = dev->client;
+       struct dtv_frontend_properties *c = &fe->dtv_property_cache;
        struct tda10071_cmd cmd;
        int ret, i, len, remaining, fw_size;
+       unsigned int uitmp;
        const struct firmware *fw;
        u8 *fw_file = TDA10071_FIRMWARE;
        u8 tmp, buf[4];
@@ -971,19 +866,19 @@ static int tda10071_init(struct dvb_frontend *fe)
                }
 
                /*  download firmware */
-               ret = tda10071_wr_reg(dev, 0xe0, 0x7f);
+               ret = regmap_write(dev->regmap, 0xe0, 0x7f);
                if (ret)
                        goto error_release_firmware;
 
-               ret = tda10071_wr_reg(dev, 0xf7, 0x81);
+               ret = regmap_write(dev->regmap, 0xf7, 0x81);
                if (ret)
                        goto error_release_firmware;
 
-               ret = tda10071_wr_reg(dev, 0xf8, 0x00);
+               ret = regmap_write(dev->regmap, 0xf8, 0x00);
                if (ret)
                        goto error_release_firmware;
 
-               ret = tda10071_wr_reg(dev, 0xf9, 0x00);
+               ret = regmap_write(dev->regmap, 0xf9, 0x00);
                if (ret)
                        goto error_release_firmware;
 
@@ -1002,7 +897,7 @@ static int tda10071_init(struct dvb_frontend *fe)
                        if (len > (dev->i2c_wr_max - 1))
                                len = (dev->i2c_wr_max - 1);
 
-                       ret = tda10071_wr_regs(dev, 0xfa,
+                       ret = regmap_bulk_write(dev->regmap, 0xfa,
                                (u8 *) &fw->data[fw_size - remaining], len);
                        if (ret) {
                                dev_err(&client->dev,
@@ -1012,11 +907,11 @@ static int tda10071_init(struct dvb_frontend *fe)
                }
                release_firmware(fw);
 
-               ret = tda10071_wr_reg(dev, 0xf7, 0x0c);
+               ret = regmap_write(dev->regmap, 0xf7, 0x0c);
                if (ret)
                        goto error;
 
-               ret = tda10071_wr_reg(dev, 0xe0, 0x00);
+               ret = regmap_write(dev->regmap, 0xe0, 0x00);
                if (ret)
                        goto error;
 
@@ -1024,11 +919,11 @@ static int tda10071_init(struct dvb_frontend *fe)
                msleep(250);
 
                /* firmware status */
-               ret = tda10071_rd_reg(dev, 0x51, &tmp);
+               ret = regmap_read(dev->regmap, 0x51, &uitmp);
                if (ret)
                        goto error;
 
-               if (tmp) {
+               if (uitmp) {
                        dev_info(&client->dev, "firmware did not run\n");
                        ret = -EFAULT;
                        goto error;
@@ -1042,7 +937,7 @@ static int tda10071_init(struct dvb_frontend *fe)
                if (ret)
                        goto error;
 
-               ret = tda10071_rd_regs(dev, cmd.len, buf, 4);
+               ret = regmap_bulk_read(dev->regmap, cmd.len, buf, 4);
                if (ret)
                        goto error;
 
@@ -1051,7 +946,7 @@ static int tda10071_init(struct dvb_frontend *fe)
                dev_info(&client->dev, "found a '%s' in warm state\n",
                         tda10071_ops.info.name);
 
-               ret = tda10071_rd_regs(dev, 0x81, buf, 2);
+               ret = regmap_bulk_read(dev->regmap, 0x81, buf, 2);
                if (ret)
                        goto error;
 
@@ -1104,7 +999,7 @@ static int tda10071_init(struct dvb_frontend *fe)
                if (ret)
                        goto error;
 
-               ret = tda10071_wr_reg_mask(dev, 0xf0, 0x01, 0x01);
+               ret = regmap_update_bits(dev->regmap, 0xf0, 0x01, 0x01);
                if (ret)
                        goto error;
 
@@ -1134,6 +1029,16 @@ static int tda10071_init(struct dvb_frontend *fe)
                        goto error;
        }
 
+       /* init stats here in order signal app which stats are supported */
+       c->strength.len = 1;
+       c->strength.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+       c->cnr.len = 1;
+       c->cnr.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+       c->post_bit_error.len = 1;
+       c->post_bit_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+       c->block_error.len = 1;
+       c->block_error.stat[0].scale = FE_SCALE_NOT_AVAILABLE;
+
        return ret;
 error_release_firmware:
        release_firmware(fw);
@@ -1258,7 +1163,11 @@ static int tda10071_probe(struct i2c_client *client,
        struct tda10071_dev *dev;
        struct tda10071_platform_data *pdata = client->dev.platform_data;
        int ret;
-       u8 u8tmp;
+       unsigned int uitmp;
+       static const struct regmap_config regmap_config = {
+               .reg_bits = 8,
+               .val_bits = 8,
+       };
 
        dev = kzalloc(sizeof(*dev), GFP_KERNEL);
        if (!dev) {
@@ -1267,36 +1176,42 @@ static int tda10071_probe(struct i2c_client *client,
        }
 
        dev->client = client;
+       mutex_init(&dev->cmd_execute_mutex);
        dev->clk = pdata->clk;
        dev->i2c_wr_max = pdata->i2c_wr_max;
        dev->ts_mode = pdata->ts_mode;
        dev->spec_inv = pdata->spec_inv;
        dev->pll_multiplier = pdata->pll_multiplier;
        dev->tuner_i2c_addr = pdata->tuner_i2c_addr;
+       dev->regmap = devm_regmap_init_i2c(client, &regmap_config);
+       if (IS_ERR(dev->regmap)) {
+               ret = PTR_ERR(dev->regmap);
+               goto err_kfree;
+       }
 
        /* chip ID */
-       ret = tda10071_rd_reg(dev, 0xff, &u8tmp);
+       ret = regmap_read(dev->regmap, 0xff, &uitmp);
        if (ret)
                goto err_kfree;
-       if (u8tmp != 0x0f) {
+       if (uitmp != 0x0f) {
                ret = -ENODEV;
                goto err_kfree;
        }
 
        /* chip type */
-       ret = tda10071_rd_reg(dev, 0xdd, &u8tmp);
+       ret = regmap_read(dev->regmap, 0xdd, &uitmp);
        if (ret)
                goto err_kfree;
-       if (u8tmp != 0x00) {
+       if (uitmp != 0x00) {
                ret = -ENODEV;
                goto err_kfree;
        }
 
        /* chip version */
-       ret = tda10071_rd_reg(dev, 0xfe, &u8tmp);
+       ret = regmap_read(dev->regmap, 0xfe, &uitmp);
        if (ret)
                goto err_kfree;
-       if (u8tmp != 0x01) {
+       if (uitmp != 0x01) {
                ret = -ENODEV;
                goto err_kfree;
        }
@@ -1336,7 +1251,6 @@ MODULE_DEVICE_TABLE(i2c, tda10071_id_table);
 
 static struct i2c_driver tda10071_driver = {
        .driver = {
-               .owner  = THIS_MODULE,
                .name   = "tda10071",
                .suppress_bind_attrs = true,
        },