Re: [git:v4l-dvb/master] V4L/DVB: tuner-xc2028: fix tuning logic to solve a regression in Australia

[Stefan Ringel <stefan.ringel@arcor.de>]

Am 19.02.2010 21:51, schrieb Mauro Carvalho Chehab:
> Stefan Ringel wrote:
>   
>> Am 19.02.2010 17:07, schrieb Patch from Mauro Carvalho Chehab:
>>     
>>>  	}
>>>  
>>>  	div = (freq - offset + DIV / 2) / DIV;
>>> @@ -1114,17 +1152,22 @@ static int xc2028_set_params(struct dvb_frontend *fe,
>>>  
>>>  	/* All S-code tables need a 200kHz shift */
>>>  	if (priv->ctrl.demod) {
>>> -		demod = priv->ctrl.demod + 200;
>>> +		/*
>>> +		 * Newer firmwares require a 200 kHz offset only for ATSC
>>> +		 */
>>> +		if (type == ATSC || priv->firm_version < 0x0302)
>>> +			demod = priv->ctrl.demod + 200;
>>>  		/*
>>>  		 * The DTV7 S-code table needs a 700 kHz shift.
>>> -		 * Thanks to Terry Wu <terrywu2009@gmail.com> for reporting this
>>>  		 *
>>>  		 * DTV7 is only used in Australia.  Germany or Italy may also
>>>  		 * use this firmware after initialization, but a tune to a UHF
>>>  		 * channel should then cause DTV78 to be used.
>>> +		 *
>>> +		 * Unfortunately, on real-field tests, the s-code offset
>>> +		 * didn't work as expected, as reported by
>>> +		 * Robert Lowery <rglowery@exemail.com.au>
>>>  		 */
>>> -		if (type & DTV7)
>>> -			demod += 500;
>>>  	}
>>>  
>>>  	return generic_set_freq(fe, p->frequency,
>>>   
>>>       
>> Hi Mauro,
>>
>> your patch doesn't work. Here is not set demod for all others (demod=0).
>>
>>     
> For DVB to properly work, you need to fill ctrl.demod at tm6000, otherwise,
> demod will be 0, and it will use some default that won't likely work.
>
>   
ctrl.demod is set in tm6000 since last month and doesn't work any more now!

Stefan Ringel

-- 
Stefan Ringel <stefan.ringel@arcor.de>