[PATCH 0/7] usb/isp1760: Cleanup and bugfixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

All this is in preparation of a larger patch meant to remove the code paths containing interrupt context BUG() calls in normal situations (like out of chip mem, more than 32 active endpoints, ...) and which would also allow more efficient use of the double-port payload chip mem on fast systems.

 isp1760-hcd.c | 1349 ++++++++++++++++++++++++++--------------------------------
 isp1760-hcd.h |   55 --
 2 files changed, 625 insertions(+), 779 deletions(-)

-- 
Arvid Brodin
Enea Services Stockholm AB

--
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


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux