Hi Gavin, On Thu, 2005-06-23 at 01:40 +0200, Johannes Stezenbach wrote: > There are a few very minor nits. Could you please clean them up > and resubmit the patch with a Signed-off-by: line? > (see dvb-kernel/SubmittingPatches) > > > +static int alps_tdbe2_pll_set(struct dvb_frontend* fe, struct dvb_frontend_parameters* params) > > +{ > > + struct ttusb* ttusb = (struct ttusb*) fe->dvb->priv; Please remove the unneeded cast from void *, too. Regards, Andreas