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.

Sign up for GitHub

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

Jump to bottom

cleanup() needs to manage *all* pins, registered or not #34

Closed
stevieb9 opened this issue Jun 24, 2017 · 2 comments
Closed

cleanup() needs to manage *all* pins, registered or not #34

stevieb9 opened this issue Jun 24, 2017 · 2 comments

Comments

@stevieb9
Copy link
Owner

This breaks I2C entirely.

Add a test file that changes pin mode to the ALT configurations
@stevieb9
Copy link
Owner Author

It looks like this is happening in the cleanup() method.

We need to have a dictionary of default pin settings, then on cleanup, sweep them all and set them back to that default.

stevieb9 added a commit that referenced this issue Jun 24, 2017
@stevieb9
work on #34; updated all tests so they don't change the I2C pin alt mode
2b9ed3a
@stevieb9
Copy link
Owner Author

Need to further investigate on how to best fix cleanup...

@stevieb9 stevieb9 changed the title t/10-register.t sets the i2c pins to IN from ALT0 cleanup() needs to manage *all* pins, registered or not Jun 25, 2017
stevieb9 added a commit that referenced this issue Jul 2, 2017
@stevieb9
work on #34; pin registration is now done with JSON
3808950 
- added _pin_registration() in Util
- added prereq of JSON in Makefile.PL
- $ENV{RPI_PINS} now contains a JSON string
stevieb9 added a commit that referenced this issue Jul 2, 2017
@stevieb9
work on #34; cleaned up t/10-register.t to conform to new registratio…
8058622 
…n standard
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stevieb9