Dive data import error with Hollis DG03

[Jef Driesen]

On 2014-03-24 10:46, Cristian Ionescu-Idbohrn wrote:
> On Mon, 24 Mar 2014, Rick Walsh wrote:
>> On 24 March 2014 10:48, Hamish Moffatt <hamish at cloud.net.au> wrote:
>> 
>> > The low_latency flag is set by default, which forces it to 1, always.
>> >
>> > setserial /dev/ttyUSB0 ^low_latency
>> >
>> > to disable it. Then you can modify the latency_timer entry.
>> >
>> 
>> When I ran the setserial command, the latency_timer value was
>> automatically set to 16 - I didn't have to change it manually.
>> Subsurface successfully downloaded my last 64 dives. Unfortunately
>> the setting isn't persistent - when I remove and replace the cable,
>> I have to run setserial again.
> 
> You may be able to make that "persistent" by creating an udev-rule.
> Start with querying udev about the device:
> 
> 	# udevadm info --query=all --name /dev/ttyUSB0
> 
> to find out what you're able to match.
> You may find some inspiration among the already installed rules.
> Check out:
> 
> 	# grep --color tty /lib/udev/rules.d/*
> 
> Use the RUN key to instruct udev to automatically execute `setserial'
> for you:
> 
> 	ACTION=="add" <more matching keys here> RUN+="setserial /dev/%p 
> ^low_latency"

Or make it persistent by disable the low_latency flag when opening the 
serial port in libdivecomputer. See the attached patch for a prototype 
implementation. On Linux, it will clear the low_latency flag, without 
setting the latency_timer value, because that requires root permissions. 
But based on the info we gathered so far, that should already do the 
trick. On Mac OS X, it seems possible to set the latency timer directly, 
although I have no idea whether that actually works or not.

Can anyone give this a try?

Jef
-------------- next part --------------
A non-text attachment was scrubbed...
Name: latency.patch
Type: text/x-diff
Size: 2706 bytes
Desc: not available
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140324/ad7dbbd6/attachment.patch>