On Wed, Nov 02, 2016 at 23:53 +0100, Uwe Hermann wrote:
> 
> On Thu, Oct 20, 2016 at 11:58:41AM +0200, Gerhard Sittig wrote:
> > +   DMM(
> > +           "peaktech-3330", fs9721,
> > +           "Peaktech", "3330", "2400/8n1/dtr=1", 2400,
> > +           FS9721_PACKET_SIZE, 0, 0, NULL,
> > +           sr_fs9721_packet_valid, sr_fs9721_parse,
> > +           sr_fs9721_01_10_temp_f_c
> 
> I assume you checked that sr_fs9721_01_10_temp_f_c is the correct
> function for this device? There are a few bits that are
> device-dependent here, even if the FS9721 chip is the same.

The patch has been sittig here for several months.  I remember
having checked hexdump(1) output from UART communication (the
chip's packet format) against the source code and the helper
variants for how the temperature gets "encoded".  Then I picked
that 01_10 routine.

Did a test with HEAD from master and a temperature probe.  The
sigrok-cli(1) output matches the meter's display in both C and F
modes.  So the entry/configuration should be good.


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.

------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to