[linux-dvb] [PATCH] Frontend or51211 - fixes SNR and signal
rustys at ieee.org
Wed Nov 22 18:17:22 CET 2006
On Mon, 2006-11-20 at 13:42 -0800, Trent Piepho wrote:
> On Sat, 18 Nov 2006, Rusty Scott wrote:
> > The attached patch makes changes to the or51211 frontend to match the SNR and
> > signal strength methods in the lgdt330x frontend. It removes an embedded log
> > function in favor of using the functions in dvb_math. Do not confuse this patch
> > with the patch just posted for the or51132 frontend. I'm not sure why 'make
> I've fixed up the patches a bit and put them into my repository,
> http://linuxtv.org/hg/~tap/v4l-dvb Does everything still look ok?
> I've tested the or51132 patch and it seems to work ok with 8-vsb. I
> haven't been able to test the or51211 patch. I'm not sure how much testing
> it got before, as it didn't compile.
Odd, it compiled and worked fine on the pcHDTV HD2000 card I used to
test it. I will grab your repo and test it to make sure but it probably
won't be until the weekend. (Thanksgiving and all that.) Do you
remember what the compile issue was?
> I'd like to verify a couple things. For the or51211, the error register is
> only 8 bits? In the or51132 it's 16 bits.
Yes, the or51211 is only an 8 bit value where the or51132 is 16 bits
> The or51211 patch removed a msleep(3) that appears between most of the i2c
> transactions. Is this ok?
> The or51132 patch removed some msleep(30) calls too. I added those back in
> since the rest of the code had them. Is there anything in the datasheet to
> suggest that these are necessary? i.e. after selecting a register you must
> wait 30 ms before trying to read it? I wonder if all those msleeps() can
> be removed.
I couldn't find anything in either datasheet suggesting they were
necessary. I removed them as a test and when everything continued to
work I left them out of the patch. I don't see any reason to leave them
in, but that should probably be a separate patch.
More information about the linux-dvb