X-Git-Url: https://sigrok.org/gitweb/?a=blobdiff_plain;f=session_driver.c;h=7d2b9bda85fd77ec251cc5dbe28b5e5481e7ca2f;hb=7b48d6e104fd461b0c12519b70dbd1c6c72824e1;hp=fe9765edee82039ed397fec330dae94963f57281;hpb=b7f09cf86dd57a59dc1bdece2cac9e4a176900b1;p=libsigrok.git diff --git a/session_driver.c b/session_driver.c index fe9765ed..7d2b9bda 100644 --- a/session_driver.c +++ b/session_driver.c @@ -1,7 +1,7 @@ /* * This file is part of the sigrok project. * - * Copyright (C) 2011 Bert Vermeulen + * Copyright (C) 2010-2012 Bert Vermeulen * * This program is free software: you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -27,7 +27,7 @@ #include "sigrok-internal.h" /* size of payloads sent across the session bus */ -#define CHUNKSIZE 4096 +#define CHUNKSIZE (512 * 1024) struct session_vdevice { char *capturefile; @@ -47,26 +47,46 @@ static int capabilities[] = { 0, }; +/** + * TODO. + * + * @param device_index TODO. + */ static struct session_vdevice *get_vdevice_by_index(int device_index) { struct sr_device_instance *sdi; struct session_vdevice *vdevice; - if (!(sdi = sr_get_device_instance(device_instances, device_index))) + /* TODO: Sanity checks on device_index. */ + + if (!(sdi = sr_dev_inst_get(device_instances, device_index))) { + sr_err("session driver: %s: device instance with device " + "index %d was not found", __func__, device_index); return NULL; + } + + /* TODO: Is sdi->priv == NULL valid? */ vdevice = sdi->priv; return vdevice; } +/** + * TODO. + * + * @param fd TODO. + * @param revents TODO. + * @param session_data TODO. + * + * @return TODO. + */ static int feed_chunk(int fd, int revents, void *session_data) { struct sr_device_instance *sdi; struct session_vdevice *vdevice; struct sr_datafeed_packet packet; struct sr_datafeed_logic logic; - uint64_t sample_period_ps; GSList *l; void *buf; int ret, got_data; @@ -86,18 +106,15 @@ static int feed_chunk(int fd, int revents, void *session_data) continue; if (!(buf = g_try_malloc(CHUNKSIZE))) { - sr_err("session: %s: buf malloc failed", __func__); - // return SR_ERR_MALLOC; - return FALSE; + sr_err("session driver: %s: buf malloc failed", + __func__); + return FALSE; /* TODO: SR_ERR_MALLOC */ } ret = zip_fread(vdevice->capfile, buf, CHUNKSIZE); if (ret > 0) { got_data = TRUE; packet.type = SR_DF_LOGIC; - sample_period_ps = 1000000000000 / vdevice->samplerate; - packet.timeoffset = sample_period_ps * (vdevice->bytes_read / vdevice->unitsize); - packet.duration = sample_period_ps * (ret / vdevice->unitsize); packet.payload = &logic; logic.length = ret; logic.unitsize = vdevice->unitsize; @@ -122,43 +139,53 @@ static int feed_chunk(int fd, int revents, void *session_data) } /* driver callbacks */ -static void hw_cleanup(void); +static int hw_cleanup(void); +/** + * TODO. + * + * @param deviceinfo TODO. + * + * @return TODO. + */ static int hw_init(const char *deviceinfo) { - hw_cleanup(); - sessionfile = g_strdup(deviceinfo); return 0; } -static void hw_cleanup(void) +/** + * TODO. + * + */ +static int hw_cleanup(void) { GSList *l; for (l = device_instances; l; l = l->next) - sr_device_instance_free(l->data); - + sr_dev_inst_free(l->data); g_slist_free(device_instances); device_instances = NULL; sr_session_source_remove(-1); g_free(sessionfile); + + return SR_OK; } static int hw_opendev(int device_index) { struct sr_device_instance *sdi; - sdi = sr_device_instance_new(device_index, SR_ST_INITIALIZING, - NULL, NULL, NULL); + sdi = sr_dev_inst_new(device_index, SR_ST_INITIALIZING, + NULL, NULL, NULL); if (!sdi) return SR_ERR; if (!(sdi->priv = g_try_malloc0(sizeof(struct session_vdevice)))) { - sr_err("session: %s: sdi->priv malloc failed", __func__); + sr_err("session driver: %s: sdi->priv malloc failed", __func__); return SR_ERR_MALLOC; } @@ -188,12 +215,18 @@ static int hw_get_status(int device_index) /* Avoid compiler warnings. */ (void)device_index; - if (sr_device_list() != NULL) + if (sr_dev_list() != NULL) return SR_OK; else return SR_ERR; } +/** + * Get the list of hardware capabilities. + * + * @return A pointer to the (hardware) capabilities of this virtual session + * driver. This could be NULL, if no such capabilities exist. + */ static int *hw_get_capabilities(void) { return capabilities; @@ -211,9 +244,13 @@ static int hw_set_configuration(int device_index, int capability, void *value) case SR_HWCAP_SAMPLERATE: tmp_u64 = value; vdevice->samplerate = *tmp_u64; + sr_info("session driver: setting samplerate to %" PRIu64, + vdevice->samplerate); break; case SR_HWCAP_CAPTUREFILE: vdevice->capturefile = g_strdup(value); + sr_info("session driver: setting capturefile to %s", + vdevice->capturefile); break; case SR_HWCAP_CAPTURE_UNITSIZE: tmp_u64 = value; @@ -224,6 +261,8 @@ static int hw_set_configuration(int device_index, int capability, void *value) vdevice->num_probes = *tmp_u64; break; default: + sr_err("session driver: %s: unknown capability %d requested", + __func__, capability); return SR_ERR; } @@ -248,20 +287,21 @@ static int hw_start_acquisition(int device_index, gpointer session_device_id) vdevice->capturefile); if (!(vdevice->archive = zip_open(sessionfile, 0, &err))) { - sr_warn("Failed to open session file '%s': zip error %d\n", - sessionfile, err); + sr_err("session driver: Failed to open session file '%s': " + "zip error %d\n", sessionfile, err); return SR_ERR; } if (zip_stat(vdevice->archive, vdevice->capturefile, 0, &zs) == -1) { - sr_warn("Failed to check capture file '%s' in session file '%s'.", - vdevice->capturefile, sessionfile); + sr_err("session driver: Failed to check capture file '%s' in " + "session file '%s'.", vdevice->capturefile, sessionfile); return SR_ERR; } - if (!(vdevice->capfile = zip_fopen(vdevice->archive, vdevice->capturefile, 0))) { - sr_warn("Failed to open capture file '%s' in session file '%s'.", - vdevice->capturefile, sessionfile); + if (!(vdevice->capfile = zip_fopen(vdevice->archive, + vdevice->capturefile, 0))) { + sr_err("session driver: Failed to open capture file '%s' in " + "session file '%s'.", vdevice->capturefile, sessionfile); return SR_ERR; } @@ -269,12 +309,12 @@ static int hw_start_acquisition(int device_index, gpointer session_device_id) sr_session_source_add(-1, 0, 0, feed_chunk, session_device_id); if (!(packet = g_try_malloc(sizeof(struct sr_datafeed_packet)))) { - sr_err("session: %s: packet malloc failed", __func__); + sr_err("session driver: %s: packet malloc failed", __func__); return SR_ERR_MALLOC; } if (!(header = g_try_malloc(sizeof(struct sr_datafeed_header)))) { - sr_err("session: %s: header malloc failed", __func__); + sr_err("session driver: %s: header malloc failed", __func__); return SR_ERR_MALLOC; } @@ -285,7 +325,6 @@ static int hw_start_acquisition(int device_index, gpointer session_device_id) gettimeofday(&header->starttime, NULL); header->samplerate = vdevice->samplerate; header->num_logic_probes = vdevice->num_probes; - header->num_analog_probes = 0; sr_session_bus(session_device_id, packet); g_free(header); g_free(packet);