similar to: [nut-commits] svn commit r1072 - in trunk: . drivers

I think having this logic buried within libhid/libusb (libusb:libusb_open(), line 179 to 206) is ultimately a mistake, albeit one that I am probably responsible for. Would it make sense to confine libhid to low-level operations, and leave the decision of trying to reopen vs. retrying to open to the high-level driver, in this case usbhid-ups? I envision that the code in usbhid-ups:reconnect_ups()

> ---------- Forwarded message ---------- > From: Arjen de Korte > To: nut-commits at lists.alioth.debian.org > Date: Mon, 27 Aug 2007 18:33:00 +0000 > Subject: svn commit r1076 - in trunk: . drivers > Author: adkorte-guest > Date: Mon Aug 27 18:33:00 2007 > New Revision: 1076 > > Log: > * drivers/libhid.[ch],libshut.[ch],libusb.[ch]: > - Cut the libshut and

> Author: aquette > Date: Thu Jun 21 07:43:46 2007 > New Revision: 971 > > Log: > fix communication lost status handling > > Modified: > trunk/ChangeLog > trunk/drivers/usbhid-ups.c > > Modified: trunk/ChangeLog > ============================================================================== > --- trunk/ChangeLog (original) > +++ trunk/ChangeLog

Arjen de Korte wrote: > > Author: adkorte-guest > Date: Fri Aug 10 20:23:27 2007 > New Revision: 1041 > > Log: > * drivers/usbhid.[ch]: > - Moved the struct & data for ups.status processing to usbhid.c, since this is only used in there. Subdrivers report the status through text messages, they don't set the status bits directly. > - Rewrote (flattened) the

Charles Lepple wrote: > Author: clepple-guest > Date: Fri Aug 15 23:49:44 2008 > New Revision: 1520 > > Log: > drivers/tripplite_usb.c: Increase send retries to 10 (as needed by some > protocol 2001 units) Would you mind if I made an attempt to re-engineer the reconnection and USB error handling code of this driver? Best regards, Arjen

> Author: agordeev-guest > Date: Wed Oct 3 00:09:21 2007 > New Revision: 1136 > > Log: > - drivers/megatec_usb.c: added ability to do subdriver-specific > initialization, and moved some shared code to agiler init function > since krauler doesn't need it. [...] I'm not very thrilled about this patch: 1) Down at the subdriver level, one shouldn't use

Gentlemen, I have determined that on my UPS (APC Back-UPS ES 650 FW:818.w1.D USB FW:w1), the variable UPS.PowerSummary.RemainingCapacityLimit is writeable. I am setting this variable to start shutdown a little earlier. Appended is an unconditional patch for apc-hid.h. If, as indicated by the original code, there are APC UPS devices that don't allow setting of this variable, it becomes

Trying to build nut 2.6.0 on OpenBSD 4.8, I get a compiler error. Here's my configure: ./configure --with-user=_ups --with-group=_ups \ --sysconfdir=/etc/ups --with-usb=no ... and the result: -e Configuration summary: ====================== build serial drivers: yes build USB drivers: no build SNMP drivers: no build neon based XML driver: no build Powerman PDU client driver:

On Mar 15, 2010, at 8:40 AM, Charles Lepple wrote: > Author: clepple-guest > Date: Mon Mar 15 12:40:49 2010 > New Revision: 2402 > URL: http://boxster.ghz.cc/projects/nut/changeset/2402 > > Log: > Generate ChangeLog with svn2cl. > > This commit removes the manually-generated ChangeLog, and adds the > authors file > necessary to map SVN usernames to full names.

> Author: adkorte-guest > Date: Mon Feb 25 20:40:41 2008 > New Revision: 1334 > > Log: > Allocate twice the maximum number of reports we expect (MAX_REPORTS), to > allow for bogus/duplicate reports. Retrospectively, this is gross hack at best and probably only good to fix the problem temporarily. I think I have a better solution available (in a similar way as r1210, that was

Does this require a new entry in scripts/hotplug/* and scripts/udev/*? -- Peter Alexander Gordeev wrote: > > Author: agordeev-guest > Date: Sat May 26 12:04:26 2007 > New Revision: 924 > > Log: > data/driver.list: added SVEN Power Smart RM 2000 to the list. > > Modified: > trunk/ChangeLog > trunk/data/driver.list > > Modified: trunk/ChangeLog >

On Nov 26, 2007 6:38 PM, Arjen de Korte <adkorte-guest at alioth.debian.org> wrote: > Author: adkorte-guest > Date: Mon Nov 26 18:38:18 2007 > New Revision: 1164 > > Log: > Allow longer commands in the 'phoenix' subdriver. > > Modified: > trunk/ChangeLog > trunk/drivers/megatec_usb.c FYI, the shutdown still doesn't work. -- Carlos Rodrigues

Arjen, Wow, that was quick... Thanks! (It was taking me a while to sort out all of the different CFLAGS settings; hence, the tracker item.) I would take issue with the following assertion in Makefile.am: "In any case, CFLAGS are only -I options, so there is no harm." since this is what got us into trouble in the first place... but I guess that was there before you edited it as

2007/3/8, Charles Lepple <clepple-guest@alioth.debian.org>: > Author: clepple-guest > Date: Thu Mar 8 04:15:51 2007 > New Revision: 848 > > Modified: > branches/tl_usb_fbsd/drivers/Makefile.am > branches/tl_usb_fbsd/drivers/libhid-uhid.c > Log: > Work-in-progress... maybe I need to look at how the SHUT subdriver works. > ... simply use the same interface

On 6/6/06, Carlos Rodrigues wrote: > Author: carlosefr-guest > Date: Tue Jun 6 21:16:30 2006 > New Revision: 430 > > Added: > branches/Testing/docs/megatec.txt > branches/Testing/drivers/megatec.c > branches/Testing/drivers/megatec.h > branches/Testing/man/megatec.8 Just a note on adding drivers to the Testing branch: if you use 'svn copy' instead

Arnaud, this commit refers to files called libshut.c and libshut.h, yet no such file has been added to SVN. Perhaps take this opportunity to run "svn status" and check for other files that might be missing from SVN. -- Peter > ---------- Forwarded message ---------- > From: Arnaud Quette > To: nut-commits@lists.alioth.debian.org > Date: Fri, 16 Jun 2006 13:40:59 +0000

> +22 = Gamatronic UPSs with alarm interface > + [CP=RTS] [OL=CTS] [LB=\-DCD] [SD=DTR] > + > +22 = CyberPower SL seriess > + [CP=RTS] [OL=CTS] [LB=\-CAR] [SD=DTR] > + > + This part of the patch is no good. While TIOCM_CAR is equivalent to TIOCM_CD (at least in some implementations) CAR certainly isn't handled the same as DCD by genericups. It won't be

Just a little tiny detail : ============================================================================== > --- trunk/scripts/hotplug/libhid.usermap (original) > +++ trunk/scripts/hotplug/libhid.usermap Fri Aug 25 19:18:19 2006 > @@ -21,5 +21,7 @@ > libhidups 0x0003 0x09ae 0x1003 0x0000 0x0000 0x00 0x00 0x00 0x00

Peter, looking back on rev 543, it looks like my 'make depend' ended up removing 'rhino.h', but also removed the rule for rhino.o as well. Did I miss a step somewhere? On 10/9/06, Peter Selinger <selinger-guest@costa.debian.org> wrote: > - updated dependencies for rhino -- - Charles Lepple

Charles Lepple wrote: > > Author: clepple-guest > Date: Mon Dec 4 03:00:57 2006 > New Revision: 609 > > Modified: > trunk/ChangeLog > trunk/configure.in > Log: > Undoing changeset [608]. It fails with this error: > > configure: error: conditional "HAVE_LIBUSB" was never defined. > Usually this means the macro was only invoked