X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=src%2Fhardware%2Fhameg-hmo%2Fapi.c;h=fb03d04cdd86ecc615d71235dd28f1471a3abe16;hb=41812aca436805b0614f2a8f31cf2f8ce494aea0;hp=ff045f16425e61ad091be85868c2111845cb39fe;hpb=155b680da482cea2381becb73c51cfb838bff31e;p=libsigrok.git diff --git a/src/hardware/hameg-hmo/api.c b/src/hardware/hameg-hmo/api.c index ff045f16..fb03d04c 100644 --- a/src/hardware/hameg-hmo/api.c +++ b/src/hardware/hameg-hmo/api.c @@ -23,13 +23,16 @@ #define SERIALCOMM "115200/8n1/flow=1" SR_PRIV struct sr_dev_driver hameg_hmo_driver_info; -static struct sr_dev_driver *di = &hameg_hmo_driver_info; static const char *manufacturers[] = { "HAMEG", }; -static const int32_t hwopts[] = { +static const uint32_t drvopts[] = { + SR_CONF_OSCILLOSCOPE, +}; + +static const uint32_t scanopts[] = { SR_CONF_CONN, SR_CONF_SERIALCOMM, }; @@ -41,7 +44,7 @@ enum { CG_DIGITAL, }; -static int init(struct sr_context *sr_ctx) +static int init(struct sr_dev_driver *di, struct sr_context *sr_ctx) { return std_init(sr_ctx, di, LOG_PREFIX); } @@ -75,21 +78,22 @@ static struct sr_dev_inst *hmo_probe_serial_device(struct sr_scpi_dev_inst *scpi if (check_manufacturer(hw_info->manufacturer) != SR_OK) goto fail; - if (!(sdi = sr_dev_inst_new(0, SR_ST_ACTIVE, - hw_info->manufacturer, hw_info->model, - hw_info->firmware_version))) { - goto fail; - } + sdi = g_malloc0(sizeof(struct sr_dev_inst)); + sdi->status = SR_ST_ACTIVE; + sdi->vendor = g_strdup(hw_info->manufacturer); + sdi->model = g_strdup(hw_info->model); + sdi->version = g_strdup(hw_info->firmware_version); + sdi->serial_num = g_strdup(hw_info->serial_number); + sdi->driver = &hameg_hmo_driver_info; + sdi->inst_type = SR_INST_SCPI; + sdi->conn = scpi; + sr_scpi_hw_info_free(hw_info); hw_info = NULL; - if (!(devc = g_try_malloc0(sizeof(struct dev_context)))) - goto fail; + devc = g_malloc0(sizeof(struct dev_context)); - sdi->driver = di; sdi->priv = devc; - sdi->inst_type = SR_INST_SCPI; - sdi->conn = scpi; if (hmo_init_device(sdi) != SR_OK) goto fail; @@ -105,48 +109,36 @@ fail: sr_scpi_hw_info_free(hw_info); if (sdi) sr_dev_inst_free(sdi); - if (devc) - g_free(devc); + g_free(devc); return NULL; } -static GSList *scan(GSList *options) +static GSList *scan(struct sr_dev_driver *di, GSList *options) { - return sr_scpi_scan(di->priv, options, hmo_probe_serial_device); + return sr_scpi_scan(di->context, options, hmo_probe_serial_device); } -static GSList *dev_list(void) +static GSList *dev_list(const struct sr_dev_driver *di) { - return ((struct drv_context *)(di->priv))->instances; + return ((struct drv_context *)(di->context))->instances; } static void clear_helper(void *priv) { - unsigned int i; struct dev_context *devc; - struct scope_config *model; devc = priv; - model = devc->model_config; hmo_scope_state_free(devc->model_state); - for (i = 0; i < model->analog_channels; ++i) - g_slist_free(devc->analog_groups[i].channels); - - for (i = 0; i < model->digital_pods; ++i) { - g_slist_free(devc->digital_groups[i].channels); - g_free(devc->digital_groups[i].name); - } - g_free(devc->analog_groups); g_free(devc->digital_groups); g_free(devc); } -static int dev_clear(void) +static int dev_clear(const struct sr_dev_driver *di) { return std_dev_clear(di, clear_helper); } @@ -176,9 +168,9 @@ static int dev_close(struct sr_dev_inst *sdi) return SR_OK; } -static int cleanup(void) +static int cleanup(const struct sr_dev_driver *di) { - dev_clear(); + dev_clear(di); return SR_OK; } @@ -187,7 +179,7 @@ static int check_channel_group(struct dev_context *devc, const struct sr_channel_group *cg) { unsigned int i; - struct scope_config *model; + const struct scope_config *model; model = devc->model_config; @@ -195,11 +187,11 @@ static int check_channel_group(struct dev_context *devc, return CG_NONE; for (i = 0; i < model->analog_channels; ++i) - if (cg == &devc->analog_groups[i]) + if (cg == devc->analog_groups[i]) return CG_ANALOG; for (i = 0; i < model->digital_pods; ++i) - if (cg == &devc->digital_groups[i]) + if (cg == devc->digital_groups[i]) return CG_DIGITAL; sr_err("Invalid channel group specified."); @@ -207,13 +199,13 @@ static int check_channel_group(struct dev_context *devc, return CG_INVALID; } -static int config_get(int key, GVariant **data, const struct sr_dev_inst *sdi, +static int config_get(uint32_t key, GVariant **data, const struct sr_dev_inst *sdi, const struct sr_channel_group *cg) { int ret, cg_type; unsigned int i; struct dev_context *devc; - struct scope_config *model; + const struct scope_config *model; struct scope_state *state; if (!sdi || !(devc = sdi->priv)) @@ -227,7 +219,7 @@ static int config_get(int key, GVariant **data, const struct sr_dev_inst *sdi, state = devc->model_state; switch (key) { - case SR_CONF_NUM_TIMEBASE: + case SR_CONF_NUM_HDIV: *data = g_variant_new_int32(model->num_xdivs); ret = SR_OK; break; @@ -242,7 +234,7 @@ static int config_get(int key, GVariant **data, const struct sr_dev_inst *sdi, return SR_ERR_CHANNEL_GROUP; } else if (cg_type == CG_ANALOG) { for (i = 0; i < model->analog_channels; ++i) { - if (cg != &devc->analog_groups[i]) + if (cg != devc->analog_groups[i]) continue; *data = g_variant_new_int32(model->num_ydivs); ret = SR_OK; @@ -259,7 +251,7 @@ static int config_get(int key, GVariant **data, const struct sr_dev_inst *sdi, return SR_ERR_CHANNEL_GROUP; } else if (cg_type == CG_ANALOG) { for (i = 0; i < model->analog_channels; ++i) { - if (cg != &devc->analog_groups[i]) + if (cg != devc->analog_groups[i]) continue; *data = g_variant_new("(tt)", (*model->vdivs)[state->analog_channels[i].vdiv][0], @@ -290,7 +282,7 @@ static int config_get(int key, GVariant **data, const struct sr_dev_inst *sdi, return SR_ERR_CHANNEL_GROUP; } else if (cg_type == CG_ANALOG) { for (i = 0; i < model->analog_channels; ++i) { - if (cg != &devc->analog_groups[i]) + if (cg != devc->analog_groups[i]) continue; *data = g_variant_new_string((*model->coupling_options)[state->analog_channels[i].coupling]); ret = SR_OK; @@ -331,14 +323,14 @@ static GVariant *build_tuples(const uint64_t (*array)[][2], unsigned int n) return g_variant_builder_end(&gvb); } -static int config_set(int key, GVariant *data, const struct sr_dev_inst *sdi, +static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sdi, const struct sr_channel_group *cg) { int ret, cg_type; unsigned int i, j; char command[MAX_COMMAND_SIZE], float_str[30]; struct dev_context *devc; - struct scope_config *model; + const struct scope_config *model; struct scope_state *state; const char *tmp; uint64_t p, q; @@ -389,7 +381,7 @@ static int config_set(int key, GVariant *data, const struct sr_dev_inst *sdi, q != (*model->vdivs)[i][1]) continue; for (j = 1; j <= model->analog_channels; ++j) { - if (cg != &devc->analog_groups[j - 1]) + if (cg != devc->analog_groups[j - 1]) continue; state->analog_channels[j - 1].vdiv = i; g_ascii_formatd(float_str, sizeof(float_str), "%E", (float) p / q); @@ -471,7 +463,7 @@ static int config_set(int key, GVariant *data, const struct sr_dev_inst *sdi, if (strcmp(tmp, (*model->coupling_options)[i]) != 0) continue; for (j = 1; j <= model->analog_channels; ++j) { - if (cg != &devc->analog_groups[j - 1]) + if (cg != devc->analog_groups[j - 1]) continue; state->analog_channels[j-1].coupling = i; @@ -503,34 +495,40 @@ static int config_set(int key, GVariant *data, const struct sr_dev_inst *sdi, return ret; } -static int config_list(int key, GVariant **data, const struct sr_dev_inst *sdi, +static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *sdi, const struct sr_channel_group *cg) { - int cg_type; - struct dev_context *devc; - struct scope_config *model; + int cg_type = CG_NONE; + struct dev_context *devc = NULL; + const struct scope_config *model = NULL; - if (!sdi || !(devc = sdi->priv)) - return SR_ERR_ARG; - - if ((cg_type = check_channel_group(devc, cg)) == CG_INVALID) - return SR_ERR; + if (sdi && (devc = sdi->priv)) { + if ((cg_type = check_channel_group(devc, cg)) == CG_INVALID) + return SR_ERR; - model = devc->model_config; + model = devc->model_config; + } switch (key) { + case SR_CONF_SCAN_OPTIONS: + *data = g_variant_new_fixed_array(G_VARIANT_TYPE_UINT32, + scanopts, ARRAY_SIZE(scanopts), sizeof(uint32_t)); + break; case SR_CONF_DEVICE_OPTIONS: if (cg_type == CG_NONE) { - *data = g_variant_new_fixed_array(G_VARIANT_TYPE_INT32, - model->hw_caps, model->num_hwcaps, - sizeof(int32_t)); + if (model) + *data = g_variant_new_fixed_array(G_VARIANT_TYPE_UINT32, + model->devopts, model->num_devopts, sizeof(uint32_t)); + else + *data = g_variant_new_fixed_array(G_VARIANT_TYPE_UINT32, + drvopts, ARRAY_SIZE(drvopts), sizeof(uint32_t)); } else if (cg_type == CG_ANALOG) { - *data = g_variant_new_fixed_array(G_VARIANT_TYPE_INT32, - model->analog_hwcaps, model->num_analog_hwcaps, - sizeof(int32_t)); + *data = g_variant_new_fixed_array(G_VARIANT_TYPE_UINT32, + model->analog_devopts, model->num_analog_devopts, + sizeof(uint32_t)); } else { - *data = g_variant_new_fixed_array(G_VARIANT_TYPE_INT32, - NULL, 0, sizeof(int32_t)); + *data = g_variant_new_fixed_array(G_VARIANT_TYPE_UINT32, + NULL, 0, sizeof(uint32_t)); } break; case SR_CONF_COUPLING: @@ -540,14 +538,20 @@ static int config_list(int key, GVariant **data, const struct sr_dev_inst *sdi, g_strv_length((char **)*model->coupling_options)); break; case SR_CONF_TRIGGER_SOURCE: + if (!model) + return SR_ERR_ARG; *data = g_variant_new_strv(*model->trigger_sources, g_strv_length((char **)*model->trigger_sources)); break; case SR_CONF_TRIGGER_SLOPE: + if (!model) + return SR_ERR_ARG; *data = g_variant_new_strv(*model->trigger_slopes, g_strv_length((char **)*model->trigger_slopes)); break; case SR_CONF_TIMEBASE: + if (!model) + return SR_ERR_ARG; *data = build_tuples(model->timebases, model->num_timebases); break; case SR_CONF_VDIV: @@ -567,7 +571,7 @@ SR_PRIV int hmo_request_data(const struct sr_dev_inst *sdi) char command[MAX_COMMAND_SIZE]; struct sr_channel *ch; struct dev_context *devc; - struct scope_config *model; + const struct scope_config *model; devc = sdi->priv; model = devc->model_config; @@ -635,7 +639,7 @@ static int hmo_setup_channels(const struct sr_dev_inst *sdi) gboolean *pod_enabled, setup_changed; char command[MAX_COMMAND_SIZE]; struct scope_state *state; - struct scope_config *model; + const struct scope_config *model; struct sr_channel *ch; struct dev_context *devc; struct sr_scpi_dev_inst *scpi; @@ -723,6 +727,9 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, void *cb_data) devc = sdi->priv; digital_added = FALSE; + g_slist_free(devc->enabled_channels); + devc->enabled_channels = NULL; + for (l = sdi->channels; l; l = l->next) { ch = l->data; if (!ch->enabled) @@ -802,5 +809,5 @@ SR_PRIV struct sr_dev_driver hameg_hmo_driver_info = { .dev_close = dev_close, .dev_acquisition_start = dev_acquisition_start, .dev_acquisition_stop = dev_acquisition_stop, - .priv = NULL, + .context = NULL, };