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

Overlapping pin configurations can generate weird error messages #336

Open
diyelectromusic opened this issue Aug 16, 2022 · 1 comment
Open

Comments

@diyelectromusic
Copy link
Contributor

diyelectromusic commented Aug 16, 2022

We probably need some configuration sanity checking on the range of possibilities for GPIO button configurations now. In particular, if the rotary encoder is configured with duplicate GPIO pins by mistake, there is an assert failure that isn't very easy to track down. It might be quite handy to eliminate (or at least warn about) other potential clashes with "fixed pins" - e.g. with I2C, the LCD, I2S, MIDI (depending on the other settings of course).

I suspect we ought to do some checking on the buttons in general or maybe at least echo the current "in use" configuration to the display to aid debugging configuration issues?

Aside: I was also wondering if it was possible to have two hardware buttons configured for the same job? I was initially thinking of a board that allows either the rotary encoder switch for select, but also a separate hardware button for select too, but having dropped the RE switch configuration, this is no longer possible.

I know this is now somewhat contradictory to my above comment about checking for duplicates! :)

Kevin

@BobanSpasic
Copy link

BobanSpasic commented Aug 16, 2022

I know that this is a promoting of my software, but here you have some sanity check:
screen3

As soon as some pin is assigned, it is removed from the selection list for other pins (buttons are exception)

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

No branches or pull requests

2 participants