[linux-dvb] Re: DIB3000MC autosearch

[Mario Rossi]

There are 2 different things:

1) the value of seq:
old driver used a table. Full autosearch (i.e. INVERSION_AUTO,
TRANSMISSION_AUTO, GUARD_AUTO) = 11. No autosearch = 0

new driver. Autosearch = 7 (7 is not present in the old table), No
autosearch = 0.
(In the old table, autosearch for TRANSMISSION & GUARD corresponds to 6)

2) how seq is written to the register 5
OLD DRIVER : (seq & 0x0f) << 4) | (1 << 8)
line 286 here http://linuxtv.org/hg/v4l-dvb?f=c8f23ba912d8;file=linux/drivers/media/dvb/frontends/dib3000mc.c;style=gitweb

check macro DIB3000MC_SEQ_TPS(seq,1)
line 71 here
http://linuxtv.org/hg/v4l-dvb?f=1bd99532ee0f;file=linux/drivers/media/dvb/frontends/dib3000mc_priv.h;style=gitweb

NEW DRIVER : seq directly
line 502 in the current version of dib3000mc.c

As you said the value 11 does inversion search as well (which might be
not needed), but I think the important issue is the second.

Hope I've explained in a decent way.

I have a question: if the board can do autotune, why do we bother with
setting the right parameters?

Regards

On 11/6/06, Patrick Boettcher <patrick.boettcher at desy.de> wrote:
> On Thu, 2 Nov 2006, Mario Rossi wrote:
>
> > Hi Patrick,
> > did you have time to check the patch I sent recently that fixes the
> > autosearch for dib3000mc in my case?
> >
> > http://www.linuxtv.org/pipermail/linux-dvb/2006-October/013907.html
> >
> > Actually the following one (100% equivalent to the one above) is more
> > in line with the treatment of seq in the old driver and what happens
> > in the dib7000 code.
>
> Please give me some time to understand the solution. I think only one of
> the two things should fix it (putting seq to 11 also searches for
> inversion which is not necessary).
>
> Anyway, thanks for digging out this thing.
>
> Patrick.