buppu3 / tnCart

FPGA cartridge for MSX
BSD 3-Clause "New" or "Revised" License
26 stars 6 forks source link

Error build the FPGA synth #1

Closed DFL1973 closed 2 months ago

DFL1973 commented 2 months ago

Hello, I'm trying to build the code using Gowin V1.9.9.03 Education (64 bit) build (73833) but I got an error message.

ERROR (EX3652) : An enum variable may only be assigned the same enum typed variable or one of its values("\rtl\src\peripheral\video\video_upscan.sv":296)

I got the same error even on the previous commit.

Another "issue": in the readme you write something like we need to disable all other features in config.sv to use the as V9900, but at the moment all the features are selected as "ENABLE". So, are the code building "as is" or I need to change the config?

Thanks for sharing.

buppu3 commented 2 months ago

ERROR (EX3652)

GoWin V1.9.9.03 is stricter on syntax than previous versions and seems to cause errors when compositing "video_upscan.sv". This issue will be fixed in the next commit.

config.sv

Sorry, the description of the config in the Readme was out of date and did not match the current situation. Older versions could not be built with all features enabled because FPGA resource usage exceeded 100%, but this version has been optimized to reduce the size of each module, so all settings can be built as "ENABLE". Still, more than 90% of the resources are used, so if new features are added to the V9990 emulation in the future, it may not be possible to set all features to "ENABLE".