Opened 18 years ago

Closed 17 years ago

Last modified 17 years ago

#4637 closed patch (fixed)

Allow a little leeway before rejecting double rate deinterlacers (bob/greedyh/yadif)

Reported by: mark.kendall@… Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: Video Playback Version: unknown
Severity: medium Keywords: easy
Cc: Ticket locked: no

Description

In summary, the nvidia drivers increase the htotal of the accepted modeline to ensure it is divisible by 8. This changes the calculated refresh rate and can lower it below the 2* framerate threshold needed for bob etc.

The attached patch adds a little tolerance to the accept/reject decision to ensure the best deinterlacing options can still be used.

I'm seeing this on several frontends where the EDID reported timings for 50Hz modelines give a refresh rate of 50Hz exactly, but by the time the drivers have adjusted the modeline the refresh rate drops to circa 49.9Hz.

Attachments (1)

vsync.diff​ (1.1 KB ) - added by anonymous 18 years ago.

Download all attachments as: .zip

Change History (4)

by anonymous, 18 years ago

Attachment: vsync.diff​ added

comment:1 by laga, 17 years ago

Component: mythtv → Video Playback
Keywords: easy added

comment:2 by markk, 17 years ago

Resolution: → fixed
Status: new → closed

(In [19287]) Allow a small amount of tolerance before rejecting double rate deinterlacers due to low refresh rates. This ensures bobdeint etc will still work when the video driver (specifically the nvidia proprietary driver) adjusts the htotal to ensure it is divisble by 8. Closes #4637

comment:3 by thomas@…, 17 years ago

I applied [19287] to 0.21 fixes and it works there, too. Please backport.

Note: See TracTickets for help on using tickets.