Hi,

On Sun, May 03, 2015 at 12:45:57AM +0200, Daniel Gröber wrote:
> I'm trying to add some basic support for the newer DSL models so
> here's a patch adding support for loading the FX2 and FPGA fimware for
> the DSLogic Pro and DSCope. 

Look good, merged! Sorry for the delay.

One small fix though, I think this was a copy-paste error?


>       if (devc->dslogic) {
> +             if (strcmp(devc->profile->model, "DSLogic") == 0) {
> +                     fpga_firmware = DSLOGIC_FPGA_FIRMWARE;
> +             } else if(strcmp(devc->profile->model, "DSLogic Pro") == 0) {
> +                     fpga_firmware = DSLOGIC_PRO_FPGA_FIRMWARE;
> +             } else if(strcmp(devc->profile->model, "DSCope") == 0) {
> +                     fpga_firmware = DSLOGIC_PRO_FPGA_FIRMWARE;

Changed this to DSCOPE_FPGA_FIRMWARE, but please confirm whether that's
correct.

Unrelated, but the lsusb seems to return "DSLogic" for both DSLogic and
DSLogic Pro, right? If you have one of those new devices, could you
please post a full "lsusb -v" as root before and after firmware upload?

You can post those here, or put them in the wiki directly (ask on IRC
if you don't have a wiki account already). If you own the devices, some
self-made photos and the usual info for the device wiki pages would be
nice as well. We have a page for the DSLogic already, but none for
DSCope or DSLogic Pro, yet.


Thanks, Uwe.
-- 
http://hermann-uwe.de | http://randomprojects.org | http://sigrok.org

------------------------------------------------------------------------------
Monitor 25 network devices or servers for free with OpManager!
OpManager is web-based network management software that monitors 
network devices and physical & virtual servers, alerts via email & sms 
for fault. Monitor 25 devices for free with no restriction. Download now
http://ad.doubleclick.net/ddm/clk/292181274;119417398;o
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to