Auto increment is broken for system bus accesses of 64 bit.

chipsalliance / Cores-VeeR-EH1

VeeR EH1 core

Apache License 2.0

808 stars 219 forks source link

Auto increment is broken for system bus accesses of 64 bit. #34

Closed joeski65 closed 4 years ago

joeski65 commented 4 years ago

Hi,

If there is an 8-byte access, the sbaddress will not auto increment. A mistake in code means that the address will be incremented by 8 if it's a 16-Byte access (illegal)

dbg.sv:272

({4{(sbcs_reg[19:17] == 3'b100)}} & 4'b1000); should be ({4{(sbcs_reg[19:17] == 3'b011)}} & 4'b1000);

Thanks!

tmw-wdc commented 4 years ago

Thank you for reporting this issue! It will be fixed in the next SweRV EH1 release.

Thanks, Thomas