BLAHAJ
/
libsigrok
10
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

lecroy-xstream: Minor whitespace and consistency fixes.

This commit is contained in:
Uwe Hermann 2017-03-02 20:05:52 +01:00
parent 9de47e9e71
commit ea257cdc23
3 changed files with 51 additions and 58 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

43
src/hardware/lecroy-xstream/api.c
View File

@ -140,8 +140,7 @@ static int dev_close(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)
const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
{
int ret;
unsigned int i;
@ -158,6 +157,7 @@ static int config_get(uint32_t key, GVariant **data,
model = devc->model_config;
state = devc->model_state;
*data = NULL;
switch (key) {
case SR_CONF_NUM_HDIV:
*data = g_variant_new_int32(model->num_xdivs);
@ -165,8 +165,8 @@ static int config_get(uint32_t key, GVariant **data,
break;
case SR_CONF_TIMEBASE:
*data = g_variant_new("(tt)",
model->timebases[state->timebase].p,
model->timebases[state->timebase].q);
model->timebases[state->timebase].p,
model->timebases[state->timebase].q);
ret = SR_OK;
break;
case SR_CONF_NUM_VDIV:
@ -182,8 +182,8 @@ static int config_get(uint32_t key, GVariant **data,
if (cg != devc->analog_groups[i])
continue;
*data = g_variant_new("(tt)",
model->vdivs[state->analog_channels[i].vdiv].p,
model->vdivs[state->analog_channels[i].vdiv].q);
model->vdivs[state->analog_channels[i].vdiv].p,
model->vdivs[state->analog_channels[i].vdiv].q);
ret = SR_OK;
}
break;
@ -200,11 +200,9 @@ static int config_get(uint32_t key, GVariant **data,
ret = SR_OK;
break;
case SR_CONF_COUPLING:
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;
}
@ -220,6 +218,7 @@ static int config_get(uint32_t key, GVariant **data,
default:
ret = SR_ERR_NA;
}
return ret;
}
@ -242,8 +241,8 @@ static GVariant *build_tuples(const struct sr_rational *array, unsigned int n)
return g_variant_builder_end(&gvb);
}
static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sdi,
const struct sr_channel_group *cg)
static int config_set(uint32_t key, GVariant *data,
const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
{
int ret;
unsigned int i, j;
@ -358,7 +357,6 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
}
break;
case SR_CONF_COUPLING:
tmp = g_variant_get_string(data, NULL);
for (i = 0; (*model->coupling_options)[i]; i++) {
@ -367,7 +365,7 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
for (j = 1; j <= model->analog_channels; j++) {
if (cg != devc->analog_groups[j - 1])
continue;
state->analog_channels[j-1].coupling = i;
state->analog_channels[j - 1].coupling = i;
g_snprintf(command, sizeof(command),
"C%d:COUPLING %s", j, tmp);
@ -396,11 +394,12 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
return ret;
}
static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *sdi,
const struct sr_channel_group *cg)
static int config_list(uint32_t key, GVariant **data,
const struct sr_dev_inst *sdi, const struct sr_channel_group *cg)
{
struct dev_context *devc = NULL;
const struct scope_config *model = NULL;
(void)cg;
if (sdi) {
@ -416,15 +415,12 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
case SR_CONF_DEVICE_OPTIONS:
if (!cg) {
*data = g_variant_new_fixed_array(G_VARIANT_TYPE_UINT32,
model->devopts,
model->num_devopts,
sizeof(uint32_t));
model->devopts, model->num_devopts, sizeof(uint32_t));
break;
}
*data = g_variant_new_fixed_array(G_VARIANT_TYPE_UINT32,
model->analog_devopts,
model->num_analog_devopts,
sizeof(uint32_t));
model->analog_devopts, model->num_analog_devopts,
sizeof(uint32_t));
break;
case SR_CONF_COUPLING:
*data = g_variant_new_strv(*model->coupling_options,
@ -472,7 +468,7 @@ SR_PRIV int lecroy_xstream_request_data(const struct sr_dev_inst *sdi)
return SR_ERR;
g_snprintf(command, sizeof(command),
"COMM_FORMAT DEF9,WORD,BIN;C%d:WAVEFORM?", ch->index+1);
"COMM_FORMAT DEF9,WORD,BIN;C%d:WAVEFORM?", ch->index + 1);
return sr_scpi_send(sdi->conn, command);
}
@ -498,7 +494,7 @@ static int lecroy_setup_channels(const struct sr_dev_inst *sdi)
if (ch->enabled == state->analog_channels[ch->index].state)
break;
g_snprintf(command, sizeof(command), "C%d:TRACE %s",
ch->index+1, ch->enabled ? "ON" : "OFF");
ch->index + 1, ch->enabled ? "ON" : "OFF");
if (sr_scpi_send(scpi, command) != SR_OK)
return SR_ERR;
@ -576,6 +572,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
free_enabled:
g_slist_free(devc->enabled_channels);
devc->enabled_channels = NULL;
return ret;
}

65
src/hardware/lecroy-xstream/protocol.c
View File

@ -99,7 +99,7 @@ static const uint32_t lecroy_analog_devopts[] = {
static const char *lecroy_coupling_options[] = {
"A1M", // AC with 1 MOhm termination
"D50", // DC with 50 Ohm termination
"D50", // DC with 50 Ohm termination
"D1M", // DC with 1 MOhm termination
"GND",
"OVL",
@ -243,7 +243,7 @@ static void scope_state_dump(const struct scope_config *config,
}
tmp = sr_period_string(((float)config->timebases[state->timebase].q) /
((float)config->timebases[state->timebase].p));
((float)config->timebases[state->timebase].p));
sr_info("Current timebase: %s", tmp);
g_free(tmp);
@ -258,8 +258,7 @@ static void scope_state_dump(const struct scope_config *config,
}
static int scope_state_get_array_option(const char *resp,
const char *(*array)[],
int *result)
const char *(*array)[], int *result)
{
unsigned int i;
@ -293,7 +292,7 @@ static int array_float_get(gchar *value, const struct sr_rational *aval,
return SR_ERR;
for (int i = 0; i < array_len; i++) {
if (sr_rational_eq(&rval, aval+i)) {
if (sr_rational_eq(&rval, aval + i)) {
*result = i;
return SR_OK;
}
@ -311,13 +310,13 @@ static int analog_channel_state_get(struct sr_scpi_dev_inst *scpi,
char *tmp_str;
for (i = 0; i < config->analog_channels; i++) {
g_snprintf(command, sizeof(command), "C%d:TRACE?", i+1);
g_snprintf(command, sizeof(command), "C%d:TRACE?", i + 1);
if (sr_scpi_get_bool(scpi, command,
&state->analog_channels[i].state) != SR_OK)
&state->analog_channels[i].state) != SR_OK)
return SR_ERR;
g_snprintf(command, sizeof(command), "C%d:VDIV?", i+1);
g_snprintf(command, sizeof(command), "C%d:VDIV?", i + 1);
if (sr_scpi_get_string(scpi, command, &tmp_str) != SR_OK)
return SR_ERR;
@ -332,19 +331,19 @@ static int analog_channel_state_get(struct sr_scpi_dev_inst *scpi,
g_free(tmp_str);
state->analog_channels[i].vdiv = j;
g_snprintf(command, sizeof(command), "C%d:OFFSET?", i+1);
g_snprintf(command, sizeof(command), "C%d:OFFSET?", i + 1);
if (sr_scpi_get_float(scpi, command, &state->analog_channels[i].vertical_offset) != SR_OK)
return SR_ERR;
g_snprintf(command, sizeof(command), "C%d:COUPLING?", i+1);
g_snprintf(command, sizeof(command), "C%d:COUPLING?", i + 1);
if (sr_scpi_get_string(scpi, command, &tmp_str) != SR_OK)
return SR_ERR;
if (scope_state_get_array_option(tmp_str, config->coupling_options,
&state->analog_channels[i].coupling) != SR_OK)
&state->analog_channels[i].coupling) != SR_OK)
return SR_ERR;
g_free(tmp_str);
@ -365,12 +364,12 @@ SR_PRIV int lecroy_xstream_update_sample_rate(const struct sr_dev_inst *sdi)
config = devc->model_config;
if (sr_scpi_get_float(sdi->conn, "MEMORY_SIZE?", &memsize) != SR_OK)
return SR_ERR;
return SR_ERR;
if (sr_scpi_get_float(sdi->conn, "TIME_DIV?", &timediv) != SR_OK)
return SR_ERR;
state->sample_rate = 1/((timediv * config->num_xdivs) / memsize);
state->sample_rate = 1 / ((timediv * config->num_xdivs) / memsize);
return SR_OK;
}
@ -378,8 +377,8 @@ SR_PRIV int lecroy_xstream_update_sample_rate(const struct sr_dev_inst *sdi)
SR_PRIV int lecroy_xstream_state_get(struct sr_dev_inst *sdi)
{
struct dev_context *devc;
struct scope_state *state
; const struct scope_config *config;
struct scope_state *state;
const struct scope_config *config;
unsigned int i;
char *tmp_str, *tmp_str2, *tmpp, *p, *key;
char command[MAX_COMMAND_SIZE];
@ -412,7 +411,7 @@ SR_PRIV int lecroy_xstream_state_get(struct sr_dev_inst *sdi)
key = tmpp = NULL;
tmp_str2 = tmp_str;
i = 0;
while((p = strtok_r(tmp_str2, ",", &tmpp))) {
while ((p = strtok_r(tmp_str2, ",", &tmpp))) {
tmp_str2 = NULL;
if (i == 0) {
/* trigger type */
@ -426,11 +425,9 @@ SR_PRIV int lecroy_xstream_state_get(struct sr_dev_inst *sdi)
i++;
}
if (!trig_source || scope_state_get_array_option(trig_source, config->trigger_sources,
&state->trigger_source) != SR_OK)
if (!trig_source || scope_state_get_array_option(trig_source, config->trigger_sources, &state->trigger_source) != SR_OK)
return SR_ERR;
g_snprintf(command, sizeof(command), "%s:TRIG_SLOPE?", trig_source);
if (sr_scpi_get_string(sdi->conn, command, &tmp_str) != SR_OK)
return SR_ERR;
@ -493,22 +490,21 @@ SR_PRIV int lecroy_xstream_init_device(struct sr_dev_inst *sdi)
}
if (model_index == -1) {
sr_dbg("Unsupported LECROY device.");
sr_dbg("Unsupported LeCroy device.");
return SR_ERR_NA;
}
devc->analog_groups = g_malloc0(sizeof(struct sr_channel_group*) *
scope_models[model_index].analog_channels);
scope_models[model_index].analog_channels);
/* Add analog channels. */
for (i = 0; i < scope_models[model_index].analog_channels; i++) {
g_snprintf(command, sizeof(command), "C%d:TRACE?", i+1);
g_snprintf(command, sizeof(command), "C%d:TRACE?", i + 1);
if (sr_scpi_get_bool(sdi->conn, command, &channel_enabled) != SR_OK)
return SR_ERR;
g_snprintf(command, sizeof(command), "C%d:VDIV?", i+1);
g_snprintf(command, sizeof(command), "C%d:VDIV?", i + 1);
ch = sr_channel_new(sdi, i, SR_CHANNEL_ANALOG, channel_enabled,
(*scope_models[model_index].analog_names)[i]);
@ -546,19 +542,18 @@ static int lecroy_waveform_2_x_to_analog(GByteArray *data,
int16_t *waveform_data;
unsigned int i, num_samples;
data_float = g_malloc(desc->version_2_x.wave_array_count * sizeof(float));
data_float = g_malloc(desc->version_2_x.wave_array_count * sizeof(float));
num_samples = desc->version_2_x.wave_array_count;
waveform_data = (int16_t *)(data->data +
+ desc->version_2_x.wave_descriptor_length
+ desc->version_2_x.user_text_len);
for(i = 0; i < num_samples; i++)
for (i = 0; i < num_samples; i++)
data_float[i] = (float)waveform_data[i]
* desc->version_2_x.vertical_gain
+ desc->version_2_x.vertical_offset;
analog->data = data_float;
analog->num_samples = num_samples;
@ -578,13 +573,14 @@ static int lecroy_waveform_2_x_to_analog(GByteArray *data,
meaning->mq = SR_MQ_CURRENT;
meaning->unit = SR_UNIT_AMPERE;
} else {
/* default to voltage */
/* Default to voltage. */
meaning->mq = SR_MQ_VOLTAGE;
meaning->unit = SR_UNIT_VOLT;
}
meaning->mqflags = 0;
spec->spec_digits = 3;
return SR_OK;
}
@ -603,7 +599,9 @@ static int lecroy_waveform_to_analog(GByteArray *data,
return lecroy_waveform_2_x_to_analog(data, desc, analog);
}
sr_err("Waveformat template '%.16s' not supported\n", desc->template_name);
sr_err("Waveformat template '%.16s' not supported.",
desc->template_name);
return SR_ERR;
}
@ -619,6 +617,7 @@ SR_PRIV int lecroy_xstream_receive_data(int fd, int revents, void *cb_data)
struct sr_analog_meaning meaning;
struct sr_analog_spec spec;
char buf[8];
(void)fd;
(void)revents;
@ -643,11 +642,10 @@ SR_PRIV int lecroy_xstream_receive_data(int fd, int revents, void *cb_data)
if (ch->type != SR_CHANNEL_ANALOG)
return SR_ERR;
/*
* Pass on the received data of the channel(s).
*/
/* Pass on the received data of the channel(s). */
if (sr_scpi_read_data(sdi->conn, buf, 4) != 4) {
sr_err("reading header failed\n");
sr_err("Reading header failed.");
return TRUE;
}
@ -675,7 +673,6 @@ SR_PRIV int lecroy_xstream_receive_data(int fd, int revents, void *cb_data)
g_slist_free(meaning.channels);
g_free(analog.data);
/*
* Advance to the next enabled channel. When data for all enabled
* channels was received, then flush potentially queued logic data,

1
src/hardware/lecroy-xstream/protocol.h
View File

@ -94,7 +94,6 @@ struct dev_context {
uint64_t num_frames;
uint64_t frame_limit;
};
SR_PRIV int lecroy_xstream_init_device(struct sr_dev_inst *sdi);