Hi Hans, Am Freitag, den 24.04.2015, 08:43 +0200 schrieb Hans Verkuil: > Hi Philipp, > > Thank you for the patch, but I have one question: > > On 04/23/2015 03:52 PM, Philipp Zabel wrote: > > Use the VIVID_WEBCAM_SIZES constant where appropriate and add a 1920x1080 pixel > > frame size setting with frame rates of 2 fps and 5 fps. > > Why add both 2 and 5 fps? Is there a reason why you want both of those fps values? > > Just wondering. I just wanted to quickly test 1080p at 5 fps, so I didn't change that afterwards for the patch. 5 fps also seems like the next logical step. webcam_intervals needs to be twice the size of webcam_sizes, the comment above webcam_intervals told me to add two intervals for every new element in webcam_sizes. For the second interval, I didn't think much about the actual value. Choosing 2 fps next was probably influenced by our monetary system. To keep data rates similar to 720p at 10 fps and 15 fps, 1080p at 4 fps and 6 fps, respectively, would be the better choice. regards Philipp -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
