]> sigrok.org Git - libsigrok.git/blobdiff - src/hardware/rdtech-dps/api.c
rdtech-dps: research to reduce serial comm transfer volume again
[libsigrok.git] / src / hardware / rdtech-dps / api.c
index 9a08c792a35cd4fc6ab8f838bd8a0a519b78b275..1c67ea49d1715d5b32cb3b18b445288623438cdc 100644 (file)
@@ -3,6 +3,7 @@
  *
  * Copyright (C) 2018 James Churchill <pelrun@gmail.com>
  * Copyright (C) 2019 Frank Stettner <frank-stettner@gmx.net>
+ * Copyright (C) 2021 Gerhard Sittig <gerhard.sittig@gmx.net>
  *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  */
 
 #include <config.h>
+
 #include <math.h>
+#include <string.h>
+
 #include "protocol.h"
 
 static const uint32_t scanopts[] = {
@@ -48,53 +52,91 @@ static const uint32_t devopts[] = {
        SR_CONF_OVER_CURRENT_PROTECTION_THRESHOLD | SR_CONF_GET | SR_CONF_SET,
 };
 
-/* Model ID, model name, max current, max voltage, max power */
+/* Model ID, model name, max current/voltage/power, current/voltage digits. */
 static const struct rdtech_dps_model supported_models[] = {
-       { 3005, "DPS3005",  5, 30,  160, 3, 2 },
-       { 5005, "DPS5005",  5, 50,  250, 3, 2 },
-       { 5205, "DPH5005",  5, 50,  250, 3, 2 },
-       { 5015, "DPS5015", 15, 50,  750, 2, 2 },
-       { 5020, "DPS5020", 20, 50, 1000, 2, 2 },
-       { 8005, "DPS8005",  5, 80,  408, 3, 2 },
+       { MODEL_DPS, 3005, "DPS3005",  5, 30,  160, 3, 2 },
+       { MODEL_DPS, 5005, "DPS5005",  5, 50,  250, 3, 2 },
+       { MODEL_DPS, 5205, "DPH5005",  5, 50,  250, 3, 2 },
+       { MODEL_DPS, 5015, "DPS5015", 15, 50,  750, 2, 2 },
+       { MODEL_DPS, 5020, "DPS5020", 20, 50, 1000, 2, 2 },
+       { MODEL_DPS, 8005, "DPS8005",  5, 80,  408, 3, 2 },
+       /* All RD specs taken from the 2020.12.2 instruction manual. */
+       { MODEL_RD , 6006, "RD6006" ,  6, 60,  360, 3, 2 },
+       { MODEL_RD , 6012, "RD6012" , 12, 60,  720, 2, 2 },
+       { MODEL_RD , 6018, "RD6018" , 18, 60, 1080, 2, 2 },
 };
 
 static struct sr_dev_driver rdtech_dps_driver_info;
+static struct sr_dev_driver rdtech_rd_driver_info;
 
-static struct sr_dev_inst *probe_device(struct sr_modbus_dev_inst *modbus)
+static struct sr_dev_inst *probe_device(struct sr_modbus_dev_inst *modbus,
+       enum rdtech_dps_model_type model_type)
 {
-       const struct rdtech_dps_model *model = NULL;
-       struct dev_context *devc;
-       struct sr_dev_inst *sdi;
+       static const char *type_prefix[] = {
+               [MODEL_DPS] = "DPS",
+               [MODEL_RD]  = "RD",
+       };
+
        uint16_t id, version;
-       unsigned int i;
+       uint32_t serno;
+       int ret;
+       const struct rdtech_dps_model *model, *supported;
+       size_t i;
+       struct sr_dev_inst *sdi;
+       struct dev_context *devc;
 
-       int ret = rdtech_dps_get_model_version(modbus, &id, &version);
+       ret = rdtech_dps_get_model_version(modbus,
+               model_type, &id, &version, &serno);
+       sr_dbg("probe: ret %d, type %s, model %u, vers %u, snr %u.",
+               ret, type_prefix[model_type], id, version, serno);
        if (ret != SR_OK)
                return NULL;
-       for (i = 0; i < ARRAY_SIZE(supported_models); i++)
-               if (id == supported_models[i].id) {
-                       model = &supported_models[i];
-                       break;
-               }
-       if (model == NULL) {
-               sr_err("Unknown model: %d.", id);
+       model = NULL;
+       for (i = 0; i < ARRAY_SIZE(supported_models); i++) {
+               supported = &supported_models[i];
+               if (model_type != supported->model_type)
+                       continue;
+               if (id != supported->id)
+                       continue;
+               model = supported;
+               break;
+       }
+       if (!model) {
+               sr_err("Unknown model: %s%u.", type_prefix[model_type], id);
                return NULL;
        }
 
-       sdi = g_malloc0(sizeof(struct sr_dev_inst));
+       sdi = g_malloc0(sizeof(*sdi));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup("RDTech");
-       sdi->model = g_strdup(model->name);
-       sdi->version = g_strdup_printf("v%d", version);
+       switch (model_type) {
+       case MODEL_DPS:
+               sdi->model = g_strdup(model->name);
+               sdi->version = g_strdup_printf("v%u", version);
+               sdi->driver = &rdtech_dps_driver_info;
+               break;
+       case MODEL_RD:
+               sdi->model = g_strdup(model->name);
+               sdi->version = g_strdup_printf("v%u.%u",
+                       version / 100, version % 100);
+               if (serno)
+                       sdi->serial_num = g_strdup_printf("%u", serno);
+               sdi->driver = &rdtech_rd_driver_info;
+               break;
+       default:
+               sr_err("Programming error, unhandled DPS/DPH/RD device type.");
+               g_free(sdi->vendor);
+               g_free(sdi);
+               return NULL;
+       }
        sdi->conn = modbus;
-       sdi->driver = &rdtech_dps_driver_info;
        sdi->inst_type = SR_INST_MODBUS;
 
        sr_channel_new(sdi, 0, SR_CHANNEL_ANALOG, TRUE, "V");
-       sr_channel_new(sdi, 0, SR_CHANNEL_ANALOG, TRUE, "I");
-       sr_channel_new(sdi, 0, SR_CHANNEL_ANALOG, TRUE, "P");
+       sr_channel_new(sdi, 1, SR_CHANNEL_ANALOG, TRUE, "I");
+       sr_channel_new(sdi, 2, SR_CHANNEL_ANALOG, TRUE, "P");
 
-       devc = g_malloc0(sizeof(struct dev_context));
+       devc = g_malloc0(sizeof(*devc));
        sr_sw_limits_init(&devc->limits);
        devc->model = model;
        devc->current_multiplier = pow(10.0, model->current_digits);
@@ -105,30 +147,63 @@ static struct sr_dev_inst *probe_device(struct sr_modbus_dev_inst *modbus)
        return sdi;
 }
 
+static struct sr_dev_inst *probe_device_dps(struct sr_modbus_dev_inst *modbus)
+{
+       return probe_device(modbus, MODEL_DPS);
+}
+
+static struct sr_dev_inst *probe_device_rd(struct sr_modbus_dev_inst *modbus)
+{
+       return probe_device(modbus, MODEL_RD);
+}
+
 static int config_compare(gconstpointer a, gconstpointer b)
 {
        const struct sr_config *ac = a, *bc = b;
        return ac->key != bc->key;
 }
 
-static GSList *scan(struct sr_dev_driver *di, GSList *options)
+static GSList *scan(struct sr_dev_driver *di, GSList *options,
+       enum rdtech_dps_model_type model_type)
 {
+       static const char *default_serialcomm_dps = "9600/8n1";
+       static const char *default_serialcomm_rd = "115200/8n1";
+
        struct sr_config default_serialcomm = {
                .key = SR_CONF_SERIALCOMM,
-               .data = g_variant_new_string("9600/8n1"),
+               .data = NULL,
        };
        struct sr_config default_modbusaddr = {
                .key = SR_CONF_MODBUSADDR,
                .data = g_variant_new_uint64(1),
        };
-       GSList *opts = options, *devices;
+       GSList *opts, *devices;
+       const char *serialcomm;
+       struct sr_dev_inst *(*probe_func)(struct sr_modbus_dev_inst *modbus);
+
+       /* TODO See why di->context isn't available yet at this time. */
+       serialcomm = NULL;
+       probe_func = NULL;
+       if (di->context == &rdtech_dps_driver_info || model_type == MODEL_DPS) {
+               serialcomm = default_serialcomm_dps;
+               probe_func = probe_device_dps;
+       }
+       if (di->context == &rdtech_rd_driver_info || model_type == MODEL_RD) {
+               serialcomm = default_serialcomm_rd;
+               probe_func = probe_device_rd;
+       }
+       if (!probe_func)
+               return NULL;
+       if (serialcomm && *serialcomm)
+               default_serialcomm.data = g_variant_new_string(serialcomm);
 
+       opts = options;
        if (!g_slist_find_custom(options, &default_serialcomm, config_compare))
                opts = g_slist_prepend(opts, &default_serialcomm);
        if (!g_slist_find_custom(options, &default_modbusaddr, config_compare))
                opts = g_slist_prepend(opts, &default_modbusaddr);
 
-       devices = sr_modbus_scan(di->context, opts, probe_device);
+       devices = sr_modbus_scan(di->context, opts, probe_func);
 
        while (opts != options)
                opts = g_slist_delete_link(opts, opts);
@@ -138,14 +213,32 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        return devices;
 }
 
+static GSList *scan_dps(struct sr_dev_driver *di, GSList *options)
+{
+       return scan(di, options, MODEL_DPS);
+}
+
+static GSList *scan_rd(struct sr_dev_driver *di, GSList *options)
+{
+       return scan(di, options, MODEL_RD);
+}
+
 static int dev_open(struct sr_dev_inst *sdi)
 {
-       struct sr_modbus_dev_inst *modbus = sdi->conn;
+       struct sr_modbus_dev_inst *modbus;
+       struct rdtech_dps_state state;
+       int ret;
 
+       modbus = sdi->conn;
        if (sr_modbus_open(modbus) < 0)
                return SR_ERR;
 
-       rdtech_dps_set_reg(sdi, REG_LOCK, 1);
+       memset(&state, 0, sizeof(state));
+       state.lock = TRUE;
+       state.mask |= STATE_LOCK;
+       ret = rdtech_dps_set_state(sdi, &state);
+       if (ret != SR_OK)
+               return ret;
 
        return SR_OK;
 }
@@ -153,12 +246,16 @@ static int dev_open(struct sr_dev_inst *sdi)
 static int dev_close(struct sr_dev_inst *sdi)
 {
        struct sr_modbus_dev_inst *modbus;
+       struct rdtech_dps_state state;
 
        modbus = sdi->conn;
        if (!modbus)
                return SR_ERR_BUG;
 
-       rdtech_dps_set_reg(sdi, REG_LOCK, 0);
+       memset(&state, 0, sizeof(state));
+       state.lock = FALSE;
+       state.mask |= STATE_LOCK;
+       (void)rdtech_dps_set_state(sdi, &state);
 
        return sr_modbus_close(modbus);
 }
@@ -167,8 +264,9 @@ static int config_get(uint32_t key, GVariant **data,
        const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
 {
        struct dev_context *devc;
+       struct rdtech_dps_state state;
        int ret;
-       uint16_t ivalue;
+       const char *cc_text;
 
        (void)cg;
 
@@ -181,51 +279,101 @@ static int config_get(uint32_t key, GVariant **data,
                ret = sr_sw_limits_config_get(&devc->limits, key, data);
                break;
        case SR_CONF_ENABLED:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_ENABLE, &ivalue)) == SR_OK)
-                       *data = g_variant_new_boolean(ivalue);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_OUTPUT_ENABLED))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_boolean(state.output_enabled);
                break;
        case SR_CONF_REGULATION:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_CV_CC, &ivalue)) != SR_OK)
-                       break;
-               *data = g_variant_new_string((ivalue == MODE_CC) ? "CC" : "CV");
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_REGULATION_CC))
+                       return SR_ERR_DATA;
+               cc_text = state.regulation_cc ? "CC" : "CV";
+               *data = g_variant_new_string(cc_text);
                break;
        case SR_CONF_VOLTAGE:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_UOUT, &ivalue)) == SR_OK)
-                       *data = g_variant_new_double((float)ivalue / devc->voltage_multiplier);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_VOLTAGE))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_double(state.voltage);
                break;
        case SR_CONF_VOLTAGE_TARGET:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_USET, &ivalue)) == SR_OK)
-                       *data = g_variant_new_double((float)ivalue / devc->voltage_multiplier);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_VOLTAGE_TARGET))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_double(state.voltage_target);
                break;
        case SR_CONF_CURRENT:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_IOUT, &ivalue)) == SR_OK)
-                       *data = g_variant_new_double((float)ivalue / devc->current_multiplier);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_CURRENT))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_double(state.current);
                break;
        case SR_CONF_CURRENT_LIMIT:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_ISET, &ivalue)) == SR_OK)
-                       *data = g_variant_new_double((float)ivalue / devc->current_multiplier);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_CURRENT_LIMIT))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_double(state.current_limit);
                break;
        case SR_CONF_OVER_VOLTAGE_PROTECTION_ENABLED:
-               *data = g_variant_new_boolean(TRUE);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_PROTECT_ENABLED))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_boolean(state.protect_enabled);
                break;
        case SR_CONF_OVER_VOLTAGE_PROTECTION_ACTIVE:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_PROTECT, &ivalue)) == SR_OK)
-                       *data = g_variant_new_boolean(ivalue == STATE_OVP);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_PROTECT_OVP))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_boolean(state.protect_ovp);
                break;
        case SR_CONF_OVER_VOLTAGE_PROTECTION_THRESHOLD:
-               if ((ret = rdtech_dps_get_reg(sdi, PRE_OVPSET, &ivalue)) == SR_OK)
-                       *data = g_variant_new_double((float)ivalue / devc->voltage_multiplier);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_OUTPUT_ENABLED))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_double(state.ovp_threshold);
                break;
        case SR_CONF_OVER_CURRENT_PROTECTION_ENABLED:
-               *data = g_variant_new_boolean(TRUE);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_PROTECT_ENABLED))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_boolean(state.protect_enabled);
                break;
        case SR_CONF_OVER_CURRENT_PROTECTION_ACTIVE:
-               if ((ret = rdtech_dps_get_reg(sdi, REG_PROTECT, &ivalue)) == SR_OK)
-                       *data = g_variant_new_boolean(ivalue == STATE_OCP);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_PROTECT_OCP))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_boolean(state.protect_ocp);
                break;
        case SR_CONF_OVER_CURRENT_PROTECTION_THRESHOLD:
-               if ((ret = rdtech_dps_get_reg(sdi, PRE_OCPSET, &ivalue)) == SR_OK)
-                       *data = g_variant_new_double((float)ivalue / devc->current_multiplier);
+               ret = rdtech_dps_get_state(sdi, &state, ST_CTX_CONFIG);
+               if (ret != SR_OK)
+                       return ret;
+               if (!(state.mask & STATE_OCP_THRESHOLD))
+                       return SR_ERR_DATA;
+               *data = g_variant_new_double(state.ocp_threshold);
                break;
        default:
                return SR_ERR_NA;
@@ -238,29 +386,37 @@ static int config_set(uint32_t key, GVariant *data,
        const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
 {
        struct dev_context *devc;
+       struct rdtech_dps_state state;
 
        (void)cg;
 
        devc = sdi->priv;
+       memset(&state, 0, sizeof(state));
 
        switch (key) {
        case SR_CONF_LIMIT_SAMPLES:
        case SR_CONF_LIMIT_MSEC:
                return sr_sw_limits_config_set(&devc->limits, key, data);
        case SR_CONF_ENABLED:
-               return rdtech_dps_set_reg(sdi, REG_ENABLE, g_variant_get_boolean(data));
+               state.output_enabled = g_variant_get_boolean(data);
+               state.mask |= STATE_OUTPUT_ENABLED;
+               return rdtech_dps_set_state(sdi, &state);
        case SR_CONF_VOLTAGE_TARGET:
-               return rdtech_dps_set_reg(sdi, REG_USET,
-                       g_variant_get_double(data) * devc->voltage_multiplier);
+               state.voltage_target = g_variant_get_double(data);
+               state.mask |= STATE_VOLTAGE_TARGET;
+               return rdtech_dps_set_state(sdi, &state);
        case SR_CONF_CURRENT_LIMIT:
-               return rdtech_dps_set_reg(sdi, REG_ISET,
-                       g_variant_get_double(data) * devc->current_multiplier);
+               state.current_limit = g_variant_get_double(data);
+               state.mask |= STATE_CURRENT_LIMIT;
+               return rdtech_dps_set_state(sdi, &state);
        case SR_CONF_OVER_VOLTAGE_PROTECTION_THRESHOLD:
-               return rdtech_dps_set_reg(sdi, PRE_OVPSET,
-                       g_variant_get_double(data) * devc->voltage_multiplier);
+               state.ovp_threshold = g_variant_get_double(data);
+               state.mask |= STATE_OVP_THRESHOLD;
+               return rdtech_dps_set_state(sdi, &state);
        case SR_CONF_OVER_CURRENT_PROTECTION_THRESHOLD:
-               return rdtech_dps_set_reg(sdi, PRE_OCPSET,
-                       g_variant_get_double(data) * devc->current_multiplier);
+               state.ocp_threshold = g_variant_get_double(data);
+               state.mask |= STATE_OCP_THRESHOLD;
+               return rdtech_dps_set_state(sdi, &state);
        default:
                return SR_ERR_NA;
        }
@@ -298,23 +454,20 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
        struct sr_modbus_dev_inst *modbus;
-       uint16_t registers[3];
        int ret;
 
        modbus = sdi->conn;
        devc = sdi->priv;
 
-       /* Prefill actual states */
-       ret = rdtech_dps_read_holding_registers(modbus, REG_PROTECT, 3, registers);
+       /* Seed internal state from current data. */
+       ret = rdtech_dps_seed_receive(sdi);
        if (ret != SR_OK)
                return ret;
-       devc->actual_ovp_state = RB16(registers + 0) == STATE_OVP;
-       devc->actual_ocp_state = RB16(registers + 0) == STATE_OCP;
-       devc->actual_regulation_state = RB16(registers + 1);
-       devc->actual_output_state = RB16(registers + 2);
 
-       if ((ret = sr_modbus_source_add(sdi->session, modbus, G_IO_IN, 10,
-                       rdtech_dps_receive_data, (void *)sdi)) != SR_OK)
+       /* Register the periodic data reception callback. */
+       ret = sr_modbus_source_add(sdi->session, modbus, G_IO_IN, 10,
+                       rdtech_dps_receive_data, (void *)sdi);
+       if (ret != SR_OK)
                return ret;
 
        sr_sw_limits_acquisition_start(&devc->limits);
@@ -341,7 +494,7 @@ static struct sr_dev_driver rdtech_dps_driver_info = {
        .api_version = 1,
        .init = std_init,
        .cleanup = std_cleanup,
-       .scan = scan,
+       .scan = scan_dps,
        .dev_list = std_dev_list,
        .dev_clear = std_dev_clear,
        .config_get = config_get,
@@ -354,3 +507,23 @@ static struct sr_dev_driver rdtech_dps_driver_info = {
        .context = NULL,
 };
 SR_REGISTER_DEV_DRIVER(rdtech_dps_driver_info);
+
+static struct sr_dev_driver rdtech_rd_driver_info = {
+       .name = "rdtech-rd",
+       .longname = "RDTech RD series power supply",
+       .api_version = 1,
+       .init = std_init,
+       .cleanup = std_cleanup,
+       .scan = scan_rd,
+       .dev_list = std_dev_list,
+       .dev_clear = std_dev_clear,
+       .config_get = config_get,
+       .config_set = config_set,
+       .config_list = config_list,
+       .dev_open = dev_open,
+       .dev_close = dev_close,
+       .dev_acquisition_start = dev_acquisition_start,
+       .dev_acquisition_stop = dev_acquisition_stop,
+       .context = NULL,
+};
+SR_REGISTER_DEV_DRIVER(rdtech_rd_driver_info);