Cleaning up CLEAR_FEATURE (endpoint)

- Reverting old fix that no longer seems relevant after numerous recent USB fixes

diff --git a/Output/pjrcUSB/arm/usb_dev.c b/Output/pjrcUSB/arm/usb_dev.c
index 1cc4f139a4954b3248d9d13e3b1fcbd168691c21..125500a655eb13c250646406e0430fa4e1e02c9e 100644 (file)
--- a/Output/pjrcUSB/arm/usb_dev.c
+++ b/Output/pjrcUSB/arm/usb_dev.c
@@ -330,27 +330,21 @@ static void usb_setup()
                endpoint0_stall();
                return;
 
-       case 0x0102: // CLEAR_FEATURE (interface)
+       case 0x0102: // CLEAR_FEATURE (endpoint)
                i = setup.wIndex & 0x7F;
                if ( i > NUM_ENDPOINTS || setup.wValue != 0 )
                {
                        endpoint0_stall();
                        return;
                }
-               warn_print("CLEAR_FEATURE - Interface");
-               //(*(uint8_t *)(&USB0_ENDPT0 + setup.wIndex * 4)) &= ~0x02;
+               (*(uint8_t *)(&USB0_ENDPT0 + setup.wIndex * 4)) &= ~0x02;
                // TODO: do we need to clear the data toggle here?
-               //break;
-
-               // FIXME: Clearing causes keyboard to freeze, likely an invalid clear
-               // XXX: Ignoring seems to work, though this may not be the ideal behaviour -HaaTa
-               endpoint0_stall();
-               return;
+               goto send;
 
        case 0x0300: // SET_FEATURE (device)
        case 0x0301: // SET_FEATURE (interface)
                // TODO: Currently ignoring, perhaps useful? -HaaTa
-               warn_print("SET_FEATURE");
+               warn_print("SET_FEATURE - Device/Interface");
                endpoint0_stall();
                return;