GNOME Bugzilla – Bug 752825
v4l2: Patch to remove limit on framerate
Last modified: 2015-08-16 13:40:45 UTC
Created attachment 308079 [details] [review] Patch to remove limit on v4l2src framerate The current implementation of v4l2 has a hard-coded limit of 100 fps for v4l2 devices. This limits certain applications that may require an even higher framerate and it also limits cameras with hardware cropping being applied to them, which can also lead to framerates higher than 100. This patch remedies this issue. It first sets the maximum possible framerate to G_MAXINT/1 to ensure that any future application needing higher framerates is not capped. This patch also changes the default framerate so it is set to 100, as it would be for all applications that do not set a framerate before this patch.
Review of attachment 308079 [details] [review]: Indeed, this limit was totally arbitrary. Keeping it in the fixate function is probably a good idea for the default case. So I think this patch should go in.
Comment on attachment 308079 [details] [review] Patch to remove limit on v4l2src framerate Thanks! Please notice the commit message format for further submissions. Author: Dimitrios Katsaros <patcherwork@gmail.com> Date: Fri Jul 24 15:10:05 2015 +0200 v4l2: Allow framerate to be large then 100pfs This limit was arbitrary. We still fixate near 100pfs for compatibility. https://bugzilla.gnome.org/show_bug.cgi?id=752825
Duly noted, I will keep the commit format in mind in the future. Thanks for the quick response!