[linux-dvb] Re: refactoring

[Andrew de Quincey <adq_dvb@lidskialf.net>]

On Monday 11 Oct 2004 12:43, Gerd Knorr wrote:
> On Mon, Oct 11, 2004 at 12:20:11PM +0100, Andrew de Quincey wrote:
> > On Monday 11 Oct 2004 12:06, Johannes Stezenbach wrote:
> > > On Mon, Oct 11, 2004 at 10:59:42AM +0200, Gerd Knorr wrote:
> >
> > Actually, it would have to be:
> >
> > struct dvb_frontend* at76c651_attach(struct i2c_adapter *adap,
> >         const struct at76c651_config* config,
> >         struct dvb_adapter* dvb,
>
> Any reason why you pass down that one?  I'd let dvb_register_frontend
> fill that, i.e. initialize the fe like this:
>
>     struct dvb_frontend *fe;
>
>     fe = at76c651_attach(adap, &my_card_config.at76c651, *adap_priv);
>     if (NULL == fe)
>  /* cry for help */
>     dvb_frontend_register(dvb_adapter, fe);
>
> Maybe also this way:
>
>     fe->dvb = dvb_adapter;
>     dvb_frontend_register(fe);

The dvb_frontend structure is part of the dvb_frontend.c library - so that 
library should really be allocating/deallocating it.