Re: [sigrok-devel] Wiegand decoder

2016-05-11 Thread Uwe Hermann
Hi, On Tue, May 10, 2016 at 06:57:13PM +1000, Sean Burford wrote: > Thanks. I think I've addressed all of your points. Great, thanks a lot! All 3 patches are merged. Added one more minor change, the bitwidth_ms option values are now integers instead of strings. Cheers, Uwe. -- http://hermann

Re: [sigrok-devel] Wiegand decoder

2016-05-10 Thread Sean Burford
Hi, Thanks. I think I've addressed all of your points. The only style point I really disagree with is using an instance variable for the samplenum, but we're not multithreaded so that should be fine. I'm not exactly sure how you would like the dump directory structure to look so I gave it a goo

Re: [sigrok-devel] Wiegand decoder

2016-05-09 Thread Uwe Hermann
Hi, On Wed, Apr 27, 2016 at 10:51:37AM +1000, Sean Burford wrote: > I've put together a basic Wiegand decoder that extracts bit streams from > wiegand signals. It stops at the binary level and doesn't extract facility > codes/identity numbers etc. This is on the list of possible future > decoder

[sigrok-devel] Wiegand decoder

2016-04-26 Thread Sean Burford
Hi, I've put together a basic Wiegand decoder that extracts bit streams from wiegand signals. It stops at the binary level and doesn't extract facility codes/identity numbers etc. This is on the list of possible future decoders at https://sigrok.org/wiki/Protocol_decoders#Possible_candidates_for