Skip to content
This repository has been archived by the owner on Dec 14, 2023. It is now read-only.

Revert "Delay appending the HID descriptors until .begin()" #63

Merged
merged 1 commit into from
Dec 17, 2019

Conversation

algernon
Copy link

Reverts #62, because it causes issues on the Dygma Raise. With this applied, the keyboard either doesn't register:

[  +0.110901] usb 2-3.4.2: New USB device found, idVendor=1209, idProduct=2201, bcdDevice= 1.00                                                    
[  +0.000006] usb 2-3.4.2: New USB device strings: Mfr=1, Product=2, SerialNumber=3                                                                
[  +0.000003] usb 2-3.4.2: Product: Dygma                                                                                                          
[  +0.000003] usb 2-3.4.2: Manufacturer: Raise                                                                                                     
[  +0.000002] usb 2-3.4.2: SerialNumber: BF00F8E650515157382E314AFF062B0BCkbio01                                                                   
[  +0.012923] cdc_acm 2-3.4.2:1.0: ttyACM0: USB ACM device                                                                                         
[  +0.004397] input: Raise Dygma as /devices/pci0000:00/0000:00:14.0/usb2/2-3/2-3.4/2-3.4.2/2-3.4.2:1.2/0003:1209:2201.0010/input/input37          
[  +0.064089] hid-generic 0003:1209:2201.0010: input,hidraw4: USB HID v1.01 Mouse [Raise Dygma] on usb-0000:00:14.0-3.4.2/input2                   
[  +0.005623] usbhid 2-3.4.2:1.3: can't add hid device: -75                                                                                        
[  +0.000020] usbhid: probe of 2-3.4.2:1.3 failed with error -75

Or, when using the driver/hid branch of Kaleidoscope (keyboardio/Kaleidoscope#771), the keyboard registers, but whenever I press a key:

[  +1.738457] usb 2-3.4.2: input irq status -75 received

If that -75 is -errno, then it's EOVERFLOW: "Value too large for defined data type".

@algernon algernon added the bug label Dec 17, 2019
@algernon algernon merged commit e12c1f4 into master Dec 17, 2019
@algernon algernon deleted the revert-62-late-descriptors branch December 17, 2019 13:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant