This series adds support for fractions in the control framework, and a way to obtain the min and max values of compound controls such as v4l2_fract. Next it adds the V4L2_CID_MPEG_VIDEO_ENC_FRAME_RATE control to set the framerate for the encoder. The next patch adds support for the V4L2_BUF_FLAG_TOO_SMALL flag to signal that the capture buffer was too small. The final patch adds the encoder spec (unchanged from v3). Michael, can you add support for V4L2_CID_MPEG_VIDEO_ENC_FRAME_RATE to your encoder driver? Let me know if something isn't working. I need to add a test control for this to vivid as well and add support for this to v4l2-ctl, that's on my TODO list. Open questions: 1) Existing encoder drivers use S_PARM to signal the frameperiod, but as discussed in Lyon this should be the rate at which frames are submitted for encoding, which can be different from V4L2_CID_MPEG_VIDEO_ENC_FRAME_RATE. But given the current use-cases I was wondering if calling S_PARM should set the ENC_FRAME_RATE control as well, and you would need to explicitly set the control after S_PARM if the two are not the same. This would mean that existing applications that don't know about the control can keep working. 2) I do believe that we need a RELEASE/DEL/DESTROY_BUFS ioctl in order to do proper handling of the V4L2_BUF_FLAG_TOO_SMALL case. I am inclined to postpone adding this flag until we have that ioctl. We can still merge the stateful encoder spec if we mention that that is work in progress. Regards, Hans Hans Verkuil (4): v4l2-ctrls: add support for v4l2_fract types v4l2-ctrls: add support for V4L2_CTRL_WHICH_MIN/MAX_VAL v4l2-controls.h: add V4L2_CID_MPEG_VIDEO_ENC_FRAME_RATE videodev2.h: add V4L2_BUF_FLAG_TOO_SMALL flag Tomasz Figa (1): media: docs-rst: Document memory-to-memory video encoder interface Documentation/media/uapi/v4l/buffer.rst | 9 + Documentation/media/uapi/v4l/dev-encoder.rst | 608 ++++++++++++++++++ Documentation/media/uapi/v4l/dev-mem2mem.rst | 1 + .../media/uapi/v4l/ext-ctrls-codec.rst | 8 + Documentation/media/uapi/v4l/pixfmt-v4l2.rst | 5 + Documentation/media/uapi/v4l/v4l2.rst | 2 + .../media/uapi/v4l/vidioc-encoder-cmd.rst | 51 +- .../media/uapi/v4l/vidioc-g-ext-ctrls.rst | 15 +- .../media/uapi/v4l/vidioc-queryctrl.rst | 6 + .../media/videodev2.h.rst.exceptions | 3 + .../media/common/videobuf2/videobuf2-core.c | 12 +- .../media/common/videobuf2/videobuf2-v4l2.c | 4 + drivers/media/i2c/imx214.c | 4 +- drivers/media/v4l2-core/v4l2-ctrls.c | 222 ++++++- include/media/v4l2-ctrls.h | 72 ++- include/media/videobuf2-core.h | 4 + include/uapi/linux/v4l2-controls.h | 1 + include/uapi/linux/videodev2.h | 6 + 18 files changed, 980 insertions(+), 53 deletions(-) create mode 100644 Documentation/media/uapi/v4l/dev-encoder.rst -- 2.23.0
