[linux-dvb] PATCH: New driver for Zarlink ZL10313 based Compro S350/S300

[Tino Reichardt]

* "Jan D. Louw" <jd.louw at mweb.co.za> wrote:
> 
> >> * "Jan D. Louw" <jd.louw at mweb.co.za> wrote:
> > Hi Tino,
> > 
> > I had a quick look at the zl10036 datasheet and I can't see anything 
> > obviously wrong. The formulas look OK. There is one wrong assignment:
> > 
> > if (!priv->has_loopthrough) buf[0] |= 0x01;

will be fixed

> > 
> > and some wrong comments:
> > 
> >         buf[6] = 0xd3;   /*  6: from datasheet */
> >         buf[7] = 0x40;   /*  7: from datasheet */

I will fix this... - the comment was copy'n paste :(

> > 
> > but this shouldn't cause it not to work. I suspect this is some obscure 
> > writing order bug. The zl10038 datasheet does provide more info on 
> > registers 8-10 which might prove useful.
> > 
> 
> Also, maybe some hardware problem (like pin SLEEP asserted) or wrong 
> crystal freq?

The crystal freq is okay. I will look @ the SLEEP PIN issue, maybe thats
the fault. Also Matthias Schwarzott has done some work on the patches,
maybe he has found the thing already ?


Thanks for the fast response ;)

-- 
regards, TR