Hi,

There is a regression in the UART decoder on head compared to the
release 0.3.1. I was looking for a bug tracker on SourceForge, but I
could not find it, so I posted the bug report here.

This is the command line I used to test for correctness:
$ cd sigrok-dumps/uart/hello_world
$ sigrok-cli -i 8n1/hello_world_8n1_9600.sr -a uart:baudrate=9600

I used git bisect (a first for me) to find the responsible commit, but
I did not look into the code for a fix, at least not yet:

04c63eb3e2f7f033a1b534cc3dc840129842375d is the first bad commit
commit 04c63eb3e2f7f033a1b534cc3dc840129842375d
Author: Uwe Hermann <[email protected]>
Date:   Wed Jul 11 22:19:31 2012 +0200
    srd: Performance improvements for various PDs.
    Ignore/skip identical samples in most (low-level) PDs, as we're usually
    (but not necessarily always) only interested in pin changes.
    This yields a significant performance improvement for the PDs.
    The mechanism was already used in the 'i2s', 'jtag', and 'lpc' PDs, but not
    yet in all supported low-level decoders. The following PDs now also use
    this mechanism: 'dcf77', 'i2c', 'spi', 'uart', and 'usb_signalling'.
    Thanks Lars-Peter Clausen <[email protected]> for bringing this to our
    attention.

Regards,
Iztok Jeras

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
sigrok-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to