On Thu, Oct 19, 2006 at 10:32:39AM -0600, Grant Grundler wrote:
>  /*
>   * Extract/implement a data field from/to a little endian report (bit array).
> + *
> + * Code sort-of follows HID spec:
> + *     http://www.usb.org/developers/devclass_docs/HID1_11.pdf
> + *
> + * While the USB HID spec allows unlimited length bit fields in "report
> + * descriptors",

To be sligtly pedantic, extract() isn't actually used on report
descriptors, but rather report fields. The length of the field is set in
the descriptor but the actual data to be extracted lives in the report.
So perhaps, "...bit fields in reports..." would be more accurate.

> most devices never use more than 16 bits.

I disagree with that comment a bit. UPSes routinely have 24 bit wide
fields as well. I'd really like to avoid promoting the idea that all HID
devices are simple little keyboards and mice and anything else is some
weird exception, especially because these exceptions are perfectly
standards compliant. I'd compromise on a change from "most" to "many" if
you feel strongly about keeping the sentence.

> + * One model of UPS is claimed to report "LINEV" as a 32-bit field.

I'd prefer to remove the word "claimed" and say instead, "One model of APC
UPS reports "LINEV" as a 32-bit field, for example."

With the above wording tweaks:
Acked-By: Adam Kropelin <[EMAIL PROTECTED]>

--Adam


-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to