[libcamera-devel,v3,24/27] gst: libcamerasrc: Add a TODO comment

Message ID 20200306202637.525587-25-nicolas@ndufresne.ca
State Accepted
Headers show
Series
  • GStreamer Element for libcamera
Related show

Commit Message

Nicolas Dufresne March 6, 2020, 8:26 p.m. UTC
From: Nicolas Dufresne <nicolas.dufresne@collabora.com>

This is to guide upcoming contributors toward what is left to do to get
toward a production ready element.

Signed-off-by: Nicolas Dufresne <nicolas.dufresne@collabora.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 src/gstreamer/gstlibcamerasrc.cpp | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

Patch

diff --git a/src/gstreamer/gstlibcamerasrc.cpp b/src/gstreamer/gstlibcamerasrc.cpp
index b04e9f1..8d0d0e1 100644
--- a/src/gstreamer/gstlibcamerasrc.cpp
+++ b/src/gstreamer/gstlibcamerasrc.cpp
@@ -7,6 +7,25 @@ 
  */
 
 /**
+ * \todo The following is a list of items that needs implementation in the GStreamer plugin
+ *  - Implement GstElement::send_event
+ *    + Allowing application to send EOS
+ *    + Allowing application to use FLUSH/FLUSH_STOP
+ *    + Prevent the main thread from accessing streaming thread
+ *  - Implement renegotiation (even if slow)
+ *  - Implement GstElement::request-new-pad (multi stream)
+ *    + Evaluate if a single streaming thread is fine
+ *  - Add application driven request (snapshot)
+ *  - Add framerate control
+ *  - Add buffer importation support
+ *
+ *  Requires new libcamera API:
+ *  - Add framerate negotiation support
+ *  - Add colorimetry support
+ *  - Add timestamp support
+ *  - Use unique names to select the camera devices
+ *  - Add GstVideoMeta support (strides and offsets)
+ *
  * \todo libcamera UVC drivers picks the lowest possible resolution first, this
  * should be fixed so that we get a decent resolution and framerate for the
  * role by default.