merge wren with data out to update only relevant data I/O register?

Question

merge wren with data out to update only relevant data I/O register?

laforest opened this issue 10 years ago · comments

Charles Eric LaForest commented 10 years ago

Currently, doing an I/O write updates all data registers, but only the wren for the updated I/O port changes, indicating where the data must go. This seems messy and maybe wasteful of power.

Now that fixing issue #38 adds a register stage, maybe we should use the wren signal to update only the data register of the same I/O port?

Charles Eric LaForest · Answer 1 · Tue May 17 2016 22:02:14 GMT+0800 (China Standard Time)

Add to this: there is a bug where the internal wren signal is only 1-bit wide, not WRITE_PORT_COUNT bits wide. This was hidden because most tests instantiate only one write port. Will change default to 2 or 4 ports.