X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=src%2Fhardware%2Fzeroplus-logic-cube%2Fapi.c;h=1081b81c2b461ec4cf059a9f8f0dc8c6df80dfd5;hb=4af1f68f9a69daa22064c4b99831f6dee5c5fbb2;hp=5d8c534f27eb56c7f8c86b3462649886f84ef848;hpb=4f840ce965b1c30c5ab75afecc56193cbaf5c1b3;p=libsigrok.git diff --git a/src/hardware/zeroplus-logic-cube/api.c b/src/hardware/zeroplus-logic-cube/api.c index 5d8c534f..1081b81c 100644 --- a/src/hardware/zeroplus-logic-cube/api.c +++ b/src/hardware/zeroplus-logic-cube/api.c @@ -17,6 +17,7 @@ * along with this program. If not, see . */ +#include #include "protocol.h" #define VENDOR_NAME "ZEROPLUS" @@ -30,7 +31,7 @@ struct zp_model { uint16_t vid; uint16_t pid; - char *model_name; + const char *model_name; unsigned int channels; unsigned int sample_depth; /* In Ksamples/channel */ unsigned int max_sampling_freq; @@ -50,7 +51,7 @@ static const struct zp_model zeroplus_models[] = { {0x0c12, 0x700e, "LAP-C(16032)", 16, 32, 100}, {0x0c12, 0x7016, "LAP-C(162000)", 16, 2048, 200}, {0x0c12, 0x7100, "AKIP-9101", 16, 256, 200}, - { 0, 0, 0, 0, 0, 0 } + ALL_ZERO }; static const uint32_t devopts[] = { @@ -76,7 +77,6 @@ static const char *channel_names[] = { "B0", "B1", "B2", "B3", "B4", "B5", "B6", "B7", "C0", "C1", "C2", "C3", "C4", "C5", "C6", "C7", "D0", "D1", "D2", "D3", "D4", "D5", "D6", "D7", - NULL, }; SR_PRIV struct sr_dev_driver zeroplus_logic_cube_driver_info; @@ -175,7 +175,7 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options) (void)options; - drvc = di->priv; + drvc = di->context; devices = NULL; @@ -183,12 +183,7 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options) libusb_get_device_list(drvc->sr_ctx->libusb_ctx, &devlist); /* TODO: Errors. */ for (i = 0; devlist[i]; i++) { - ret = libusb_get_device_descriptor(devlist[i], &des); - if (ret != 0) { - sr_err("Failed to get device descriptor: %s.", - libusb_error_name(ret)); - continue; - } + libusb_get_device_descriptor(devlist[i], &des); if ((ret = libusb_open(devlist[i], &hdl)) < 0) continue; @@ -261,11 +256,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options) return devices; } -static GSList *dev_list(const struct sr_dev_driver *di) -{ - return ((struct drv_context *)(di->priv))->instances; -} - static int dev_open(struct sr_dev_inst *sdi) { struct sr_dev_driver *di = sdi->driver; @@ -276,13 +266,9 @@ static int dev_open(struct sr_dev_inst *sdi) int device_count, ret, i; char connection_id[64]; - drvc = di->priv; + drvc = di->context; usb = sdi->conn; - - if (!(devc = sdi->priv)) { - sr_err("%s: sdi->priv was NULL", __func__); - return SR_ERR_ARG; - } + devc = sdi->priv; device_count = libusb_get_device_list(drvc->sr_ctx->libusb_ctx, &devlist); @@ -382,44 +368,30 @@ static int dev_close(struct sr_dev_inst *sdi) return SR_OK; } -static int cleanup(const struct sr_dev_driver *di) -{ - return std_dev_clear(di, NULL); -} - 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; + GVariant *range[2]; (void)cg; + if (!sdi) + return SR_ERR_ARG; + + devc = sdi->priv; + switch (key) { case SR_CONF_SAMPLERATE: - if (sdi) { - devc = sdi->priv; - *data = g_variant_new_uint64(devc->cur_samplerate); - sr_spew("Returning samplerate: %" PRIu64 "Hz.", - devc->cur_samplerate); - } else - return SR_ERR_ARG; + *data = g_variant_new_uint64(devc->cur_samplerate); break; case SR_CONF_CAPTURE_RATIO: - if (sdi) { - devc = sdi->priv; - *data = g_variant_new_uint64(devc->capture_ratio); - } else - return SR_ERR_ARG; + *data = g_variant_new_uint64(devc->capture_ratio); break; case SR_CONF_VOLTAGE_THRESHOLD: - if (sdi) { - GVariant *range[2]; - devc = sdi->priv; - range[0] = g_variant_new_double(devc->cur_threshold); - range[1] = g_variant_new_double(devc->cur_threshold); - *data = g_variant_new_tuple(range, 2); - } else - return SR_ERR_ARG; + range[0] = g_variant_new_double(devc->cur_threshold); + range[1] = g_variant_new_double(devc->cur_threshold); + *data = g_variant_new_tuple(range, 2); break; default: return SR_ERR_NA; @@ -526,8 +498,7 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst * return SR_OK; } -static int dev_acquisition_start(const struct sr_dev_inst *sdi, - void *cb_data) +static int dev_acquisition_start(const struct sr_dev_inst *sdi) { struct dev_context *devc; struct sr_usb_dev_inst *usb; @@ -552,10 +523,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, if (sdi->status != SR_ST_ACTIVE) return SR_ERR_DEV_CLOSED; - if (!(devc = sdi->priv)) { - sr_err("%s: sdi->priv was NULL", __func__); - return SR_ERR_ARG; - } + devc = sdi->priv; if (analyzer_add_triggers(sdi) != SR_OK) { sr_err("Failed to configure triggers."); @@ -592,20 +560,15 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, sr_info("Ramsize trigger = 0x%x.", ramsize_trigger); sr_info("Memory size = 0x%x.", memory_size); - /* Send header packet to the session bus. */ - std_session_send_df_header(cb_data, LOG_PREFIX); + std_session_send_df_header(sdi, LOG_PREFIX); /* Check for empty capture */ if ((status & STATUS_READY) && !stop_address) { - packet.type = SR_DF_END; - sr_session_send(cb_data, &packet); + std_session_send_df_end(sdi, LOG_PREFIX); return SR_OK; } - if (!(buf = g_try_malloc(PACKET_SIZE))) { - sr_err("Packet buffer malloc failed."); - return SR_ERR_MALLOC; - } + buf = g_malloc(PACKET_SIZE); /* Check if the trigger is in the samples we are throwing away */ trigger_now = now_address == trigger_address || @@ -684,7 +647,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, logic.length = (trigger_offset - samples_read) * 4; logic.unitsize = 4; logic.data = buf + buf_offset; - sr_session_send(cb_data, &packet); + sr_session_send(sdi, &packet); len -= logic.length; samples_read += logic.length / 4; buf_offset += logic.length; @@ -694,7 +657,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, /* Send out trigger */ packet.type = SR_DF_TRIGGER; packet.payload = NULL; - sr_session_send(cb_data, &packet); + sr_session_send(sdi, &packet); } /* Send out data (or data after trigger) */ @@ -703,32 +666,23 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, logic.length = len; logic.unitsize = 4; logic.data = buf + buf_offset; - sr_session_send(cb_data, &packet); + sr_session_send(sdi, &packet); samples_read += len / 4; } analyzer_read_stop(usb->devhdl); g_free(buf); - packet.type = SR_DF_END; - sr_session_send(cb_data, &packet); + std_session_send_df_end(sdi, LOG_PREFIX); return SR_OK; } /* TODO: This stops acquisition on ALL devices, ignoring dev_index. */ -static int dev_acquisition_stop(struct sr_dev_inst *sdi, void *cb_data) +static int dev_acquisition_stop(struct sr_dev_inst *sdi) { - struct dev_context *devc; struct sr_usb_dev_inst *usb; - struct sr_datafeed_packet packet; - - packet.type = SR_DF_END; - sr_session_send(cb_data, &packet); - if (!(devc = sdi->priv)) { - sr_err("%s: sdi->priv was NULL", __func__); - return SR_ERR_BUG; - } + std_session_send_df_end(sdi, LOG_PREFIX); usb = sdi->conn; analyzer_reset(usb->devhdl); @@ -742,9 +696,9 @@ SR_PRIV struct sr_dev_driver zeroplus_logic_cube_driver_info = { .longname = "ZEROPLUS Logic Cube LAP-C series", .api_version = 1, .init = init, - .cleanup = cleanup, + .cleanup = std_cleanup, .scan = scan, - .dev_list = dev_list, + .dev_list = std_dev_list, .dev_clear = NULL, .config_get = config_get, .config_set = config_set, @@ -753,5 +707,5 @@ SR_PRIV struct sr_dev_driver zeroplus_logic_cube_driver_info = { .dev_close = dev_close, .dev_acquisition_start = dev_acquisition_start, .dev_acquisition_stop = dev_acquisition_stop, - .priv = NULL, + .context = NULL, };