Re: [PATCH v2 33/33] target/riscv: Make pmp.h target_ulong agnostic

[Anton Johansson via]

On 01/10/25, Philippe Mathieu-Daudé wrote:
> On 1/10/25 09:33, Anton Johansson wrote:
> > The pmp.h header is exposed through cpu.h.  pmp_table_t is also used in
> > CPUArchState.  CSR declaraions are only used in target/ and are moved to
> 
> Typo "declarations".
> 
> > csr.h.  In pmp.h, addr_reg is widened to 64 bits and the privilege mode
> > parameter is fixed to 8 bits, similar to previous commits.
> 
> Worth a privilege_mode_t typedef?

I think this makes sense, will add in v3!