2008/10/4 Arjen de Korte <[EMAIL PROTECTED]>: > Citeren Arnaud Quette <[EMAIL PROTECTED]>: >> >> you missed to add an entry in drivers/tripplite-hid.c >> ->tripplite_claim() for that ;-) > > And beware that you need to put it in the correct place as well. Some Tripp > Lite HID devices (incorrectly) report battery voltage in decivolts so we > need to apply a correction factor here. So there are basically two groups in > this function. One that reports battery voltage in decivolts (productid > 1003, 2005 and 2007) and one that reports battery voltage in volts > (productid 3012, 4002, 4003). Before committing this change, we need to see > the output of 'upsc' to know what it reports.
In the attached patch I've added it to the first group, as I'd expect it to behave just like 1003 (the avr550u model). I've also attached the output of upsc. > >> and an entry in the hotplug script for the sake of completion. > > Don't forget the udev and hal scripts either. We probably need to come up > with some automation here, so that we have to put this in only one place and > have these scripts generated automagically. That would be much easier; in the attached patch I added the ups info to hal, hotplug, udev, and drivers/tripplite-hid.c (btw, why isn't that info stored in tripplite-hid.h?). > > Best regards, Arjen > -- > Please keep list traffic on the list > Cheers, -- Atomo64 - Raphael Please avoid sending me Word, PowerPoint or Excel attachments. See http://www.gnu.org/philosophy/no-word-attachments.html
avr750u.2.patch
Description: Binary data
upsc.stdout
Description: Binary data
