Andrew de Quincey wrote:
On Friday 14 April 2006 14:11, Patrick Boettcher wrote:
On Fri, 14 Apr 2006, Andrew de Quincey wrote:
OK, after consulting with Manu a bit, my original idea isn't going to
work well. Here is an updated patch.
The initialisation sequence would now be:
struct dvb_frontend *fe = stv0299_attach();
stb6100_attach(fe);
The stb6100 attaches itself to the stv0299 using the (*attach_tuner)()
callback.
1) What do you think about adding a void *priv_tuner to dvb_frontend?
And a tuner would use that, instead of passing a void* in the argument of
the function pointer?
yeah, sounds much neater.
2) Why not creating a struct dvb_tuner and struct dvb_tuner_ops? and a
dvb_frontend would then also contain an optional pointer to a
dvb_tuner-struct?
With that idea we could even move the dvb_frontend_ops to a dvb_demod and
a dvb_demod_ops-struct and a dvb_frontend would consist of dvb_demod
+ dvb_tuner.
Just my thoughts - maybe too radical. But what about the first one?
LOL! Look at my latest patch :)
hehe
_______________________________________________
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
