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

pin: implement driver.Pin interface #215

Closed
wants to merge 2 commits into from
Closed

pin: implement driver.Pin interface #215

wants to merge 2 commits into from

Conversation

deadprogram
Copy link
Member

This PR implements the drivers.Pin interface so that many drivers can remove another dependency on the machine package.

The following drivers however still use machine.Pin each for various reasons:

  • HD44780
  • ILI9341
  • L293x
  • L9110x
  • MCP3008
  • microbitmatrix
  • shifter
  • wifinina
  • ws2812

Each of these will require some resolution or just leave them how they are.

The drivers that use the new pin interface each has a very rudimentary unit test in place to serve as a starting point for more complete testing of that driver.

The new make unit-test task run the units for all drivers except those that have dependencies on machine.

@deadprogram
Copy link
Member Author

Closing in favor of a somewhat different approach once tinygo-org/tinygo#1537 or something else like that exists.

@deadprogram deadprogram deleted the pin-interface branch April 22, 2022 06:44
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.

1 participant