On Fri, 2019-08-09 at 15:37 +0200, Daniel Franke wrote:
>
> I want to contribute to the sigrok project.

Great, that's always appreciated.

> I have imported the sigrokdecoder in Github under
>
> https://github.com/DerDan/Libsigrokdecode
>
> i have made changes to the uart decoder:
>
> *     measure the idle time between to uart frames
> *     number the data frames (shown in startbit)
>
> both are optional via setup dialog.

The presence of the other branch made me think:  Would it perhaps
be better to not modify the UART decoder (and change its output
in the process), but instead stack something on top of UART which
can count frames and measure time between frames?  That makes the
use automatically optional, and won't touch the UART decoder
(won't complicate the already long configuration dialog).

Ideally the frame counter could get re-used for other protocols,
too.  Think SPI transfers or I2C transactions, which you may find
yourself as often wanting to count or measure.  If the first stab
at it still isn't generic enough, and exclusively works with
UART, that's fine as well, yet I'd feel the stacked decoder still
could be preferred.

> what are the next steps?

You did see the documentation in the source code, right?  And for
more responsive interaction beyond the slow ML I also recommend
joining the developers on IRC.  Just don't leave after a few
minutes yet, humans have the tendency to take longer than that to
respond in meaningful ways. :)


virtually yours
Gerhard Sittig
--
     If you don't understand or are scared by any of the above
             ask your parents or an adult to help you.


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

Reply via email to