V4L/DVB (12166): cx23885: add FIXME comment above set_frontend override
authorMichael Krufky <mkrufky@kernellabs.com>
Tue, 16 Jun 2009 20:08:17 +0000 (17:08 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Sun, 5 Jul 2009 17:30:01 +0000 (14:30 -0300)
add FIXME comment to indicate that the set_frontend override is a
temporary hack.  This will be done a better way in the next kernel.

Signed-off-by: Michael Krufky <mkrufky@kernellabs.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/cx23885/cx23885-dvb.c
drivers/media/video/cx23885/cx23885.h

index a851a81e257e2119e4cf1e53ed1d40caa88a863a..86ac529e62be0be042ea6912225ddf9d7d1d89bc 100644 (file)
@@ -527,6 +527,7 @@ static int dvb_register(struct cx23885_tsport *port)
                                   &hauppauge_hvr127x_config);
                }
 
+               /* FIXME: temporary hack */
                /* define bridge override to set_frontend */
                port->set_frontend_save = fe0->dvb.frontend->ops.set_frontend;
                fe0->dvb.frontend->ops.set_frontend = cx23885_dvb_set_frontend;
index f4e22b0af7d2b9cf458b8095d9327cc3ce5f26a7..214a55e943b783dc7a6cfaab95bb25f4fe504ee1 100644 (file)
@@ -288,6 +288,8 @@ struct cx23885_tsport {
        /* Allow a single tsport to have multiple frontends */
        u32                        num_frontends;
        void                       *port_priv;
+
+       /* FIXME: temporary hack */
        int (*set_frontend_save) (struct dvb_frontend *,
                                  struct dvb_frontend_parameters *);
 };