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

Support for Altera DE2 board added #38

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Support for Altera DE2 board added #38

wants to merge 13 commits into from

Conversation

tmd-set
Copy link
Contributor

@tmd-set tmd-set commented Mar 5, 2014

We have just created a DE2 port of ORPSoC. It is based on DE1 port, pin mapping and pll was altered. It works now as some simple programs such as HelloWorld can be run, but it may be not be completed and need more testing and updates.

tmd-set and others added 4 commits March 5, 2014 16:22
The generic system has been broken for a long time and only adds
confusion. It has been more or less replaced with or1200-generic
@olofk
Copy link
Member

olofk commented Mar 5, 2014

Thanks for this. Just wondering when you say that it works for simple programs. Have you tried it on a real DE2 board, or only in simulation?

No error while using Icarus but Modelsim complains.

Add missing stdlib.h and ctype.h required for NULL, free() and
isspace().

Signed-off-by: Franck Jullien <franck.jullien@gmail.com>
@tmd-set
Copy link
Contributor Author

tmd-set commented Mar 6, 2014

@olofk
The man who wrote this DE2 port tested it on his DE2-70 Board. I saw it worked with timerled (from orconf2013) and HelloWorld program (https://code.google.com/p/vak-opensource/source/browse/trunk/hardware/s3esk-openrisc/hello-uart/hello.c) . Did it work to you?

@fjullien
Newly generated wb_intercon.v and wb_intercon.vh still doesn't include wb8_xxx signals as in my most recent commit.
tmd-set@52940b7#commitcomment-5581892
tmd-set@52940b7#commitcomment-5581894

@fjullien
Copy link
Member

fjullien commented Mar 6, 2014

Yeah I'm sorry I did a mistake. wb8_xx signals are not created. However, the wb_resize is now included in the generated interconnection.

@tmd-set
Copy link
Contributor Author

tmd-set commented Mar 6, 2014

Anyway, may be I pull this request too soon, there are a lot of problems in my orpsoc-cores repo now as it have some error after updated from the parent repo. May be I should put another one when I finished fix them.

@olofk
Copy link
Member

olofk commented Mar 6, 2014

@tmd-set I haven't got a de2 board myself, but I trust you if you say it works. Great job!
The pull request has become a bit messy however, so when you have fixed the things that @fjullien commented on, it might be best to just do a new pull request

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