di0ib/controller
Archived
1
0
Fork 0
Code Releases Activity

Adding additional case statements for SET_FEATURE and CLEAR_FEATURE

Browse Source 
- Ignoring (which is done by default)
- Device, Interface and Endpoint variants
This commit is contained in:
Jacob Alexander 2015-03-07 00:50:42 -08:00
parent b2dfd63d4b
commit 55346314b8
1 changed files with 12 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
Output/pjrcUSB/arm/usb_dev.c
View File

@ -300,7 +300,7 @@ static void usb_setup()
data = reply_buffer; data = reply_buffer;
break; break;
case 0x0082: // GET_STATUS (endpoint) case 0x0082: // GET_STATUS (endpoint)
if (setup.wIndex > NUM_ENDPOINTS) if ( setup.wIndex > NUM_ENDPOINTS )
{ {
// TODO: do we need to handle IN vs OUT here? // TODO: do we need to handle IN vs OUT here?
endpoint0_stall(); endpoint0_stall();
@ -313,7 +313,12 @@ static void usb_setup()
data = reply_buffer; data = reply_buffer;
datalen = 2; datalen = 2;
break; break;
case 0x0102: // CLEAR_FEATURE (endpoint) case 0x0100: // CLEAR_FEATURE (device)
case 0x0101: // CLEAR_FEATURE (interface)
// TODO: Currently ignoring, perhaps useful? -HaaTa
endpoint0_stall();
return;
case 0x0102: // CLEAR_FEATURE (interface)
i = setup.wIndex & 0x7F; i = setup.wIndex & 0x7F;
if ( i > NUM_ENDPOINTS || setup.wValue != 0 ) if ( i > NUM_ENDPOINTS || setup.wValue != 0 )
{ {
@ -328,6 +333,11 @@ static void usb_setup()
// XXX: Ignoring seems to work, though this may not be the ideal behaviour -HaaTa // XXX: Ignoring seems to work, though this may not be the ideal behaviour -HaaTa
endpoint0_stall(); endpoint0_stall();
return; return;
case 0x0300: // SET_FEATURE (device)
case 0x0301: // SET_FEATURE (interface)
// TODO: Currently ignoring, perhaps useful? -HaaTa
endpoint0_stall();
return;
case 0x0302: // SET_FEATURE (endpoint) case 0x0302: // SET_FEATURE (endpoint)
i = setup.wIndex & 0x7F; i = setup.wIndex & 0x7F;
if ( i > NUM_ENDPOINTS || setup.wValue != 0 ) if ( i > NUM_ENDPOINTS || setup.wValue != 0 )