X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=hardware%2Flink-mso19%2Flink-mso19.c;h=5eaf9bd31d23f22d34ff6c6fbba9a47ef138cb5b;hb=d261dbbfcc73;hp=138ffe970125de629fd56b528c716552fc9cd045;hpb=5c64390e5ac8a0052d8b7eeb49c781d86027c814;p=libsigrok.git diff --git a/hardware/link-mso19/link-mso19.c b/hardware/link-mso19/link-mso19.c index 138ffe97..5eaf9bd3 100644 --- a/hardware/link-mso19/link-mso19.c +++ b/hardware/link-mso19/link-mso19.c @@ -37,7 +37,7 @@ #define NUM_PROBES 8 -static int capabilities[] = { +static int hwcaps[] = { SR_HWCAP_LOGIC_ANALYZER, // SR_HWCAP_OSCILLOSCOPE, // SR_HWCAP_PAT_GENERATOR, @@ -48,6 +48,11 @@ static int capabilities[] = { 0, }; +/* + * Probes are numbered 0 to 7. + * + * See also: http://www.linkinstruments.com/images/mso19_1113.gif + */ static const char *probe_names[NUM_PROBES + 1] = { "0", "1", @@ -91,10 +96,10 @@ static struct sr_samplerates samplerates = { supported_samplerates, }; -static GSList *device_instances = NULL; +static GSList *dev_insts = NULL; -static int mso_send_control_message(struct sr_device_instance *sdi, - uint16_t payload[], int n) +static int mso_send_control_message(struct sr_dev_inst *sdi, + uint16_t payload[], int n) { int fd = sdi->serial->fd; int i, w, ret, s = n * 2 + sizeof(mso_head) + sizeof(mso_foot); @@ -137,53 +142,52 @@ ret: return ret; } -static int mso_reset_adc(struct sr_device_instance *sdi) +static int mso_reset_adc(struct sr_dev_inst *sdi) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; uint16_t ops[2]; - ops[0] = mso_trans(REG_CTL1, (mso->ctlbase1 | BIT_CTL1_RESETADC)); - ops[1] = mso_trans(REG_CTL1, mso->ctlbase1); - mso->ctlbase1 |= BIT_CTL1_ADC_UNKNOWN4; + ops[0] = mso_trans(REG_CTL1, (ctx->ctlbase1 | BIT_CTL1_RESETADC)); + ops[1] = mso_trans(REG_CTL1, ctx->ctlbase1); + ctx->ctlbase1 |= BIT_CTL1_ADC_UNKNOWN4; - sr_dbg("Requesting ADC reset"); + sr_dbg("mso19: Requesting ADC reset"); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_reset_fsm(struct sr_device_instance *sdi) +static int mso_reset_fsm(struct sr_dev_inst *sdi) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; uint16_t ops[1]; - mso->ctlbase1 |= BIT_CTL1_RESETFSM; - ops[0] = mso_trans(REG_CTL1, mso->ctlbase1); + ctx->ctlbase1 |= BIT_CTL1_RESETFSM; + ops[0] = mso_trans(REG_CTL1, ctx->ctlbase1); - sr_dbg("Requesting ADC reset"); + sr_dbg("mso19: Requesting ADC reset"); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_toggle_led(struct sr_device_instance *sdi, int state) +static int mso_toggle_led(struct sr_dev_inst *sdi, int state) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; uint16_t ops[1]; - mso->ctlbase1 &= ~BIT_CTL1_LED; + ctx->ctlbase1 &= ~BIT_CTL1_LED; if (state) - mso->ctlbase1 |= BIT_CTL1_LED; - ops[0] = mso_trans(REG_CTL1, mso->ctlbase1); + ctx->ctlbase1 |= BIT_CTL1_LED; + ops[0] = mso_trans(REG_CTL1, ctx->ctlbase1); - sr_dbg("Requesting LED toggle"); + sr_dbg("mso19: Requesting LED toggle"); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_check_trigger(struct sr_device_instance *sdi, - uint8_t *info) +static int mso_check_trigger(struct sr_dev_inst *sdi, uint8_t *info) { uint16_t ops[] = { mso_trans(REG_TRIGGER, 0) }; char buf[1]; int ret; - sr_dbg("Requesting trigger state"); + sr_dbg("mso19: Requesting trigger state"); ret = mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); if (info == NULL || ret != SR_OK) return ret; @@ -193,108 +197,107 @@ static int mso_check_trigger(struct sr_device_instance *sdi, ret = SR_ERR; *info = buf[0]; - sr_dbg("Trigger state is: 0x%x", *info); + sr_dbg("mso19: Trigger state is: 0x%x", *info); return ret; } -static int mso_read_buffer(struct sr_device_instance *sdi) +static int mso_read_buffer(struct sr_dev_inst *sdi) { uint16_t ops[] = { mso_trans(REG_BUFFER, 0) }; - sr_dbg("Requesting buffer dump"); + sr_dbg("mso19: Requesting buffer dump"); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_arm(struct sr_device_instance *sdi) +static int mso_arm(struct sr_dev_inst *sdi) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; uint16_t ops[] = { - mso_trans(REG_CTL1, mso->ctlbase1 | BIT_CTL1_RESETFSM), - mso_trans(REG_CTL1, mso->ctlbase1 | BIT_CTL1_ARM), - mso_trans(REG_CTL1, mso->ctlbase1), + mso_trans(REG_CTL1, ctx->ctlbase1 | BIT_CTL1_RESETFSM), + mso_trans(REG_CTL1, ctx->ctlbase1 | BIT_CTL1_ARM), + mso_trans(REG_CTL1, ctx->ctlbase1), }; - sr_dbg("Requesting trigger arm"); + sr_dbg("mso19: Requesting trigger arm"); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_force_capture(struct sr_device_instance *sdi) +static int mso_force_capture(struct sr_dev_inst *sdi) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; uint16_t ops[] = { - mso_trans(REG_CTL1, mso->ctlbase1 | 8), - mso_trans(REG_CTL1, mso->ctlbase1), + mso_trans(REG_CTL1, ctx->ctlbase1 | 8), + mso_trans(REG_CTL1, ctx->ctlbase1), }; - sr_dbg("Requesting forced capture"); + sr_dbg("mso19: Requesting forced capture"); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_dac_out(struct sr_device_instance *sdi, uint16_t val) +static int mso_dac_out(struct sr_dev_inst *sdi, uint16_t val) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; uint16_t ops[] = { mso_trans(REG_DAC1, (val >> 8) & 0xff), mso_trans(REG_DAC2, val & 0xff), - mso_trans(REG_CTL1, mso->ctlbase1 | BIT_CTL1_RESETADC), + mso_trans(REG_CTL1, ctx->ctlbase1 | BIT_CTL1_RESETADC), }; - sr_dbg("Setting dac word to 0x%x", val); + sr_dbg("mso19: Setting dac word to 0x%x", val); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_clkrate_out(struct sr_device_instance *sdi, uint16_t val) +static int mso_clkrate_out(struct sr_dev_inst *sdi, uint16_t val) { uint16_t ops[] = { mso_trans(REG_CLKRATE1, (val >> 8) & 0xff), mso_trans(REG_CLKRATE2, val & 0xff), }; - sr_dbg("Setting clkrate word to 0x%x", val); + sr_dbg("mso19: Setting clkrate word to 0x%x", val); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_configure_rate(struct sr_device_instance *sdi, - uint32_t rate) +static int mso_configure_rate(struct sr_dev_inst *sdi, uint32_t rate) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; unsigned int i; int ret = SR_ERR; for (i = 0; i < ARRAY_SIZE(rate_map); i++) { if (rate_map[i].rate == rate) { - mso->ctlbase2 = rate_map[i].slowmode; + ctx->ctlbase2 = rate_map[i].slowmode; ret = mso_clkrate_out(sdi, rate_map[i].val); if (ret == SR_OK) - mso->cur_rate = rate; + ctx->cur_rate = rate; return ret; } } return ret; } -static inline uint16_t mso_calc_raw_from_mv(struct mso *mso) +static inline uint16_t mso_calc_raw_from_mv(struct context *ctx) { return (uint16_t) (0x200 - - ((mso->dso_trigger_voltage / mso->dso_probe_attn) / - mso->vbit)); + ((ctx->dso_trigger_voltage / ctx->dso_probe_attn) / + ctx->vbit)); } -static int mso_configure_trigger(struct sr_device_instance *sdi) +static int mso_configure_trigger(struct sr_dev_inst *sdi) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; uint16_t ops[16]; - uint16_t dso_trigger = mso_calc_raw_from_mv(mso); + uint16_t dso_trigger = mso_calc_raw_from_mv(ctx); dso_trigger &= 0x3ff; - if ((!mso->trigger_slope && mso->trigger_chan == 1) || - (mso->trigger_slope && - (mso->trigger_chan == 0 || - mso->trigger_chan == 2 || - mso->trigger_chan == 3))) + if ((!ctx->trigger_slope && ctx->trigger_chan == 1) || + (ctx->trigger_slope && + (ctx->trigger_chan == 0 || + ctx->trigger_chan == 2 || + ctx->trigger_chan == 3))) dso_trigger |= 0x400; - switch (mso->trigger_chan) { + switch (ctx->trigger_chan) { case 1: dso_trigger |= 0xe000; case 2: @@ -314,7 +317,7 @@ static int mso_configure_trigger(struct sr_device_instance *sdi) break; } - switch (mso->trigger_outsrc) { + switch (ctx->trigger_outsrc) { case 1: dso_trigger |= 0x800; break; @@ -327,40 +330,40 @@ static int mso_configure_trigger(struct sr_device_instance *sdi) } - ops[0] = mso_trans(5, mso->la_trigger); - ops[1] = mso_trans(6, mso->la_trigger_mask); + ops[0] = mso_trans(5, ctx->la_trigger); + ops[1] = mso_trans(6, ctx->la_trigger_mask); ops[2] = mso_trans(3, dso_trigger & 0xff); ops[3] = mso_trans(4, (dso_trigger >> 8) & 0xff); ops[4] = mso_trans(11, - mso->dso_trigger_width / SR_HZ_TO_NS(mso->cur_rate)); + ctx->dso_trigger_width / SR_HZ_TO_NS(ctx->cur_rate)); /* Select the SPI/I2C trigger config bank */ - ops[5] = mso_trans(REG_CTL2, (mso->ctlbase2 | BITS_CTL2_BANK(2))); + ops[5] = mso_trans(REG_CTL2, (ctx->ctlbase2 | BITS_CTL2_BANK(2))); /* Configure the SPI/I2C protocol trigger */ - ops[6] = mso_trans(REG_PT_WORD(0), mso->protocol_trigger.word[0]); - ops[7] = mso_trans(REG_PT_WORD(1), mso->protocol_trigger.word[1]); - ops[8] = mso_trans(REG_PT_WORD(2), mso->protocol_trigger.word[2]); - ops[9] = mso_trans(REG_PT_WORD(3), mso->protocol_trigger.word[3]); - ops[10] = mso_trans(REG_PT_MASK(0), mso->protocol_trigger.mask[0]); - ops[11] = mso_trans(REG_PT_MASK(1), mso->protocol_trigger.mask[1]); - ops[12] = mso_trans(REG_PT_MASK(2), mso->protocol_trigger.mask[2]); - ops[13] = mso_trans(REG_PT_MASK(3), mso->protocol_trigger.mask[3]); - ops[14] = mso_trans(REG_PT_SPIMODE, mso->protocol_trigger.spimode); + ops[6] = mso_trans(REG_PT_WORD(0), ctx->protocol_trigger.word[0]); + ops[7] = mso_trans(REG_PT_WORD(1), ctx->protocol_trigger.word[1]); + ops[8] = mso_trans(REG_PT_WORD(2), ctx->protocol_trigger.word[2]); + ops[9] = mso_trans(REG_PT_WORD(3), ctx->protocol_trigger.word[3]); + ops[10] = mso_trans(REG_PT_MASK(0), ctx->protocol_trigger.mask[0]); + ops[11] = mso_trans(REG_PT_MASK(1), ctx->protocol_trigger.mask[1]); + ops[12] = mso_trans(REG_PT_MASK(2), ctx->protocol_trigger.mask[2]); + ops[13] = mso_trans(REG_PT_MASK(3), ctx->protocol_trigger.mask[3]); + ops[14] = mso_trans(REG_PT_SPIMODE, ctx->protocol_trigger.spimode); /* Select the default config bank */ - ops[15] = mso_trans(REG_CTL2, mso->ctlbase2); + ops[15] = mso_trans(REG_CTL2, ctx->ctlbase2); return mso_send_control_message(sdi, ARRAY_AND_SIZE(ops)); } -static int mso_configure_threshold_level(struct sr_device_instance *sdi) +static int mso_configure_threshold_level(struct sr_dev_inst *sdi) { - struct mso *mso = sdi->priv; + struct context *ctx = sdi->priv; - return mso_dac_out(sdi, la_threshold_map[mso->la_threshold]); + return mso_dac_out(sdi, la_threshold_map[ctx->la_threshold]); } static int mso_parse_serial(const char *iSerial, const char *iProduct, - struct mso *mso) + struct context *ctx) { unsigned int u1, u2, u3, u4, u5, u6; @@ -368,26 +371,26 @@ static int mso_parse_serial(const char *iSerial, const char *iProduct, /* FIXME: This code is in the original app, but I think its * used only for the GUI */ /* if (strstr(iProduct, "REV_02") || strstr(iProduct, "REV_03")) - mso->num_sample_rates = 0x16; + ctx->num_sample_rates = 0x16; else - mso->num_sample_rates = 0x10; */ + ctx->num_sample_rates = 0x10; */ /* parse iSerial */ if (iSerial[0] != '4' || sscanf(iSerial, "%5u%3u%3u%1u%1u%6u", &u1, &u2, &u3, &u4, &u5, &u6) != 6) return SR_ERR; - mso->hwmodel = u4; - mso->hwrev = u5; - mso->serial = u6; - mso->vbit = u1 / 10000; - if (mso->vbit == 0) - mso->vbit = 4.19195; - mso->dac_offset = u2; - if (mso->dac_offset == 0) - mso->dac_offset = 0x1ff; - mso->offset_range = u3; - if (mso->offset_range == 0) - mso->offset_range = 0x17d; + ctx->hwmodel = u4; + ctx->hwrev = u5; + ctx->serial = u6; + ctx->vbit = u1 / 10000; + if (ctx->vbit == 0) + ctx->vbit = 4.19195; + ctx->dac_offset = u2; + if (ctx->dac_offset == 0) + ctx->dac_offset = 0x1ff; + ctx->offset_range = u3; + if (ctx->offset_range == 0) + ctx->offset_range = 0x17d; /* * FIXME: There is more code on the original software to handle @@ -398,16 +401,16 @@ static int mso_parse_serial(const char *iSerial, const char *iProduct, return SR_OK; } -static int hw_init(const char *deviceinfo) +static int hw_init(const char *devinfo) { - struct sr_device_instance *sdi; + struct sr_dev_inst *sdi; int devcnt = 0; struct udev *udev; struct udev_enumerate *enumerate; - struct udev_list_entry *devices, *dev_list_entry; - struct mso *mso; + struct udev_list_entry *devs, *dev_list_entry; + struct context *ctx; - deviceinfo = deviceinfo; + devinfo = devinfo; /* It's easier to map usb<->serial using udev */ /* @@ -416,14 +419,14 @@ static int hw_init(const char *deviceinfo) */ udev = udev_new(); if (!udev) { - sr_warn("Failed to initialize udev."); + sr_err("mso19: Failed to initialize udev."); goto ret; } enumerate = udev_enumerate_new(udev); udev_enumerate_add_match_subsystem(enumerate, "usb-serial"); udev_enumerate_scan_devices(enumerate); - devices = udev_enumerate_get_list_entry(enumerate); - udev_list_entry_foreach(dev_list_entry, devices) { + devs = udev_enumerate_get_list_entry(enumerate); + udev_list_entry_foreach(dev_list_entry, devs) { const char *syspath, *sysname, *idVendor, *idProduct, *iSerial, *iProduct; char path[32], manufacturer[32], product[32], hwrev[32]; @@ -436,8 +439,8 @@ static int hw_init(const char *deviceinfo) parent = udev_device_get_parent_with_subsystem_devtype( dev, "usb", "usb_device"); if (!parent) { - sr_warn("Unable to find parent usb device for %s", - sysname); + sr_err("mso19: Unable to find parent usb device for %s", + sysname); continue; } @@ -455,60 +458,59 @@ static int hw_init(const char *deviceinfo) s = strcspn(iProduct, " "); if (s > sizeof(product) || strlen(iProduct) - s > sizeof(manufacturer)) { - sr_warn("Could not parse iProduct: %s", iProduct); + sr_err("mso19: Could not parse iProduct: %s", iProduct); continue; } strncpy(product, iProduct, s); product[s] = 0; strcpy(manufacturer, iProduct + s); - if (!(mso = g_try_malloc0(sizeof(struct mso)))) { - sr_err("mso19: %s: mso malloc failed", __func__); + if (!(ctx = g_try_malloc0(sizeof(struct context)))) { + sr_err("mso19: %s: ctx malloc failed", __func__); continue; /* TODO: Errors handled correctly? */ } - if (mso_parse_serial(iSerial, iProduct, mso) != SR_OK) { - sr_warn("Invalid iSerial: %s", iSerial); - goto err_free_mso; + if (mso_parse_serial(iSerial, iProduct, ctx) != SR_OK) { + sr_err("mso19: Invalid iSerial: %s", iSerial); + goto err_free_ctx; } - sprintf(hwrev, "r%d", mso->hwrev); + sprintf(hwrev, "r%d", ctx->hwrev); /* hardware initial state */ - mso->ctlbase1 = 0; + ctx->ctlbase1 = 0; { /* Initialize the protocol trigger configuration */ int i; - for (i = 0; i < 4; i++) - { - mso->protocol_trigger.word[i] = 0; - mso->protocol_trigger.mask[i] = 0xff; + for (i = 0; i < 4; i++) { + ctx->protocol_trigger.word[i] = 0; + ctx->protocol_trigger.mask[i] = 0xff; } - mso->protocol_trigger.spimode = 0; + ctx->protocol_trigger.spimode = 0; } - sdi = sr_device_instance_new(devcnt, SR_ST_INITIALIZING, - manufacturer, product, hwrev); + sdi = sr_dev_inst_new(devcnt, SR_ST_INITIALIZING, + manufacturer, product, hwrev); if (!sdi) { - sr_warn("Unable to create device instance for %s", - sysname); - goto err_free_mso; + sr_err("mso19: Unable to create device instance for %s", + sysname); + goto err_free_ctx; } /* save a pointer to our private instance data */ - sdi->priv = mso; + sdi->priv = ctx; - sdi->serial = sr_serial_device_instance_new(path, -1); + sdi->serial = sr_serial_dev_inst_new(path, -1); if (!sdi->serial) - goto err_device_instance_free; + goto err_dev_inst_free; - device_instances = g_slist_append(device_instances, sdi); + dev_insts = g_slist_append(dev_insts, sdi); devcnt++; continue; -err_device_instance_free: - sr_device_instance_free(sdi); -err_free_mso: - free(mso); +err_dev_inst_free: + sr_dev_inst_free(sdi); +err_free_ctx: + g_free(ctx); } udev_enumerate_unref(enumerate); @@ -518,37 +520,41 @@ ret: return devcnt; } -static void hw_cleanup(void) +static int hw_cleanup(void) { GSList *l; - struct sr_device_instance *sdi; + struct sr_dev_inst *sdi; + int ret; + ret = SR_OK; /* Properly close all devices. */ - for (l = device_instances; l; l = l->next) { - sdi = l->data; + for (l = dev_insts; l; l = l->next) { + if (!(sdi = l->data)) { + /* Log error, but continue cleaning up the rest. */ + sr_err("mso19: %s: sdi was NULL, continuing", __func__); + ret = SR_ERR_BUG; + continue; + } if (sdi->serial->fd != -1) serial_close(sdi->serial->fd); - if (sdi->priv != NULL) - { - free(sdi->priv); - sdi->priv = NULL; - } - sr_device_instance_free(sdi); + sr_dev_inst_free(sdi); } - g_slist_free(device_instances); - device_instances = NULL; + g_slist_free(dev_insts); + dev_insts = NULL; + + return ret; } -static int hw_opendev(int device_index) +static int hw_dev_open(int dev_index) { - struct sr_device_instance *sdi; - struct mso *mso; + struct sr_dev_inst *sdi; + struct context *ctx; int ret = SR_ERR; - if (!(sdi = sr_get_device_instance(device_instances, device_index))) + if (!(sdi = sr_dev_inst_get(dev_insts, dev_index))) return ret; - mso = sdi->priv; + ctx = sdi->priv; sdi->serial->fd = serial_open(sdi->serial->port, O_RDWR); if (sdi->serial->fd == -1) return ret; @@ -561,31 +567,31 @@ static int hw_opendev(int device_index) /* FIXME: discard serial buffer */ - mso_check_trigger(sdi, &mso->trigger_state); - sr_dbg("trigger state: 0x%x", mso->trigger_state); + mso_check_trigger(sdi, &ctx->trigger_state); + sr_dbg("mso19: trigger state: 0x%x", ctx->trigger_state); ret = mso_reset_adc(sdi); if (ret != SR_OK) return ret; - mso_check_trigger(sdi, &mso->trigger_state); - sr_dbg("trigger state: 0x%x", mso->trigger_state); + mso_check_trigger(sdi, &ctx->trigger_state); + sr_dbg("mso19: trigger state: 0x%x", ctx->trigger_state); // ret = mso_reset_fsm(sdi); // if (ret != SR_OK) // return ret; - sr_dbg("Finished %s", __func__); + sr_dbg("mso19: Finished %s", __func__); // return SR_ERR; return SR_OK; } -static int hw_closedev(int device_index) +static int hw_dev_close(int dev_index) { - struct sr_device_instance *sdi; + struct sr_dev_inst *sdi; - if (!(sdi = sr_get_device_instance(device_instances, device_index))) { + if (!(sdi = sr_dev_inst_get(dev_insts, dev_index))) { sr_err("mso19: %s: sdi was NULL", __func__); return SR_ERR; /* TODO: SR_ERR_ARG? */ } @@ -597,22 +603,22 @@ static int hw_closedev(int device_index) sdi->status = SR_ST_INACTIVE; } - sr_dbg("finished %s", __func__); + sr_dbg("mso19: finished %s", __func__); return SR_OK; } -static void *hw_get_device_info(int device_index, int device_info_id) +static void *hw_dev_info_get(int dev_index, int dev_info_id) { - struct sr_device_instance *sdi; - struct mso *mso; + struct sr_dev_inst *sdi; + struct context *ctx; void *info = NULL; - if (!(sdi = sr_get_device_instance(device_instances, device_index))) + if (!(sdi = sr_dev_inst_get(dev_insts, dev_index))) return NULL; - mso = sdi->priv; + ctx = sdi->priv; - switch (device_info_id) { - case SR_DI_INSTANCE: + switch (dev_info_id) { + case SR_DI_INST: info = sdi; break; case SR_DI_NUM_PROBES: /* FIXME: How to report analog probe? */ @@ -628,35 +634,35 @@ static void *hw_get_device_info(int device_index, int device_info_id) info = "01"; /* FIXME */ break; case SR_DI_CUR_SAMPLERATE: - info = &mso->cur_rate; + info = &ctx->cur_rate; break; } return info; } -static int hw_get_status(int device_index) +static int hw_dev_status_get(int dev_index) { - struct sr_device_instance *sdi; + struct sr_dev_inst *sdi; - if (!(sdi = sr_get_device_instance(device_instances, device_index))) + if (!(sdi = sr_dev_inst_get(dev_insts, dev_index))) return SR_ST_NOT_FOUND; return sdi->status; } -static int *hw_get_capabilities(void) +static int *hw_hwcap_get_all(void) { - return capabilities; + return hwcaps; } -static int hw_set_configuration(int device_index, int capability, void *value) +static int hw_dev_config_set(int dev_index, int hwcap, void *value) { - struct sr_device_instance *sdi; + struct sr_dev_inst *sdi; - if (!(sdi = sr_get_device_instance(device_instances, device_index))) + if (!(sdi = sr_dev_inst_get(dev_insts, dev_index))) return SR_ERR; - switch (capability) { + switch (hwcap) { case SR_HWCAP_SAMPLERATE: return mso_configure_rate(sdi, *(uint64_t *) value); case SR_HWCAP_PROBECONFIG: @@ -677,8 +683,8 @@ static int hw_set_configuration(int device_index, int capability, void *value) /* FIXME: Pass errors? */ static int receive_data(int fd, int revents, void *user_data) { - struct sr_device_instance *sdi = user_data; - struct mso *mso = sdi->priv; + struct sr_dev_inst *sdi = user_data; + struct context *ctx = sdi->priv; struct sr_datafeed_packet packet; struct sr_datafeed_logic logic; uint8_t in[1024], logic_out[1024]; @@ -692,11 +698,11 @@ static int receive_data(int fd, int revents, void *user_data) return FALSE; /* No samples */ - if (mso->trigger_state != MSO_TRIGGER_DATAREADY) { - mso->trigger_state = in[0]; - if (mso->trigger_state == MSO_TRIGGER_DATAREADY) { + if (ctx->trigger_state != MSO_TRIGGER_DATAREADY) { + ctx->trigger_state = in[0]; + if (ctx->trigger_state == MSO_TRIGGER_DATAREADY) { mso_read_buffer(sdi); - mso->buffer_n = 0; + ctx->buffer_n = 0; } else { mso_check_trigger(sdi, NULL); } @@ -704,20 +710,20 @@ static int receive_data(int fd, int revents, void *user_data) } /* the hardware always dumps 1024 samples, 24bits each */ - if (mso->buffer_n < 3072) { - memcpy(mso->buffer + mso->buffer_n, in, s); - mso->buffer_n += s; + if (ctx->buffer_n < 3072) { + memcpy(ctx->buffer + ctx->buffer_n, in, s); + ctx->buffer_n += s; } - if (mso->buffer_n < 3072) + if (ctx->buffer_n < 3072) return FALSE; /* do the conversion */ for (i = 0; i < 1024; i++) { /* FIXME: Need to do conversion to mV */ - analog_out[i] = (mso->buffer[i * 3] & 0x3f) | - ((mso->buffer[i * 3 + 1] & 0xf) << 6); - logic_out[i] = ((mso->buffer[i * 3 + 1] & 0x30) >> 4) | - ((mso->buffer[i * 3 + 2] & 0x3f) << 2); + analog_out[i] = (ctx->buffer[i * 3] & 0x3f) | + ((ctx->buffer[i * 3 + 1] & 0xf) << 6); + logic_out[i] = ((ctx->buffer[i * 3 + 1] & 0x30) >> 4) | + ((ctx->buffer[i * 3 + 2] & 0x3f) << 2); } packet.type = SR_DF_LOGIC; @@ -725,7 +731,7 @@ static int receive_data(int fd, int revents, void *user_data) logic.length = 1024; logic.unitsize = 1; logic.data = logic_out; - sr_session_bus(mso->session_id, &packet); + sr_session_bus(ctx->session_id, &packet); // Dont bother fixing this yet, keep it "old style" /* @@ -733,26 +739,26 @@ static int receive_data(int fd, int revents, void *user_data) packet.length = 1024; packet.unitsize = sizeof(double); packet.payload = analog_out; - sr_session_bus(mso->session_id, &packet); + sr_session_bus(ctx->session_id, &packet); */ packet.type = SR_DF_END; - sr_session_bus(mso->session_id, &packet); + sr_session_bus(ctx->session_id, &packet); return TRUE; } -static int hw_start_acquisition(int device_index, gpointer session_device_id) +static int hw_dev_acquisition_start(int dev_index, gpointer session_dev_id) { - struct sr_device_instance *sdi; - struct mso *mso; + struct sr_dev_inst *sdi; + struct context *ctx; struct sr_datafeed_packet packet; struct sr_datafeed_header header; int ret = SR_ERR; - if (!(sdi = sr_get_device_instance(device_instances, device_index))) + if (!(sdi = sr_dev_inst_get(dev_insts, dev_index))) return ret; - mso = sdi->priv; + ctx = sdi->priv; /* FIXME: No need to do full reconfigure every time */ // ret = mso_reset_fsm(sdi); @@ -760,15 +766,15 @@ static int hw_start_acquisition(int device_index, gpointer session_device_id) // return ret; /* FIXME: ACDC Mode */ - mso->ctlbase1 &= 0x7f; -// mso->ctlbase1 |= mso->acdcmode; + ctx->ctlbase1 &= 0x7f; +// ctx->ctlbase1 |= ctx->acdcmode; - ret = mso_configure_rate(sdi, mso->cur_rate); + ret = mso_configure_rate(sdi, ctx->cur_rate); if (ret != SR_OK) return ret; /* set dac offset */ - ret = mso_dac_out(sdi, mso->dac_offset); + ret = mso_dac_out(sdi, ctx->dac_offset); if (ret != SR_OK) return ret; @@ -795,49 +801,51 @@ static int hw_start_acquisition(int device_index, gpointer session_device_id) // if (ret != SR_OK) // return ret; - mso_check_trigger(sdi, &mso->trigger_state); + mso_check_trigger(sdi, &ctx->trigger_state); ret = mso_check_trigger(sdi, NULL); if (ret != SR_OK) return ret; - mso->session_id = session_device_id; + ctx->session_id = session_dev_id; sr_source_add(sdi->serial->fd, G_IO_IN, -1, receive_data, sdi); packet.type = SR_DF_HEADER; packet.payload = (unsigned char *) &header; header.feed_version = 1; gettimeofday(&header.starttime, NULL); - header.samplerate = mso->cur_rate; + header.samplerate = ctx->cur_rate; // header.num_analog_probes = 1; header.num_logic_probes = 8; - sr_session_bus(session_device_id, &packet); + sr_session_bus(session_dev_id, &packet); return ret; } /* FIXME */ -static void hw_stop_acquisition(int device_index, gpointer session_device_id) +static int hw_dev_acquisition_stop(int dev_index, gpointer session_dev_id) { struct sr_datafeed_packet packet; - device_index = device_index; + dev_index = dev_index; packet.type = SR_DF_END; - sr_session_bus(session_device_id, &packet); + sr_session_bus(session_dev_id, &packet); + + return SR_OK; } -SR_PRIV struct sr_device_plugin link_mso19_plugin_info = { +SR_PRIV struct sr_dev_plugin link_mso19_plugin_info = { .name = "link-mso19", .longname = "Link Instruments MSO-19", .api_version = 1, .init = hw_init, .cleanup = hw_cleanup, - .opendev = hw_opendev, - .closedev = hw_closedev, - .get_device_info = hw_get_device_info, - .get_status = hw_get_status, - .get_capabilities = hw_get_capabilities, - .set_configuration = hw_set_configuration, - .start_acquisition = hw_start_acquisition, - .stop_acquisition = hw_stop_acquisition, + .dev_open = hw_dev_open, + .dev_close = hw_dev_close, + .dev_info_get = hw_dev_info_get, + .dev_status_get = hw_dev_status_get, + .hwcap_get_all = hw_hwcap_get_all, + .dev_config_set = hw_dev_config_set, + .dev_acquisition_start = hw_dev_acquisition_start, + .dev_acquisition_stop = hw_dev_acquisition_stop, };