[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-23 Thread Holger Waechtler
Johannes Stezenbach wrote: Felix Domke wrote: Johannes Stezenbach wrote: The V4 API has: unsigned int frame_rate; /* in frames per 1000sec */ I'm against this. The reason is that arbitrary framerates are not allowed in MPEG-2 (correct me if i'm wrong, i'm currently too lazy to read

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-23 Thread Andreas Oberritter
On Tue, 2003-12-23 at 14:12, Holger Waechtler wrote: the nom/denom notation is in opposite to the frames/msec notation always correct and can display all usual framerates without rounding - so I'd vote for this one. Do you know any mpeg decoder which offers these values instead of the few

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-23 Thread Johannes Stezenbach
Andreas Oberritter wrote: On Tue, 2003-12-23 at 14:12, Holger Waechtler wrote: the nom/denom notation is in opposite to the frames/msec notation always correct and can display all usual framerates without rounding - so I'd vote for this one. Do you know any mpeg decoder which offers

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-23 Thread Johannes Stezenbach
Holger Waechtler wrote: the nom/denom notation is in opposite to the frames/msec notation always correct and can display all usual framerates without rounding - so I'd vote for this one. It's frames / (1000 seconds). And I think it's accurate enough. AFAIK this value is not used in the

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-22 Thread Holger Waechtler
Carsten Juttner wrote: Felix Domke wrote: Johannes Stezenbach wrote: The V4 API has: unsigned int frame_rate; /* in frames per 1000sec */ I'm against this. The reason is that arbitrary framerates are not allowed in MPEG-2 (correct me if i'm wrong, i'm currently too lazy to read 13818-2

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-22 Thread Johannes Stezenbach
Felix Domke wrote: Johannes Stezenbach wrote: The V4 API has: unsigned int frame_rate; /* in frames per 1000sec */ I'm against this. The reason is that arbitrary framerates are not allowed in MPEG-2 (correct me if i'm wrong, i'm currently too lazy to read 13818-2 again...),

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-21 Thread Johannes Stezenbach
On Sat, Dec 20, 2003 at 12:15:25AM +0100, Andreas Oberritter wrote: I'd like to add this patch to dvb-kernel's linux/dvb/video.h, so userspace programs can adjust video system settings (PAL/NTSC) depending on the frame rate and video size. I think it won't break binary compatibility. The

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-21 Thread Felix Domke
Johannes Stezenbach wrote: The V4 API has: unsigned int frame_rate; /* in frames per 1000sec */ I'm against this. The reason is that arbitrary framerates are not allowed in MPEG-2 (correct me if i'm wrong, i'm currently too lazy to read 13818-2 again...), and there's no hardware being

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-20 Thread Marcus Metzler
Andreas Oberritter writes: [1 text/plain] On Sat, 2003-12-20 at 01:06, Marcus Metzler wrote: I'd like to add this patch to dvb-kernel's linux/dvb/video.h, so userspace programs can adjust video system settings (PAL/NTSC) depending on the frame rate and video size. I think it

[linux-dvb] Re: [PATCH] new video event and ioctl

2003-12-19 Thread Andreas Oberritter
On Sat, 2003-12-20 at 01:06, Marcus Metzler wrote: I'd like to add this patch to dvb-kernel's linux/dvb/video.h, so userspace programs can adjust video system settings (PAL/NTSC) depending on the frame rate and video size. I think it won't break binary compatibility. The frame rate