[linux-dvb] Re: Missing LOCK with tda10045h

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

Again, sorry about the delay, been very busy at work for the last few days.... 
And I've become slightly addicted to Gothic 2 :)

On Friday 20 June 2003 20:04, Henning Glawe wrote:
> On Fri, Jun 20, 2003 at 08:09:09PM +0200, Robert Schlabbach wrote:
> > Try setting everything to AUTO, except Bandwidth. The TDA10045H can
> > autodetect all the parameters, but if you provide them, Andrew's code
> > disabled the autodetection, IIRC. I think autodetection is fast enough to
> > always utilize it.
> >
> > That might make the scan work as well...
>
> the scan works now, but only scanning the last tuned transponder (with
> option -c), which makes the 'scan' in an unknown environment a bit
> useless ;)

So scan itself is unable to tune to DVB signals... but you can tune using 
other programs?

> is somebody already working on this issue ? Though I don't know the dvb
> driver infrastructure at all, I have some C coding experience, so maybe
> I look at it...

Attached is a patch which forces AUTO on all parameters it can. 

This sounds like scan is supplying specific DVB parameters (e.g. FEC_2_3), but 
expecting the driver to always use AUTO anyway.. I don't know whether scan 
should be fixed, or the driver hacked to force auto. Anyone got any opinions?
Index: linux/drivers/media/dvb/frontends/tda10045h.c
===================================================================
RCS file: /cvs/linuxtv/dvb-kernel/linux/drivers/media/dvb/frontends/tda10045h.c,v
retrieving revision 1.4
diff -u -p -r1.4 tda10045h.c
--- linux/drivers/media/dvb/frontends/tda10045h.c	19 Jun 2003 09:18:02 -0000	1.4
+++ linux/drivers/media/dvb/frontends/tda10045h.c	21 Jun 2003 13:12:35 -0000
@@ -671,6 +671,10 @@ int tda10045h_set_fe(struct dvb_i2c_bus 
 	if (tmp < 0)
 		return tmp;
 
+        fe_params->u.ofdm.code_rate_HP = FEC_AUTO;
+        fe_params->u.ofdm.guard_interval = GUARD_INTERVAL_AUTO;
+        fe_params->u.ofdm.transmission_mode = TRANSMISSION_MODE_AUTO;
+	  
 	// Set standard params.. or put them to auto
 	if ((fe_params->u.ofdm.code_rate_HP == FEC_AUTO) ||
 	    (fe_params->u.ofdm.code_rate_LP == FEC_AUTO) ||