Changing names for ram address and data randomization (retry)

tillitis / tillitis-key1

Board designs, FPGA verilog, firmware for TKey, the flexible and open USB security key 🔑

https://www.tillitis.se

382 stars 24 forks source link

Open secworks opened 1 month ago

secworks commented 1 month ago

This PR fixes issue #219 - again. This PR does change the digests, but it is expected according to Yosys.

mchack-work commented 1 week ago

Do we use this or the other branch for going forward?

dehanj commented 1 week ago

Do we use this or the other branch for going forward?

Both PRs have the same digests, so that is good.

This one have updates to the fw and tk1_mem.h as well, which the other does not have. So my suggestion is to use this one.

We might want to squash a bit, i think it is enough with 2 commits. One for the FPGA, and one for FW.