salut la French Connection ;)<br><br><div class="gmail_quote">2011/5/17 trantore <span dir="ltr"><<a href="mailto:quesoyjamon@aol.fr">quesoyjamon@aol.fr</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div><div></div><div class="h5">Aurélien Grenotton <agrenott <at> <a href="http://gmail.com" target="_blank">gmail.com</a>> writes:<br>
<br>
><br>
> Hello,<br>
><br>
> I've recently bought two cheap UPS<br>
(<a href="http://www.ldlc.com/fiche/PB00109242.html" target="_blank">http://www.ldlc.com/fiche/PB00109242.html</a>).<br>
> They're not supported natively by NUT (even though the monitoring<br>
> software is UPSilon from megatec).<br>
> After some investigation, they almost use the krauler-type<br>
> communication already supported in the blazer_usb driver. However, the<br>
> UPS won't reply if the language ID is not 0x4095...<br>
> Therefore, I cloned the krauler_command, adding the language ID and<br>
> unicode to ASCII conversion (taken from libusb). I also enhanced the<br>
> error!retry mechanism, as the UPS fails to reply quite often.<br>
><br>
> Please see the attached patch file.<br>
><br>
> Any comment would me more than welcome.<br>
><br>
> Rgds,<br>
> Aurélien<br>
><br>
<br>
</div></div>Hi Aurélien!<br>
<br>
I'll try your patch... I'm very interested in a UPSilon replacement!<br>
<br>
Please, see my posts at:<br>
<a href="http://comments.gmane.org/gmane.comp.monitoring.nut.devel/5012" target="_blank">http://comments.gmane.org/gmane.comp.monitoring.nut.devel/5012</a><br>
<br>
<br>
I suggest two changes:<br>
<br>
1) Rename the dialet from "ldlc" to "upsilon", as SEVERAL no-name UPS uses this<br>
(horrible) software.<br></blockquote><div> <br>valid<br><br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
2) Add the USB ID "0001:0000" (vendorid=0001 productid=0000) to map directly<br>
"upsilon" in "blazer_usb"<br></blockquote><div> <br>no, this pair is already declared, and another reason for the below.<br><br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Finally, can someone accept the patch and can add it to the main branch?<br></blockquote></div><br>no, sorry. but you'll get it fixed for 2.6.1 next week.<br><br>though I'm not familiar with this driver, there is a smarter and mandatory way to solve this:<br>
since this VID:PID couple is already bound to krauler, the way to go is to use addvar()/getvar() and declare a new driver variable ("upsilon" for example).<br>then krauler_command() can switch using this between usb_get_string() and usb_get_string_simple()...<br>
<br>@Arjen: your take on this?<br><br>cheers,<br>Arnaud<br>-- <br>Linux / Unix Expert R&D - Eaton - <a href="http://powerquality.eaton.com" target="_blank">http://powerquality.eaton.com</a><br>Network UPS Tools (NUT) Project Leader - <a href="http://www.networkupstools.org/" target="_blank">http://www.networkupstools.org/</a><br>
Debian Developer - <a href="http://www.debian.org" target="_blank">http://www.debian.org</a><br>Free Software Developer - <a href="http://arnaud.quette.free.fr/" target="_blank">http://arnaud.quette.free.fr/</a><br><br>