[Nut-upsdev] Driver fix for Ippon BCP800 (06da/0003), blazer_usb.c

[Charles Lepple]

On Mon, Aug 24, 2009 at 9:52 AM, Arjen de Korte <nut+devel at de-korte.org> wrote:
> Citeren Danya Alexeyevsky <me.dendik at gmail.com>:
>
>> Thanks to some older discussions on this list I found it quite easy to
>> eventually track down problem with using USB cable with some more of
>> Megatec/Q1 UPSes. The trick (which is done in megatec_usb, but not in
>> blazer_usb) is to read in the junk that somehow gets into read buffers
>> before sending any commands. (I inserted debug statements so it's easy
>> to see that the data read is really junk: in my case it's most often a
>> middle piece of previous query and sometimes is appended with some
>> fragments of /proc/self/environment or something).
>
> Thanks for this report. I have slightly modified your patch, but it should
> work out the same.
>
>> I will appreciate if you apply the patch to fix it (although probably
>> it's better to have some such flush function defined in usb wrappers
>> for every needing driver to use):
>
> The fact that we need to flush the IO buffers, really means that the USB to
> serial implementation is broken, so I'd prefer to do this explicitly in the
> driver.

Arjen,

what is the status of this patch?

I am not too familiar with the inner workings of the megatec/blazer
UPSes, but does it make sense to add this as a configuration file
option?

For reference, here is the corresponding Debian bug:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=539747

-- 
- Charles Lepple