tillitis / tillitis-key1

Board designs, FPGA verilog, firmware for TKey, the flexible and open USB security key 🔑
https://www.tillitis.se
395 stars 24 forks source link

Try to remove all VERSION fields from cores #175

Closed secworks closed 5 months ago

secworks commented 6 months ago

Each core have NAME0, NAME1 and VERSION strings. The purpose is to allow feature discovery. But the version field is not updated, not documented (what features a given core have at a given version). We could possibly save FPGA resources by removing all VERSION strings from core. However keep a version field at top level. Which we then should update for a given release of the whole design. And document in release notes.

This issue is about trying to remove in-core version fields and see how we can save.

secworks commented 5 months ago

Going through all cores, the only core with VERSION is the tk1 core. That version is connected to the release and it used. This means that we don't have a big opportunity to save resources.