search for: utab

...l drivers. DESCRIPTION OF CHANGES: - In newhidups.h, add an extra line to the subdriver_s struct; void (*extendvartable)(void); so it becomes; struct subdriver_s { char *name; int (*claim)(HIDDevice *hd); usage_tables_t *utab; hid_info_t *hid2nut; int (*shutdown)(int ondelay, int offdelay); char *(*format_model)(HIDDevice *hd); char *(*format_mfr)(HIDDevice *hd); char *(*format_serial)(HIDDevice *hd); void (*extendvartable)(v...

...desirable results and errors. --- drivers/libhid.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/drivers/libhid.c b/drivers/libhid.c index f6ec644..1cec74a 100644 --- a/drivers/libhid.c +++ b/drivers/libhid.c @@ -301,6 +301,13 @@ void HIDDumpTree(hid_dev_handle_t udev, usage_tables_t *utab) continue; } #else + /* Skip non-feature reports, primarily for OpenUPS - but + * as HIDGetItemData() only cares about feature items, + * do it for all. + */ + if (pData->Type != ITEM_FEATURE) + continue; + if ((vendorID == 0x0463) || (vendorID == 0x047c)) { if ((pData-&...

...> ============================================================================== > --- trunk/drivers/libhid.c (original) > +++ trunk/drivers/libhid.c Mon Jan 8 10:52:18 2007 > @@ -561,8 +561,9 @@ > > /* Get data value */ > if (HIDGetItemValue(udev, path, &value, utab) > 0) > - upsdebugx(1, "Path: %s, Type: %s, Value: %f", path, type, value); > - > + upsdebugx(1, "Path: %s, Type: %s, ReportID: 0x%02x, Offset: %i, Size: %i, Value: %f", > + path, type, pData->ReportID, pData->Offset, pData->Size, value); &g...

Hi, I've been checking out NUT with an OpenUPS board over the last couple of weekends, and have noticed that it doesn't seem to report sensible values. This lead me to investigate usbhid-ups and delve into various issues. As mentioned in a github issue, one of the problems is with the report descriptor - dumping this from the usbhid-ups debug output and picking through it reveals that

.../* Apparently, we are reconnecting, so > + * NUT-to-HID translation is still good */ > + if (item->hiddata != NULL) > + break; > + > /* Get pointer to the corresponding HIDData_t item */ > item->hiddata = HIDGetItemData(udev, item->hidpath, subdriver->utab); > - > if (item->hiddata == NULL) > continue; > > - /* Avoid redundancy when multiple defines (RO/RW) > - * ups.status and ups.alarm is not set during > - HU_WALKMODE_INIT, so these don't need to be > - caught here. */ > - if (dstate_ge...