[REVIEW PATCH 65/86] e4000: remove .set_config() which was for controls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



That custom DVB callback is not needed anymore for setting gain
controls as those are now implemented using V4L2 control framework.

That change was proposed by Mauro.

Cc: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx>
Signed-off-by: Antti Palosaari <crope@xxxxxx>
---
 drivers/media/tuners/e4000.c | 68 --------------------------------------------
 drivers/media/tuners/e4000.h |  6 ----
 2 files changed, 74 deletions(-)

diff --git a/drivers/media/tuners/e4000.c b/drivers/media/tuners/e4000.c
index 77318e9..019dc62 100644
--- a/drivers/media/tuners/e4000.c
+++ b/drivers/media/tuners/e4000.c
@@ -381,73 +381,6 @@ static int e4000_get_if_frequency(struct dvb_frontend *fe, u32 *frequency)
 	return 0;
 }
 
-static int e4000_set_config(struct dvb_frontend *fe, void *priv_cfg)
-{
-	struct e4000_priv *priv = fe->tuner_priv;
-	struct e4000_ctrl *ctrl = priv_cfg;
-	int ret;
-	u8 buf[2];
-	u8 u8tmp;
-	dev_dbg(&priv->client->dev, "%s: lna=%d mixer=%d if=%d\n", __func__,
-			ctrl->lna_gain, ctrl->mixer_gain, ctrl->if_gain);
-
-	if (fe->ops.i2c_gate_ctrl)
-		fe->ops.i2c_gate_ctrl(fe, 1);
-
-	if (ctrl->lna_gain == INT_MIN && ctrl->if_gain == INT_MIN)
-		u8tmp = 0x17;
-	else if (ctrl->lna_gain == INT_MIN)
-		u8tmp = 0x19;
-	else if (ctrl->if_gain == INT_MIN)
-		u8tmp = 0x16;
-	else
-		u8tmp = 0x10;
-
-	ret = e4000_wr_reg(priv, 0x1a, u8tmp);
-	if (ret)
-		goto err;
-
-	if (ctrl->mixer_gain == INT_MIN)
-		u8tmp = 0x15;
-	else
-		u8tmp = 0x14;
-
-	ret = e4000_wr_reg(priv, 0x20, u8tmp);
-	if (ret)
-		goto err;
-
-	if (ctrl->lna_gain != INT_MIN) {
-		ret = e4000_wr_reg(priv, 0x14, ctrl->lna_gain);
-		if (ret)
-			goto err;
-	}
-
-	if (ctrl->mixer_gain != INT_MIN) {
-		ret = e4000_wr_reg(priv, 0x15, ctrl->mixer_gain);
-		if (ret)
-			goto err;
-	}
-
-	if (ctrl->if_gain != INT_MIN) {
-		buf[0] = e4000_if_gain_lut[ctrl->if_gain].reg16_val;
-		buf[1] = e4000_if_gain_lut[ctrl->if_gain].reg17_val;
-		ret = e4000_wr_regs(priv, 0x16, buf, 2);
-		if (ret)
-			goto err;
-	}
-
-	if (fe->ops.i2c_gate_ctrl)
-		fe->ops.i2c_gate_ctrl(fe, 0);
-
-	return 0;
-err:
-	if (fe->ops.i2c_gate_ctrl)
-		fe->ops.i2c_gate_ctrl(fe, 0);
-
-	dev_dbg(&priv->client->dev, "%s: failed=%d\n", __func__, ret);
-	return ret;
-}
-
 static int e4000_set_lna_gain(struct dvb_frontend *fe)
 {
 	struct e4000_priv *priv = fe->tuner_priv;
@@ -630,7 +563,6 @@ static const struct dvb_tuner_ops e4000_tuner_ops = {
 	.init = e4000_init,
 	.sleep = e4000_sleep,
 	.set_params = e4000_set_params,
-	.set_config = e4000_set_config,
 
 	.get_if_frequency = e4000_get_if_frequency,
 };
diff --git a/drivers/media/tuners/e4000.h b/drivers/media/tuners/e4000.h
index d86de6d..989f2ea 100644
--- a/drivers/media/tuners/e4000.h
+++ b/drivers/media/tuners/e4000.h
@@ -40,12 +40,6 @@ struct e4000_config {
 	u32 clock;
 };
 
-struct e4000_ctrl {
-	int lna_gain;
-	int mixer_gain;
-	int if_gain;
-};
-
 #if IS_ENABLED(CONFIG_MEDIA_TUNER_E4000)
 extern struct v4l2_ctrl_handler *e4000_get_ctrl_handler(
 		struct dvb_frontend *fe
-- 
1.8.5.3

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux