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 :) _______________________________________________ linux-dvb@xxxxxxxxxxx http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
