]> sigrok.org Git - libsigrok.git/commitdiff
drivers: Drop unneeded or duplicate comments.
authorUwe Hermann <redacted>
Wed, 19 Jul 2017 19:29:09 +0000 (21:29 +0200)
committerUwe Hermann <redacted>
Fri, 21 Jul 2017 13:23:56 +0000 (15:23 +0200)
Drop various comments which are not really needed, too verbose, document
obvious things, are duplicated across all drivers, or simply incorrect.

116 files changed:
src/hardware/agilent-dmm/api.c
src/hardware/agilent-dmm/protocol.h
src/hardware/appa-55ii/api.c
src/hardware/appa-55ii/protocol.h
src/hardware/arachnid-labs-re-load-pro/api.c
src/hardware/arachnid-labs-re-load-pro/protocol.h
src/hardware/asix-sigma/api.c
src/hardware/asix-sigma/protocol.c
src/hardware/asix-sigma/protocol.h
src/hardware/atten-pps3xxx/api.c
src/hardware/atten-pps3xxx/protocol.h
src/hardware/baylibre-acme/protocol.h
src/hardware/beaglelogic/api.c
src/hardware/beaglelogic/protocol.h
src/hardware/brymen-bm86x/api.c
src/hardware/brymen-bm86x/protocol.h
src/hardware/brymen-dmm/api.c
src/hardware/brymen-dmm/protocol.h
src/hardware/cem-dt-885x/api.c
src/hardware/cem-dt-885x/protocol.h
src/hardware/center-3xx/api.c
src/hardware/center-3xx/protocol.h
src/hardware/chronovu-la/api.c
src/hardware/chronovu-la/protocol.h
src/hardware/colead-slm/api.c
src/hardware/colead-slm/protocol.h
src/hardware/conrad-digi-35-cpu/api.c
src/hardware/conrad-digi-35-cpu/protocol.c
src/hardware/conrad-digi-35-cpu/protocol.h
src/hardware/demo/protocol.h
src/hardware/dreamsourcelab-dslogic/api.c
src/hardware/dreamsourcelab-dslogic/protocol.h
src/hardware/fluke-dmm/api.c
src/hardware/fluke-dmm/protocol.h
src/hardware/ftdi-la/api.c
src/hardware/ftdi-la/protocol.h
src/hardware/fx2lafw/api.c
src/hardware/fx2lafw/protocol.h
src/hardware/gmc-mh-1x-2x/api.c
src/hardware/gmc-mh-1x-2x/protocol.c
src/hardware/gmc-mh-1x-2x/protocol.h
src/hardware/gwinstek-gds-800/protocol.h
src/hardware/hameg-hmo/protocol.h
src/hardware/hp-3457a/protocol.c
src/hardware/hp-3457a/protocol.h
src/hardware/hung-chang-dso-2100/protocol.h
src/hardware/ikalogic-scanalogic2/api.c
src/hardware/ikalogic-scanalogic2/protocol.c
src/hardware/ikalogic-scanalogic2/protocol.h
src/hardware/ikalogic-scanaplus/api.c
src/hardware/ikalogic-scanaplus/protocol.h
src/hardware/kecheng-kc-330b/protocol.c
src/hardware/kecheng-kc-330b/protocol.h
src/hardware/kern-scale/api.c
src/hardware/kern-scale/protocol.h
src/hardware/korad-kaxxxxp/api.c
src/hardware/korad-kaxxxxp/protocol.h
src/hardware/lascar-el-usb/protocol.h
src/hardware/lecroy-logicstudio/protocol.h
src/hardware/lecroy-xstream/api.c
src/hardware/lecroy-xstream/protocol.h
src/hardware/link-mso19/api.c
src/hardware/link-mso19/protocol.h
src/hardware/manson-hcs-3xxx/api.c
src/hardware/manson-hcs-3xxx/protocol.c
src/hardware/manson-hcs-3xxx/protocol.h
src/hardware/maynuo-m97/api.c
src/hardware/maynuo-m97/protocol.h
src/hardware/mic-985xx/api.c
src/hardware/mic-985xx/protocol.h
src/hardware/motech-lps-30x/api.c
src/hardware/motech-lps-30x/protocol.c
src/hardware/motech-lps-30x/protocol.h
src/hardware/norma-dmm/api.c
src/hardware/norma-dmm/protocol.c
src/hardware/norma-dmm/protocol.h
src/hardware/openbench-logic-sniffer/protocol.h
src/hardware/pce-322a/api.c
src/hardware/pce-322a/protocol.h
src/hardware/pipistrello-ols/api.c
src/hardware/pipistrello-ols/protocol.c
src/hardware/pipistrello-ols/protocol.h
src/hardware/rigol-ds/protocol.h
src/hardware/rohde-schwarz-sme-0x/protocol.h
src/hardware/saleae-logic-pro/protocol.h
src/hardware/saleae-logic16/protocol.h
src/hardware/scpi-pps/api.c
src/hardware/scpi-pps/profiles.c
src/hardware/scpi-pps/protocol.h
src/hardware/serial-dmm/api.c
src/hardware/serial-dmm/protocol.h
src/hardware/sysclk-lwla/api.c
src/hardware/sysclk-lwla/lwla.h
src/hardware/sysclk-lwla/lwla1016.c
src/hardware/sysclk-lwla/lwla1034.c
src/hardware/sysclk-lwla/protocol.c
src/hardware/sysclk-lwla/protocol.h
src/hardware/teleinfo/api.c
src/hardware/teleinfo/protocol.h
src/hardware/testo/api.c
src/hardware/testo/protocol.h
src/hardware/tondaj-sl-814/api.c
src/hardware/tondaj-sl-814/protocol.h
src/hardware/uni-t-dmm/api.c
src/hardware/uni-t-dmm/protocol.c
src/hardware/uni-t-dmm/protocol.h
src/hardware/uni-t-ut32x/protocol.h
src/hardware/victor-dmm/api.c
src/hardware/victor-dmm/protocol.h
src/hardware/yokogawa-dlm/api.c
src/hardware/yokogawa-dlm/protocol.c
src/hardware/yokogawa-dlm/protocol.h
src/hardware/zeroplus-logic-cube/api.c
src/hardware/zeroplus-logic-cube/protocol.h
src/hwdriver.c
src/lcr/es51919.c

index 052e503cd297fe2a20855b368522f575e36600f0..d38cdf0c56a0dbcd65611231fedc3a25757a2fc6 100644 (file)
@@ -316,7 +316,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 10ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 10,
                        agdmm_receive_data, (void *)sdi);
index 2510d7360fd5cd940bfe246184d20221b6c5dcc1..85b5235743eecd703671f97fd38eba6c6125eb8c 100644 (file)
@@ -68,13 +68,11 @@ struct agdmm_profile {
        const struct agdmm_recv *recvs;
 };
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
        const struct agdmm_profile *profile;
        struct sr_sw_limits limits;
        int data_source;
 
-       /* Runtime. */
        const struct agdmm_job *jobs;
        int current_job;
        gboolean job_running;
index eab203dd09ac418b36d723f218b5f6c6ce62ad37..aecfc262960a7f1f387b189128c523a16a2b030c 100644 (file)
@@ -192,7 +192,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 
        std_session_send_df_header(sdi);
 
-       /* Poll every 50ms, or whenever some data comes in. */
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
                        appa_55ii_receive_data, (void *)sdi);
 
index 248d672e1970b15e38352ddf61b1de02e3ca7396..171ac63670cb959391e8cfc640144042f8846c37 100644 (file)
@@ -36,13 +36,10 @@ enum {
        DATA_SOURCE_MEMORY,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Acquisition settings */
        struct sr_sw_limits limits;
        gboolean data_source; /**< Whether to read live samples or memory */
 
-       /* Temporary state across callbacks */
        uint8_t buf[APPA_55II_BUF_SIZE];
        unsigned int buf_len;
        uint8_t log_buf[64];
index 9c1fe532b28cda007df115a20ef7294213f730f9..11a6e3059119ad51d466061985d10624bf58a9ef 100644 (file)
@@ -153,7 +153,6 @@ static int config_list(uint32_t key, GVariant **data,
        GVariantBuilder gvb;
 
        if (!cg) {
-               /* No channel group: global options. */
                return STD_CONFIG_LIST(key, data, sdi, cg, scanopts, drvopts, devopts);
        } else {
                switch (key) {
@@ -288,7 +287,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
                return SR_ERR;
        }
 
-       /* Poll every 100ms, or whenever some data comes in. */
        serial_source_add(sdi->session, serial, G_IO_IN, 100,
                          reloadpro_receive_data, (void *)sdi);
 
index 8307053b27b2198d3f6409440b630d01540b4751..d954fcacc441f968fbcab827809ec96838042162 100644 (file)
@@ -29,7 +29,6 @@
 
 #define RELOADPRO_BUFSIZE 100
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
        uint8_t buf[RELOADPRO_BUFSIZE];
index bc6c87e86ebd51baf49e5c710f7daa587416b48b..163032c335079dc730300ee35894ffab18cc3ba5 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/*
- * ASIX SIGMA/SIGMA2 logic analyzer driver
- */
-
 #include <config.h>
 #include "protocol.h"
 
@@ -85,8 +81,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
 
        ftdi_init(&devc->ftdic);
 
-       /* Look for SIGMAs. */
-
        if ((ret = ftdi_usb_find_all(&devc->ftdic, &devlist,
            USB_VENDOR, USB_PRODUCT)) <= 0) {
                if (ret < 0)
@@ -116,7 +110,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        devc->capture_ratio = 50;
        devc->use_triggers = 0;
 
-       /* Register SIGMA device. */
        sdi = g_malloc0(sizeof(struct sr_dev_inst));
        sdi->status = SR_ST_INITIALIZING;
        sdi->vendor = g_strdup(USB_VENDOR_NAME);
@@ -127,7 +120,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
 
        sdi->priv = devc;
 
-       /* We will open the device again when we need it. */
        ftdi_list_free(&devlist);
 
        return std_scan_complete(di, g_slist_append(NULL, sdi));
index 01382e754d587af365dc9658e879e0941c60403d..dd4efd0d271847a7a81f5b0f5fcd70ae6d018d09 100644 (file)
@@ -677,7 +677,6 @@ SR_PRIV int sigma_convert_trigger(const struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
-
 /* Software trigger to determine exact trigger position. */
 static int get_trigger_offset(uint8_t *samples, uint16_t last_sample,
                              struct sigma_trigger *t)
index 888217699c8d93b06db924a639ca8865aaa2e0ba..98eb1a6837c89fe556b5e5ec676647de7946da44 100644 (file)
@@ -255,7 +255,6 @@ struct sigma_state {
        uint16_t lastsample;
 };
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
        struct ftdi_context ftdic;
        uint64_t cur_samplerate;
index c88914f9139aca7bd077abcf4eee83b062219170..0a16ca5f2c5b245d7c51455666abb1ddd9e6654f 100644 (file)
@@ -195,7 +195,6 @@ static int config_get(uint32_t key, GVariant **data, const struct sr_dev_inst *s
        devc = sdi->priv;
 
        if (!cg) {
-               /* No channel group: global options. */
                switch (key) {
                case SR_CONF_CHANNEL_CONFIG:
                        *data = g_variant_new_string(channel_modes[devc->channel_mode]);
@@ -263,7 +262,6 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
        ret = SR_OK;
        devc = sdi->priv;
        if (!cg) {
-               /* No channel group: global options. */
                switch (key) {
                case SR_CONF_CHANNEL_CONFIG:
                        sval = g_variant_get_string(data, NULL);
@@ -294,7 +292,6 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
                        return SR_ERR_NA;
                }
        } else {
-               /* Channel group specified: per-channel options. */
                /* We only ever have one channel per channel group in this driver. */
                ch = cg->channels->data;
                channel = ch->index;
@@ -342,7 +339,6 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
        devc = (sdi) ? sdi->priv : NULL;
 
        if (!cg) {
-               /* No channel group: global options. */
                switch (key) {
                case SR_CONF_SCAN_OPTIONS:
                case SR_CONF_DEVICE_OPTIONS:
@@ -360,8 +356,6 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
                        return SR_ERR_NA;
                }
        } else {
-               /* Channel group specified: per-channel options. */
-
                /* We only ever have one channel per channel group in this driver. */
                ch = cg->channels->data;
                channel = ch->index;
index b9a256d17ab820daf73e631890a70bc6ad05278b..f9409c2d1f7c01dcf61533fe3454a74c2b09809d 100644 (file)
@@ -71,15 +71,11 @@ struct per_channel_config {
        gboolean output_enabled_set;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        const struct pps_model *model;
 
-       /* Acquisition state */
        gboolean acquisition_running;
 
-       /* Operational state */
        gboolean config_dirty;
        struct per_channel_config *config;
        /* Blocking write timeout for packet. */
@@ -91,7 +87,6 @@ struct dev_context {
        int channel_mode_set;
        gboolean over_current_protection_set;
 
-       /* Temporary state across callbacks */
        uint8_t packet[PACKET_SIZE];
        int packet_size;
 
index 8717debd42cda396eb514983ab237a291355d48e..2425c992648f72ebdce4e0525bb8f79ac36d2147 100644 (file)
@@ -49,7 +49,6 @@ enum probe_type {
        PROBE_TEMP,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        uint64_t samplerate;
        struct sr_sw_limits limits;
index e35a9a7347e8ee6410c8871e4ef815392a25ba14..e450a094027317bd06551aeb2b64cee0fe037adc 100644 (file)
@@ -21,7 +21,6 @@
 #include "protocol.h"
 #include "beaglelogic.h"
 
-/* Scan options */
 static const uint32_t scanopts[] = {
        SR_CONF_NUM_LOGIC_CHANNELS,
 };
@@ -30,7 +29,6 @@ static const uint32_t drvopts[] = {
        SR_CONF_LOGIC_ANALYZER,
 };
 
-/* Hardware capabilities */
 static const uint32_t devopts[] = {
        SR_CONF_CONTINUOUS,
        SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
@@ -40,7 +38,6 @@ static const uint32_t devopts[] = {
        SR_CONF_NUM_LOGIC_CHANNELS | SR_CONF_GET,
 };
 
-/* Trigger matching capabilities */
 static const int32_t soft_trigger_matches[] = {
        SR_TRIGGER_ZERO,
        SR_TRIGGER_ONE,
@@ -120,10 +117,8 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        beaglelogic_set_sampleunit(devc);
        beaglelogic_close(devc);
 
-       /* Signal */
        sr_info("BeagleLogic device found at "BEAGLELOGIC_DEV_NODE);
 
-       /* Fill the channels */
        for (i = 0; i < maxch; i++)
                sr_channel_new(sdi, i, SR_CHANNEL_LOGIC, TRUE,
                                channel_names[i]);
index e058a60ab6560af593b1a8e83716f0ab5fa70dbc..5df27dcdcfbe8bdf44f06641a32c36ac84204049 100644 (file)
 
 #define SAMPLEUNIT_TO_BYTES(x) ((x) == 1 ? 1 : 2)
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        int max_channels;
        uint32_t fw_ver;
 
-       /* Acquisition settings: see beaglelogic.h */
        uint64_t cur_samplerate;
        uint64_t limit_samples;
        uint32_t sampleunit;
@@ -49,7 +46,6 @@ struct dev_context {
        uint32_t bufunitsize;
        uint32_t buffersize;
 
-       /* Operational state */
        int fd;
        GPollFD pollfd;
        int last_error;
index d9575c00ffeb71bb2f9f9b601f366bf8a98b15dc..3b4da51611a489f6a8d1668e66fe842276dfef5e 100644 (file)
@@ -101,7 +101,6 @@ static int dev_open(struct sr_dev_inst *sdi)
        if ((ret = sr_usb_open(drvc->sr_ctx->libusb_ctx, usb)) < 0)
                return SR_ERR;
 
-       /* Detach kernel drivers which grabbed this device (if any). */
        if (libusb_kernel_driver_active(usb->devhdl, 0) == 1) {
                ret = libusb_detach_kernel_driver(usb->devhdl, 0);
                if (ret < 0) {
@@ -112,7 +111,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                devc->detached_kernel_driver = 1;
        }
 
-       /* Claim interface 0. */
        if ((ret = libusb_claim_interface(usb->devhdl, 0)) < 0) {
                sr_err("Failed to claim interface 0: %s.",
                       libusb_error_name(ret));
index 4cba7674b7de5ac1a3fa2d4cbc556713d7b85bca..c5c4d596807330ee3ce994abddf93d34df86136c 100644 (file)
 
 #define LOG_PREFIX "brymen-bm86x"
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Acquisition settings */
        struct sr_sw_limits sw_limits;
-
-       /* Operational state */
-       int detached_kernel_driver;/**< Whether kernel driver was detached or not */
-
-       /* Temporary state across callbacks */
+       int detached_kernel_driver; /**< Whether kernel driver was detached or not */
        int interrupt_pending;
 };
 
index fb66d9bfc0fc90294edb69404271eedb48a1c764..d859314375033146b0c13c3052cf6067782c3fdd 100644 (file)
@@ -110,13 +110,10 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        if (!conn)
                return NULL;
 
-       if (serialcomm) {
-               /* Use the provided comm specs. */
+       if (serialcomm)
                devices = brymen_scan(di, conn, serialcomm);
-       } else {
-               /* But 9600/8n1 should work all of the time. */
+       else
                devices = brymen_scan(di, conn, "9600/8n1/dtr=1/rts=1");
-       }
 
        return devices;
 }
@@ -149,7 +146,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->sw_limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 50ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
                        brymen_dmm_receive_data, (void *)sdi);
index 7e02180ec1de3e31005150da0da185a70602f14e..d5090ee63b3783524d378ce8b523a68fd0c708a7 100644 (file)
@@ -38,7 +38,6 @@ enum packet_len_status {
        PACKET_INVALID_HEADER,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits sw_limits;
 
index 4bdbc076c212df008b086220766b43a7b2455dce..79689cd9e10ef1da90b781966f225935fef93634 100644 (file)
@@ -330,7 +330,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 
        std_session_send_df_header(sdi);
 
-       /* Poll every 100ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 150,
                        cem_dt_885x_receive_data, (void *)sdi);
index bb0984225e0dc5de2dfa5a3a5951aa2778f0c627..f183ed11be26773eeee9b0f495b0e94da1105a3e 100644 (file)
@@ -87,23 +87,18 @@ enum {
        DATA_SOURCE_MEMORY,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Device state */
        enum sr_mqflag cur_mqflags;
        int recording;
        int cur_meas_range;
        int cur_data_source;
 
-       /* Acquisition settings */
        uint64_t limit_samples;
 
-       /* Operational state */
        int state;
        uint64_t num_samples;
        gboolean enable_data_source_memory;
 
-       /* Temporary state across callbacks */
        unsigned char cmd;
        unsigned char token;
        int buf_len;
index 5e34f696c1f7fc43273c8a07a01a25a84b851ef4..3f517a22e2059e4a1257dbea39e81f1444b9ef0b 100644 (file)
@@ -109,13 +109,10 @@ static GSList *scan(GSList *options, int idx)
        if (!conn)
                return NULL;
 
-       if (serialcomm) {
-               /* Use the provided comm specs. */
+       if (serialcomm)
                devices = center_scan(conn, serialcomm, idx);
-       } else {
-               /* Try the default. */
+       else
                devices = center_scan(conn, center_devs[idx].conn, idx);
-       }
 
        return std_scan_complete(center_devs[idx].di, devices);
 }
@@ -149,7 +146,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, int idx)
 
        std_session_send_df_header(sdi);
 
-       /* Poll every 500ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 500,
                      center_devs[idx].receive_data, (void *)sdi);
index fbbdaf3010c59377a3a2d3b6c3089af1a0480ce9..6bd2c27c20c1c4f4dc26ca5a0388ba14c0c173ae 100644 (file)
@@ -51,7 +51,6 @@ extern SR_PRIV const struct center_dev_info center_devs[];
 
 #define SERIAL_BUFSIZE 256
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits sw_limits;
 
index d02ef3ef587fa9a4039a7e940e852972f6ca9886..3bf5609544763b13f37d7aff975083eac1cdbc54 100644 (file)
@@ -65,7 +65,6 @@ static int add_device(int model, struct libusb_device_descriptor *des,
 
        ret = SR_OK;
 
-       /* Allocate memory for our private device context. */
        devc = g_malloc0(sizeof(struct dev_context));
 
        /* Set some sane defaults. */
@@ -97,7 +96,6 @@ static int add_device(int model, struct libusb_device_descriptor *des,
        /* We now know the device, set its max. samplerate as default. */
        devc->cur_samplerate = devc->prof->max_samplerate;
 
-       /* Register the device with libsigrok. */
        sdi = g_malloc0(sizeof(struct sr_dev_inst));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup("ChronoVu");
@@ -231,7 +229,6 @@ static int dev_open(struct sr_dev_inst *sdi)
 
        devc = sdi->priv;
 
-       /* Allocate memory for the FTDI context and initialize it. */
        if (!(devc->ftdic = ftdi_new())) {
                sr_err("Failed to initialize libftdi.");
                return SR_ERR;
@@ -240,7 +237,6 @@ static int dev_open(struct sr_dev_inst *sdi)
        sr_dbg("Opening %s device (%04x:%04x).", devc->prof->modelname,
               devc->usb_vid, devc->usb_pid);
 
-       /* Open the device. */
        if ((ret = ftdi_usb_open_desc(devc->ftdic, devc->usb_vid,
                        devc->usb_pid, devc->prof->iproduct, NULL)) < 0) {
                sr_err("Failed to open FTDI device (%d): %s.",
@@ -248,21 +244,18 @@ static int dev_open(struct sr_dev_inst *sdi)
                goto err_ftdi_free;
        }
 
-       /* Purge RX/TX buffers in the FTDI chip. */
        if ((ret = ftdi_usb_purge_buffers(devc->ftdic)) < 0) {
                sr_err("Failed to purge FTDI buffers (%d): %s.",
                       ret, ftdi_get_error_string(devc->ftdic));
                goto err_ftdi_free;
        }
 
-       /* Enable flow control in the FTDI chip. */
        if ((ret = ftdi_setflowctrl(devc->ftdic, SIO_RTS_CTS_HS)) < 0) {
                sr_err("Failed to enable FTDI flow control (%d): %s.",
                       ret, ftdi_get_error_string(devc->ftdic));
                goto err_ftdi_free;
        }
 
-       /* Wait 100ms. */
        g_usleep(100 * 1000);
 
        return SR_OK;
index 609f37d9c5888b8ad1f6636037ddb826d91b018d..1e2036f7bab1c7134f6d242700699d26ed6e0139 100644 (file)
@@ -44,28 +44,18 @@ enum {
 struct cv_profile {
        int model;
        const char *modelname;
-       const char *iproduct; /* USB iProduct string */
+       const char *iproduct;
        unsigned int num_channels;
        uint64_t max_samplerate;
        const int num_trigger_matches;
        float trigger_constant;
 };
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
-       /** Device profile struct for this device. */
        const struct cv_profile *prof;
-
-       /** FTDI device context (used by libftdi). */
        struct ftdi_context *ftdic;
-
-       /** The currently configured samplerate of the device. */
        uint64_t cur_samplerate;
-
-       /** The current sampling limit (in ms). */
        uint64_t limit_msec;
-
-       /** The current sampling limit (in number of samples). */
        uint64_t limit_samples;
 
        /**
@@ -124,7 +114,6 @@ struct dev_context {
        uint64_t samplerates[255];
 };
 
-/* protocol.c */
 extern SR_PRIV const char *cv_channel_names[];
 extern const struct cv_profile cv_profiles[];
 SR_PRIV void cv_fill_samplerates_if_needed(const struct sr_dev_inst *sdi);
index 7db1549be8ff78b3c19cdd67150bbf4afe437472..5281178f13531d21027fb1cd67ca4ab293bba5ad 100644 (file)
@@ -110,7 +110,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 150ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 150,
                        colead_slm_receive_data, (void *)sdi);
index 8b298bb7648b132bd901502a0932c7d8d2713525..1849a865193300736e32bb571a0388b10eaef08f 100644 (file)
@@ -31,7 +31,6 @@ enum {
        COMMAND_SENT,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
 
index d1454acbf404f7f5aba5ca28decf8e82a702be24..141a3d9715eedd55d7d2903eafa780d6f35b8f9a 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Conrad DIGI 35 CPU</em> power supply driver
- *
- * @internal
- */
-
 #include <config.h>
 #include "protocol.h"
 
index 732bec62c7805169daaa74a840950c2dab01b626..78f7d1c34c0f7136db9396f374b590fd45c440de 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- * <em>Conrad DIGI 35 CPU</em> power supply driver
- * @internal
- */
-
 #include <config.h>
 #include "protocol.h"
 
index 99d2cb7041e2da670edb847469fcaa1318208ed3..b02af9e2a49cdf0960b3656d228c37ef3b2ee3e6 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Conrad DIGI 35 CPU</em> power supply driver
- *
- * @internal
- */
-
 #ifndef LIBSIGROK_HARDWARE_CONRAD_DIGI_35_CPU_PROTOCOL_H
 #define LIBSIGROK_HARDWARE_CONRAD_DIGI_35_CPU_PROTOCOL_H
 
index 7930624d463ed94c4a6258455421d79fca4d33a2..0b13ebfba2ef9b4a217d980bc997eab46ed8b475 100644 (file)
@@ -34,7 +34,6 @@
 /* Size of the analog pattern space per channel. */
 #define ANALOG_BUFSIZE                 4096
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
        uint64_t cur_samplerate;
        uint64_t limit_samples;
@@ -105,9 +104,6 @@ enum {
 
 /* Analog patterns we can generate. */
 enum {
-       /**
-        * Square wave.
-        */
        PATTERN_SQUARE,
        PATTERN_SINE,
        PATTERN_TRIANGLE,
index 08dded0140364826bd61bf73d45b4affc8c06ab9..6427cfdc7c5cb579d834fb43fab2ad7fc2faf9db 100644 (file)
@@ -67,7 +67,6 @@ static const uint32_t devopts[] = {
        SR_CONF_CLOCK_EDGE | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
 };
 
-/* Names assigned to available edge slope choices. */
 static const char *const signal_edge_names[] = {
        [DS_EDGE_RISING] = "rising",
        [DS_EDGE_FALLING] = "falling",
@@ -223,7 +222,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                        }
                }
 
-               /* Skip if the device was not found. */
                if (!prof)
                        continue;
 
index 467389bd7b4afd972cecea687fecbadf1a281d9e..f71506649fd4ba213eb2b4a882fc27f890905989 100644 (file)
@@ -112,16 +112,13 @@ struct dev_context {
         */
        int64_t fw_updated;
 
-       /* Supported samplerates */
        const uint64_t *samplerates;
        int num_samplerates;
 
-       /* Device/capture settings */
        uint64_t cur_samplerate;
        uint64_t limit_samples;
        uint64_t capture_ratio;
 
-       /* Operational settings */
        gboolean acq_aborted;
 
        unsigned int sent_samples;
index 50add95d7be20778f0a736a85de8c53b51234910..d91a2807e8cacd0989b72f171b2f16aede276252 100644 (file)
@@ -210,7 +210,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 100ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
                        fluke_receive_data, (void *)sdi);
index b8afc9000bf268710e82f6bd91f63cecf6b03516..36ece3c9eda9c7d7aee5a8d0b46e57afe8eca182 100644 (file)
@@ -46,12 +46,10 @@ struct flukedmm_profile {
        int timeout;
 };
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
        const struct flukedmm_profile *profile;
        struct sr_sw_limits limits;
 
-       /* Runtime. */
        char buf[FLUKEDMM_BUFSIZE];
        int buflen;
        int64_t cmd_sent_at;
index c2914eea89f4f47fc47fa58980ed1f953736210b..c59ff9ad493e0572ef950fa918eb9bc185fcd268 100644 (file)
@@ -111,7 +111,6 @@ static void scan_device(struct ftdi_context *ftdic,
                return;
        }
 
-       /* Allocate memory for our private device context. */
        devc = g_malloc0(sizeof(struct dev_context));
 
        /* Allocate memory for the incoming data. */
@@ -138,7 +137,6 @@ static void scan_device(struct ftdi_context *ftdic,
        }
        sr_dbg("Found an FTDI device: %s.", model);
 
-       /* Register the device with libsigrok. */
        sdi = g_malloc0(sizeof(struct sr_dev_inst));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = vendor;
@@ -214,7 +212,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                }
        }
 
-       /* Allocate memory for the FTDI context (ftdic) and initialize it. */
        ftdic = ftdi_new();
        if (!ftdic) {
                sr_err("Failed to initialize libftdi.");
@@ -273,7 +270,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                goto err_ftdi_free;
        }
 
-       /* Purge RX/TX buffers in the FTDI chip. */
        ret = ftdi_usb_purge_buffers(devc->ftdic);
        if (ret < 0) {
                sr_err("Failed to purge FTDI RX/TX buffers (%d): %s.",
@@ -281,7 +277,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                goto err_dev_open_close_ftdic;
        }
 
-       /* Reset the FTDI bitmode. */
        ret = ftdi_set_bitmode(devc->ftdic, 0x00, BITMODE_RESET);
        if (ret < 0) {
                sr_err("Failed to reset the FTDI chip bitmode (%d): %s.",
index d5ba9b09cbb0149e1115d17034788fc27345d7b6..495c4dbbccf1c1264e1bce566c41a2d50d48a0bb 100644 (file)
@@ -36,7 +36,6 @@ struct ftdi_chip_desc {
        char *channel_names[];
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct ftdi_context *ftdic;
        const struct ftdi_chip_desc *desc;
index b9d43428799cdf35cfac164c5bc6cc9efdd188b7..9e5b24241e42fc92f3f90841b1202700578493b2 100644 (file)
@@ -268,7 +268,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                        }
                }
 
-               /* Skip if the device was not found. */
                if (!prof)
                        continue;
 
index f5ef67c99bbb3fa2cd8225554bf9d0beb4997561..1403c8ab09adf3a49ad53e3ff289c3877cd15f9e 100644 (file)
@@ -98,16 +98,13 @@ struct dev_context {
         */
        int64_t fw_updated;
 
-       /* Supported samplerates */
        const uint64_t *samplerates;
        int num_samplerates;
 
-       /* Device/capture settings */
        uint64_t cur_samplerate;
        uint64_t limit_samples;
        uint64_t capture_ratio;
 
-       /* Operational settings */
        gboolean trigger_fired;
        gboolean acq_aborted;
        gboolean sample_wide;
index 7443f487eed26440f45503cb9b7c64eb35756ad4..91f12560074d3c9dfc8eb01681800802c70d4efe 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * Gossen Metrawatt Metrahit 1x/2x drivers
- *
- * @internal
- */
-
 #include <config.h>
 #include <string.h>
 #include "protocol.h"
@@ -388,7 +380,6 @@ static int dev_acquisition_start_1x_2x_rs232(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 40ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 40,
                        gmc_mh_1x_2x_receive_data, (void *)sdi);
@@ -408,7 +399,6 @@ static int dev_acquisition_start_2x_bd232(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 40ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 40,
                        gmc_mh_2x_receive_data, (void *)sdi);
index 5820a0ec89aa1e1429490cb976fa61f3dfd53c5e..53efdf4255f7500d60c4f07ef36ac6e537f4dff3 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * Gossen Metrawatt Metrahit 1x/2x drivers
- *
- * @internal
- */
-
 #include <config.h>
 #include <math.h>
 #include <string.h>
 #include "protocol.h"
 
-/* Internal Headers */
 static guchar calc_chksum_14(guchar *dta);
 static int chk_msg14(struct sr_dev_inst *sdi);
 
@@ -1283,9 +1274,7 @@ static void create_cmd_14(guchar addr, guchar func, guchar *params, guchar *buf)
        }
 }
 
-/** Request one measurement from 2x multimeter (msg 8).
- *
- */
+/** Request one measurement from 2x multimeter (msg 8). */
 int req_meas14(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -1361,7 +1350,8 @@ int req_stat14(const struct sr_dev_inst *sdi, gboolean power_on)
        return SR_OK;
 }
 
-/** Decode model in "send mode".
+/**
+ * Decode model in "send mode".
  *
  * @param[in] mcode Model code.
  * @return Model code.
index 52563d95e0ab9597cac755a57a59feccd4df07ca..ed759694861d2b59c8e6b75c626a7313b18012d8 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * Gossen Metrawatt Metrahit 1x/2x drivers
- *
- * @internal
- */
-
 #ifndef LIBSIGROK_HARDWARE_GMC_MH_1X_2X_PROTOCOL_H
 #define LIBSIGROK_HARDWARE_GMC_MH_1X_2X_PROTOCOL_H
 
@@ -84,15 +76,11 @@ enum model {
        METRAHIT_29S            = METRAHIT_28S + 1,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        enum model model;       /**< Model code. */
 
-       /* Acquisition settings */
        struct sr_sw_limits limits;
 
-       /* Operational state */
        gboolean settings_ok;   /**< Settings msg received yet. */
        int msg_type;       /**< Message type (MSGID_INF, ...). */
        int msg_len;        /**< Message length (valid when msg, curr. type known).*/
@@ -112,12 +100,10 @@ struct dev_context {
        int64_t req_sent_at;    /**< Request sent. */
        gboolean response_pending; /**< Request sent, response is pending. */
 
-       /* Temporary state across callbacks */
        uint8_t buf[GMC_BUFSIZE];       /**< Buffer for read callback */
        int buflen;                     /**< Data len in buf */
 };
 
-/* Forward declarations */
 SR_PRIV int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sdi,
                const struct sr_channel_group *cg);
 SR_PRIV int gmc_decode_model_bd(uint8_t mcode);
index 382e87b8cac5b3253df981486837c4a2374681be..d5c2e5a0de185698debe6d78b649234f00d0c86a 100644 (file)
@@ -44,7 +44,6 @@ enum gds_state
        WAIT_FOR_TRANSFER_OF_CHANNEL_DATA_COMPLETE,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        enum gds_state state;
        uint64_t cur_acq_frame;
index ccdcb90cf3d3fe4b4ddce0e8280c387c46502159..74ef3da9a0afb137f613089f692092a846f360ff 100644 (file)
@@ -92,7 +92,6 @@ struct scope_state {
        uint64_t sample_rate;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        const void *model_config;
        void *model_state;
index aa8ab5284bc3664feaa8cffee781221eb633d683..f0197290f3b6d0e4825194335309d8a1a0df8d42 100644 (file)
@@ -25,6 +25,7 @@
 static int set_mq_volt(struct sr_scpi_dev_inst *scpi, enum sr_mqflag flags);
 static int set_mq_amp(struct sr_scpi_dev_inst *scpi, enum sr_mqflag flags);
 static int set_mq_ohm(struct sr_scpi_dev_inst *scpi, enum sr_mqflag flags);
+
 /*
  * The source for the frequency measurement can be either AC voltage, AC+DC
  * voltage, AC current, or AC+DC current. Configuring this is not yet
index dda19a23610e7e75e7df82f579da648bd059cb80..3fef0efaf8044132c25888dfcbc8da5d0402b5bf 100644 (file)
@@ -57,13 +57,10 @@ enum channel_conn {
        CONN_REAR,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        /* Information about rear card option, or NULL if unknown */
        const struct rear_card_info *rear_card;
 
-       /* Acquisition settings */
        enum sr_mq measurement_mq;
        enum sr_mqflag measurement_mq_flags;
        enum sr_unit measurement_unit;
@@ -73,7 +70,6 @@ struct dev_context {
        unsigned int num_active_channels;
        struct sr_channel *current_channel;
 
-       /* Operational state */
        enum acquisition_state acq_state;
        enum channel_conn input_loc;
        uint64_t num_samples;
index 7168bd69ff9c0491e57d960ef838304240c7715e..f7268f3b133c4cceb96b9ef174b1e2c665a015c2 100644 (file)
@@ -29,9 +29,7 @@
 #define MAX_RETRIES 4
 #define NUM_CHANNELS 2
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Acquisition settings */
        GSList *enabled_channel;
        uint8_t channel;
        uint8_t rate;
@@ -42,7 +40,6 @@ struct dev_context {
        uint8_t offset[2];
        uint8_t gain[2];
 
-       /* Operational state */
        uint64_t frame_limit;
        uint64_t frame;
        uint64_t probe[2];
@@ -51,7 +48,6 @@ struct dev_context {
        uint8_t retries;
        gboolean adc2;
 
-       /* Temporary state across callbacks */
        float *samples;
        float factor;
        gboolean state_known;
index 8b9e7f169d9fe1b7bbaeb48d80f5a790027c04cd..8005fe074332062936d1a3643bdefe2266e1e910 100644 (file)
@@ -181,10 +181,6 @@ static int dev_open(struct sr_dev_inst *sdi)
        if (sr_usb_open(drvc->sr_ctx->libusb_ctx, usb) != SR_OK)
                return SR_ERR;
 
-       /*
-        * Determine if a kernel driver is active on this interface and, if so,
-        * detach it.
-        */
        if (libusb_kernel_driver_active(usb->devhdl, USB_INTERFACE) == 1) {
                ret = libusb_detach_kernel_driver(usb->devhdl, USB_INTERFACE);
                if (ret < 0) {
index c58050bbb102e316c3d580108793405f81415f8c..6dd66999bd2b3e7fcb921904f80b00a47207021a 100644 (file)
@@ -640,10 +640,6 @@ SR_PRIV int sl2_get_device_info(struct sr_dev_driver *di,
        if (sr_usb_open(drvc->sr_ctx->libusb_ctx, &usb) != SR_OK)
                return SR_ERR;
 
-       /*
-        * Determine if a kernel driver is active on this interface and, if so,
-        * detach it.
-        */
        if (libusb_kernel_driver_active(usb.devhdl, USB_INTERFACE) == 1) {
                ret = libusb_detach_kernel_driver(usb.devhdl,
                        USB_INTERFACE);
index d7a273574d78276d1504dbb4f616a62d16d42dfa..bb9de6663b6c1cde6cbfd5f19fe1580b4043c12c 100644 (file)
@@ -120,7 +120,6 @@ enum {
        STATE_WAIT_DEVICE_READY
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        /* Current selected samplerate. */
        uint64_t samplerate;
index 67da9b39c1a3952533aeaa5992e2f78846aef104..86c1402e511e635ec2fe4538140f143d555befe3 100644 (file)
@@ -38,7 +38,6 @@ static const uint32_t devopts[] = {
        SR_CONF_SAMPLERATE | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
 };
 
-/* Channels are numbered 1-9. */
 static const char *channel_names[] = {
        "1", "2", "3", "4", "5", "6", "7", "8", "9",
 };
@@ -67,7 +66,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
 
        (void)options;
 
-       /* Allocate memory for our private device context. */
        devc = g_malloc0(sizeof(struct dev_context));
 
        /* Allocate memory for the incoming compressed samples. */
@@ -82,13 +80,11 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                goto err_free_compressed_buf;
        }
 
-       /* Allocate memory for the FTDI context (ftdic) and initialize it. */
        if (!(devc->ftdic = ftdi_new())) {
                sr_err("Failed to initialize libftdi.");
                goto err_free_sample_buf;
        }
 
-       /* Check for the device and temporarily open it. */
        ret = ftdi_usb_open_desc(devc->ftdic, USB_VENDOR_ID, USB_DEVICE_ID,
                                 USB_IPRODUCT, NULL);
        if (ret < 0) {
@@ -99,7 +95,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                goto err_free_ftdic;
        }
 
-       /* Register the device with libsigrok. */
        sdi = g_malloc0(sizeof(struct sr_dev_inst));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup(USB_VENDOR_NAME);
@@ -109,14 +104,13 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        for (i = 0; i < ARRAY_SIZE(channel_names); i++)
                sr_channel_new(sdi, i, SR_CHANNEL_LOGIC, TRUE, channel_names[i]);
 
-       /* Close device. We'll reopen it again when we need it. */
        scanaplus_close(devc);
 
        return std_scan_complete(di, g_slist_append(NULL, sdi));
 
        scanaplus_close(devc);
 err_free_ftdic:
-       ftdi_free(devc->ftdic); /* NOT free() or g_free()! */
+       ftdi_free(devc->ftdic);
 err_free_sample_buf:
        g_free(devc->sample_buf);
 err_free_compressed_buf:
@@ -134,7 +128,6 @@ static int dev_open(struct sr_dev_inst *sdi)
 
        devc = sdi->priv;
 
-       /* Select interface A, otherwise communication will fail. */
        ret = ftdi_set_interface(devc->ftdic, INTERFACE_A);
        if (ret < 0) {
                sr_err("Failed to set FTDI interface A (%d): %s", ret,
@@ -142,7 +135,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                return SR_ERR;
        }
 
-       /* Open the device. */
        ret = ftdi_usb_open_desc(devc->ftdic, USB_VENDOR_ID, USB_DEVICE_ID,
                                 USB_IPRODUCT, NULL);
        if (ret < 0) {
@@ -151,14 +143,12 @@ static int dev_open(struct sr_dev_inst *sdi)
                return SR_ERR;
        }
 
-       /* Purge RX/TX buffers in the FTDI chip. */
        if ((ret = ftdi_usb_purge_buffers(devc->ftdic)) < 0) {
                sr_err("Failed to purge FTDI RX/TX buffers (%d): %s.",
                       ret, ftdi_get_error_string(devc->ftdic));
                goto err_dev_open_close_ftdic;
        }
 
-       /* Reset the FTDI bitmode. */
        ret = ftdi_set_bitmode(devc->ftdic, 0xff, BITMODE_RESET);
        if (ret < 0) {
                sr_err("Failed to reset the FTDI chip bitmode (%d): %s.",
@@ -166,7 +156,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                goto err_dev_open_close_ftdic;
        }
 
-       /* Set FTDI bitmode to "sync FIFO". */
        ret = ftdi_set_bitmode(devc->ftdic, 0xff, BITMODE_SYNCFF);
        if (ret < 0) {
                sr_err("Failed to put FTDI chip into sync FIFO mode (%d): %s.",
@@ -174,7 +163,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                goto err_dev_open_close_ftdic;
        }
 
-       /* Set the FTDI latency timer to 2. */
        ret = ftdi_set_latency_timer(devc->ftdic, 2);
        if (ret < 0) {
                sr_err("Failed to set FTDI latency timer (%d): %s.",
@@ -182,7 +170,6 @@ static int dev_open(struct sr_dev_inst *sdi)
                goto err_dev_open_close_ftdic;
        }
 
-       /* Set the FTDI read data chunk size to 64kB. */
        ret = ftdi_read_data_set_chunksize(devc->ftdic, 64 * 1024);
        if (ret < 0) {
                sr_err("Failed to set FTDI read data chunk size (%d): %s.",
index f797832449cbbe9c5341c3eae744520fcd1c6f2f..fde2e4e2ef8a174265d64292f4a08e5b3b08759a 100644 (file)
 
 #define COMPRESSED_BUF_SIZE (64 * 1024)
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
-       /** FTDI device context (used by libftdi). */
        struct ftdi_context *ftdic;
-
-       /** The current sampling limit (in ms). */
        uint64_t limit_msec;
-
-       /** The current sampling limit (in number of samples). */
        uint64_t limit_samples;
 
        uint8_t *compressed_buf;
index 1ee5c5e2f43bca66d3d2f22b0d7d32bcc98c3bd2..1bbe5746b36ec2fe666afc23202c13dc93166829 100644 (file)
@@ -20,6 +20,7 @@
 #include <config.h>
 #include <string.h>
 #include "protocol.h"
+
 extern const uint64_t kecheng_kc_330b_sample_intervals[][2];
 
 SR_PRIV int kecheng_kc_330b_handle_events(int fd, int revents, void *cb_data)
index e2090d84dfdc999fa1ffe4d1fa490e7cd099ce00..e750ff472ce6d801f080a973883f6ed321d57ca5 100644 (file)
@@ -66,9 +66,7 @@ enum {
        DEVICE_INACTIVE,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Acquisition settings */
        uint64_t limit_samples;
        int sample_interval;
        int alarm_low;
@@ -76,7 +74,6 @@ struct dev_context {
        enum sr_mqflag mqflags;
        int data_source;
 
-       /* Operational state */
        int state;
        gboolean config_dirty;
        uint64_t num_samples;
@@ -84,9 +81,7 @@ struct dev_context {
        struct libusb_transfer *xfer;
        unsigned char buf[128];
 
-       /* Temporary state across callbacks */
        gint64 last_live_request;
-
 };
 
 SR_PRIV int kecheng_kc_330b_handle_events(int fd, int revents, void *cb_data);
index 283e5b17e553c39c815012ba2c6d3e70c70fd42e..7b7d3d082093d170fe8a53553dcd311934b45477 100644 (file)
@@ -147,7 +147,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 50ms, or whenever some data comes in. */
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
                      kern_scale_receive_data, (void *)sdi);
 
index 934aff02064a5c3660ff5e9264ebc67eabca7aa3..836eb91b0404014ac7670cae80467f656743271c 100644 (file)
@@ -46,7 +46,6 @@ struct scale_info {
 
 #define SCALE_BUFSIZE 256
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
 
index 04a26834114efbfe865216b5fa362042651f2b67..acd50961f25a1487f32ace54a56cb98e169b6164 100644 (file)
@@ -26,17 +26,13 @@ static const uint32_t scanopts[] = {
 };
 
 static const uint32_t drvopts[] = {
-       /* Device class */
        SR_CONF_POWER_SUPPLY,
 };
 
 static const uint32_t devopts[] = {
-       /* Device class */
-       /* Acquisition modes. */
        SR_CONF_CONTINUOUS,
        SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
        SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET,
-       /* Device configuration */
        SR_CONF_VOLTAGE | SR_CONF_GET,
        SR_CONF_VOLTAGE_TARGET | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
        SR_CONF_CURRENT | SR_CONF_GET,
@@ -129,7 +125,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        sr_dbg("Found: %s %s (idx %d, ID '%s').", models[model_id].vendor,
                models[model_id].name, model_id, models[model_id].id);
 
-       /* Init device instance, etc. */
        sdi = g_malloc0(sizeof(struct sr_dev_inst));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup(models[model_id].vendor);
index e88360e7edcf0f10b0dc734746e766fac6a1114b..2423ce99a8e4f672d8d1aa0296546711d8f9afe5 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- * Korad KAxxxxP power supply driver
- * @internal
- */
-
 #ifndef LIBSIGROK_HARDWARE_KORAD_KAXXXXP_PROTOCOL_H
 #define LIBSIGROK_HARDWARE_KORAD_KAXXXXP_PROTOCOL_H
 
@@ -70,17 +64,13 @@ enum {
        KAXXXXP_RECALL,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        const struct korad_kaxxxxp_model *model; /**< Model information. */
 
-       /* Acquisition settings */
        struct sr_sw_limits limits;
        int64_t req_sent_at;
        gboolean reply_pending;
 
-       /* Operational state */
        float current;          /**< Last current value [A] read from device. */
        float current_max;      /**< Output current set. */
        float voltage;          /**< Last voltage value [V] read from device. */
@@ -92,7 +82,6 @@ struct dev_context {
        gboolean ocp_enabled;    /**< Output current protection enabled. */
        gboolean ovp_enabled;    /**< Output voltage protection enabled. */
 
-       /* Temporary state across callbacks */
        int target;              /**< What reply to expect. */
        int program;             /**< Program to store or recall. */
        char reply[6];
index 9e7b716783f94ea5a8d45158eac1c43ac2ca7764..14699096792959ca7d2fdec3900901983cf394c5 100644 (file)
@@ -39,9 +39,9 @@
 #define SLEEP_US_LONG (5 * 1000)
 #define SLEEP_US_SHORT (1 * 1000)
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        const struct elusb_profile *profile;
+
        /* Generic EL-USB */
        unsigned char config[MAX_CONFIGBLOCK_SIZE];
        unsigned int log_size;
@@ -50,6 +50,7 @@ struct dev_context {
        unsigned int logged_samples;
        unsigned int rcvd_samples;
        uint64_t limit_samples;
+
        /* Model-specific */
        /* EL-USB-CO: these are something like scaling and calibration values
         * fixed per device, used to convert the sample values to CO ppm. */
index 1ac3d0fc0bf69e386513396aa271f6adcba3cbcd..f6a0ea0992b91a46ec1872fa8cc0ab794c878f46 100644 (file)
@@ -33,7 +33,6 @@
 
 struct samplerate_info;
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct libusb_transfer *intr_xfer;
        struct libusb_transfer *bulk_xfer;
index 281a3871bf0050d74414fdc381770bf63d7bb15d..140308dc62f67dfc23be4a3936fad7cae2f82195 100644 (file)
@@ -504,6 +504,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 
        devc = sdi->priv;
        scpi = sdi->conn;
+
        /* Preset empty results. */
        g_slist_free(devc->enabled_channels);
        devc->enabled_channels = NULL;
index 0200c197d4b3659ff7dc0572bfbdc32e2a482e4f..266504e6290852f21949ce3cc3ca26a2f5473958 100644 (file)
@@ -76,7 +76,6 @@ struct scope_state {
        uint64_t sample_rate;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        const void *model_config;
        void *model_state;
index 6069c71b007d072aeb79f33c4d2c8b45ff4dd383..47ace67db499065df25161660bbad1b1f6d8ba2a 100644 (file)
@@ -374,7 +374,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
-/* This stops acquisition on ALL devices, ignoring dev_index. */
 static int dev_acquisition_stop(struct sr_dev_inst *sdi)
 {
        stop_acquisition(sdi);
index 4c8a66b26a22f0e241501be5dcc2e50f27492cea..5097d053d4b8ead07182e1de37ac71fb866da6d3 100644 (file)
@@ -84,7 +84,6 @@ struct mso_prototrig {
        uint8_t spimode;
 };
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
        /* info */
        uint8_t hwmodel;
@@ -97,10 +96,11 @@ struct dev_context {
        uint16_t offset_range;
        uint64_t limit_samples;
        uint64_t num_samples;
+
        /* register cache */
        uint8_t ctlbase1;
        uint8_t ctlbase2;
-       /* state */
+
        uint8_t la_threshold;
        uint64_t cur_rate;
        uint8_t dso_probe_attn;
index 4015ec76c89ad9413bcf67ba551736e8472f56e4..ab1e3b9bb954602e23d604d86fc6c4f47ce9e356 100644 (file)
  * along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Manson HCS-3xxx series</em> power supply driver
- *
- * @internal
- */
-
 #include <config.h>
 #include "protocol.h"
 
@@ -35,17 +27,13 @@ static const uint32_t scanopts[] = {
 };
 
 static const uint32_t drvopts[] = {
-       /* Device class */
        SR_CONF_POWER_SUPPLY,
 };
 
 static const uint32_t devopts[] = {
-       /* Device class */
-       /* Acquisition modes. */
        SR_CONF_CONTINUOUS,
        SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
        SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET,
-       /* Device configuration */
        SR_CONF_VOLTAGE | SR_CONF_GET,
        SR_CONF_VOLTAGE_TARGET | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
        SR_CONF_CURRENT | SR_CONF_GET,
@@ -137,7 +125,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
        }
        g_strfreev(tokens);
 
-       /* Init device instance, etc. */
        sdi = g_malloc0(sizeof(struct sr_dev_inst));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup("Manson");
@@ -337,7 +324,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        devc->reply_pending = FALSE;
        devc->req_sent_at = 0;
 
-       /* Poll every 10ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 10,
                        hcs_receive_data, (void *)sdi);
index 19d581802582fe169ec442c3c3e00074347b4a09..30017d1cdcbaeea092676db753ca31bdab4d8b42 100644 (file)
  * along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Manson HCS-3xxx Series</em> power supply driver
- *
- * @internal
- */
-
 #include <config.h>
 #include "protocol.h"
 
index 93cc54755b43410d77ffd9f7af59e84478019eb9..da47afb0c56f92eaf0e10874ef534a7005114289 100644 (file)
  * along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Manson HCS-3xxx Series</em> power supply driver
- *
- * @internal
- */
-
 #ifndef LIBSIGROK_HARDWARE_MANSON_HCS_3XXX_PROTOCOL_H
 #define LIBSIGROK_HARDWARE_MANSON_HCS_3XXX_PROTOCOL_H
 
@@ -66,7 +58,6 @@ struct hcs_model {
        double current[3]; /**< Min, max, step */
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        const struct hcs_model *model; /**< Model information. */
 
index 08bc16159820b0902921e84683812727ab54007c..970e843733d19299daba96fad30078db0512ea12 100644 (file)
@@ -365,7 +365,6 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
        devc = (sdi) ? sdi->priv : NULL;
 
        if (!cg) {
-               /* No channel group: global options. */
                return STD_CONFIG_LIST(key, data, sdi, cg, scanopts, drvopts, devopts);
        } else {
                switch (key) {
index a2e39f585bde5a2ef0560135c350ae5ec625e3e4..31589d901d745e19312e51b1df260efa32fa378d 100644 (file)
@@ -34,15 +34,9 @@ struct maynuo_m97_model {
        unsigned int max_power;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        const struct maynuo_m97_model *model;
-
-       /* Acquisition settings */
        struct sr_sw_limits limits;
-
-       /* Operational state */
        int expecting_registers;
 };
 
index 3af30a2add2c45945719ba34837de3f476e0fead..cec9a73fd278a7e128175cf888cadae3141616c0 100644 (file)
@@ -116,13 +116,10 @@ static GSList *scan(GSList *options, int idx)
        if (!conn)
                return NULL;
 
-       if (serialcomm) {
-               /* Use the provided comm specs. */
+       if (serialcomm)
                devices = mic_scan(conn, serialcomm, idx);
-       } else {
-               /* Try the default. */
+       else
                devices = mic_scan(conn, mic_devs[idx].conn, idx);
-       }
 
        return devices;
 }
@@ -166,7 +163,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi, int idx)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 100ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 100,
                      mic_devs[idx].receive_data, (void *)sdi);
index d4ad06d914b2d067055966584c385f37f238bb7d..c9ba0944a6aaf68004d2bf1e74fe069d485ec90f 100644 (file)
@@ -51,7 +51,6 @@ extern SR_PRIV const struct mic_dev_info mic_devs[];
 
 #define SERIAL_BUFSIZE 256
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
 
index d5a834f7e972376c093f7426dc1cb0e1f138d322..6164e4ace3cc3bcf2f3d136e0a459ed0128d9565 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Motech LPS-30x series</em> power supply driver
- *
- * @internal
- */
-
 #include <config.h>
 #include <ctype.h>
 #include <math.h>
@@ -38,31 +30,23 @@ SR_PRIV int lps_cmd_ok(struct sr_serial_dev_inst *serial, const char *fmt, ...);
 SR_PRIV int lps_cmd_reply(char *reply, struct sr_serial_dev_inst *serial, const char *fmt, ...);
 SR_PRIV int lps_query_status(struct sr_dev_inst *sdi);
 
-/* Serial communication parameters */
 #define SERIALCOMM "2400/8n1/dtr=1/rts=1/flow=0"
 
 #define VENDOR_MOTECH "Motech"
 
-/** Driver scanning options. */
 static const uint32_t scanopts[] = {
        SR_CONF_CONN,
        SR_CONF_SERIALCOMM,
 };
 
-/** Driver capabilities generic. */
 static const uint32_t drvopts[] = {
-       /* Device class */
        SR_CONF_POWER_SUPPLY,
 };
 
-/** Hardware capabilities generic. */
 static const uint32_t devopts[] = {
-       /* Device class */
-       /* Acquisition modes. */
        SR_CONF_CONTINUOUS,
        SR_CONF_LIMIT_SAMPLES | SR_CONF_GET | SR_CONF_SET,
        SR_CONF_LIMIT_MSEC | SR_CONF_GET | SR_CONF_SET,
-       /* Device configuration */
        SR_CONF_CHANNEL_CONFIG | SR_CONF_GET | SR_CONF_SET | SR_CONF_LIST,
 };
 
@@ -75,7 +59,7 @@ static const uint32_t devopts_ch12[] = {
        SR_CONF_ENABLED | SR_CONF_GET | SR_CONF_SET,
 };
 
-/** Hardware capabilities channel 3. (LPS-304/305 only). */
+/** Hardware capabilities channel 3 (LPS-304/305 only). */
 static const uint32_t devopts_ch3[] = {
        SR_CONF_VOLTAGE | SR_CONF_GET,
        SR_CONF_ENABLED | SR_CONF_GET | SR_CONF_SET,
@@ -133,8 +117,7 @@ static const struct lps_modelspec models[] = {
        },
 };
 
-/** Send command to device with va_list.
- */
+/** Send command to device with va_list. */
 SR_PRIV int lps_send_va(struct sr_serial_dev_inst *serial, const char *fmt, va_list args)
 {
        int retc;
@@ -155,8 +138,7 @@ SR_PRIV int lps_send_va(struct sr_serial_dev_inst *serial, const char *fmt, va_l
        return SR_OK;
 }
 
-/** Send command to device.
- */
+/** Send command to device. */
 SR_PRIV int lps_send_req(struct sr_serial_dev_inst *serial, const char *fmt, ...)
 {
        int retc;
@@ -367,8 +349,7 @@ SR_PRIV int lps_read_reply(struct sr_serial_dev_inst *serial, char **buf, int *b
        return SR_ERR; /* Timeout! */
 }
 
-/** Scan for LPS-300 series device.
- */
+/** Scan for LPS-300 series device. */
 static GSList *do_scan(lps_modelid modelid, struct sr_dev_driver *drv, GSList *options)
 {
        struct sr_dev_inst *sdi;
@@ -523,7 +504,6 @@ static int config_get(uint32_t key, GVariant **data, const struct sr_dev_inst *s
        devc = sdi->priv;
 
        if (!cg) {
-               /* No channel group: global options. */
                switch (key) {
                case SR_CONF_LIMIT_SAMPLES:
                case SR_CONF_LIMIT_MSEC:
@@ -538,6 +518,7 @@ static int config_get(uint32_t key, GVariant **data, const struct sr_dev_inst *s
                /* We only ever have one channel per channel group in this driver. */
                ch = cg->channels->data;
                ch_idx = ch->index;
+
                switch (key) {
                case SR_CONF_VOLTAGE:
                        *data = g_variant_new_double(devc->channel_status[ch_idx].output_voltage_last);
@@ -582,7 +563,6 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
                return SR_ERR_NA;
 
        if (!cg) {
-               /* No channel group: global options. */
                switch (key) {
                case SR_CONF_LIMIT_MSEC:
                case SR_CONF_LIMIT_SAMPLES:
@@ -609,7 +589,6 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
                        return SR_ERR_NA;
                }
        } else {
-               /* Channel group specified: per-channel options. */
                /* We only ever have one channel per channel group in this driver. */
                ch = cg->channels->data;
                ch_idx = ch->index;
@@ -679,7 +658,6 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
 
        devc = (sdi) ? sdi->priv : NULL;
 
-       /* Device options, independent from channel groups. */
        if (!cg) {
                switch (key) {
                case SR_CONF_SCAN_OPTIONS:
@@ -699,9 +677,10 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
                }
        }
 
-       /* Device options, depending on channel groups. */
+       /* We only ever have one channel per channel group in this driver. */
        ch = cg->channels->data;
        ch_idx = ch->index;
+
        switch (key) {
        case SR_CONF_DEVICE_OPTIONS:
                if ((ch_idx == 0) || (ch_idx == 1)) /* CH1, CH2 */
index 867cbb45197fca719b7c5d553548e9fc19446d81..c952ed0bc6bf7b91f02a20ef8369b2ff93336a65 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Motech LPS-30x series</em> power supply driver
- *
- * @internal
- */
-
 #include <config.h>
 #include <errno.h>
 #include <string.h>
index cb74148f468dcf82c75871cef191747c8239f164..99e453eba15ff76f3c3466b0cb81c51f1a9140b2 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Motech LPS-30x series</em> power supply driver
- *
- * @internal
- */
-
 #ifndef LIBSIGROK_HARDWARE_MOTECH_LPS_30X_PROTOCOL_H
 #define LIBSIGROK_HARDWARE_MOTECH_LPS_30X_PROTOCOL_H
 
@@ -93,22 +85,17 @@ struct channel_status {
        gdouble output_current_max;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        const struct lps_modelspec *model;
 
-       /* Acquisition status */
        gboolean acq_running;           /**< Acquisition is running. */
        struct sr_sw_limits limits;
        acquisition_req acq_req;        /**< Current request. */
        uint8_t acq_req_pending;        /**< Request pending. 0=none, 1=reply, 2=OK */
 
-       /* Operational state */
        struct channel_status channel_status[MAX_CHANNELS];
        guint8 tracking_mode;           /**< 0=off, 1=Tracking from CH1, 2=Tracking from CH2. */
 
-       /* Temporary state across callbacks */
        int64_t req_sent_at;    /**< Request sent. */
        gchar buf[LINELEN_MAX]; /**< Buffer for read callback */
        int buflen;             /**< Data len in buf */
index 96d69b12f1e07dfbd1c99917ef66b20a8b79ae47..094fe951cf165b671604f762fc31539176e75a58 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/** @file
- *  Norma DM9x0/Siemens B102x DMMs driver.
- *  @internal
- */
-
 #include <config.h>
 #include "protocol.h"
 
@@ -192,7 +187,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 100ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 100,
                        norma_dmm_receive_data, (void *)sdi);
index 804fc36e37b77c5474bfd39467e38d4cb9267a4c..39924176d484c4ddab04f126a4d245c8dba888c0 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * Norma DM9x0/Siemens B102x DMMs driver.
- *
- * @internal
- */
-
 #include <config.h>
 #include "protocol.h"
 
index bec6107046b51e0a11f5f66a6e7aca1b973c1f78..9891741ee88e0101b34be1bf51deb37c316f7ec2 100644 (file)
 #include <libsigrok/libsigrok.h>
 #include "libsigrok-internal.h"
 
-/**
- * @file
- *
- * Norma DM9x0/Siemens B102x DMMs driver.
- *
- * @internal
- */
-
 #define LOG_PREFIX "norma-dmm"
 
 #define NMADMM_BUFSIZE 256
@@ -57,21 +49,16 @@ struct nmadmm_req {
 /** Strings for requests. */
 extern const struct nmadmm_req nmadmm_requests[];
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        int type;               /**< DM9x0, e.g. 5 = DM950 */
 
-       /* Acquisition settings */
        struct sr_sw_limits limits;
 
-       /* Operational state */
        int last_req;                   /**< Last request. */
        int64_t req_sent_at;            /**< Request sent. */
        gboolean last_req_pending;      /**< Last request not answered yet. */
        int lowbatt;                    /**< Low battery. 1=low, 2=critical. */
 
-       /* Temporary state across callbacks */
        uint8_t buf[NMADMM_BUFSIZE];    /**< Buffer for read callback */
        int buflen;                     /**< Data len in buf */
 };
index 4525ba2ccb750707bfa9db8dfa31402d00a176cc..723da636b326aa9472225cc063ccf9904b510f92 100644 (file)
 #define FLAG_FILTER                (1 << 1)
 #define FLAG_DEMUX                 (1 << 0)
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
-       /* Fixed device settings */
        int max_channels;
        uint32_t max_samples;
        uint32_t max_samplerate;
        uint32_t protocol_version;
 
-       /* Acquisition settings */
        uint64_t cur_samplerate;
        uint32_t cur_samplerate_divider;
        uint64_t limit_samples;
@@ -86,7 +83,6 @@ struct dev_context {
        int num_stages;
        uint16_t flag_reg;
 
-       /* Operational states */
        unsigned int num_transfers;
        unsigned int num_samples;
        int num_bytes;
@@ -94,7 +90,6 @@ struct dev_context {
        int cnt_samples;
        int cnt_samples_rle;
 
-       /* Temporary variables */
        unsigned int rle_count;
        unsigned char sample[4];
        unsigned char tmp_sample[4];
index 1ac30aed0cf5a771571fb9f0da76dca064bc05f8..66983c9c55a8b682d39851617b697fa4062000f8 100644 (file)
@@ -306,7 +306,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 
        std_session_send_df_header(sdi);
 
-       /* Poll every 150ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 150,
                        pce_322a_receive_data, (void *)sdi);
index 094b3ab47793949d93c8c49fe51c84b5639a9fb3..338e37e3af454b7d676194eb685d88814ad64544 100644 (file)
@@ -68,17 +68,13 @@ enum {
        MEM_STATE_GET_MEMORY_BLOCK,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        uint64_t cur_mqflags;
        uint8_t cur_meas_range;
 
-       /* Acquisition settings */
        uint8_t cur_data_source;
        uint64_t limit_samples;
 
-       /* Operational state */
        uint64_t num_samples;
 
        /* Memory reading state */
@@ -88,7 +84,6 @@ struct dev_context {
        uint16_t memory_block_counter; /* Number of memory blocks retrieved so far. */
        uint8_t memory_block_cursor; /* Number of bytes retrieved in current memory block. */
 
-       /* Temporary state across callbacks. */
        uint8_t buffer[BUFFER_SIZE];
        int buffer_len;
        int buffer_skip; /* Number of bytes to skip in memory mode. */
index f16b659e1aabad0bdc66e605e9e22a249874f3c1..8dd474cf478bf3dc0c21ed3169aa1b8ccb964032 100644 (file)
@@ -91,31 +91,24 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
 
        devices = NULL;
 
-       /* Allocate memory for our private device context. */
        devc = g_malloc0(sizeof(struct dev_context));
 
-       /* Device-specific settings */
        devc->max_samplebytes = devc->max_samplerate = devc->protocol_version = 0;
 
-       /* Acquisition settings */
        devc->limit_samples = devc->capture_ratio = 0;
        devc->trigger_at = -1;
        devc->channel_mask = 0xffffffff;
        devc->flag_reg = 0;
 
-       /* Allocate memory for the incoming ftdi data. */
        devc->ftdi_buf = g_malloc0(FTDI_BUF_SIZE);
 
-       /* Allocate memory for the FTDI context (ftdic) and initialize it. */
        if (!(devc->ftdic = ftdi_new())) {
                sr_err("Failed to initialize libftdi.");
                goto err_free_ftdi_buf;;
        }
 
-       /* Try to open the FTDI device */
-       if (p_ols_open(devc) != SR_OK) {
+       if (p_ols_open(devc) != SR_OK)
                goto err_free_ftdic;
-       }
 
        /* The discovery procedure is like this: first send the Reset
         * command (0x00) 5 times, since the device could be anywhere
@@ -166,7 +159,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                goto err_close_ftdic;
        }
 
-       /* Close device. We'll reopen it again when we need it. */
        p_ols_close(devc);
 
        /* Parse the metadata. */
@@ -184,7 +176,7 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
 err_close_ftdic:
        p_ols_close(devc);
 err_free_ftdic:
-       ftdi_free(devc->ftdic); /* NOT free() or g_free()! */
+       ftdi_free(devc->ftdic);
 err_free_ftdi_buf:
        g_free(devc->ftdi_buf);
        g_free(devc);
index dbe79c6a5a385283e5912d1b12231ab40cc6565c..66430acb9544ad25f6e148e4d3b895d86ef48e2e 100644 (file)
@@ -73,7 +73,6 @@ SR_PRIV int p_ols_open(struct dev_context *devc)
 
        /* Note: Caller checks devc and devc->ftdic. */
 
-       /* Select interface B, otherwise communication will fail. */
        ret = ftdi_set_interface(devc->ftdic, INTERFACE_B);
        if (ret < 0) {
                sr_err("Failed to set FTDI interface B (%d): %s", ret,
@@ -81,7 +80,6 @@ SR_PRIV int p_ols_open(struct dev_context *devc)
                return SR_ERR;
        }
 
-       /* Check for the device and temporarily open it. */
        ret = ftdi_usb_open_desc(devc->ftdic, USB_VENDOR_ID, USB_DEVICE_ID,
                                 USB_IPRODUCT, NULL);
        if (ret < 0) {
@@ -92,14 +90,12 @@ SR_PRIV int p_ols_open(struct dev_context *devc)
                return SR_ERR;
        }
 
-       /* Purge RX/TX buffers in the FTDI chip. */
        if ((ret = ftdi_usb_purge_buffers(devc->ftdic)) < 0) {
                sr_err("Failed to purge FTDI RX/TX buffers (%d): %s.",
                       ret, ftdi_get_error_string(devc->ftdic));
                goto err_open_close_ftdic;
        }
 
-       /* Reset the FTDI bitmode. */
        ret = ftdi_set_bitmode(devc->ftdic, 0xff, BITMODE_RESET);
        if (ret < 0) {
                sr_err("Failed to reset the FTDI chip bitmode (%d): %s.",
@@ -107,7 +103,6 @@ SR_PRIV int p_ols_open(struct dev_context *devc)
                goto err_open_close_ftdic;
        }
 
-       /* Set the FTDI latency timer to 16. */
        ret = ftdi_set_latency_timer(devc->ftdic, 16);
        if (ret < 0) {
                sr_err("Failed to set FTDI latency timer (%d): %s.",
@@ -115,7 +110,6 @@ SR_PRIV int p_ols_open(struct dev_context *devc)
                goto err_open_close_ftdic;
        }
 
-       /* Set the FTDI read data chunk size to 64kB. */
        ret = ftdi_read_data_set_chunksize(devc->ftdic, 64 * 1024);
        if (ret < 0) {
                sr_err("Failed to set FTDI read data chunk size (%d): %s.",
index c2bd2ccb902ba5cb91f79bcbd8254b32b9dd78ba..c1aaac4b37139142bda3ad3bbba0b301f607a392 100644 (file)
 #define FLAG_FILTER                (1 << 1)
 #define FLAG_DEMUX                 (1 << 0)
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
-       /** FTDI device context (used by libftdi). */
        struct ftdi_context *ftdic;
        uint8_t *ftdi_buf;
 
-       /* Fixed device settings */
        int max_channels;
        uint32_t max_samplebytes;
        uint32_t max_samplerate;
        uint32_t protocol_version;
 
-       /* Acquisition settings */
        uint64_t cur_samplerate;
        uint32_t cur_samplerate_divider;
        uint32_t max_samples;
@@ -101,7 +97,6 @@ struct dev_context {
        int num_stages;
        uint16_t flag_reg;
 
-       /* Operational states */
        unsigned int num_transfers;
        unsigned int num_samples;
        int num_bytes;
@@ -109,7 +104,6 @@ struct dev_context {
        unsigned int cnt_samples;
        int cnt_samples_rle;
 
-       /* Temporary variables */
        unsigned int rle_count;
        unsigned char sample[4];
        unsigned char tmp_sample[4];
index e77797047eac5c160d21ff21e4d1e1b424f5b394..3b17299114bb244e16685662174b1b5a4e496353 100644 (file)
@@ -89,9 +89,7 @@ enum wait_events {
        WAIT_STOP,    /* Wait for scope stopping (only single shots) */
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Device model */
        const struct rigol_ds_model *model;
        enum data_format format;
 
@@ -127,8 +125,6 @@ struct dev_context {
        float trigger_level;
        char *coupling[MAX_ANALOG_CHANNELS];
 
-       /* Operational state */
-
        /* Number of frames received in total. */
        uint64_t num_frames;
        /* GSList entry for the current channel. */
index 7b66c814de32d46e9ffd24faab3d0fc2bb59a309..65e5c6d193bcaa43ce1d19923eb6261cc9f995be 100644 (file)
@@ -41,9 +41,7 @@ struct rs_device_model {
        double power_min;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        const struct rs_device_model *model_config;
 };
 
index ca80f258ef4956e79bf4e1207d56593d1c2833ba..9844454381463ae3bfe91b9b4e10e0286a9f5b28 100644 (file)
  */
 #define CONV_BUFFER_SIZE (2 * 8 * 16384 + CONV_BATCH_SIZE)
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Acquisition settings */
        unsigned int dig_channel_cnt;
        uint16_t dig_channel_mask;
        uint16_t dig_channel_masks[16];
        uint64_t dig_samplerate;
 
-       /* Operational state */
        uint32_t lfsr;
 
-       /* Temporary state across callbacks */
        unsigned int num_transfers;
        unsigned int submitted_transfers;
        struct libusb_transfer **transfers;
 
-       /* Conversion buffer */
        uint8_t *conv_buffer;
        unsigned int conv_size;
        unsigned int batch_index;
index 9eead9a9e772ffc80b728c12c058f1df36cbf758..003358169871a1f144833f5c4641966413cfc3e3 100644 (file)
@@ -41,7 +41,6 @@ enum fpga_variant {
        FPGA_VARIANT_MCUPRO    /* mcupro clone v4.6 with Actel FPGA */
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        /** Distinguishing between original Logic16 and clones */
        enum fpga_variant fpga_variant;
index 7448796298ddb082a777d123ed75b428e303bb0e..cad2a8747fd83cdbacbc3708928132e7e720ad3c 100644 (file)
@@ -439,7 +439,6 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
                return SR_ERR_ARG;
 
        if (cg)
-               /* Channel group specified. */
                select_channel(sdi, cg->channels->data);
 
        devc = sdi->priv;
@@ -523,7 +522,6 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
        devc = (sdi) ? sdi->priv : NULL;
 
        if (!cg) {
-               /* No channel group: global options. */
                switch (key) {
                case SR_CONF_SCAN_OPTIONS:
                case SR_CONF_DEVICE_OPTIONS:
@@ -555,7 +553,6 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
                        return SR_ERR_NA;
                }
        } else {
-               /* Channel group specified. */
                /*
                 * Per-channel-group options depending on a channel are actually
                 * done with the first channel. Channel groups in PPS can have
index 75be9efdf525c5d08a0eecdb0ea3d7d0e543e12d..f3c58dcaacb03d5c6b0f1be1a0359ee627ecce65 100644 (file)
@@ -130,7 +130,6 @@ static const struct scpi_command chroma_61604_cmd[] = {
 };
 
 /* Chroma 62000 series DC source */
-
 static const uint32_t chroma_62000_devopts[] = {
        SR_CONF_CONTINUOUS,
 };
@@ -307,7 +306,6 @@ static const struct scpi_command rigol_dp800_cmd[] = {
 };
 
 /* HP 663xx series */
-
 static const uint32_t hp_6630a_devopts[] = {
        SR_CONF_CONTINUOUS,
        SR_CONF_ENABLED | SR_CONF_SET,
@@ -572,6 +570,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                agilent_n5700a_cmd,
                .probe_channels = NULL,
        },
+
        /* Agilent N5767A */
        { "Agilent", "N5767A", 0,
                ARRAY_AND_SIZE(agilent_n5700a_devopts),
@@ -581,6 +580,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                agilent_n5700a_cmd,
                .probe_channels = NULL,
        },
+
        /* Chroma 61604 */
        { "Chroma", "61604", 0,
                ARRAY_AND_SIZE(chroma_61604_devopts),
@@ -590,6 +590,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                chroma_61604_cmd,
                .probe_channels = NULL,
        },
+
        /* Chroma 62000 series */
        { "Chroma", "620[0-9]{2}P-[0-9]{2,3}-[0-9]{1,3}", 0,
                ARRAY_AND_SIZE(chroma_62000_devopts),
@@ -599,6 +600,7 @@ SR_PRIV const struct scpi_pps pps_profiles[] = {
                chroma_62000_cmd,
                .probe_channels = chroma_62000p_probe_channels,
        },
+
        /* HP 6633A */
        { "HP", "6633A", 0,
                ARRAY_AND_SIZE(hp_6630a_devopts),
index 80593ceca8ad54ed205e7db214a9bcd56f4f64d2..d4425e725573cb0813267cc7f4be8ff02adea283 100644 (file)
@@ -136,17 +136,13 @@ enum acq_states {
        STATE_STOP,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        const struct scpi_pps *device;
 
-       /* Operational state */
        gboolean beeper_was_set;
        struct channel_spec *channels;
        struct channel_group_spec *channel_groups;
 
-       /* Temporary state across callbacks */
        struct sr_channel *cur_channel;
 };
 
index 687b35b9ccae873eae627f586cb8ae3b31393a57..7fb688655f674695c6085cdba561a1ddee9d6274 100644 (file)
@@ -168,7 +168,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        sr_sw_limits_acquisition_start(&devc->limits);
        std_session_send_df_header(sdi);
 
-       /* Poll every 50ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
                      receive_data, (void *)sdi);
index 9749345604589328e7b619b3e0f55b39cf145ca5..c80180ce0b976c089036b8d26d4b50d249b3ef53 100644 (file)
@@ -60,7 +60,6 @@ struct dmm_info {
 
 #define DMM_BUFSIZE 256
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
 
index 1ff70eeb8dce3eba523dc9793963f9bf0c6e14c8..71a358b98a05e122c089cabf4264c4948d4c9c40 100644 (file)
 #include <libsigrok-internal.h>
 #include "protocol.h"
 
-/* Supported device scan options.
- */
 static const uint32_t scanopts[] = {
        SR_CONF_CONN,
 };
 
-/* Driver capabilities.
- */
 static const uint32_t drvopts[] = {
        SR_CONF_LOGIC_ANALYZER,
 };
 
-/* Supported trigger match conditions.
- */
 static const int32_t trigger_matches[] = {
        SR_TRIGGER_ZERO,
        SR_TRIGGER_ONE,
@@ -47,22 +41,16 @@ static const int32_t trigger_matches[] = {
        SR_TRIGGER_FALLING,
 };
 
-/* Names assigned to available trigger sources.
- */
 static const char *const trigger_source_names[] = {
        [TRIGGER_CHANNELS] = "CH",
        [TRIGGER_EXT_TRG] = "TRG",
 };
 
-/* Names assigned to available edge slope choices.
- */
 static const char *const signal_edge_names[] = {
        [EDGE_POSITIVE] = "r",
        [EDGE_NEGATIVE] = "f",
 };
 
-/* Create a new sigrok device instance for the indicated LWLA model.
- */
 static struct sr_dev_inst *dev_inst_new(const struct model_info *model)
 {
        struct sr_dev_inst *sdi;
@@ -70,7 +58,6 @@ static struct sr_dev_inst *dev_inst_new(const struct model_info *model)
        int i;
        char name[8];
 
-       /* Initialize private device context. */
        devc = g_malloc0(sizeof(struct dev_context));
        devc->model = model;
        devc->active_fpga_config = FPGA_NOCONF;
@@ -78,16 +65,13 @@ static struct sr_dev_inst *dev_inst_new(const struct model_info *model)
        devc->samplerate = model->samplerates[0];
        devc->channel_mask = (UINT64_C(1) << model->num_channels) - 1;
 
-       /* Create sigrok device instance. */
        sdi = g_malloc0(sizeof(struct sr_dev_inst));
        sdi->status = SR_ST_INACTIVE;
        sdi->vendor = g_strdup(VENDOR_NAME);
        sdi->model = g_strdup(model->name);
        sdi->priv = devc;
 
-       /* Generate list of logic channels. */
        for (i = 0; i < model->num_channels; i++) {
-               /* The LWLA series simply number channels from CH1 to CHxx. */
                g_snprintf(name, sizeof(name), "CH%d", i + 1);
                sr_channel_new(sdi, i, SR_CHANNEL_LOGIC, TRUE, name);
        }
@@ -148,8 +132,6 @@ static struct sr_dev_inst *dev_inst_new_matching(GSList *conn_matches,
        return sdi;
 }
 
-/* Scan for SysClk LWLA devices and create a device instance for each one.
- */
 static GSList *scan(struct sr_dev_driver *di, GSList *options)
 {
        GSList *conn_devices, *devices, *node;
@@ -236,8 +218,6 @@ static int drain_usb(struct sr_usb_dev_inst *usb, unsigned int endpoint)
        return SR_OK;
 }
 
-/* Open and initialize device.
- */
 static int dev_open(struct sr_dev_inst *sdi)
 {
        struct drv_context *drvc;
@@ -300,7 +280,6 @@ static int dev_open(struct sr_dev_inst *sdi)
        return ret;
 }
 
-/* Shutdown and close device. */
 static int dev_close(struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -344,8 +323,6 @@ static int has_devopt(const struct model_info *model, uint32_t key)
        return FALSE;
 }
 
-/* Read device configuration setting.
- */
 static int config_get(uint32_t key, GVariant **data, const struct sr_dev_inst *sdi,
                      const struct sr_channel_group *cg)
 {
@@ -426,8 +403,6 @@ static int lookup_index(GVariant *value, const char *const *table, int len)
        return -1;
 }
 
-/* Write device configuration setting.
- */
 static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sdi,
                      const struct sr_channel_group *cg)
 {
@@ -501,8 +476,6 @@ static int config_set(uint32_t key, GVariant *data, const struct sr_dev_inst *sd
        return SR_OK;
 }
 
-/* Apply channel configuration change.
- */
 static int config_channel_set(const struct sr_dev_inst *sdi,
                              struct sr_channel *ch, unsigned int changes)
 {
@@ -532,8 +505,7 @@ static int config_channel_set(const struct sr_dev_inst *sdi,
        return SR_OK;
 }
 
-/* Derive trigger masks from the session's trigger configuration.
- */
+/* Derive trigger masks from the session's trigger configuration. */
 static int prepare_trigger_masks(const struct sr_dev_inst *sdi)
 {
        uint64_t trigger_mask, trigger_values, trigger_edge_mask;
@@ -598,8 +570,6 @@ static int prepare_trigger_masks(const struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
-/* Apply current device configuration to the hardware.
- */
 static int config_commit(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -688,8 +658,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        return lwla_start_acquisition(sdi);
 }
 
-/* Request that a running capture operation be stopped.
- */
 static int dev_acquisition_stop(struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -704,8 +672,6 @@ static int dev_acquisition_stop(struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
-/* SysClk LWLA driver descriptor.
- */
 static struct sr_dev_driver sysclk_lwla_driver_info = {
        .name = "sysclk-lwla",
        .longname = "SysClk LWLA series",
index 35e33f79ef8a8cf48069559cfb26925467f9abd5..bf5318d1e64d4c7c7cd21d0ea5a89f43d2121aa4 100644 (file)
@@ -69,8 +69,7 @@ struct sr_usb_dev_inst;
  */
 #define PACKET_SIZE            (5000 * 4 * 5)
 
-/** LWLA protocol command ID codes.
- */
+/** LWLA protocol command ID codes. */
 enum command_id {
        CMD_READ_REG    = 1,
        CMD_WRITE_REG   = 2,
@@ -89,22 +88,19 @@ enum status_flag {
        STATUS_MEM_AVAIL = 1 << 6,
 };
 
-/** LWLA1034 run-length encoding states.
- */
+/** LWLA1034 run-length encoding states. */
 enum rle_state {
        RLE_STATE_DATA,
        RLE_STATE_LEN
 };
 
-/** Register address/value pair.
- */
+/** Register address/value pair. */
 struct regval {
        unsigned int reg;
        uint32_t val;
 };
 
-/** LWLA sample acquisition and decompression state.
- */
+/** LWLA sample acquisition and decompression state. */
 struct acquisition_state {
        uint64_t samples_max;   /* maximum number of samples to process */
        uint64_t samples_done;  /* number of samples sent to the session bus */
index d16a35e71175a2c8f58f18a5bf8d180921d1a442..4b412fb59a5a287a5b692ada9f166231c3565c20 100644 (file)
 #include "lwla.h"
 #include "protocol.h"
 
-/* Number of logic channels.
- */
+/* Number of logic channels. */
 #define NUM_CHANNELS   16
 
-/* Unit size for the sigrok logic datafeed.
- */
+/* Unit size for the sigrok logic datafeed. */
 #define UNIT_SIZE      ((NUM_CHANNELS + 7) / 8)
 
-/* Size of the acquisition buffer in device memory units.
- */
+/* Size of the acquisition buffer in device memory units. */
 #define MEMORY_DEPTH   (256 * 1024)    /* 256k x 32 bit */
 
-/* Capture memory read start address.
- */
+/* Capture memory read start address. */
 #define READ_START_ADDR        2
 
-/* Number of device memory units (32 bit) to read at a time.
- */
+/* Number of device memory units (32 bit) to read at a time. */
 #define READ_CHUNK_LEN 250
 
-/** LWLA1016 register addresses.
- */
+/** LWLA1016 register addresses. */
 enum reg_addr {
        REG_CHAN_MASK   = 0x1000, /* bit mask of enabled channels */
 
@@ -64,15 +58,13 @@ enum reg_addr {
        REG_DIV_COUNT   = 0x10BC, /* write */
 };
 
-/** Flag bits for REG_MEM_CTRL.
- */
+/** Flag bits for REG_MEM_CTRL. */
 enum mem_ctrl_flag {
        MEM_CTRL_RESET  = 1 << 0,
        MEM_CTRL_WRITE  = 1 << 1,
 };
 
-/** Flag bits for REG_CAP_CTRL.
- */
+/** Flag bits for REG_CAP_CTRL. */
 enum cap_ctrl_flag {
        CAP_CTRL_FIFO32_FULL    = 1 << 0, /* "fifo32_ful" bit */
        CAP_CTRL_FIFO64_FULL    = 1 << 1, /* "fifo64_ful" bit */
@@ -83,22 +75,19 @@ enum cap_ctrl_flag {
        CAP_CTRL_CNTR_NOT_ENDR  = 1 << 6, /* "cntr_not_endr" bit */
 };
 
-/* Available FPGA configurations.
- */
+/* Available FPGA configurations. */
 enum fpga_config {
        FPGA_100 = 0,   /* 100 MS/s, no compression */
        FPGA_100_TS,    /* 100 MS/s, timing-state mode */
 };
 
-/* FPGA bitstream resource filenames.
- */
+/* FPGA bitstream resource filenames. */
 static const char bitstream_map[][32] = {
        [FPGA_100]      = "sysclk-lwla1016-100.rbf",
        [FPGA_100_TS]   = "sysclk-lwla1016-100-ts.rbf",
 };
 
-/* Demangle incoming sample data from the transfer buffer.
- */
+/* Demangle incoming sample data from the transfer buffer. */
 static void read_response(struct acquisition_state *acq)
 {
        uint32_t *in_p, *out_p;
@@ -136,8 +125,7 @@ static void read_response(struct acquisition_state *acq)
        acq->samples_done += run_samples;
 }
 
-/* Demangle and decompress incoming sample data from the transfer buffer.
- */
+/* Demangle and decompress incoming sample data from the transfer buffer. */
 static void read_response_rle(struct acquisition_state *acq)
 {
        uint32_t *in_p;
@@ -233,8 +221,7 @@ static int test_read_memory(const struct sr_dev_inst *sdi,
        return SR_OK;
 }
 
-/* Select and transfer FPGA bitstream for the current configuration.
- */
+/* Select and transfer FPGA bitstream for the current configuration. */
 static int apply_fpga_config(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -259,8 +246,7 @@ static int apply_fpga_config(const struct sr_dev_inst *sdi)
        return ret;
 }
 
-/* Perform initialization self test.
- */
+/* Perform initialization self test. */
 static int device_init_check(const struct sr_dev_inst *sdi)
 {
        static const struct regval mem_reset[] = {
@@ -443,8 +429,7 @@ static int handle_response(const struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
-/* Model descriptor for the LWLA1016.
- */
+/* Model descriptor for the LWLA1016. */
 SR_PRIV const struct model_info lwla1016_info = {
        .name = "LWLA1016",
        .num_channels = NUM_CHANNELS,
index a4bda3683016c9f4918ba8c0e08ad1ba1092c280..63ca46f58065474c86136f5923a43e66e0236c30 100644 (file)
 #include "lwla.h"
 #include "protocol.h"
 
-/* Number of logic channels.
- */
+/* Number of logic channels. */
 #define NUM_CHANNELS   34
 
-/* Bit mask covering all logic channels.
- */
+/* Bit mask covering all logic channels. */
 #define ALL_CHANNELS_MASK      ((UINT64_C(1) << NUM_CHANNELS) - 1)
 
-/* Unit size for the sigrok logic datafeed.
- */
+/* Unit size for the sigrok logic datafeed. */
 #define UNIT_SIZE      ((NUM_CHANNELS + 7) / 8)
 
-/* Size of the acquisition buffer in device memory units.
- */
+/* Size of the acquisition buffer in device memory units. */
 #define MEMORY_DEPTH   (256 * 1024)    /* 256k x 36 bit */
 
-/* Capture memory read start address.
- */
+/* Capture memory read start address. */
 #define READ_START_ADDR        4
 
 /* Number of device memory units (36 bit) to read at a time. Slices of 8
@@ -56,8 +51,7 @@
  */
 #define READ_CHUNK_LEN (28 * 8)
 
-/* Bit mask for the RLE repeat-count-follows flag.
- */
+/* Bit mask for the RLE repeat-count-follows flag. */
 #define RLE_FLAG_LEN_FOLLOWS   (UINT64_C(1) << 35)
 
 /* Start index and count for bulk long register reads.
@@ -67,8 +61,7 @@
 #define READ_LREGS_START       LREG_MEM_FILL
 #define READ_LREGS_COUNT       (LREG_STATUS + 1 - READ_LREGS_START)
 
-/** LWLA1034 register addresses.
- */
+/** LWLA1034 register addresses. */
 enum reg_addr {
        REG_MEM_CTRL    = 0x1074, /* capture buffer control */
        REG_MEM_FILL    = 0x1078, /* capture buffer fill level */
@@ -82,15 +75,13 @@ enum reg_addr {
        REG_LONG_HIGH   = 0x10BC, /* long register high word */
 };
 
-/** Flag bits for REG_MEM_CTRL.
- */
+/** Flag bits for REG_MEM_CTRL. */
 enum mem_ctrl_flag {
        MEM_CTRL_WRITE   = 1 << 0, /* "wr1rd0" bit */
        MEM_CTRL_CLR_IDX = 1 << 1, /* "clr_idx" bit */
 };
 
-/* LWLA1034 long register addresses.
- */
+/* LWLA1034 long register addresses. */
 enum long_reg_addr {
        LREG_CHAN_MASK  = 0,    /* channel enable mask */
        LREG_DIV_COUNT  = 1,    /* clock divider max count */
@@ -107,8 +98,7 @@ enum long_reg_addr {
        LREG_TEST_ID    = 100,  /* constant test ID */
 };
 
-/** Flag bits for LREG_CAP_CTRL.
- */
+/** Flag bits for LREG_CAP_CTRL. */
 enum cap_ctrl_flag {
        CAP_CTRL_TRG_EN       = 1 << 0, /* "trg_en" bit */
        CAP_CTRL_CLR_TIMEBASE = 1 << 2, /* "do_clr_timebase" bit */
@@ -117,8 +107,7 @@ enum cap_ctrl_flag {
        CAP_CTRL_CLR_COUNTER  = 1 << 6, /* "clr_cntr0" bit */
 };
 
-/* Available FPGA configurations.
- */
+/* Available FPGA configurations. */
 enum fpga_config {
        FPGA_OFF = 0,   /* FPGA shutdown config */
        FPGA_INT,       /* internal clock config */
@@ -126,8 +115,7 @@ enum fpga_config {
        FPGA_EXTNEG,    /* external clock, falling edge config */
 };
 
-/* FPGA bitstream resource filenames.
- */
+/* FPGA bitstream resource filenames. */
 static const char bitstream_map[][32] = {
        [FPGA_OFF]      = "sysclk-lwla1034-off.rbf",
        [FPGA_INT]      = "sysclk-lwla1034-int.rbf",
@@ -135,8 +123,7 @@ static const char bitstream_map[][32] = {
        [FPGA_EXTNEG]   = "sysclk-lwla1034-extneg.rbf",
 };
 
-/* Read 64-bit long register.
- */
+/* Read 64-bit long register. */
 static int read_long_reg(const struct sr_usb_dev_inst *usb,
                         uint32_t addr, uint64_t *value)
 {
@@ -164,8 +151,7 @@ static int read_long_reg(const struct sr_usb_dev_inst *usb,
        return SR_OK;
 }
 
-/* Queue access sequence for a long register write.
- */
+/* Queue access sequence for a long register write. */
 static void queue_long_regval(struct acquisition_state *acq,
                              uint32_t addr, uint64_t value)
 {
@@ -175,8 +161,7 @@ static void queue_long_regval(struct acquisition_state *acq,
        lwla_queue_regval(acq, REG_LONG_STROBE, 0);
 }
 
-/* Helper to fill in the long register bulk write command.
- */
+/* Helper to fill in the long register bulk write command. */
 static inline void bulk_long_set(struct acquisition_state *acq,
                                 unsigned int idx, uint64_t value)
 {
@@ -186,8 +171,7 @@ static inline void bulk_long_set(struct acquisition_state *acq,
        acq->xfer_buf_out[4 * idx + 6] = LWLA_WORD_3(value);
 }
 
-/* Helper for dissecting the response to a long register bulk read.
- */
+/* Helper for dissecting the response to a long register bulk read. */
 static inline uint64_t bulk_long_get(const struct acquisition_state *acq,
                                     unsigned int idx)
 {
@@ -317,8 +301,7 @@ static int detect_short_transfer_quirk(const struct sr_dev_inst *sdi)
        return SR_ERR;
 }
 
-/* Select and transfer FPGA bitstream for the current configuration.
- */
+/* Select and transfer FPGA bitstream for the current configuration. */
 static int apply_fpga_config(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -347,8 +330,7 @@ static int apply_fpga_config(const struct sr_dev_inst *sdi)
        return ret;
 }
 
-/* Perform initialization self test.
- */
+/* Perform initialization self test. */
 static int device_init_check(const struct sr_dev_inst *sdi)
 {
        uint64_t value;
@@ -369,8 +351,7 @@ static int device_init_check(const struct sr_dev_inst *sdi)
        return detect_short_transfer_quirk(sdi);
 }
 
-/* Set up the device in preparation for an acquisition session.
- */
+/* Set up the device in preparation for an acquisition session. */
 static int setup_acquisition(const struct sr_dev_inst *sdi)
 {
        static const struct regval capture_init[] = {
@@ -565,8 +546,7 @@ static int handle_response(const struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
-/** Model descriptor for the LWLA1034.
- */
+/** Model descriptor for the LWLA1034. */
 SR_PRIV const struct model_info lwla1034_info = {
        .name = "LWLA1034",
        .num_channels = NUM_CHANNELS,
index 85997249143213304062396e5bea67361ec1a8ae..45c0795210629e812f25981425a95b9666795447 100644 (file)
@@ -22,8 +22,7 @@
 #include "protocol.h"
 #include "lwla.h"
 
-/* Submit an already filled-in USB transfer.
- */
+/* Submit an already filled-in USB transfer. */
 static int submit_transfer(struct dev_context *devc,
                           struct libusb_transfer *xfer)
 {
@@ -40,8 +39,7 @@ static int submit_transfer(struct dev_context *devc,
        return SR_OK;
 }
 
-/* Set up transfer for the next register in a write sequence.
- */
+/* Set up transfer for the next register in a write sequence. */
 static void next_reg_write(struct acquisition_state *acq)
 {
        struct regval *regval;
@@ -56,8 +54,7 @@ static void next_reg_write(struct acquisition_state *acq)
        acq->xfer_out->length = 4 * sizeof(acq->xfer_buf_out[0]);
 }
 
-/* Set up transfer for the next register in a read sequence.
- */
+/* Set up transfer for the next register in a read sequence. */
 static void next_reg_read(struct acquisition_state *acq)
 {
        unsigned int addr;
@@ -70,8 +67,7 @@ static void next_reg_read(struct acquisition_state *acq)
        acq->xfer_out->length = 2 * sizeof(acq->xfer_buf_out[0]);
 }
 
-/* Decode the response to a register read request.
- */
+/* Decode the response to a register read request. */
 static int read_reg_response(struct acquisition_state *acq)
 {
        uint32_t value;
@@ -87,8 +83,7 @@ static int read_reg_response(struct acquisition_state *acq)
        return SR_OK;
 }
 
-/* Enter a new state and submit the corresponding request to the device.
- */
+/* Enter a new state and submit the corresponding request to the device. */
 static int submit_request(const struct sr_dev_inst *sdi,
                          enum protocol_state state)
 {
@@ -123,8 +118,7 @@ static int submit_request(const struct sr_dev_inst *sdi,
        return submit_transfer(devc, acq->xfer_out);
 }
 
-/* Evaluate and act on the response to a capture status request.
- */
+/* Evaluate and act on the response to a capture status request. */
 static void handle_status_response(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -160,8 +154,7 @@ static void handle_status_response(const struct sr_dev_inst *sdi)
        }
 }
 
-/* Evaluate and act on the response to a capture length request.
- */
+/* Evaluate and act on the response to a capture length request. */
 static void handle_length_response(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -191,8 +184,7 @@ static void handle_length_response(const struct sr_dev_inst *sdi)
        submit_request(sdi, STATE_READ_PREPARE);
 }
 
-/* Evaluate and act on the response to a capture memory read request.
- */
+/* Evaluate and act on the response to a capture memory read request. */
 static void handle_read_response(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -250,8 +242,7 @@ static void handle_read_response(const struct sr_dev_inst *sdi)
        submit_request(sdi, STATE_READ_FINISH);
 }
 
-/* Destroy and unset the acquisition state record.
- */
+/* Destroy and unset the acquisition state record. */
 static void clear_acquisition_state(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
@@ -269,8 +260,7 @@ static void clear_acquisition_state(const struct sr_dev_inst *sdi)
        }
 }
 
-/* USB I/O source callback.
- */
+/* USB I/O source callback. */
 static int transfer_event(int fd, int revents, void *cb_data)
 {
        const struct sr_dev_inst *sdi;
@@ -321,8 +311,7 @@ static int transfer_event(int fd, int revents, void *cb_data)
        return G_SOURCE_REMOVE;
 }
 
-/* USB output transfer completion callback.
- */
+/* USB output transfer completion callback. */
 static void LIBUSB_CALL transfer_out_completed(struct libusb_transfer *transfer)
 {
        const struct sr_dev_inst *sdi;
@@ -386,8 +375,7 @@ static void LIBUSB_CALL transfer_out_completed(struct libusb_transfer *transfer)
        }
 }
 
-/* USB input transfer completion callback.
- */
+/* USB input transfer completion callback. */
 static void LIBUSB_CALL transfer_in_completed(struct libusb_transfer *transfer)
 {
        const struct sr_dev_inst *sdi;
@@ -448,8 +436,7 @@ static void LIBUSB_CALL transfer_in_completed(struct libusb_transfer *transfer)
        }
 }
 
-/* Set up the acquisition state record.
- */
+/* Set up the acquisition state record. */
 static int init_acquisition_state(const struct sr_dev_inst *sdi)
 {
        struct dev_context *devc;
index ebfc704a97417b7737f8cd1c7fb11ac0fba93639..027f3f55606863b44b0330f4b13e4ffff04edae6 100644 (file)
 
 struct acquisition_state;
 
-/* USB vendor and product IDs.
- */
+/* USB vendor and product IDs. */
 enum {
        USB_VID_SYSCLK   = 0x2961,
        USB_PID_LWLA1016 = 0x6688,
        USB_PID_LWLA1034 = 0x6689,
 };
 
-/* USB device characteristics.
- */
+/* USB device characteristics. */
 enum {
        USB_CONFIG      = 1,
        USB_INTERFACE   = 0,
        USB_TIMEOUT_MS  = 1000,
 };
 
-/** USB device end points.
- */
+/** USB device end points. */
 enum usb_endpoint {
        EP_COMMAND = 2,
        EP_CONFIG  = 4,
        EP_REPLY   = 6 | LIBUSB_ENDPOINT_IN
 };
 
-/** LWLA1034 clock sources.
- */
+/** LWLA1034 clock sources. */
 enum clock_source {
        CLOCK_INTERNAL = 0,
        CLOCK_EXT_CLK,
 };
 
-/** LWLA1034 trigger sources.
- */
+/** LWLA1034 trigger sources. */
 enum trigger_source {
        TRIGGER_CHANNELS = 0,
        TRIGGER_EXT_TRG,
 };
 
-/** Edge choices for the LWLA1034 external clock and trigger inputs.
- */
+/** Edge choices for the LWLA1034 external clock and trigger inputs. */
 enum signal_edge {
        EDGE_POSITIVE = 0,
        EDGE_NEGATIVE,
@@ -96,8 +90,7 @@ enum {
        FPGA_NOCONF = -1,
 };
 
-/** Acquisition protocol states.
- */
+/** Acquisition protocol states. */
 enum protocol_state {
        /* idle states */
        STATE_IDLE = 0,
@@ -114,8 +107,6 @@ enum protocol_state {
        STATE_READ_REQUEST,
 };
 
-/** Private, per-device-instance driver context.
- */
 struct dev_context {
        uint64_t samplerate;    /* requested samplerate */
        uint64_t limit_msec;    /* requested capture duration in ms */
@@ -142,8 +133,7 @@ struct dev_context {
        enum signal_edge cfg_trigger_slope;     /* ext trigger slope setting */
 };
 
-/** LWLA model descriptor.
- */
+/** LWLA model descriptor. */
 struct model_info {
        char name[12];
        int num_channels;
index 759280986e7bea5394573bfaf8fd47b172295e60..f7e52aace77e639d389c7a5877b6a50be901919a 100644 (file)
@@ -153,7 +153,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 
        std_session_send_df_header(sdi);
 
-       /* Poll every 50ms, or whenever some data comes in. */
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
                        teleinfo_receive_data, (void *)sdi);
 
index f450547ec6b664b00d75a2ec6507eb2ab84b38fd..b617074fd74293b311181a6780f7d041d05756dd 100644 (file)
@@ -37,15 +37,9 @@ enum optarif {
 
 #define TELEINFO_BUF_SIZE 256
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Acquisition settings */
        struct sr_sw_limits sw_limits;
-
-       /* Operational state */
        enum optarif optarif; /**< The device mode (which measures are reported) */
-
-       /* Temporary state across callbacks */
        uint8_t buf[TELEINFO_BUF_SIZE];
        int buf_len;
 };
index b208ff724ccaad304bc0fcf5b58816325947fabd..9ad817396257fa6e8142f9d5702924973fb70fba 100644 (file)
@@ -111,7 +111,7 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
 
                usb_get_port_path(devlist[i], connection_id, sizeof(connection_id));
 
-               /* Hardcode the 435 for now.*/
+               /* Hardcode the 435 for now. */
                if (strcmp(product, "testo 435/635/735"))
                        continue;
 
index 654eb6977a7fabbb5d4fc591ae60f351f5dfa681..a6def0b5343330bcdc1ed7367e5c0c95738a3d3c 100644 (file)
@@ -51,19 +51,13 @@ struct testo_model {
        const uint8_t *request;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Model-specific information */
        const struct testo_model *model;
-
-       /* Acquisition settings */
        struct sr_sw_limits sw_limits;
 
-       /* Operational state */
        uint8_t channel_units[MAX_CHANNELS];
        int num_channels;
 
-       /* Temporary state across callbacks */
        struct libusb_transfer *out_transfer;
        uint8_t reply[MAX_REPLY_SIZE];
        int reply_size;
index be1ea82b5a074875ff72249007592461d2260ba3..788bc88439bad3dc38cf16bdf4d38d3c952ff3f1 100644 (file)
@@ -121,7 +121,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 
        sr_sw_limits_acquisition_start(&devc->limits);
 
-       /* Poll every 500ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 500,
                      tondaj_sl_814_receive_data, (void *)sdi);
index 8aa8ad9728aebcafea1cbbe8d0a91a4fde3336c7..b7cbd2180ef990aa063197d1c1ef95c466ad9430 100644 (file)
@@ -26,7 +26,6 @@
 
 #define LOG_PREFIX "tondaj-sl-814"
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
        int state;
index 5730633cac52e683a0593af505d5e05b8be41f83..61936c779006c3347cc528d6c89a6a880cbf3c61 100644 (file)
@@ -137,7 +137,7 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
 
        std_session_send_df_header(sdi);
 
-       sr_session_source_add(sdi->session, -1, 0, 10 /* poll_timeout */,
+       sr_session_source_add(sdi->session, -1, 0, 10,
                        uni_t_dmm_receive_data, (void *)sdi);
 
        return SR_OK;
index 4114aec4d831954ccc5d4f705112088a740a764d..111770838f077310559b8fbbbc765c995734d2c1 100644 (file)
@@ -103,7 +103,6 @@ static int hid_chip_init(struct sr_dev_inst *sdi, uint16_t baudrate)
 
        usb = sdi->conn;
 
-       /* Detach kernel drivers which grabbed this device (if any). */
        if (libusb_kernel_driver_active(usb->devhdl, 0) == 1) {
                ret = libusb_detach_kernel_driver(usb->devhdl, 0);
                if (ret < 0) {
@@ -113,7 +112,6 @@ static int hid_chip_init(struct sr_dev_inst *sdi, uint16_t baudrate)
                }
        }
 
-       /* Claim interface 0. */
        if ((ret = libusb_claim_interface(usb->devhdl, 0)) < 0) {
                sr_err("Failed to claim interface 0: %s.",
                       libusb_error_name(ret));
index 4158833f0ee72cfae6bd32de521e774610e664f0..4258d413d4a3bad72fee264d3f4adb56b2c5cc78 100644 (file)
@@ -45,7 +45,6 @@ struct dmm_info {
 
 #define DMM_BUFSIZE            256
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
 
index 969ccfa65d3a4212cd8759914899d1b422706cae..f499dcde2fd5e13d00178bbf446e1f419283f64b 100644 (file)
@@ -48,18 +48,14 @@ enum {
        CMD_GET_STORED = 7,
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /* Acquisition settings */
        uint64_t limit_samples;
        gboolean data_source;
 
-       /* Operational state */
        uint64_t num_samples;
        unsigned char buf[8];
        struct libusb_transfer *xfer;
 
-       /* Temporary state across callbacks */
        unsigned char packet[32];
        int packet_len;
 };
index d5d7e4a7d079b0386a13dd738d7f71fa52198f12..fd13421f8e954977261773743b9b0873fbaa088b 100644 (file)
@@ -105,8 +105,6 @@ static int dev_open(struct sr_dev_inst *sdi)
        if (ret != SR_OK)
                return ret;
 
-       /* The device reports as HID class, so the kernel would have
-        * claimed it. */
        if (libusb_kernel_driver_active(usb->devhdl, 0) == 1) {
                if ((ret = libusb_detach_kernel_driver(usb->devhdl, 0)) < 0) {
                        sr_err("Failed to detach kernel driver: %s.",
index 5c07d37320ad88bf5f5dea51e4ef1c59677e7f67..fe665e43f320c1d08a58265531fc3d579b7ee676 100644 (file)
@@ -28,7 +28,6 @@
 
 #define DMM_DATA_SIZE 14
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        struct sr_sw_limits limits;
 };
index 650174a09234ae7ee1a52a82fe33f822b8655e36..cc0cacbd3737201255f9e1b1e50cda6f8f607c29 100644 (file)
@@ -476,10 +476,6 @@ static int config_list(uint32_t key, GVariant **data, const struct sr_dev_inst *
        devc = (sdi) ? sdi->priv : NULL;
        model = (devc) ? devc->model_config : NULL;
 
-       /*
-        * If cg is NULL, only the SR_CONF_DEVICE_OPTIONS that are not
-        * specific to a channel group must be returned.
-        */
        if (!cg) {
                switch (key) {
                case SR_CONF_SCAN_OPTIONS:
@@ -611,7 +607,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        devc->current_channel = devc->enabled_channels;
        dlm_channel_data_request(sdi);
 
-       /* Call our callback when data comes in or after 5ms. */
        sr_scpi_source_add(sdi->session, scpi, G_IO_IN, 5,
                        dlm_data_receive, (void *)sdi);
 
index 6955baaeec23bf2e11f894d7247d21880c3d3231..ead078fa1507bbd9f1a1770ad5e70a41aa099fac 100644 (file)
  * along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-/**
- * @file
- *
- * <em>Yokogawa DL/DLM series</em> oscilloscope driver
- * @internal
- */
-
 #include <config.h>
 #include "scpi.h"
 #include "protocol.h"
index 80def3f9de550d752f54dc7371f0525ae9e13190..ac0f4f1bd7b4758f3073ff25ce71dc243898045b 100644 (file)
@@ -95,7 +95,6 @@ struct scope_state {
        uint32_t samples_per_frame;
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
        const void *model_config;
        void *model_state;
index 4ffcced83489a054c847790db46c66f8e5b0c924..e0124ed8ba5eb49422819303bdb17265bc57da9e 100644 (file)
@@ -203,12 +203,11 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                                prof = &zeroplus_models[j];
                        }
                }
-               /* Skip if the device was not found. */
+
                if (!prof)
                        continue;
                sr_info("Found ZEROPLUS %s.", prof->model_name);
 
-               /* Register the device with libsigrok. */
                sdi = g_malloc0(sizeof(struct sr_dev_inst));
                sdi->status = SR_ST_INACTIVE;
                sdi->vendor = g_strdup(VENDOR_NAME);
@@ -216,7 +215,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                sdi->serial_num = g_strdup(serial_num);
                sdi->connection_id = g_strdup(connection_id);
 
-               /* Allocate memory for our private driver context. */
                devc = g_malloc0(sizeof(struct dev_context));
                sdi->priv = devc;
                devc->prof = prof;
@@ -232,7 +230,6 @@ static GSList *scan(struct sr_dev_driver *di, GSList *options)
                devc->memory_size = MEMORY_SIZE_8K;
                // memset(devc->trigger_buffer, 0, NUM_TRIGGER_STAGES);
 
-               /* Fill in channellist according to this device's profile. */
                for (j = 0; j < devc->num_channels; j++)
                        sr_channel_new(sdi, j, SR_CHANNEL_LOGIC, TRUE,
                                        channel_names[j]);
@@ -627,7 +624,6 @@ static int dev_acquisition_start(const struct sr_dev_inst *sdi)
        return SR_OK;
 }
 
-/* TODO: This stops acquisition on ALL devices, ignoring dev_index. */
 static int dev_acquisition_stop(struct sr_dev_inst *sdi)
 {
        struct sr_usb_dev_inst *usb;
index cd83f9b1c37660e77a4ddcda9dd8acb8cc70bbb9..62aa02551f6fa59055a28de62f9e920f85a41612 100644 (file)
@@ -30,7 +30,6 @@
 
 #define LOG_PREFIX "zeroplus"
 
-/* Private, per-device-instance driver context. */
 struct dev_context {
        uint64_t cur_samplerate;
        uint64_t max_samplerate;
index 5378d74adf32a4460e2dd16482ff707d8fecbbf9..41f71abeef61b3018ab94c395383f98ad696acf3 100644 (file)
@@ -861,6 +861,9 @@ SR_API int sr_config_commit(const struct sr_dev_inst *sdi)
  * @param[in] cg The channel group on the device instance for which to list
  *            the values, or NULL. If this device instance doesn't
  *            have channel groups, this must not be != NULL.
+ *            If cg is NULL, this function will return the "common" device
+ *            instance options that are channel-group independent. Otherwise
+ *            it will return the channel-group specific options.
  * @param[in] key The configuration key (SR_CONF_*).
  * @param[in,out] data A pointer to a GVariant where the list will be stored.
  *                The caller is given ownership of the GVariant and must thus
index eeb724ec20cef49de7992374d84c0860cafa33f6..02e6dd023df0d1fbc74bba9f0a15ecb450fb788a 100644 (file)
@@ -387,15 +387,11 @@ static const char *const models[] = {
        "NONE", "PARALLEL", "SERIES", "AUTO",
 };
 
-/** Private, per-device-instance driver context. */
 struct dev_context {
-       /** The number of frames. */
        struct dev_limit_counter frame_count;
 
-       /** The time limit counter. */
        struct dev_time_counter time_count;
 
-       /** Data buffer. */
        struct dev_buffer *buf;
 
        /** The frequency of the test signal (index to frequencies[]). */
@@ -886,7 +882,6 @@ SR_PRIV int es51919_serial_acquisition_start(const struct sr_dev_inst *sdi)
 
        std_session_send_df_header(sdi);
 
-       /* Poll every 50ms, or whenever some data comes in. */
        serial = sdi->conn;
        serial_source_add(sdi->session, serial, G_IO_IN, 50,
                          receive_data, (void *)sdi);