From c9c94390c520ad496082222858c0f1285f1432df Mon Sep 17 00:00:00 2001
From: Matthias Vogelgesang <matthias.vogelgesang@kit.edu>
Date: Fri, 3 Feb 2012 11:09:54 +0100
Subject: Callback function have to notify caller about buffer

If they return a UCA_BUFFER_KEEP they have to call uca_cam_release_buffer() in
the future. If returning UCA_BUFFER_RELEASE, this is done by the caller.
---
 src/uca-cam.h |  1 +
 src/uca.c     |  7 +++++++
 src/uca.h     | 25 ++++++++++++++++++++++---
 3 files changed, 30 insertions(+), 3 deletions(-)

(limited to 'src')

diff --git a/src/uca-cam.h b/src/uca-cam.h
index cba0305..08b7941 100644
--- a/src/uca-cam.h
+++ b/src/uca-cam.h
@@ -63,6 +63,7 @@ typedef struct uca_camera_priv {
     uint32_t (*stop_recording) (struct uca_camera_priv *cam);
     uint32_t (*trigger) (struct uca_camera_priv *cam);
     uint32_t (*register_callback) (struct uca_camera_priv *cam, uca_cam_grab_callback callback, void *user);
+    uint32_t (*release_buffer) (struct uca_camera_priv *cam, void *buffer);
     uint32_t (*grab) (struct uca_camera_priv *cam, char *buffer, void *meta_data);
     uint32_t (*readout) (struct uca_camera_priv *cam);
 
diff --git a/src/uca.c b/src/uca.c
index 4a23e69..1d07ea4 100644
--- a/src/uca.c
+++ b/src/uca.c
@@ -341,6 +341,13 @@ uint32_t uca_cam_register_callback(struct uca_camera *cam, uca_cam_grab_callback
     return priv->register_callback(priv, callback, user);
 }
 
+uint32_t uca_cam_release_buffer(struct uca_camera *cam, void *buffer)
+{
+    struct uca_camera_priv *priv = cam->priv;
+    if (priv->release_buffer != NULL)
+        return priv->release_buffer(priv, buffer);
+}
+
 uint32_t uca_cam_grab(struct uca_camera *cam, char *buffer, void *meta_data)
 {
     struct uca_camera_priv *priv = cam->priv;
diff --git a/src/uca.h b/src/uca.h
index 5baf472..6721e58 100644
--- a/src/uca.h
+++ b/src/uca.h
@@ -216,6 +216,15 @@ enum uca_cam_state {
     UCA_CAM_READOUT         /**< Camera recorded and is currently in readout mode */
 };
 
+/**
+ * Specify if the callback function keeps the buffer and will call
+ * ufo_cam_release_buffer() at later time or if after returning the buffer can
+ * be released automatically.
+ */
+enum uca_buffer_status {
+    UCA_BUFFER_KEEP,
+    UCA_BUFFER_RELEASE
+};
 
 /**
  * A uca_property_t describes a vendor-independent property used by cameras and
@@ -253,14 +262,14 @@ union uca_value {
  * receive data as soon as it is delivered.
  *
  * \param[in] image_number Current frame number
- * \param[in] buffer Image data
+ * \param[in] buffer Image data.
  * \param[in] meta_data Meta data provided by the camera specifying per-frame
  *   data.
  * \param[in] user User data registered in uca_cam_register_callback()
  *
  * \note The meta data parameter is not yet specified but just a place holder.
  */
-typedef void (*uca_cam_grab_callback) (uint64_t image_number, void *buffer, void *meta_data, void *user);
+typedef enum uca_buffer_status (*uca_cam_grab_callback) (uint64_t image_number, void *buffer, void *meta_data, void *user);
 
 extern const char *uca_unit_map[];      /**< maps unit numbers to corresponding strings */
 
@@ -454,7 +463,6 @@ uint32_t uca_cam_stop_recording(struct uca_camera *cam);
  */
 uint32_t uca_cam_trigger(struct uca_camera *cam);
 
-
 /**
  * Register callback for given frame grabber. To actually start receiving
  * frames, call uca_grabber_acquire().
@@ -466,6 +474,17 @@ uint32_t uca_cam_trigger(struct uca_camera *cam);
  */
 uint32_t uca_cam_register_callback(struct uca_camera *cam, uca_cam_grab_callback callback, void *user);
 
+/**
+ * Release the buffer that was given in the grab callback.
+ *
+ * \param[in] cam A uca_camera object
+ * \param[in] buffer The buffer that was passed to the callback
+ *
+ * \see uca_cam_register_callback(), uca_cam_grab_callback
+ * \since 0.5
+ */
+uint32_t uca_cam_release_buffer(struct uca_camera *cam, void *buffer);
+
 /**
  * \brief Grab one image from the camera
  * 
-- 
cgit v1.2.3