Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Drop ALT support #457

Closed
wants to merge 3 commits into from
Closed

Conversation

davewongillies
Copy link
Contributor

@davewongillies davewongillies commented May 9, 2023

Its working and I've managed to flash the keyboard but two smallish issues that may or may not matter:

  • The keyboard identifies as Massdrop Inc. ALT Keyboard [VIA] Vial app but not vial.rocks. I can't figure out where the [VIA] bit is coming from.
  • I followed the porting guide and enabled vialrgb mode but using the vial app the lighting tab doesn't show up in the app. It seems to somewhat work on vial.rocks, but color selection is broken with some errors such as Uncaught [object ErrorEvent] and Uncaught RuntimeError: memory access out of bounds. Is there anything else I should be enabling? Or is it possible that this keyboard might not be able to fully support vialrgb?

The regular keycodes for changing RGB lighting works though.

@xyzz
Copy link
Contributor

xyzz commented May 14, 2023

The keyboard identifies as Massdrop Inc. ALT Keyboard [VIA] Vial app but not vial.rocks. I can't figure out where the [VIA] bit is coming from.

That happens because the atsam port doesn't support overriding the serial number which is what the desktop Vial application uses to detect Vial keyboards. I recall this line needs to be commented out for a usable Vial support:

#define USB_DEVICE_SERIAL_USE_BOOTLOADER_SERIAL

@xyzz
Copy link
Contributor

xyzz commented Feb 10, 2024

Closing because there's been no update for over a month.

@xyzz xyzz closed this Feb 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants