Request: Upgrade nix scripts & example codes to use nmigen-v0.1rc1 #1

Merged
sb10q merged 3 commits from nmigen-v0.1rc1 into master 2 years ago
harry commented 2 years ago
Collaborator

Major changes as follows:

  • eda/nmigen.nix: now uses version 0.1rc1 of nmigen on PyPi.
  • eda/nmigen-boards.nix: now uses 835c175, which also changes the syntax used by the example nmigen code (see below for details).
  • overlay.nix: now uses version 0.9 of yosys on GitHub, with the right berkeley-abc version.
  • cores/minerva.nix: now uses 0518621, which uses and requires nmigen v0.1rc1 (from PyPi).
  • examples/simplesoc_ecp5.py: now uses syntax from nmigen v0.1rc1:
    • Resource name of the LEDs is now "led" (not "user_led")
    • A Memory object must now be initiated with keyword-only arguments
Major changes as follows: * `eda/nmigen.nix`: now uses version 0.1rc1 of nmigen on [PyPi](https://pypi.org/project/nmigen/0.1rc1/). * `eda/nmigen-boards.nix`: now uses [`835c175`](https://github.com/m-labs/nmigen-boards/commit/835c175d7cf9d143aea2c7dbc0c870ede655cfc2), which also changes the syntax used by the example nmigen code (see below for details). * `overlay.nix`: now uses version 0.9 of yosys on [GitHub](https://github.com/YosysHQ/yosys/tree/yosys-0.9), with the right berkeley-abc [version](https://github.com/berkeley-abc/abc/commit/3709744c60696c5e3f4cc123939921ce8107fe04). * `cores/minerva.nix`: now uses [`0518621`](https://github.com/lambdaconcept/minerva/commit/0518621134dd7a551cac5758d9171a24d0c1578f), which uses and requires nmigen v0.1rc1 (from [PyPi](https://pypi.org/project/nmigen/0.1rc1/)). * `examples/simplesoc_ecp5.py`: now uses syntax from nmigen v0.1rc1: * Resource name of the LEDs is now `"led"` (not `"user_led"`) * A `Memory` object must now be initiated with keyword-only arguments
sb10q closed this pull request 2 years ago
sb10q deleted branch nmigen-v0.1rc1 2 years ago
The pull request has been merged as 9710ebd9d5.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.