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

pwm breathing with phase offset #3

Closed
nmigen-issue-migration opened this issue Jun 6, 2019 · 2 comments
Closed

pwm breathing with phase offset #3

nmigen-issue-migration opened this issue Jun 6, 2019 · 2 comments

Comments

@nmigen-issue-migration
Copy link

Issue by zignig
Thursday Jun 06, 2019 at 13:47 GMT
Originally opened as m-labs/nmigen-boards#3

First part of some blinky bling.

zignig included the following code: https://github.com/m-labs/nmigen-boards/pull/3/commits
@nmigen-issue-migration
Copy link
Author

Comment by whitequark
Thursday Jun 06, 2019 at 13:51 GMT

Let's not do this. The only purpose of the built-in blinky is to make sure that all parts of the toolchain actually work. It's not supposed to be any kind of nMigen demo or anything, just a litmus check for the entire stack.

I originally wrote it to only blink a single user_led, even, but then it turned out that one of my devboards has a faulty LED, so I made it blink all of them instead.

@nmigen-issue-migration
Copy link
Author

Comment by zignig
Thursday Jun 06, 2019 at 13:55 GMT

Ok, for sure. I was just pleased that I got it working in 2 hours :)

I will keep building blinky cores elsewhere , but yes , flash all for litmus.

rroohhh pushed a commit to rroohhh/nmigen-boards that referenced this issue Aug 4, 2020
@whitequark
hdl.ir: detect elaboratables that are created but not used.
44711b7 
Requres every elaboratable to inherit from Elaboratable, but still
accepts ones that do not, with a warning.

Fixes amaranth-lang#3.
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
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nmigen-issue-migration