Skip to content

Jira 895, Set all pins to INPUT on sketch startup #489

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

Merged
merged 1 commit into from
Mar 21, 2017

Conversation

bigdinotech
Copy link
Contributor

No description provided.

@bigdinotech
Copy link
Contributor Author

@SidLeung @eriknyquist ready for review

@eriknyquist eriknyquist added this to the Elnath milestone Mar 21, 2017
Copy link
Contributor

@eriknyquist eriknyquist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks fine. I haven't tested it though

@SidLeung SidLeung changed the title Set all pins to INPUT on sketch startup Jira 895, Set all pins to INPUT on sketch startup Mar 21, 2017
@SidLeung SidLeung requested review from russmcinnis and noelpaz March 21, 2017 22:29
@SidLeung SidLeung modified the milestones: Deneb, Elnath Mar 21, 2017
@SidLeung
Copy link
Contributor

@russmcinnis @noelpaz , please perform system testing on this PR.

Please note that this PR superseded the following PR,

#483

@ndgbuilder ndgbuilder merged commit 6b33e73 into arduino:master Mar 21, 2017
Copy link
Contributor

@russmcinnis russmcinnis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tested by using a sketch to set all pins as outputs and set the pins high. I then ran a sketch without calling pinMode() and did a digitalRead() of all pins giving them high and low states. The digitalRead() always matched the stimulus. So this looks good to me.

@russmcinnis
Copy link
Contributor

tested again with PR490(471, 477, 480, 484 and 489) passed system test and the pin stimulus test to prove all pins default to input.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants