[janderwald] 51443: [USBEHCI_NEW] - Fix build - Implement retrieving configuration descriptor (USBHI_GetUsbDescriptors) - Implement function to build a setup packet from urb (taken from mjmartin us...
24 Apr
2011
24 Apr
'11
9:26 a.m.
Author: janderwald
Date: Sun Apr 24 09:26:22 2011
New Revision: 51443
URL: http://svn.reactos.org/svn/reactos?rev=51443&view=rev
Log:
[USBEHCI_NEW]
- Fix build
- Implement retrieving configuration descriptor (USBHI_GetUsbDescriptors)
- Implement function to build a setup packet from urb (taken from mjmartin usbehci
driver)
Modified:
branches/usb-bringup/drivers/usb/usbehci_new/hub_controller.cpp
branches/usb-bringup/drivers/usb/usbehci_new/interfaces.h
branches/usb-bringup/drivers/usb/usbehci_new/usb_device.cpp
branches/usb-bringup/drivers/usb/usbehci_new/usb_request.cpp
Modified: branches/usb-bringup/drivers/usb/usbehci_new/hub_controller.cpp
URL:
http://svn.reactos.org/svn/reactos/branches/usb-bringup/drivers/usb/usbehci…
==============================================================================
--- branches/usb-bringup/drivers/usb/usbehci_new/hub_controller.cpp [iso-8859-1]
(original)
+++ branches/usb-bringup/drivers/usb/usbehci_new/hub_controller.cpp [iso-8859-1] Sun Apr
24 09:26:22 2011
@@ -1050,7 +1050,7 @@
//
// FIXME: retrieve values
//
- UsbHubDescriptor->bNumberOfPorts = PortCount;
+ UsbHubDescriptor->bNumberOfPorts = (UCHAR)PortCount;
UsbHubDescriptor->wHubCharacteristics = 0x0012;
UsbHubDescriptor->bPowerOnToPowerGood = 0x01;
UsbHubDescriptor->bHubControlCurrent = 0x00;
@@ -1842,8 +1842,6 @@
{
PUSBDEVICE UsbDevice;
CHubController * Controller;
- NTSTATUS Status;
- PURB Urb;
DPRINT1("USBHI_GetUsbDescriptors\n");
@@ -1851,7 +1849,10 @@
// sanity check
//
PC_ASSERT(DeviceDescriptorBuffer);
+ PC_ASSERT(DeviceDescriptorBufferLength);
PC_ASSERT(*DeviceDescriptorBufferLength >= sizeof(USB_DEVICE_DESCRIPTOR));
+ PC_ASSERT(ConfigDescriptorBufferLength);
+ PC_ASSERT(*ConfigDescriptorBufferLength >= sizeof(USB_CONFIGURATION_DESCRIPTOR));
//
// first get controller
@@ -1890,54 +1891,14 @@
*DeviceDescriptorBufferLength = sizeof(USB_DEVICE_DESCRIPTOR);
//
- // allocate urb
- //
- Urb = (PURB)ExAllocatePoolWithTag(NonPagedPool, sizeof(URB), TAG_USBEHCI);
- if (!Urb)
- {
- //
- // no memory
- //
- return STATUS_INSUFFICIENT_RESOURCES;
- }
-
- //
- // zero request
- //
- RtlZeroMemory(Urb, sizeof(URB));
-
- //
- // initialize request
- //
- Urb->UrbHeader.Function = URB_FUNCTION_GET_DESCRIPTOR_FROM_DEVICE;
- Urb->UrbHeader.Length = sizeof(_URB_CONTROL_DESCRIPTOR_REQUEST);
- Urb->UrbControlDescriptorRequest.DescriptorType =
USB_CONFIGURATION_DESCRIPTOR_TYPE;
- Urb->UrbControlDescriptorRequest.TransferBuffer = ConfigDescriptorBuffer;
- Urb->UrbControlDescriptorRequest.TransferBufferLength =
*ConfigDescriptorBufferLength;
-
- //
- // submit urb
- //
- //Status = UsbDevice->SubmitUrb(Urb);
- UNIMPLEMENTED
-
- if (NT_SUCCESS(Status))
- {
- //
- // TransferBufferLength holds the number of bytes transferred
- //
- *ConfigDescriptorBufferLength =
Urb->UrbControlDescriptorRequest.TransferBufferLength;
- }
-
- //
- // free urb
- //
- ExFreePoolWithTag(Urb, TAG_USBEHCI);
+ // get configuration descriptor
+ //
+
UsbDevice->GetConfigurationDescriptors((PUSB_CONFIGURATION_DESCRIPTOR)ConfigDescriptorBuffer,
*ConfigDescriptorBufferLength, ConfigDescriptorBufferLength);
//
// complete the request
//
- return Status;
+ return STATUS_SUCCESS;
}
NTSTATUS
@@ -2195,7 +2156,11 @@
// set length returned
//
*LengthReturned = ControllerInfo->ActualLength;
- return STATUS_NOT_IMPLEMENTED;
+
+ //
+ // done
+ //
+ return STATUS_SUCCESS;
}
NTSTATUS
Modified: branches/usb-bringup/drivers/usb/usbehci_new/interfaces.h
URL:
http://svn.reactos.org/svn/reactos/branches/usb-bringup/drivers/usb/usbehci…
==============================================================================
--- branches/usb-bringup/drivers/usb/usbehci_new/interfaces.h [iso-8859-1] (original)
+++ branches/usb-bringup/drivers/usb/usbehci_new/interfaces.h [iso-8859-1] Sun Apr 24
09:26:22 2011
@@ -748,6 +748,15 @@
virtual NTSTATUS SubmitIrp(PIRP Urb) = 0;
+//-----------------------------------------------------------------------------------------
+//
+// GetConfigurationDescriptors
+//
+// Description: returns one or more configuration descriptors
+
+ virtual VOID GetConfigurationDescriptors(IN PUSB_CONFIGURATION_DESCRIPTOR
ConfigDescriptorBuffer,
+ IN ULONG BufferLength,
+ OUT PULONG OutBufferLength) = 0;
};
typedef IUSBDevice *PUSBDEVICE;
Modified: branches/usb-bringup/drivers/usb/usbehci_new/usb_device.cpp
URL:
http://svn.reactos.org/svn/reactos/branches/usb-bringup/drivers/usb/usbehci…
==============================================================================
--- branches/usb-bringup/drivers/usb/usbehci_new/usb_device.cpp [iso-8859-1] (original)
+++ branches/usb-bringup/drivers/usb/usbehci_new/usb_device.cpp [iso-8859-1] Sun Apr 24
09:26:22 2011
@@ -65,6 +65,8 @@
virtual void GetDeviceDescriptor(PUSB_DEVICE_DESCRIPTOR DeviceDescriptor);
virtual UCHAR GetConfigurationValue();
virtual NTSTATUS SubmitIrp(PIRP Irp);
+ virtual VOID GetConfigurationDescriptors(IN PUSB_CONFIGURATION_DESCRIPTOR
ConfigDescriptorBuffer, IN ULONG BufferLength, OUT PULONG OutBufferLength);
+
// local function
virtual NTSTATUS CommitIrp(PIRP Irp);
@@ -113,7 +115,6 @@
IN ULONG PortStatus)
{
NTSTATUS Status;
- USB_DEFAULT_PIPE_SETUP_PACKET CtrlSetup;
//
// initialize members
@@ -835,6 +836,36 @@
//
return Status;
}
+//----------------------------------------------------------------------------------------
+VOID
+CUSBDevice::GetConfigurationDescriptors(
+ IN PUSB_CONFIGURATION_DESCRIPTOR ConfigDescriptorBuffer,
+ IN ULONG BufferLength,
+ OUT PULONG OutBufferLength)
+{
+ //
+ // sanity check
+ //
+ PC_ASSERT(BufferLength >= sizeof(USB_CONFIGURATION_DESCRIPTOR));
+ PC_ASSERT(ConfigDescriptorBuffer);
+ PC_ASSERT(OutBufferLength);
+
+ //
+ // FIXME: support multiple configurations
+ //
+ PC_ASSERT(m_DeviceDescriptor.bNumConfigurations == 1);
+
+ //
+ // copy first configuration descriptor
+ //
+ RtlCopyMemory(ConfigDescriptorBuffer,
&m_ConfigurationDescriptors[0].ConfigurationDescriptor,
sizeof(USB_CONFIGURATION_DESCRIPTOR));
+
+ //
+ // store length
+ //
+ *OutBufferLength = sizeof(USB_CONFIGURATION_DESCRIPTOR);
+}
+
//----------------------------------------------------------------------------------------
NTSTATUS
Modified: branches/usb-bringup/drivers/usb/usbehci_new/usb_request.cpp
URL:
http://svn.reactos.org/svn/reactos/branches/usb-bringup/drivers/usb/usbehci…
==============================================================================
--- branches/usb-bringup/drivers/usb/usbehci_new/usb_request.cpp [iso-8859-1] (original)
+++ branches/usb-bringup/drivers/usb/usbehci_new/usb_request.cpp [iso-8859-1] Sun Apr 24
09:26:22 2011
@@ -56,6 +56,7 @@
NTSTATUS CreateQueueHead(PQUEUE_HEAD *OutQueueHead);
ULONG GetDeviceAddress();
NTSTATUS BuildSetupPacket();
+ NTSTATUS BuildSetupPacketFromURB();
// constructor / destructor
CUSBRequest(IUnknown *OuterUnknown){}
@@ -83,7 +84,6 @@
// transfer buffer MDL
//
PMDL m_TransferBufferMDL;
-
//
// caller provided setup packet
@@ -181,7 +181,6 @@
{
PIO_STACK_LOCATION IoStack;
PURB Urb;
- NTSTATUS Status;
//
// sanity checks
@@ -409,6 +408,10 @@
DPRINT1("USB_ENDPOINT_TYPE_ISOCHRONOUS not implemented\n");
Status = STATUS_NOT_IMPLEMENTED;
break;
+ default:
+ PC_ASSERT(FALSE);
+ Status = STATUS_NOT_IMPLEMENTED;
+ break;
}
if (NT_SUCCESS(Status))
@@ -838,11 +841,6 @@
PHYSICAL_ADDRESS PhysicalAddress;
//
- // FIXME: generate setup packet from urb request
- //
- PC_ASSERT(m_SetupPacket);
-
- //
// allocate common buffer setup packet
//
Status = m_DmaManager->Allocate(sizeof(USB_DEFAULT_PIPE_SETUP_PACKET),
(PVOID*)&m_DescriptorPacket, &PhysicalAddress);
@@ -862,10 +860,167 @@
RtlCopyMemory(m_DescriptorPacket, m_SetupPacket,
sizeof(USB_DEFAULT_PIPE_SETUP_PACKET));
m_DescriptorSetupPacket = PhysicalAddress;
}
+ else
+ {
+ //
+ // build setup packet from urb
+ //
+ Status = BuildSetupPacketFromURB();
+ }
//
// done
//
+ return Status;
+}
+
+
+NTSTATUS
+CUSBRequest::BuildSetupPacketFromURB()
+{
+ PIO_STACK_LOCATION IoStack;
+ PURB Urb;
+ NTSTATUS Status = STATUS_NOT_IMPLEMENTED;
+
+ //
+ // sanity checks
+ //
+ PC_ASSERT(m_Irp);
+ PC_ASSERT(m_DescriptorPacket);
+
+ //
+ // get stack location
+ //
+ IoStack = IoGetCurrentIrpStackLocation(m_Irp);
+
+ //
+ // get urb
+ //
+ Urb = (PURB)IoStack->Parameters.Others.Argument1;
+
+ //
+ // zero descriptor packet
+ //
+ RtlZeroMemory(m_DescriptorPacket, sizeof(USB_DEFAULT_PIPE_SETUP_PACKET));
+
+
+ switch (Urb->UrbHeader.Function)
+ {
+ /* CLEAR FEATURE */
+ case URB_FUNCTION_CLEAR_FEATURE_TO_DEVICE:
+ case URB_FUNCTION_CLEAR_FEATURE_TO_INTERFACE:
+ case URB_FUNCTION_CLEAR_FEATURE_TO_ENDPOINT:
+ UNIMPLEMENTED
+ break;
+
+ /* GET CONFIG */
+ case URB_FUNCTION_GET_CONFIGURATION:
+ m_DescriptorPacket->bRequest = USB_REQUEST_GET_CONFIGURATION;
+ m_DescriptorPacket->bmRequestType.B = 0x80;
+ m_DescriptorPacket->wLength = 1;
+ break;
+
+ /* GET DESCRIPTOR */
+ case URB_FUNCTION_GET_DESCRIPTOR_FROM_DEVICE:
+ m_DescriptorPacket->bRequest = USB_REQUEST_GET_DESCRIPTOR;
+ m_DescriptorPacket->wValue.LowByte =
Urb->UrbControlDescriptorRequest.Index;
+ m_DescriptorPacket->wValue.HiByte =
Urb->UrbControlDescriptorRequest.DescriptorType;
+ m_DescriptorPacket->wIndex.W =
Urb->UrbControlDescriptorRequest.LanguageId;
+ m_DescriptorPacket->wLength =
Urb->UrbControlDescriptorRequest.TransferBufferLength;
+ m_DescriptorPacket->bmRequestType.B = 0x80;
+ break;
+
+ /* GET INTERFACE */
+ case URB_FUNCTION_GET_INTERFACE:
+ m_DescriptorPacket->bRequest = USB_REQUEST_GET_CONFIGURATION;
+ m_DescriptorPacket->wIndex.W = Urb->UrbControlGetStatusRequest.Index;
+ m_DescriptorPacket->bmRequestType.B = 0x80;
+ m_DescriptorPacket->wLength = 1;
+ break;
+
+ /* GET STATUS */
+ case URB_FUNCTION_GET_STATUS_FROM_DEVICE:
+ m_DescriptorPacket->bRequest = USB_REQUEST_GET_STATUS;
+ ASSERT(Urb->UrbControlGetStatusRequest.Index == 0);
+ m_DescriptorPacket->wIndex.W = Urb->UrbControlGetStatusRequest.Index;
+ m_DescriptorPacket->bmRequestType.B = 0x80;
+ m_DescriptorPacket->wLength = 2;
+ break;
+
+ case URB_FUNCTION_GET_STATUS_FROM_INTERFACE:
+ m_DescriptorPacket->bRequest = USB_REQUEST_GET_STATUS;
+ ASSERT(Urb->UrbControlGetStatusRequest.Index != 0);
+ m_DescriptorPacket->wIndex.W = Urb->UrbControlGetStatusRequest.Index;
+ m_DescriptorPacket->bmRequestType.B = 0x81;
+ m_DescriptorPacket->wLength = 2;
+ break;
+
+ case URB_FUNCTION_GET_STATUS_FROM_ENDPOINT:
+ m_DescriptorPacket->bRequest = USB_REQUEST_GET_STATUS;
+ ASSERT(Urb->UrbControlGetStatusRequest.Index != 0);
+ m_DescriptorPacket->wIndex.W = Urb->UrbControlGetStatusRequest.Index;
+ m_DescriptorPacket->bmRequestType.B = 0x82;
+ m_DescriptorPacket->wLength = 2;
+ break;
+
+ /* SET ADDRESS */
+
+ /* SET CONFIG */
+ case URB_FUNCTION_SELECT_CONFIGURATION:
+ m_DescriptorPacket->bRequest = USB_REQUEST_SET_CONFIGURATION;
+ m_DescriptorPacket->wValue.W =
Urb->UrbSelectConfiguration.ConfigurationDescriptor->bConfigurationValue;
+ m_DescriptorPacket->wIndex.W = 0;
+ m_DescriptorPacket->wLength = 0;
+ m_DescriptorPacket->bmRequestType.B = 0x00;
+ break;
+
+ /* SET DESCRIPTOR */
+ case URB_FUNCTION_SET_DESCRIPTOR_TO_DEVICE:
+ case URB_FUNCTION_SET_DESCRIPTOR_TO_INTERFACE:
+ case URB_FUNCTION_SET_DESCRIPTOR_TO_ENDPOINT:
+ UNIMPLEMENTED
+ break;
+
+ /* SET FEATURE */
+ case URB_FUNCTION_SET_FEATURE_TO_DEVICE:
+ m_DescriptorPacket->bRequest = USB_REQUEST_SET_FEATURE;
+ ASSERT(Urb->UrbControlGetStatusRequest.Index == 0);
+ m_DescriptorPacket->wIndex.W = Urb->UrbControlGetStatusRequest.Index;
+ m_DescriptorPacket->bmRequestType.B = 0x80;
+ break;
+
+ case URB_FUNCTION_SET_FEATURE_TO_INTERFACE:
+ m_DescriptorPacket->bRequest = USB_REQUEST_SET_FEATURE;
+ ASSERT(Urb->UrbControlGetStatusRequest.Index == 0);
+ m_DescriptorPacket->wIndex.W = Urb->UrbControlGetStatusRequest.Index;
+ m_DescriptorPacket->bmRequestType.B = 0x81;
+ break;
+
+ case URB_FUNCTION_SET_FEATURE_TO_ENDPOINT:
+ m_DescriptorPacket->bRequest = USB_REQUEST_SET_FEATURE;
+ ASSERT(Urb->UrbControlGetStatusRequest.Index == 0);
+ m_DescriptorPacket->wIndex.W = Urb->UrbControlGetStatusRequest.Index;
+ m_DescriptorPacket->bmRequestType.B = 0x82;
+ break;
+
+ /* SET INTERFACE*/
+ case URB_FUNCTION_SELECT_INTERFACE:
+ m_DescriptorPacket->bRequest = USB_REQUEST_SET_INTERFACE;
+ m_DescriptorPacket->wValue.W =
Urb->UrbSelectInterface.Interface.AlternateSetting;
+ m_DescriptorPacket->wIndex.W =
Urb->UrbSelectInterface.Interface.InterfaceNumber;
+ m_DescriptorPacket->wLength = 0;
+ m_DescriptorPacket->bmRequestType.B = 0x01;
+ break;
+
+ /* SYNC FRAME */
+ case URB_FUNCTION_SYNC_RESET_PIPE_AND_CLEAR_STALL:
+ UNIMPLEMENTED
+ break;
+ default:
+ UNIMPLEMENTED
+ break;
+ }
+
return Status;
}
4995
days inactive
4995
days old
0 comments
1 participants
participants (1)
-
janderwald＠svn.reactos.org