di0ib
/
controller
Sledovat 1
Oblíbit 0
Rozštěpit 0
Zdrojový kód Vydání 0 Aktivita
Procházet zdrojové kódy

Fixing Linux NKRO Delete bug 

- Fixed descriptor to not include USB Code 156 (Clear)
- This USB Code affects repeating Delete
simple
Jacob Alexander před 9 roky
rodič
1da4903816
revize
11fb8bd77d
6 změnil soubory, kde provedl 107 přidání a 41 odebrání
Jednotný pohled Ukázat statistiku rozdílových dat
  1. 22
    9
      Output/pjrcUSB/arm/usb_desc.c
  2. 19
    6
      Output/pjrcUSB/arm/usb_keyboard.c
  3. 18
    5
      Output/pjrcUSB/avr/usb_keyboard_serial.c
  4. 22
    9
      Output/pjrcUSB/avr/usb_keyboard_serial.h
  5. 22
    9
      Output/pjrcUSB/output_com.c
  6. 4
    3
      Output/pjrcUSB/output_com.h

+ 22
- 9
Output/pjrcUSB/arm/usb_desc.c Zobrazit soubor

// See http://www.usb.org/developers/hidpage/Hut1_12v2.pdf Chapter 10 // See http://www.usb.org/developers/hidpage/Hut1_12v2.pdf Chapter 10
// Or Macros/PartialMap/usb_hid.h // Or Macros/PartialMap/usb_hid.h
// //
// 50 must be excluded due to a Linux bug with bitmaps (not useful anyways)
// 50 (ISO \ due to \ bug) and 156 (Clear due to Delete bug) must be excluded
// due to a Linux bug with bitmaps (not useful anyways)
// 165-175 are reserved/unused as well as 222-223 and 232-65535 // 165-175 are reserved/unused as well as 222-223 and 232-65535
// 224-231 are used for modifiers (see above) // 224-231 are used for modifiers (see above)
// //
// //
// Packing of bitmaps are as follows: // Packing of bitmaps are as follows:
// 4-49 : 6 bytes + 1 Report ID byte (0x04-0x31) ( 46 bits + 2 padding bits for 6 bytes total) // 4-49 : 6 bytes + 1 Report ID byte (0x04-0x31) ( 46 bits + 2 padding bits for 6 bytes total)
// 51-164 : 20 bytes + 1 Report ID byte (0x33-0xA4) (114 bits + 6 padding bits for 15 bytes total)
// 51-155 : 14 bytes + 1 Report ID byte (0x33-0x9B) (105 bits + 6 padding bits for 15 bytes total)
// 157-164 : 1 byte + 1 Report ID byte (0x9D-0xA4) ( 8 bits)
// 176-221 : 6 bytes + 1 Report ID byte (0xB0-0xDD) ( 46 bits + 2 padding bits for 6 bytes total) // 176-221 : 6 bytes + 1 Report ID byte (0xB0-0xDD) ( 46 bits + 2 padding bits for 6 bytes total)
// //
// 4-49 (6 bytes/46 bits)
// 4-49 (6 bytes/46 bits) - MainKeys
0x85, 0x03, // Report ID (3), 0x85, 0x03, // Report ID (3),
0x75, 0x01, // Report Size (1), 0x75, 0x01, // Report Size (1),
0x95, 0x2E, // Report Count (46), 0x95, 0x2E, // Report Count (46),


// Should pad 2 bits according to the spec, but OSX doesn't like this -HaaTa // Should pad 2 bits according to the spec, but OSX doesn't like this -HaaTa


// 51-164 (15 bytes/160 bits)
// 51-155 (14 bytes/105 bits) - SecondaryKeys
0x85, 0x04, // Report ID (4), 0x85, 0x04, // Report ID (4),
0x75, 0x01, // Report Size (1), 0x75, 0x01, // Report Size (1),
0x95, 0xA0, // Report Count (160),
0x95, 0x69, // Report Count (105),
0x15, 0x00, // Logical Minimum (0), 0x15, 0x00, // Logical Minimum (0),
0x25, 0x01, // Logical Maximum (1), 0x25, 0x01, // Logical Maximum (1),
0x05, 0x07, // Usage Page (Key Codes), 0x05, 0x07, // Usage Page (Key Codes),
0x19, 0x33, // Usage Minimum (51), 0x19, 0x33, // Usage Minimum (51),
0x29, 0xA4, // Usage Maximum (164),
0x29, 0x9B, // Usage Maximum (155),
0x81, 0x02, // Input (Data, Variable, Absolute, Bitfield), 0x81, 0x02, // Input (Data, Variable, Absolute, Bitfield),


// Should pad 6 bits according to the spec, but OSX doesn't like this -HaaTa // Should pad 6 bits according to the spec, but OSX doesn't like this -HaaTa


// 176-221 (6 bytes/46 bits)
// 157-164 (1 byte/8 bits) - TertiaryKeys
0x85, 0x05, // Report ID (5), 0x85, 0x05, // Report ID (5),
0x75, 0x01, // Report Size (1), 0x75, 0x01, // Report Size (1),
0x95, 0x08, // Report Count (8),
0x15, 0x00, // Logical Minimum (0),
0x25, 0x01, // Logical Maximum (1),
0x05, 0x07, // Usage Page (Key Codes),
0x19, 0x9D, // Usage Minimum (157),
0x29, 0xA4, // Usage Maximum (164),
0x81, 0x02, // Input (Data, Variable, Absolute, Bitfield),

// 176-221 (6 bytes/46 bits) - QuartiaryKeys
0x85, 0x06, // Report ID (6),
0x75, 0x01, // Report Size (1),
0x95, 0x2D, // Report Count (45), 0x95, 0x2D, // Report Count (45),
0x15, 0x00, // Logical Minimum (0), 0x15, 0x00, // Logical Minimum (0),
0x25, 0x01, // Logical Maximum (1), 0x25, 0x01, // Logical Maximum (1),
0x05, 0x01, // Usage Page (Generic Desktop), 0x05, 0x01, // Usage Page (Generic Desktop),
0x09, 0x80, // Usage (System Control), 0x09, 0x80, // Usage (System Control),
0xA1, 0x01, // Collection (Application), 0xA1, 0x01, // Collection (Application),
0x85, 0x06, // Report ID (6),
0x85, 0x07, // Report ID (7),
0x75, 0x08, // Report Size (8), 0x75, 0x08, // Report Size (8),
0x95, 0x01, // Report Count (1), 0x95, 0x01, // Report Count (1),
0x16, 0x81, 0x00, // Logical Minimum (129), 0x16, 0x81, 0x00, // Logical Minimum (129),
0x05, 0x0c, // Usage Page (Consumer), 0x05, 0x0c, // Usage Page (Consumer),
0x09, 0x01, // Usage (Consumer Control), 0x09, 0x01, // Usage (Consumer Control),
0xA1, 0x01, // Collection (Application), 0xA1, 0x01, // Collection (Application),
0x85, 0x07, // Report ID (7),
0x85, 0x08, // Report ID (8),
0x75, 0x10, // Report Size (16), 0x75, 0x10, // Report Size (16),
0x95, 0x01, // Report Count (1), 0x95, 0x01, // Report Count (1),
0x16, 0x20, 0x00, // Logical Minimum (32), 0x16, 0x20, 0x00, // Logical Minimum (32),

+ 19
- 6
Output/pjrcUSB/arm/usb_keyboard.c Zobrazit soubor

{ {
*tx_buf++ = 0x04; // ID *tx_buf++ = 0x04; // ID


// 51-164 (Middle 15 bytes)
memcpy( tx_buf, USBKeys_Keys + 6, 15 );
tx_packet->len = 16;
// 51-155 (Middle 14 bytes)
memcpy( tx_buf, USBKeys_Keys + 6, 14 );
tx_packet->len = 15;


// Send USB Packet // Send USB Packet
usb_tx( NKRO_KEYBOARD_ENDPOINT, tx_packet ); usb_tx( NKRO_KEYBOARD_ENDPOINT, tx_packet );
{ {
*tx_buf++ = 0x05; // ID *tx_buf++ = 0x05; // ID


// 157-164 (Next byte)
memcpy( tx_buf, USBKeys_Keys + 20, 1 );
tx_packet->len = 2;

// Send USB Packet
usb_tx( NKRO_KEYBOARD_ENDPOINT, tx_packet );
USBKeys_Changed &= ~USBKeyChangeState_TertiaryKeys; // Mark sent
}
// Check quartiary key section
else if ( USBKeys_Changed & USBKeyChangeState_QuartiaryKeys )
{
*tx_buf++ = 0x06; // ID

// 176-221 (last 6 bytes) // 176-221 (last 6 bytes)
memcpy( tx_buf, USBKeys_Keys + 21, 6 ); memcpy( tx_buf, USBKeys_Keys + 21, 6 );
tx_packet->len = 7; tx_packet->len = 7;


// Send USB Packet // Send USB Packet
usb_tx( NKRO_KEYBOARD_ENDPOINT, tx_packet ); usb_tx( NKRO_KEYBOARD_ENDPOINT, tx_packet );
USBKeys_Changed &= ~USBKeyChangeState_TertiaryKeys; // Mark sent
USBKeys_Changed &= ~USBKeyChangeState_QuartiaryKeys; // Mark sent
} }
// Check system control keys // Check system control keys
else if ( USBKeys_Changed & USBKeyChangeState_System ) else if ( USBKeys_Changed & USBKeyChangeState_System )
{ {
*tx_buf++ = 0x06; // ID
*tx_buf++ = 0x07; // ID
*tx_buf = USBKeys_SysCtrl; *tx_buf = USBKeys_SysCtrl;
tx_packet->len = 2; tx_packet->len = 2;


// Check consumer control keys // Check consumer control keys
else if ( USBKeys_Changed & USBKeyChangeState_Consumer ) else if ( USBKeys_Changed & USBKeyChangeState_Consumer )
{ {
*tx_buf++ = 0x07; // ID
*tx_buf++ = 0x08; // ID
*tx_buf++ = (uint8_t)(USBKeys_ConsCtrl & 0x00FF); *tx_buf++ = (uint8_t)(USBKeys_ConsCtrl & 0x00FF);
*tx_buf = (uint8_t)(USBKeys_ConsCtrl >> 8); *tx_buf = (uint8_t)(USBKeys_ConsCtrl >> 8);
tx_packet->len = 3; tx_packet->len = 3;

+ 18
- 5
Output/pjrcUSB/avr/usb_keyboard_serial.c Zobrazit soubor

{ {
UEDATX = 0x04; // ID UEDATX = 0x04; // ID


// 51-164 (Middle 15 bytes)
for ( uint8_t byte = 6; byte < 21; byte++ )
// 51-155 (Middle 14 bytes)
for ( uint8_t byte = 6; byte < 20; byte++ )
UEDATX = USBKeys_Keys[ byte ]; UEDATX = USBKeys_Keys[ byte ];


UEINTX = 0; // Finished with ID UEINTX = 0; // Finished with ID
{ {
UEDATX = 0x05; // ID UEDATX = 0x05; // ID


// 157-164 (Next byte)
for ( uint8_t byte = 20; byte < 21; byte++ )
UEDATX = USBKeys_Keys[ byte ];

UEINTX = 0; // Finished with ID

USBKeys_Changed &= ~USBKeyChangeState_TertiaryKeys; // Mark sent
}
// Check quartiary key section
if ( USBKeys_Changed & USBKeyChangeState_TertiaryKeys )
{
UEDATX = 0x06; // ID

// 176-221 (last 6 bytes) // 176-221 (last 6 bytes)
for ( uint8_t byte = 21; byte < 27; byte++ ) for ( uint8_t byte = 21; byte < 27; byte++ )
UEDATX = USBKeys_Keys[ byte ]; UEDATX = USBKeys_Keys[ byte ];


UEINTX = 0; // Finished with ID UEINTX = 0; // Finished with ID


USBKeys_Changed &= ~USBKeyChangeState_TertiaryKeys; // Mark sent
USBKeys_Changed &= ~USBKeyChangeState_QuartiaryKeys; // Mark sent
} }
// Check system control keys // Check system control keys
if ( USBKeys_Changed & USBKeyChangeState_System ) if ( USBKeys_Changed & USBKeyChangeState_System )
{ {
UEDATX = 0x06; // ID
UEDATX = 0x07; // ID
UEDATX = USBKeys_SysCtrl; UEDATX = USBKeys_SysCtrl;
UEINTX = 0; // Finished with ID UEINTX = 0; // Finished with ID


// Check consumer control keys // Check consumer control keys
if ( USBKeys_Changed & USBKeyChangeState_Consumer ) if ( USBKeys_Changed & USBKeyChangeState_Consumer )
{ {
UEDATX = 0x07; // ID
UEDATX = 0x08; // ID
UEDATX = (uint8_t)(USBKeys_ConsCtrl & 0x00FF); UEDATX = (uint8_t)(USBKeys_ConsCtrl & 0x00FF);
UEDATX = (uint8_t)(USBKeys_ConsCtrl >> 8); UEDATX = (uint8_t)(USBKeys_ConsCtrl >> 8);
UEINTX = 0; // Finished with ID UEINTX = 0; // Finished with ID

+ 22
- 9
Output/pjrcUSB/avr/usb_keyboard_serial.h Zobrazit soubor

// See http://www.usb.org/developers/hidpage/Hut1_12v2.pdf Chapter 10 // See http://www.usb.org/developers/hidpage/Hut1_12v2.pdf Chapter 10
// Or Macros/PartialMap/usb_hid.h // Or Macros/PartialMap/usb_hid.h
// //
// 50 must be excluded due to a Linux bug with bitmaps (not useful anyways)
// 50 (ISO \ due to \ bug) and 156 (Clear due to Delete bug) must be excluded
// due to a Linux bug with bitmaps (not useful anyways)
// 165-175 are reserved/unused as well as 222-223 and 232-65535 // 165-175 are reserved/unused as well as 222-223 and 232-65535
// 224-231 are used for modifiers (see above) // 224-231 are used for modifiers (see above)
// //
// //
// Packing of bitmaps are as follows: // Packing of bitmaps are as follows:
// 4-49 : 6 bytes + 1 Report ID byte (0x04-0x31) ( 46 bits + 2 padding bits for 6 bytes total) // 4-49 : 6 bytes + 1 Report ID byte (0x04-0x31) ( 46 bits + 2 padding bits for 6 bytes total)
// 51-164 : 20 bytes + 1 Report ID byte (0x33-0xA4) (114 bits + 6 padding bits for 15 bytes total)
// 51-155 : 14 bytes + 1 Report ID byte (0x33-0x9B) (105 bits + 6 padding bits for 15 bytes total)
// 157-164 : 1 byte + 1 Report ID byte (0x9D-0xA4) ( 8 bits)
// 176-221 : 6 bytes + 1 Report ID byte (0xB0-0xDD) ( 46 bits + 2 padding bits for 6 bytes total) // 176-221 : 6 bytes + 1 Report ID byte (0xB0-0xDD) ( 46 bits + 2 padding bits for 6 bytes total)
// //
// 4-49 (6 bytes/46 bits)
// 4-49 (6 bytes/46 bits) - MainKeys
0x85, 0x03, // Report ID (3), 0x85, 0x03, // Report ID (3),
0x75, 0x01, // Report Size (1), 0x75, 0x01, // Report Size (1),
0x95, 0x2E, // Report Count (46), 0x95, 0x2E, // Report Count (46),


// Should pad 2 bits according to the spec, but OSX doesn't like this -HaaTa // Should pad 2 bits according to the spec, but OSX doesn't like this -HaaTa


// 51-164 (15 bytes/160 bits)
// 51-155 (14 bytes/105 bits) - SecondaryKeys
0x85, 0x04, // Report ID (4), 0x85, 0x04, // Report ID (4),
0x75, 0x01, // Report Size (1), 0x75, 0x01, // Report Size (1),
0x95, 0xA0, // Report Count (160),
0x95, 0x69, // Report Count (105),
0x15, 0x00, // Logical Minimum (0), 0x15, 0x00, // Logical Minimum (0),
0x25, 0x01, // Logical Maximum (1), 0x25, 0x01, // Logical Maximum (1),
0x05, 0x07, // Usage Page (Key Codes), 0x05, 0x07, // Usage Page (Key Codes),
0x19, 0x33, // Usage Minimum (51), 0x19, 0x33, // Usage Minimum (51),
0x29, 0xA4, // Usage Maximum (164),
0x29, 0x9B, // Usage Maximum (155),
0x81, 0x02, // Input (Data, Variable, Absolute, Bitfield), 0x81, 0x02, // Input (Data, Variable, Absolute, Bitfield),


// Should pad 6 bits according to the spec, but OSX doesn't like this -HaaTa // Should pad 6 bits according to the spec, but OSX doesn't like this -HaaTa


// 176-221 (6 bytes/46 bits)
// 157-164 (1 byte/8 bits) - TertiaryKeys
0x85, 0x05, // Report ID (5), 0x85, 0x05, // Report ID (5),
0x75, 0x01, // Report Size (1), 0x75, 0x01, // Report Size (1),
0x95, 0x08, // Report Count (8),
0x15, 0x00, // Logical Minimum (0),
0x25, 0x01, // Logical Maximum (1),
0x05, 0x07, // Usage Page (Key Codes),
0x19, 0x9D, // Usage Minimum (157),
0x29, 0xA4, // Usage Maximum (164),
0x81, 0x02, // Input (Data, Variable, Absolute, Bitfield),

// 176-221 (6 bytes/46 bits) - QuartiaryKeys
0x85, 0x06, // Report ID (6),
0x75, 0x01, // Report Size (1),
0x95, 0x2D, // Report Count (45), 0x95, 0x2D, // Report Count (45),
0x15, 0x00, // Logical Minimum (0), 0x15, 0x00, // Logical Minimum (0),
0x25, 0x01, // Logical Maximum (1), 0x25, 0x01, // Logical Maximum (1),
0x05, 0x01, // Usage Page (Generic Desktop), 0x05, 0x01, // Usage Page (Generic Desktop),
0x09, 0x80, // Usage (System Control), 0x09, 0x80, // Usage (System Control),
0xA1, 0x01, // Collection (Application), 0xA1, 0x01, // Collection (Application),
0x85, 0x06, // Report ID (6),
0x85, 0x07, // Report ID (7),
0x75, 0x08, // Report Size (8), 0x75, 0x08, // Report Size (8),
0x95, 0x01, // Report Count (1), 0x95, 0x01, // Report Count (1),
0x16, 0x81, 0x00, // Logical Minimum (129), 0x16, 0x81, 0x00, // Logical Minimum (129),
0x05, 0x0c, // Usage Page (Consumer), 0x05, 0x0c, // Usage Page (Consumer),
0x09, 0x01, // Usage (Consumer Control), 0x09, 0x01, // Usage (Consumer Control),
0xA1, 0x01, // Collection (Application), 0xA1, 0x01, // Collection (Application),
0x85, 0x07, // Report ID (7),
0x85, 0x08, // Report ID (8),
0x75, 0x10, // Report Size (16), 0x75, 0x10, // Report Size (16),
0x95, 0x01, // Report Count (1), 0x95, 0x01, // Report Count (1),
0x16, 0x20, 0x00, // Logical Minimum (32), 0x16, 0x20, 0x00, // Logical Minimum (32),

+ 22
- 9
Output/pjrcUSB/output_com.c Zobrazit soubor

break; break;
} }
// First 6 bytes // First 6 bytes
else if ( key >= 4 && key <= 50 )
else if ( key >= 4 && key <= 49 )
{ {
// Lookup (otherwise division or multiple checks are needed to do alignment) // Lookup (otherwise division or multiple checks are needed to do alignment)
uint8_t keyPos = key - (4 - 0); // Starting position in array
// Starting at 0th position, each byte has 8 bits, starting at 4th bit
uint8_t keyPos = key + (0 * 8 - 4); // Starting position in array, Ignoring 4 keys
switch ( keyPos ) switch ( keyPos )
{ {
byteLookup( 0 ); byteLookup( 0 );


USBKeys_Changed |= USBKeyChangeState_MainKeys; USBKeys_Changed |= USBKeyChangeState_MainKeys;
} }
// Next 15 bytes
else if ( key >= 51 && key <= 164 )
// Next 14 bytes
else if ( key >= 51 && key <= 155 )
{ {
// Lookup (otherwise division or multiple checks are needed to do alignment) // Lookup (otherwise division or multiple checks are needed to do alignment)
uint8_t keyPos = key - (51 - 48); // Starting position in array
// Starting at 6th byte position, each byte has 8 bits, starting at 51st bit
uint8_t keyPos = key + (6 * 8 - 51); // Starting position in array
switch ( keyPos ) switch ( keyPos )
{ {
byteLookup( 6 ); byteLookup( 6 );
byteLookup( 17 ); byteLookup( 17 );
byteLookup( 18 ); byteLookup( 18 );
byteLookup( 19 ); byteLookup( 19 );
byteLookup( 20 );
} }


USBKeys_Changed |= USBKeyChangeState_SecondaryKeys; USBKeys_Changed |= USBKeyChangeState_SecondaryKeys;
} }
// Next byte
else if ( key >= 157 && key <= 164 )
{
// Lookup (otherwise division or multiple checks are needed to do alignment)
uint8_t keyPos = key + (20 * 8 - 157); // Starting position in array, Ignoring 6 keys
switch ( keyPos )
{
byteLookup( 20 );
}

USBKeys_Changed |= USBKeyChangeState_TertiaryKeys;
}
// Last 6 bytes // Last 6 bytes
else if ( key >= 176 && key <= 221 ) else if ( key >= 176 && key <= 221 )
{ {
// Lookup (otherwise division or multiple checks are needed to do alignment) // Lookup (otherwise division or multiple checks are needed to do alignment)
uint8_t keyPos = key - (176 - 168); // Starting position in array
uint8_t keyPos = key + (21 * 8 - 176); // Starting position in array
switch ( keyPos ) switch ( keyPos )
{ {
byteLookup( 21 ); byteLookup( 21 );
byteLookup( 26 ); byteLookup( 26 );
} }


USBKeys_Changed |= USBKeyChangeState_TertiaryKeys;
USBKeys_Changed |= USBKeyChangeState_QuartiaryKeys;
} }
// Received 0x00 // Received 0x00
// This is a special USB Code that internally indicates a "break" // This is a special USB Code that internally indicates a "break"
// Invalid key // Invalid key
else else
{ {
warn_msg("USB Code not within 4-164 (0x4-0xA4) or 176-221 (0xB0-0xDD) NKRO Mode: ");
warn_msg("USB Code not within 4-49 (0x4-0x31), 51-155 (0x33-0x9B), 157-164 (0x9D-0xA4), 176-221 (0xB0-0xDD) or 224-231 (0xE0-0xE7) NKRO Mode: ");
printHex( key ); printHex( key );
print( NL ); print( NL );
break; break;

+ 4
- 3
Output/pjrcUSB/output_com.h Zobrazit soubor

USBKeyChangeState_MainKeys = 0x02, USBKeyChangeState_MainKeys = 0x02,
USBKeyChangeState_SecondaryKeys = 0x04, USBKeyChangeState_SecondaryKeys = 0x04,
USBKeyChangeState_TertiaryKeys = 0x08, USBKeyChangeState_TertiaryKeys = 0x08,
USBKeyChangeState_System = 0x10,
USBKeyChangeState_Consumer = 0x20,
USBKeyChangeState_All = 0x3F,
USBKeyChangeState_QuartiaryKeys = 0x10,
USBKeyChangeState_System = 0x20,
USBKeyChangeState_Consumer = 0x40,
USBKeyChangeState_All = 0x7F,
} USBKeyChangeState; } USBKeyChangeState;