Opened 12 years ago

Closed 11 years ago

Last modified 11 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 12 years ago.

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by anonymous

Attachment: vsync.diff added

comment:1 Changed 12 years ago by laga

Component: mythtvVideo Playback
Keywords: easy added

comment:2 Changed 11 years ago by markk

Resolution: fixed
Status: newclosed

(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 Changed 11 years ago by thomas@…

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

Note: See TracTickets for help on using tickets.