Hello. On 23-02-2011 18:16, Arvid Brodin wrote:
This patch series is meant to clean up the code (removing more of the legacy from the original and quite frankly horrible Philips drivers), and also contain some small bug fixes.
* Patch 1: Moves to native-endian ptds. This helps users with platform bus connected isp176xs with endianness problems and fixes a bug with non-32-bit io transfers on big endian, straight-connected hardware.
* Patch 2: Removes the redundant hw_hext list pointer from struct isp1760_qtd, removes some unused #defines, removes redundant "urb" member from struct inter_packet_info.
* Patch 3: Cleans up payload address handling and encapsulates within struct isp1760_qtd.
* Patch 4: Removes redundant "data_buffer" member from struct inter_packet_info.
* Patch 5: Consolidates printouts to use dev_XXX functions instead of an assortment of printks and driver specific macros. Removes some unused code snippets and struct members. Removes some unused function parameters and #defines. Remove the "queue_entry" variable name which has different but related meanings in different places and use "slot" only.
* Patch 6: Replace the period calculation for INT packets with something readable. Probably fixes a bug with quick insertion/removal of several USB devices simultaneously (hub control).
* Patch 7: Remove "toggle" member from struct isp1760_qtd, and store toggle status in struct isp1760_qh only.
Why these described changes weren't reflected in the patch subjects? Naming all 7 patches the same way is a not a good idea.
WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
