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

Changes to be committed: #76

Open
wants to merge 19 commits into
base: six-buttons
Choose a base branch
from
Open

Conversation

bndlfm
Copy link

@bndlfm bndlfm commented Jun 14, 2022

modified:   common.c
    modified:   common.h

@bndlfm
Copy link
Author

bndlfm commented Jun 15, 2022

I've got a strong feeling of imposters syndrome right now. I haven't really wrote any C since reading my father's copy of KR when I was 11. I mainly just do shell scripting.

Although basically all I did was change some arrays. This merges your work on Six Button support with: the CI stuff, pull request #68 (multiple physical devices attached to the same system), the packaging for distros commit, and adds support for the PC Sensor FS2016BT2A_D [1a86:e02d].

One unusual thing regarding these pedals is that the default udev rules provided don't seem to detect them. However I have little experience with udev rules so that may be my fault. Footswitch is unable to recognize them over bluetooth as well but looking at the code it seems that was intentional.

This is the udev rules I use for the FT2016BT2A_D.

40-footswitch.rules.txt

I've been using this fork of mine for months with no problems aside from the udev snafu.

Thank you for taking the time to look at my pr/comment, I figured it was about time for me to try and contribute back to the community.

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.

6 participants