[linux-dvb] Re: [video4linux-cvs] Add support for the Avermedia 777 DVB-T card

[Michael Krufky]

Jose Alberto Reguero wrote:

>El Lunes, 30 de Enero de 2006 22:27, Hartmut Hackmann escribió:
>  
>
>>Ok, at the first glance there is only a minor issue with this patch and
>>it will not give optimum performance if the Philips recommendatios are
>>right.
>>I was not aware that dvb-pll already just provides the byte lists, so
>>we can agree to move the tuning code for MT352 to dvb-pll as a compromise.
>>    
>>
>I think that this patch solves the issue you mentioned. I think that now is the same code that without dvb-pll.
>  
>
>--- v4l-dvb/linux/drivers/media/dvb/frontends/dvb-pll.c	2006-01-31 09:46:16.000000000 +0100
>+++ v4l-dvb.new/linux/drivers/media/dvb/frontends/dvb-pll.c	2006-01-31 09:29:20.000000000 +0100
>@@ -391,15 +391,15 @@
> 	.setbw = td1316_bw,
> 	.count = 9,
> 	.entries = {
>-		{ 130000000, 36166000, 166666, 0xca, 0x60},
>-		{ 160000000, 36166000, 166666, 0xca, 0xa0},
>-		{ 200000000, 36166000, 166666, 0xca, 0xc0},
>-		{ 290000000, 36166000, 166666, 0xca, 0x60},
>-		{ 420000000, 36166000, 166666, 0xca, 0xa0},
>-		{ 480000000, 36166000, 166666, 0xca, 0xc0},
>-		{ 620000000, 36166000, 166666, 0xca, 0x60},
>-		{ 830000000, 36166000, 166666, 0xca, 0xa0},
>-		{ 895000000, 36166000, 166666, 0xca, 0xe0},
>+		{ 938340000, 36166000, 166666, 0xca, 0x60},
>+		{ 123834000, 36166000, 166666, 0xca, 0xa0},
>+		{ 163834000, 36166000, 166666, 0xca, 0xc0},
>+		{ 253834000, 36166000, 166666, 0xca, 0x60},
>+		{ 383834000, 36166000, 166666, 0xca, 0xa0},
>+		{ 443834000, 36166000, 166666, 0xca, 0xc0},
>+		{ 583834000, 36166000, 166666, 0xca, 0x60},
>+		{ 793834000, 36166000, 166666, 0xca, 0xa0},
>+		{ 858834000, 36166000, 166666, 0xca, 0xe0},
> 	},
> };
> EXPORT_SYMBOL(dvb_pll_philips_td1316);
>
Jose-

The patch above doesnt make any sense...  The ranges should be in order 
from min to max... Hartmut has already applied your last patch, but this 
one is obviously incorrect.....

Is the current code in Hg not working correctly?

If not, then the patch above should be corrected, but I'm not sure that 
I understand what you're trying to do, here.

Regards,

Michael Krufky