Commit 6714049e authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] tda10086: convert set_fontend to use DVBv5 parameters

Instead of using dvb_frontend_parameters struct, that were
designed for a subset of the supported standards, use the DVBv5
cache information.

Also, fill the supported delivery systems at dvb_frontend_ops
struct.
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent b2a29b57
...@@ -267,7 +267,7 @@ static int tda10086_send_burst (struct dvb_frontend* fe, fe_sec_mini_cmd_t minic ...@@ -267,7 +267,7 @@ static int tda10086_send_burst (struct dvb_frontend* fe, fe_sec_mini_cmd_t minic
} }
static int tda10086_set_inversion(struct tda10086_state *state, static int tda10086_set_inversion(struct tda10086_state *state,
struct dvb_frontend_parameters *fe_params) struct dtv_frontend_properties *fe_params)
{ {
u8 invval = 0x80; u8 invval = 0x80;
...@@ -292,7 +292,7 @@ static int tda10086_set_inversion(struct tda10086_state *state, ...@@ -292,7 +292,7 @@ static int tda10086_set_inversion(struct tda10086_state *state,
} }
static int tda10086_set_symbol_rate(struct tda10086_state *state, static int tda10086_set_symbol_rate(struct tda10086_state *state,
struct dvb_frontend_parameters *fe_params) struct dtv_frontend_properties *fe_params)
{ {
u8 dfn = 0; u8 dfn = 0;
u8 afs = 0; u8 afs = 0;
...@@ -303,7 +303,7 @@ static int tda10086_set_symbol_rate(struct tda10086_state *state, ...@@ -303,7 +303,7 @@ static int tda10086_set_symbol_rate(struct tda10086_state *state,
u32 tmp; u32 tmp;
u32 bdr; u32 bdr;
u32 bdri; u32 bdri;
u32 symbol_rate = fe_params->u.qpsk.symbol_rate; u32 symbol_rate = fe_params->symbol_rate;
dprintk ("%s %i\n", __func__, symbol_rate); dprintk ("%s %i\n", __func__, symbol_rate);
...@@ -367,13 +367,13 @@ static int tda10086_set_symbol_rate(struct tda10086_state *state, ...@@ -367,13 +367,13 @@ static int tda10086_set_symbol_rate(struct tda10086_state *state,
} }
static int tda10086_set_fec(struct tda10086_state *state, static int tda10086_set_fec(struct tda10086_state *state,
struct dvb_frontend_parameters *fe_params) struct dtv_frontend_properties *fe_params)
{ {
u8 fecval; u8 fecval;
dprintk ("%s %i\n", __func__, fe_params->u.qpsk.fec_inner); dprintk("%s %i\n", __func__, fe_params->fec_inner);
switch(fe_params->u.qpsk.fec_inner) { switch (fe_params->fec_inner) {
case FEC_1_2: case FEC_1_2:
fecval = 0x00; fecval = 0x00;
break; break;
...@@ -409,9 +409,9 @@ static int tda10086_set_fec(struct tda10086_state *state, ...@@ -409,9 +409,9 @@ static int tda10086_set_fec(struct tda10086_state *state,
return 0; return 0;
} }
static int tda10086_set_frontend(struct dvb_frontend* fe, static int tda10086_set_frontend(struct dvb_frontend *fe)
struct dvb_frontend_parameters *fe_params)
{ {
struct dtv_frontend_properties *fe_params = &fe->dtv_property_cache;
struct tda10086_state *state = fe->demodulator_priv; struct tda10086_state *state = fe->demodulator_priv;
int ret; int ret;
u32 freq = 0; u32 freq = 0;
...@@ -452,12 +452,12 @@ static int tda10086_set_frontend(struct dvb_frontend* fe, ...@@ -452,12 +452,12 @@ static int tda10086_set_frontend(struct dvb_frontend* fe,
tda10086_write_mask(state, 0x10, 0x40, 0x40); tda10086_write_mask(state, 0x10, 0x40, 0x40);
tda10086_write_mask(state, 0x00, 0x01, 0x00); tda10086_write_mask(state, 0x00, 0x01, 0x00);
state->symbol_rate = fe_params->u.qpsk.symbol_rate; state->symbol_rate = fe_params->symbol_rate;
state->frequency = fe_params->frequency; state->frequency = fe_params->frequency;
return 0; return 0;
} }
static int tda10086_get_frontend(struct dvb_frontend* fe, struct dvb_frontend_parameters *fe_params) static int tda10086_get_frontend(struct dvb_frontend *fe, struct dtv_frontend_properties *fe_params)
{ {
struct tda10086_state* state = fe->demodulator_priv; struct tda10086_state* state = fe->demodulator_priv;
u8 val; u8 val;
...@@ -467,7 +467,7 @@ static int tda10086_get_frontend(struct dvb_frontend* fe, struct dvb_frontend_pa ...@@ -467,7 +467,7 @@ static int tda10086_get_frontend(struct dvb_frontend* fe, struct dvb_frontend_pa
dprintk ("%s\n", __func__); dprintk ("%s\n", __func__);
/* check for invalid symbol rate */ /* check for invalid symbol rate */
if (fe_params->u.qpsk.symbol_rate < 500000) if (fe_params->symbol_rate < 500000)
return -EINVAL; return -EINVAL;
/* calculate the updated frequency (note: we convert from Hz->kHz) */ /* calculate the updated frequency (note: we convert from Hz->kHz) */
...@@ -516,34 +516,34 @@ static int tda10086_get_frontend(struct dvb_frontend* fe, struct dvb_frontend_pa ...@@ -516,34 +516,34 @@ static int tda10086_get_frontend(struct dvb_frontend* fe, struct dvb_frontend_pa
tmp |= 0xffffff00; tmp |= 0xffffff00;
tmp = (tmp * 480 * (1<<1)) / 128; tmp = (tmp * 480 * (1<<1)) / 128;
tmp = ((state->symbol_rate/1000) * tmp) / (1000000/1000); tmp = ((state->symbol_rate/1000) * tmp) / (1000000/1000);
fe_params->u.qpsk.symbol_rate = state->symbol_rate + tmp; fe_params->symbol_rate = state->symbol_rate + tmp;
/* the FEC */ /* the FEC */
val = (tda10086_read_byte(state, 0x0d) & 0x70) >> 4; val = (tda10086_read_byte(state, 0x0d) & 0x70) >> 4;
switch(val) { switch(val) {
case 0x00: case 0x00:
fe_params->u.qpsk.fec_inner = FEC_1_2; fe_params->fec_inner = FEC_1_2;
break; break;
case 0x01: case 0x01:
fe_params->u.qpsk.fec_inner = FEC_2_3; fe_params->fec_inner = FEC_2_3;
break; break;
case 0x02: case 0x02:
fe_params->u.qpsk.fec_inner = FEC_3_4; fe_params->fec_inner = FEC_3_4;
break; break;
case 0x03: case 0x03:
fe_params->u.qpsk.fec_inner = FEC_4_5; fe_params->fec_inner = FEC_4_5;
break; break;
case 0x04: case 0x04:
fe_params->u.qpsk.fec_inner = FEC_5_6; fe_params->fec_inner = FEC_5_6;
break; break;
case 0x05: case 0x05:
fe_params->u.qpsk.fec_inner = FEC_6_7; fe_params->fec_inner = FEC_6_7;
break; break;
case 0x06: case 0x06:
fe_params->u.qpsk.fec_inner = FEC_7_8; fe_params->fec_inner = FEC_7_8;
break; break;
case 0x07: case 0x07:
fe_params->u.qpsk.fec_inner = FEC_8_9; fe_params->fec_inner = FEC_8_9;
break; break;
} }
...@@ -701,7 +701,7 @@ static void tda10086_release(struct dvb_frontend* fe) ...@@ -701,7 +701,7 @@ static void tda10086_release(struct dvb_frontend* fe)
} }
static struct dvb_frontend_ops tda10086_ops = { static struct dvb_frontend_ops tda10086_ops = {
.delsys = { SYS_DVBS },
.info = { .info = {
.name = "Philips TDA10086 DVB-S", .name = "Philips TDA10086 DVB-S",
.type = FE_QPSK, .type = FE_QPSK,
...@@ -722,8 +722,8 @@ static struct dvb_frontend_ops tda10086_ops = { ...@@ -722,8 +722,8 @@ static struct dvb_frontend_ops tda10086_ops = {
.sleep = tda10086_sleep, .sleep = tda10086_sleep,
.i2c_gate_ctrl = tda10086_i2c_gate_ctrl, .i2c_gate_ctrl = tda10086_i2c_gate_ctrl,
.set_frontend_legacy = tda10086_set_frontend, .set_frontend = tda10086_set_frontend,
.get_frontend_legacy = tda10086_get_frontend, .get_frontend = tda10086_get_frontend,
.get_tune_settings = tda10086_get_tune_settings, .get_tune_settings = tda10086_get_tune_settings,
.read_status = tda10086_read_status, .read_status = tda10086_read_status,
......
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