Commit a7d44baa authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab

[media] cx23885-dvb: Remove a dirty hack that would require DVBv3

The cx23885-dvb driver has a dirty hack:
	1) it hooks the DVBv3 legacy call to FE_SET_FRONTEND;
	2) it uses internally the DVBv3 struct to decide some
	   configs.

Replace it by a change during the gate control. This will
likely work, but requires testing. Anyway, the current way
will break, as soon as we stop copying data for DVBv3 for
pure DVBv5 calls.

Compile-tested only.

Cc: Michael Krufky <mkrufky@linuxtv.org>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent c6f56e7d
...@@ -111,6 +111,8 @@ static void dvb_buf_release(struct videobuf_queue *q, ...@@ -111,6 +111,8 @@ static void dvb_buf_release(struct videobuf_queue *q,
cx23885_free_buffer(q, (struct cx23885_buffer *)vb); cx23885_free_buffer(q, (struct cx23885_buffer *)vb);
} }
static int cx23885_dvb_set_frontend(struct dvb_frontend *fe);
static void cx23885_dvb_gate_ctrl(struct cx23885_tsport *port, int open) static void cx23885_dvb_gate_ctrl(struct cx23885_tsport *port, int open)
{ {
struct videobuf_dvb_frontends *f; struct videobuf_dvb_frontends *f;
...@@ -125,6 +127,12 @@ static void cx23885_dvb_gate_ctrl(struct cx23885_tsport *port, int open) ...@@ -125,6 +127,12 @@ static void cx23885_dvb_gate_ctrl(struct cx23885_tsport *port, int open)
if (fe && fe->dvb.frontend && fe->dvb.frontend->ops.i2c_gate_ctrl) if (fe && fe->dvb.frontend && fe->dvb.frontend->ops.i2c_gate_ctrl)
fe->dvb.frontend->ops.i2c_gate_ctrl(fe->dvb.frontend, open); fe->dvb.frontend->ops.i2c_gate_ctrl(fe->dvb.frontend, open);
/*
* FIXME: Improve this path to avoid calling the
* cx23885_dvb_set_frontend() every time it passes here.
*/
cx23885_dvb_set_frontend(fe->dvb.frontend);
} }
static struct videobuf_queue_ops dvb_qops = { static struct videobuf_queue_ops dvb_qops = {
...@@ -479,15 +487,15 @@ static struct xc5000_config mygica_x8506_xc5000_config = { ...@@ -479,15 +487,15 @@ static struct xc5000_config mygica_x8506_xc5000_config = {
.if_khz = 5380, .if_khz = 5380,
}; };
static int cx23885_dvb_set_frontend(struct dvb_frontend *fe, static int cx23885_dvb_set_frontend(struct dvb_frontend *fe)
struct dvb_frontend_parameters *param)
{ {
struct dtv_frontend_properties *p = &fe->dtv_property_cache;
struct cx23885_tsport *port = fe->dvb->priv; struct cx23885_tsport *port = fe->dvb->priv;
struct cx23885_dev *dev = port->dev; struct cx23885_dev *dev = port->dev;
switch (dev->board) { switch (dev->board) {
case CX23885_BOARD_HAUPPAUGE_HVR1275: case CX23885_BOARD_HAUPPAUGE_HVR1275:
switch (param->u.vsb.modulation) { switch (p->modulation) {
case VSB_8: case VSB_8:
cx23885_gpio_clear(dev, GPIO_5); cx23885_gpio_clear(dev, GPIO_5);
break; break;
...@@ -507,31 +515,6 @@ static int cx23885_dvb_set_frontend(struct dvb_frontend *fe, ...@@ -507,31 +515,6 @@ static int cx23885_dvb_set_frontend(struct dvb_frontend *fe,
return 0; return 0;
} }
static int cx23885_dvb_fe_ioctl_override(struct dvb_frontend *fe,
unsigned int cmd, void *parg,
unsigned int stage)
{
int err = 0;
switch (stage) {
case DVB_FE_IOCTL_PRE:
switch (cmd) {
case FE_SET_FRONTEND:
err = cx23885_dvb_set_frontend(fe,
(struct dvb_frontend_parameters *) parg);
break;
}
break;
case DVB_FE_IOCTL_POST:
/* no post-ioctl handling required */
break;
}
return err;
};
static struct lgs8gxx_config magicpro_prohdtve2_lgs8g75_config = { static struct lgs8gxx_config magicpro_prohdtve2_lgs8g75_config = {
.prod = LGS8GXX_PROD_LGS8G75, .prod = LGS8GXX_PROD_LGS8G75,
.demod_address = 0x19, .demod_address = 0x19,
...@@ -1151,7 +1134,7 @@ static int dvb_register(struct cx23885_tsport *port) ...@@ -1151,7 +1134,7 @@ static int dvb_register(struct cx23885_tsport *port)
/* register everything */ /* register everything */
ret = videobuf_dvb_register_bus(&port->frontends, THIS_MODULE, port, ret = videobuf_dvb_register_bus(&port->frontends, THIS_MODULE, port,
&dev->pci->dev, adapter_nr, mfe_shared, &dev->pci->dev, adapter_nr, mfe_shared,
cx23885_dvb_fe_ioctl_override); NULL);
if (ret) if (ret)
goto frontend_detach; goto frontend_detach;
......
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