]> sigrok.org Git - sigrok-test.git/blobdiff - decoder/runtc.c
runtc: minor nits in usage(), add missing options
[sigrok-test.git] / decoder / runtc.c
index 364e43f284e89bb4f6e2de5081ea15fe49796f64..3d0bfd8f0c845b2ab2104e979fd3d1b18420fdbb 100644 (file)
 #include <sched.h>
 #endif
 
-int debug = FALSE;
-int statistics = FALSE;
-char *coverage_report;
+static int debug = FALSE;
+static int statistics = FALSE;
+static char *coverage_report;
+static struct sr_context *ctx;
 
 struct channel {
        char *name;
@@ -52,17 +53,17 @@ struct option {
 };
 
 struct pd {
-       char *name;
+       const char *name;
        GSList *channels;
        GSList *options;
 };
 
 struct output {
-       char *pd;
+       const char *pd;
        int type;
-       char *class;
+       const char *class;
        int class_idx;
-       char *outfile;
+       const char *outfile;
        int outfd;
 };
 
@@ -73,12 +74,12 @@ struct cvg {
        GSList *missed_lines;
 };
 
-struct cvg *get_mod_cov(PyObject *py_cov, char *module_name);
-void cvg_add(struct cvg *dst, struct cvg *src);
-struct cvg *cvg_new(void);
-gboolean find_missed_line(struct cvg *cvg, char *linespec);
+static struct cvg *get_mod_cov(PyObject *py_cov, const char *module_name);
+static void cvg_add(struct cvg *dst, const struct cvg *src);
+static struct cvg *cvg_new(void);
+static gboolean find_missed_line(struct cvg *cvg, const char *linespec);
 
-static void logmsg(char *prefix, FILE *out, const char *format, va_list args)
+static void logmsg(const char *prefix, FILE *out, const char *format, va_list args)
 {
        if (prefix)
                fprintf(out, "%s", prefix);
@@ -130,27 +131,30 @@ static int srd_log(void *cb_data, int loglevel, const char *format, va_list args
        return SRD_OK;
 }
 
-static void usage(char *msg)
+static void usage(const char *msg)
 {
        if (msg)
                fprintf(stderr, "%s\n", msg);
 
-       printf("Usage: runtc [-dPpoiOf]\n");
-       printf("  -d  Debug\n");
-       printf("  -P  <protocol decoder>\n");
-       printf("  -p  <channelname=channelnum> (optional)\n");
-       printf("  -o  <channeloption=value> (optional)\n");
+       printf("Usage: runtc [-dPpoiOfcS]\n");
+       printf("  -d  (enables debug output)\n");
+       printf("  -P <protocol decoder>\n");
+       printf("  -p <channelname=channelnum> (optional)\n");
+       printf("  -o <channeloption=value> (optional)\n");
        printf("  -i <input file>\n");
        printf("  -O <output-pd:output-type[:output-class]>\n");
        printf("  -f <output file> (optional)\n");
        printf("  -c <coverage report> (optional)\n");
+       printf("  -S  (enables statistics)\n");
        exit(msg ? 1 : 0);
 
 }
 
-/* This is a neutered version of libsigrokdecode's py_str_as_str(). It
+/*
+ * This is a neutered version of libsigrokdecode's py_str_as_str(). It
  * does no error checking, but then the only strings it processes are
- * generated by Python's repr(), so are known good. */
+ * generated by Python's repr(), so are known good.
+ */
 static char *py_str_as_str(const PyObject *py_str)
 {
        PyObject *py_encstr;
@@ -171,12 +175,12 @@ static void srd_cb_py(struct srd_proto_data *pdata, void *cb_data)
        GString *out;
        char *s;
 
-       DBG("Python output from %s", pdata->pdo->di->inst_id);
+       DBG("Python output from %s", pdata->pdo->di->decoder->id);
        op = cb_data;
        pydata = pdata->data;
        DBG("ptr %p", pydata);
 
-       if (strcmp(pdata->pdo->di->inst_id, op->pd))
+       if (strcmp(pdata->pdo->di->decoder->id, op->pd))
                /* This is not the PD selected for output. */
                return;
 
@@ -191,7 +195,7 @@ static void srd_cb_py(struct srd_proto_data *pdata, void *cb_data)
        out = g_string_sized_new(128);
        g_string_printf(out, "%" PRIu64 "-%" PRIu64 " %s: %s\n",
                        pdata->start_sample, pdata->end_sample,
-                       pdata->pdo->di->inst_id, s);
+                       pdata->pdo->di->decoder->id, s);
        g_free(s);
        if (write(op->outfd, out->str, out->len) == -1)
                ERR("SRD_OUTPUT_PYTHON callback write failure!");
@@ -207,11 +211,11 @@ static void srd_cb_bin(struct srd_proto_data *pdata, void *cb_data)
        GString *out;
        unsigned int i;
 
-       DBG("Binary output from %s", pdata->pdo->di->inst_id);
+       DBG("Binary output from %s", pdata->pdo->di->decoder->id);
        op = cb_data;
        pdb = pdata->data;
 
-       if (strcmp(pdata->pdo->di->inst_id, op->pd))
+       if (strcmp(pdata->pdo->di->decoder->id, op->pd))
                /* This is not the PD selected for output. */
                return;
 
@@ -225,7 +229,7 @@ static void srd_cb_bin(struct srd_proto_data *pdata, void *cb_data)
        out = g_string_sized_new(128);
        g_string_printf(out, "%" PRIu64 "-%" PRIu64 " %s:",
                        pdata->start_sample, pdata->end_sample,
-                       pdata->pdo->di->inst_id);
+                       pdata->pdo->di->decoder->id);
        for (i = 0; i < pdb->size; i++) {
                g_string_append_printf(out, " %.2x", pdb->data[i]);
        }
@@ -244,26 +248,26 @@ static void srd_cb_ann(struct srd_proto_data *pdata, void *cb_data)
        int i;
        char **dec_ann;
 
-       DBG("Annotation output from %s", pdata->pdo->di->inst_id);
+       DBG("Annotation output from %s", pdata->pdo->di->decoder->id);
        op = cb_data;
        pda = pdata->data;
        dec = pdata->pdo->di->decoder;
-       if (strcmp(pdata->pdo->di->inst_id, op->pd))
+       if (strcmp(pdata->pdo->di->decoder->id, op->pd))
                /* This is not the PD selected for output. */
                return;
 
-       if (op->class_idx != -1 && op->class_idx != pda->ann_format)
+       if (op->class_idx != -1 && op->class_idx != pda->ann_class)
                /*
                 * This output takes a specific annotation class,
                 * but not the one that just came in.
                 */
                return;
 
-       dec_ann = g_slist_nth_data(dec->annotations, pda->ann_format);
+       dec_ann = g_slist_nth_data(dec->annotations, pda->ann_class);
        line = g_string_sized_new(256);
        g_string_printf(line, "%" PRIu64 "-%" PRIu64 " %s: %s:",
                        pdata->start_sample, pdata->end_sample,
-                       pdata->pdo->di->inst_id, dec_ann[0]);
+                       pdata->pdo->di->decoder->id, dec_ann[0]);
        for (i = 0; pda->ann_text[i]; i++)
                g_string_append_printf(line, " \"%s\"", pda->ann_text[i]);
        g_string_append(line, "\n");
@@ -276,19 +280,22 @@ static void srd_cb_ann(struct srd_proto_data *pdata, void *cb_data)
 static void sr_cb(const struct sr_dev_inst *sdi,
                const struct sr_datafeed_packet *packet, void *cb_data)
 {
+       static int samplecnt = 0;
        const struct sr_datafeed_logic *logic;
        struct srd_session *sess;
        GVariant *gvar;
        uint64_t samplerate;
        int num_samples;
-       static int samplecnt = 0;
+       struct sr_dev_driver *driver;
 
        sess = cb_data;
 
+       driver = sr_dev_inst_driver_get(sdi);
+
        switch (packet->type) {
        case SR_DF_HEADER:
                DBG("Received SR_DF_HEADER");
-               if (sr_config_get(sdi->driver, sdi, NULL, SR_CONF_SAMPLERATE,
+               if (sr_config_get(driver, sdi, NULL, SR_CONF_SAMPLERATE,
                                &gvar) != SR_OK) {
                        ERR("Getting samplerate failed");
                        break;
@@ -308,10 +315,11 @@ static void sr_cb(const struct sr_dev_inst *sdi,
        case SR_DF_LOGIC:
                logic = packet->payload;
                num_samples = logic->length / logic->unitsize;
-               DBG("Received SR_DF_LOGIC: %d samples", num_samples);
+               DBG("Received SR_DF_LOGIC (%"PRIu64" bytes, unitsize = %d).",
+                       logic->length, logic->unitsize);
                srd_session_send(sess, samplecnt, samplecnt + num_samples,
-                               logic->data, logic->length);
-               samplecnt += logic->length / logic->unitsize;
+                               logic->data, logic->length, logic->unitsize);
+               samplecnt += num_samples;
                break;
        case SR_DF_END:
                DBG("Received SR_DF_END");
@@ -320,7 +328,7 @@ static void sr_cb(const struct sr_dev_inst *sdi,
 
 }
 
-static int run_testcase(char *infile, GSList *pdlist, struct output *op)
+static int run_testcase(const char *infile, GSList *pdlist, struct output *op)
 {
        struct srd_session *sess;
        struct srd_decoder *dec;
@@ -331,23 +339,27 @@ static int run_testcase(char *infile, GSList *pdlist, struct output *op)
        struct option *option;
        GVariant *gvar;
        GHashTable *channels, *opts;
-       GSList *pdl, *l;
-       int idx;
+       GSList *pdl, *l, *devices;
+       int idx, i;
        int max_channel;
        char **decoder_class;
        struct sr_session *sr_sess;
+       gboolean is_number;
+       const char *s;
 
        if (op->outfile) {
                if ((op->outfd = open(op->outfile, O_CREAT|O_WRONLY, 0600)) == -1) {
                        ERR("Unable to open %s for writing: %s", op->outfile,
-                                       strerror(errno));
+                                       g_strerror(errno));
                        return FALSE;
                }
        }
 
-       if (sr_session_load(infile, &sr_sess) != SR_OK)
+       if (sr_session_load(ctx, infile, &sr_sess) != SR_OK)
                return FALSE;
 
+       sr_session_dev_list(sr_sess, &devices);
+
        if (srd_session_new(&sess) != SRD_OK)
                return FALSE;
        sr_session_datafeed_callback_add(sr_sess, sr_cb, sess);
@@ -378,7 +390,22 @@ static int run_testcase(char *infile, GSList *pdlist, struct output *op)
                                (GDestroyNotify)g_variant_unref);
                for (l = pd->options; l; l = l->next) {
                        option = l->data;
-                       g_hash_table_insert(opts, option->key, option->value);
+
+                       is_number = TRUE;
+                       s = g_variant_get_string(option->value, NULL);
+                       for (i = 0; i < (int)strlen(s); i++) {
+                               if (!isdigit(s[i]))
+                                       is_number = FALSE;
+                       }
+
+                       if (is_number) {
+                               /* Integer option value */
+                               g_hash_table_insert(opts, option->key,
+                                 g_variant_new_int64(strtoull(s, NULL, 10)));
+                       } else {
+                               /* String option value */
+                               g_hash_table_insert(opts, option->key, option->value);
+                       }
                }
                if (!(di = srd_inst_new(sess, pd->name, opts)))
                        return FALSE;
@@ -397,14 +424,16 @@ static int run_testcase(char *infile, GSList *pdlist, struct output *op)
                                g_variant_ref_sink(gvar);
                                g_hash_table_insert(channels, channel->name, gvar);
                        }
-                       if (srd_inst_channel_set_all(di, channels,
-                                       (max_channel + 8) / 8) != SRD_OK)
+
+                       if (srd_inst_channel_set_all(di, channels) != SRD_OK)
                                return FALSE;
                        g_hash_table_destroy(channels);
                }
 
-               /* If this is not the first decoder in the list, stack it
-                * on top of the previous one. */
+               /*
+                * If this is not the first decoder in the list, stack it
+                * on top of the previous one.
+                */
                if (prev_di) {
                        if (srd_inst_stack(sess, prev_di, di) != SRD_OK) {
                                ERR("Failed to stack decoder instances.");
@@ -492,7 +521,7 @@ static PyObject *start_coverage(GSList *pdlist)
        return py_cov;
 }
 
-struct cvg *get_mod_cov(PyObject *py_cov, char *module_name)
+static struct cvg *get_mod_cov(PyObject *py_cov, const char *module_name)
 {
        PyObject *py_mod, *py_pathlist, *py_path, *py_func, *py_pd;
        PyObject *py_result, *py_missed, *py_item;
@@ -509,7 +538,7 @@ struct cvg *get_mod_cov(PyObject *py_cov, char *module_name)
        py_pathlist = PyObject_GetAttrString(py_mod, "__path__");
        for (i = 0; i < PyList_Size(py_pathlist); i++) {
                py_path = PyList_GetItem(py_pathlist, i);
-        PyUnicode_FSConverter(PyList_GetItem(py_pathlist, i), &py_path);
+               PyUnicode_FSConverter(PyList_GetItem(py_pathlist, i), &py_path);
                path = PyBytes_AS_STRING(py_path);
                if (!(d = opendir(path))) {
                        ERR("Invalid module path '%s'", path);
@@ -560,7 +589,7 @@ struct cvg *get_mod_cov(PyObject *py_cov, char *module_name)
        return cvg_mod;
 }
 
-struct cvg *cvg_new(void)
+static struct cvg *cvg_new(void)
 {
        struct cvg *cvg;
 
@@ -569,7 +598,7 @@ struct cvg *cvg_new(void)
        return cvg;
 }
 
-gboolean find_missed_line(struct cvg *cvg, char *linespec)
+static gboolean find_missed_line(struct cvg *cvg, const char *linespec)
 {
        GSList *l;
 
@@ -580,7 +609,7 @@ gboolean find_missed_line(struct cvg *cvg, char *linespec)
        return FALSE;
 }
 
-void cvg_add(struct cvg *dst, struct cvg *src)
+static void cvg_add(struct cvg *dst, const struct cvg *src)
 {
        GSList *l;
        char *linespec;
@@ -678,7 +707,6 @@ static int report_coverage(PyObject *py_cov, GSList *pdlist)
 
 int main(int argc, char **argv)
 {
-       struct sr_context *ctx;
        PyObject *coverage;
        GSList *pdlist;
        struct pd *pd;