]> sigrok.org Git - libsigrok.git/blobdiff - src/hardware/korad-kaxxxxp/api.c
korad-kaxxxxp: factor voltage/current capability out of models[] table
[libsigrok.git] / src / hardware / korad-kaxxxxp / api.c
index 1f82f2278631b013c840fd413ab4739aa1da69c4..1b35b4a58b8168a5e972aac9b6677b26b15812f5 100644 (file)
@@ -24,6 +24,7 @@
 static const uint32_t scanopts[] = {
        SR_CONF_CONN,
        SR_CONF_SERIALCOMM,
+       SR_CONF_FORCE_DETECT,
 };
 
 static const uint32_t drvopts[] = {
@@ -31,6 +32,7 @@ static const uint32_t drvopts[] = {
 };
 
 static const uint32_t devopts[] = {
+       SR_CONF_CONN | SR_CONF_GET,
        SR_CONF_CONTINUOUS,
        SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
        SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET,
@@ -44,65 +46,84 @@ static const uint32_t devopts[] = {
        SR_CONF_OVER_VOLTAGE_PROTECTION_ENABLED | SR_CONF_GET | SR_CONF_SET,
 };
 
+/* Voltage and current ranges. Values are: Min, max, step. */
+static const double volts_30[] = { 0, 31, 0.01, };
+static const double volts_60[] = { 0, 61, 0.01, };
+static const double amps_3[] = { 0, 3.1, 0.001, };
+static const double amps_5[] = { 0, 5.1, 0.001, };
+
 static const struct korad_kaxxxxp_model models[] = {
        /* Device enum, vendor, model, ID reply, channels, voltage, current */
-       {VELLEMAN_PS3005D, "Velleman", "PS3005D",
-               "VELLEMANPS3005DV2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {VELLEMAN_LABPS3005D, "Velleman", "LABPS3005D",
-               "VELLEMANLABPS3005DV2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {KORAD_KA3005P, "Korad", "KA3005P",
-               "KORADKA3005PV2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
+       {KORAD_KA3005P, "Korad", "KA3005P", "KORADKA3005PV2.0",
+               1, volts_30, amps_5},
        /* Sometimes the KA3005P has an extra 0x01 after the ID. */
-       {KORAD_KA3005P_0X01, "Korad", "KA3005P",
-               "KORADKA3005PV2.0\x01", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
+       {KORAD_KA3005P_0X01, "Korad", "KA3005P", "KORADKA3005PV2.0\x01",
+               1, volts_30, amps_5},
        /* Sometimes the KA3005P has an extra 0xBC after the ID. */
-       {KORAD_KA3005P_0XBC, "Korad", "KA3005P",
-               "KORADKA3005PV2.0\xBC", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {KORAD_KA3005P_V42, "Korad", "KA3005P",
-               "KORAD KA3005P V4.2", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {KORAD_KA3005P_V55, "Korad", "KA3005P",
-               "KORAD KA3005P V5.5", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {KORAD_KD3005P, "Korad", "KD3005P",
-               "KORAD KD3005P V2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {KORAD_KD3005P_V20_NOSP, "Korad", "KD3005P",
-               "KORADKD3005PV2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {RND_320_KD3005P, "RND", "KD3005P",
-               "RND 320-KD3005P V4.2", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {RND_320_KA3005P, "RND", "KA3005P",
-               "RND 320-KA3005P V5.5", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {RND_320K30PV, "RND", "KA3005P",
-               "RND 320-KA3005P V2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {TENMA_72_2550_V2, "Tenma", "72-2550",
-               "TENMA72-2550V2.0", 1, {0, 61, 0.01}, {0, 3.1, 0.001}},
-       {TENMA_72_2540_V20, "Tenma", "72-2540",
-               "TENMA72-2540V2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {TENMA_72_2540_V21, "Tenma", "72-2540",
-               "TENMA 72-2540 V2.1", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {TENMA_72_2540_V52, "Tenma", "72-2540",
-               "TENMA 72-2540 V5.2", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {TENMA_72_2535_V21, "Tenma", "72-2535",
-               "TENMA 72-2535 V2.1", 1, {0, 31, 0.01}, {0, 3.1, 0.001}},
-       {STAMOS_SLS31_V20, "Stamos Soldering", "S-LS-31",
-               "S-LS-31 V2.0", 1, {0, 31, 0.01}, {0, 5.1, 0.001}},
-       {KORAD_KD6005P, "Korad", "KD6005P",
-               "KORAD KD6005P V2.2", 1, {0, 61, 0.01}, {0, 5.1, 0.001}},
+       {KORAD_KA3005P_0XBC, "Korad", "KA3005P", "KORADKA3005PV2.0\xBC",
+               1, volts_30, amps_5},
+       {KORAD_KA3005P_V42, "Korad", "KA3005P", "KORAD KA3005P V4.2",
+               1, volts_30, amps_5},
+       {KORAD_KA3005P_V55, "Korad", "KA3005P", "KORAD KA3005P V5.5",
+               1, volts_30, amps_5},
+       {KORAD_KD3005P_V20, "Korad", "KD3005P", "KORAD KD3005P V2.0",
+               1, volts_30, amps_5},
+       {KORAD_KD3005P_V20_NOSP, "Korad", "KD3005P", "KORADKD3005PV2.0",
+               1, volts_30, amps_5},
+       {KORAD_KD3005P_V21_NOSP, "Korad", "KD3005P", "KORADKD3005PV2.1",
+               1, volts_30, amps_5},
+       {KORAD_KD3005P_V41, "Korad", "KD3005P", "KORAD KD3005P V4.1",
+               1, volts_30, amps_5},
+       {KORAD_KD3005P_V68, "Korad", "KD3005P", "KORAD KD3005P V6.8",
+               1, volts_30, amps_5},
+       {KORAD_KD6005P, "Korad", "KD6005P", "KORAD KD6005P V2.2",
+               1, volts_60, amps_5},
+       {RND_320_KA3005P, "RND", "KA3005P", "RND 320-KA3005P V5.5",
+               1, volts_30, amps_5},
+       {RND_320_KD3005P, "RND", "KD3005P", "RND 320-KD3005P V4.2",
+               1, volts_30, amps_5},
+       {RND_320K30PV, "RND", "KA3005P", "RND 320-KA3005P V2.0",
+               1, volts_30, amps_5},
+       {STAMOS_SLS31_V20, "Stamos Soldering", "S-LS-31", "S-LS-31 V2.0",
+               1, volts_30, amps_5},
+       {TENMA_72_2535_V21, "Tenma", "72-2535", "TENMA 72-2535 V2.1",
+               1, volts_30, amps_3},
+       {TENMA_72_2540_V20, "Tenma", "72-2540", "TENMA72-2540V2.0",
+               1, volts_30, amps_5},
+       {TENMA_72_2540_V21, "Tenma", "72-2540", "TENMA 72-2540 V2.1",
+               1, volts_30, amps_5},
+       {TENMA_72_2540_V52, "Tenma", "72-2540", "TENMA 72-2540 V5.2",
+               1, volts_30, amps_5},
+       {TENMA_72_2550_V2, "Tenma", "72-2550", "TENMA72-2550V2.0",
+               1, volts_60, amps_3},
+       {TENMA_72_2710_V66, "Tenma", "72-2710", "TENMA 72-2710 V6.6",
+               1, volts_30, amps_5},
+       {VELLEMAN_LABPS3005D, "Velleman", "LABPS3005D", "VELLEMANLABPS3005DV2.0",
+               1, volts_30, amps_5},
+       {VELLEMAN_PS3005D, "Velleman", "PS3005D", "VELLEMANPS3005DV2.0",
+               1, volts_30, amps_5},
        ALL_ZERO
 };
 
 static GSList *scan(struct sr_dev_driver *di, GSList *options)
 {
+       static const char *serno_prefix = " SN:";
+
        struct dev_context *devc;
        GSList *l;
        struct sr_dev_inst *sdi;
        struct sr_config *src;
        const char *conn, *serialcomm;
+       const char *force_detect;
        struct sr_serial_dev_inst *serial;
        char reply[50];
-       int i, model_id;
-       unsigned int len;
+       int ret, i, model_id;
+       size_t len;
+       char *serno;
 
        conn = NULL;
        serialcomm = NULL;
+       force_detect = NULL;
 
        for (l = options; l; l = l->next) {
                src = l->data;
@@ -113,6 +134,9 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                case SR_CONF_SERIALCOMM:
                        serialcomm = g_variant_get_string(src->data, NULL);
                        break;
+               case SR_CONF_FORCE_DETECT:
+                       force_detect = g_variant_get_string(src->data, NULL);
+                       break;
                default:
                        sr_err("Unknown option %d, skipping.", src->key);
                        break;
@@ -123,45 +147,64 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                return NULL;
        if (!serialcomm)
                serialcomm = "9600/8n1";
+       if (force_detect && !*force_detect)
+               force_detect = NULL;
 
        serial = sr_serial_dev_inst_new(conn, serialcomm);
        if (serial_open(serial, SERIAL_RDWR) != SR_OK)
                return NULL;
 
-       serial_flush(serial);
-
-       /* Get the device model. */
+       /*
+        * Prepare a receive buffer for the identification response that
+        * is large enough to hold the longest known model name, and an
+        * optional serial number. Communicate the identification request.
+        */
        len = 0;
        for (i = 0; models[i].id; i++) {
-               if (strlen(models[i].id) > len)
+               if (len < strlen(models[i].id))
                        len = strlen(models[i].id);
        }
+       len += strlen(serno_prefix) + 12;
+       if (len > sizeof(reply) - 1)
+               len = sizeof(reply) - 1;
+       sr_dbg("Want max %zu bytes.", len);
 
-       /*
-        * Some models also include the serial number:
-        * RND 320-KD3005P V4.2 SN:59834414
-        */
-       len += 12;
-
-       memset(&reply, 0, sizeof(reply));
-       sr_dbg("Want max %d bytes.", len);
-       if ((korad_kaxxxxp_send_cmd(serial, "*IDN?") < 0))
+       ret = korad_kaxxxxp_send_cmd(serial, "*IDN?");
+       if (ret < 0)
                return NULL;
 
-       /* i is used here for debug purposes only. */
-       if ((i = korad_kaxxxxp_read_chars(serial, len, reply)) < 0)
+       ret = korad_kaxxxxp_read_chars(serial, len, reply);
+       if (ret < 0)
                return NULL;
-       sr_dbg("Received: %d, %s", i, reply);
-       model_id = -1;
+       sr_dbg("Received: %d, %s", ret, reply);
 
-       /* Truncate before serial number. */
-       char *sn = g_strrstr(reply, " SN:");
-       if (sn)
-               *sn = '\0';
+       /*
+        * Isolate the optional serial number at the response's end.
+        * Lookup the response's model ID in the list of known models.
+        */
+       serno = g_strrstr(reply, serno_prefix);
+       if (serno) {
+               *serno = '\0';
+               serno += strlen(serno_prefix);
+       }
 
+       model_id = -1;
        for (i = 0; models[i].id; i++) {
-               if (!g_strcmp0(models[i].id, reply))
+               if (g_strcmp0(models[i].id, reply) != 0)
+                       continue;
+               model_id = i;
+               break;
+       }
+       if (model_id < 0 && force_detect) {
+               sr_warn("Found model ID '%s' is unknown, trying '%s' spec.",
+                       reply, force_detect);
+               for (i = 0; models[i].id; i++) {
+                       if (strcmp(models[i].id, force_detect) != 0)
+                               continue;
+                       sr_info("Found replacement, using it instead.");
                        model_id = i;
+                       break;
+               }
        }
        if (model_id < 0) {
                sr_err("Unknown model ID '%s' detected, aborting.", reply);
@@ -174,8 +217,11 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup(models[model_id].vendor);
        sdi->model = g_strdup(models[model_id].name);
+       if (serno)
+               sdi->serial_num = g_strdup(serno);
        sdi->inst_type = SR_INST_SERIAL;
        sdi->conn = serial;
+       sdi->connection_id = g_strdup(conn);
 
        sr_channel_new(sdi, 0, SR_CHANNEL_ANALOG, TRUE, "V");
        sr_channel_new(sdi, 1, SR_CHANNEL_ANALOG, TRUE, "I");
@@ -224,6 +270,9 @@ static int config_get(uint32_t key, GVariant **data,
        case SR_CONF_LIMIT_SAMPLES:
        case SR_CONF_LIMIT_MSEC:
                return sr_sw_limits_config_get(&devc->limits, key, data);
+       case SR_CONF_CONN:
+               *data = g_variant_new_string(sdi->connection_id);
+               break;
        case SR_CONF_VOLTAGE:
                korad_kaxxxxp_get_value(sdi->conn, KAXXXXP_VOLTAGE, devc);
                *data = g_variant_new_double(devc->voltage);