Hello Stefan,

thanks for your quick reply.

On Tue, Dec 18, 2018 at 06:38:19PM +0100, Stefan Brüns wrote:
> On Dienstag, 18. Dezember 2018 17:32:49 CET Alexander Dahl wrote:
> > The scope itself is currently not supported by sigrok, but some older
> > model is: https://sigrok.org/wiki/GW_Instek_GDS-800_series
> > 
> > The GDS-1000 Series seems to use the same serial protocol over USB,
> > but I can not just give sigrok the path to the device /dev/ttyACM0 and
> > it just works, that's not the case. :-/
> 
> The driver checks the response of the standard SCPI command "*IDN?". The 
> device response is split into a manufacturer and device/model part, and then 
> compared with expected values.
> 
> See src/hardware/gwinstek-gds-800/api.c, lines 51ff in libsigrok.
> 
> Currently, it expects the manufacturer to be "GW" and the model name to start 
> with "GDS-8". You have to extend this part to support both device series.

Yes it does, here's what I get when calling sigrok-cli, so at least
sigrok sends that *idn? command:

% sigrok-cli -l 5 --driver gwinstek-gds-800:conn=/dev/ttyACM0 --show
sr: [00:00.000001] log: libsigrok loglevel set to 5.
sr: [00:00.000030] backend: libsigrok 0.6.0-git-28877994/4:0:0 (rt: 
0.6.0-git-28877994/4:0:0).
sr: [00:00.000075] backend: Libs: glib 2.50.3 (rt: 2.50.3/5003:3), libzip 
1.1.2, libserialport 0.1.1/1:0:1 (rt: 0.1.1/1:0:1), libusb-1.0 1.0.21.11156 API 
0x01000105, libftdi 1.3.
sr: [00:00.000080] backend: Host: x86_64-pc-linux-gnu, little-endian.
sr: [00:00.000087] backend: SCPI backends: TCP, RPC, serial, USBTMC.
sr: [00:00.000102] backend: Firmware search paths:
sr: [00:00.000116] backend:  - /home/adahl/.local/share/sigrok-firmware
sr: [00:00.000119] backend:  - /usr/local/share/sigrok-firmware
sr: [00:00.000122] backend:  - /usr/share/sigrok-firmware
sr: [00:00.000125] backend:  - /usr/share/sigrok-firmware
sr: [00:00.000129] backend:  - /usr/local/share/sigrok-firmware
sr: [00:00.000143] backend: Sanity-checking all drivers.
sr: [00:00.000149] backend: Sanity-checking all input modules.
sr: [00:00.000154] backend: Sanity-checking all output modules.
sr: [00:00.000158] backend: Sanity-checking all transform modules.
srd: libsigrokdecode loglevel set to 5.
Driver functions:
    Oscilloscope
sr: [00:00.002698] hwdriver: sr_config_list(): key 2147418112 (NULL) sdi (nil) 
cg NULL -> [uint32 20000, 20001]
Scan options:
    conn
    serialcomm
sr: [00:00.002734] hwdriver: sr_config_list(): key 2147418112 (NULL) sdi (nil) 
cg NULL -> [uint32 20000, 20001]
sr: [00:00.002744] scpi: Opening serial device /dev/ttyACM0.
sr: [00:00.002752] serial: Opening serial port '/dev/ttyACM0' (flags 1).
sr: [00:00.008035] serial: Flushing serial port /dev/ttyACM0.
sr: [00:00.008063] serial: Wrote 6/6 bytes.
sr: [00:00.008068] scpi_serial: Successfully sent SCPI command: '*IDN?'.
sr: [00:00.048255] serial: Read 28/2048 bytes.
sr: [00:00.048281] scpi_serial: Received terminator
sr: [00:00.048289] scpi: Got response: 'GW,GDS-1102A,EK142755,V1.10', length 27.
sr: [00:00.048319] serial: Closing serial port /dev/ttyACM0.
sr: [00:00.068746] hwdriver: Scan found 0 devices (gwinstek-gds-800).
No devices found.

> Also, it might be necessary to extend the driver to support 

There's a different programming manual for the GDS1000A: 

https://gwinstek.com/en-global/products/downloadSeriesDownNew/11980/1030

The command set seems to be different, so it's maybe not that simple?

> > I could hardcopy a measurement to SD card however, which gives me one
> > CSV file per channel with analog values. According to libsigrok
> > master, analog input from CSV is still an open TODO: "Add support for
> > analog input data? (optional)".
> 
> Does it support exporting the data as binary? If yes, libsigrok can read 
> analog data in raw binary format.

I don't think so. :-/

> Adding analog support to the CSV importer would be very very welcome, 
> although 
> this is likely a quite large and orthogonal task.

Yeah I got some hints on that in IRC, but I guess that's something
which could be done by more than the person with a certain device,
those CVS files could easily be shared. Maybe I find some spare time
in the next days.

Greets
Alex

-- 
/"\ ASCII RIBBON | »With the first link, the chain is forged. The first
\ / CAMPAIGN     | speech censured, the first thought forbidden, the
 X  AGAINST      | first freedom denied, chains us all irrevocably.«
e \ HTML MAIL    | (Jean-Luc Picard, quoting Judge Aaron Satie)

Attachment: signature.asc
Description: PGP signature

_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to