summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNathan Conrad <[email protected]>2019-09-20 08:46:17 -0400
committerNathan Conrad <[email protected]>2019-09-20 08:46:17 -0400
commita8a65d6ceae244971405a13bc493ee923861be1f (patch)
tree6351018e65af887acd08bc8e63ba3cc23f27ebce
parent05164c5a27eaad25f28a4fc8926297d6cd32e3fc (diff)
Use cached HID descriptor.
-rw-r--r--src/class/hid/hid_device.c23
1 files changed, 3 insertions, 20 deletions
diff --git a/src/class/hid/hid_device.c b/src/class/hid/hid_device.c
index 7dabdbc17..fee44e30c 100644
--- a/src/class/hid/hid_device.c
+++ b/src/class/hid/hid_device.c
@@ -205,32 +205,15 @@ bool hidd_control_request(uint8_t rhport, tusb_control_request_t const * p_reque
if (p_request->bRequest == TUSB_REQ_GET_DESCRIPTOR && desc_type == HID_DESC_TYPE_HID)
{
- // FIXME: Should check which is the active configuration, but no callback in usbd currently exists to do that
TU_VERIFY(p_hid->hid_descriptor != NULL);
- tusb_desc_configuration_t const* desc_cfg =
- (tusb_desc_configuration_t const*) tud_descriptor_configuration_cb(0);
- uint8_t const * p_desc = ((uint8_t const*) desc_cfg) + sizeof(tusb_desc_configuration_t);
- uint8_t const * desc_end = ((uint8_t const*) desc_cfg) + desc_cfg->wTotalLength;
-
- while( p_desc < desc_end )
- {
- tusb_hid_descriptor_hid_t *p_desc_hid =(tusb_hid_descriptor_hid_t*)p_desc;
- if(p_desc_hid->bDescriptorType == HID_DESC_TYPE_HID) {
- tud_control_xfer(rhport, p_request, (void*) p_desc_hid, p_desc_hid->bLength);
- break;
- }
- p_desc += p_desc_hid->bLength; // next desc
- }
- if(p_desc >= desc_end)
- {
- return false;
- }
+ TU_VERIFY(tud_control_xfer(rhport, p_request, (void*) p_hid->hid_descriptor, p_hid->hid_descriptor->bLength));
}
else if (p_request->bRequest == TUSB_REQ_GET_DESCRIPTOR && desc_type == HID_DESC_TYPE_REPORT)
{
uint8_t const * desc_report = tud_hid_descriptor_report_cb();
tud_control_xfer(rhport, p_request, (void*) desc_report, p_hid->report_desc_len);
- }else
+ }
+ else
{
return false; // stall unsupported request
}