Remove some unneeded double-spaces.

[libsigrok.git] / src / sw_limits.c

diff --git a/src/sw_limits.c b/src/sw_limits.c
index eb1e5c39a00abca1010a0264346004291285c75b..bea14692749a679f6be57c0cf2451b769ac4a856 100644 (file)
--- a/src/sw_limits.c
+++ b/src/sw_limits.c
@@ -17,6 +17,12 @@
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
+/**
+ * @file
+ * Software limits helper functions
+ * @internal
+ */
+
 #include <config.h>
 #include <stdio.h>
 #include <stdint.h>
@@ -25,12 +31,34 @@
 #include <libsigrok/libsigrok.h>
 #include "libsigrok-internal.h"
 
+#define LOG_PREFIX "sw_limits"
+
+/**
+ * Initialize a software limit instance
+ *
+ * Must be called before any other operations are performed on a struct
+ * sr_sw_limits and should typically be called after the data structure has been
+ * allocated.
+ *
+ * @param limits the software limit instance to initialize
+ */
 SR_PRIV void sr_sw_limits_init(struct sr_sw_limits *limits)
 {
        limits->limit_samples = 0;
        limits->limit_msec = 0;
 }
 
+/**
+ * Get software limit configuration
+ *
+ * Retrieve the currently configured software limit for the specified key.
+ * Should be called from the drivers config_get() callback.
+ *
+ * @param limits software limit instance
+ * @param key config item key
+ * @param data config item data
+ * @return SR_ERR_NA if @p key is not a supported limit, SR_OK otherwise
+ */
 SR_PRIV int sr_sw_limits_config_get(struct sr_sw_limits *limits, uint32_t key,
        GVariant **data)
 {
@@ -48,6 +76,17 @@ SR_PRIV int sr_sw_limits_config_get(struct sr_sw_limits *limits, uint32_t key,
        return SR_OK;
 }
 
+/**
+ * Set software limit configuration
+ *
+ * Configure software limit for the specified key. Should be called from the
+ * drivers config_set() callback.
+ *
+ * @param limits software limit instance
+ * @param key config item key
+ * @param data config item data
+ * @return SR_ERR_NA if @p key is not a supported limit, SR_OK otherwise
+ */
 SR_PRIV int sr_sw_limits_config_set(struct sr_sw_limits *limits, uint32_t key,
        GVariant *data)
 {
@@ -65,30 +104,65 @@ SR_PRIV int sr_sw_limits_config_set(struct sr_sw_limits *limits, uint32_t key,
        return SR_OK;
 }
 
+/**
+ * Start a new data acquisition session
+ *
+ * Resets the internal accounting for all software limits. Usually should be
+ * called from the drivers acquisition_start() callback.
+ *
+ * @param limits software limits instance
+ */
 SR_PRIV void sr_sw_limits_acquisition_start(struct sr_sw_limits *limits)
 {
        limits->samples_read = 0;
        limits->start_time = g_get_monotonic_time();
 }
 
+/**
+ * Check if any of the configured software limits has been reached
+ *
+ * Usually should be called at the end of the drivers work function after all
+ * processing has been done.
+ *
+ * @param limits software limits instance
+ * @returns TRUE if any of the software limits has been reached and the driver
+ *               should stop data acquisition, otherwise FALSE.
+ */
 SR_PRIV gboolean sr_sw_limits_check(struct sr_sw_limits *limits)
 {
        if (limits->limit_samples) {
-               if (limits->samples_read >= limits->limit_samples)
+               if (limits->samples_read >= limits->limit_samples) {
+                       sr_dbg("Requested number of samples (%" PRIu64
+                              ") reached.", limits->limit_samples);
                        return TRUE;
+               }
        }
 
        if (limits->limit_msec) {
                guint64 now;
                now = g_get_monotonic_time();
                if (now > limits->start_time &&
-                       now - limits->start_time > limits->limit_msec)
+                       now - limits->start_time > limits->limit_msec) {
+                       sr_dbg("Requested sampling time (%" PRIu64
+                              "ms) reached.", limits->limit_msec / 1000);
                        return TRUE;
+               }
        }
 
        return FALSE;
 }
 
+/**
+ * Update the amount samples that have been read
+ *
+ * Update the amount of samples that have been read in the current data
+ * acquisition run. For each invocation @p samples_read will be accumulated and
+ * once the configured sample limit has been reached sr_sw_limits_check() will
+ * return TRUE.
+ *
+ * @param limits software limits instance
+ * @param samples_read the amount of samples that have been read
+ */
 SR_PRIV void sr_sw_limits_update_samples_read(struct sr_sw_limits *limits,
        uint64_t samples_read)
 {