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

Added TinyBX gpio , fix on programmer. #76

Closed
wants to merge 5 commits into from
Closed

Added TinyBX gpio , fix on programmer. #76

wants to merge 5 commits into from

Conversation

zignig
Copy link
Contributor

@zignig zignig commented Jun 2, 2019

No description provided.

@whitequark
Copy link
Contributor

I don't really understand what's happening here. Why did you delete TinyFPGA B?

@zignig
Copy link
Contributor Author

zignig commented Jun 2, 2019

Renamed it to BX to make the naming consistent.

I think my rebase got munted.

@whitequark
Copy link
Contributor

OK, so I've renamed it upstream in commit 37152c7.

Regarding other parts of this PR:

  1. I've added .bin suffix in every programmer in commit e4ebe03.
  2. I don't think there is any need to have a blinky for every single platform. The purpose of the blinky example is to demonstrate how to use the platform API in general, and I would expect anyone who intends to use nMigen to have no trouble adapting it.
  3. The "gpio" resources really should be defined as a "connector" in terms of oMigen. But there is no such feature in nMigen yet, because it needs a redesign and so it is not implemented at all yet. For now, I would say use a local patch, and once connectors are in nMigen, we can upstream that.

@whitequark whitequark closed this Jun 2, 2019
@whitequark
Copy link
Contributor

See #77.

@whitequark
Copy link
Contributor

And I've added GPIOs in 2ca0834.

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