Skip to content

Adding arty support for minion.#6

Open
nchronas wants to merge 2 commits intolowRISC:minion-v0.4from
nchronas:arty_2
Open

Adding arty support for minion.#6
nchronas wants to merge 2 commits intolowRISC:minion-v0.4from
nchronas:arty_2

Conversation

@nchronas
Copy link
Collaborator

To Do before merging:

  • Add an example program for the arty (blink LEDs and transmit a message through the UART).
  • Add clk_wiz_1 (sd_clk_wiz) IP for SD clock generation.

@nchronas
Copy link
Collaborator Author

Example program in PR.
clk_wiz_1 IP will not be added in the repo.

Tested in arty.

@nchronas nchronas changed the title WIP: Adding arty support for minion. Adding arty support for minion. Aug 24, 2017
@asb
Copy link
Member

asb commented Aug 25, 2017

Shouldn't top_arty.sv be in pulpino/fpga/rtl? It might be better named arty_top.sv for consistency with nexys_top.sv.

@nchronas
Copy link
Collaborator Author

I named & placed the file, based on the this file.

I think the files found in pulpino/fpga/rtl folder, are taken for the original pulpino core, and are not suitlabe for the minion_soc.

There isn't a problem, rename and or moving it to a different place.
Let me know what you want me to do.

@asb
Copy link
Member

asb commented Aug 25, 2017

Thanks, looks like it's my mistake and that file is best left where it is, unless Jonathan disagrees.

@jrrk
Copy link
Member

jrrk commented Sep 18, 2017

pulpino/fpga/rtl appears to be a zync specific adaptor layer. Along with a lot of other superfluous files it could be removed from our version of the repository. Since I don't have an Arty, I cannot comment further.

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.

3 participants