MiSTer-devel / SMS_MiSTer

Sega Master System for MiSTer
45 stars 43 forks source link

Cannot compile the core #137

Closed jotego closed 1 year ago

jotego commented 1 year ago

It looks like there is something wrong in the core files at the moment (commit 6be6652) as attempting to compile it generates errores related to the PLL:

Error (12006): Node instance "pll_hdmi" instantiates undefined entity "pll_hdmi". Ensure that required library paths are specified correctly, define the specified entity, or change the instantiation. If this entity represents Intel FPGA or third-party IP, generate the synthesis files for the IP. Error (12006): Node instance "pll_cfg" instantiates undefined entity "pll_cfg". Ensure that required library paths are specified correctly, define the specified entity, or change the instantiation. If this entity represents Intel FPGA or third-party IP, generate the synthesis files for the IP. Error (12006): Node instance "pll_audio" instantiates undefined entity "pll_audio". Ensure that required library paths are specified correctly, define the specified entity, or change the instantiation. If this entity represents Intel FPGA or third-party IP, generate the synthesis files for the IP. Error (12006): Node instance "pll" instantiates undefined entity "pll". Ensure that required library paths are specified correctly, define the specified entity, or change the instantiation. If this entity represents Intel FPGA or third-party IP, generate the synthesis files for the IP. Error (12006): Node instance "pll_cfg" instantiates undefined entity "pll_cfg". Ensure that required library paths are specified correctly, define the specified entity, or change the instantiation. If this entity represents Intel FPGA or third-party IP, generate the synthesis files for the IP.

jotego commented 1 year ago

The MiST version is broken too.

birdybro commented 1 year ago

Are you using Quartus 17.0.2 for the MiSTer version and Quartus 13.1 for the MiST version?

Those spaces should have no impact on compilation, it's commented out. I'll investigate...

This is the kind of error that often occurs when using a newer version of Quartus than the core supports (because the altera megafunction wizard would have to be reran to upgrade the IP used in it otherwise). See example here --> https://misterfpga.org/viewtopic.php?t=370

I will look into the MiST version and see if I can fix it, it's probably because I instantiated new ports and their top-level doesn't know how to deal with it, but it would probably be best for @gyurco or @robinsonb5 to take a look to be safe.

EDIT: See --> https://github.com/RobertPeip/PlayStation_MiSTer/issues/2#issuecomment-976308925 - I think you are running Quartus 20 again :)

EDIT2: Confirmed no compilation errors here:

image

jotego commented 1 year ago

Thank you, I tried Q20 indeed. Sorry for making the same mistake twice.

birdybro commented 1 year ago

It's no worries! You have a very complex tool chain with your other project. It's bound to happen.