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

Fixed issues with duplicate vendor and product IDs #2

Merged
merged 7 commits into from
Feb 10, 2023
Merged

Fixed issues with duplicate vendor and product IDs #2

merged 7 commits into from
Feb 10, 2023

Conversation

mckset
Copy link
Owner

@mckset mckset commented Feb 9, 2023

The driver now grabs a list of all USB devices that match the vendor ID and product ID of the Mini Keydial and does one of the following depending upon how the driver is ran:

  • As root - The driver looks for the device without a description and selects that instead. For some reason, Huion decided that the Keydial didn't need a description while other products got one
  • As a user - The driver lists all devices that match the vendor ID and product ID along with the bus and device number and displays attached devices using the "lsusb" command. This is only here because getting device descriptions require root permission
  • With the '-a' flag - The driver acts as it did before and grabs the first matching device

New flags:

  • '-a' - Assumes the first matching device is the Mini Keydial
  • '-h' - Displays a help message

Other fixes:

  • The driver should find the device all the time now
  • Added a check to make sure the driver has permission to access the USB before running

A test patch to fix the issue where devices with the same ID as the keydial are selected before the keydial. If the code is ran as root, this process is automatic but if it is ran as a user, you will be prompted to select your device before the driver can run.
Also added the '-a' and '-h' flag
@mckset mckset merged commit 743bbc7 into main Feb 10, 2023
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.

1 participant