matthieu castet wrote:
>
> > in 2.6.16 the code in driver/media/video/bttv-vbi.c was changed
> > a little bit. Beside other things, the constant 244 for the vbi
> > offset was replaced by a #define VBI_OFFSET 128.
> >
> > Afaics, the old value 244 was correct - was the change to 128
> > intentionally?
>
> You can have some comments about that in the git log :
> http://www.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=67f1570a0659abba5efbf55cc986187af61bdd52;hp=7e57819169d4f9a1d7af55fb645ece3fb981e2e3
Hmmm... regarding VBI_OFFSET there's this:
| - V4L2_(G|S|TRY)_FMT returned incorrect VBI start lines for PAL-M,
| NTSC-JP, and PAL-60. They also returned an inaccurate VBI offset.
I remember that I tried to figure out how to calculate the VBI offset
from the Bt8xx specs a couple years ago but resigned - somehow the
specs are wrong. But given a teletext signal and the teletext specs
you can measure its value[1] and 244 is pretty accurate for PAL.
Regarding the 64 bit arithmetic there's this:
| - V4L2_(S|TRY)_FMT did not expect very large or small VBI start or
| count values, returning wrong (but safe) counts due to an overflow.
Wow, previously the driver produced (safe) garbage when given garbage
and now it produces more accurate (safe) garbage??? I don't get it.
[My suspicion is that it was only inserted to shut up warnings...]
Ciao, ET.
[1] You know that the 7th peak of the teletext clock run-in is at
12us (+0.4us/-1.0us) from the falling hsync. You look at which
sample the 7th peak is at its max. The difference between this
number and 12e-6 * Fs (426 for Fs=35468950) is the VBI_OFFSET.
Btw, hsync ends at 166 (PAL/Fs=35468950) respectively 134 (NTSC/
Fs=28636363). So any VBI_OFFSET lower than that would show the
hsync in the samples...
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
