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

RemoteHardwareModule.cpp: Fix digitalReads() pinModes(mask, INPUT_PULLUP) #2459

Merged
merged 1 commit into from May 4, 2023

Conversation

IhorNehrutsa
Copy link
Contributor

// The Arduino docs show to run pinMode(). But, when testing, found it is best not to.
// If the line below is uncommented, read will flip the pin to the default of the second
// argument in pinModes(), which will make the read turn the PIN "on".
// pinModes(mask, INPUT_PULLUP);

This comment describes an unfortunate misunderstanding caused by incorrect mask calculation from #2454

@github-actions
Copy link
Contributor

github-actions bot commented May 3, 2023

🤖 Pull request artifacts

file commit
pr2459-firmware-2.1.11.4ad3a13.zip 4ad3a13

thebentern added a commit to meshtastic/artifacts that referenced this pull request May 3, 2023
@thebentern thebentern merged commit e360c62 into meshtastic:master May 4, 2023
44 checks passed
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