On Mon, 11 May 2009 05:35:51 +0200 hermann pitton <hermann-pitton@xxxxxxxx> wrote: > Dmitri, > > Am Sonntag, den 10.05.2009, 08:52 +1000 schrieb Dmitri Belimov: > > Hi All. > > > > > [snip] > > > > > > > > > > Channel designations I dug out of ivtv-tune: > > > > > > > > > > S38 439.250 MHz (European cable) > > > > > H18 439.250 MHz (SECAM France) > > > > > 47 440.250 MHz (PAL China) > > > > > 059 440.250 MHz (PAL Argentina) > > > > > > > > > > come close, but are unaffected by the change from 442 to 441 > > > > > as the bandswitch cutover point. These channels fall right > > > > > on top of the cutover, but are not affected by the proposed > > > > > change in any meaningful way. The VHF-High filter and VCO > > > > > would still be used. Dmitri's proposed change is a "don't > > > > > care" unless the cutover point is changed to 440 MHz. > > > > > > > > > > > > > > > Let's pretend that the proposed cutover point is 440 MHz. > > > > > > NO! it is not > > > > > > Dmitri, > > > > > > can you cut one off and tell us what it is all about ? > > > > > > Unless you do so, all other is pointless and I likely stop to > > > participate in such stuff. > > > > Sorry my delay. I lost subject of discussion. What main question?? > > > > 1. AGC TOP of RF part - I think need support for MK3 > > 2. Changing to 441MHz is not critical. We can write some > > information about this case to Wiki or docs. > > for 2.: Discussed to the end if you stay at 441MHz. If you still want > to have it in, just send a patch and no more info is needed. (Likely > Andy is giving only examples for more difficult cases, sorry.) This is my patch. diff -r b40d628f830d linux/drivers/media/common/tuners/tuner-types.c --- a/linux/drivers/media/common/tuners/tuner-types.c Fri Apr 24 01:46:41 2009 -0300 +++ b/linux/drivers/media/common/tuners/tuner-types.c Tue Apr 28 03:35:42 2009 +1000 @@ -558,8 +558,8 @@ static struct tuner_range tuner_fm1216me_mk3_pal_ranges[] = { { 16 * 158.00 /*MHz*/, 0x8e, 0x01, }, - { 16 * 442.00 /*MHz*/, 0x8e, 0x02, }, - { 16 * 999.99 , 0x8e, 0x04, }, + { 16 * 441.00 /*MHz*/, 0x8e, 0x02, }, + { 16 * 864.00 , 0x8e, 0x04, }, }; static struct tuner_params tuner_fm1216me_mk3_params[] = { Signed-off-by: Beholder Intl. Ltd. Dmitry Belimov <d.belimov@xxxxxxxxx> > for 1.: I would like to be absolutely sure, that we are talking about > the same tuner. I want to have the exact filters on it at least. > > 3.: That is what Andy noted. Following the Philips datasheet for TOP, > it should be added for negative modulation, positive modulation and FM > accordingly. (2 and 3 are out of discussion) > > If you still have some sort of Secam fire and can improve it, we must > know the tuner you are on "exactly". If it is the original Philips, > without any such TOP suggestions over the full range in recent > datasheets (???), I assume you might have them, I would say you can > proceed, if you have shown that you are really still on the same > tuner. Ok. I read the datasheet, make photos inside tuner. With my best regards, Dmitry. > Cheers, > Hermann > > > > > >
diff -r b40d628f830d linux/drivers/media/common/tuners/tuner-types.c --- a/linux/drivers/media/common/tuners/tuner-types.c Fri Apr 24 01:46:41 2009 -0300 +++ b/linux/drivers/media/common/tuners/tuner-types.c Tue Apr 28 03:35:42 2009 +1000 @@ -558,8 +558,8 @@ static struct tuner_range tuner_fm1216me_mk3_pal_ranges[] = { { 16 * 158.00 /*MHz*/, 0x8e, 0x01, }, - { 16 * 442.00 /*MHz*/, 0x8e, 0x02, }, - { 16 * 999.99 , 0x8e, 0x04, }, + { 16 * 441.00 /*MHz*/, 0x8e, 0x02, }, + { 16 * 864.00 , 0x8e, 0x04, }, }; static struct tuner_params tuner_fm1216me_mk3_params[] = { Signed-off-by: Beholder Intl. Ltd. Dmitry Belimov <d.belimov@xxxxxxxxx>