On Tue, May 14, 2013 at 11:19:40PM -0400, Scott Talbert wrote:
> One thing I don't quite like - I hard coded the report length to 64.
> We used to read this from the descriptor, but hidapi does not
> provide a way to get this.  Not sure what to do about that.

We should ask on their devel list. It's lame we can't get to the descriptor
info.

> +            hid_info.ver = cur_dev->release_number;
> +            h_dev = hid_open(cur_dev->vendor_id, cur_dev->product_id, NULL);

One thing I noticed is that we should use hid_force_open()

> +    int err = hid_write(h_dev, data, USB_PACKET_LENGTH);

Eh? There's not even an hid_write in their docs. hid_interrupt_write is the
closest I see.

> +    int err = hid_read_timeout(h_dev, data, USB_PACKET_LENGTH, timeout);

Same here.

-- 
Phil Dibowitz                             p...@ipom.com
Open Source software and tech docs        Insanity Palace of Metallica
http://www.phildev.net/                   http://www.ipom.com/

"Be who you are and say what you feel, because those who mind don't matter
 and those who matter don't mind."
 - Dr. Seuss

Attachment: signature.asc
Description: Digital signature

------------------------------------------------------------------------------
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d
_______________________________________________
concordance-devel mailing list
concordance-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/concordance-devel

Reply via email to