Import data from Scubapro Aladin Sport Matrix

[Berthold Stoeger]

On Mittwoch, 1. November 2017 09:42:04 CET Jef Driesen wrote:
> On 31-10-17 23:35, Berthold Stoeger wrote:
> > On Dienstag, 31. Oktober 2017 11:29:00 CET Jef Driesen wrote:
> >> I believe you have the model number wrong. I think it should be 0x17,
> > 
> >> and your test seems to confirm that:
> > Well, I put a printf directly after "rc = scubapro_g2_transfer (device,
> > cmd_model, sizeof (cmd_model), model, sizeof (model));" and guess what?
> > I got both, 0x17 *and* 0xa5 (the latter much more commonly). There seems
> > to be some race. Will investigate further - tomorrow.
> > 
> > But when manually sending 0x10 manually with my test program, I
> > consistently get a 0x17, which therefore seems to be the correct model.
> > Sigh.
> 
> That's indeed strange. Probably something is still wrong. Can you share the
> logs? Maybe we can spot something?

Nothing wrong - turns out the culprit was my own "inject dive data" code (a5 
is the start of a new dive). That's what you get for trying to be clever. :(

The correct model number is 0x17 - I will send you logs later.

@Linus: patch attached. Sorry about that.

Berthold
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-model-number-of-the-Scubapro-Aladin-Sport-Matrix.patch
Type: text/x-patch
Size: 1589 bytes
Desc: not available
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20171101/d4ad1c8d/attachment.bin>