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

Makefile: Correct parameters for the Orange Crab 85F #336

Merged
merged 1 commit into from
Oct 13, 2021

Conversation

paulusmack
Copy link
Collaborator

The Orange Crab v2.1 with 85k LUT FPGA uses a LFE5U-85F device, not
an LFE5UM5G device. Specifying the wrong device causes the FPGA to
give a configuration error and not load the bitstream. Also add flags
to indicate that the device is the highest speed grade (the -8 grade)
and that the bitstream should be compressed, and can be loaded at
38.8MHz.

Signed-off-by: Paul Mackerras [email protected]

@paulusmack
Copy link
Collaborator Author

And... now with setting the OPENOCD_DEVICE_CONFIG variable correctly.

@umarcor
Copy link
Contributor

umarcor commented Oct 3, 2021

I believe this might be motivated because the OrangeCrab that @antonblanchard got from @gregdavill, and which was used to write and test the initial makefiles in this repo, is different from OrangeCrab v2.1. See #294 (comment). Hence, maybe it's desirable to support two OrangeCrab boards/variants.

@gregdavill
Copy link

OrangeCrab that @antonblanchard got from @gregdavill, and which was used to write and test the initial makefiles in this repo, is different from OrangeCrab v2.1

This is correct. There should probably be a way to support for "non-standard" boards. But I'm not sure how that would look.

The existing orange crab target is for an older board with a
LFE5UM5G-85F device.  Newer orange crab boards (v0.21) have a
LFE5U-85F device in the -8 speed grade, so make a new target for them
called ORANGE-CRAB-0.21.

Also add flags to ecppack to indicate that the bitstream should be
compressed and can be loaded at 38.8MHz.

Signed-off-by: Paul Mackerras <[email protected]>
@paulusmack
Copy link
Collaborator Author

I updated the patch to leave the existing ORANGE-CRAB target as is and add a new one called ORANGE-CRAB-0.21. However, now the checks are failing because the ghdl container image has a bug that causes the error:

ERROR: Assert `is_fully_const() && GetSize(chunks_) <= 1' failed in kernel/rtlil.cc:4532.

I believe that has been fixed upstream, so hopefully the container images will get updated shortly.

@umarcor
Copy link
Contributor

umarcor commented Oct 13, 2021

@paulusmack, please, try restarting the CI run.

@mikey
Copy link
Collaborator

mikey commented Oct 13, 2021

@umarcor Thanks, restarting did fix it.

@mikey mikey merged commit 8a03050 into antonblanchard:master Oct 13, 2021
@umarcor
Copy link
Contributor

umarcor commented Oct 13, 2021

FTR, it was not just a restart. I did trigger the image build workflow in hdl/containers before I posted my previous comment. So, Paul was correct.

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.

4 participants