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

Missing files #2

Closed
jotego opened this issue Nov 21, 2021 · 7 comments
Closed

Missing files #2

jotego opened this issue Nov 21, 2021 · 7 comments

Comments

@jotego
Copy link

jotego commented Nov 21, 2021

It looks like some files are missing for compilation:

imagen

Could you please add them?
Thank you

@RobertPeip
Copy link
Owner

The files are checked in (sys folder).
Are you sure you use Quartus 17?

I know of at least 3 people who could build the core as checked in, so I don't know what the reason is...

@birdybro
Copy link
Contributor

birdybro commented Nov 21, 2021

I was able to build the core just now to test (and multiple commits previously in a row as he's been updating the repo). It took awhile because my computer's slow, but wanted to make sure it works independently of Robert's setup too.
gHdUgjVjT0
Here's the software version and OS I'm using:

  • Quartus Prime Standard Edition 17.0.0
  • Windows 11 Pro x64

Must be something different about your configuration. @jotego do you use Quartus on Linux? If so maybe there is a case-sensitivity issue happening. Maybe you are trying to port it to your forked framework? Since these instances are in the MiSTer Framework here --> https://github.com/RobertPeip/PlayStation_MiSTer/tree/main/sys

@Kitrinx
Copy link

Kitrinx commented Nov 21, 2021

Can confirm, all files are present and builds work fine using Quartus 17.

@thehughhefner
Copy link

I have the same issue as Jotego

@jotego
Copy link
Author

jotego commented Nov 22, 2021

@birdybro I'm using the git repository as it is, but I run Quartus 17 (or 20) from linux. So it would be case sensitive for file names.

@birdybro
Copy link
Contributor

birdybro commented Nov 22, 2021

@jotego Yes it's probably case-sensitivity then without looking at it deeper myself. The errors generated from using version 20 instead of 17 are a little different when I've ran into them, usually just a couple strings in the top-level's QSF that need to be modified since they just changed the name of some of those variables for whatever reason.

EDIT: It's not a case sensitivity issue in this repo's end AFAICT, it might be somewhere else.

image

The error must be generated from pll_hdmi.qip being run through. It's possible your development environment is different than a standard Quartus 17 install. That would be my first guess.

I think you use a script to setup Quartus in your environment, yes? And I know @thehughhefner has contributed to your repos before, so maybe you are both using the same development environment in Linux?

@jotego
Copy link
Author

jotego commented Nov 23, 2021

I can confirm that it works with Quartus 17. I was trying Quartus 20 before. Thanks for your kind replies.

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

No branches or pull requests

5 participants