i2p/haiku-beta6
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

USB: Add default max packet size for SuperSpeed bulk endpoints.

Browse Source 
We already had one for High-Speed endpoints, but SuperSpeed endpoints
also have a fixed size, similar to Control endpoints.

May help with some devices failing to configure.
This commit is contained in:
Augustin Cavalier
2022-06-29 10:36:36 -04:00
parent 5b5da451b3
commit 00096a9d1f
1 changed files with 14 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/add-ons/kernel/bus_managers/usb/Pipe.cpp
+14 -5
View File
@@ -186,9 +186,18 @@ BulkPipe::InitCommon(int8 deviceAddress, uint8 endpointAddress,
usb_speed speed, pipeDirection direction, size_t maxPacketSize, usb_speed speed, pipeDirection direction, size_t maxPacketSize,
uint8 interval, int8 hubAddress, uint8 hubPort) uint8 interval, int8 hubAddress, uint8 hubPort)
{ {
// some devices have bogus descriptors // See comments in ControlPipe::InitCommon.
if (speed == USB_SPEED_HIGHSPEED && maxPacketSize != 512) switch (speed) {
maxPacketSize = 512; case USB_SPEED_HIGHSPEED:
maxPacketSize = 512;
break;
case USB_SPEED_SUPERSPEED:
maxPacketSize = 1024;
break;
default:
break;
}
Pipe::InitCommon(deviceAddress, endpointAddress, speed, direction, Pipe::InitCommon(deviceAddress, endpointAddress, speed, direction,
maxPacketSize, interval, hubAddress, hubPort); maxPacketSize, interval, hubAddress, hubPort);
@@ -355,8 +364,8 @@ ControlPipe::InitCommon(int8 deviceAddress, uint8 endpointAddress,
uint8 interval, int8 hubAddress, uint8 hubPort) uint8 interval, int8 hubAddress, uint8 hubPort)
{ {
// The USB 2.0 spec section 5.5.3 gives fixed max packet sizes for the // The USB 2.0 spec section 5.5.3 gives fixed max packet sizes for the
// different speeds. The USB 3.1 specs defines the max packet size to a // different speeds. The USB 3.1 specs defines some fixed max packet sizes,
// fixed 512 for control endpoints in 9.6.6. Some devices ignore these // including for control endpoints in 9.6.6. Some devices ignore these
// values and use bogus ones, so we restrict them here. // values and use bogus ones, so we restrict them here.
switch (speed) { switch (speed) {
case USB_SPEED_LOWSPEED: case USB_SPEED_LOWSPEED: