search

va7deo / zerowing

Toaplan V1 system for MiSTer FPGA
GNU General Public License v2.0
46 stars 14 forks source link

Update sys and fixes for MiSTer integration #3

Closed birdybro closed 2 years ago

birdybro commented 2 years ago

Mostly to use as a reference, and was for my own learning, this is not stable.

Proposed changes to more easily integrate into the MiSTer project.

  1. Updated sys (framework) to latest
  2. Renamed core to match typical MiSTer arcade core naming scheme
  3. moved files around to match typical organizational hierarchy of mister cores
  4. Ported to Quartus 17.0.2 to match the template recommendations and to aid in developer collaboration

After doing all this, I saw some timing problems in the timing analyzer, but I was unable to compile your core as is in either 17.0.2 or 20.1, so I wasn't sure if this was expected.

Submitting as a draft for posterity, pending plans by other people, and for review in case I got something wrong.

birdybro commented 2 years ago

Some instability with the core after I made these changes, so not sure what went wrong. Just a proposal, no pressure, and mostly for my own learning...

birdybro commented 2 years ago

Timing changes were observed with the fitter and synthesis changes seen in https://github.com/va7deo/zerowing/pull/3/commits/620bf3669d1986a374fd7c42befae8996d4f8266 .

Before: image

After: image

Any option that says it's saving you Area only should probably be turned off. All settings that say they will improve timing or performance should be set to the performance or timing optimized setting. That's what I think I did with this last change, for reference.

birdybro commented 2 years ago

Closing this now as it's no longer needing to be explored for my learning purposes.