Commit 01ce5a79 authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] mt2031: remove fake implementaion of get_bandwidth()

This driver implements a fake get_bandwidth() callback. In
reallity, the tuner driver won't adjust its low-pass
filter based on a bandwidth, and were just providing a fake
method for demods to read whatever was "set".

This code is useless, as none of the drivers that use
this tuner seems to require a get_bandwidth() callback.

While here, convert set_params to use the DVBv5 way to pass
parameters to tuners.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 2676c258
...@@ -95,6 +95,7 @@ static int mt2131_writeregs(struct mt2131_priv *priv,u8 *buf, u8 len) ...@@ -95,6 +95,7 @@ static int mt2131_writeregs(struct mt2131_priv *priv,u8 *buf, u8 len)
static int mt2131_set_params(struct dvb_frontend *fe, static int mt2131_set_params(struct dvb_frontend *fe,
struct dvb_frontend_parameters *params) struct dvb_frontend_parameters *params)
{ {
struct dtv_frontend_properties *c = &fe->dtv_property_cache;
struct mt2131_priv *priv; struct mt2131_priv *priv;
int ret=0, i; int ret=0, i;
u32 freq; u32 freq;
...@@ -105,12 +106,8 @@ static int mt2131_set_params(struct dvb_frontend *fe, ...@@ -105,12 +106,8 @@ static int mt2131_set_params(struct dvb_frontend *fe,
u8 lockval = 0; u8 lockval = 0;
priv = fe->tuner_priv; priv = fe->tuner_priv;
if (fe->ops.info.type == FE_OFDM)
priv->bandwidth = params->u.ofdm.bandwidth;
else
priv->bandwidth = 0;
freq = params->frequency / 1000; // Hz -> kHz freq = c->frequency / 1000; /* Hz -> kHz */
dprintk(1, "%s() freq=%d\n", __func__, freq); dprintk(1, "%s() freq=%d\n", __func__, freq);
f_lo1 = freq + MT2131_IF1 * 1000; f_lo1 = freq + MT2131_IF1 * 1000;
...@@ -193,14 +190,6 @@ static int mt2131_get_frequency(struct dvb_frontend *fe, u32 *frequency) ...@@ -193,14 +190,6 @@ static int mt2131_get_frequency(struct dvb_frontend *fe, u32 *frequency)
return 0; return 0;
} }
static int mt2131_get_bandwidth(struct dvb_frontend *fe, u32 *bandwidth)
{
struct mt2131_priv *priv = fe->tuner_priv;
dprintk(1, "%s()\n", __func__);
*bandwidth = priv->bandwidth;
return 0;
}
static int mt2131_get_status(struct dvb_frontend *fe, u32 *status) static int mt2131_get_status(struct dvb_frontend *fe, u32 *status)
{ {
struct mt2131_priv *priv = fe->tuner_priv; struct mt2131_priv *priv = fe->tuner_priv;
...@@ -263,7 +252,6 @@ static const struct dvb_tuner_ops mt2131_tuner_ops = { ...@@ -263,7 +252,6 @@ static const struct dvb_tuner_ops mt2131_tuner_ops = {
.set_params = mt2131_set_params, .set_params = mt2131_set_params,
.get_frequency = mt2131_get_frequency, .get_frequency = mt2131_get_frequency,
.get_bandwidth = mt2131_get_bandwidth,
.get_status = mt2131_get_status .get_status = mt2131_get_status
}; };
...@@ -281,7 +269,6 @@ struct dvb_frontend * mt2131_attach(struct dvb_frontend *fe, ...@@ -281,7 +269,6 @@ struct dvb_frontend * mt2131_attach(struct dvb_frontend *fe,
return NULL; return NULL;
priv->cfg = cfg; priv->cfg = cfg;
priv->bandwidth = 6000000; /* 6MHz */
priv->i2c = i2c; priv->i2c = i2c;
if (mt2131_readreg(priv, 0, &id) != 0) { if (mt2131_readreg(priv, 0, &id) != 0) {
......
...@@ -38,7 +38,6 @@ struct mt2131_priv { ...@@ -38,7 +38,6 @@ struct mt2131_priv {
struct i2c_adapter *i2c; struct i2c_adapter *i2c;
u32 frequency; u32 frequency;
u32 bandwidth;
}; };
#endif /* __MT2131_PRIV_H__ */ #endif /* __MT2131_PRIV_H__ */
......
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