siglent-sds: Drop obsolete SR_ST_ACTIVE checks.

[libsigrok.git] / src / hardware / siglent-sds / api.c

diff --git a/src/hardware/siglent-sds/api.c b/src/hardware/siglent-sds/api.c
index 2d0a6b95394ade3395f14c59e30a7522762c5128..a1b1ed71d4c490aed37dfa145275d92f59830b22 100644 (file)
--- a/src/hardware/siglent-sds/api.c
+++ b/src/hardware/siglent-sds/api.c
@@ -124,9 +124,6 @@ static const uint64_t vdivs[][2] = {
        { 100, 1 },
 };
 
-#define NUM_TIMEBASE   ARRAY_SIZE(timebases)
-#define NUM_VDIV       ARRAY_SIZE(vdivs)
-
 static const char *trigger_sources[] = {
        "CH1", "CH2", "Ext", "Ext /5", "AC Line",
        "D0", "D1", "D2", "D3", "D4", "D5", "D6", "D7",
@@ -169,11 +166,9 @@ enum series {
        SDS2000X,
 };
 
-/* short name, full name, USB name */
+/* short name, full name */
 static const struct siglent_sds_vendor supported_vendors[] = {
-       [SIGLENT] = {
-               "Siglent", "Siglent Technologies", "Siglent Technologies Co,. Ltd.",
-       },
+       [SIGLENT] = {"Siglent", "Siglent Technologies"},
 };
 
 #define VENDOR(x) &supported_vendors[x]
@@ -326,7 +321,7 @@ static struct sr_dev_inst *probe_device(struct sr_scpi_dev_inst *scpi)
                        devc->digital_group);
        }
 
-       for (i = 0; i < NUM_TIMEBASE; i++) {
+       for (i = 0; i < ARRAY_SIZE(timebases); i++) {
                if (!memcmp(&devc->model->min_timebase, &timebases[i], sizeof(uint64_t[2])))
                        devc->timebases = &timebases[i];
 
@@ -334,12 +329,12 @@ static struct sr_dev_inst *probe_device(struct sr_scpi_dev_inst *scpi)
                        devc->num_timebases = &timebases[i] - devc->timebases + 1;
        }
 
-       for (i = 0; i < NUM_VDIV; i++) {
+       for (i = 0; i < ARRAY_SIZE(vdivs); i++) {
                devc->vdivs = &vdivs[i];
                if (!memcmp(&devc->model->series->min_vdiv,
                        &vdivs[i], sizeof(uint64_t[2]))) {
                        devc->vdivs = &vdivs[i];
-                       devc->num_vdivs = NUM_VDIV - i;
+                       devc->num_vdivs = ARRAY_SIZE(vdivs) - i;
                        break;
                }
        }
@@ -376,8 +371,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                return SR_ERR;
        }
 
-       sdi->status = SR_ST_ACTIVE;
-
        return SR_OK;
 }
 
@@ -437,7 +430,7 @@ static int config_get(uint32_t key, GVariant **data,
                break;
        case SR_CONF_SAMPLERATE:
                siglent_sds_get_dev_cfg_horizontal(sdi);
-               *data = g_variant_new_uint64(devc->sampleRate);
+               *data = g_variant_new_uint64(devc->samplerate);
                break;
        case SR_CONF_TRIGGER_SOURCE:
                if (!strcmp(devc->trigger_source, "ACL"))
@@ -540,9 +533,6 @@ static int config_set(uint32_t key, GVariant *data,
 
        devc = sdi->priv;
 
-       if (sdi->status != SR_ST_ACTIVE)
-               return SR_ERR_DEV_CLOSED;
-
        /* If a channel group is specified, it must be a valid one. */
        if (cg && !g_slist_find(sdi->channel_groups, cg)) {
                sr_err("Invalid channel group specified.");
@@ -583,7 +573,6 @@ static int config_set(uint32_t key, GVariant *data,
                        devc->trigger_level = t_dbl;
                break;
        case SR_CONF_TIMEBASE:
-               sr_dbg("Setting device Timebase");
                g_variant_get(data, "(tt)", &p, &q);
                for (i = 0; i < devc->num_timebases; i++) {
                        char *cmd;
@@ -592,16 +581,16 @@ static int config_set(uint32_t key, GVariant *data,
                                devc->timebase = (float) p / q;
                                switch (q) {
                                case 1:
-                                       cmd = g_strdup_printf("%luS", p);
+                                       cmd = g_strdup_printf("%" PRIu64 "S", p);
                                        break;
                                case 1000:
-                                       cmd = g_strdup_printf("%luMS", p);
+                                       cmd = g_strdup_printf("%" PRIu64 "MS", p);
                                        break;
                                case 1000000:
-                                       cmd = g_strdup_printf("%luUS", p);
+                                       cmd = g_strdup_printf("%" PRIu64 "US", p);
                                        break;
                                case 100000000:
-                                       cmd = g_strdup_printf("%luNS", p);
+                                       cmd = g_strdup_printf("%" PRIu64 "NS", p);
                                        break;
                                }
                                sr_dbg("Setting device timebase: TDIV %s.", cmd);
@@ -658,13 +647,13 @@ static int config_set(uint32_t key, GVariant *data,
                                        cmd = "";
                                        switch (q) {
                                        case 1:
-                                               cmd = g_strdup_printf("%luV", p);
+                                               cmd = g_strdup_printf("%" PRIu64 "V", p);
                                                break;
                                        case 1000:
-                                               cmd = g_strdup_printf("%luMV", p);
+                                               cmd = g_strdup_printf("%" PRIu64 "MV", p);
                                                break;
                                        case 100000:
-                                               cmd = g_strdup_printf("%luUV", p);
+                                               cmd = g_strdup_printf("%" PRIu64 "UV", p);
                                                break;
                                        }
                                        return siglent_sds_config_set(sdi, "C%d:VDIV %s", i + 1, cmd);
@@ -712,7 +701,7 @@ static int config_set(uint32_t key, GVariant *data,
                                                return ret;
                                        }
                                }
-                               sr_err("Invalid probe factor: %"PRIu64".", p);
+                               sr_err("Invalid probe factor: %" PRIu64 ".", p);
                                return SR_ERR_ARG;
                        }
                }
@@ -732,7 +721,7 @@ static int config_set(uint32_t key, GVariant *data,
                break;
        case SR_CONF_SAMPLERATE:
                siglent_sds_get_dev_cfg_horizontal(sdi);
-               data = g_variant_new_uint64(devc->sampleRate);
+               data = g_variant_new_uint64(devc->samplerate);
                break;
        default:
                return SR_ERR_NA;
@@ -874,11 +863,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        struct sr_channel *ch;
        struct sr_datafeed_packet packet;
        gboolean some_digital;
-       GSList *l;
-       GSList *d;
-
-       if (sdi->status != SR_ST_ACTIVE)
-               return SR_ERR_DEV_CLOSED;
+       GSList *l, *d;
 
        scpi = sdi->conn;
        devc = sdi->priv;
@@ -987,11 +972,6 @@ static int dev_acquisition_stop(struct sr_dev_inst *sdi)
 
        devc = sdi->priv;
 
-       if (sdi->status != SR_ST_ACTIVE) {
-               sr_err("Device inactive, can't stop acquisition.");
-               return SR_ERR;
-       }
-
        std_session_send_df_end(sdi);
 
        g_slist_free(devc->enabled_channels);