Hi,

On Wed, Feb 24, 2016 at 09:58:35PM +0100, Andreas Zschunke wrote:
> I added support of the Hantek 4032l logic analyzer to the libsigrok 
> library. The source can be pulled or used from:
>      https://github.com/zschunky/libsigrok.git

Thanks a lot for working on this!


> This is the 1st version of the implementation. I will continue to clean 
> it up and advance it. For now only one stage of triggers is supported 
> with only one channel being rising/falling/edge. All other channels can 
> still be used as trigger with 0 or 1. The threshold voltage is currently 
> fixed to 1.5V. Sample size per channel ranges from 2kb to 64Mb in 512 
> byte steps (given size will be adjusted if it does not fit into 512 byte 
> steps).
> 
> Feel free to try it out. Feedback and code review is welcome (it was my 
> first time I had to deal with sigrok code and libusb).

Just a few basic points to make future reviews easier, could you please
fix the following things:

 - Please use the "new-driver" script from our sigrok-util repo to generate
   the initial driver framework (api.c and protocol.[ch]). Keep that
   generated patch as the first commit, and add your actual
   implementation on top of that as a second commit. The libsigrok API
   calls go into api.c, the other stuff (protocol implementation) into
   protocol.c usually.

 - Please use the same coding style as in the rest of the sigrok
   code-base, i.e. one tab for indentation, foo_bar() function names
   instead of camel-case or similar, #defines and enums are UPPER_CASE etc.
   We're basically using the Linux kernel conventions mostly, see HACKING.

 - Once you're done with a first version of the driver please drop all
   the obsolete (commented out) debug code and such so that we can
   commit a first clean version to the upstream repo. The driver doesn't
   have to fully support all features yet for that, but the code that
   gets in should be reasonably clean.

Other than that the code looks pretty promising, please let us know when
the next iteration is ready for review/testing.


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

------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to