Re: [PATCH 08/13] target: Include missing 'cpu.h'

[Richard Henderson]

On 2/9/22 02:22, Philippe Mathieu-Daudé wrote:
> These target-specific files use the target-specific CPU state
> but lack to include "cpu.h"; i.e.:
>
>      ../target/riscv/pmp.h:61:23: error: unknown type name 'CPURISCVState'
>      void pmpcfg_csr_write(CPURISCVState *env, uint32_t reg_index,
>                            ^
>      ../target/nios2/mmu.h:43:18: error: unknown type name 'CPUNios2State'
>      void mmu_flip_um(CPUNios2State *env, unsigned int um);
>                       ^
>      ../target/microblaze/mmu.h:88:19: error: unknown type name 'CPUMBState'; 
> did you mean 'CPUState'?
>      uint32_t mmu_read(CPUMBState *env, bool ea, uint32_t rn);
>                        ^~~~~~~~~~
>                        CPUState
>
> Signed-off-by: Philippe Mathieu-Daudé<f4bug@amsat.org>
> ---
>   target/microblaze/mmu.h | 2 ++
>   target/mips/internal.h  | 1 +
>   target/nios2/mmu.h      | 2 ++
>   target/riscv/pmp.h      | 2 ++
>   4 files changed, 7 insertions(+)
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~