<div>Joachim and Paul, this letter is mostly for you.</div><div><div>š</div><div>I've attached a modified version of <span title="Branch head">coretest_hashes branch of the test/novena_base repository. Here is a list of what was done.<br /></span></div><div>š</div><div><span title="Branch head">1. EIM arbiter was updated to take advantage of 3 additional address lines, that bunnie routed from the CPU to the FPGA. Now we have 19 address lines instead of 16, that means 19-2=17 effective bits when using 32-bit access.<br /></span></div><div>š</div><div><span title="Branch head">2. In the doc directory there's a draft version of current EIM memory map.</span></div><div>š</div><div><span title="Branch head">3. I've figured out why you guys could not use read and write signals from the arbiter the way they were supposed to be used. I was wrong when I expected Joachim's cores to have registered outputs. They have a combinatorial output in fact. EIM arbiter's minimum latency is 1 cycle, so we have to register data coming out of cores. I've added these three lines to every core wrapper (sha1.v, sha256.v and sha512.v):</span></div><div><span style="font-family:courier new,courier;" title="Branch head">š reg [31 : 0]šš tmp_read_data_reg;<br />š always @(posedge clk) tmp_read_data_reg <= tmp_read_data;<br />š assign read_data = tmp_read_data_reg;</span></div><div>š</div><div>4. Joachim told me, that we are going to have different types of cores (HASH, RNG, CIPHER and so on), so I redesigned EIM multiplexor to have separate modules for every core type. RNG and CIPHER selectors right now are just templates with some dummy registers. Here is what was modified in the HASH multiplexor:</div><div>š</div><div>4a. Core number 0 was added. It is not an actual HASH core, but a set of global (board-level) registers. I've added three registers so far: board type, bitstream version and one writeable dummy general-purpose register.</div><div>š</div><div>4b. Core instantiation was made conditional to allow selecting of what cores to actually implement. We can have a project that offers a large number of cores, so people can disable unnecessary cores to speed up compile time and to save some slices for something else.</div><div>š</div><div>4c. I have disconnected .error() output from cores. As far as I understand it gets asserted when some non-existent register is being addressed. In most projects that I've seen writes to empty regions of memory are discarded and reads return zeroes. If you really need this kind of error checking, please re-connect this output as needed.</div><div>š</div><div>4d. core_selector.v has an instruction on how to add new HASH cores to our design.</div><div>š</div><div>5. TC11() was added to hash_tester.c to check that we can read global board-level registers and that we have access to segments other than HASH. The last check reads dummy registers from RNG and CIPHER segments (which are just templates now), this effectively tests the 3 new added address bits.</div><div>š</div><div>Joachim and Paul, last time we agreed that I don't need write access to the repository right now. Could you please evaluate my changes and upload them if you agree with them. Thank you.</div><div>š</div><div>-- <br />With best regards,</div><div>Pavel Shatov</div></div>