pulp-platform / riscv-dbg

RISC-V Debug Support for our PULP RISC-V Cores
Other
197 stars 70 forks source link

[fix] Realign addresses to 64-bit #145

Closed creinwar closed 1 year ago

creinwar commented 1 year ago

The read/write logic in dm_mem.sv only looks at the request address and does not take the byte strobes into account which causes problems when trying to write the Halted, Going, Resuming and Exception locations from a bus that is larger than 32 bit as that would require misaligned requests. This PR addresses this by realigning the relevant addresses to 64 bits, which makes the DM work for 32 and 64-bit harts.

bluewww commented 1 year ago

I'll double check otherwise looks good. Could you also please update the documentation (it describes the debug module addresses).