]> sigrok.org Git - libsigrok.git/blobdiff - src/hardware/scpi-pps/profiles.c
scpi-pps: Support for the EEZ PSU series
[libsigrok.git] / src / hardware / scpi-pps / profiles.c
index efec3ce9b49bcfd8e5443296c99f0ac6d4237e67..e947f744a0dab2d71e3db594735740a0a50ac178 100644 (file)
@@ -274,6 +274,177 @@ static int chroma_62000p_probe_channels(struct sr_dev_inst *sdi,
        return SR_OK;
 }
 
+/* Envox EEZ PSU Series */
+static const uint32_t eez_psu_devopts[] = {
+       SR_CONF_CONTINUOUS,
+       SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET,
+};
+
+static const uint32_t eez_psu_devopts_cg[] = {
+       SR_CONF_VOLTAGE | SR_CONF_GET,
+       SR_CONF_VOLTAGE_TARGET | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
+       SR_CONF_CURRENT | SR_CONF_GET,
+       SR_CONF_CURRENT_LIMIT | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
+       SR_CONF_ENABLED | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_REGULATION | SR_CONF_GET,
+       SR_CONF_OVER_VOLTAGE_PROTECTION_ACTIVE | SR_CONF_GET,
+       SR_CONF_OVER_VOLTAGE_PROTECTION_THRESHOLD | SR_CONF_GET,
+       SR_CONF_OVER_CURRENT_PROTECTION_ACTIVE | SR_CONF_GET,
+       SR_CONF_OVER_CURRENT_PROTECTION_THRESHOLD | SR_CONF_GET,
+};
+
+static const struct scpi_command eez_psu_cmd[] = {
+       { SCPI_CMD_REMOTE, "SYST:REMOTE" },
+       { SCPI_CMD_LOCAL, "SYST:LOCAL" },
+       { SCPI_CMD_SELECT_CHANNEL, ":INST:NSEL %s" },
+       { SCPI_CMD_GET_MEAS_VOLTAGE, ":MEAS:VOLT?" },
+       { SCPI_CMD_GET_MEAS_CURRENT, ":MEAS:CURR?" },
+       { SCPI_CMD_GET_MEAS_POWER, ":MEAS:POWER?" },
+       { SCPI_CMD_GET_OUTPUT_REGULATION, ":OUTP:MODE?" },
+       { SCPI_CMD_GET_VOLTAGE_TARGET, ":SOUR:VOLT?" },
+       { SCPI_CMD_SET_VOLTAGE_TARGET, ":SOUR:VOLT %.2f" },
+       { SCPI_CMD_GET_CURRENT_LIMIT, ":SOUR:CURR?" },
+       { SCPI_CMD_SET_CURRENT_LIMIT, ":SOUR:CURR %.6f" },
+       { SCPI_CMD_GET_OUTPUT_ENABLED, ":OUTP?" },
+       { SCPI_CMD_SET_OUTPUT_ENABLE, ":OUTP ON" },
+       { SCPI_CMD_SET_OUTPUT_DISABLE, ":OUTP OFF" },
+       { SCPI_CMD_GET_OVER_CURRENT_PROTECTION_THRESHOLD, ":SOUR:CURR:PROT?" },
+       { SCPI_CMD_GET_OVER_VOLTAGE_PROTECTION_THRESHOLD, ":SOUR:VOLT:PROT?" },
+       ALL_ZERO
+};
+
+/*
+ * The EEZ BB3 protocol currently specifies up to six channels. The older
+ * EEZ PSU design only has room for two in its enclosure design.
+ *
+ * If a future model's SCPI spec allows more than six models then we can
+ * extend this to support more.
+ */
+static const char *eez_psu_channel_names[] = { "1", "2", "3", "4", "5", "6", };
+
+static int eez_psu_probe_channels(struct sr_dev_inst *sdi,
+               struct sr_scpi_hw_info *hw_info,
+               struct channel_spec **channels, unsigned int *num_channels,
+               struct channel_group_spec **channel_groups,
+               unsigned int *num_channel_groups)
+{
+       struct sr_scpi_dev_inst *scpi;
+       int ret, intval;
+       size_t i, channel_count;
+       double limit_val;
+       const char *channel_name;
+
+       /*
+        * The EEZ PSU family is intended by the designer to be end-user
+        * customizable, so this is intentionally a little more dynamic
+        * than strictly necessary for the "stock" models, to make it
+        * more likely to automatically support end-user upgrades of the
+        * various ranges.
+        *
+        * The BB3 in particular supports various different modular
+        * power supply frontends that offer different voltage/current
+        * limits and different numbers of independent channels, such as
+        * three PSU modules that have two channels each for a total of
+        * six controllable channels.
+        *
+        * This currently supports both the original EEZ PSU design
+        * (H24005, when in its stock build configuration) and the
+        * successor EEZ BB3 design.
+        */
+
+       scpi = sdi->conn;
+       ret = sr_scpi_get_int(scpi, ":SYST:CHAN:COUN?", &intval);
+       if (ret != SR_OK) {
+               sr_err("Failed to probe EEZ PSU channel count.");
+               return ret;
+       }
+       if (intval < 0) {
+               sr_err("Suspicious channel count %d, ignoring.", intval);
+               return SR_ERR_DATA;
+       }
+       channel_count = intval;
+       if (channel_count > ARRAY_SIZE(eez_psu_channel_names)) {
+               /*
+                * No known EEZ PSU specifies more than six channels at
+                * the time of writing, so it would be weird to get here
+                * but we'll allow it to be robust.
+                */
+               sr_warn("Only using first %zu of %zu EEZ PSU channels.",
+                       channel_count, ARRAY_SIZE(eez_psu_channel_names));
+               channel_count = ARRAY_SIZE(eez_psu_channel_names);
+       }
+
+       sr_spew("EEZ PSU (%s) has channel count %zu.",
+               hw_info->model, channel_count);
+
+       *channels = g_malloc0(sizeof(**channels) * channel_count);
+       *channel_groups = g_malloc0(sizeof(**channel_groups) * channel_count);
+       for (i = 0; i < channel_count; i++) {
+               channel_name = eez_psu_channel_names[i];
+
+               /*
+                * Select the channel to prepare for our various "get"
+                * calls below.
+                */
+               ret = sr_scpi_send(scpi, ":INST:NSEL %s", channel_name);
+               if (ret != SR_OK) {
+                       sr_err("Failed to select %s to retrieve its limits.",
+                               channel_name);
+                       return ret;
+               }
+
+               (*channel_groups)[i].name = channel_name;
+               (*channel_groups)[i].channel_index_mask = CH_IDX(i);
+               (*channel_groups)[i].features = PPS_OVP | PPS_OCP;
+               (*channel_groups)[i].mqflags = SR_MQFLAG_DC;
+
+               (*channels)[i].name = channel_name;
+
+               ret = sr_scpi_get_double(scpi,
+                       ":SYST:CHAN:INFO:CURR?", &limit_val);
+               if (ret != SR_OK) {
+                       sr_err("Failed to read the current limit for %s.",
+                               channel_name);
+                       return ret;
+               }
+               (*channels)[i].current[0] = 0.0;
+               (*channels)[i].current[1] = limit_val;
+               (*channels)[i].current[2] = 0.01; /* Programming resolution. */
+               (*channels)[i].current[3] = 2; /* Spec digits. */
+               (*channels)[i].current[4] = 2; /* Encoding digits. */
+
+               ret = sr_scpi_get_double(scpi,
+                       ":SYST:CHAN:INFO:VOLT?", &limit_val);
+               if (ret != SR_OK) {
+                       sr_err("Failed to read the voltage limit for %s.",
+                               channel_name);
+                       return ret;
+               }
+               (*channels)[i].voltage[0] = 0.0;
+               (*channels)[i].voltage[1] = limit_val;
+               (*channels)[i].voltage[2] = 0.01; /* Programming resolution. */
+               (*channels)[i].voltage[3] = 2; /* Spec digits. */
+               (*channels)[i].voltage[4] = 2; /* Encoding digits. */
+
+               ret = sr_scpi_get_double(scpi,
+                       ":SYST:CHAN:INFO:POW?", &limit_val);
+               if (ret != SR_OK) {
+                       sr_err("Failed to read the power limit for %s.",
+                               channel_name);
+                       return ret;
+               }
+               (*channels)[i].power[0] = 0.0;
+               (*channels)[i].power[1] = limit_val;
+               (*channels)[i].power[2] = 0.01; /* Programming resolution. */
+               (*channels)[i].power[3] = 2; /* Spec digits. */
+               (*channels)[i].power[4] = 2; /* Encoding digits. */
+       }
+       *num_channels = *num_channel_groups = channel_count;
+
+       return SR_OK;
+}
+
 /* Rigol DP700 series */
 static const uint32_t rigol_dp700_devopts[] = {
        SR_CONF_CONTINUOUS,
@@ -442,15 +613,26 @@ static const uint32_t hp_6630a_devopts_cg[] = {
        SR_CONF_CURRENT | SR_CONF_GET,
        SR_CONF_VOLTAGE_TARGET | SR_CONF_SET | SR_CONF_LIST,
        SR_CONF_CURRENT_LIMIT | SR_CONF_SET | SR_CONF_LIST,
+       SR_CONF_OVER_VOLTAGE_PROTECTION_ACTIVE | SR_CONF_GET,
        SR_CONF_OVER_VOLTAGE_PROTECTION_THRESHOLD | SR_CONF_SET | SR_CONF_LIST,
        SR_CONF_OVER_CURRENT_PROTECTION_ENABLED | SR_CONF_SET,
+       SR_CONF_OVER_CURRENT_PROTECTION_ACTIVE | SR_CONF_GET,
+       SR_CONF_OVER_TEMPERATURE_PROTECTION_ACTIVE | SR_CONF_GET,
        SR_CONF_REGULATION | SR_CONF_GET,
 };
 
+static const struct channel_spec hp_6632a_ch[] = {
+       { "1", { 0, 20.475, 0.005, 3, 4 }, { 0, 5.1188, 0.00125, 4, 5 }, { 0, 104.80743 }, FREQ_DC_ONLY, { 0, 22, 0.1 }, NO_OCP_LIMITS },
+};
+
 static const struct channel_spec hp_6633a_ch[] = {
        { "1", { 0, 51.188, 0.0125, 3, 4 }, { 0, 2.0475, 0.0005, 4, 5 }, { 0, 104.80743 }, FREQ_DC_ONLY, { 0, 55, 0.25 }, NO_OCP_LIMITS },
 };
 
+static const struct channel_spec hp_6634a_ch[] = {
+       { "1", { 0, 102.38, 0.025, 3, 4 }, { 0, 1.0238, 0.00025, 4, 5 }, { 0, 104.81664 }, FREQ_DC_ONLY, { 0, 110, 0.5 }, NO_OCP_LIMITS },
+};
+
 static const struct channel_group_spec hp_6630a_cg[] = {
        { "1", CH_IDX(0), PPS_OVP | PPS_OCP, SR_MQFLAG_DC },
 };
@@ -462,13 +644,104 @@ static const struct scpi_command hp_6630a_cmd[] = {
        { SCPI_CMD_GET_MEAS_CURRENT, "IOUT?" },
        { SCPI_CMD_SET_VOLTAGE_TARGET, "VSET %.4f" },
        { SCPI_CMD_SET_CURRENT_LIMIT, "ISET %.4f" },
+       { SCPI_CMD_GET_OVER_VOLTAGE_PROTECTION_ACTIVE, "STS?" },
+       { SCPI_CMD_SET_OVER_VOLTAGE_PROTECTION_THRESHOLD, "OVSET %.4f" },
        { SCPI_CMD_SET_OVER_CURRENT_PROTECTION_ENABLE, "OCP 1" },
        { SCPI_CMD_SET_OVER_CURRENT_PROTECTION_DISABLE, "OCP 0" },
-       { SCPI_CMD_SET_OVER_VOLTAGE_PROTECTION_THRESHOLD, "OVSET %.4f" },
+       { SCPI_CMD_GET_OVER_CURRENT_PROTECTION_ACTIVE, "STS?" },
+       { SCPI_CMD_GET_OVER_TEMPERATURE_PROTECTION_ACTIVE, "STS?" },
        { SCPI_CMD_GET_OUTPUT_REGULATION, "STS?" },
        ALL_ZERO
 };
 
+static int hp_6630a_init_acquisition(const struct sr_dev_inst *sdi)
+{
+       struct sr_scpi_dev_inst *scpi;
+
+       scpi = sdi->conn;
+
+       /*
+        * Monitor CV (1), CC+ (2), UR (4), OVP (8), OTP (16), OCP (64) and
+        * CC- (256) bits of the Status Register for the FAULT? query.
+        */
+       return sr_scpi_send(scpi, "UNMASK 607");
+}
+
+static int hp_6630a_update_status(const struct sr_dev_inst *sdi)
+{
+       struct sr_scpi_dev_inst *scpi;
+       int ret;
+       int fault;
+       gboolean cv, cc_pos, unreg, cc_neg;
+       gboolean regulation_changed;
+       char *regulation;
+
+       scpi = sdi->conn;
+
+       /*
+        * Use the FAULT register (only 0->1 transitions), this way multiple set
+        * regulation bits in the STS/ASTS registers are ignored. In rare cases
+        * we will miss some changes (1->0 transitions, e.g. no regulation at all),
+        * but SPS/ASPS doesn't work either, unless all states are stored and
+        * compared to the states in STS/ASTS.
+        * TODO: Use SPoll or SRQ when SCPI over GPIB is used.
+        */
+       ret = sr_scpi_get_int(scpi, "FAULT?", &fault);
+       if (ret != SR_OK)
+               return ret;
+
+       /* OVP */
+       if (fault & (1 << 3))
+               sr_session_send_meta(sdi, SR_CONF_OVER_VOLTAGE_PROTECTION_ACTIVE,
+                       g_variant_new_boolean(fault & (1 << 3)));
+
+       /* OCP */
+       if (fault & (1 << 6))
+               sr_session_send_meta(sdi, SR_CONF_OVER_CURRENT_PROTECTION_ACTIVE,
+                       g_variant_new_boolean(fault & (1 << 6)));
+
+       /* OTP */
+       if (fault & (1 << 4))
+               sr_session_send_meta(sdi, SR_CONF_OVER_TEMPERATURE_PROTECTION_ACTIVE,
+                       g_variant_new_boolean(fault & (1 << 4)));
+
+       /* CV */
+       cv = (fault & (1 << 0));
+       regulation_changed = (fault & (1 << 0));
+       /* CC+ */
+       cc_pos = (fault & (1 << 1));
+       regulation_changed = (fault & (1 << 1)) | regulation_changed;
+       /* UNREG */
+       unreg = (fault & (1 << 2));
+       regulation_changed = (fault & (1 << 2)) | regulation_changed;
+       /* CC- */
+       cc_neg = (fault & (1 << 9));
+       regulation_changed = (fault & (1 << 9)) | regulation_changed;
+
+       if (regulation_changed) {
+               if (cv && !cc_pos && !cc_neg && !unreg)
+                       regulation = "CV";
+               else if (cc_pos && !cv && !cc_neg && !unreg)
+                       regulation = "CC";
+               else if (cc_neg && !cv && !cc_pos && !unreg)
+                       regulation = "CC-";
+               else if (unreg && !cv && !cc_pos && !cc_neg)
+                       regulation = "UR";
+               else if (!cv && !cc_pos && !cc_neg && !unreg)
+                       regulation = "";
+               else {
+                       sr_dbg("Undefined regulation for HP 66xxA "
+                               "(CV=%i, CC+=%i, CC-=%i, UR=%i).",
+                               cv, cc_pos, cc_neg, unreg);
+                       return FALSE;
+               }
+               sr_session_send_meta(sdi, SR_CONF_REGULATION,
+                       g_variant_new_string(regulation));
+       }
+
+       return SR_OK;
+}
+
 /* HP 663xB series */
 static const uint32_t hp_6630b_devopts[] = {
        SR_CONF_CONTINUOUS,
@@ -490,6 +763,22 @@ static const uint32_t hp_6630b_devopts_cg[] = {
        SR_CONF_REGULATION | SR_CONF_GET,
 };
 
+static const struct channel_spec hp_6611c_ch[] = {
+       { "1", { 0, 8.19, 0.002, 3, 4 }, { 0, 5.1188, 0.00125, 4, 5 }, { 0, 41.92297 }, FREQ_DC_ONLY, { 0, 12, 0.06 }, NO_OCP_LIMITS },
+};
+
+static const struct channel_spec hp_6612c_ch[] = {
+       { "1", { 0, 20.475, 0.005, 3, 4 }, { 0, 2.0475, 0.0005, 4, 5 }, { 0, 41.92256 }, FREQ_DC_ONLY, { 0, 22, 0.1 }, NO_OCP_LIMITS },
+};
+
+static const struct channel_spec hp_6613c_ch[] = {
+       { "1", { 0, 51.188, 0.0125, 3, 4 }, { 0, 1.0238, 0.00025, 4, 5 }, { 0, 52.40627 }, FREQ_DC_ONLY, { 0, 55, 0.25 }, NO_OCP_LIMITS },
+};
+
+static const struct channel_spec hp_6614c_ch[] = {
+       { "1", { 0, 102.38, 0.025, 3, 4 }, { 0, 0.5118, 0.000125, 4, 5 }, { 0, 52.39808 }, FREQ_DC_ONLY, { 0, 110, 0.5 }, NO_OCP_LIMITS },
+};
+
 static const struct channel_spec hp_6631b_ch[] = {
        { "1", { 0, 8.19, 0.002, 3, 4 }, { 0, 10.237, 0.00263, 4, 5 }, { 0, 83.84103 }, FREQ_DC_ONLY, { 0, 12, 0.06 }, NO_OCP_LIMITS },
 };
@@ -498,6 +787,10 @@ static const struct channel_spec hp_6632b_ch[] = {
        { "1", { 0, 20.475, 0.005, 3, 4 }, { 0, 5.1188, 0.00132, 4, 5 }, { 0, 104.80743 }, FREQ_DC_ONLY, { 0, 22, 0.1 }, NO_OCP_LIMITS },
 };
 
+static const struct channel_spec hp_66312a_ch[] = {
+       { "1", { 0, 20.475, 0.0001, 4, 5 }, { 0, 2.0475, 0.0001, 4, 5 }, { 0, 41.92256 }, FREQ_DC_ONLY, { 0, 22, 0.01 }, NO_OCP_LIMITS },
+};
+
 static const struct channel_spec hp_66332a_ch[] = {
        { "1", { 0, 20.475, 0.005, 3, 4 }, { 0, 5.1188, 0.00132, 4, 5 }, { 0, 104.80743 }, FREQ_DC_ONLY, { 0, 22, 0.1 }, NO_OCP_LIMITS },
 };
@@ -515,6 +808,10 @@ static const struct channel_group_spec hp_6630b_cg[] = {
 };
 
 static const struct scpi_command hp_6630b_cmd[] = {
+       /*
+        * SCPI_CMD_REMOTE and SCPI_CMD_LOCAL are not used when GPIB is used,
+        * otherwise the device will report (non critical) error 602.
+        */
        { SCPI_CMD_REMOTE, "SYST:REM" },
        { SCPI_CMD_LOCAL, "SYST:LOC" },
        { SCPI_CMD_GET_OUTPUT_ENABLED, "OUTP:STAT?" },
@@ -538,7 +835,7 @@ static const struct scpi_command hp_6630b_cmd[] = {
        ALL_ZERO
 };
 
-static int hp_6630b_init_aquisition(const struct sr_dev_inst *sdi)
+static int hp_6630b_init_acquisition(const struct sr_dev_inst *sdi)
 {
        struct sr_scpi_dev_inst *scpi;
        int ret;
@@ -652,8 +949,8 @@ static int hp_6630b_update_status(const struct sr_dev_inst *sdi)
                /*
                 * Check if output state has changed, due to one of the
                 * questionable states changed.
-                * NOTE: The output state is send even if it hasn't changed, but that
-                * only happends rarely.
+                * NOTE: The output state is sent even if it hasn't changed,
+                * but that only happens rarely.
                 */
                ret = sr_scpi_get_bool(scpi, "OUTP:STAT?", &output_enabled);
                if (ret != SR_OK)
@@ -685,7 +982,7 @@ static int hp_6630b_update_status(const struct sr_dev_inst *sdi)
        }
 
        if (regulation_changed) {
-               if (cv && !cc_pos && !cc_neg &&!unreg)
+               if (cv && !cc_pos && !cc_neg && !unreg)
                        regulation = "CV";
                else if (cc_pos && !cv && !cc_neg && !unreg)
                        regulation = "CC";
@@ -693,11 +990,11 @@ static int hp_6630b_update_status(const struct sr_dev_inst *sdi)
                        regulation = "CC-";
                else if (unreg && !cv && !cc_pos && !cc_neg)
                        regulation = "UR";
-               else if (!cv && !cc_pos && !cc_neg &&!unreg)
-                       /* This happends in case of OCP active */
+               else if (!cv && !cc_pos && !cc_neg && !unreg)
+                       /* This happens in case of OCP active. */
                        regulation = "";
                else {
-                       /* This happends from time to time (CV and CC+ active). */
+                       /* This happens from time to time (CV and CC+ active). */
                        sr_dbg("Undefined regulation for HP 66xxB "
                                "(CV=%i, CC+=%i, CC-=%i, UR=%i).",
                                cv, cc_pos, cc_neg, unreg);
@@ -710,6 +1007,53 @@ static int hp_6630b_update_status(const struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
+/* Owon P4000 series */
+static const uint32_t owon_p4000_devopts[] = {
+       SR_CONF_CONTINUOUS,
+       SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET
+};
+
+static const uint32_t owon_p4000_devopts_cg[] = {
+       SR_CONF_ENABLED | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_VOLTAGE | SR_CONF_GET,
+       SR_CONF_VOLTAGE_TARGET | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
+       SR_CONF_CURRENT | SR_CONF_GET,
+       SR_CONF_CURRENT_LIMIT | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
+       SR_CONF_OVER_VOLTAGE_PROTECTION_THRESHOLD | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_OVER_CURRENT_PROTECTION_THRESHOLD | SR_CONF_GET | SR_CONF_SET,
+};
+
+static const struct channel_spec owon_p4603_ch[] = {
+       { "1", { 0.01, 60, 0.001, 3, 3 }, { 0.001, 3, 0.001, 3, 3 }, { 0, 180, 0, 3, 3 }, FREQ_DC_ONLY, { 0.01, 61, 0.001}, { 0.001, 3.1, 0.001} },
+};
+
+static const struct channel_spec owon_p4305_ch[] = {
+       { "1", { 0.01, 30, 0.001, 3, 3 }, { 0.001, 5, 0.001, 3, 3 }, { 0, 180, 0, 3, 3 }, FREQ_DC_ONLY, { 0.01, 31, 0.001}, { 0.001, 3.1, 0.001} },
+};
+
+static const struct channel_group_spec owon_p4000_cg[] = {
+       { "1", CH_IDX(0), PPS_OVP | PPS_OCP, SR_MQFLAG_DC },
+};
+
+static const struct scpi_command owon_p4000_cmd[] = {
+       { SCPI_CMD_GET_MEAS_VOLTAGE, "MEAS:VOLT?" },
+       { SCPI_CMD_GET_MEAS_CURRENT, "MEAS:CURR?" },
+       { SCPI_CMD_GET_MEAS_POWER, "MEAS:POW?" },
+       { SCPI_CMD_GET_VOLTAGE_TARGET, "VOLT?" },
+       { SCPI_CMD_SET_VOLTAGE_TARGET, "VOLT %.6f" },
+       { SCPI_CMD_GET_CURRENT_LIMIT, "CURR?" },
+       { SCPI_CMD_SET_CURRENT_LIMIT, "CURR %.6f" },
+       { SCPI_CMD_GET_OUTPUT_ENABLED, "OUTP?" },
+       { SCPI_CMD_SET_OUTPUT_ENABLE, "OUTP 1" },
+       { SCPI_CMD_SET_OUTPUT_DISABLE, "OUTP 0" },
+       { SCPI_CMD_GET_OVER_VOLTAGE_PROTECTION_THRESHOLD, "VOLT:LIM?" },
+       { SCPI_CMD_SET_OVER_VOLTAGE_PROTECTION_THRESHOLD, "VOLT:LIM %.6f" },
+       { SCPI_CMD_GET_OVER_CURRENT_PROTECTION_THRESHOLD, "CURR:LIM?" },
+       { SCPI_CMD_SET_OVER_CURRENT_PROTECTION_THRESHOLD, "CURR:LIM %.6f" },
+       ALL_ZERO
+};
+
 /* Philips/Fluke PM2800 series */
 static const uint32_t philips_pm2800_devopts[] = {
        SR_CONF_CONTINUOUS,
@@ -911,6 +1255,83 @@ static const struct scpi_command rs_hmc8043_cmd[] = {
        ALL_ZERO
 };
 
+static const uint32_t rs_hmp4040_devopts[] = {
+       SR_CONF_CONTINUOUS,
+       SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET,
+};
+
+static const uint32_t rs_hmp4040_devopts_cg[] = {
+       SR_CONF_OVER_VOLTAGE_PROTECTION_ENABLED | SR_CONF_GET,
+       SR_CONF_OVER_VOLTAGE_PROTECTION_ACTIVE | SR_CONF_GET,
+       SR_CONF_OVER_VOLTAGE_PROTECTION_THRESHOLD | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_VOLTAGE | SR_CONF_GET,
+       SR_CONF_VOLTAGE_TARGET | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
+       SR_CONF_CURRENT | SR_CONF_GET,
+       SR_CONF_CURRENT_LIMIT | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
+       SR_CONF_ENABLED | SR_CONF_GET | SR_CONF_SET,
+       SR_CONF_OVER_TEMPERATURE_PROTECTION_ACTIVE | SR_CONF_GET,
+       SR_CONF_REGULATION | SR_CONF_GET,
+};
+
+static const struct channel_spec rs_hmp2020_ch[] = {
+       { "1", { 0, 32.050, 0.001, 3, 4 }, { 0.001, 10.01, 0.0002, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+       { "2", { 0, 32.050, 0.001, 3, 4 }, { 0.001,  5.01, 0.0001, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+};
+
+static const struct channel_spec rs_hmp2030_ch[] = {
+       { "1", { 0, 32.050, 0.001, 3, 4 }, { 0.001,  5.01, 0.0001, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+       { "2", { 0, 32.050, 0.001, 3, 4 }, { 0.001,  5.01, 0.0001, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+       { "3", { 0, 32.050, 0.001, 3, 4 }, { 0.001,  5.01, 0.0001, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+};
+
+static const struct channel_spec rs_hmp4040_ch[] = {
+       { "1", { 0, 32.050, 0.001, 3, 4 }, { 0.001, 10.01, 0.0002, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+       { "2", { 0, 32.050, 0.001, 3, 4 }, { 0.001, 10.01, 0.0002, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+       { "3", { 0, 32.050, 0.001, 3, 4 }, { 0.001, 10.01, 0.0002, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+       { "4", { 0, 32.050, 0.001, 3, 4 }, { 0.001, 10.01, 0.0002, 3, 4 }, { 0, 0, 0, 0, 4 }, FREQ_DC_ONLY, NO_OVP_LIMITS, NO_OCP_LIMITS },
+};
+
+static const struct channel_group_spec rs_hmp4040_cg[] = {
+       { "1", CH_IDX(0), PPS_OVP | PPS_OTP, SR_MQFLAG_DC },
+       { "2", CH_IDX(1), PPS_OVP | PPS_OTP, SR_MQFLAG_DC },
+       { "3", CH_IDX(2), PPS_OVP | PPS_OTP, SR_MQFLAG_DC },
+       { "4", CH_IDX(3), PPS_OVP | PPS_OTP, SR_MQFLAG_DC },
+};
+
+/*
+ * Developer's note: Currently unused device commands. Some of them
+ * are not in use because SCPI_CMD codes are not defined yet.
+ *   OUTP:GEN
+ *   VOLT? MAX, CURR? MAX
+ *   VOLT:PROT:CLE (could set SR_CONF_OVER_VOLTAGE_PROTECTION_ACTIVE)
+ *   VOLT:PROT:MODE
+ *   FUSE:STAT, FUSE:TRIP?, FUSE:LINK, FUSE:UNL
+ *   ARB:...
+ *   SYST:LOC, SYST:REM, SYST:RWL, SYST:MIX
+ *   SYST:BEEP:IMM
+ */
+static const struct scpi_command rs_hmp4040_cmd[] = {
+       { SCPI_CMD_REMOTE, "SYST:REM" },
+       { SCPI_CMD_LOCAL, "SYST:LOC" },
+       { SCPI_CMD_SELECT_CHANNEL, "INST:NSEL %s" },
+       { SCPI_CMD_GET_MEAS_VOLTAGE, "MEAS:VOLT?" },
+       { SCPI_CMD_GET_MEAS_CURRENT, "MEAS:CURR?" },
+       { SCPI_CMD_GET_VOLTAGE_TARGET, "VOLT?" },
+       { SCPI_CMD_SET_VOLTAGE_TARGET, "VOLT %.6f" },
+       { SCPI_CMD_GET_CURRENT_LIMIT, "CURR?" },
+       { SCPI_CMD_SET_CURRENT_LIMIT, "CURR %.6f" },
+       { SCPI_CMD_GET_OUTPUT_ENABLED, "OUTP?" },
+       { SCPI_CMD_SET_OUTPUT_ENABLE, "OUTP ON" },
+       { SCPI_CMD_SET_OUTPUT_DISABLE, "OUTP OFF" },
+       { SCPI_CMD_GET_OUTPUT_REGULATION, "STAT:QUES:INST:ISUM%s:COND?" },
+       { SCPI_CMD_GET_OVER_VOLTAGE_PROTECTION_ACTIVE, "VOLT:PROT:TRIP?" },
+       { SCPI_CMD_GET_OVER_VOLTAGE_PROTECTION_THRESHOLD, "VOLT:PROT:LEV?" },
+       { SCPI_CMD_SET_OVER_VOLTAGE_PROTECTION_THRESHOLD, "VOLT:PROT:LEV %.6f" },
+       { SCPI_CMD_GET_OVER_TEMPERATURE_PROTECTION_ACTIVE, "STAT:QUES:INST:ISUM%s:COND?" },
+       ALL_ZERO
+};
+
 SR_PRIV const struct scpi_pps pps_profiles[] = {
        /* Agilent N5763A */
        { "Agilent", "N5763A", SCPI_DIALECT_UNKNOWN, 0,
@@ -920,7 +1341,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(agilent_n5700a_cg),
                agilent_n5700a_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
@@ -932,7 +1353,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(agilent_n5700a_cg),
                agilent_n5700a_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
@@ -944,7 +1365,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(bk_9130_cg),
                bk_9130_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
@@ -956,7 +1377,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(chroma_61604_cg),
                chroma_61604_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
@@ -968,10 +1389,80 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                NULL, 0,
                chroma_62000_cmd,
                .probe_channels = chroma_62000p_probe_channels,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+
+       /*
+        * Envox EEZ PSU Series
+        * The documented identification strings disagree with the behavior
+        * of at least some real units (returning "EEZ"). The first of these
+        * is the documented one, while the second seems to be returned by
+        * firmware v1.02 and earlier.
+        */
+       { "Envox", "^EEZ H24005 ", SCPI_DIALECT_UNKNOWN, 0,
+               ARRAY_AND_SIZE(eez_psu_devopts),
+               ARRAY_AND_SIZE(eez_psu_devopts_cg),
+               NULL, 0,
+               NULL, 0,
+               eez_psu_cmd,
+               .probe_channels = eez_psu_probe_channels,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+       { "EEZ", "^PSU ", SCPI_DIALECT_UNKNOWN, 0,
+               ARRAY_AND_SIZE(eez_psu_devopts),
+               ARRAY_AND_SIZE(eez_psu_devopts_cg),
+               NULL, 0,
+               NULL, 0,
+               eez_psu_cmd,
+               .probe_channels = eez_psu_probe_channels,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+
+       /* Envox EEZ BB3 Series */
+       { "Envox", "^BB3 ", SCPI_DIALECT_UNKNOWN, 0,
+               ARRAY_AND_SIZE(eez_psu_devopts),
+               ARRAY_AND_SIZE(eez_psu_devopts_cg),
+               NULL, 0,
+               NULL, 0,
+               eez_psu_cmd,
+               .probe_channels = eez_psu_probe_channels,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
+       /*
+        * This entry is for testing the HP COMP language with a HP 6632B power
+        * supply switched to the COMP language ("SYST:LANG COMP"). When used,
+        * disable the entry for the HP 6632B below!
+        */
+       /*
+       { "HP", "6632B", SCPI_DIALECT_HP_COMP, 0,
+               ARRAY_AND_SIZE(hp_6630a_devopts),
+               ARRAY_AND_SIZE(hp_6630a_devopts_cg),
+               ARRAY_AND_SIZE(hp_6632a_ch),
+               ARRAY_AND_SIZE(hp_6630a_cg),
+               hp_6630a_cmd,
+               .probe_channels = NULL,
+               hp_6630a_init_acquisition,
+               hp_6630a_update_status,
+       },
+       */
+
+       /* HP 6632A */
+       { "HP", "6632A", SCPI_DIALECT_HP_COMP, 0,
+               ARRAY_AND_SIZE(hp_6630a_devopts),
+               ARRAY_AND_SIZE(hp_6630a_devopts_cg),
+               ARRAY_AND_SIZE(hp_6632a_ch),
+               ARRAY_AND_SIZE(hp_6630a_cg),
+               hp_6630a_cmd,
+               .probe_channels = NULL,
+               hp_6630a_init_acquisition,
+               hp_6630a_update_status,
+       },
+
        /* HP 6633A */
        { "HP", "6633A", SCPI_DIALECT_HP_COMP, 0,
                ARRAY_AND_SIZE(hp_6630a_devopts),
@@ -980,8 +1471,68 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(hp_6630a_cg),
                hp_6630a_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
-               .update_status = NULL,
+               hp_6630a_init_acquisition,
+               hp_6630a_update_status,
+       },
+
+       /* HP 6634A */
+       { "HP", "6634A", SCPI_DIALECT_HP_COMP, 0,
+               ARRAY_AND_SIZE(hp_6630a_devopts),
+               ARRAY_AND_SIZE(hp_6630a_devopts_cg),
+               ARRAY_AND_SIZE(hp_6634a_ch),
+               ARRAY_AND_SIZE(hp_6630a_cg),
+               hp_6630a_cmd,
+               .probe_channels = NULL,
+               hp_6630a_init_acquisition,
+               hp_6630a_update_status,
+       },
+
+       /* HP 6611C */
+       { "HP", "6611C", SCPI_DIALECT_HP_66XXB, PPS_OTP,
+               ARRAY_AND_SIZE(hp_6630b_devopts),
+               ARRAY_AND_SIZE(hp_6630b_devopts_cg),
+               ARRAY_AND_SIZE(hp_6611c_ch),
+               ARRAY_AND_SIZE(hp_6630b_cg),
+               hp_6630b_cmd,
+               .probe_channels = NULL,
+               hp_6630b_init_acquisition,
+               hp_6630b_update_status,
+       },
+
+       /* HP 6612C */
+       { "HP", "6612C", SCPI_DIALECT_HP_66XXB, PPS_OTP,
+               ARRAY_AND_SIZE(hp_6630b_devopts),
+               ARRAY_AND_SIZE(hp_6630b_devopts_cg),
+               ARRAY_AND_SIZE(hp_6612c_ch),
+               ARRAY_AND_SIZE(hp_6630b_cg),
+               hp_6630b_cmd,
+               .probe_channels = NULL,
+               hp_6630b_init_acquisition,
+               hp_6630b_update_status,
+       },
+
+       /* HP 6613C */
+       { "HP", "6613C", SCPI_DIALECT_HP_66XXB, PPS_OTP,
+               ARRAY_AND_SIZE(hp_6630b_devopts),
+               ARRAY_AND_SIZE(hp_6630b_devopts_cg),
+               ARRAY_AND_SIZE(hp_6613c_ch),
+               ARRAY_AND_SIZE(hp_6630b_cg),
+               hp_6630b_cmd,
+               .probe_channels = NULL,
+               hp_6630b_init_acquisition,
+               hp_6630b_update_status,
+       },
+
+       /* HP 6614C */
+       { "HP", "6614C", SCPI_DIALECT_HP_66XXB, PPS_OTP,
+               ARRAY_AND_SIZE(hp_6630b_devopts),
+               ARRAY_AND_SIZE(hp_6630b_devopts_cg),
+               ARRAY_AND_SIZE(hp_6614c_ch),
+               ARRAY_AND_SIZE(hp_6630b_cg),
+               hp_6630b_cmd,
+               .probe_channels = NULL,
+               hp_6630b_init_acquisition,
+               hp_6630b_update_status,
        },
 
        /* HP 6631B */
@@ -992,7 +1543,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(hp_6630b_cg),
                hp_6630b_cmd,
                .probe_channels = NULL,
-               hp_6630b_init_aquisition,
+               hp_6630b_init_acquisition,
                hp_6630b_update_status,
        },
 
@@ -1004,7 +1555,19 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(hp_6630b_cg),
                hp_6630b_cmd,
                .probe_channels = NULL,
-               hp_6630b_init_aquisition,
+               hp_6630b_init_acquisition,
+               hp_6630b_update_status,
+       },
+
+       /* HP 66312A */
+       { "HP", "66312A", SCPI_DIALECT_HP_66XXB, PPS_OTP,
+               ARRAY_AND_SIZE(hp_6630b_devopts),
+               ARRAY_AND_SIZE(hp_6630b_devopts_cg),
+               ARRAY_AND_SIZE(hp_66312a_ch),
+               ARRAY_AND_SIZE(hp_6630b_cg),
+               hp_6630b_cmd,
+               .probe_channels = NULL,
+               hp_6630b_init_acquisition,
                hp_6630b_update_status,
        },
 
@@ -1016,7 +1579,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(hp_6630b_cg),
                hp_6630b_cmd,
                .probe_channels = NULL,
-               hp_6630b_init_aquisition,
+               hp_6630b_init_acquisition,
                hp_6630b_update_status,
        },
 
@@ -1028,7 +1591,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(hp_6630b_cg),
                hp_6630b_cmd,
                .probe_channels = NULL,
-               hp_6630b_init_aquisition,
+               hp_6630b_init_acquisition,
                hp_6630b_update_status,
        },
 
@@ -1040,7 +1603,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(hp_6630b_cg),
                hp_6630b_cmd,
                .probe_channels = NULL,
-               hp_6630b_init_aquisition,
+               hp_6630b_init_acquisition,
                hp_6630b_update_status,
        },
 
@@ -1052,7 +1615,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(rigol_dp700_cg),
                rigol_dp700_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
        { "Rigol", "^DP712$", SCPI_DIALECT_UNKNOWN, 0,
@@ -1062,7 +1625,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(rigol_dp700_cg),
                rigol_dp700_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
@@ -1074,7 +1637,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(rigol_dp820_cg),
                rigol_dp800_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
        { "Rigol", "^DP831A$", SCPI_DIALECT_UNKNOWN, PPS_OTP,
@@ -1084,7 +1647,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(rigol_dp830_cg),
                rigol_dp800_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
        { "Rigol", "^(DP832|DP832A)$", SCPI_DIALECT_UNKNOWN, PPS_OTP,
@@ -1094,7 +1657,29 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(rigol_dp830_cg),
                rigol_dp800_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+
+       /* Owon P4000 series */
+       { "OWON", "^P4305$", SCPI_DIALECT_UNKNOWN, 0,
+               ARRAY_AND_SIZE(owon_p4000_devopts),
+               ARRAY_AND_SIZE(owon_p4000_devopts_cg),
+               ARRAY_AND_SIZE(owon_p4305_ch),
+               ARRAY_AND_SIZE(owon_p4000_cg),
+               owon_p4000_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+       { "OWON", "^P4603$", SCPI_DIALECT_UNKNOWN, 0,
+               ARRAY_AND_SIZE(owon_p4000_devopts),
+               ARRAY_AND_SIZE(owon_p4000_devopts_cg),
+               ARRAY_AND_SIZE(owon_p4603_ch),
+               ARRAY_AND_SIZE(owon_p4000_cg),
+               owon_p4000_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
@@ -1106,7 +1691,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                NULL, 0,
                philips_pm2800_cmd,
                philips_pm2800_probe_channels,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 
@@ -1118,7 +1703,70 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                ARRAY_AND_SIZE(rs_hmc8043_cg),
                rs_hmc8043_cmd,
                .probe_channels = NULL,
-               .init_aquisition = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+
+       /* Hameg / Rohde&Schwarz HMP4000 series */
+       /* TODO Match on regex, pass scpi_pps item to .probe_channels(). */
+       { "HAMEG", "HMP4030", SCPI_DIALECT_HMP, 0,
+               ARRAY_AND_SIZE(rs_hmp4040_devopts),
+               ARRAY_AND_SIZE(rs_hmp4040_devopts_cg),
+               rs_hmp4040_ch, 3,
+               rs_hmp4040_cg, 3,
+               rs_hmp4040_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+       { "HAMEG", "HMP4040", SCPI_DIALECT_HMP, 0,
+               ARRAY_AND_SIZE(rs_hmp4040_devopts),
+               ARRAY_AND_SIZE(rs_hmp4040_devopts_cg),
+               ARRAY_AND_SIZE(rs_hmp4040_ch),
+               ARRAY_AND_SIZE(rs_hmp4040_cg),
+               rs_hmp4040_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+       { "ROHDE&SCHWARZ", "HMP2020", SCPI_DIALECT_HMP, 0,
+               ARRAY_AND_SIZE(rs_hmp4040_devopts),
+               ARRAY_AND_SIZE(rs_hmp4040_devopts_cg),
+               rs_hmp2020_ch, 2,
+               rs_hmp4040_cg, 2,
+               rs_hmp4040_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+       { "ROHDE&SCHWARZ", "HMP2030", SCPI_DIALECT_HMP, 0,
+               ARRAY_AND_SIZE(rs_hmp4040_devopts),
+               ARRAY_AND_SIZE(rs_hmp4040_devopts_cg),
+               rs_hmp2030_ch, 3,
+               rs_hmp4040_cg, 3,
+               rs_hmp4040_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+       { "ROHDE&SCHWARZ", "HMP4030", SCPI_DIALECT_HMP, 0,
+               ARRAY_AND_SIZE(rs_hmp4040_devopts),
+               ARRAY_AND_SIZE(rs_hmp4040_devopts_cg),
+               rs_hmp4040_ch, 3,
+               rs_hmp4040_cg, 3,
+               rs_hmp4040_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
+               .update_status = NULL,
+       },
+       { "ROHDE&SCHWARZ", "HMP4040", SCPI_DIALECT_HMP, 0,
+               ARRAY_AND_SIZE(rs_hmp4040_devopts),
+               ARRAY_AND_SIZE(rs_hmp4040_devopts_cg),
+               ARRAY_AND_SIZE(rs_hmp4040_ch),
+               ARRAY_AND_SIZE(rs_hmp4040_cg),
+               rs_hmp4040_cmd,
+               .probe_channels = NULL,
+               .init_acquisition = NULL,
                .update_status = NULL,
        },
 };