Remove unused OUTPUT endpoint to simplify

The HIDPowerDevice HID report descriptor only contains INPUT and FEATURE reports, and no OUTPUT reports. It therefore makes little sense to configure a USB input endpoint for INPUT reports.

This also makes the implementation more similar to the upstream Arduino code on https://github.com/arduino/ArduinoCore-avr/blob/master/libraries/HID/src/HID.cpp

Author: forderud

src/HID/HID.cpp

@@ -31,10 +31,9 @@ int HID_::getInterface(uint8_t* interfaceCount)
 {
     *interfaceCount += 1; // uses 1
     HIDDescriptor hidInterface = {
-        D_INTERFACE(pluggedInterface, 2, USB_DEVICE_CLASS_HUMAN_INTERFACE, HID_SUBCLASS_NONE, HID_PROTOCOL_NONE),
+        D_INTERFACE(pluggedInterface, 1, USB_DEVICE_CLASS_HUMAN_INTERFACE, HID_SUBCLASS_NONE, HID_PROTOCOL_NONE),
         D_HIDREPORT(descriptorSize),
-        D_ENDPOINT(USB_ENDPOINT_IN(HID_TX), USB_ENDPOINT_TYPE_INTERRUPT, USB_EP_SIZE, 0x14),
-        D_ENDPOINT(USB_ENDPOINT_OUT(HID_RX), USB_ENDPOINT_TYPE_INTERRUPT, USB_EP_SIZE, 0x0A)
+        D_ENDPOINT(USB_ENDPOINT_IN(HID_TX), USB_ENDPOINT_TYPE_INTERRUPT, USB_EP_SIZE, 0x14)
     };
     return USB_SendControl(0, &hidInterface, sizeof(hidInterface));
 }
@@ -254,12 +253,11 @@ bool HID_::setup(USBSetup& setup)
     return false;
 }
 
-HID_::HID_(void) : PluggableUSBModule(2, 1, epType),
+HID_::HID_(void) : PluggableUSBModule(1, 1, epType),
                    rootNode(NULL), descriptorSize(0),
                    protocol(HID_REPORT_PROTOCOL), idle(1)
 {
     epType[0] = EP_TYPE_INTERRUPT_IN;
-        epType[1] = EP_TYPE_INTERRUPT_OUT;
     PluggableUSB().plug(this);
 }
 

src/HID/HID.h

@@ -139,7 +139,7 @@ class HID_ : public PluggableUSBModule
     uint8_t getShortName(char* name) override;
 
 private:
-    uint8_t epType[2];
+    uint8_t epType[1];
 
     HIDSubDescriptor* rootNode;
     uint16_t descriptorSize;