diff options
Diffstat (limited to 'tmk_core/protocol')
| -rw-r--r-- | tmk_core/protocol/arm_atsam/usb/udi_device_conf.h | 10 | ||||
| -rw-r--r-- | tmk_core/protocol/chibios/usb_main.c | 45 | ||||
| -rw-r--r-- | tmk_core/protocol/chibios/usb_util.c | 2 | ||||
| -rw-r--r-- | tmk_core/protocol/lufa/lufa.c | 34 | ||||
| -rw-r--r-- | tmk_core/protocol/lufa/usb_util.c | 2 | ||||
| -rw-r--r-- | tmk_core/protocol/usb_descriptor.c | 6 | ||||
| -rw-r--r-- | tmk_core/protocol/usb_descriptor_common.h | 4 | ||||
| -rw-r--r-- | tmk_core/protocol/vusb/usb_util.c | 2 | ||||
| -rw-r--r-- | tmk_core/protocol/vusb/vusb.c | 7 | 
9 files changed, 64 insertions, 48 deletions
| diff --git a/tmk_core/protocol/arm_atsam/usb/udi_device_conf.h b/tmk_core/protocol/arm_atsam/usb/udi_device_conf.h index 9c9d94789d..1c0983115c 100644 --- a/tmk_core/protocol/arm_atsam/usb/udi_device_conf.h +++ b/tmk_core/protocol/arm_atsam/usb/udi_device_conf.h @@ -23,6 +23,10 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  #include "compiler.h"  #include "usb_protocol_hid.h" +#ifndef USB_POLLING_INTERVAL_MS +#    define USB_POLLING_INTERVAL_MS 10 +#endif +  #ifdef VIRTSER_ENABLE  // because CDC uses IAD (interface association descriptor  // per USB Interface Association Descriptor Device Class Code and Use Model 7/23/2003 Rev 1.0) @@ -118,7 +122,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  #define UDI_HID_KBD_EP_IN KEYBOARD_IN_EPNUM  #define NEXT_IN_EPNUM_1 (KEYBOARD_IN_EPNUM + 1)  #define UDI_HID_KBD_EP_SIZE KEYBOARD_EPSIZE -#define KBD_POLLING_INTERVAL 10 +#define KBD_POLLING_INTERVAL USB_POLLING_INTERVAL_MS  #ifndef UDI_HID_KBD_STRING_ID  #    define UDI_HID_KBD_STRING_ID 0  #endif @@ -128,7 +132,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  #    define NEXT_IN_EPNUM_2 (MOUSE_IN_EPNUM + 1)  #    define UDI_HID_MOU_EP_IN MOUSE_IN_EPNUM  #    define UDI_HID_MOU_EP_SIZE MOUSE_EPSIZE -#    define MOU_POLLING_INTERVAL 10 +#    define MOU_POLLING_INTERVAL USB_POLLING_INTERVAL_MS  #    ifndef UDI_HID_MOU_STRING_ID  #        define UDI_HID_MOU_STRING_ID 0  #    endif @@ -141,7 +145,7 @@ along with this program.  If not, see <http://www.gnu.org/licenses/>.  #    define UDI_HID_EXK_EP_IN EXTRAKEY_IN_EPNUM  #    define NEXT_IN_EPNUM_3 (EXTRAKEY_IN_EPNUM + 1)  #    define UDI_HID_EXK_EP_SIZE EXTRAKEY_EPSIZE -#    define EXTRAKEY_POLLING_INTERVAL 10 +#    define EXTRAKEY_POLLING_INTERVAL USB_POLLING_INTERVAL_MS  #    ifndef UDI_HID_EXK_STRING_ID  #        define UDI_HID_EXK_STRING_ID 0  #    endif diff --git a/tmk_core/protocol/chibios/usb_main.c b/tmk_core/protocol/chibios/usb_main.c index d04302acae..e5edd74dcb 100644 --- a/tmk_core/protocol/chibios/usb_main.c +++ b/tmk_core/protocol/chibios/usb_main.c @@ -415,14 +415,18 @@ static inline void usb_event_wakeup_handler(void) {  #endif /* SLEEP_LED_ENABLE */  } +bool last_suspend_state = false; +  void usb_event_queue_task(void) {      usbevent_t event;      while (usb_event_queue_dequeue(&event)) {          switch (event) {              case USB_EVENT_SUSPEND: +                last_suspend_state = true;                  usb_event_suspend_handler();                  break;              case USB_EVENT_WAKEUP: +                last_suspend_state = false;                  usb_event_wakeup_handler();                  break;              default: @@ -464,6 +468,9 @@ static void usb_event_cb(USBDriver *usbp, usbevent_t event) {                  qmkusbConfigureHookI(&drivers.array[i].driver);              }              osalSysUnlockFromISR(); +            if (last_suspend_state) { +                usb_event_queue_enqueue(USB_EVENT_WAKEUP); +            }              return;          case USB_EVENT_SUSPEND:              usb_event_queue_enqueue(USB_EVENT_SUSPEND); @@ -518,7 +525,7 @@ static uint16_t get_hword(uint8_t *p) {   * Other Device    Required    Optional    Optional    Optional    Optional    Optional   */ -static uint8_t set_report_buf[2] __attribute__((aligned(2))); +static uint8_t set_report_buf[2] __attribute__((aligned(4)));  static void    set_led_transfer_cb(USBDriver *usbp) {      if (usbp->setup[6] == 2) { /* LSB(wLength) */          uint8_t report_id = set_report_buf[0]; @@ -705,7 +712,7 @@ void init_usb_driver(USBDriver *usbp) {      chVTObjectInit(&keyboard_idle_timer);  } -void restart_usb_driver(USBDriver *usbp) { +__attribute__((weak)) void restart_usb_driver(USBDriver *usbp) {      usbStop(usbp);      usbDisconnectBus(usbp); @@ -903,7 +910,8 @@ static void send_extra(uint8_t report_id, uint16_t data) {          return;      } -    report_extra_t report = {.report_id = report_id, .usage = data}; +    static report_extra_t report; +    report = (report_extra_t){.report_id = report_id, .usage = data};      usbStartTransmitI(&USB_DRIVER, SHARED_IN_EPNUM, (uint8_t *)&report, sizeof(report_extra_t));      osalSysUnlock(); @@ -1051,45 +1059,44 @@ void virtser_task(void) {  #ifdef JOYSTICK_ENABLE  void send_joystick_packet(joystick_t *joystick) { -    joystick_report_t rep = { +    static joystick_report_t rep; +    rep = (joystick_report_t) {  #    if JOYSTICK_AXES_COUNT > 0          .axes = -            { -                joystick->axes[0], +        { joystick->axes[0],  #        if JOYSTICK_AXES_COUNT >= 2 -                joystick->axes[1], +          joystick->axes[1],  #        endif  #        if JOYSTICK_AXES_COUNT >= 3 -                joystick->axes[2], +          joystick->axes[2],  #        endif  #        if JOYSTICK_AXES_COUNT >= 4 -                joystick->axes[3], +          joystick->axes[3],  #        endif  #        if JOYSTICK_AXES_COUNT >= 5 -                joystick->axes[4], +          joystick->axes[4],  #        endif  #        if JOYSTICK_AXES_COUNT >= 6 -                joystick->axes[5], +          joystick->axes[5],  #        endif -            }, +        },  #    endif  // JOYSTICK_AXES_COUNT>0  #    if JOYSTICK_BUTTON_COUNT > 0 -        .buttons = -            { -                joystick->buttons[0], +        .buttons = { +            joystick->buttons[0],  #        if JOYSTICK_BUTTON_COUNT > 8 -                joystick->buttons[1], +            joystick->buttons[1],  #        endif  #        if JOYSTICK_BUTTON_COUNT > 16 -                joystick->buttons[2], +            joystick->buttons[2],  #        endif  #        if JOYSTICK_BUTTON_COUNT > 24 -                joystick->buttons[3], +            joystick->buttons[3],  #        endif -            } +        }  #    endif  // JOYSTICK_BUTTON_COUNT>0      }; diff --git a/tmk_core/protocol/chibios/usb_util.c b/tmk_core/protocol/chibios/usb_util.c index 5945e8a8de..e32d6ebfa4 100644 --- a/tmk_core/protocol/chibios/usb_util.c +++ b/tmk_core/protocol/chibios/usb_util.c @@ -16,6 +16,6 @@  #include <hal.h>  #include "usb_util.h" -void usb_disable(void) { usbStop(&USBD1); } +void usb_disconnect(void) { usbStop(&USBD1); }  bool usb_connected_state(void) { return usbGetDriverStateI(&USBD1) == USB_ACTIVE; } diff --git a/tmk_core/protocol/lufa/lufa.c b/tmk_core/protocol/lufa/lufa.c index 63619fdb3b..4ac079e168 100644 --- a/tmk_core/protocol/lufa/lufa.c +++ b/tmk_core/protocol/lufa/lufa.c @@ -314,45 +314,44 @@ static void Console_Task(void) {  void send_joystick_packet(joystick_t *joystick) {      uint8_t timeout = 255; -    joystick_report_t r = { +    static joystick_report_t; +    r = (joystick_report_t) {  #    if JOYSTICK_AXES_COUNT > 0          .axes = -            { -                joystick->axes[0], +        { joystick->axes[0],  #        if JOYSTICK_AXES_COUNT >= 2 -                joystick->axes[1], +          joystick->axes[1],  #        endif  #        if JOYSTICK_AXES_COUNT >= 3 -                joystick->axes[2], +          joystick->axes[2],  #        endif  #        if JOYSTICK_AXES_COUNT >= 4 -                joystick->axes[3], +          joystick->axes[3],  #        endif  #        if JOYSTICK_AXES_COUNT >= 5 -                joystick->axes[4], +          joystick->axes[4],  #        endif  #        if JOYSTICK_AXES_COUNT >= 6 -                joystick->axes[5], +          joystick->axes[5],  #        endif -            }, +        },  #    endif  // JOYSTICK_AXES_COUNT>0  #    if JOYSTICK_BUTTON_COUNT > 0 -        .buttons = -            { -                joystick->buttons[0], +        .buttons = { +            joystick->buttons[0],  #        if JOYSTICK_BUTTON_COUNT > 8 -                joystick->buttons[1], +            joystick->buttons[1],  #        endif  #        if JOYSTICK_BUTTON_COUNT > 16 -                joystick->buttons[2], +            joystick->buttons[2],  #        endif  #        if JOYSTICK_BUTTON_COUNT > 24 -                joystick->buttons[3], +            joystick->buttons[3],  #        endif -            } +        }  #    endif  // JOYSTICK_BUTTON_COUNT>0      }; @@ -768,7 +767,8 @@ static void send_extra(uint8_t report_id, uint16_t data) {      if (USB_DeviceState != DEVICE_STATE_Configured) return; -    report_extra_t r = {.report_id = report_id, .usage = data}; +    static report_extra_t r; +    r = (report_extra_t){.report_id = report_id, .usage = data};      Endpoint_SelectEndpoint(SHARED_IN_EPNUM);      /* Check if write ready for a polling interval around 10ms */ diff --git a/tmk_core/protocol/lufa/usb_util.c b/tmk_core/protocol/lufa/usb_util.c index 9e943a21b9..9691eff1e4 100644 --- a/tmk_core/protocol/lufa/usb_util.c +++ b/tmk_core/protocol/lufa/usb_util.c @@ -17,7 +17,7 @@  #include "usb_util.h"  #include "wait.h" -void usb_disable(void) { +void usb_disconnect(void) {      USB_Disable();      USB_DeviceState = DEVICE_STATE_Unattached;  } diff --git a/tmk_core/protocol/usb_descriptor.c b/tmk_core/protocol/usb_descriptor.c index ba7760f283..7a4a790315 100644 --- a/tmk_core/protocol/usb_descriptor.c +++ b/tmk_core/protocol/usb_descriptor.c @@ -351,7 +351,7 @@ const USB_Descriptor_Device_t PROGMEM DeviceDescriptor = {          .Size                   = sizeof(USB_Descriptor_Device_t),          .Type                   = DTYPE_Device      }, -    .USBSpecification           = VERSION_BCD(1, 1, 0), +    .USBSpecification           = VERSION_BCD(2, 0, 0),  #if VIRTSER_ENABLE      .Class                      = USB_CSCP_IADDeviceClass, @@ -953,10 +953,10 @@ const USB_Descriptor_String_t PROGMEM ProductString = {  #if defined(SERIAL_NUMBER)  const USB_Descriptor_String_t PROGMEM SerialNumberString = {      .Header = { -        .Size                   = USB_STRING_LEN(sizeof(STR(SERIAL_NUMBER)) - 1), // Subtract 1 for null terminator +        .Size                   = USB_STRING_LEN(sizeof(SERIAL_NUMBER) - 1), // Subtract 1 for null terminator          .Type                   = DTYPE_String      }, -    .UnicodeString              = LSTR(SERIAL_NUMBER) +    .UnicodeString              = USBSTR(SERIAL_NUMBER)  };  #endif diff --git a/tmk_core/protocol/usb_descriptor_common.h b/tmk_core/protocol/usb_descriptor_common.h index b1f602c82e..ce0cf09763 100644 --- a/tmk_core/protocol/usb_descriptor_common.h +++ b/tmk_core/protocol/usb_descriptor_common.h @@ -16,6 +16,10 @@  #pragma once +// Prefix string literal with L for descriptors +#define USBCONCAT(a, b) a##b +#define USBSTR(s) USBCONCAT(L, s) +  /////////////////////  // RAW Usage page and ID configuration diff --git a/tmk_core/protocol/vusb/usb_util.c b/tmk_core/protocol/vusb/usb_util.c index 602854dbe6..4ee2d3188b 100644 --- a/tmk_core/protocol/vusb/usb_util.c +++ b/tmk_core/protocol/vusb/usb_util.c @@ -16,7 +16,7 @@  #include <usbdrv/usbdrv.h>  #include "usb_util.h" -void usb_disable(void) { usbDeviceDisconnect(); } +void usb_disconnect(void) { usbDeviceDisconnect(); }  bool usb_connected_state(void) {      usbPoll(); diff --git a/tmk_core/protocol/vusb/vusb.c b/tmk_core/protocol/vusb/vusb.c index 9362fbde78..98cebf6012 100644 --- a/tmk_core/protocol/vusb/vusb.c +++ b/tmk_core/protocol/vusb/vusb.c @@ -272,7 +272,8 @@ static void send_extra(uint8_t report_id, uint16_t data) {      last_id   = report_id;      last_data = data; -    report_extra_t report = {.report_id = report_id, .usage = data}; +    static report_extra_t report; +    report = (report_extra_t){.report_id = report_id, .usage = data};      if (usbInterruptIsReadyShared()) {          usbSetInterruptShared((void *)&report, sizeof(report_extra_t));      } @@ -598,10 +599,10 @@ const PROGMEM usbStringDescriptor_t usbStringDescriptorProduct = {  #if defined(SERIAL_NUMBER)  const PROGMEM usbStringDescriptor_t usbStringDescriptorSerial = {      .header = { -        .bLength         = USB_STRING_LEN(sizeof(STR(SERIAL_NUMBER)) - 1), +        .bLength         = USB_STRING_LEN(sizeof(SERIAL_NUMBER) - 1),          .bDescriptorType = USBDESCR_STRING      }, -    .bString             = LSTR(SERIAL_NUMBER) +    .bString             = USBSTR(SERIAL_NUMBER)  };  #endif | 
